Elastic

This is an output writer that writes data directly into an elastic search instance.
Define output writer as follows:
    "outputWriters" : [
        "@class" : "com.googlecode.jmxtrans.model.output.elastic.ElasticWriter",
        "connectionUrl" : "http://<IP_ADDRESS>:9200"
      } ]

[ ]   File: jmxtrans-core/src/main/java/com/googlecode/jmxtrans/model/naming/StringUtils.java:L25-32
 Thanks for the correction, but that is a comment which does not tell us much more than the name of the method. We should probably just remove it (and also remove the @param name    the name).
I tend to be fairly strongly opinionated with comments : if they do not add anything to what is already in the code, they should be removed...
[ ]   File: jmxtrans-output/jmxtrans-output-elastic/src/main/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriter.java:L1-210
 It seems that result can be null, so the condition should probably be if (result == null || !result.isSucceeded()). Though even better would be to understand why thi result can be null.
[ ]   File: jmxtrans-output/jmxtrans-output-elastic/src/main/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriter.java:L1-210
 In case of error, we probably want to throw an exception to ensure that this writer is disabled. Unless this can be a transient error. So probably in case of communication error, we have a good chance of recovery, otherwise probably not so much ...
[ ]   File: jmxtrans-output/jmxtrans-output-elastic/src/test/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriterTests.java:L1-106
 You're missing a mock for the second call to jestClient.execute().
[ ]   File: jmxtrans-output/jmxtrans-output-elastic/src/test/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriterTests.java:L1-106
 We should also verify that call to Elastic that sends the data is actually made and that it contains the results that were given on inputs.
[x]   File: jmxtrans-output/jmxtrans-output-elastic/pom.xml:L1-69
 Could you move <version/> to the main pom? That way we have all versions of all dependencies in a centralized place.
[x]   File: jmxtrans-output/jmxtrans-output-elastic/src/main/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriter.java:L1-238
 Seems that the creation of the JestClient would be better placed in the constructor. That way you can ensure the this object is always in a valid state. I know, we have already too much creation logic in the constructors of OutputWriters, but a cleanup is on the way...
[x]   File: pom.xml:L653-663
 This should have been fixed already by #318 with a profile activated only for JDK >=8. This fix makes the travis build fail as it is run on Java 7. Could you rebase your pull request to include latest fixes ?
 Any chance we could load it from a classpath resource instead ?
[x]   File: jmxtrans-output/jmxtrans-output-elastic/src/main/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriter.java:L1-238
 I'm not an expert on ElasticSerach and Jest, but my guess is that if the creation of mappings fail, this output writer will be unuseable. So you should probably just let the exception bubble up and not hide the issue.
[x]   File: jmxtrans-output/jmxtrans-output-elastic/src/main/java/com/googlecode/jmxtrans/model/output/elastic/ElasticWriter.java:L1-238
 You should extract a computeAlias() method from this block (possibly with a better name).
[x]   General Comment
 Looks good ! Thanks for the contribution ! I have a few mostly minor comments. Could you have a look and tell me if those comments make sense to you ?

This pull request is now managed with CodeReviewHub. Refresh.

Looks good ! Thanks for the contribution ! I have a few mostly minor comments. Could you have a look and tell me if those comments make sense to you ?

:+1:

Sorry to be a pain, but it seems that I have a few more comments...

:+1:

When running this against a elastic server, I see the following non-numeric value passing along in the log file:
17:04:05.442 [pool-17-thread-2] WARN  c.g.j.m.output.elastic.ElasticWriter - Unable to submit non-numeric value to Elastic: [javax.management.openmbean.TabularDataSupport(tabularType=javax.management.openmbean.TabularType(name=Map<java.lang.String,java.lang.management.MemoryUsage>,rowType=javax.management.openmbean.CompositeType(name=Map<java.lang.String,java.lang.management.MemoryUsage>,items=((itemName=key,itemType=javax.management.openmbean.SimpleType(name=java.lang.String)),(itemName=value,itemType=javax.management.openmbean.CompositeType(name=java.lang.management.MemoryUsage,items=((itemName=committed,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=init,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=max,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=used,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long))))))),indexNames=(key)),contents={ <<<... REMOVED VERY LONG STRING ... >>> javax.management.openmbean.CompositeDataSupport(compositeType=javax.management.openmbean.CompositeType(name=java.lang.management.MemoryUsage,items=((itemName=committed,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=init,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=max,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=used,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)))),contents={committed=178978816, init=178978816, max=724828160, used=12257128})})})}, epoch=1439737445281]]
17:04:05.447 [pool-17-thread-2] DEBUG c.g.jmxtrans.jmx.JmxQueryProcessor - Finished running outputWriters for query: Query [obj=java.lang:name=ConcurrentMarkSweep,type=GarbageCollector, useObjDomainAsKey:false, resultAlias=null, attr=[LastGcInfo]]
Any idea what this is?

Seems to be a bug in JmxResultProcessor. You should never see tabular data in an output writer. Having a quick look at the code, it looks like that could happen if a tabular data contains other tabular data... Could that be the case ?

Merging this PR as it looks ready to me. The last bug reported by @stokpop is most probably outside of the code changed in this PR.


v3.0.0

[ ] check if metadata plotting of module scores requires fixing if the score was not calculated due to missing genes
[ ] HVG selection only has an effect on the scaled slot. report number of selected HVGs somehow (e.g., text file)
[ ] subsampled heatmaps too thin now ie +x inches for width
[ ] testing on ASD data
[ ] #21
[ ] #1
[ ] release version 3.0.0

Check if DAG is current


PE-D debugging

Add different colours for task types
Fix task bug for high priority
Prevent UI text overflow
Fix task list sorting bugs and toggling task isDone bugs
Update task Description to accept only ASCII characters
Update UserGuide for doneTask and Description

Codecov Report

Merging #238 (716a4b6) into master (99b2d01) will increase coverage by 0.52%.
The diff coverage is 48.14%.

@@             Coverage Diff              @@
##             master     #238      +/-   ##
============================================
+ Coverage     62.93%   63.46%   +0.52%
- Complexity      830      835       +5
============================================
  Files           138      138
  Lines          3157     3148       -9
  Branches        409      412       +3
============================================
+ Hits           1987     1998      +11
+ Misses         1026     1006      -20
  Partials        144      144

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

.../java/seedu/address/model/task/UniqueTaskList.java
91.83% <ø> (ø)

...c/main/java/seedu/address/ui/DeadlineTaskCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/EventTaskCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MemberListCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/StudentCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/ToDoTaskCard.java
0.00% <0.00%> (ø)

...ain/java/seedu/address/model/task/Description.java
50.00% <50.00%> (ø)

src/main/java/seedu/address/model/task/Task.java
44.30% <62.50%> (+6.97%)
:arrow_up:

...du/address/logic/commands/MarkTaskDoneCommand.java
94.44% <100.00%> (+0.32%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Installed 4.1.7, but reports 4.1.6

Description
I just downloaded and installed 4.1.7, but it reports as 4.1.6 in Home Assistant and the logs.
This is with the Windows installer System.Bridge_4.1.7_x64-setup.exe

2024-04-06 14:50:22 INFO (MainThread) [systembridgebackend.Application] System Bridge 4.1.6: Startup

This is probably also why Home Assistant detects there is an update available for 4.1.7.
What Platform / OS are you running?
Windows
What version are you running?
System Bridge 4.1.7 (at least that was the expectation)
Anything in the logs or a references that might be useful?
No response
Additional information
No response

Ow, I now see that this is probably the same as #2998, but not 100% sure.
Please close if duplicate

Yep same as #2998


[ISTHMUS]support the window function translation from sql to substrait

To fix #48 , the work will be split into two PRs. This the first one to support window function translation from sql to substrait direction.

@jacques-n , can you take a look at this PR?


Can't run 'rcnn_demo' in Matlab(CPU Mode)

Hello,

First of all, thank you for providing a well documented tutorial for rcnn.
I followed the tutorial. I got to the point where I executed the command:
key = caffe('get_init_key');
and it returns me -2 as the tutorial says. Everything seemed to be going well until that point. When I tried running `rcnn_demo('ILSVRC13', 0) ; I got this

I traced the execution flow of the program and I found out that it stopped at here:

rcnn_model.cnn.init_key = ...
caffe('init', rcnn_model.cnn.definition_file, rcnn_model.cnn.binary_file);

It already passes 3 arguments to caffe. Why does MATLAB still output the following error message?

Error using caffe
Expected 3 arguments, got 2

And MATLAB also displays a warning:

Warning: Could not find appropriate function on path loading function handle
/home/rbg/working/rcnn-ilsvrc/imdb/imdb_from_ilsvrc13.m>@(i)fullfile(imdb.image_dir,[imdb.image_ids{i},'.',imdb.extension])

In rcnn_load_model at 21
In rcnn_demo at 59

/home/rbg/working/rcnn-ilsvrc/imdb/imdb_from_ilsvrc13.m>@(i)fullfile(imdb.image_dir,[imdb.image_ids{i},'.',imdb.extension]) seems to be a path on  Dr. Girshick's(the writer) computer. This is very strange. Does this warning matter?
I use the latest version of Caffe and MATLAB 2014a to run the code.

Have solved it. It turns out the latest Caffe changed some of it matlab intefaces. Have to use Caffe v0.999 and Matlab 2012b to run rcnn.

@kaishenchang
hi,it means if i want to run rcnn,i must install caffe v0.999 and matlab 2012b?

@DaChaoXc  I dont wanna to change caffe and matlab version, so is there another method?

@kaishenchang   My caffe version is caffe-rc2,and I run the command key=caffe('get_init_key');It returns -2. When I do the rcnn_demo,I encountered the same problem .Is it necessary to run in caffe v0.999? Can caffe V0.999 run in CPU model?I hope you can help me.Thanks

Can you tell me how to make CPU only?


Remove support for iframes

There's really no good use for them in emails and most clients don't seem to support iframes either.

A lot of clients support iframes (e.g. thunderbird). Removing support would provoke to not to work some emails that are correctly shown in other clients.
In my opinion, iframes support shouldn't be removed, and at least be available to enable it in configuration


Access Control

Private and Public Access: Determine which parts of your application should be accessible to all users and which should be private.
Content Ownership and Editing Rights: Implement checks to ensure users can only edit or delete their own content.

[ ] Remove delete button in frontend for messages not authored by current user


Current status: all mentions of Elk and Mastodon in Nimbus

In short
Since this repo started as a fork of Elk, all the mentions/code specific to Elk and Mastodont need to be replaced with Nimbus and Bluesky. There is already some work that was done in #4, but there is still a lot to be done.
This issue aims to list all the files that needs to be updated to complete the full change from Elk to Nimbus.
Quick notes
About the data
The data used in this issue came from runing these two commands in the root of this repository :
# for Elk references
grep --exclude={README.md,pnpm-lock.yaml,package.json}  --exclude-dir={node_modules,.nuxt} -rnw '.' -e "elk"

# for Mastodon references
grep --exclude={README.md,pnpm-lock.yaml,package.json}  --exclude-dir={node_modules,.nuxt} -rnw '.' -e "masto"

The raw output of these two commands is available in this gist
About the classification
In order to make this easier to read, I have tried to categorize the files in 3 categories:

Config files: (mostly) root files that reference "elk" or "masto(don)" and should be straightforward to update, but it might impact the whole project deployment/dev environnement/GitHub config).
Content to update: files that includes "elk" or "masto(don)", but that should not need to update the logic.
Logic to replace: files that are tightly coupled to Elk or Mastodon specific code and that needs more than just a rename/search and replace.

The items insidde the categories are sorted in alphabetical order. in Items that are duplicated (aka present in both the "Elk" and "Mastodon" list) are in italic and bold.

[!NOTE]
I might have mislabeled some files since I currently do not know Elk nor Bluesky very well, feel free to correct the classification of a file if needed.

Elk references

[x] Config files

[x] .github/ISSUE_TEMPLATE/config.yml #12
[x] .gitignore #12
[x] eslint.config.js #12
[x] docker-compose.yaml #12
[x] Dockerfile #12
[x] netlify.toml #12
[x] public/_redirects #12
[x] unocss.config.ts #7

[ ] Content to update

[x] components/account/AccountFollowButton.vue #7
[x] components/account/AccountMoreButton.vue #7
[ ] components/common/CommonPreviewPrompt.vue
[ ] components/help/HelpPreview.vue
[ ] components/nav/NavFooter.vue
[x] components/nav/NavSideItem.vue #7
[ ]  components/notification/NotificationSubscribePushNotificationError.vue
[ ] components/publish/PublishWidget.vue
[x] components/status/StatusActions.vue #7
[x] components/status/StatusActionButton.vue #7
[x] components/status/StatusActionsMore.vue #7
[x] components/tag/TagActionButton.vue #7
[ ] composables/about.ts
[x] docs/content/1.guide/1.index.md #8
[x] docs/content/1.guide/3.contributing.md #8
[x] docs/content/2.deployment/1.netlify.md #8
[x] docs/content/80.pwa.md #8
[x] docs/content/99.privacy.md #8
[ ] docs/components/global/TranslationState.vue
[ ] modules/tauri/index.ts
[ ] pages/settings/about/index.vue
[ ] service-worker/nimbus-sw.ts
[ ] server/utils/shared.ts

[ ] Logic to replace

[ ] utils/elk-idb
[x] pages/settings/language/index.vue #12
[ ] service-worker/notification.ts
[x] scripts/prepare-translation-status.ts #12
[ ] tests/nuxt/content-rich.test.ts
[ ] tests/unit/permalinks.test.ts

Mastodon references

[ ] Config files

[ ] nuxt.config.ts
[ ] .github/renovate.json5
[ ] .vscode/settings.json

[ ] Content to update

[x] docs/content/1.guide/3.contributing.md #8
[ ] Locales (json files inside locales folder)

[ ] Logic to replace.

[ ] components/account/AccountAvatar.vue
[ ] components/account/AccountBigAvatar.vue
[ ] components/account/AccountBigCard.vue
[ ] components/account/AccountCard.vue
[ ] components/account/AccountDisplayName.vue
[ ] components/account/AccountFollowButton.vue
[ ] components/account/AccountFollowRequestButton.vue
[ ] components/account/AccountHandle.vue
[ ] components/account/AccountHeader.vue
[ ] components/account/AccountHoverCard.vue
[ ] components/account/AccountHoverWrapper.vue
[ ] components/account/AccountInfo.vue
[ ] components/account/AccountInlineInfo.vue
[ ] components/account/AccountMoreButton.vue
[ ] components/account/AccountMoved.vue
[ ] components/account/AccountPaginator.vue
[ ]  components/account/AccountPostsFollowers.vue
[ ] components/account/AccountRolesIndicator.vue
[ ] components/account/TagHoverWrapper.vue
[ ] components/common/CommonPaginator.vue
[ ] components/common/CommonTrending.vue
[ ] components/common/CommonTrendingCharts.vue
[ ] components/conversation/ConversationCard.vue
[ ] components/conversation/ConversationPaginator.vue
[ ] components/list/Account.vue
[ ] components/list/ListEntry.vue
[ ] components/modal/ModalMediaPreviewCarousel.vue
[ ] components/modal/ModalContainer.vue
[ ] components/notification/NotificationCard.vue
[ ] components/notification/NotificationPaginator.vue
[ ] components/publish/PublishWidget.vue
[ ] components/publish/PublishAttachment.vue
[x] components/publish/PublishWidgetList 7cb8ad01d5634bf0bfcb2c1d964ec1acab47ef10
[ ] components/report/ReportModal.vue
[ ] components/settings/SettingsProfileMetadata.vue
[ ] components/search/SearchAccountInfo.vue
[ ] components/search/SearchHashtagInfo.vue
[ ] components/status/StatusAccountDetails.vue
[ ] components/status/StatusActions.vue
[ ] components/status/StatusActionsMore.vue
[ ] components/status/StatusAttachment.vue
[ ] components/status/StatusBody.vue
[ ] components/status/StatusEmbeddedMedia.vue
[ ] components/status/StatusCard.vue
[ ] components/status/StatusContent.vue
[ ] components/status/StatusDetails.vue
[ ] components/status/StatusLink.vue
[ ] components/status/StatusMedia.vue
[ ] components/status/StatusPoll.vue
[ ] components/status/StatusPreviewCard.vue
[ ] components/status/StatusPreviewCardMoreFromAuthor.vue
[ ] components/status/StatusPreviewCardNormal.vue
[ ] components/status/StatusPreviewGitHub.vue
[ ] components/status/StatusPreviewStackBlitz.vue
[ ] components/status/StatusReplyingTo.vue
[ ] components/status/StatusTranslation.vue
[x] components/status/StatusVisibilityIndicator.vue 7cb8ad01d5634bf0bfcb2c1d964ec1acab47ef10
[ ] components/tag/TagCard.vue
[ ] components/tag/TagActionButton.vue
[ ] components/tag/TagCardPaginator.vue
[ ] components/timeline/TimelineHome.vue
[ ] components/timeline/TimelinePaginator.vue
[ ] components/timeline/TimelinePublic.vue
[ ] components/tiptap/TiptapHashtagList.vue
[ ] components/tiptap/TiptapMentionList.vue
[ ] composables/cache.ts
[ ] composables/command.ts
[ ] composables/content-parse.ts
[ ] composables/dialog.ts
[ ] composables/masto/* x10
[ ] composables/paginator.ts
[ ] composables/push-notifications/createPushSubscription.ts
[ ] composables/push-notifications/types.ts
[ ] composables/push-notifications/usePushManager.ts
[ ] composables/notification.ts
[ ] composables/users.ts
[ ] composables/timeline.ts
[ ] composables/thread.ts
[ ] constants/index.ts
[ ] middleware/1.permalink.global.ts
[ ] pages/[[server]]/lists/index.vue
[ ]<EMAIL_ADDRESS>[x] pages/intent/post.vue 7cb8ad01d5634bf0bfcb2c1d964ec1acab47ef10
[ ] pages/notifications.vue
[ ] pages/notifications/[filter].vue
[ ] pages/settings/profile/appearance.vue
[ ] plugins/0.setup-users.ts
[ ] service-worker/types.ts
[ ] tests/nuxt/content-rich.test.ts
[ ] tests/setup.ts
[ ] tests/unit/reorder-timeline.test.ts
[ ] types/index.ts

We also need to change Mastodon from all json files inside the locales folder.

Hi, I created #39 to update references in "Content to update".


Truncated Log Display in Unraid

Describe the bug
When looking at the log from within Unraid Docker section, the first ~10 characters are cut off.
To Reproduce
Unraid > Docker tab > left click on Machinaris icon > Logs > missing log text
Steps to reproduce the behavior, e.g.:

Navigate to Unraid webUI
Click on Docker tab (assuming Machinaris is already installed)
Click on Machinaris icon to show options
Select Logs
See error

Expected behavior
It is expected to see all of the desired text without being cut off.
System setup:

OS: Unraid
Docker version number: 20.10.5
Machinaris branch: main
Machinaris version number: Machinaris 0.5.4

Config
Name:
machinaris
Overview:
Building upon the official Chia binaries, Machinaris also offers a plot manager, log monitor, and a simple WebUI for Unraid.
Repository:
ghcr.io/guydavis/machinaris
Network Type:
Custom : br0
Fixed IP address (optional):
<IP_ADDRESS>
Subnet: <IP_ADDRESS>/24
Console shell command:
Shell
Privileged:
OFF
appdata:
/mnt/user/appdata/machinaris
EDITREMOVE
Holds app data for the container. Not for temp plotting or final plot storage.
plots:
/mnt/disk1/
EDITREMOVE
Holds completed Chia plots for farming.
plotting:
/mnt/cache/
EDITREMOVE
Your fast temp space for plotting, ideally a cache pool or unassigned SSD.
chia_protocol:
8444
EDITREMOVE
Sync port to the network. Port-forward on your router!
web:
8926
EDITREMOVE
Machinaris local web app port.
plots_dir:
<IP_ADDRESS>_Public
EDITREMOVE
A colon-separated list of all in-container folders holding your completed plots.
flax_protocol:
6888
EDITREMOVE
Sync port to the network. Port-forward on your router!
Show more settings ...
Show docker allocations ...
Add another Path, Port, Variable, Label or Device
# paste your Docker run or Docker compose here
I am unsure of how to do this, since it is run as a container

Additional context & screenshots

Thanks for the details report.  Yes, I've seen this and it's annoying for sure.  Unfortunately, the issue seems to be in the Unraid Docker Logs window as other containers have reported it as well.  The workaround seems to be just executing:
docker logs machinaris

from an Unraid console/shell as the full log output is then shown, not cut-off:

Hi!  Thanks again for reporting this.  Turned out that it was related to the use of the -t Extra Parameter on the Machinaris docker config in Unraid.  Up until the just released v0.5.5 of Machinaris, Plotman required a tty to function, however I've now corrected that  as part of a fix for TrueNAS users.
So, now the -t is no longer required  in the "Extra Parameters" field of the Unraid Docker configuration for Machinaris.  A great side-effect of this is that the weird log window truncation that you found above, no longer occurs when this -t flag is removed.  Please update to the latest Machinaris and remove the -t flag, then Apply in Unraid admin UI.  This should fix it.
Existing Unraid users will need to update their configs, but I've updated the Unraid application template to skip this flag for new users.  Thanks again!


use requre for storing data for tests

replace storing requests objects by requre project.

[x] regenerate test data for ogr
[x] remove old functionality after packit will be fixed as well, not now in this PR

rebased and used in #231


v8/js engine string cache use age

Hey Jack!
Thank you again for this amazing library!
I'n using parts of the code in https://github.com/Bnaya/objectbuffer
V8 and other js engines are trying to deduplicate strings references, and also hold strings as rope data structure.
In my use-case, i'm doing many decoding of the same string over and over again, and i'm not sure how to actual memory in the js engine side gonna end up.
I was wondering if you made any analysis regarding that topic and if you cloud share some findings
Thanks!
Bnaya

You raise a good point about a valid concern. I am also interested in knowing how well this library handles strings. However, I do not know how to make an analysis of such data. I'm not that deep in the V8 engine internals. Perhaps you could point me to a article explaining what to do. Thank you.


Update Testnet MPC Contract Address

It looks like there was a new contract deployed (although it is also not providing signatures).
I immediately ran into the issues upon change:

EVM Address was different (despite using the same derivationPath).
default gas was too low "Smart contract panicked: panicked at contract/src/lib.rs:345:9:\nInsufficient gas provided. Provided: 200.0 Tgas Required: 300.0 Tgas". Hence the change of everything to 300.
Recursion depth exceeded - aka "Smart contract panicked: Signature was not provided in time. Please, try again.".

TODO Later:
Add the following additional stuff to connection config:
// const connectionConfig = {
//   networkId: "testnet",
//   keyStore: new keyStores.BrowserLocalStorageKeyStore(),
//   nodeUrl: "https://rpc.testnet.near.org",
//   walletUrl: "https://testnet.mynearwallet.com/",
//   helperUrl: "https://helper.testnet.near.org",
//   explorerUrl: "https://testnet.nearblocks.io",
// };

Merging without approval  (no logical changes).


[refactor](jdbc catalog) refactor jdbc catalog get databases logic

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


fix: allow consistent traversals

Fixes https://github.com/dragonflydb/dragonfly/issues/2820

The mutex would contend between migrations.
We have users using 20k connections with bullmq workload and 10+ connection rate that migrate. You grab mutex for the whole time you create a bottleneck.


keychain not available

I just did brew install buildkite/cli/bk then tried to bk configure buildkite but it wants to store my credentials in a keyring file:
$ bk --keyring-keychain=login configure buildkite
──── Configuring Buildkite GraphQL credentials

Create a GraphQL token at https://buildkite.com/user/api-access-tokens/new. Make sure to tick the GraphQL scope at the bottom.

GraphQL Token<EMAIL_ADDRESS>✅

Enter passphrase to unlock /Users/sj26/.buildkite/keyring/:
^C

I'm on macOS (11.2.2) so want it to be stored in my keychain. But that doesn't seem to be an available option, only "pass" and "file" are available:
$ bk help
usage: bk [<flags>] <command> [<args> ...]

Manage buildkite from the command-line

Flags:
  --help           Show context-sensitive help (also try --help-long and --help-man).
  --version        Show application version.
  --debug          Show debugging output
  --debug-graphql  Show requests and responses for graphql
  --keyring-backend=KEYRING-BACKEND
                   Keyring backend to use: [pass file]
  --keyring-file-dir="~/.buildkite/keyring/"
                   Use the file keyring-backend with a specific directory
  --keyring-keychain=KEYRING-KEYCHAIN
                   Use the macOS keychain keyring-backend with a specific keychain (defaults to login)
  --keyring-pass-dir=KEYRING-PASS-DIR
                   Use the pass password manager with a specific directory
  --keyring-pass-cmd=KEYRING-PASS-CMD
                   Name of the pass executable
  --keyring-pass-prefix=KEYRING-PASS-PREFIX
                   Prefix to prepend to the item path stored in pass

$ bk --version
1.1.0

“solved” in #115 /  https://github.com/buildkite/cli/releases/tag/v2.0.0 😅


ROS2 Wrapper for Depth-Anything

Hey, I made a ROS2 package for using Depth-Anything with Rviz and Gazebo. Here is the link: https://github.com/polatztrk/depth_anything_ros

Thank you! I will update the README.


URL signature expired

The CDN URL generated have an invalid signature. This wasn't always the case, they were working fine up until now.
How do I refresh the signature token in the CDN URL?
The CDN returns a 403: URL signature expired error
My dependencies and versions
"dependencies": {
    "@fortawesome/fontawesome-svg-core": "^6.4.0",
    "@fortawesome/react-fontawesome": "^0.2.0",
    "@strapi/plugin-i18n": "4.9.1",
    "@strapi/plugin-users-permissions": "4.9.1",
    "@strapi/provider-upload-aws-s3": "^4.9.1",
    "@strapi/strapi": "4.9.1",
    "@webvibe-io/strapi-plugin-instagram": "^1.2.5",
    "better-sqlite3": "7.4.6",
    "pg": "^8.7.1"
  }

The URL's saved in the plugin expire after 1 week, normally the URL would be updated if the image already exists but in the current code it simply discards the new information. Had the same problem so fixed this in this commit. Which is featured in my PR #21

Thanks. I actually saw your PR this morning and was glad that a possible fix is already in place. When are you expecting to get the PR merged?

Fixed this in my copy/continuation of this package @Ramon404 . https://github.com/SvenWesterlaken/strapi-plugin-instagram-images


"Truly Random" isn't actually random?

randomColor({hue: 'random',luminosity: 'random',count: 54});

More of a pet peeve than an severe issue: On the demo page, you have the above code for Truly Random colors, but it produces dark colors more often than not, so it appears biased and/or constrained to certain saturations and luminosities (which means its not truly random).
Comparison of randomColor on the left, to fully random RGB values (on the right).

Hmm, good catch, I had wondered that myself. I'm not sure why this is happening, it should picking between 0 and 100 for both Saturation and Brightness in the HSB color space. Will investigate.
Code for Brightness: https://github.com/davidmerfield/randomColor/blob/master/randomColor.js#L172-L174
Code for Saturation: https://github.com/davidmerfield/randomColor/blob/master/randomColor.js#L129-L131


use primary and secondary in language

@GraysonNull I don't know if we should drop the old API endpoints without a deprecation. It's not a documented API, but we do point people at it sometimes. How about leaving both the old and new endpoints in the code for a few versions before removing the old endpoints?

Ah i didn't realize those kurl endpoints were public facing. Yeah agreed we shouldn't change them then. I'll duplicate them with updated language for now and we can phase them out after a few versions like you suggested. Thanks


Support customization of NatsOrganizationApp related_names

Codecov Report

Merging #29 (ec195de) into main (47610a7) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ Coverage   76.03%   76.21%   +0.18%
==========================================
  Files          22       23       +1
  Lines         651      656       +5
==========================================
+ Hits          495      500       +5
  Misses        156      156

Impacted Files
Coverage Δ

...o_nats_nkeys/migrations/0009_auto_20221105_1710.py
100.00% <100.00%> (ø)

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


 One thing that is super frustrating is trying to see the form submission progress i.e. submitting form 3 out of 5.
Here is how they render on Android 4.4.2, notice the very readable black on black:

This PR makes that text readable by using the standard notification builder:

On new devices:

The tests are failing because https://github.com/dimagi/commcare-odk/pull/1144 needs to get merged into master (and 2.27)


Argument 'picture' must be a picture that can be used as a Icon.

************** Exception Text **************
System.ArgumentException: Argument 'picture' must be a picture that can be used as a Icon.
at System.Drawing.Icon.Initialize(Int32 width, Int32 height)
at System.Drawing.Icon..ctor(String fileName, Int32 width, Int32 height)
at AudioSwitch.Classes.DeviceIcons.GetIcon(String iconPath)
at AudioSwitch.Classes.EndPoints.GetAllDeviceList()
at AudioSwitch.Forms.FormSettings.FormSettings_Load(Object sender, EventArgs e)
at System.Windows.Forms.Form.OnLoad(EventArgs e)
at System.Windows.Forms.Form.OnCreateControl()
at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
at System.Windows.Forms.Control.CreateControl()
at System.Windows.Forms.Control.WmShowWindow(Message& m)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
at System.Windows.Forms.Form.WmShowWindow(Message& m)
at System.Windows.Forms.Form.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies **************
mscorlib
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1080.0 built by: NETFXREL3STAGE
CodeBase: file:///C:/Windows/Microsoft.NET/Framework/v4.0.30319/mscorlib.dll
AudioSwitch
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.1.3
CodeBase: file:///C:/Program%20Files%20(x86)/AudioSwitch/AudioSwitch.exe
System.Windows.Forms
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1038.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
System
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1081.0 built by: NETFXREL3STAGE
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
System.Drawing
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1078.0 built by: NETFXREL3STAGE
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
System.Xml
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1064.2 built by: NETFXREL3STAGE
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
System.Configuration
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1038.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
System.Core
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1038.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
Microsoft.GeneratedCode
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1064.2 built by: NETFXREL3STAGE
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
Accessibility
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.6.1038.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.
For example:

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.
ErrorLog.txt

I realize that you have probably moved on from this however I had the same issue you had and instead of modifying the source I discovered that it wasn't playing nice with my elgato sound capture device. The icon in both the playback and recording devices was referencing to the elgato sound capture .exe file instead of an .ico file. Therefore I changed it to one of the system default icons and the application worked normally. Hope this helps!

Okay, so it's a invalid path issue by the device manufacturer


Drag a simple object to create a more complex object

Hi
I have a dnd-list of users with many fields and arrays
I need to drop into it a dnd-draggable of a applicant with only name and email so they can fill all the details.
right now dragging an applicant into the users list creates like a shallow copy of a user and the  applicant data isn't seen.
In dnd-drop i can drill down the event.currentTarget and set the fields there but it seems hacky
is there a better way to implement something like this?

Hi
If anyone's interested this can be achieved very simply by modifying the html template of dnd-list to accommodate different model structures.

@safhac hi, I'm running into the exact same problem and I'm not sure I understand what you mean. How did you solve this? Thanks!


Merch Store Interface

Design the UI for the in-game store where players can purchase badges and other items using credits.
In-game store UI is accessible and lists all available items for purchase.
Players can purchase items using credits, with successful transactions.
Store shows accurate stock, prices, and descriptions for each item. bug, good first issue, with new enhancement

This issue is In progress


[release-1.13] Update branch for update_deps

Please provide a description of this PR:
Update branch used in update_deps

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

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.13
d1a30c996cb4f310ebcaab3f8ec030481abfc304
link
true
/test unit-tests_istio_release-1.13

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

/test unit-tests_istio_release-1.13


error during training ( make: * [data/foo/foo.traineddata] Error)

Hi, I created everything correctly. After the 'make training' command, some of my files were not created and I am getting this error:
Config file is optional, continuing...
Failed to read data from: data//foo/foo.config
Failed to read data from: data//radical-stroke.txt
Error reading radical code table data//radical-stroke.txt
Makefile:123: recipe for target 'data/foo/foo.traineddata' failed
make: * [data/foo/foo.traineddata] Error

@kba
@zdenop
@jimregan

oh. yes i have this error . HELP

posting screenshots does not help much. Input data could help more.

I created everything correctly.

So why do you get errors like these?
Failed to read data from: data//foo/foo.config
Failed to read data from: data//radical-stroke.txt
Error reading radical code table data//radical-stroke.txt

i have this error
Connecting to github.com
Unable to establish SSL connection.
Makefile:317: recipe for target 'data/radical-stroke.txt' failed

I have this issue too but it resolved. Simply, rm data/radical-stroke.txt to download it, make sure that your internet is good.


W-11891317 Exchange Added GraphQL API info jh

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

In index.adoc I found a typo, it says GraghQL instead of GraphQL. The rest seems ok to me (focus on GraphQL)

fixed


port SYNONYM to Python

maintained the effect of the original logic without all the crazy array manipulations.

ahhh the simplicity! thank you!


Higher ros camera frame rate

ros2 topic hz reports the desired 5 Hz, but rqt_image_view and rviz show only a fraction of that.
Try reliable instead of best effort?
https://www.youtube.com/watch?v=i9g27hR9TWU

ros2 bag record -a

ros2 bag info rosbag2_2024_04_05-09_00_49/

Files:             rosbag2_2024_04_05-09_00_49_0.mcap
Bag size:          206.7 MiB
Storage id:        mcap
ROS Distro:        rolling
Duration:          9.321s
Start:             Apr  5 2024 09:01:23.392<PHONE_NUMBER>.392)
End:               Apr  5 2024 09:01:32.714<PHONE_NUMBER>.714)
Messages:          3543
Topic information: Topic: /robot/depth/image | Type: sensor_msgs/msg/Image | Count: 29 | Serialization Format: cdr
                   Topic: /robot/color/image_color | Type: sensor_msgs/msg/Image | Count: 39 | Serialization Format: cdr
                   Topic: /tf_static | Type: tf2_msgs/msg/TFMessage | Count: 1 | Serialization Format: cdr
                   Topic: /rosout | Type: rcl_interfaces/msg/Log | Count: 10 | Serialization Format: cdr
                   Topic: /car/cloud | Type: sensor_msgs/msg/PointCloud2 | Count: 99 | Serialization Format: cdr
                   Topic: /car/color/camera_info | Type: sensor_msgs/msg/CameraInfo | Count: 53 | Serialization Format: cdr
                   Topic: /events/write_split | Type: rosbag2_interfaces/msg/WriteSplitEvent | Count: 0 | Serialization Format: cdr
                   Topic: /car/color/image_color | Type: sensor_msgs/msg/Image | Count: 49 | Serialization Format: cdr
                   Topic: /car/depth/camera_info | Type: sensor_msgs/msg/CameraInfo | Count: 53 | Serialization Format: cdr
                   Topic: /clock | Type: rosgraph_msgs/msg/Clock | Count: 1491 | Serialization Format: cdr
                   Topic: /robot/color/camera_info | Type: sensor_msgs/msg/CameraInfo | Count: 45 | Serialization Format: cdr
                   Topic: /robot/depth/camera_info | Type: sensor_msgs/msg/CameraInfo | Count: 45 | Serialization Format: cdr
                   Topic: /tf | Type: tf2_msgs/msg/TFMessage | Count: 1491 | Serialization Format: cdr
                   Topic: /car/depth/image | Type: sensor_msgs/msg/Image | Count: 49 | Serialization Format: cdr
                   Topic: /robot/cloud | Type: sensor_msgs/msg/PointCloud2 | Count: 89 | Serialization Format: cdr
                   Topic: /parameter_events | Type: rcl_interfaces/msg/ParameterEvent | Count: 0 | Serialization Format: cdr
Service:           0
Service information:

So the frame rate is there in the mcap (though it doesn't have to deserialize, it's an issue with rqt image view and rviz that the full rates aren't seen


docs: sphinx 1.4.2

Signed-off-by: Tibor Simko<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling c44d45fb91640036a79bdb7bf187db17c11bd193 on tiborsimko:sphinx-1.4 into de90ddc2e2810c8256575377005968b62a8ba4b3 on inveniosoftware:master.


Crash on OS X 10.6.8

dyld: Symbol not found: _kSecDigestSHA1
  Referenced from: /Applications/Spotifree.app/Contents/MacOS/../Frameworks/Sparkle.framework/Versions/A/Sparkle
  Expected in: /System/Library/Frameworks/Security.framework/Versions/A/Security
 in /Applications/Spotifree.app/Contents/MacOS/../Frameworks/Sparkle.framework/Versions/A/Sparkle
Trace/BPT trap

Sparkle 1.9 claims to be compatible with 10.6, and I can't find any similar issues in their repository. Not sure if it's the problem in their library or if we are doing something wrong.

As I just read it's not compatible with 10.6 anymore. I don't know, whether we should switch back to Sparkle 1.5b, because of that.

Sorry, don't know how I could have misread that :)

[...] switch back to Sparkle 1.5b

Are we using any specific features from 1.9? By the way, I see only 1.8 on their website.

No. I'll switch to 1.6.1. It is the most recent version which supports Snow Leopard


Fix bug discoverd while testing

Get registered name when preferred name doesn't exist.
fix focus issue on input component

Coverage decreased (-0.0004%) to 83.519% when pulling fd0256006a982ea49895698d3237710e96bbc42b on CAB-3457 into e90fc12e2db0b0f67e167e7b7ff929b4e7a8d717 on develop.


Read spacing table from OpenDB

Hi I'm tring to read the spacing table of a layer in LEF. The table is a 2d table with indices of width and length.
I can use the getV55SpacingTable() to read the entries of the table. But how do I read the index array of width and length?
I can only use the getMaxWideDRCRange() and getMinWideDRCRange() to get the max and min of width and length, but how about the values in the middle?

Sorry, didn't see the getSpacing().


Retrieving group names from OBJ file using hitResult or anything similar

I am using ModelRenderable to render a 3D object. The source asset is an .obj file with groups denoted by g and vertices denoted by vt and v.
I want to use hitResult (or any other thing), that would return me the name denoted in the .obj file under g when a user taps anywhere within that group.
How do I do that?
Please let me know if my question doesn't make sense.

If you want a hit result against a submesh of an obj, you would have to split that out into a separate model file, and instantiate it independently. Then you could get hit results directly associated with that model.

Is it possible for your to give me an example or how to do this?


Use https instead of git protocol in Gemfiles

Removes Bundler warnings: "The git source git://github.com/... uses the
git protocol, which transmits data without encryption. Disable this
warning with bundle config git.allow_insecure true, or switch to the
https protocol to keep your data secure."
See https://github.com/bundler/bundler/pull/2569 for more info.

Euwe Backport details:
$ git log
commit f0baaf4668815e0dac61b946889d24306bb02126
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Sep 29 17:44:47 2016 -0400

    Merge pull request #11585 from chrisarcand/use-https-git-links

    Use https instead of git protocol in Gemfiles
    (cherry picked from commit ab04015cabfd1246496c759982fc9260304db421)


[Search Project Manager] shows 1Password icon for vs-code remote projects

Extension
https://github.com/raycast/extensions/tree/main/extensions/visual-studio-code-project-manager
Description
I noticed that the icon for my vs-code remote project is the 1Password file icon. Can I configure that? Even better: it would be nice if I could customise icons for my projects. Maybe even by supporting favicons.
Who will benefit from this feature?
People managing multiple projects, of which some are Remote-SSH.
Anything else?

Currently, no own data is stored in the extension. All information is loaded from the Visual Studio Extension.
Therefore, it is not possible to define your own icon for a project.


support subresource integrity [was: CDN does not offer specific versions]

According to the documentation, the MathJax CDN does not offer any endpoints for specific versions, but only for latest patch versions. This makes using subresource integrity with the CDN effectively impossible. I understand the desire to give users the latest patches--this is one reason to use the CDN after all--but it should be possible to use the CDN without compromising security.

@dpvc @zorkow any thoughts?

but it should be possible to use the CDN without compromising security.

While I'm not strictly against this request, this particular argument strikes me as odd. Using a third-party CDN seems to always compromise security. After all, it could shut down or our domain might accidentally expire and be bought up by some spyware-injecting company.

@pkra The great thing about subresource integrity is that it solves exactly this problem. If the resource served is not byte-for-byte identical to what is expected, the script will simply be ignored. So if a spyware-injecting company buys the domain and injects spyware, the worst thing that can happen to the user is that their equations don't render.

I'm not sure I understand what the attack scenario would be, and how having
control over versions would help.
On 1 Feb 2016 8:23 a.m., "Peter Krautzberger"<EMAIL_ADDRESS>wrote:

@dpvc https://github.com/dpvc @zorkow https://github.com/zorkow any
thoughts?
but it should be possible to use the CDN without compromising security.
While I'm not strictly against this request, this particular argument
strikes me as odd. Using a third-party CDN seems to always compromise
security. After all, it could shut down or our domain might accidentally
expire and be bought up by some spyware-injecting company.
—
Reply to this email directly or view it on GitHub
https://github.com/mathjax/MathJax/issues/1366#issuecomment-177838349.

@pkra The great thing about subresource integrity is that it solves exactly this problem. If the resource served is not byte-for-byte identical to what is expected, the script will simply be ignored.

Thanks for repeating that.
@zorkow

I'm not sure I understand what the attack scenario would be, and how having
control over versions would help.

See the linked spec.

or https://blog.cloudflare.com/an-introduction-to-javascript-based-ddos/

So it's a fancy name for distributing a hash. Not very practical from a
MathJax POV.
But we could always offer the latest.
On 1 Feb 2016 8:51 a.m., "Peter Krautzberger"<EMAIL_ADDRESS>wrote:

or https://blog.cloudflare.com/an-introduction-to-javascript-based-ddos/
—
Reply to this email directly or view it on GitHub
https://github.com/mathjax/MathJax/issues/1366#issuecomment-177859740.

Not very practical from a MathJax POV.

Yeah, I was wondering about this.
@alexbecker I suspect this would require not just CDN maintenance but cause lots of actual implementation work in our own resource loading facilities; for legacy browser support, MathJax loads its resources by injecting script tags into the DOM and I suspect we would need to inject them with the correct hashes. So that seems difficult, especially since we're starting to work on 3.0 which will see a completely different approach to the resource loading.

Of course, if there needs to be some work on the code then PRs are more than welcome. If not, even better.

The <script> tag injections makes this much harder, yes. The hashes could be hard-coded in, but they'd have to be recomputed with every release, and any circular references would break. A build tool would probably be required to automate the process. It probably isn't worth it currently, but subresource integrity is worth keeping in mind for 3.0 if you remove the script injections.
In my specific use-case, I was able to use mathjax-node to precompile my equations and get better security and performance than a verified script from the CDN, so thanks @pkra for maintaining that!

The hashes could be hard-coded in, but they'd have to be recomputed with every release, and any circular references would break.

Thanks for confirming.

It probably isn't worth it currently, but subresource integrity is worth keeping in mind for 3.0 if you remove the script injections.

Definitely. Thanks for filing the issue and helping us understand the problem!

Unless I misunderstand what is being requested, this seems to be resolved by now.  For example, I can do this:
    <script type="text/javascript"
            src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-AMS-MML_HTMLorMML"
            integrity="sha384-Ra6zh6uYMmH5ydwCqqMoykyf1T/+ZcnOQfFPhDrp2kI4OIxadnhsvvA2vv9A7xYv"
            crossorigin="anonymous">
    </script>

It is true that I had to compute the integrity hash myself, using
curl 'https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-AMS-MML_HTMLorMML' -o mathjax.js
openssl dgst -sha384 -binary mathjax.js | openssl base64 -A

So maybe that could be included with the documentation for each release, for example.  But it certainly seems like the original issue can be closed.

Yes, this is no longer an issue now that the CloudFlare CDN has replaced MathJax's CDN and offers specific versions. The issue was that previously you couldn't specify version 2.7.1 for example, so the resource would regularly change on you.

Please note that this will likely be insufficient.
MathJax 2.x loads its resources by inserting script tags and is not set up to pass integrity hashes along.
If you need proper SRI in 2.x, you will have to either whitelist all components or use a single-file build. (In v3., this won't be a problem anymore as the loading will occur differently.)

@pkra It works on a simple page of mine, but I'm wondering if there are more complicated tex or maybe different configs that might screw it up.  Do you have any examples?

Oh, I see.  There are more scripts that are loaded, but they don't have SRI; only the first one that I enter manually has SRI.  So it's still a valid issue.

Mathjax should be able to have and use integrity of all the extension files. It shouldn't be difficult. All the integrity is listed in https://cdnjs.com/libraries/mathjax.
FYI: https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity


HDDS-8036. Unprotected flush in SCMHADBTransactionBuffer

What changes were proposed in this pull request?
Found by  https://issues.apache.org/jira/browse/HDDS-8031?page=com.atlassian.jira.plugin.system.issuetabpanels%3Aall-tabpanel
The issue is found due to the lack of lock protection.
The addToBuffer/removeFromBuffer may be concurrently called with flush, which may add KV into a closed batch and cause the JVM crash.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8036
How was this patch tested?
Original Error test:
https://github.com/Xushaohong/ozone/actions/runs/4280636237/jobs/7452661792
POC fix test:
https://github.com/Xushaohong/ozone/actions/runs/4281772162/jobs/7455208505

cc @tanvipenumudy @sumitagrawl can you please take a look as well.

@Xushaohong , It is better to add guard init() with the write lock since

it won't hurt the performance; and
it is easier to maintain the code (someone may call init() outside Ratis in the future.)

Thanks @Xushaohong for the fix, @kaijchen, @szetszwo for the review.


asynchronously check for version update

This will start a thread to check the version to avoid having a delay for the network request. It will report to the log what's going on.

one possibly weird thing: the version status prints before the banner

one possibly weird thing: the version status prints before the banner

force-output after printing the banner? I assume checking for the version actually takes longer but just gets flushed first

Nix the banner
On Thu, Nov 14, 2019 at 2:58 PM Robert Smith<EMAIL_ADDRESS>wrote:

one possibly weird thing: the version status prints before the banner
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/rigetti/quilc/pull/478?email_source=notifications&email_token=AEAK273XCO224WYZ3LN3IO3QTXJZ5A5CNFSM4JNSSP22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEDT26I#issuecomment-554122617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEAK27YOHUNTGKUEJXXXEG3QTXJZ5ANCNFSM4JNSSP2Q
.

--
Mark Skilbeck
Junior Quantum Engineer
<EMAIL_ADDRESS>+1<PHONE_NUMBER>
rigetti.com

one possibly weird thing: the version status prints before the banner

force-output after printing the banner? I assume checking for the version actually takes longer but just gets flushed first

I tried this. Doesn't seem to help.

Actually, testing in the dockerized version does print the log after the banner. And also speeds instantiation up.

This will need porting to QVM also, FYI.

quilc-tests unhappy. maybe disable version check when calling %entry-point in the "faithfulness" test?

Weird. Running the tests via SLIME: no issue. Running via make test-quilc: process seems to hang.


Fix references to outdated 'nativeExecutables' in error messages

In 991b7ff6a7d4fcaba7c59922793acf8209dfc8fa 'nativeExecutables' was renamed to 'nativeDistributions', however a later commit added error messages that still reference the old DSL names.

I don't think the CI failures are related to these changes.


The Products that were deleted and went through Provisioning of the Deletion are still shown in both Old and New Registry.

Description:
120. The Products that were deleted and went through Provisioning of the Deletion are still shown in both Old and New Registry. Example on Prod: b88b53

After testing fix in Test Env I still can see the product that went through deletion process on UI of both Current and New Registry:

The Deletion Process ran in Provisioner and has sent a callback to the Registry App:

We had a call with Junmin to check it again and it works good. There is a double callback from the Provisioner fulfilment pipeline which overlapped marking the product deleted in database, and the logic for the old Registry didn't work well but the Project was already deleted in RequestedProejcts table so it couldn't be found by the New Registry. If the New Registry logic works first everything is displayed well in both UIs.


fixed endpoint selector for postgres service

The existing selector doesn't match the postgres-torus pod. If you run
kubectl get endpoints
NAME                  ENDPOINTS          AGE
postgres-torus        <none>             48m

The pod should be available in the ENDPOINTS
After changing  the label on the deployment pod you can see the pod
kubectl get endpoints
NAME                  ENDPOINTS          AGE
postgres-torus        <IP_ADDRESS>:5432   49m

:shipit:

:+1:


chore: some cleanup from uv branch

Pulling out a bit of cleanup, especially the better minimums.

@henryiii the pybind11 lower pin seems to have made the rpm builds not compatible. There's also wheel for F38, but that can be skipped. I have pinged the maintainers for pybind11 update. But should I add packit automation to pybind11, and should it be under the scikit-build group (the difference is mostly that all builds in nightly/release are synchronized with the main/tag releases)

Indeed the numpy 2.0 will be major release effort on Fedora, but I don't see it in the upcoming change proposals yet. But anyway for test dependencies we should lower (or eliminate the bound if you are ok with that) until we find a combination that fails. Fedora packaging is not a good metric for that though, maybe we should try packaging at least for epel9 or start with epel10 (probably 1-2 years away).

You could add it, I'd be in favor, though I'm not the only maintainer of pybind11. :) I am not sure about groups, they are separate projects but have a lot of synergy.

Got it, I'll make a quick PR there. Indeed because of the synergy it would be nice to have them tested, and currently having them in the same copr repo is the only clean way to do it

I'd like the bound because I'm hoping to test the minimums. Unfortunately, we'll probably have to list a different minimum version per Python version if we pick anything lower than 2.11 down to whatever you want as a minimum. (and Python 3.13 will require at least 2.12, if not higher).


Use Hash for $service_limits of systemd class

create_resource('systemd::service_limits', $service_limits) of systemd class (init.pp) fails unless $service_limits is set to Hash.

Maybe you're using a key that is not known in the data type? Check in https://github.com/camptocamp/puppet-systemd/blob/master/types/servicelimits.pp and add if needed.

Maybe you're using a key that is not known in the data type?

No. This is not a Data type problem. create_resource function needs Hash: https://puppet.com/docs/puppet/latest/function.html#createresources I think that $service_limits in systemd class now don't work.

https://github.com/camptocamp/puppet-systemd/pull/52 does the same thing, but also adds a test. Didn't realize you already had a PR for this.

Oh. #52 is better than my PR. I'll close this.


[AutoDiff] Properly collect inout parameter adjoints

Apparently, the parameter index calculation in #37861 was not always correct in presence of other pullback parameters (e.g. captures and non-differentiated args). Collect all inout parameters and collect inout parameter adjoints correctly.
Resolves SR-15891

Tagging @BradLarson @rxwei

Is SR-15793 related to this? It seems to stem from incorrect index calculation, and it has a nested pullback calculation involving an inout parameter. The inout resides in the inner function, unlike SR-15891 where inout is in the outer function's parameter list. I tested the bug's reproducer and this PR's current commits have not changed its behavior.

Well, if it did not change the behavior, then it is not related...

@swift-ci please test


Use the current user's SDKMAN installation

When su-ing to another user, environment variables carry over.
As a result, sdk will point to the installation of the original user,
not the one running the current shell.
Thus: Reset $SDKMAN_DIR if the directory it points at
belongs to a user other than the current one.
(Sorry, I didn't have the time to fulfill the prerequisites. I empathize, but that doesn't change my time prioritization.)

It occurs to me that SDKMAN_VERSION is similarly affected. Is there a reason against not setting it to $(cat ${SDKMAN_DIR}/var/version)?

To run a command as another user, don't use su, use sudo instead. By default, sudo will clear the current user's environment variables before starting the command.
sudo --user=<user> <cmd...>


Make stub files appear PHP code

Q
A

Type
improvement

BC Break
no

Summary
Make stub files appear PHP code here on GitHub by changing syntax highlighting.

Voilà: https://github.com/szepeviktor/xml/blob/patch-1/stubs/Psl.phpstub

Thanks!


Add animated background

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
I want to add an animated background under the background component.
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

Proceed @saurav2208


Sporadic loss of routes

It seems that routes can be lost sporadically for some time. One observed instance is when 2 nodes were connected to all 6 public nodes. At times, 1 node lost it's selected route for the other node. This node is physically close to the 2 belgian peers (< 1ms latency), in case that matters. In the other direction the subnet was not lost (or at least it is being lost a lot less).
Note that babel does have instances were valid routes exist which aren't immediately selected, ususally a seqno request is sent here.

After cross referencing logs on the public nodes, it seems no retractions are sent from there. So that means the route loss is in fact happening in the node itself

Code to send a seqno request in this case has already been added some time ago, though there is no release of this yet

After some more debugging, what seems to be happening is the following:

There is a selected route for some subnet, with a given metric.
The metric decreases as a result of an update. The route entry is updated with the new metric but no update is triggered.
A new update comes in for the route with a higher metric. The update is validated against the source table and found to be feasible.
After this, but before the route entry is updated in the route table, an update is sent out for the subnet. This uses the last recorded metric, which is lower than the announced metric in the source table. As a result, the metric in the source table decreases.
The route entry is updated as part of the update, and route selection is ran.
Due to the update we sent in the meantime, the route entry is now unfeasible, and the route is lost

All in all, the problem is that the source table is not locked for the duration of the update call. This would however create additional contention on the source table. For now we will not do anything and see how the additional seqno requests work out.


Make project naming consistent

Make project naming consistent
:recycle: Current situation & Problem
Since the project was originally created from the CardinalKit Template Application, many areas of the project are still labeled as "Template".
:bulb: Proposed solution
Changes all "Template" occurrences in the project to "HealthGPT" to reduce confusion/maintain consistency.
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[X] I agree to follow the Code of Conduct and Contributing Guidelines.

Will continue with #6.


Adding gccgo option along with IBMPOWER and IBMZ to the build should not...

... affect current gc builds.
Addresses #9207
Signed-off-by: Srini Brahmaroutu<EMAIL_ADDRESS>Since there are issues I see around gccgo, I would like to get this PR merged. It will allow build to work with gccgo selectively, I have tested on X86( with/without gccgo) , IBM Power and IBM Z systems. Please bear in mind that when enabling build with gccgo we add a flag to the build.  "make gccgo=true arch=x86 binary"

@tianon If we agree on the idea, I need to somehow preprocess Dockerfile.in before feeding into jenkins, jenkins job does not seem to call “make”
Having a Dockerfile.in or Dockerfile at the root, is minor, but only one of them should exist. I would like to see if you agree with the format of the Dockerfile and the preprocessing logic,

I'm also still not seeing why we can't include gccgo in the main Dockerfile as-is.  If we added a project/make/gccgo (or project/make/binary-gccgo), we could even be sure to be compiling with it every time to make sure that once it's working, we don't break it.

Yes I want a test, sorry there is no way we are going to add gccgo and not
test it to make sure we never break it. We need a test. +1 for a
project/make/gccgo
On Mon, Feb 23, 2015 at 5:45 PM, Tianon Gravi<EMAIL_ADDRESS>wrote:

I'm also still not seeing why we can't include gccgo in the main
Dockerfile as-is. If we added a project/make/gccgo (or
project/make/binary-gccgo), we could even be sure to be compiling with it
every time to make sure that once it's working, we don't break it.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10953#issuecomment-75683174.

Also NOT LGTM for removing the Dockerfile from being checked in

@jfrazelle  I have build this code with "make gccgo=true arch=X86 test" for building on X86, Ubuntu14.04 and all tests ran fine. I also tests on IBM Power and IBM Z.

Referencing #10990

@tianon @brahmaroutu what should we do with this one? Can we close it?

@tiborvass you can close this, we split most of the code here into several PRs.

Thanks!


User data sharing settings

Hard-coded (for now) settings:

Adult names and contact info shared with cluster
Adult photos shared by default but user configurable
Kids not shared at all
Household listing available only for own community

Cluster admin and above see all

Originally created by Tom Smyth at 2017-01-11 02:28, migrated from redmine-#5695

Description updated (diff)

By Tom Smyth at 2017-01-11 18:06

Tags set to people

By Tom Smyth at 2017-01-15 08:02

Status changed from New to Closed

By Tom Smyth at 2017-01-18 19:19

Version set to 3.0

By Tom Smyth at 2017-01-18 19:19


Update the Pull Request template to include a "Verification" section

One idea point of feedback around pull requests is that it's often not clear the level of verification expected from reviewers - should reviewers be patching all changes to be validating them, only specific ones. For new features, it's also sometimes unclear how a reviewer can functionally test things (i.e. exactly what should be tested, how to engage the scenarios, etc). One idea is to provide an optional section on the PR template where people can call out specifics they want reviewed (i.e. please patch my change and test ABC out, or please pay attention to this specific thing)

Can we provide example instructions like command line commands for how to check out the fork? I feel like people will be more likely to check out the fork if they have steps that can copy/paste

Done - I spent some time looking to see if github markdown supports variables (in that, is there some macro/variable I would use such that {PR_ID} would be auto-replaced by the actual ID of the pull request, so that people who were reviewing could literally copy paste and the commands would just work without their intervention in substituting values)
I didn't have any luck so I think this is the next best thing.

/azp run mrtk_docs


helm: Review pod toleration of Hubble Relay deployment

Currently, the hubble-relay deployment tolerates to be scheduled on any node:
https://github.com/cilium/cilium/blob/f191f9eed4b9ab1d09c4bd8a6bf2456011acea52/install/kubernetes/cilium/charts/hubble-relay/templates/deployment.yaml#L72-L73
This means that hubble-relay can end up deployed on tainted nodes (such as the master node). It is unclear as to why this toleration is needed.
We should review the current toleration and either make it less permissive (e.g. by using the default toleration), or add comment as to why it needs to be scheduled on e.g. the master node.

@gandro to give an update on this. If you have a single node cluster, where that node behaves has the master, hubble-relay won't be scheduled into that node.
In the case of cilium-agent & cilium-operator we need to make sure it's running on all nodes since it is an important part of the infrastructure so the tolerations need to be permissive as possible. In case of hubble-relay relay it will depend on important hubble-relay is to be running in the cluster. (Also in the 1 node cluster I mentioned)

@aanm Could it be something like this?
affinity:
  nodeAntiAffinity:
    preferredDuringSchedulingIgnoredDuringExecution:
    - weight: 1
      preference:
        matchExpressions:
        - key: node-role.kubernetes.io/master
          operator: DoesNotExist

That could be an option, that also works on a 1-node clusters??

@aanm That should work. https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#node-affinity The second paragraph.


Missed semicolon

In this section, the semicolon is missed at the first line
object o  = "object"
dynamic d = "dynamic";

string s1 = o; // Fails at compile-time -- no conversion exists
string s2 = d; // Compiles and succeeds at run-time
int i     = d; // Compiles but fails at run-time -- no conversion exists

[Enter feedback here]

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

ID: 7be4c24e-9e92-6b75-2b8e-2cb6cb072b5b
Version Independent ID: 450064c8-720e-3b90-18d6-7bd5f183857e
Content: Conversions - C# language specification
Content Source: spec/conversions.md
Product: dotnet-csharp
Technology: csharp-spec
GitHub Login: @dotnet-bot
Microsoft Alias: wiwagn

Thanks for noting this @dmytrostrembetskyi
I've fixed it in PRs in the Microsoft spec, and the ECMA draft standard.


dashed affects arrowhead, not just body

\begin{tikzcd}
	\bullet & \bullet
	\arrow[Rightarrow, dashed, from=1-1, to=1-2]
\end{tikzcd}

Without nfold:

With nfold:

(Also affects dotted, etc.)

Thank you for reporting this issue and the extensive testing!


Agrifood Content Validation Issue for github.io website

Section link1, link2:

Reason:
Link validation Issue: Broken links.

@rohit-ganguly, @lmazuel, @bhargav-kansagara  for notification.

@ChenxiJiang333 please help on this issue.

Close this issue as we'll put it into a summarzied one.


Keep teminal open ?

Hi,
I added a "Create Terminal" action to launch NodeJS script with path "C:\Program Files\nodejs\node.exe" and argument "app.js".
The button works and fire a terminal which launch the app (Whatever the project, I almost always call my file "app.js").
The problem is that when an application terminates (whether due to an error or manually), the terminal closes so not verry easy to read the error...
Is there a way to keep the terminal open ?

Hey,
I did some testing and I think the reason the terminal closes it's because the path, which in this case is NodeJS, acts as the shell for the terminal.
Is there a reason for you to be using the create terminal action instead of execute terminal command action?
** To use the execute terminal command, you need to have at least one active terminal. It should have at least one by default, but if for any reason you close it accidentally, you can just open a new one and everything will work just fine.
This was fixed in the new version that I'm extensively testing before releasing, since I rewrote the entire thing.
Best,
Nicollas

Hi,
Thank for the fast answer ! Of course there is a reason... no, there is no reason. I just didn't notice the existence of this action
(even though it is the one with the longest name!)... I had to go too fast, being in full discovery of the Stream Deck.
Sorry for the inconvenience and I take this opportunity to say thank you for this plugin, it will save me a lot of time!


How to use it?

I git cloned and installed analyzeMFT, but I don't know how to test it, for example, on my D:\. Can you give an example (for Windows) about how to analyze the MFT of an exisiting drive?
Thanks.

Greetings,
You need to extract the $MFT from the file system first and then point analyzeMFT at it.
-David

On Jun 23, 2017, at 2:27 PM, josephernest<EMAIL_ADDRESS>wrote:
I git cloned and installed analyzeMFT, but I don't know how to test it, for example, on my D:. Can you give an example (for Windows) about how to analyze the MFT of an exisiting drive?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dkovar/analyzeMFT/issues/49, or mute the thread https://github.com/notifications/unsubscribe-auth/AB_YUB1M1t4LEwubsfK9ryZUWcETNzwOks5sHAOagaJpZM4OD5cI.

Thanks. How to extract the MFT of my NTFS D:\ with Windows?

Hello,
Check out this article (
https://whereismydata.wordpress.com/2009/06/05/forensics-what-is-the-mft/).
There are plenty of Forensics articles out there that can help guide and
teach, but my recommendation would be to research the $MFT file itself so
that you can understand what it is and what its purpose is and THEN extract
and rip it.

Devon Ackerman

GCFA, GCFE, CFCE, CDFC, CICP, CCE
http://linkedin.com/devonackermanDefinitive DFIR Compendium Project
http://www.aboutdfir.com
Ransomware Research Project
https://goo.gl/b9R8DE
APT Groups & Operations
https://goo.gl/QEayyo

https://goo.gl/QEayyo[linkedin]
https://www.linkedin.com/in/devonackerman | [twitter]
https://twitter.com/aei4n6
On Fri, Jun 23, 2017 at 2:31 PM, josephernest<EMAIL_ADDRESS>wrote:

Thanks. How to extract the MFT of my NTFS D:\ with Windows?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dkovar/analyzeMFT/issues/49#issuecomment-310740763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXIP_ek7-CVKc119lXL8KkY0M8GYSO4gks5sHASKgaJpZM4OD5cI
.

Thanks @aei4n6 I read a few related articles. But I can imagine there exists a ready-to-use tool on Windows that can extract/display the $MFT file?

My humble recommendation would be something like AccessData's FTK Imager to
expose and scrape out the system/hidden $MFT file itself.

Devon Ackerman

GCFA, GCFE, CFCE, CDFC, CICP, CCE
http://linkedin.com/devonackermanDefinitive DFIR Compendium Project
http://www.aboutdfir.com
Ransomware Research Project
https://goo.gl/b9R8DE
APT Groups & Operations
https://goo.gl/QEayyo

https://goo.gl/QEayyo[linkedin]
https://www.linkedin.com/in/devonackerman | [twitter]
https://twitter.com/aboutdfir
On Fri, Jun 23, 2017 at 2:41 PM, josephernest<EMAIL_ADDRESS>wrote:

Thanks @aei4n6 https://github.com/aei4n6 I read a few related articles.
But I can imagine there exists a ready-to-use tool on Windows that can
extract/display the $MFT file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dkovar/analyzeMFT/issues/49#issuecomment-310743021,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXIP_a4wZmwraOOBT5Ydfnu8aotkzuNSks5sHAbIgaJpZM4OD5cI
.

I am going to close this as I have included a comprehensive USAGE.md in the directory.


Images not showing in tvOS 14?

Description
Just updated my test apple tv device to tvOS 14.0 and it seems that all  are not rendering at all?
React Native version:
System:
OS: macOS 10.15.6
CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
Memory: 138.16 MB / 32.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 10.17.0 - /usr/local/opt/node@10/bin/node
Yarn: 1.13.0 - ~/.nvm/versions/node/v8.10.0/bin/yarn
npm: 6.11.3 - /usr/local/opt/node@10/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.0, DriverKit 19.0, macOS 10.15, tvOS 14.0, watchOS 7.0
Android SDK:
API Levels: 23, 24, 26, 28, 29
Build Tools: 28.0.3, 29.0.0
System Images: android-27 | Android TV Intel x86 Atom, android-28 | Android TV Intel x86 Atom, android-29 | Google APIs Intel x86 Atom, android-Q | Android TV Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 4.0 AI-193.69<IP_ADDRESS>26763
Xcode: 12.0/12A7209 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_131 - /usr/bin/javac
Python: 2.7.13 - /Library/Frameworks/Python.framework/Versions/2.7/bin/python
npmPackages:
@react-native-community/cli: ^4.10.0 => 4.12.0
react: ^16.11.0 => 16.13.1
react-native: Not Found
react-native-tvos:  0.63.1-0
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
Run app on real apple tv device on tvOS 14.0
Expected Results
Images should render

Related to this react-native issue?
Image cannot show image in iOS 14 #29279

@olafurnsig thanks for the heads up.  I'll merge the core commit (https://github.com/facebook/react-native/commit/123423c2a9258c9af25ca9bffe1f10c42a176bf3) to the current release branch.

Fix is merged to the tvos-v0.63.1 branch, and will go out with the next release.  I've tested it with the RNTester project and verified that it works.

@dlowder-salesforce When can we expect a release to NPM? I was mistakenly pointing to the NPM 0.63.1-1 source and wondering why it wasn't working - I had to switch to https to resolve the image loading issue.

Fixed in https://github.com/react-native-tvos/react-native-tvos/releases/tag/v0.63.1-2


Running downloadTool in VS2017 agent can't find temp dir

The agent logs the following error when calling downloadTool
Directory does not exist: D:\a\_temp\d0ca8410-0ac0-4515-b4cb-ea3dd05e1dd3
My task source code can be found here:
https://github.com/cdesewell/terragrunt-azure-dev-ops

Edited your comment for clarity because github was escaping the backslashes.
It seems to be working fine for me - minimal repro and Succeeding build
Are you running anything before your task? Is it possible that they're somehow deleting the temp dir?

Hey thanks for your response. I've got it working now.

Ok awesome! I'm going to close, feel free to comment if you'd like to reopen/have other questions


No Devices found

Describe The Bug:
I have a Roomba 980 and a Braava M6 but neither are showing up when using username/password. I have double checked the credentials to show they are correct. There are no devices under the device tab and no error logs either.
To Reproduce:
Use username and password to try and connect roombas to this plugin.
Expected behavior:
Both iRobots to appear in devices tab
Logs:
No logs to show

Config:
{
            "name": "iRobot",
            "email"<EMAIL_ADDRESS>            "password": "*************",
            "roombas": [
                {
                    "autoConfig": true,
                    "info": {
                        "ver": 2
                    }
                }
            ],
            "manualDiscovery": false,
            "lowBattery": 20,
            "offAction": "pause:dock",
            "status": "cycle:clean",
            "eveStatus": "inverted:cycle:clean",
            "bin": "filter:contact",
            "ignoreMultiRoomBin": true,
            "hideStuckSensor": false,
            "disableMultiRoom": false,
            "platform": "iRobotPlatform"
        }

Node Version:
v16.13.0
NPM Version:
v8.1.0
Homebridge Version:
v1.4.0
Plugin Version:
v2.1.16
Operating System:
Raspberian

Please try installing the latest beta. It might have more info

I‘ve got the same combination of robots. With the latest stable version I am experiencing the same behavior. With the latest beta I am getting timeout errors for both all the time. They do show up in the home app but as unresponsive.

I'm noticing something similar in the latest beta. My 980 shows up and is able to be turned on and off via the home and homebridge app. However the Braava jet M6 is always unresponsive via the home app, but works in the homebridge app.
Either way since I turned on this plugin I've noticed slow down of the whole homebridge system. Other accessories taking much longer to connect to then normal.

What is the software version on your m6?

My M6 is on 3.20.7

Ok, thanks. I will look into this later today

I'm also running 3.20.7 on M6 which appears to be the latest.

Any news on this?

Please try the latest beta:


Children not receiving context on subsequent renders

Children receive context on the initial render, but not on subsequent renders.
https://github.com/trueadm/inferno/pull/358 demonstrates this behavior with a failing test

I haven't had a chance to attempt to debug this (I need to dig into the source code more), but I figured I'd might as well make an issue and PR with a failing test.
Awesome work on Inferno 😄

Fixed in dev branch (1.0)

Great, thanks! Would you mind publishing to npm?

This has been pushed to NPM as alpha10.
On Fri, Sep 16, 2016 at 11:58 PM, Ryan Tsao<EMAIL_ADDRESS>wrote:

Great, thanks! Would you mind publishing to npm?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/trueadm/inferno/issues/359#issuecomment-247727830,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcw_nJ1r0ApifSaxYnPxOxIsyfegxrxks5qqx8egaJpZM4J-ntf
.


Multiple document upload

Is your feature request related to a problem? Please describe.
Applicants aren't submitting all pages of their income documents. For example, the Parks Scholarship Program asks for applicants' 1040 tax document, pages 1 and 2. Applicants may be taking a photo of the document from their phone. But they are only able to upload one image or document at a time.
As a work-around, we ask applicants "Do you have additional pages to upload?" and if "YES" an additional document upload question is visible. To encourage applicants to upload all necessary income pages, it would be helpful if CiviForm allowed for:

Multiple uploads on one screen (currently, we can only do one document upload question per screen)
Allow multiple documents to be uploaded. For example, should be able to upload multiple photographs for different pages of an income document. (currently, applicants can only select one document to upload at a time)
Allow for drag-and-drop

What priority is this?
High priority--it's causing delays for applicants and program staff. The work-around is to email, mail or fax in the income documents, which is much less secure and leads to delays in processing time.
Associated civic entity
City of Seattle -- Applicants, Program Admins, TIs
Done when:
Applicants can upload multiple documents on the same screen or multiple documents on the same upload question.

To review with Product Design committee @isabelcshaw and @msprenke

Hi! Agreed this is a problem (and looks like others have felt the same: https://github.com/civiform/civiform/issues/5591).
I've worked with the file upload question before, so I have a bit of background on how it works and why it doesn't support multiple files right now -- essentially, when you click "Save & next", AWS only accepts one file and nothing else.
I have a quick follow-up question: Do you always know how many pages an applicant will need to upload, or does it vary per applicant?
For this specific program, it sounds like there's always a page 1 and optionally a page 2. I'm wondering if you could skip the "Do you have additional pages to upload?" question, and instead have one required file upload question and then a second optional file upload question. That way, applicants will at least see the "Upload the second page to the document if you have it" question and be less likely to not upload it.
Again, I don't think this workaround is great (especially if you may need more than 2 pages) and I think it'd be great to have the file upload question allow multiple files at once -- just proposing a workaround in case they unblock you for the moment.
(Also we can chat more in Product Design committee, don't feel obligated to reply to this comment!)

@caitlinshk, responding to your questions here so we have it all in the same place. We do not always know how many pages/documents an applicant may need to upload for a particular upload question. For example, we may ask an applicant to show their rental income and prove that with a copy of the lease agreement. I personally have had all manner of different sized lease agreements in my life. I like your work around suggestion. We are currently attempting to do this with our Weatherization program. Here is a snap shot of what that looks like. The document upload section of this program is currently 36 screens long. However, it is unlikely that an applicant will have all 8 types of income and need to click past each screen.

I believe this is being actively worked? moving to in progress.  Please correct if that is not the case

Just confirming that this is being worked on and close to complete. To summarize the behavior we've currently landed on:
All file upload questions will accept multiple files (no follow up questions required, by default, and admins can set a maximum for the question if necessary.


Returning functions in place

After seeing issue #67, was curious as to what people think as to adding this capability to all functions?
Some of them, like df.limit_column_characters(), already return in place. I don't think it will be hard to extend to others.

:thinking: :thinking: :thinking:
This one is a tough one for me to respond to.
On one hand, I can see why that might be desirable by some users who are used to the inplace=True kwarg.
On the other hand, I think adding it in as an option means an additional thing to check for when any new function is submitted in. Conceptually, it's "just one less thing" to maintain if we just leave that argument out.
With a fluent API, I don't think in-place modifications are a thing.
That said, I'm willing to be persuaded by a counter-example to my general slant against in-place modification of a dataframe, if one can provide an example where in-place modification accomplishes something substantial that a method chaining interface cannot accomplish.

I’m actually more inclined have the default be to return functions in place this is for a few reasons:

It doesn’t change the method chaning syntax
If people want to apply separate functions outside of the initial janitor section it’s merely ‘df.add_column()’ rather than ‘df=df.add_column’. I think the former is cleaner.
Since Pandas is used for a variety of reasons, with EDA being a big one, it could be a big problem to change dataframes in place since one’s work could be inadvertently clobbered. My assumption is that the janitor functions should run once at the beginning and then later people will work with Pandas after the data is cleaned. If someone did clobber a dataframe, I would assume he would just “start over”

@szuckerman, let me see if I can clarify my understanding of the definition of in-place vs. method chaining.
Taking the example from the README, regular pandas syntax looks as follows:
df = pd.DataFrame(...)  # create a pandas DataFrame somehow.
del df['column1']  # delete a column from the dataframe.
df = df.dropna(subset=['column2', 'column3'])  # drop rows that have empty values in column 2 and 3.
df = df.rename({'column2': 'unicorns', 'column3': 'dragons'})  # rename column2 and column3
df['newcolumn'] = ['iterable', 'of', 'items']  # add a new column.

First off, with in-place modifications at each line, we do:
df = pd.DataFrame(...)
df.add_column()

That being the case, some of the functions already work in place.

@szuckerman yes, that's a valid concern, especially for consistency purposes.
I think the hack that we've implicitly used, but not explicitly stated, is to piggy back off pandas operations. If modification takes place in-place anyways (to prevent memory copying), then it happens in-place; if not, then we don't really care. For a pareto-distributed set of users, 80% will have datasets that fit in-memory, while 20% will have @zbarry's kind of needs, where dataframes are too large for a single computer's RAM.
At the moment, I don't think we're necessarily at a point where too many people will complain about the memory-usage architecture of pyjanitor. Perhaps when a few related issues come together, then it might be an issue? (I will defer to your experience, @szuckerman, as the more seasoned software engineer here, as to whether we should enforce consistency w.r.t. memory usage at this early point in the project.)

@zbarry makes a good point that "since you can't get the intermediate dataframes out for each step anyway" returning in place is more performant.
However, after doing some research it doesn't look like returning objects in place makes too much of a difference. I think the method-chaining syntax saves most people from the concern of making mistakes (i.e. by accidentally running a method that works in place by default), as given as one of the examples on stackoverflow.
As for future considerations of the package, I think that to really determine whether methods should default to being in place or not depends on "usability". Will users be confused when dataframes are mutated in place?  Later we can run tests, profile memory, etc. to determine if there's a difference between a method and its in place counterpart.

Yeah, usability is probably core to the idea of pyjanitor even. That being said, I think having a (short) paragraph in the readme + in the notebook example I will (eventually) create as well as having a clear boilerplate message in all the function docs would convey this pretty well. If you make it such that all functions that are relevant w.r.t. this discussion operate on the same principle of in place, I think this gives a clear expection to users as to what approach pyjanitor takes for operations on DataFrames (that being that they should expect mutation).
Because I'm procrastinating, I ran a simple speed test for adding a column to a DataFrame with pandas' .assign vs. using the df[my_col] = data explicitly (which is what pyjanitors .add_column does):

The slowdown with the data copy in this particular operation is a pretty consistent 1.5x. This could definetely add up when users chain multiple operations (I think it's possible that some ops could have even worse slowdown multipliers).

Slowdown multiplier when adding to a DataFrame that has multiple columns already is pretty horrible:

@zbarry those are great tests. Yes, I think it's worth adding to the docstrings (but without user warnings popping up - those are too easily abused) that for the relevant functions, underneath the hood, we do change the original dataframe in place for performance reasons.

without user warnings popping up

Hmm... I was thinking of adding a pyqt5 dependency & popping up a message box warning you upon pyjanitor import of the mutation operations. Shame.

Random idea: what about having colorful badges in the docs that you can see at a glance whether there is mutation / something else critical to know that a function does to your dataframe?
Like:

kind of stuff but with appropriate labels.


Form property is not set to $dirty until the user has entered a valid credit card number

If the user enters bad input and/or the tabs off the credit card field the $dirty flag is not set on the form object.
i.e. formname.field name.$dirty is not set to true until a valid input has been received.
Can be reproduced by using the try it page: http://embed.plnkr.co/uE47aZ/preview and entering in anything like 4242 and tabbing off.
Can there be an option to allow invalid input to cause the form side of things to update?
<input type="text" class="input-text" id="cardNumber" data-cc-number data-cc-eager-type="payment.creditCardType" name="cardNumber" data-ng-model-options="{ allowInvalid: true }" data-ng-model="payment.creditCardNumber">

    Bad credit card #

The ngModelController is updated but the value in the view doesn't kick in until after the first time the user has entered a valid cc number.  This is happening even with allowInvalid the model options.

I can't reproduce this either. Entering 4242 into the card number results in ng-dirty being applied to both the input and the form. Both of those are set when a view value update is committed.
Any ideas on why all these weird issues are popping up for you?

In the example above paymentForm.cardNumber.$dirty never gets set to true if you start typing?


SQL value with "xx"  parse error when use encrypt

Which version of ShardingSphere did you use?
4.0.0-RC3
Which project did you use? Sharding-JDBC or Sharding-Proxy?
sharding-jdbc
Expected behavior
insert error
Actual behavior

Reason analyze (If you can)
parse error for sql like this:
String sql = "insert into fix_test(id,name,password) values (4,"you.wang@xxx.com",'123456')";

Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

@xiyelife String sql = "insert into fix_test(id,name,password) values (4,"you.wang@xxx.com",'123456')"; SQL you provided is wrong.
You can try to modify INSERT_GENERATED_KEY_SQL of EncryptStatementTest to "INSERT INTO t_encrypt(pwd) VALUES ('\"you.wang@xxx.com\"')" , and run the test method assertInsertWithExecuteWithGeneratedKey. This works well.

@SteNicholas the sql above has been formated by github, the sql is real like this in the code (idea)：

I think this writing should be right


[PDI-13353] - When having duplicate DB Connection names (difference in capitals) PDI defaults to the first

handle NULL returned from the dialog
trim connection's names
add tests

@mattyb149, @brosander, review it please

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
ui/src/org/pentaho/di/ui/repository/repositoryexplorer/controllers/ConnectionsController.java
ui/test-src/org/pentaho/di/ui/repository/repositoryexplorer/controllers/ConnectionsControllerTest.java
Checkstyle violations:
No errors found
License header violations:
No violations found
Unit test coverage change:
org.pentaho.di.ui.repository.repositoryexplorer.controllers.ConnectionsController:

instructionPercentCoverageChange : +0.72%
branchPercentCoverageChange : +2.35%
linePercentCoverageChange : +1.57%
complexityPercentCoverageChange : +0.94%


Add Source : KvSource

It provides connection to many KV Store like Consul, Etcd or Zookeeper : using libkv and  mapstructure packages to loas data into the struct
The whole configuration structure is stored, using architecture like this pattern :

Key : <prefix1>/<prefix2>/.../<fieldNameLevel1>/<fieldNameLevel2>/.../<fieldName>
Value : <value>

It handels :

All mapstructure features(bool, int, ... , Squashed Embeded Sub struct, Pointer).
Maps with pattern : .../<MapFieldName>/<mapKey> -> <mapValue> (Struct as key not supported)
Slices (and Arraies) with pattern : .../<SliceFieldName>/<SliceIndex> -> <value>

Coverage decreased (-8.3%) to 78.81% when pulling 1fd4df0667fd4c18223d7773268e2a5a500944ec on addKV into e2aa88e235a02dd52aa1d5d9de75f9d9139d1602 on master.

Coverage decreased (-5.4%) to 81.784% when pulling 76b710925d97d4ed304f23842d44dffeca07d5d3 on addKV into e2aa88e235a02dd52aa1d5d9de75f9d9139d1602 on master.


Error when training fastspeech2

After training some iteration, i get this

error:tensorflow.python.framework.errors_impl.InvalidArgumentError: 2 root error(s) found.
(0) Invalid argument:  ConcatOp : Dimensions of inputs should match: shape[0] = [2,1139,384] vs. shape[1] = [1,1140,384]
[[{{node tf_fast_speech2/length_regulator/while/body/_1/concat}}]]
[[tf_fast_speech2/mul_2/_160]]
(1) Invalid argument:  ConcatOp : Dimensions of inputs should match: shape[0] = [2,1139,384] vs. shape[1] = [1,1140,384]
[[{{node tf_fast_speech2/length_regulator/while/body/_1/concat}}]]
0 successful operations.
0 derived errors ignored. [Op:__inference__one_step_fastspeech2_131786]

Anyone can help me to fix this error?! Thanks!

Following my guess, your dataset has some samples with total duration different from mel length, so when repeating it up, it does not match real Mel-spectrogram. You should check over your dataset and ensure that all samples have total duration equal to mel length.