fix: Fix various type definitions & tests

One test is still failing, I'd appreciate if anyone else who knows a bit more about octokit could help. I'm still looking into CI failures, tests are running locally for me.

The same problem exists with the latest Octokit btw.

I did't look into the changes, but from the error message it sounds like it might have introduced a modern JS feature that is not supported in Node 10 / 12. We can release a new breaking version of probot and stop testing in Node 10/12 as they are no longer maintained, but I'd rather not right now.

I did't look into the changes, but from the error message it sounds like it might have introduced a modern JS feature that is not supported in Node 10 / 12. We can release a new breaking version of probot and stop testing in Node 10/12 as they are no longer maintained, but I'd rather not right now.

Yes, it looks like the latest Jest requires Node 14, I'll try to get back to an older version of Jest.


Pod's DNS resolution becomes slow after applying FQDN rules to it

Describe the bug

The issue was reported in the thread: https://kubernetes.slack.com/archives/CR2J23M0X/p1691664157256979
The main symptom was a few pods which after upgrading antrea to version v1.11.2 were observed with a huge increase in DNS requests time. By analysing the DNS traffic of a problematic Pod, I found that the RPS happened to 100. The slowness may be because the userspace rate-limiting (which is 100 packets/s at the moment) dropped some DNS responses. The reason why it became very slow may be because we didn't packetin DNS over TCP before v1.11.0 so the contention was not that bad, and it became worse after all DNS over TCP started occupying the quota.
# tcpdump -r dns.pcap -n src port 53  | awk -F "." '{print $1}' | uniq -c |sort -r | head
reading from file dns.pcap, link-type EN10MB (Ethernet)
    101 15:38:43
    100 15:38:56
    100 15:38:40
    100 15:34:01
     99 15:38:44
     99 15:34:03
     99 15:33:58
     98 15:38:58
     98 15:38:04
     98 15:37:48

[ ] To resolve the issue, we should revisit the rate-limiting configuration as 100 packets/s is far away from enough. @GraysonWu
[ ] Besides, I found the userspace rate-limiting simply dropped the overflowing packets, which makes debugging issues caused by it really hard without any visibility, I think we should have a metric for it and perhaps log it periodically. @tnqn

Versions:

Antrea version (Docker image tag). v1.11.2

Related to https://github.com/antrea-io/antrea/issues/5358
While we should have reasonable defaults for the rate limits, we should also let power users change the value if they want to.


Overlapping widgets in file->preferences menu

The file->preferences menu looks like:

It has some overlapping widgets which are most likely undesired. Introduced by https://github.com/GraphiteEditor/Graphite/commit/de366f951424fcdf4463a419db3fa659910fabfd.

can you assign me this issue ,  we just need to fix the CSS in the dialogue component

Hi @0HyperCube , I have figured out the place , where the CSS have gone wrong , and fixed this in PR #2104. Actually the problem was, that the elements in the dialog was not acquiring whole space available, which is necessary for proper use of width.

Resolved by https://github.com/GraphiteEditor/Graphite/commit/99cf8f0c4f91a051b59fc2c9e5cc6c7417bdd74b.


Disable ci build when deploying/pushing to gh-pages

So we don’t have a useless circle.yml just for the gh-pages branch.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #482   +/-   ##
=======================================
  Coverage   21.44%   21.44%
=======================================
  Files         166      166
  Lines        1231     1231
=======================================
  Hits          264      264
  Misses        967      967

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 821ae6a...c490559. Read the comment docs.


Document classes with @author and @version

Every class should be documented to have a small description and the author

documented


Add further sentiment analysis to ChatMessageEvent

Added some simple analysis on the message, to make it a little more varied than "said", "asked".
Moved the strings into the the relevant json - If anyone can translate to other languages that would be good.

@drusepth @mdiller for review

@howardchung
I'm used to bitbucket sorry, so unsure how to add reviewers after the PR was created.
I have added a few screenshots with example conversations from recent games. Overall I feel this makes the story really fun to read.


ts0601_trv_maxsmart - _TZE200_thbr5z34 - temperature calibration and current temperature updates problem, unnamed switches,

Hi! First of all.... THANK YOU VERY MUCH for your work
I have 5 of AWOW MS-SH4-Z TRVs    _TZE200_thbr5z34   I would have thrown them out to rubbish if not your hard work.
All of them work properly with your quirk in ZHA but...

I am not sure about temperature calibration. When I  move G07 Temperature Calibration slider, Local Temperature offset also changes.
Sometimes, after room is heated it shows odd current temperature even if it was properly calibrated before.

Is there any way to check, how often current temp updates are made or maybe force higher updates frequency? I think, sometimes temperature changed but the device did not update it for long time. Even half an hour.

There are two switches for the device that have no name, one is probably a Child lock but the other does nothing, besides there is a list named Keypad lockout with 5 positions Unlocked/Lock1/Lock2/Lock3/Lock4 Any diffrence between them? All of them just lock keypad on my devices when set.

Is there any way to read and set schedule settings in text bar for those TRVs like this

(work days - holiday) disabled : 6-1 : 5-2 : 7

w.day example
00:00/19.0°C 05:00/21.0°C 15:00/20.0°C 19:00/21.0°C 22:00/21.0°C 23:59/19.0°C
h.day example
06:00/21.0°C 22:00/19.0°C

All issues were checked on five devices, so it is not coincidence.
Is there a chance that you will work on this quirk anytime soon?
Thanks in advance for any answers and explanations

When quirk was build originally that HA field Offset didn't exist. HA manages it with his own power.
and 3. One switch is ChildLock. Second one was built for manual temp update. That's a Zigbee - low power device. If the temp were reported each 5 minutes battery would probably drain fast. That's why I've built for my TRV's that I use manual switch to update temp. Then I've added Automation to trigger that switch every 15 minutes. Internally that sends 0 value to the room temp, which makes the device to respond with current room temp. Small hack.
Skip other lock and just forget about them.
Yes and no. I've added that internally, but HA doesn't have any controls for that.
Wanna play with it - use Zigbee Cluster approach, but it's really easier to go to TRV and set it once for life.


[Feature] Support orc_use_column_names property in session variable

Why I'm doing:
We only support to mapping orc column by position, but we need also support mapping by column name(Support Hive schema evolution).
What I'm doing:
Support it by session variable.
If you want to mapping orc by column names, set orc_use_column_names=true in session variable.
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergify backport branch-3.2 branch-3.1


how to multiprocess training?

I am using cornell university corpus for training my bot.
It takes time more than 2 days. could I split the dataset.txt file into 4 files and merge all result in 1 sqlite file finally?

It sounds like you wrote your own training class for your chat bot. Have you looked into Python's multiprocessing library?
https://docs.python.org/3.6/library/multiprocessing.html

Related #354


[types] Using the thunkAPI argument of effects creates circular type definition

When reading the state or dispatching an action from within an effect via the thunkAPI argument, both the effect inputs and the slice become any because their types depend on each other.
Functionality is unaffected.

Workaround: first save the action into a variable that is explicitly types as any:
Type error
startCountingInterval: async (_unused, thunkAPI) => {
  setInterval(() => thunkAPI.dispatch(counterSlice.actions.countIntervalTicked()), 1000);
},

Workaround
startCountingInterval: async (_unused, thunkAPI) => {
  const actionToDispatch: any = counterSlice.actions.countIntervalTicked();
  setInterval(() => thunkAPI.dispatch(actionToDispatch), 1000);
},


[Bug] my plugins load but aren't working

Describe the Bug
None of the plugins that I have installed and active are working. The notification sounds, owner tag, badges everywhere, ect. All of them are not working. The themes still work though.
To Reproduce
I do not know how it is happening, but I've reinstalled the plugins, Better Discord its self, and reload Discord more times then I can count.
Expected Behavior
I expect the plugins to work and do what they are meant to; it is annoying and kinda weird without them.
Screenshots
A few of my plugins:
https://gyazo.com/f4e73de8e88b11c0aa470adb567fc3b5
https://gyazo.com/7e069bd1d617a03237c833d0aa654013
Proof that the are not work:
https://gyazo.com/f365ddfac520d9a46c686aef16c8499b
https://gyazo.com/77d42fc6528a08161762fa9f3ec620c8
Discord Version
PTB
Additional Context
None of my plugins work at all, and as I said, my themes still do work.

It started to work again today. I will contact you guys again if I have issues!

Sounds like this was an issue with the plugins themselves anyways, better off talking to the plugin devs than blaming BBD in those cases


DAPP Store

DAPP Store screen https://zpl.io/Vq4kNlJ
There should be no "Add Dapp from a file/Uninstall Dapp" functionality on mobile yet.

Checked


Unit 'farenheit' is not supported

in the US and have the USW Insights dashboard loaded and I get the above error on the "Switch Temperature"

v2 is out. check it out!


inaccessible due to 'private' protection level

Recently I updated the cocoa pod and once updated I face the following issue.

What could be the possible reason and how can I clear this issue ?

@bharath-dev Where is that code located? These APIs are marked as private now.
Can you provide your IBAnimatable version & Swift version?

have the same errors:
IBAnimatable 5.1.0

I used it recently and was working as expecting, I'm closing now but let me know if you still have an issue.


Update callback according to SDK 6.9.1 interface.

@felixzhang0703 Can you also update the Readme.md with the information for the callbacks?

@stanley-vungle , updated README.md. Thanks for your comments!


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3984

Questions ?
Please refer to the Backport tool documentation


It is not clear if the update finished successfully or not

This issue impacts 9.0.0-rc.0 and happened while updating https://github.com/johannesjo/super-productivity and https://github.com/SAP/cloud-commerce-spartacus-storefront.
A failed update will show a message saying both the individual migration and the update failed:
× Package install failed, see above.
The update failed. See above.
git HEAD was at 85f08a4d93e684566f751e38e2a2feffe0194cd9 before migrations.
×  Migration failed. See above for further details.

The update below did not finish successfully however, and instead just shows the last migration as successful (@devversion confirms this). In this case the migration shows an error that it seems to suggest was actually recovered from, but it fact the intent is that the migration was interrupted and should later be resumed.
>  Undecorated classes with DI migration.
   As of Angular 9, it is no longer supported to use Angular DI on a class that does not have an Angular decorator.
   Read more about this here: https://v9.angular.io/guide/migration-undecorated-classes

    This migration uses the Angular compiler internally and therefore projects that no longer build successfully after the update cannot run the migration. Please ensure there are
no AOT compilation errors and rerun the migration.. The following project failed: projects/storefrontlib/tsconfig.lib.json

    TypeError: Cannot read property 'module' of undefined

    Could not migrate all undecorated classes that use dependency
    injection. Some project targets could not be analyzed due to
    TypeScript program failures.

    Migration can be rerun with: "ng update @angular/core --from 8.0.0 --to 9.0.0 --migrate-only"

√  Migration succeeded.

A completely successful migration will look the same as the migration above though:
kamik@RED-X1C6 MINGW64 /d/sandbox/latest-app (master)
$ ng update @angular/cli @angular/core --next
The installed Angular CLI version is older than the latest published version.
Installing a temporary version to perform the update.
Installing packages for tooling via yarn.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
Installed packages for tooling via yarn.
Using package manager: 'yarn'
Collecting installed dependencies...
Found 35 dependencies.
Fetching dependency metadata from registry...
    Updating package.json with dependency @angular/cli @ "9.0.0-rc.0" (was "8.3.17")...
    Updating package.json with dependency @angular/core @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/language-service @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/compiler-cli @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/animations @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/platform-browser @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/platform-browser-dynamic @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/common @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/compiler @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/forms @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular/router @ "9.0.0-rc.0" (was "8.2.13")...
    Updating package.json with dependency @angular-devkit/build-angular @ "0.900.0-rc.0" (was "0.803.17")...
UPDATE package.json (1636 bytes)
√ Packages installed successfully.
** Executing migrations of package '@angular/cli' **

>  Update an Angular CLI project to version 9.

** Executing migrations of package '@angular/core' **

>  Static flag migration.
   Removes the `static` flag from dynamic queries.
   As of Angular 9, the "static" flag defaults to false and is no longer required for your view and content queries.
   Read more about this here: https://v9.angular.io/guide/migration-dynamic-flag
√  Migration succeeded.

>  Missing @Injectable migration.
   In Angular 9, enforcement of @Injectable decorators for DI is a bit stricter.
   Read more about this here: https://v9.angular.io/guide/migration-injectable
√  Migration succeeded.

>  ModuleWithProviders migration.
   In Angular 9, the ModuleWithProviders type without a generic has been deprecated.
   This migration adds the generic where it is missing.
   Read more about this here: https://v9.angular.io/guide/migration-module-with-providers
√  Migration succeeded.

>  NGCC postinstall migration.
   Adds an ngcc invocation to npm/yarn's postinstall script.
   Read more about this here: https://v9.angular.io/guide/migration-ngcc
UPDATE package.json (1743 bytes)
√ Packages installed successfully.
√  Migration succeeded.

>  Renderer to Renderer2 migration.
   As of Angular 9, the Renderer class is no longer available.
   Renderer2 should be used instead.
   Read more about this here: https://v9.angular.io/guide/migration-renderer
√  Migration succeeded.

>  Undecorated classes with decorated fields migration.
   As of Angular 9, it is no longer supported to have Angular field decorators on a class that does not have an Angular decorator.
   Read more about this here: https://v9.angular.io/guide/migration-undecorated-classes
√  Migration succeeded.

>  Undecorated classes with DI migration.
   As of Angular 9, it is no longer supported to use Angular DI on a class that does not have an Angular decorator.
   Read more about this here: https://v9.angular.io/guide/migration-undecorated-classes
√  Migration succeeded.

We should confirm the update finished, was successful, or if it was interrupted midway and should later be continued.

This was previously not an issue because the CLI did not print any message stating that the migration was successful or not. Now with V9 of the CLI, it always prints Migration successful. This is not correct since migrations can be simply incomplete or failing gracefully (for various reasons).
e.g. most of the time migrations have edge cases which cannot be handled automatically. In those cases user action is required and it does not make sense to make the user think that the migration was successful/complete.

If a migration can stop midway, it shouldn't print this indication though:
    Could not migrate all undecorated classes that use dependency
    injection. Some project targets could not be analyzed due to
    TypeScript program failures.

    Migration can be rerun with: "ng update @angular/core --from 8.0.0 --to 9.0.0 --migrate-only"

This is incorrect because many packages can be updated at the same time. In this example it is only correct to use --migrate-only with @angular/core if that was the last package to be updated. But the migration does not know this.
The update procedure itself should indicate how to resume from an interrupted migration, and show the full command to resume all un-fully-migrated packages.

I don't quite follow why that would be incorrect. The migration coming from @angular/core just asks the developer to re-run the migrations for @angular/core since the migration from core failed. Previous migrations which were successful as part of @angular/core and will be scheduled twice, will just be a noop.

In this example it is only correct to use --migrate-only with @angular/core if that was the last package to be updated. But the migration does not know this.

I'm slightly confused. Doesn't it print that --migrate-only and @angular/core should be used? Since we print the message from the failing migration, we know that we are in between v7 and version 8. Hence we can propose that command in order to ask for re-running.

Agreed that this is something the CLI could do, but at the time of writing this is not available and we achieved something similar by printing such a message. We did this in version 8 too.

It's incorrect because the original command was ng update @angular/core @angular/cli --next. The migrations for @angular/cli might not have happened yet when the update process was interrupted.
So running ng update @angular/core --from 8.0.0 --to 9.0.0 --migrate-only does not get you to the same place as if the migration had not been interrupted. To do that you'd need instead to do  ng update @angular/core @angular/cli --from 8.0.0 --to 9.0.0 --migrate-only (including CLI as well).

Why would the migrations from @angular/cli be relevant though for that particular migration that failed? It's true that the CLI migrations can change things too, but from the perspective of the @angular/core migration, the only prerequisite is that there is an TS project.
If you are saying that the @angular/core migrations are dependent on migrations of the CLI, then I'm not sure if that is the right thing. I'd expect the the core migrations to be not dependent on migrations outside of the @angular/core scope. In general though.
The way you are describing the issue is that followed migrations will be interrupted. This is not the goal and not the case currently. The goal is:

All followed migrations continue executing.
But only the incomplete migration will be re-run manually (with the given command)

Other migrations as part of @angular/core ran already and will be a noop.

Ah I see. Yes I did think that the last migration interrupted the process and prevented the remaining ones from executing.
So that does sound right. We should somehow indicate that some migrations need extra attention and might require action.

Talked about this in the planning meeting today, decision for now is to simply rename "Migration succeeded." to "Migration completed." to be more clear that we don't necessarily know that it was successful. Sent out a PR to change this.
Longer term, we'll have to look in to a broader redesign which allows some migrations to fail and more clearly communicate them to the user (something like Karma where some tests pass and other fail).

Message change was merged to master and 9.0.x branches. I'll leave this issue open since it seems to be a larger challenge which will require a greater redesign to address. If that is being tracked elsewhere, then we can maybe close this issue.

Yeah this looks as good a place to track it as any.

@alan-agius4 I feel differently. An easy example is that a migration cannot handle all patterns automatically and requests the developer to manually take action. In those case the migration is not successful nor complete. This is actually quite common since migrations most of the time are limited by the possibilities of static analysis. I agree that we could rethink how errors are handled (e.g. not gracefully exiting), but in terms of this issue, it seems to unveil the general issue where the CLI makes an assumption. It's wrong to assume that a migration is complete or successful if it didn't throw. There should be an API to communicate back to the CLI what the status is (for purposes like printing how to rerun the migration). In any case though (except when an error is thrown), the CLI should not make any assumption IMO.

A larger expansion and refactoring is planned and will include a structured migration system.  However, at the current juncture the migrations will need to work within the current confines of the system.  This unfortunately means that the migrations themselves will need to contain more boilerplate and infrastructure than should ideally be needed. But this will improve as update is updated.


[SPARK-23674] Add Spark ML Listener for Tracking ML Pipeline Status

coauthored with @merlintang  mingjie tang<EMAIL_ADDRESS>What changes were proposed in this pull request?
In order to keep track of the status of Spark ML pipeline, trait MLListenEvent to monitor the jira proposed events was added; trait MLListener  used onEvent method to overide doPostEvent in ListenerBus and post the events to specific listener.
In Pipeline.scala, PipelineStage now extends with ListenerBus, so that the related events can be posted to specific listener by doPostEvent. All pipeline related events were posted to all registered listeners by postToAll
In ReadWrite.scala, MLWriter now extends with ListenerBus, so that the save-related events can be posted to specific listener
How was this patch tested?
When testing the features, a recorder was created as a mutable buffer to catch/listen to the actual pipeline events, all the events were added to the recorder by the overridden onEvent method in the newly created MLListener, which was add to the object(pipeline/newPipelineModel/pipelineWritter/pipelineModelWritter) corresponding to the operation(fit/transform/save/save) associated with each type of event, finally the actual captured events were compared with the expected events specified in the tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@jmwdpk can you update this pr, since there is conflict. I have update this pr. https://github.com/merlintang/spark/commits/SPARK-23674

Can one of the admins verify this patch?

@holdenk can you look at this PR? thanks in advance.

Can one of the admins verify this patch?

ok to test

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


[WIP] Adjustments for upstream functional CI

This PR will be used to track all adjustments required for the new CI system and remove code for legacy CI

test

test

test

test

test

test

test

test

test

test

test

test

test

test


Unexpected token

Hi, the components looks great!
Thanks to publish it!
I don't not why but I'm having a problem:
gulp-notify: [Laravel Elixir] Browserify Failed!: Unexpected token

.../node_modules/vue-videobg/src/VideoBackground.vue:1
<template>
^
ParseError: Unexpected token

If I copy the component outside of the package of node_modules folder it's working...

Thanks!


Allow an max height prop for subrows container

Allow a max height prop for sub rows container so that we can show an scroll on all subrows

no idea what this issue is for. TanStack Table does not have components/props for components.


A short description of the bug

Hi,
How to change the mysql port for running quicksart.sh .which all files i need to change that.

Note: iam running the tag version 9.5

hey @resulraveendran - you can customize your quickstart deployment by specifying a docker-compose file to use when issuing the quickstart command (check out the docs here: https://datahubproject.io/docs/quickstart/#customization)
In the docker-compose file you can always change the port from 3306 to any port you want as long as it doesn't conflict with something else in use. Just change all of the instances of 3306 in the docker-compose file the docs tell you to use with whatever port you'd like!
Also i'll use this as an opportunity to suggest directing future questions like this towards our slack community as it's a great place to ask questions and have discussions around all thing datahub! http://slack.datahubproject.io/
Let me know if this resolves your problem and then I'll close this issue

@resulraveendran - going to close this issue for now, but feel free to ask more questions in our slack community or re-open this issue if you're still having problems.


[BUG] eventbox onclick Script Succeeding/Failing Seemingly At Random

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
I have a liitle crypto widget which calls a script to update the logo and price variables, thereby changing the current crypto in the widget:

The script works fine if called directly from the command line (even if called repeatedly), but fails maybe 50% of the time when called through clicking the eventbox. I should mention that I have ruled out the API rate-limit being the issue.
Here is the .yuck for the widget:
;(defpoll bitcoin :interval "60s" "runhaskell $HOME/.dotfiles/home-manager/modules/eww/config/scripts/crypto/GetCryptoPrice.hs 0")
(defpoll bitcoin :interval "60s" "runhaskell $HOME/.config/eww/scripts/crypto/GetCryptoPrice.hs 0")
;(defpoll cardano :interval "60s" "runhaskell $HOME/.dotfiles/home-manager/modules/eww/config/scripts/crypto/GetCryptoPrice.hs 1")
(defpoll cardano :interval "60s" "runhaskell $HOME/.config/eww/scripts/crypto/GetCryptoPrice.hs 1")
;(defpoll ethereum :interval "60s" "runhaskell $HOME/.dotfiles/home-manager/modules/eww/config/scripts/crypto/GetCryptoPrice.hs 2")
(defpoll ethereum :interval "60s" "runhaskell $HOME/.config/eww/scripts/crypto/GetCryptoPrice.hs 2")

; The inner box blow is there below so that the price fetching scripts work,
; if I declare them in the literal they won't fire.

(defwidget crypto []
  (eventbox
    :class "crypto"
    :onclick "runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs ${logo}"
    :timeout "10s"
    (box
      :class "cryptoInnerBox"
      :valign "center"
      :space-evenly false
      :spacing 7
      (label :visible false :text "${bitcoin}")
      (label :visible false :text "${ethereum}")
      (image
        :image-height 25
        :image-width 25
        :path "${logo}"
      )
      (literal :valign "center" :content price)
    )
  )
)

(defvar logo "/home/isaac/.config/eww/images/cardano.png")
(defvar price "(label :text \"$\{cardano\}\")")

And here is the .hs for the  script:
import Data.List (find)
import Network.HTTP.Client qualified as HTTP
import Network.HTTP.Simple (Request, parseRequest)
import Network.HostName (getHostName)
import Safe (headMay)
import System.Environment (getArgs)
import System.Process (callCommand)

main :: IO ()
main = do
  args <- getArgs
  callCommand $ case headMay args of
    Just arg ->
      "eww update logo="
        ++ logoPath'
        ++ " && eww update price=\"(label :text \\\"\\${"
        ++ name'
        ++ "}\\\")\""
      where
        logoPath' = maybe "" logoPath (cycleCrypto cryptos arg)
        name' = maybe "" name (cycleCrypto cryptos arg)
    Nothing -> ""

cycleCrypto :: [Crypto] -> String -> Maybe Crypto
cycleCrypto cryptos str =
  case find ((== str) . logoPath) cryptos of
    Just x -> case dropWhile (/= x) (cycle cryptos) of
      x' : y : _ -> Just y
    Nothing -> Nothing

-- Datatypes:
data Crypto = Crypto
  { api :: Request,
    logoPath :: String,
    name :: String
  }
  deriving (Show, Eq)

instance Eq HTTP.Request where
  (==) :: Request -> Request -> Bool
  req1 == req2 =
    (HTTP.host req1, HTTP.path req1, HTTP.queryString req1)
      == (HTTP.host req2, HTTP.path req2, HTTP.queryString req2)

-- Functions:
makeCrypto :: String -> Crypto
makeCrypto str =
  Crypto
    { api = req1,
      logoPath = logoPath',
      name = str
    }
  where
    Right req1 = parseRequest ("https://api.coingecko.com/api/v3/simple/price?ids=" ++ str ++ "&vs_currencies=cad")
    logoPath' = "/home/isaac/.config/eww/images/" ++ str ++ ".png"

-- Data:
cryptos :: [Crypto]
cryptos = map makeCrypto ["bitcoin", "cardano", "ethereum"]

The logs show this in the ~50% of cases where clicking the event box does not change the crypto (the rest of the time it works with no log output):
 2023-04-25T17:11:42.815Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/cardano.png timed out
 2023-04-25T17:11:43.182Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/ethereum.png timed out
 2023-04-25T17:11:43.182Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/ethereum.png timed out
 2023-04-25T17:11:43.203Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/ethereum.png timed out
 2023-04-25T17:11:43.334Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/ethereum.png timed out
 2023-04-25T17:11:43.334Z ERROR eww::widgets              > WARNING: command runhaskell $HOME/.config/eww/scripts/crypto/CycleCrypto.hs /home/isaac/.config/eww/images/ethereum.png timed out

Reproducing the issue
You could spin up my config locally to reproduce the issue: https://gitlab.com/IsaacBrown92/dotfiles/-/tree/main/home-manager/modules/eww/config
Expected behaviour
Crypto widget should flip seamlessly between cryptos on click (unless API rate limit is hit).
Additional context
No response

Probably related to: https://github.com/elkowar/eww/issues/715

I see. Hopefully at some point this is addressed. I've been scripting with Haskell over the last while and I've basically learned that anywhere a Haskell script is called, it will fail some % of the time with a timeout warning in the logs. It's not fixable with the :timeout property, and it doesn't require spamming whatever the thing is that launches the script (button, input, etc) to make the error happen. It just seems to be a statistical inevitability. It's a bit frustrating because no matter how good I get things looking, they won't behave reliably if they for ex need user input (which will require running a script). It doesn't matter in cases where the script is run via polling (maybe it's also failing some amount of the time when run like this, but since the polling happens regularly enough it's not noticed). So really the issue becomes apparent with scripts that need to work everytime (like workspace switching script), not so much with polling scripts.

For now, I'd suggest appending a & to your command like this: your/script.sh &. This will run the command in the background, which effectively nullifies eww's timeout limit.


[Extensibility Request] Method 'GetPaymentInstructionsFromPostedInvoice' has scope 'OnPrem' and cannot be used for 'Extension' development

When you clone report 1306, there's an error in its code:
The application object or method 'GetPaymentInstructionsFromPostedInvoice' has scope 'OnPrem' and cannot be used for 'Extension' development. For more information, see: https://docs.microsoft.com/dynamics365/business-central/dev-itpro/developer/devenv-compilation-scope-overview.ALAL0296
It could be interesting if standard reports, especially document reports don't use any OnPrem scoped procedures/objects, since those reports are cloned a lot, or at least provide a non-onprem overload?
Or does Microsoft wants us to start using scope OnPrem too? ;-)

I guess the main reason, why this procedure is OnPrem, is that this procedure shouldn't be used anymore since it is using an obsoletete field "Payment Instructions" of the SalesHeader.
The use of this procedure is probably mainly legacy code, that should be cleaned up.
#if not CLEAN19
    [Scope('OnPrem')]
    procedure GetPaymentInstructionsFromPostedInvoice(SalesInvoiceHeader: Record "Sales Invoice Header"): Text
    var
        TypeHelper: Codeunit "Type Helper";
        InStream: InStream;
    begin
        if not SalesInvoiceHeader."Payment Instructions".HasValue then
            exit('');

        SalesInvoiceHeader."Payment Instructions".CreateInStream(InStream, TEXTENCODING::Windows);
        exit(TypeHelper.ReadAsTextWithSeparator(InStream, TypeHelper.LFSeparator));
    end;
#endif

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Dependencies] [Security] Upgrade to Gulp4

This fixes multiple bugs (#6549 #5762 #6214 ) and implements #3793 by upgrading from Gulp 3.9.1 to 4.0.0 which was released a few months ago.
The primary goal of this is to incorporate a series of vulnerability fixes (minimatch, hoek, and lodash) that were introduced within the dependencies of gulp, and as a result were patched starting in Gulp v4.
The basic upgrade flow works like this:

Upgrade Gulp from 3.9.1 to 4.0.0
Upgrade gulp-help to the version that specifically supports Gulp4 (1.6.1 to chmontgomery/gulp-help#gulp4)
Upgrade additional dependencies that fix additional audit issues (gulp-concat-css, gulp-copy, gulp-less, gulp-watch, and github)
Remove run-sequence which is now deprecated and natively replaced with gulp.series
Fix calc() errors that arose out of interpolated strings with the latest version of gulp-less

With those tasks complete, npm install works and completes as intended with no errors and 0 security vulnerabilities.

This is great this will help a bunch!

May be worth waiting until this is merged

I wasnt able to get the tasks to run with this PR. Is this a work in progress?

Alright gulp-copy 4 is merged I'll take another look now to make sure the tasks run

@jlukic Okay this is now complete. You can run npm install and gulp and all tasks will successfully complete with no errors and no security vulnerabilities.

@acconrad Great, I will take a look at this when I have some time!

@acconrad I just did some testing and it all looks good. The only problem I can find is when running gulp build-docs it will crash with TypeError [ERR_INVALID_CALLBACK]: Callback must be a function. It seems line 83 in tasks/docs/build.js is the problem because in newer versions of node the fs.writeFile function throws an error when there is no callback. If I remember correctly this was introduced in node v9.

Cool @hammy2899 fixed!

@hammy2899 added a commit that should get your PR to work now :)

All of the calc variables need to pull from their respective variables (instead of being hard-coded) otherwise they wont adjust properly with user themes.

Maybe related to
https://github.com/less/less.js/issues/3191
https://github.com/less/less.js/pull/3223

Good find @jlukic I had trouble finding that resolution, all fixed now!

I've merged #6512 which updates calcs for compatibility with less 3.5.
Can you take a look at that and see if it can fit into your PR? A simple way to confirm is to inspect an attached menu and verify its calc(100% + (-1px * 2))

@jlukic okay merged in. i verified everything looked right and the build still passes when building from npm and gulp.

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

@jlukic @hammy2899 bump any chance we can get this merged in while it's fully passing and there are no conflicts? would be great to get security issues resolved where I'm using this

bump @jlukic @hammy2899 can we get this merged while there are no conflicts and everything is working properly? would love to remove all of these security vulnerabilities

bump @jlukic @hammy2899 can we get this merged while there are no conflicts and everything is working properly? would love to remove all of these security vulnerabilities

bump @jlukic @hammy2899 can we get this merged while there are no conflicts and everything is working properly? would love to remove all of these security vulnerabilities

sorry about all of the bumps, this was during Github's outage this weekend and it kept telling me that no comments were going through (I even tried deleting all of them and they keep coming back)

I was on vacation with family in Ireland, will probably have a chance to look this weekend or next.

This ready to merge?

This ready to merge?

@acconrad @jlukic ./tasks/config/npm/gulpfile.js need to be updated to match the new gulpfile.js.

@acconrad Have you noticed file.path no longer exists in watch.js?
From watch.js:
gulp
    .watch([
      source.config,
      source.definitions   + '/**/*.less',
      source.site          + '/**/*.{overrides,variables}',
      source.themes        + '/**/*.{overrides,variables}'
    ], function(file) {

    ...

    gulp.src(file.path)
              ^
    ...

For me, gulp watch outputs an undefined exception after editing .less or an override.
If it indeed does, a solution might look something like the following:
  const cssWatcher = gulp.watch([
    source.config,
    source.definitions   + '/**/*.less',
    source.site          + '/**/*.{overrides,variables}',
    source.themes        + '/**/*.{overrides,variables}'
  ]);

  cssWatcher.on('change', (path) => {

    ...

    gulp.src(path)

    ...

For the CSS watch and the remaining watches also.

@pgobin it wasn't there before I started this PR and this PR doesn't touch file.path so I would recommend you fix this in a separate PR

Are we ready to merge this? Still would love to get this one in for my team!

Is there anything that stops this PR from being merged? Any help needed?

@maxhq Give https://github.com/fomantic/Fomantic-UI a try it has gulp4 support and is activity maintained

ping

@acconrad This will most likely never be merged, its over 1 year now since last release of SUI (https://github.com/Semantic-Org/Semantic-UI/releases)
I would recommend anyone looking at this to take a look at Fomantic-UI the official community fork which has gulp4 support.

Upgraded yesterday to Formantic, supersmooth, no issues


Adjust CSS to Center Black Moon Icon Inside Circular Toggle Button

The toggle button lacks a visual indicator of its function, making it unclear to users.

https://github.com/pradyumna14/To-Do-List-Application/pull/1#issue-1945822745

I want to do this


[proof system] Reducing use of the term plonk

The term plonk is overloaded and confusing in our codebase; this PR reduces use of that term to disambiguate meaning.
Please review based on whether the suggested names make sense; we can worry about the implications in terms of breaking changes once we're in alignment about whether the names are good names.
Resolves #922

Added this note to the PR description:
Please review based on whether the suggested names make sense; we can worry about the implications in terms of breaking changes once we're in alignment about whether the names are good names.

Added this note to the PR description:
Please review based on whether the suggested names make sense; we can worry about the implications in terms of breaking changes once we're in alignment about whether the names are good names.

I'm going to drop my request for changes, because I like these names with no objections beyond what's in already-seen comments, and CI will prevent us from accidentally landing this without making the corresponding circuit changes.

OK I've updated the PR to avoid making any changes from ram to memory.
@jbruestle how do the terms here feel to you?

Discussed this with proof system team at October 16th sync -- closing this for now. We can revisit this again in the future as appropriate, perhaps in conjunction with audits and/or implementing LogUp


Application crashes when user navigates to "Phone number" page while creating & restoring account

Frequency: 100%
Repro on build version: Testflight Build V1.23.2
Repro on devices: iPhone 13 mini (15.1.1), iPhone 12 (14.7.1)
Does Not Repro on build version: Testflight Build V1.23.1, Android Internal Build V1.23.2
Does Not Repro on devices: iPhone 13 mini (15.1.1), iPhone 12 (14.7.1), Samsung Galaxy A8+(8.0), One plus 7T(11.0)
Pre-condition:

User must have installed the application
User must be on Create Account Step 1 of 3 screen & added photo & name into respective fields.

Repro Steps:

Tap on "Next" button after adding photo and name.
Enter a valid PIN on Create PIN step 2of 3 screen.
Enter same valid PIIN again to confirm PIN & observe.

Current Behavior: Application crashes when user navigates to phone number page while creating & restoring account
Expected Behavior: Application must not crash when user navigates to phone number page while creating & restoring account & able to complete both flows.
Investigation: This issue is not reproducible on Android Platform
Impact: User is blocked to create or restore account.
Attachment:
App crashes.mp4
Celo Crash Logs

Hey @MuckT, We have verified this issue on Test flight build V1.23.2 and observed that application is crashing when user opens the valora application.
Device: iPhone Xs Max(13.5.1)
Attachment: App crash.mp4
Thanks..!!

Hey @MuckT, We have verified this issue using latest Test flight Build V1.23.3 & observed that issue is fixed now. The app is not crashing on app load, also application is not crashing on "Phone number" page and user is able to complete the create & restore account flow.
Devices: iPhone Xs Max(13.5.1), iPhone 13 mini (15.1.1), iPhone 12 (14.7.1)
Attachment:
App not cashing on phone number screen.mp4
App not crashing on app load.mp4
Thanks..!!


Compile on save in Visual Studio 2013 does not emit reference tags in declaration output when saving

I'm using Visual Studio 2013 Update 3 with TypeScript 1.1 on Windows 8.1.
I have a solution with multiple projects that use declaration files to reference each other. A core project emits a declaration file, and a single JavaScript output. Each dependent project copies these into their own hierarchy. The core definition file references a number of other definition files.
When I do a rebuild, the output definition file contains the references at the top of the file. This is correct behaviour, in my case. However, when I save any of the files in the project, the declaration file is emitted again, but with no references at the top. Comments are still preserved in the declarations (and obey the setting for comment preservation), but references are gone. This breaks my entire project.
Replication is simple:

Create a new project in VS 2013 from the "HTML Application with TypeScript" template.
Using the Project Properties screen, ensure that "Compile on save" and "Keep comments in JavaScript output" are enabled, and define an output file.
Create a test.d.ts file with something valid in it.
Reference test.d.ts from your generated app.ts.
Make a change in app.ts.
Do a rebuild of the project.
Examine app.d.ts. It will contain the reference to test.d.ts
Save app.ts (even without modifying it).
Examine app.d.ts. The references are gone.
Do a rebuild and examine app.d.ts. The references are back.

A workaround for this is to disable Compile on save in projects that generate definition files that require reference emission.

I can not reproduce this any more. please reopen if still an issue.


[Snyk] Upgrade fast-glob from 3.3.0 to 3.3.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade fast-glob from 3.3.0 to 3.3.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2023-07-22.

Release notes

Package name: fast-glob

3.3.1 - 2023-07-22
Full Changelog: 3.3.0...3.3.1

This release fixes a regression for cases where the ignore option is used with a string (#403, #404).
The public interface of this package does not support a string as the value for the ignore option since 2018 year (release).
So, in the next major release, we will reintroduce method implementations that do not involve strings in the ignore option.

3.3.0 - 2023-06-30
Full Changelog: 3.2.12...3.3.0

🚀 Improvements
Method aliases
New methods (glob, globSync, globStream) have been added in addition to the current methods (default import, sync, stream), which eliminate the need to rename the method when importing. In addition, an async alias has been added for the default import, which makes it possible to use this packet with ESM.
Method to convert paths to globs
A new method (convertPathToPattern) has been added in this release to convert a path to a pattern. The primary goal is to enable users to avoid processing Windows paths in each location where this package is used by utilities from third-party packages.
See more details in the pull request.
🐛 Bug fixes

In the past, we mishandled patterns that contained slashes when the baseNameMatch option was enabled, which went against the documented behavior. (#312)
Several problems with matching patterns that contain brace expansion have been resolved.  The primary issue solved is when the pattern has duplicate slashes after it is expanded (#394), or the micromatch package does not correctly generate a regular expression (#365).
All negative patterns will now have the dot option enabled when matching paths. Previously, the !**/* patterns did not exclude hidden files (start with a dot). (#343)
The issue that led to duplicates in the results when overlapping or duplicate patterns were present among the patterns has been fixed. At the moment, we are only talking about leading dot. Other cases are not included. For example, running with the patterns ['./file.md', 'file.md', '*'] will now only include file.md once in the results. (#190)

📖 Documentation
A clarifying note has been added for the concurrency option, which provides more detailed information about the Thread Pool utilization.
⚙ Infrastructure

The benchmark in CI is now running on Node.js 20.
The benchmark now uses the public package bencho instead of an in-house implementation. You may want to try this solution for your packages and provide feedback.

🥇 New Contributors

@ josh-hemphill made their first contribution in #383
@ mairaw made their first contribution in #401

    from fast-glob GitHub release notes

Commit messages

Package name: fast-glob

4ba3fcd 3.3.1
e4ee035 Merge pull request #407 from ivanhofer/patch-1
7303d0f fix typo
697caf7 Merge pull request #406 from mrmlnc/ISSUE-404_fix_patterns.reduce
2a0c808 fix: cast the ignore field to the array
c089430 build: fix build benchmark files
9a1361f docs: remove funding
bbc9c28 docs: update benchmark section

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Design Review] Sign & Submit

Close #449
Changes

Content updates

Screenshots - Updated May 16, 12:30 ET

@natalia-fitzgerald All updated.
Follow-up ticket for Date/time here: https://github.com/cfpb/sbl-frontend/issues/531


Fix grpc protobuf encoder/decoder

compatibility fix for older protobuf generated with github.com/golang/protobuf in grpc plugins
Resolves #3239

It seems that the CI check didn't like it.
Would recompilation of your plugin fix the issue?

github.com/golang/protobuf/proto is deprecated, we're not going back to it.

Is it possible to disable linter for this specific place?


release: can we release a first alpha?

I just saw that there is no alpha released of invenio-deposit. Would it be possible to release one so that we can install B2Share with pip install without using the devel requirements?
I can make the PR if it is ok.
Thanks.

Done

Thanks again @lnielsen. However it seems that pypi didn't get updated. https://pypi.python.org/pypi/invenio-deposit/1.0.0a1 => 404

Ups, fixed.


Node port number has a constructor in the number field during LiveView

Summary
Node port number has the constructor "Just" in the number field using LiveView
Steps to reproduce
Steps to reproduce the behavior:

Perform pioneer exercise 3
Install Cardano Node 1.12.0
Launch Node in LiveView

Expected behavior
Should only contain the port number
System info (please complete the following information):

OS: Ubuntu
Version: 20.04
Node version: 1.12.0

Screenshots and attachments

I see this same issue building for and running on mainnet.

The question here is what we want, and is it important. The port number is just one bit of info, and currently it will be wrong for some configurations.
The general info is the addresses (yes, plural) that the node is listening on for remote connections. So that's a list of IP addresses on local interfaces, with corresponding port numbers. Some of those will be IPv6 so the addresses can be huge.
And currently this info will be missing/wrong if the node is configured to use systemd socket activation. The live view currently takes its port number from the CLI. But this is wrong. It misses out if it get specified in the config file and it misses the case of systemd socket activation, where we get passed pre-created sockets rather than creating them ourselves.
So what do we actually want?

I believe this was addressed by https://github.com/input-output-hk/cardano-node/pull/1076


Uploading a second stash is only possible after a page refresh

If you upload your current stash, leave the page open, play some more and try to upload the new version nothing happens. You have to reload the page manually and upload the latest version afterwards to actually see the changes.
tested in the latest chrome (92)

Fixed. :)


Disk Choice problem

When it asks you to choose your disk for installation, there is only a number next to the top choice, but the other disks dont have a number next to it, so cant be selected

hey there, I cannot reproduce this bug or issue kindly give me log or snapshot of the issue to track it down.

It also happens here!

yep, thats the same error that i was getting.

it was not a array. so converted it to be an array. which will solve this issue. thank you guys


Fix wrong example on how to set the drawing unix runtime switch.

Summary
The runtime switch should not go under a runtimeOptions node.
Related to: https://github.com/dotnet/runtime/issues/63027

This still does not work. I have following runtimeconfig.json
{
"runtimeOptions": {
"tfm": "net6.0",
"includedFrameworks": [
{
"name": "Microsoft.NETCore.App",
"version": "6.0.0"
},
{
"name": "Microsoft.AspNetCore.App",
"version": "6.0.0"
}
]
},
"configProperties": {
"System.GC.Server": true,
"System.Reflection.Metadata.MetadataUpdater.IsSupported": false,
"System.Reflection.NullabilityInfoContext.IsSupported": true,
"System.Runtime.Serialization.EnableUnsafeBinaryFormatterSerialization": false,
"System.Drawing.EnableUnixSupport": true
}
}
and it still throws PlatformNotSupportedException
Microsoft.Reporting.NETCore.LocalProcessingException: An error occurred during local report processing.
---> System.TypeInitializationException: The type initializer for 'Microsoft.ReportingServices.Rendering.ImageRenderer.Renderer' threw an exception.
---> System.PlatformNotSupportedException: System.Drawing.Common is not supported on this platform.
at System.Drawing.Image..ctor()
at System.Drawing.Bitmap..ctor(Int32 width, Int32 height)
at Microsoft.ReportingServices.Rendering.ImageRenderer.Renderer..cctor()

@patelhitul The switch was removed in .NET 7, however, I see you are targeting .NET 6, so I'm not sure. I wonder if the setting is being overwritten somehow. @tarekgh Can you help?

I have created a small console app on Ubuntu v22 which is targeting net6.0. I have created runtimeconfig.template.json file with the following contents:
{
      "configProperties": {
         "System.Drawing.EnableUnixSupport": true
      }
}

after compiling it, it produced the file bin/Debug/net6.0/SystemDrawingApp.runtimeconfig.json with the following contents:
{
  "runtimeOptions": {
    "tfm": "net6.0",
    "framework": {
      "name": "Microsoft.NETCore.App",
      "version": "6.0.0"
    },
    "configProperties": {
      "System.Drawing.EnableUnixSupport": true
    }
  }
}

I have the following code in the app
using System.Drawing;

Bitmap bitmap = new Bitmap(10, 10);
Console.WriteLine($"{bitmap.Width} ... {bitmap. Height}");

and that ran fine without throwing any exception. If I remove the switch, the app will start throwing PlatformNotSupportedException as expected. I am running with .NET 6 version 6.0.401.
Conclusion, it looks to me it is correct to have configProperties section under the runtimeOptions and this issue is not correct.
@patelhitul if you still run into the problem, could you please provide a small app demo the problem? just ensure configProperties section under the runtimeOptions  in the runtime config file.


styledMapType doesn´t work

Hi, there seems to be a problem with the styledMapType which does not use the values i set in the config area. I would like to get the theme as shown in the wiki. Thanks

Im also having the same issue. Cant figure out how to get it like the example.  I already found the comma error on my own. Did you find out what I need in config?


[SPARK-19763][SQL]qualified external datasource table location stored in catalog

What changes were proposed in this pull request?
If we create a external datasource table with a non-qualified location , we should qualified it to store in catalog.
CREATE TABLE t(a string)
USING parquet
LOCATION '/path/xx'

CREATE TABLE t1(a string, b string)
USING parquet
PARTITIONED BY(b)
LOCATION '/path/xx'

when we get the table from catalog, the location should be qualified, e.g.'file:/path/xxx'
How was this patch tested?
unit test added

Test build #73556 has started for PR 17095 at commit 570ce24.

Test build #73556 has finished for PR 17095 at commit 570ce24.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73556/
Test FAILed.

Merged build finished. Test FAILed.

Test build #73565 has started for PR 17095 at commit 55c525e.

Test build #73565 has finished for PR 17095 at commit 55c525e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73565/
Test FAILed.

Merged build finished. Test FAILed.

Test build #73582 has started for PR 17095 at commit 18ec570.

Test build #73582 has finished for PR 17095 at commit 18ec570.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73582/
Test PASSed.

Test build #73591 has started for PR 17095 at commit 22bef8b.

Test build #73591 has finished for PR 17095 at commit 22bef8b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73591/
Test PASSed.

cc @cloud-fan

retest this please

Test build #73760 has started for PR 17095 at commit 7e08045.

Test build #73760 has finished for PR 17095 at commit 7e08045.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73760/
Test FAILed.

Test build #73773 has started for PR 17095 at commit 9932b03.

Test build #73773 has finished for PR 17095 at commit 9932b03.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73773/
Test PASSed.

Test build #74258 has started for PR 17095 at commit 0919fea.

Test build #74259 has started for PR 17095 at commit c7e837c.

thanks, merging to master!

Test build #74259 has finished for PR 17095 at commit c7e837c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74259/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #74258 has finished for PR 17095 at commit 0919fea.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 public static class LongWrapper
 public static class IntWrapper
case class ResolveInlineTables(conf: CatalystConf) extends Rule[LogicalPlan]
case class CostBasedJoinReorder(conf: CatalystConf) extends Rule[LogicalPlan] with PredicateHelper
  case class JoinPlan(itemIds: Set[Int], plan: LogicalPlan, joinConds: Set[Expression], cost: Cost)
case class Cost(rows: BigInt, size: BigInt)
abstract class RepartitionOperation extends UnaryNode
case class FlatMapGroupsWithState(
class CSVOptions(
class UnivocityParser(
trait WatermarkSupport extends UnaryExecNode
case class FlatMapGroupsWithStateExec(

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74258/
Test PASSed.

Merged build finished. Test PASSed.

I suspect that this PR is the cause of consistent failures in the maven build, in the HiveCatalogedDDLSuite unit test: https://spark-tests.appspot.com/test-details?suite_name=org.apache.spark.sql.hive.execution.HiveCatalogedDDLSuite&test_name=create+temporary+view+using
Based on the error message: https://spark-tests.appspot.com/test-logs/408097945 it looks like the way the path is getting re-written (I think by the code in this PR) is causing Hadoop's path code to barf.  The create temporary view using unit test is the only one in that suite that reads from a CSV file, which would explain why that's the only one that's failing.  @windpiger or @cloud-fan would one of you mind looking into this?

Sounds like this was caused by a different PR (see the comment on the JIRA) and is now being fixed, so never mind here!


No Eq

Any thoughts on not depending on the Eq for a faster lookup and smaller footprint when collisions are not a major issue?

Do you have a realistic application where this behavior would actually be acceptable?

I have a tokenized search that has the suggestions paired with a hashable token (some subset of the suggestion's text). In the search if a clash occurs the user will not choose the incorrect suggestions.
If the tokens could be garbage collected and just the hashes remained there would be considerable savings with the amount of memory used.

So what you're really looking for is an IntMap or WordMap implemented as an array-mapped trie?

A very intriguing suggestion. I'll look into array-mapped tries.
Thank you


Locust stopped working after Flast 2.0 got released

in setup.py I can see:
"flask>=1.1.2",
I guess it should be hardcoded to ==1.1.2 for now.
it crashes with:
File "/root/.local/share/virtualenvs/xxxxxxx/lib/python3.6/site-packages/locust/web.py", line 102, in __init__
app.jinja_options["extensions"].append("jinja2.ext.do")
KeyError: 'extensions'

I also faced the same error, I tried to google for a solution.
Based on what chfaber shared in this StackOverflow post , the workaround would be to replace app.jinja_options['extensions'].append('jinja2.ext.do') with app.jinja_env.add_extension('jinja2.ext.do')
However, I am not familiar with how Flask and Jinja work together, so I can't offer much explanation. Hope this still helps!


CellRangeRenderer with MultiGrid - How to tell which grid it is operating on?

Is there a way to get which grid in a multi grid the cellRangeRenderer is operating on?
I can currently get it by accessing the parent prop and checking the className - parent._scrollingContainer.className
I was wondering if there was a less hacky way to do this.
Thank you!

I have the same doubt. I need to render an overlay only for one particular grid. @jrweinb have you got any solution?
Thanks

@darshan-raj - you can check the className on the parent prop...for example, this is how I am using it:
cellRangeRenderer = (props: Object) => {
        const renderedCells = defaultCellRangeRenderer(props);
        const {
            columnStartIndex,
            columnStopIndex,
            parent,
        } = props;
        if (parent._scrollingContainer && parent._scrollingContainer.className.includes('grid-header') && columnStopIndex !== 0) {
            return [...renderedCells, ...this.props.cachedHeaderElements];
        }
        return renderedCells;
    }

@jrweinb @darshan-raj here is a less hacky solution 😄
class MyGrid extends Component {
  constructor(props) {
    super(props);
    this.multiGridRef = React.createRef();
  }

  renderCellRange = props => {
    const children = defaultCellRangeRenderer(props);

    if (props.parent === this.multiGridRef.current._bottomRightGrid) {
      // Operating on bottom right grid
    }

    return children;
  }

  render() {
    return <MultiGrid ref={this.multiGridRef} cellRangeRenderer={this.renderCellRange} />
   }
}

Unfortunately, these hacks will work in typescript only when casting to type any.


Correctly support indefinite durations

Should we also increment the package version/create a new release? After all, some flows of the last release have been fixed (e.g. revising orders, having an indefinite duration, ...).

Should we also increment the package version/create a new release? After all, some flows of the last release have been fixed (e.g. revising orders, having an indefinite duration, ...).

Yes 👍

Should we also increment the package version/create a new release? After all, some flows of the last release have been fixed (e.g. revising orders, having an indefinite duration, ...).

Yes 👍

Alright, done!


Slant Rounds: sure about dividing degree angle by 100?

Or did you mean something like this:
First at the top, add:
from math import radians

Then in process(), replace the myX/Y assignments with this:
	myY =  radians(thisMasterAngle)
	myX = -radians(thisMasterAngle)

And this change to the shear matrix:
	thisLayer.applyTransform([1, myX/2, myY, 1, 0, 0])

Or what exactly is Jacques’ method, spelled out in words?

Thank you for the suggestion. /100 was simply my closest guess. Radians is exactly what I was looking for, as it produces a mathematically natural result, and is identical to the slant in Glyphs.
Given Italic angle is 10:
old transform: applyTransform([1, -0.100000, 0.200000, 1, 0, 0])
new transform: applyTransform([1, -0.087266, 0.174533, 1, 0, 0]) radians(thisMasterAngle)

Or what exactly is Jacques’ method, spelled out in words?

Jacques @fonthausen mentioned a skew with *0.5 compensation.
vSkew = 0.5 * i | hSkew = i , where i = Italic Angle

I tried an alternative method, it produces similar results:
vSkew = i | hSkew = i + 2


SSL Certificate expired

The SSL certificate (on https://api-dev.freme-project.eu) expired today.

The automatic renewal script did not work because it needs to be executed with sudo. I executed it manually and now https works again.


Please use the Help chat room for getting challenge related help. Happy Coding!

There is a flaw in the challenge and I am reporting it.  Please address my
concern.
On Tuesday, May 3, 2016, Jonathan<EMAIL_ADDRESS>wrote:

Please use the Help https://gitter.im/FreeCodeCamp/Help chat room for
getting challenge related help. Happy Coding!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8415#issuecomment-216456688

Have you addressed the problem with this challenge?
On Tuesday, May 3, 2016, Jonathan<EMAIL_ADDRESS>wrote:

Closed #8415 https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8415.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8415#event-648500241

@Stephaceae Please checkout this link about selectors: https://developer.mozilla.org/en/docs/Web/Guide/CSS/Getting_started/Selectors.
HTML Elements don't have periods before them, only classes do.

Yeah, I got it thanks. I'm saying the challenge requires the info, but it's
not included in the challenge, or the challenges leading up to it.
This either means that the challenge should be corrected, by simply
including a section on how to use elements in a style tag.  The other
option is FCC is deliberately omitting the data to screen people out.  I
don't think that is the case, as it appears FCC designed this first set of
challenges of to be easy and engaging for noobs.
I was doing this section as review, as it has been some time since I've
used FCC, and the first time I completed the section, I did not have this
trouble.  The challenge has been modified since, and isn't fully complete.
Please stop sending me info I've already worked out for myself and instead
include that info in the challenge for others to use.
Cheers!
On Tuesday, May 3, 2016, Jonathan<EMAIL_ADDRESS>wrote:

@Stephaceae https://github.com/Stephaceae Please checkout this link
about selectors:
https://developer.mozilla.org/en/docs/Web/Guide/CSS/Getting_started/Selectors
.
HTML Elements don't have periods before them, only classes do.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8415#issuecomment-216457982


Playback rate in Chrome

The chrome browser doesn't seem to be registering changes in playback rate - that is, everything plays back at normal speed.
Specifically noticed this when looking at phrases = https://p5js.org/reference/#/p5.Phrase

I've noticed this bug as well. it was affecting the MK-1 and I fixed it with this commit which included a couple of small changes to my own code and to the p5 sound library (marked with "ericr" in comments). I never exactly figured out what was happening, but it felt like a "race condition" situation in my case: the sounds were playing back sometimes at the correct shifted rate, and sometimes at normal speed. One thing I changed in p5.sound.js was:
this.bufferSourceNode.playbackRate.linearRampToValueAtTime(Math.abs(rate), now);
to
this.bufferSourceNode.playbackRate.value = Math.abs(rate);

quite possibly this is the same issue being discussed on tonejs:
https://github.com/Tonejs/Tone.js/issues/163

Thank you, @ericrosenbaum, and thanks for the bug report + test case, @8ude! I'm sorry for my slow response.
Seems like we have some scheduling bugs... If I run the p5.phrase example on its own, outside of the p5 reference page, it works, but inside of the page, the audiocontext scheduling is less predictable. If I remove time from the call to play, it works. But in both settings, this technique that had worked previously does not work any more:  mySound.play(time, playbackRate);
@ericrosenbaum your solution of assigning the value directly instead of scheduling it looks similar to @tambien's solution for Tone.js. I'm pretty sure I've run into a bug where, once you start using the web audio api scheduler (i.e. linearRampToValueAtTime(val, time)), then value = is ignored. I'm trying to remember which browsers had that issue, and I'm wondering if that's still an issue.

Ok, value =  seems to do the trick when initializing a BufferSourceNode. Then, when changing the playback rate, I am continuing to use linearRampToValueAtTime instead of setValueAtTime because setValueAtTime doesn't work in Chrome, and p5.sound doesn't have the ability to schedule ramp playback changes. This would be a nice feature to add, though!


Online Learning page

Love the Online Learning page!!! 🥇
Here are a few things to fix:
[ ] Include Mixer and YouTube channel link.  "All videos will be live streamed on Mixer  and available for viewing later in the MakeCode YouTube Channel".
[ ] Fix broken links:
https://aka.ms/makecodearcadestreamcommunity doesn't go anywhere - suggest just pointing to https://www.twitch.tv/mr_isaacs/
https://aka.ms/makecodeadafruitstream doesn't go anywhere either - change to https://aka.ms/makecodecpxstream
[ ] I might also suggest adding links to the video titles in the descriptions

[ ] In the Educator Resources section, can we make the hyperlinked text a different color?  Right now it's a bit hidden.

Fixed?

[ ] Remove Steve Isaacs from the Schedule and Stream Description

[ ] Add sub-headers under Live Video Streaming section for "Schedule" and "Description"

Using this same issue for another change request to add the Microsoft Stores "Digital All-Stars" stream.
[ ] Add a 2nd paragraph in the Live Video Streaming section: "Microsoft Stores is proud to host a MakeCode stream with all-star athletes passionate about technology and connecting with their communities.  This week, join Brandon McManus from the Denver Broncos as he helps you build your very own football arcade game with Microsoft MakeCode Arcade!"
[ ] Update the Mixer offline image to include the Digital All-Stars video series

[ ] Add a row for the Digital All-Stars stream to our schedule: use the link https://aka.ms/makecodeDASstream

[ ] Under Available Streams, add a row for Digital All-Stars:
Digital All-Stars
12 PM PST / 3 PM EST
Join your favorite all-star athletes as they use MakeCode Arcade to code games with you at home!
Use this image -

@ganicke

Hi there.  I got some good feedback from the advertising media team about our Online Learning landing page.  We're going to be driving page views as the primary metric, and Mixer + YouTube Playlist views as secondary metrics, so we need to make these CTA's a bit clearer.
Online Learning landing page v2.pptx

Let me know if we can make these changes...

i'll do a pass!

Add John Park under the Video Playlist section.
MakeCode with Adafruit's John Park
Join Adafruit’s John Park as he builds a MakeCode project each week. Covers fundamental concepts and techniques using real-world projects in his studio.

Requesting another update to the Online Learning Page - remove the Mixer link and iframe.
Does Twitch have an api/iframe we can use to embed the schedule or live videos?
Also, can we add the WeCodeMakeCode stream?
Title: WeCodeMakeCode
Link: aka.ms/makecodeWCMCstream
Description: Learn how to build cool Minecraft mods like Snoman vs. Monsters, and Arcade games like Hungry Snake with WeCodeMakeCode! For intermediate to advanced level.
Image:

Yes they do. We can add it next week.

From: Jacqueline Russell<EMAIL_ADDRESS>Sent: Sunday, June 28, 2020 6:00:13 PM
To: microsoft/pxt<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [microsoft/pxt] Online Learning page (#6749)
Requesting another update to the Online Learning Page - remove the Mixer link and iframe.
Does Twitch have an api/iframe we can use to embed the schedule or live videos?
Also, can we add the WeCodeMakeCode stream?
Title: WeCodeMakeCode
Link: aka.ms/makecodeWCMCstream
Description: Learn how to build cool Minecraft mods like Snoman vs. Monsters, and Arcade games like Hungry Snake with WeCodeMakeCode! For intermediate to advanced level.
Image:
[WeCodeMakeCode playlist]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F22108954%2F85963145-2b74c900-b969-11ea-9606-9514ee42c85d.png&data=02|01|jhalleux%40microsoft.com|6fc27876abbe42afbce808d81bc7c7fd|72f988bf86f141af91ab2d7cd011db47|1|0|637289892150584950&sdata=9qAqeId45whUnXFuLc8%2B86j9sUjKpNDMS7kp3lBuU2U%3D&reserved=0
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fpxt%2Fissues%2F6749%23issuecomment-650849969&data=02|01|jhalleux%40microsoft.com|6fc27876abbe42afbce808d81bc7c7fd|72f988bf86f141af91ab2d7cd011db47|1|0|637289892150594943&sdata=b66y%2F8%2BlJPYe4wRSUinkFhyMuUFHc4R4PAuvU64Q3q0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKI7VJ462RMYYT4IARDRY7RR3ANCNFSM4LSEGLFQ&data=02|01|jhalleux%40microsoft.com|6fc27876abbe42afbce808d81bc7c7fd|72f988bf86f141af91ab2d7cd011db47|1|0|637289892150594943&sdata=Dv%2BEvcOK0WrwoBZ1LVbbzkyj460dB4ABrvzU5y3ezgg%3D&reserved=0.

This is all pretty old. I think this was the 'tracking' issue for the effort at the time.


[Feature Request]: Implementing Chat App in style share

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The chat app will help user to generate tailwind components, new ideas etc...
I will use gemini api for the same.
Use Case
AI integration
Benefits
User experience will increase.
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

what do you mean by chat app? an ai chat bot?

no like there will be a new route where user can come and ask ai for components and all

rather than that maybe we can do one thing which is that on each post we give the user a button "Customize with AI" which redirects to a separate page where the user can send his/her query and the AI will customize and provide accordingly. We can send the first message to the AI with the component after that the user can send messages and interact with the AI.

Okay ! I didn't implemented such thing previously but i will try and the idea is interesting ! I will try this aproach :) give me sometime


Add capability to pass custom properties to the trackMetric api

We would like to add custom properties to our metrics telemetry. Currently the contracts data model for metrics in the SDK has properties on the MetricsData object, but the api doesn't allow them as arguments.

Great! PR looks good to me


upgrade gulp-sass to support node.js latest

upgrades node-sass so the build supports node.js 12

@cancerberoSgx sorry for lately reply, so far the node version is doesn't matter for react-bootstrap-table2, so i will suggest people to use node8 is enough. thanks


Fixed bug that overwrites attributes as empty

Attributes that are passed by the function get overwritten as blank in the begining of the else block so that they don't get rendered.
I've fixed this little bug with removing the overwrite line and renaming the $attr as $attribute so that attributes can be rendered correctly.

sorry for delay, bro.

no problem :)


Custom emoji list

It should be possible to provide a configuration file with additional emoji entries.

Closed by #12


Hot fix for naming suggestions template

This template is only available to image model beta testers, but has been broken for a few weeks apparently.
This doesn't fully update the layout, but it does make it render without errors and be legible.

Coverage: 93.506%. Remained the same when pulling 522b8c57c6910036f834f8cb94eec8c23c15d862 on nimmo-naming-suggestions-template into efde94dc2b73a9036383ea432f22118d101806fd on main.


Do not "absorb" mouse scrolling if no scrollbar is visible

It makes sense to prevent scroll events from propagating further up when there's a scrollbar in the table, but when there isn't one (ie. when the list isn't long enough), it makes sense to not absorb the scroll events so there won't be an invisible mouse scrolling black hole on the page.
Scrolling your mouse wheel while the cursor is over a scrollbar-less table should scroll the page, making the table feel like an integrate part of the page.

Seconded. I just had to hack around this by drawing a giant invisible DIV rectangle all over the page so that the table, intentionally without scrollbars, didn't make the rest of the page non-scrollable.

@gsklee @benshimmin Sorry we know this is bad and are planing on fixing this for the next release. For now a hacky fix for this if you know you don't need any scrolling is to set overflowX="hidden" overflowY="hidden" http://jsbin.com/valoco/12/edit.

This should be fixed in the latest release :)

@pieterv :100:


[PhpstanPhpConfig] Add ignore error option

Added ignore error option with test that fails.

https://github.com/symplify/symplify/issues/2915

That looks good :+1:
Could you solve the slashes issue? Then it's ready to merge

The problem is, it's located in Nette\Neon in the encode function and I guess it's meant to work that way.
https://github.com/symplify/symplify/blob/master/packages/package-builder/src/Neon/NeonPrinter.php#L30

I could do something, like replace / single slashes before and readd them.
$neonContent = $this->fixReAddSingleSlashed($neonContent);

Don't know if that should be the way to go?
Any suggestions?

I think it's best to leave it this way and by convention always use double slashes. This works in phpstan.neon.
$parameters->set(
        OPTION::IGNORE_ERRORS,
        [
            "#Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition\\:\\:children\\(\\)#"
        ]
    );

parameters:
    level: max
    paths:
        - src
        - tests

    reportUnmatchedIgnoredErrors: false
    parallel:
        maximumNumberOfProcesses: 6

    ignoreErrors:
        - "#Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition\\:\\:children\\(\\)#"

So if it's fine by you, I would suggest changing the test and add it to the documentation.

Don't know if that should be the way to go?
Any suggestions?

If it make input compatile to output, go for it  :+1:

I think it's best to leave it this way and by convention always use double slashes. This works in phpstan.neon.

It works, but it's not compatible. The input must be equal to output, otherwise there will be bugs.

I understand that input and output should be equal but the problem is:
"#Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition\:\:children\(\)#"

Will in php never be processed as-is because the double slashes are handled as escape of a single slash.
Since it goes from php to neon in this cause outputting as file that goes back from neon to php.
So I don't know how i could solve this, so input matches output in this case.

$neonContent = Neon::encode($phpStanNeon, Encoder::BLOCK);
https://github.com/symplify/symplify/blob/master/packages/package-builder/src/Neon/NeonPrinter.php#L30

Where do we call it during conversion?

https://github.com/symplify/symplify/blob/master/packages/phpstan-php-config/src/PHPStanPHPToNeonConverter.php#L80

Good!
So the fix should be applied somewhere here: https://github.com/symplify/symplify/blob/ca0d84af9733629aaefad580ffca011726c1123d/packages/package-builder/src/Neon/NeonPrinter.php#L31
There is couple of other fixing methods as you can see.
First dummy idea that commes to my head is to double slashes on the input, then cut half on the output. That should work

That's what I was looking into but the problem here is you can not differentiate the difference between a single slash and a double slash here, both come in as a single slash.

I see. I'm trying it out now and it already comes from $parameterBag with broken slashes.
Somewhere in FileLoader, could not find the place.
Let's give this a try then and see if something breaks :)

Thank you :+1:


Added dependency lib and project don't open.

I just added the dependency to my project and when trying to run it, the app doesn't open. Has anyone ever experienced this?
react-native: 0.61.2
react-native-google-signin: 3

Hi there! Sorry you're having an issue. As the issue template explains, we require that you provide a runnable example (please try reproducing the bug with our example app) that reproduces the bug in a github repository. Please try to minimize the superfluous code and focus only on reproducing the bug.
please create a new issue with this and we'll be happy to review it!


NIFI-3222 - Allow ExecuteScript to use ExpressionLangugage to define …

…the scripting engine modules path
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] ~Have you written or updated unit tests to verify your changes?~
[X] ~If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?~
[X] ~If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?~
[X] ~If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?~
[X] ~If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?~

For documentation related changes:

[X] ~Have you ensured that format looks appropriate for the output in which it is rendered?~

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@mattyb149 Would you be able to give me a hand with this PR?
Cheers!

Code LGTM, my comments would be to add this same improvement to InvokeScriptedProcessor, and update one unit test apiece to use Expression Language to set the Module Directory property, to prove that this works

fair enough. will do both

And there's a typo in Language in the commit/PR title ;)

@mattyb149 - should be all goody now

+1 LGTM, Built and ran unit tests, ran NiFi with ExecuteScript using EL for Module Directory, everything looks good. Thanks much for the improvement! Merging to master


RegEx Usage not clear or bug

** Please describe which feature you have a question about? **
I have made model with @searchable directive
Schema is as follows:
type Category @model @searchable {
title: String!
hierarchy: String
}
Now I want to search for string using RegEx in hierarchy field with searchCategory query.
My data items are stored in following fashion.
Sports#Cricket#
Sports#Cricket#T20#
My RegEx is ^Sports#[a-zA-Z0-9]+#
Which should give me
Sports#Cricket#
But getting 0 items in result of query using AppSync console
Any suggestions will be helpful. Am I using it wrong? or Is there bug in the system?
** Provide additional details**
With same RegEx Got the right result full match on following website:
https://regex101.com/

Hi @MShah-Fintech
Since the data you are searching on is analyzed data, it wont find Sports#Cricket# in the inverted table created by elasticsearch. Thats why it is coming as empty.
You can try regex for "Sports" or "cricket" separately  to get the desired result.
Regex : "[sS][a-zA-Z]+, this regex will return documents having Sports in it.

Let me know if above issue is solved. For now I am closing this issue , if you are still stuck , you can comment on this and I will reopen this issue.


Dev

About the PR

Why / Balance

Technical details

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

wrong repo bro


Alternate paths through a dependency chain

So I have a chain on three operations. However, if the first operation fails, I would like to skip to the third operation. How can I express this? My current idea is to build the full dependency chain and then in operationDidFinish check to see which operation finished. If it's the first, check the property I care about, and if it's good, let the process continue unchanged. If it's bad, cancel the current operations in the queue and then readd the third operation. Is there any way to express this behavior in cleaner fashion?

Well, you can attach a BlockCondition to the 2nd operation.
The condition will only allow an operation to execute if the block it is initialized evaluates truthfully. So you would need to test some property of your group operation inside that block. i.e. something like this...
class ThreeOperations: GroupOperation {
   let first = OperationWithProperty()
   let second: Operation()
   let third: Operation()
   init() {
      second = Operation()
      second.addDependency(first)
      third = Operation()
      third.addDependency(second)
      super.init(operations: [first, second, third])
      second.addCondition(BlockCondition { [unowned self] in
            return self.first.propertyIsSet
      })
   }
}

I didn't realize OperationConditions didn't kill the entire dependency chain. Is that configurable? If I want to kill the entire chain, how could I do that?

What do you mean by "kill the entire dependency chain"?

The dependency management of Operation is the same as NSOperation. That means that eventually all operations Finish. And when they finish, then any NSOperations which are depending on them become Ready.
The concept of a "Success" and "Failure" is something which you are attributing to whatever the task is your NSOperation subclass is performing. Neither NSOperation nor Operation knows anything about this.
When you attach a condition to an Operation where the condition evaluates to a failure, then that operation will still finish - but it will do so without executing.
This means that any operations which were depending on it will then become ready.
If this is unexpected behavior, you might find it useful to look over the docs of NSOperation again.
However, what you can do is attach a NoCancelledCondition to an operation, which is a condition which requires that all it's dependencies succeeded.

Thanks for the explanation and solution.

@jshier did NoCancelledCondition and BlockCondition work for you?

I'll have to create some tests to make sure, but the logic sounds correct. I should be able to write some tests soon to double check.


Fix PKG_CONFIG file name issue

Please refer to #43 for details.

I'll take a closer look after I get home today, but at a quick glance it looks good!  I'll likely merge it very soon.
Thanks so much for your time and effort!

Hi Cessen, I am also curious about your plan about this project. Currently it seems that there are a lot works to do, like support custom attributes in OpenEXR header, more compression algorithms and tile output. I can provide some features when I have some time.

Thanks Aaron!  I just merged.  If you need this urgently, let me know, and I'll cut a release.  Otherwise, I'll let this sit for a bit to see if additional things comes in first that can be rolled into a single release.
As for my plans for this project, ultimately I would like it to provide feature-complete bindings for OpenEXR.  But realistically, my time and motivation has declined.  For the foreseeable future I'll mainly be doing maintenance and merging other people's work, although hopefully I'll get back around to working on this properly eventually.
If you want to contribute additional OpenEXR feature support that would definitely be appreciated!  But do keep in mind that my time is going to be limited for the next several months (probably until April or so), so especially complex PRs might take me a while to get to and review.  But yeah, I'd be super happy to get more contributions!


Fix JSON example of connector message

Fix the JSON example which has an invalid syntax in line no.65-69.
I consider that they are caused by the update in #2583
I have tested my fix with Curl just in case.

Docs Build status updates of commit 11ddff9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 11ddff9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0166903:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0166903:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Background and foreground colors do not have a sufficient contrast ratio.

Report
Low-contrast text is difficult or impossible for many users to read. Learn how to provide sufficient color contrast.

This task is more about design and nothing special about coding


rhol crc/manage secrets

This PR has two distinct patches in it:

Removed useless files in manage_secrets
A tiny change that probably doesn't needs its own pull-request,
just a small cleaning

Switch rhol_crc to the new manage_secrets
The important part - switching to the new manage_secrets role
for the sake of consistency and stability.

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/849
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role

/test all

recheck

recheck

recheck

/test all

/approve
Update the small English nit on a PR if you want


As a user I can sort the contact information via name or birthday

... so that I can obtain more functionality from Address Book.

Isn't sorting the contact information via name the same as #1 ?
But sorting by birthday is okay.