plans to add to nbextensions

Just curious: any plans to add jupyter-notify to nbextensions?

Interesting!  I didn't know about nbextensions.  I'll take a look and see what that would involve.  Thanks!

After looking, it's not clear to me that pure python magics like this one are supported by nbextensions.  Those look to be javascript extensions.  I've filed an issue on that repo to ask though...


Cannot pull 'docker' image

Hi, we have problems pulling the docker image from docker hub. The problem first occurred yesterday.
docker pull docker:dind
Trying to pull repository docker.io/library/docker ...
manifest for docker.io/docker:dind not found

I also tried different tags. Pulling other images works fine...

Client:
Version:         1.13.1
API version:     1.26
Package version: docker-1.13.1-209.git7d71120.el7.centos.x86_64
Go version:      go1.10.3
Git commit:      7d71120/1.13.1
Built:           Wed Mar  2 15:25:43 2022
OS/Arch:         linux/amd64
Maybe the build docker version changed? https://stackoverflow.com/questions/75384648/pull-from-docker-hub-returns-not-found-with-older-docker-version

Oof, yeah, maybe -- Docker 1.13 has been EOL for a very long time. :see_no_evil:

For reference, this is probably because the images we publish are now OCI images: https://explore.ggcr.dev/?image=docker:dind (and I don't recall whether Docker 1.13 supported those yet or not, but it looks like maybe not)


Runtime specific carbon.pid

Currently carbon.pid is created at <CARBON_HOME> location, but this is an issue when there are multiple runtimes and all start at the same time. This needs to be changed so that the pid file is specific for each runtime.

Thusitha is now working on this feature.


NVSHAS-6703 For 5.0.0, with Antrea CNI, traffic between pods within the same node see source as unmanaged workload

Already cherry picked.


No script or notebook to train ddpg from scratch.

Why there is no script provided to train the dppg model just like you have provided imitation_learning.py for training imitation learning algorithm?
It would be helpful to replicate the results and experimentations.

Please look at this file stock_trading.py
Also, please be highly suspected of the results you got. I actually extend the project by adding some news features and found that stock market can't be predicted/acted using simple algorithms like RL or simple language model.

Thanks. I am closing the issue.

The stock_trading.py file throws an error.
Traceback (most recent call last):   File "C:\Users\hp\Envs\rl-portfolio\lib\site-packages\gym\core.py", line 203, in __del__     self.close()   File "C:\Users\hp\Envs\rl-portfolio\lib\site-packages\gym\core.py", line 164, in close     self.render(close=True)   File "E:\Pirimid\drl-portfolio-management\src\environment\portfolio.py", line 298, in render     return self._render(mode='human', close=False)   File "E:\Pirimid\drl-portfolio-management\src\environment\portfolio.py", line 295, in _render     self.plot()   File "E:\Pirimid\drl-portfolio-management\src\environment\portfolio.py", line 302, in plot     df_info = pd.DataFrame(self.infos) AttributeError: 'PortfolioEnv' object has no attribute 'infos'

If you guys can provide the requirement.txt for the code then also it would be really helpful. The conflict between the packages is really big issues in Python.

Yes. I notice some non-compatible updates on openai gym environment. I will try to make a requirement tomorrow if I have time.

I solved the error by removing dtype=float32 from two lines in the PortfolioEnv. The new lines are as follows.
`self.action_space = gym.spaces.Box(0, 1, shape=(len(self.src.asset_names) + 1,))  # include cash
get the observation space from the data min and max
self.observation_space = gym.spaces.Box(low=-np.inf, high=np.inf, shape=(len(abbreviation), window_length, history.shape[-1]))`
I have gym==0.9.3 on my machine.


Update Correct command in README.md

Correction on readme page regarding CLI command for viewing syslog logs via SysmonLogView is required.
The current provided code mentions:
sudo tail -f /var/log/syslog | /opt/sysmon/sysmonLogView
But we will get the output using sudo tail -f /var/log/syslog | sudo /opt/sysmon/sysmonLogView
Hope this helps!

Hi,
I have added another commit in the same folder for adding -y flag to dependencies installation.


Multi-level mask and wildcard in event

Test case
var server = new em.EventEmitter2({wildcard: true, delimiter: '::'})
function test (value1, value2) { console.log('e0', this.event); }
server.on('A::**::C', test);
server.emit('A::*') // not called
server.emit('A::C') // called

According documentation I think it should calls both listener.

You are subscribing to A::**::C, and there is no C in the first event name

But A::* means match any and ** allows match zero level A::**::C so
server.on('A::C',     function() { console.log('A::C Match', this.event); });
server.on('A::**::C', function() { console.log('A::C Match', this.event); });
server.emit('A::*')     // match to 'A::C'
server.emit('A::C')    // match to 'A::C' and to `A::**::C`

But I really do not use this library and it is not big problem because I do not think this have some real usecase. I just try figure out how it should works to be able implement EventEmitter on Lua language. In fact I found similar problem with
server.on('A::**', test);
server.emit('A::*')       // called
server.emit('A::*::B')    // not called

You're right.
I don't really understand the basis of allowing wildcards in emit though

Hm ... don't see any point too...
Better to close to not confuse other developers.
Only listeners uses wildcard to subscribe.
This is definitely not the publisher responsibility to use wildcard for defining the group of subscribers.


[Help Requst]Imcompatibale version with argoverse-api

Hi,
I followed the "Read Data from Dataset" chapter as the doc described. And found that the argoverse-api requires numpy==1.19 but metadrive will install numpy=1.24.2. I set numpy==1.24.2 in argoverse-api setup.py, but still failed to run python -m metadrive.examples.drive_in_argoverse:
/usr/bin/python: No module named metadrive.examples.drive_in_argoverse
Is there any solution to this problem?

Sorry, Argoverse-related API and environment have been deprecated for quite a long time. However, ScenarioNet is going to support Argoverse soon! We expect to release Argoverse convertor before the end of this month.

Thanks for your reply!


[YUNIKORN-1355] Generic example of GPU scheduling with Yunikorn

Issue
Add a generic example of GPU scheduling with Yunikorn.
Link
https://issues.apache.org/jira/browse/YUNIKORN-1355?filter=-1
Solve
Test the GPU scheduling.
Update Yunikorn site.

In other words, whether time slicing is enabled for GPUs shouldn't matter from a resource scheduling perspective, right?

That is correct this just brings the NVIDIA setup and the YuniKorn bits together.


Upload desktop integration artifacts to GCS

This creates a python script to upload artifacts to GCS, and adds a step to the integration test workflow to run it.
We can't expose these artifacts in Github workflows, as that would expose secrets: in order to run a desktop integration test it's necessary to have the decrypted google-services.json file. This script thus uploads desktop artifacts (the integration test binaries, and the google services json) to GCS instead.
To avoid duplication with the existing test_lab.py tool, common functionality has been moved into the new library file integration_testing/gcs.py
At the end of the workflow step to upload artifacts, it will report this:
Use 'gsutil cp <gs_uri> <local_path> to copy an artifact locally.
Use 'gsutil -m cp -r gs://games-auto-release-testing/201013-231404_ckjT <local_path> to copy everything.
This can be used to download artifacts for local testing/inspection. This requires the cloud sdk to be installed locally, to use the gsutil tool.

Consider adapting this presubmit test - https://github.com/firebase/firebase-ios-sdk/blob/master/scripts/check_copyright.sh

Yep, it's on our list of things to do!

Well that's new:
NDK_ROOT environment variable is not set, or NDK version is incorrect.
This build requires NDK r16b for STLPort compatibility, downloading...

curl -LSs https://dl.google.com/android/repository/android-ndk-r16b-windows-x86_64.zip --output /tmp/android-ndk-r16b.zip
curl: (16) Error in the HTTP2 framing layer
Error: Process completed with exit code 16.


Welcome screen: iFrame value should take precedence over custom HTML

In Umbraco 7.5.4 saved custom HTML & iFrame value for - the custom HTML should not be shown

Workaround: clear the custom HTML.


Pass allowed_mentions from the bot into Webhook.send

Summary
When using Webhook.send(), use the allowed_mentions set in the Bot
What is the feature request for?
The core library
The Problem
When using Webhook.send(), the webhook is using default AllowedMentions, which, with the change that webhook's permissions are calculated using the respective bot's permissions, can cause issues.
The Ideal Solution
Pass the allowed_mentions property from the bot
The Current Solution
Use the allowed_mentions kwarg
Additional Context
No response

Webhooks that aren't manually created (i.e. those from webhooks, fetch_webhook, create_webhook, and things like interaction.followup) already use the client's allowed_mentions, so this shouldn't be an issue there.
The ones that don't inherit it are Webhook.partial and Webhook.from_url, since they are mostly detached from the client/bot system and are intended to also work outside of bots; those don't have any way of accessing the client's allowed_mentions.
If you want to use allowed_mentions with those, you'll need to pass it directly into webhook.send(), as you mentioned ^^
I'm closing this as working as intended, but feel free to comment if there's anything else.


Update Win32 metadata version from 24.0.1 to 30.0.12

This update includes a rollup of fixes and improvements to the Win32 metadata.
More information about what's changed: https://github.com/microsoft/win32metadata/releases

@riverar & @glandium I will follow up with updates to support new calling convention metadata and publish new libs.


Abusive memory usage after high traffic multiprocess gunicorn

After doing some tests w/ patator to try and guess paths on an internal app, we encountered an abusive memory usage case which was not obvious. This is kind of related to: https://github.com/prometheus/client_python/issues/275
A 4GB docker was oomkilling its processes after a DoS phase, those processes were respawned by gunicorn, who was reading at boot time the .db files, triggering a 200Mb/s disk read at each bootstrap, all those bootstraps were triggering oomkills, with a resulting endless oom loop.
I discarded any incoming traffic, isolated the docker and saw the exact same thing. So, a strace on the active pids were showing a lot of read, I looked at the opened files and saw those .db files. After a brutal deletion of those files, I finally got the container to settle down to a normal CPU/RAM usage (forgot to mention that the CPU was stuck @500%, seems coherent with the db load).
I finally took a screenshot of the resulting variation of both CPU and RAM on ctop:

Some tech details:
prometheus_client==0.0.21
gunicorn==19.8.1

Your description doesn't sound right, the only time all of these files are read is when a /metric scrape is performed. Are you sure you had no incoming traffic?
The DoS takes about 2khits/s for ~ 3min and workers are logging in a loop:
2018-06-13 16:05:37 [644:gunicorn.error:271] [INFO] Booting worker with pid:

a strace gives me:
mmap(NULL, 262144, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f47ee7bd000

a lot of reads
and cpu/ram usage are looking like this on that inactive app:

and after a deletion of those .db files:

So, I don't know if the crashloop occurs on a scrape (which could makes sense) or a worker bootstrap, but it occurs after a short high traffic period.

It sounds to me like some other part of your code is likely to blame here. To verify, disable multiprocess mode by not providing the environment variable.

We anticipated that answer a bit and tried to run the app without prometheus exporter → the bug is not present anymore and I can hit the app quite relentlessly without any notable consequences.

We anticipated that answer a bit and tried to run the app without prometheus exporter

That helps, but is not the question I asked. What happens if you disable multiprocess mode?
 To have 32MB for the histogram for one process is not easy. I smell a cardinality issue.

It turns out that this was a cardinality issue on a path label.


isSelected always false

[self.indexPathsForSelectedItems containsObject:self.assets[indexPath.row]]; always return false. how can i check indexPathsForSelectedItems contains object?

selected a new asset  [self.indexPathsForSelectedItems containsObject:] return false. because the asset not in self.indexPathsForSelectedItems; but you unselected the asset, the asset  already exist in self.indexPathsForSelectedItems , return true.


Convert project to use ARC

This Apple document should help: Transitioning to ARC Release Notes.

When this is done the "-fobjc-arc" compiler flag can be removed from all QuincyKit files. The flag was introduced in commit 2fa4e43 after the update to QuincyKit 3.0.

Xcode has an assistant that might be helpful: Edit > Convert > To Objective-C ARC


Update readme.md

This one is outdated now, and the changes has gone in with other PR:s.
Thanks for opening the PR, much appreciated.


Questions for DINOv2

Hello,how does your code reduce features dim to 512,1024 after importing DINOv2? Gem pooling or adding a linear layer?

Hello, we don't use the linear layer to reduce the feature dimension. GeM pooling is used to aggregate all 768-D patch features in each image region into a 768-D vector. We concatenate 14 region (GeM) features and get 14*768-D image representation. PCA is optional for dimension reduction (to 512, 1024, or 4096, etc).


GMAIL SMTP CONFIGURATION

Hi Anton,
is this confuration in traccar.xml or default.xml?
Gmail is one of the most popular free email services. To configure Gmail for notifications in Traccar follow steps below.
Assuming that you are using two factor authentication for your Google account, first you have to create an app password:
Go to your Google Account.
On the left navigation panel, choose Security.
On the "Signing in to Google" panel, choose App Passwords. If you don’t see this option check Google support page with more details.
At the bottom, choose Select app and choose Mail.
Choose Select device and choose Other. Enter "Traccar" as a name.
Choose Generate.
The App Password is the 16-character code in the yellow bar on your device. Copy it into the configuration below.
Choose Done.
Add following lines to the Traccar configuration file:
<entry key='mail.smtp.port'>587</entry>
<entry key='mail.smtp.starttls.enable'>true</entry>
<entry key='mail.smtp.host'>smtp.gmail.com</entry>
<entry<EMAIL_ADDRESS><entry key='mail.smtp.auth'>true</entry>
<entry<EMAIL_ADDRESS><entry key='mail.smtp.password'>YourPassword</entry>

You have to replace following items in the above configuration:
<EMAIL_ADDRESS>- your Gmail address. It will be used as a sender address
YourPassword - App Password that we created in the first part. Don't include spaces
Note that recepient address is your Traccar user email address. Default "admin" is not a valid address, so if you are using default account, you need to update email address to be a valid one.
thanks for the info!

For questions please use the forum. GitHub is not the right place to ask this.

Also I don't see the reason for copy-pasting documentation. Why not just provide a link?

Also, please don't use UPPER CASE letters in title.


ScrollBehavior doesn't need to be parameterised. It is always one dimensional and always uses doubles.

ScrollBehavior2 isn't.


Fixed restoring state-db journals on startup

Currently when a block N is canonicalized, all discarded branches are removed from the canonicalization overlay immediately (unless pinned). They are also removed from the DB. The may lead to a situation, when out of two journals, for block N+1 and N+1', the former is discarded and the latter remains in the DB.
On startup the journals are loaded by trying DB keys for each block number, increasing the journal sub-index starting from 0. So if sub-index 0 is missing and sub-index 1 is there, it won't be loaded.
On kusama/polkadot this has low chance of happening, because forks of more than 1 block in length are rare and the node needs to be stopped/restarted on such a fork for the issue to happen. On rococo forks are more likely due to increased finality window and there are a lot of restart because of staling issues, which help finding this issue.
I've considered a fix that keeps track of journal counts for each block number, or involves organising journals in a linked list. Ultimately this requires updating an extra database key for each commit. So I've opted for a solution that tries a limited number of possible journals on startup. This very slightly increases startup time, but requires no extra writes when running.

The code change looks good to me.
The overhead on launch indeed doesn't seems like an issue.
I am more wondering about the new limit of 32 simultaneous fork at a given block height (is it 'D3-trivial'?).
Maybe not an issue with babe and stalled finalization (2^5), but it limits substrate possible use-cases (it may already be).

I was not thinking of babe producing 32 blocks at once (astronomical indeed), but babe producing five time 2 block over a period without finalization (finalization stalling for a quiet long time).


Invalid namspace is suggested

/cc @vasily-kirichenko

not really sure what's happening here, but it doesn't look correct ;-)

I've stared at it for some time and haven't got what's happening here either. What do you expect it to do (or not to do)?

@vasily-kirichenko Looks like @forki is trying to use this: https://github.com/fsprojects/Paket/blob/master/src/Paket.Core/PackageResolver.fs#L77
Which is in Paket.PackageResolver, but the IDE is suggesting PackageResolveer.Paket.PackageResolver.  Looks like the correct namespace is getting appended here.

Yep I think it's suggesting the current namespace.
Am 26.12.2016 17:27 schrieb "Phillip Carter"<EMAIL_ADDRESS>
@vasily-kirichenko https://github.com/vasily-kirichenko Looks like
@forki https://github.com/forki is trying to use this:
https://github.com/fsprojects/Paket/blob/master/src/Paket.
Core/PackageResolver.fs#L77
Which is in Paket.PackageResolver, but the IDE is suggesting
PackageResolveer.Paket.PackageResolver. Looks like the correct namespace
is getting appended here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/2108#issuecomment-269224712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNIcyY4WnNvtf04MaLswtINw8vQlLks5rL-r_gaJpZM4LVvKy
.

@Krzysztof-Cieslak do you see the same thing happening in ionide?

Fixed in https://github.com/Microsoft/visualfsharp/pull/2102


Separate license file?

The license is embedded in the README, can we please move it to a separate license file?

Done as part of #19, which provides a separate LICENSE file in addition to what is shown in the README.


Fixed not using hidden_activation param

self.hidden_activation was not  properly being set to hidden_activation

Thank you for this and my apologies for overlooking it until now!


Simplify build

The rules.mk file included in the sequence_transform directory now automatically calls the generator script on every build.
This removes a few lines that the user needed to paste into their rules.mk file, including one line that required a tab, which is any easy source of errors.
Only drawback is that the user cannot use the provided rules.mk file if they don't want the generator to run automatically for some reason, but I don't think anyone will actually have a use case for that.
I have additionally created a json schema file for the config file, and removed some code that is unneeded now that rules search is incompatible with the current version of the library.

I have additionally added support for using multiple rules dictionary files. These multiple files are simply recombined before processing, so they must use the same symbols. The primary use case is to separate out rules that contain personal information into a separate private dictionary that you don't push up to GitHub.
Also, the config sample file in the generator folder now acts as a default config. Users no longer need to specify every setting in their config file (only the ones they want to override), and a user can get started without creating any config file at all.
The README needs to be updated to reflect the new simplified setup.

And also you have 2 .mk files rn, rules.mk and rules_auto_generate.mk, while the first one is not used at all

and I also don't really like the name for the new .mk file, just rules.mk would be better imo


Profiles don't sync views (secondary sidebar)

Does this issue occur when all extensions are disabled?: not tested

VS Code Version: 1.77.3
OS Version: Windows 10

Steps to Reproduce:

drag something to the secondary sidebar
switch profile
close secondary sidebar
switch back to the previous profile
secondary sidebar isn't visible

I'd assume that my view is saved to the profile. If I switch to another profile, I don't want to see the secondary sidebar.
If I switch then back to the profile, I want to see the secondary sidebar.

After step 5 I opened the secondary sidebar and I see the view/content. Just to be sure, my issue is that I must open the secondary sidebar manually. I'd assume, after I switch back to this profile, the secondary sidebar is automatically opened.
Generally spoken, I'd like to have my appearance to be saved in my profile.

This is because the expansion state of a view / sidebar is not scoped to profile. It is scoped to workspace/folder you have opened. Hence closing this as designed
CC @sbatten


Fix links in release.md

This file is auto generated from k/community, I've checked it's already in sync.
The first link https://github.com/kubernetes/community/blob/master/contributors/guide/issue-triage.md is located at https://k8s.dev/docs/guide/issue-triage/.
The second one https://github.com/kubernetes/community/blob/master/contributors/devel/sig-release/cherry-picks.md do not have webpage version.
The last one https://github.com/kubernetes/community/blob/master/sig-list.md have aliases /groups and /sigs, so I pick the close one https://k8s.dev/sigs.

The updated links work, thanks for fixing the 404s
Preview of updated page:
https://deploy-preview-34719--kubernetes-io-main-staging.netlify.app/releases/release/
/approve

/lgtm


Refactor the scheduler test because it's flaky

Smells:
scheduler_test.go:144:146
	// Wait some time to pickup work and finish.
	// We have to wait at least 3 seconds for scheduler tick interval.
	time.Sleep(5 * time.Second)

This just failed with the error that the job is in an invalid status. I think this should be fortified or it will cause problems in the future. The scheduler needs to be tested in a different way, or manually kicked off at this stage, or somehow signal on a channel that it's done with its round.

Ah I see. I will refactor that 🤗


Direct Line channel - SendActivityAsync receiving 503 Service Unavailable

Using the C# SDK we are calling:
await turnContext.SendActivityAsync(result).ConfigureAwait(false);

Since this morning (roughly 7am UTC) we have started receiving a 503 Service Unavailable message when sending messages. It happens often, but not all the time. This screenshot from App Insights shows the failure:

Sending a message again after the failure succeeded.
Is there an issue with Direct Line at the moment? I've also logged a ticket with Azure support to investigate.
Thanks.

Same here
Request URL:https://directline.botframework.com/v3/directline/conversations/C46gOO3jIbO4QZl4E0cp51-h/upload?userId=1ac40575-3df6-4bc3-9474-46a83d67ee1a
Request method:POST
Remote address:<IP_ADDRESS>:8080
Status code: 503 Service Unavailable
Version:HTTP/2.0
Referrer Policy:no-referrer-when-downgrade

Just an update for anyone having the same issue.
I've had a reply from Azure support. They have said that the Direct Line channel is having a region outage at the moment and they are working on a fix.

Azure support have confirmed that the isse has been resolced. Closing it here too.
Thanks!


[SPARK-37287][SQL] Pull out dynamic partition and bucket sort from FileFormatWriter

What changes were proposed in this pull request?
FileFormatWriter.write is used by all V1 write commands including data source and hive tables. Depending on dynamic partitions, bucketed, and sort columns in the V1 write command, FileFormatWriter can add a physical sort on top of the query plan which is not visible from plan directly.
This PR (based on https://github.com/apache/spark/pull/34568) intends to pull out the physical sort added by FileFormatWriter into logical planning. It adds a new logical rule V1Writes to add logical Sort operators based on the required ordering of a V1 write command. This behavior can be controlled by the new config spark.sql.optimizer.plannedWrite.enabled (default: true).
Why are the changes needed?

Improve observability of V1 write, and unify the logic of V1 and V2 write commands.
Does this PR introduce any user-facing change?

No
How was this patch tested?

New unit tests.

Thanks @cloud-fan and @allisonwang-db for pushing on this. This is great!
I will work on supporting required partitioning for V1 in this week (https://issues.apache.org/jira/browse/SPARK-37287). The motivation is to support shuffling on bucket columns when writing Hive bucket table. cc @cloud-fan, @allisonwang-db and @ulysses-you FYI.

So there was an optimization in https://github.com/apache/spark/pull/32198 which can avoid local sort if there are only a small set of partition/bucket values.
Is there optimization gone after the changes in this PR?

That optimization is off by default. When it's turned on, we skip planned write.

Sorry all. After checking the reported use case once more, I found that that it's a false alarm. I closed the issue as Not A Problem.


KARAF-4813, RMI should not listen to all hosts

The default RMI configuration will make Karaf listen to all hosts.
Instead. it is usually good practice to, by default, listen to localhost
only to avoid possible security risks.
This patch modified the default configuration file to set RMI to
localhost.

R: @jbonofre


Readme.md links undefined

In some cases, the replaced links in the readme.md are undefined, see
http://www.rdocumentation.org/packages/dplyr/versions/0.5.0#key-data-structures

@HannesBuseyne can you take a look ?

closed by 80cb24e7cf5ea2764ec17b883dead1a52a15e24b


Http thing

Description of changes

Issue Resolved

Fixes Security vuln related to cookies.

We'll need to set cookies from the relevant responses as well.


staking command fails with 'Upgrade the Helium Ledger App to use validator commands'

Hi Community,
When I try to stake using the command:
./helium-ledger-cli validator stake 10000 ACTUALADDRESS
I get this strange error:
thread 'main' panicked at 'Upgrade the Helium Ledger App to use validator commands', src/txns/validator/mod.rs:25:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

My ledger app is already v1.0.0, don't see any newer ones via Ledger Live interface. Any thoughts would be greatly appreciated.

Hi Community,
When I try to stake using the command:
./helium-ledger-cli validator stake 10000 ACTUALADDRESS
I get this strange error:
thread 'main' panicked at 'Upgrade the Helium Ledger App to use validator commands', src/txns/validator/mod.rs:25:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

My ledger app is already v1.0.0, don't see any newer ones via Ledger Live interface. Any thoughts would be greatly appreciated.

You have to update your Ledger firmware and then update the Helium App Firmware to the latest version

Hi Community,
When I try to stake using the command:
./helium-ledger-cli validator stake 10000 ACTUALADDRESS
I get this strange error:
thread 'main' panicked at 'Upgrade the Helium Ledger App to use validator commands', src/txns/validator/mod.rs:25:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

My ledger app is already v1.0.0, don't see any newer ones via Ledger Live interface. Any thoughts would be greatly appreciated.

You have to update your Ledger firmware and then update the Helium App Firmware to the latest version

You are right, thank you very much!


console appender broken for DEBUG -- Karma test failure

TL;DR
after running 'karma test' and inspecting the console output in the chrome window that Karma opened -- there are no debug messages, only info, warn, & error
I'm submitting a bug report

Library Version:
aurelia-logging: "^1.5.0"

Please tell us about your environment:
OS X 10.14.4

Node Version:
v10.14.0

NPM Version:
6.7.0

JSPM OR Webpack AND Version
JSPM 0.16.53
(Running against local jspm install)

Browser:
Chrome

Language:
karma ?

Current behavior:
browser console output:
WARN[test logger]
INFO[test logger]
ERROR[test logger]
Expected/desired behavior:
WARN[test logger]
INFO[test logger]
ERROR[test logger]
DEBUG[test logger]

What is the motivation / use case for changing the behavior?
To see DEBUG messages in the browser console

@ajoslin103 Any ideas what might be the cause? Is this something that changed with a recent CLI update?

I don't know.  I know that .debug() hasn't worked for a while but I can't say when it stopped working
I moved the bug/error report from the framework down to this library because it's the smallest place I can make it happen
And I didn't write the code that's making it happen so I have a stronger confidence in it :)
If I had to speculate I'd say it's a change to the underlying WebKit browser technology somehow...
In Chrome & Safari I can type: console.debug('test') and I get back undefined with no echo of 'test' -- and the eager evaluation of console. does show a debug function...
In FireFox  I can type: console.debug('test') and it echo's 'test' before I get back undefined
I'm sorry, I just don't know....

@fkleuver has done a ton of work with Karma. Let's see if he just happens to know what's going offhand before we dig in further. Might be something really simple.

Well it couldn't get much simpler...  And I feel quite the simpleton as well.
In the browser inspectors, when in the 'console' view there is a filter wherein you can choose the messages to display by level -- at some point Safari and Chrome started defaulting to not include debug messages.  Both of them have options (Chrome: verbose, Safari: all) that will show debug messages when enabled.
Sorry to waste your time,
Al;

No waste at all. Always glad to not have a bug :) Good to know about the change in default behavior as well. Thanks @ajoslin103 !


Sub-Navigation menu.

having a sub-navigation menu to increases the discoverability of content in lower-level categories of your site architecture with the fewest number of clicks.

Let's add onRender methods for each of the components. It'll give you the element containing the specific component, so you can render under it. Or insert it manually.

You can use the "onRender" method to insert a sub-navigation. Added to v0.3.6.


Consider how to load and share the root certificate store across all HTTP client usages.

While examining component bounds vs actual usage, I noticed that one component was managing to use ~115KiB when it should have been using <10KiB. Doing some investigation led me to discover that this component just happened to be the first one to create an HttpClient, which meant it was responsible for (among other things) lazily loading the root certificate store in the process.
The root certificate store is worth about ~85KiB on my local development machine (Ubuntu 23.10) which isn't necessarily surprising, but certainly blows the budget of the component in the context of its bounds.
Looking deeper into rustls, we can pass it an Arc<T>-wrapped version of the root certificate store that should be used, which means we could conceptually initialize it early on and attribute that memory usage to the root allocation group instead, which would make things look nicer.
The biggest issue is just figuring out how/where to store the shared value for each access. Perhaps we could/should expose some free functions in saluki_io to initialize a OnceCell, which would then be called by saluki_app::tls::initialize_tls.. and then from that point, we'd just use that when creating HTTP clients.
:shrug:

This is going to be solved by #205 which has us loading the platform's native root certificate store and making it reusable/sharable by default when a client TLS configuration is generated.


Send out the announcement for upcoming Sunday

We have plans Detailed here to build an app on this coming Sunday.
We should send an email to the group letting them know the cool thing we are going to do!

[x] What is the email address to send the message to so it gets to our meet up members?
[ ] Is the message correct, clear, exciting?
[ ] Did we send it?

The email address to send announcements to is
<EMAIL_ADDRESS>

I think the description is exciting! I can't wait to go after reading the description! who wouldn't want to implement their own Instagram! 👯

What do you think?

I think we'll keep this one separate from the month long instagram one.
My plan is for this one to just be a one or 2 week, collection views, rest api son parsing thing.
Then for october we'll plan a bigger app, basically a custom "Friendstagram" with firebase backend, probably core data on the device (or at least a caching layer) and hopefully some animations and visual polish.

The updated description reads great! I think it's ready to send! Would you do the honor? :D

Will you send? Please.

Not a problem at all!

I will go out for a bit after sending out the email,  should be able to contribute on the 4-week outline tomorrow!

Sent! Anyway to confirm?

Looks like it is on the message board!

I got the email! Closing this issue.


Adding UI Test (fdl-deprecation)

test to verify email link sent successfully

clang-format versions disappear from homebrew every few months. main should always be fixed, but branches off of main like this one may stop working until they are synced with main again

completed in #1400

completed in #140000


Add information about contributing and releases

We should get a little bit more careful about versioning and releasing now.
Here is what I think are some minimal first steps:

Add an (empty) changelog.
Add some information about the PR and release workflow.

Should we specify a merge strategy for PRs (rebase / squash / merge etc?)

👍 I would not overdo it with processes given the early state of the project but this looks reasonable to me.

I applied all your suggested changes. I also don't want to add unnecessary processes, I mainly wanted to prepare a TODO list for when we start adding and updating versions.
Do you have any opinion on squashing vs just rebasing? I prefer a linear history, so I prefer not to have merges.

Not quite sure about the linear history thing, yet. The problem is that "rebase and merge" will loose all information about which PR the commits originated from, while squash and merge looses all the individual commits. I think I personally prefer rebase + merge commit.

What I'm advocating for is described on GitLab as semi-linear history.

What I'm advocating for is described on GitLab as semi-linear history.

But I don't think I can enforce this strategy in the settings in the GitHub repository? (Or I haven't found the correct configuration yet). I would like to choose some strategy which I can easily enforce with Github. And If I understand it correctly then GitHub only has three options which can be enabled/disabled: Merge, Rebase + FastForward Merge, Squash.

Yes, you are right. One could enforce it e.g. with a custom merge bot but that's probably not worth the effort. Let's be pragmatic and go with the selected options :+1:


Add LockWait functionality

When a lock condition is detected, the caller should wait for the lock to be released.
This is different than acquiring a lock - we do not want to duplicate the job.
However, when the lock is released, it doesn't mean the job is completed.
The requirements should be less strict than idempotency.

Added core/(d)sync/singleflight


Don't attempt to commit if we are disconnected

A little followup on #293

Coverage decreased (-0.08%) to 88.878% when pulling 83ef5aa5ff602f67a7771cacc33d1d02084a73a9 on Pchelolo:clean_queues_on_reconnect into 41ff54c849bd7a1c6308c8b09bab6aad01a7fae0 on wikimedia:master.


Rough fix to IceShatterPower

Hi Gk (and Modargo if you are also here)
So I wanted to make a start on fixing this quickly - can't actually debug at all and I don't have the dependencies here with me but this might be a start?
Will continue work when I can, assuming you haven't got to it first that is!

By the way this code might be totally faulty in loads of ways but I will check when I can

Ok I'm home now and have tested this (it actually works now for realsies)


Remove keyboard context's dictation replacement property

The KeyboardContext is a bit bloated right now, and should be broken up in smaller pieces. It also contains things that used to make sense historically, but that have become obsolete.
One thing that should removed is the  keyboardDictationReplacement property, since the concept of dictation buttons within the keyboard only makes sense on some devices.
You should now create a custom layout provider instead.

I've actually found more usages for this, and will keep it.


🌱  Add some more projects to be scanned in the cron

What kind of change does this PR introduce?
Infrastructure

[✅] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
What is the new behavior (if this is a feature change)?**
This adds projects to the weekly cron job

[N/A] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
NONE
Special notes for your reviewer
Does this PR introduce a user-facing change?
NONE

Rebased my branch 👍 ✅

can you run make add-projects and commit the changes? I'm seeing duplicates.
e.g. "github.com/gitlabphp/client," vs "github.com/GitLabPHP/Client" which is already in the list
For the future: I'll add this to the documentation somewhere and make the CI fail if it's not done.

Done ✅

Thanks @katzj sorry for the back and forth


⚠ Bofrost.it has degraded performance

In ddec2d5, Bofrost.it (https://www.bofrost.it) experienced degraded performance:

HTTP code: 200
Response time: 1534 ms

Resolved: Bofrost.it performance has improved in 870347c after 6 minutes.


chore: add semantic release and use GH for CI

@feross we need to enable the semantic pull request check in this repo, as well as removing the travis check requirement while adding the new GH actions ci

@DiegoRBaquero Alright, done


Linking with skiplist does not appear to be needed

diff --git a/Makefile b/Makefile
index bb7eea8..a80b9b3 100644
--- a/Makefile
+++ b/Makefile
@@ -21,7 +21,7 @@ CSTD +=               -std=c99 #-D_POSIX_C_SOURCE=200112L -D_C99_SOURCE
 CFLAGS +=      ${CSTD} -g ${WARN} ${CDEFS} ${CINCS} ${OPTIMIZE}

 LDFLAGS +=
-MAIN_LDFLAGS += ${LDFLAGS} -lskiplist -L${BUILD} -lhopscotch
+MAIN_LDFLAGS += ${LDFLAGS} -L${BUILD} -lhopscotch

 TEST_CFLAGS =  ${CFLAGS}
 TEST_LDFLAGS =  ${LDFLAGS} -ltheft

Running your tool on a large dataset revealed some issues. See the attached patch for fixes.
hpscotch.txt

You're absolutely right that it doesn't need the skiplist library. That was a dependency early on and never got removed. Good catch!
I'm looking at your other patch now. Both will go on the develop branch and in the next release. Thanks.

These are fixed in the 0.1.2 release. Thanks.

Thank you! I'll give it a spin too when I get back to my project :-)


Build error [gcc 4.6.1, archlinux, x86_64]

I'm trying to build libcppa from the git repository.
There is only an old_configure script file.  When I run it, it generates a Makefile. I run make and it starts to build libcppa.  Unfortunately, it soon fails with this message :
/usr/bin/g++ -std=c++0x -pedantic -Wall -Wextra -g -O0 -I/opt/local/include/ -fpermissive -Wno-deprecated-declarations  -I./ -fPIC -c src/blocking_message_queue.cpp -o src/blocking_message_queue.o
In file included from src/blocking_message_queue.cpp:8:0:
./cppa/invoke_rules.hpp:77:5: error: ‘cppa::invoke_rules_base::invoke_rules_base()’ cannot be overloaded
./cppa/invoke_rules.hpp:67:5: error: with ‘cppa::invoke_rules_base::invoke_rules_base()’
./cppa/invoke_rules.hpp:79:5: error: ‘cppa::invoke_rules_base::invoke_rules_base(cppa::invokable_list&&)’ cannot be overloaded
./cppa/invoke_rules.hpp:69:5: error: with ‘cppa::invoke_rules_base::invoke_rules_base(cppa::invokable_list&&)’
./cppa/invoke_rules.hpp:81:5: error: ‘cppa::invoke_rules_base::invoke_rules_base(cppa::invoke_rules_base&&)’ cannot be overloaded
./cppa/invoke_rules.hpp:71:5: error: with ‘cppa::invoke_rules_base::invoke_rules_base(cppa::invoke_rules_base&&)’
make[1]: *** [src/blocking_message_queue.o] Error 1

I have no ideas how libcppa is coded, and I am not sure that to use ./old_configure was the correct command to launch.
The readme file doesn't help much (and I can't help to try out this library).
As said in the title, I'm on a archlinux system, with gcc4.6.1, and with the x86_64 processor instruction set.

Hi,
thank you for the bug report. I'll update the readme file.
libcppa uses automake as build tool. The usual steps are:
autoreconf -i
./configure
make

You also need the boost-thread library.
Have fun! :)

Thank you to give the correct way to build the source.
gcc builds more files, but it anyway fails when meeting those lines :
https://github.com/Neverlord/libcppa/blob/master/cppa/invoke_rules.hpp#LC65
It doesn't accept the constructor to be twice declared, once as a private constructor, once as a protected one.
I've removed the private constructor declarations, and it worked fine.

fixed :)


undefined method

<IP_ADDRESS>
<IP_ADDRESS> ================================================================================
<IP_ADDRESS> Recipe Compile Error in /var/chef/cache/cookbooks/mariadb/recipes/_redhat_server.rb
<IP_ADDRESS> ================================================================================
<IP_ADDRESS>
<IP_ADDRESS> NoMethodError
<IP_ADDRESS> -------------
<IP_ADDRESS> undefined method use_os_native_package?' for cookbook: mariadb, recipe: _redhat_server :Chef::Recipe <IP_ADDRESS> <IP_ADDRESS> Cookbook Trace: <IP_ADDRESS> --------------- <IP_ADDRESS>   /var/chef/cache/cookbooks/mariadb/recipes/_redhat_server.rb:20:in from_file'
<IP_ADDRESS>
<IP_ADDRESS> Relevant File Content:
<IP_ADDRESS> ----------------------
<IP_ADDRESS> /var/chef/cache/cookbooks/mariadb/recipes/_redhat_server.rb:
<IP_ADDRESS>
<IP_ADDRESS>  13:  # Unless required by applicable law or agreed to in writing, software
<IP_ADDRESS>  14:  # distributed under the License is distributed on an "AS IS" BASIS,
<IP_ADDRESS>  15:  # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
<IP_ADDRESS>  16:  # See the License for the specific language governing permissions and
<IP_ADDRESS>  17:  # limitations under the License.
<IP_ADDRESS>  18:  #
<IP_ADDRESS>  19:
<IP_ADDRESS>  20>> use_os_native = use_os_native_package?(node['mariadb']['install']['prefer_os_package'],
<IP_ADDRESS>  21:                                         node['platform'],
<IP_ADDRESS>  22:                                         node['platform_version'])
<IP_ADDRESS>  23:  use_scl = use_scl_package?(node['mariadb']['install']['prefer_scl_package'],
<IP_ADDRESS>  24:                             node['platform'],
<IP_ADDRESS>  25:                             node['platform_version']) &&
<IP_ADDRESS>  26:            scl_version_available?(node['mariadb']['install']['version'])
<IP_ADDRESS>  27:
<IP_ADDRESS>  28:  # To force removing of mariadb-libs on CentOS >= 7
<IP_ADDRESS>  29:  package 'MariaDB-shared' do
<IP_ADDRESS>
<IP_ADDRESS> System Info:
<IP_ADDRESS> ------------
<IP_ADDRESS> chef_version=13.3.42
<IP_ADDRESS> platform=centos
<IP_ADDRESS> platform_version=7.3.1611
<IP_ADDRESS> ruby=ruby 2.4.1p111 (2017-03-22 revision 58053) [x86_64-linux]
<IP_ADDRESS> program_name=chef-client worker: ppid=2206;start=12:26:46;
<IP_ADDRESS> executable=/opt/chef/bin/chef-client
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS> Running handlers:
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] ERROR: Running exception handlers
<IP_ADDRESS> Running handlers complete
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] ERROR: Exception handlers complete
<IP_ADDRESS> Chef Client failed. 0 resources updated in 08 seconds
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] ERROR: undefined method `use_os_native_package?' for cookbook: mariadb, recipe: _redhat_server :Chef::Recipe
<IP_ADDRESS> [2017-08-29T12:26:54-04:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
PS C:\Users\ehaston\ChefMad\cookbooks>

Hi!
Sorry for the delayed response - can you test if this is fixed in the latest version (1.5.2)?
There has been some refactoring and the tests seem to pass.  If you still have a problem please let me know how I can replicate.
Cheers,
Stephen

Hi,
Closing issue as we have released 2.0.0 which does not use recipes anymore, if you are still having an issue please re-open
Thanks


BigQuery swallowing exceptions

Steps to reproduce:
Consider two Cloud Platform Projects "A" and "B".

Generated Service Account Key for Project "A" (in JSON Format) with role as BigQuery Metadata Viewer.
Modified the "project_id" value from "A" to "B" in the file.
Used this file to connect to BigQuery instance.
listDatasets() on com.google.cloud.bigquery.BigQuery returns an empty List.

Expected Outcome:
The operation should fail, stating cannot access Project "A" with the provided credentials.
Other Information:
Using Google Client Library: com.google.cloud:google-cloud-bigquery:jar:1.48.0
        GoogleCredentials googleCredentials;
        File credentialsFile = new File("path to the service account key file");
        try(FileInputStream fileInputStream = new FileInputStream(credentialsFile)) {
            googleCredentials = ServiceAccountCredentials.fromStream(fileInputStream);
            BigQuery bigQueryClient = BigQueryOptions.newBuilder()
                    .setCredentials(googleCredentials)
                    .build()
                    .getService();
            for (Dataset dataset : bigQueryClient.listDatasets().iterateAll()) {
                // listDatasets() returns an empty list
            }
        } catch (IOException ioException) {
            // do something
        }

[This was reported by a user]

@sduskis server is returning empty dataset and not returning any error. This can not be fixed in client.

This is similar to ##3344

@sduskis I verified all http request and response in this case and  listDataset is returning status code 200 for refresh token and listDataset api. This is same as authenticating with project A and accessing resource from project B. Changing just project id to B in json file still authenticates as A since refresh token belong to project A.
However the api we are using in this issue step is list, it just returns empty list since project B didnt grant permission to any dataset to project A.
If we use exact data set id with api bigQueryClient.getDataset("testdataset"); then it returns proper error message as below.
com.google.cloud.bigquery.BigQueryException: Access Denied: Dataset ociinternalappid:testdataset: The user<EMAIL_ADDRESS>does not have bigquery.datasets.get permission for dataset ociinternalappid:testdataset.


Replace missing PyGithub calls

Using PyGithub instead of calling Github API using requests. This brings the benefit that, when not specific commit is selected, PyGithub uses the default branch. We were building the URL to fetch content from master but this is not always the default branch (Github has switched to using main as the default, as pointed out in #296). This takes care of it automatically.
Fixes #296

I left in methods from both branches, including getRepoDescription and getLicenseURL, to remove conflicts and avoid removing necessary code --please amend if necessary


feat: update to track latest spec changes

Automated changes after Neon spec has been updated

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add test for route secrets

/cherrypick release-4.1

/assign @adambkaplan
Can I reuse the same bug for backporting tests?

Can I reuse the same bug for backporting tests?

since it needs to target the appropriate release, unfortunately not. just clone it.

@bparees this PR is against the master branch (4.2), and I used cherrypick release-4.1. On the other hand I have bugs
4.2.0 - Bug 1719965: TLS Keys Not Added to Registry Routes (VERIFIED)
4.1.z - Bug 1720270: TLS Keys Not Added to Registry Routes (POST)
Can I use the first bug for this PR, and the second bug for the PR that will be created by the cherrypick robot?

@dmage that seems fine

/assign @adambkaplan

/retest
possible flakes

flake
/retest


描述/Description
无法进入与JM相关的漫画
Version
3.1.4
使用的操作系统/Operating system
Android
日志/logs
POST https://www.jmapinodeudzn.xyz/login
DioException [bad response]: This exception was thrown because the response has a status code of 301 and RequestOptions.validateStatus was configured to throw for this status code.
The status code of 301 has the following meaning: "Redirection: further action needs to be taken in order to complete the request"
Read more about status codes at https://developer.mozilla.org/en-US/docs/Web/HTTP/Status
In order to resolve this exception you typically have either to verify and fix your request code or you have to fix the server code.
[]
截图或视频/Screenshot or video
No response

我也是


Adds basic petri dish rendering

Initializes the simulation visualizations and configurable placement of antibiotics.

 Kirby: Base Two.js display for dish


Update the composer version to PHP7.2

This change will update the composer version to php7.2 and update phpunit to v8
Support for 7.1 has ended so we should upgrade to 7.2 - Security support for 7.2 will end in Nov 2020

`PHPUnit 8.5.2 by Sebastian Bergmann and contributors.
............................                                      28 / 28 (100%)
Time: 116 ms, Memory: 20.00 MB
OK (28 tests, 42 assertions)
`

Bumping versions isn't a a BC break. https://www.doctrine-project.org/2017/07/25/php-7.1-requirement-and-composer.html

It is for PHPUnit.
Anyways, these changes have no reason to drop support of PHP 7.1. If you introduce a feature not supported in PHP 7.1 then fine, but not with these changes.

It's what the sat solver is for. This really isn't a BC break.

I think this PR is now duplicate because #105 is merged. Correct me if I'm wrong.


Listview doesn't scroll when inside a scrollview

Noticed this after upgrading to 0.7.1 from 0.6.
Below is a simple example to reproduce. I noticed it when upgrading my app that has three separate list views inside a scrolling container. List views are set to show all rows, as they're not expected to be very long. But all three are large enough to go off page.
/**
 * Sample React Native App
 * https://github.com/facebook/react-native
 */
'use strict';

var React = require('react-native');
var {
  AppRegistry,
  Component,
  ListView,
  StyleSheet,
  Text,
  ScrollView,
  View,
} = React;

class TestScrollView extends Component {
  constructor(props) {
    super(props);
    let ds = new ListView.DataSource({rowHasChanged: (r1, r2) => r1 !== r2});
    this.state = {
      dataSource: ds.cloneWithRows(["ALABAMA", "ALASKA", "AMERICAN SAMOA", "ARIZONA", "ARKANSAS", "CALIFORNIA", "COLORADO", "CONNECTICUT", "DELAWARE", "DISTRICT OF COLUMBIA", "FEDERATED STATES OF MICRONESIA", "FLORIDA", "GEORGIA", "GUAM GU", "HAWAII", "IDAHO", "ILLINOIS", "INDIANA", "IOWA", "KANSAS", "KENTUCKY", "LOUISIANA", "MAINE", "MARSHALL ISLANDS", "MARYLAND", "MASSACHUSETTS", "MICHIGAN", "MINNESOTA", "MISSISSIPPI", "MISSOURI", "MONTANA", "NEBRASKA", "NEVADA", "NEW HAMPSHIRE", "NEW JERSEY", "NEW MEXICO", "NEW YORK", "NORTH CAROLINA", "NORTH DAKOTA", "NORTHERN MARIANA ISLANDS", "OHIO", "OKLAHOMA", "OREGON", "PALAU", "PENNSYLVANIA", "PUERTO RICO", "RHODE ISLAND", "SOUTH CAROLINA", "SOUTH DAKOTA", "TENNESSEE", "TEXAS", "UTAH", "VERMONT", "VIRGIN ISLANDS", "VIRGINIA", "WASHINGTON", "WEST VIRGINIA", "WISCONSIN", "WYOMING"])
    };
  }
  render() {
    return (
      <View style={styles.container}>
        <ScrollView>
          <ListView
            dataSource={this.state.dataSource}
            renderRow={(rowData) => <Text>{rowData}</Text>}
          />
        </ScrollView>
      </View>
    );
  }
}

var styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  }
});

AppRegistry.registerComponent('TestScrollView', () => TestScrollView);

just see noder, I put five listView into a scrollView, it works well.

Are you running version 0.7? I see you have ^0.6.0 but if you haven't updated in a bit, it could be 0.6.x, which works fine.

Is it necessary to wrap your ListView with ScrollView?
Have you tried just using ListView?

I have three separate list views that show all the items, as i don't want them scrolling. So I wrap all three in a single scroll view. My above example is a simple & short one to show the issue.

I have update my project noder to 0.7.1, it works well,

@soliury i wonder if it's because your PageScrollView component is horizontal as opposed to vertical.

Doing some more experimentation with this. I seem to have scroll issues when it's a really long body of content. Do i need to manually set the height or add styling of some kind? I just have an app with a scroll view tag and 100+  components in it. I can't scroll down the page either.

Realized a change along the way must have broke how i was using the scrollview. Updating to 0.8.0-rc, using the Dimensions package has helped me fix this. I set the height of my top level view in index.ios.js to that of the screen:
class App extends Component {
  render() {
    return (
      <Navigator
        initialRoute={{
          component: ProjectList,
          navigationBar: <NavigationBar title="Projects" titleColor="#000000" style={appStyles.navigator} />,
          id: "project_list"
        }}
        renderScene={this.renderScene}
      />
    );
  }

  renderScene(route, navigator) {
    var Component = route.component;
    var navBar = route.navigationBar;

    if (navBar) {
      navBar = React.addons.cloneWithProps(navBar, {
        navigator: navigator,
        route: route
      });
    }
    var height = Dimensions.get("window").height;
    return (
      <View style={{height: height }}>
        {navBar}
        <Component {...route.props} navigator={navigator} route={route} />
      </View>
    );
  }
}

Then pass a flex 1 to the scrollview in my subcomponents. Though with this method, you'll need a scrollview in every component that requires scrolling.

how to make  scrollable ListView inside Modal ???
Modal
//need scrollable listview
/Modal

@DurgaManickam  I got the same issue, sloved by setting height of ListView.

@phoue should I get the height from Dimensions, to make the modal get the whole screen?

I had the same issue as I had the following component structure.

As wrapping the ListView inside the ScrollView the parent scroll action dominates the child scroll action which leads only ScrollView to scroll.
I resolved this issue by adding the following code as a prop to the parent View which wraps the ScrollView
onStartShouldSetResponderCapture={() => {
this.setState({ enableScrollViewScroll: true });
}}
Add  scrollEnabled={this.state.enableScrollViewScroll} prop to ScrollView
And Finally add following to the View which wraps the ListView( immediate parent )
onStartShouldSetResponderCapture={() => {
this.setState({ enableScrollViewScroll: false });
if (this.refs.myList.scrollProperties.offset === 0 && this.state.enableScrollViewScroll === false) {
this.setState({ enableScrollViewScroll: true });
}
}}
this works very well on IOS and Android both, scrolls the ListView if the touch is on List and else it scrolls the ScrollView

@sadafk831 awesome solution. Saved my time and effort at crucial time.

@sadafk831 thanks for your feedback. It helps.

@sadafk831 what is scrollProperties.offset? it throws me an undefined is not an object error.
Does FlatList have it?

@Thanmai-C I am using FlatList, not sure if this is the right way but I wrote this.refs.myList._listRef._scrollMetrics.offset instead.

@sadafk831  what is "this.refs.myList"? I am new to react-native

@sadafk831 I am using FlatList instead of listview. I am getting offset of undefined error message. Please give me someone solution


PSR2 fixer is incomplete with respect to argument lists

I believe the PSR2 level does not implement PSR2 completely - is that a known issue, or a bug?
One example is that function arguments should either all be on one line, or all on separate lines (§ 4.4, §  4.6). Additionally, ) { should be on the same line. For the following code:
<?php

function bar($a,
    $b, $c)
{
    print_r($a);
    print_r($b);
    print_r($c);
}

bar(1,
    2, 3);

the command php-cs-fixer fix example.php --level=psr2 leaves the code untouched. However I would expect it to output each argument on a new line (since some are already on a new line) as follows:
<?php

function bar(
    $a,
    $b,
    $c
) {
    print_r($a);
    print_r($b);
    print_r($c);
}

bar(
    1,
    2,
    3
);

I'm using version 1.10.

thanks for the report, I can confirm this.
(working on this, hope to wrap up a PR soon)

@SpacePossum thank you very much!!

Closing as duplicate of #1184.
Also, see #1187


Theme should differentiate python code from parsed-literal

IPython notebooks are a great way to write docs and convert them to rst. However, the theme makes the code and its output (which is given as a parsed-literal in the rst file) look identical, which makes things a little bit confusing.
This is what an example notebook looks like:

This is how readthedocs renders it

If the border or background color were changed slightly on the parsed-literal, this would make this way of writing documentation a lot easier.
This was copied from rtfd/readthedocs.org#1388

Do you have the original rst code for us to test?

Closing, this should be fixed with the pygments fixes but also the requested information was not given.


The comments duplicate will remain above the class after re-organizing.

The comments duplicate will remain above class if I use using to import namespaces.
Here is my testing code:
using System;

public class Test : object
{
    // Some comments
    public void Foo()
    {
    }
}

After re-organizing, it will look like this:
using System;

// Some comments
public class Test : object
{
    // Some comments
    public void Foo()
    {
    }
}

But if there is no using, it will work perfectly.
public class Test
{
    // Some comments
    public void Foo()
    {
    }
}

After re-organizing: (nothing change in this case)
public class Test
{
    // Some comments
    public void Foo()
    {
    }
}

I also noticed that comments above the using directive get removed after reorganizing.
Maybe this is related to the issue?
// Some comments
using System;

public class Test : object
{
}

And after re-organizing, it becomes:
using System;

public class Test : object
{
}

This is a great project, appreciate your efforts.

Thanks. I'll take a look at it. I've found a couple other quirks lately that I need to address as well.

The code checked in tagged as 0.0.2 and soon its package to VS Marketplace should have this fixed.

Hey, after further testing, I've found that this issue still persists.
I've retested the testing code I initially provided:
using System;

public class Test : object
{
    // Some comments
    public void Foo()
    {
    }
}

I discovered that this example was problematic. Since System wasn't being used, it would be removed by CSharpen, making it appear to work correctly. (I suspect this is because the removal of the using statement caused it to enter the state of the second example mentioned in the original post.)
Here's an updated testing code:
using System.Collections.Generic;

public class Test : List<int>
{
    // Some comments
    public void Foo()
    {
    }
}

This new testing code should consistently reproduce the error. I apologize for providing a problematic testing code initially.

In RE: to your initial example, I did have issues with the example code as you mentioned but I changed it a but and was able to reproduce (and resolve). But with your latest code, I can reproduce it as well. Could have been two similar issues. Will look into this latest example to resolve.

Found the issue of your last code example. It was related to do with using directives and the whitespace afterwards. This should be resolved with 0.0.3 that was just released.


FR: Suggestions for WebGLRenderer refactor

Having heard the GL side of things is slated for a major refactor, as well as having seen [some] (https://github.com/Famous/engine/issues/272#issuecomment-119031972) of the [writing] (https://github.com/Famous/engine/issues/288) on the [wall] (https://github.com/Famous/engine/issues/313), I figured now might be a good time to request certain features that might otherwise be untenable if not for an impending refactor.
So without further ado, here's my wish list:
1. Decouple the cutout system from WebGLRenderer
I would argue that the cutout system should not be an intrinsic part of WebGLRenderer. Per [issue 313] (https://github.com/Famous/engine/issues/313), barring a major leap like rasterization of DOM content (which is a long, long ways off), the only way complex cutouts can be reliably achieved is if the cutout system is exposed to the user, and the user is free to define the cutout representing their DOM content.
2. Eschew vertex-based cutouts in favor of a fragment-based system with blending support
Currently, the cutout system is based on vertex data (geometry). While this might be somewhat more efficient than alternatives, it's also not as capable. DOM images containing alpha opacity are the most glaring argument against the current system.
3. The cutout/blending system should support n layers
Specifically, n GL layers. Imagine the following scenario:
You have DOM content containing alpha opacity, some Famous GL content, as well as foreign (i.e. non-Famous) GL content. All of it needs to play nicely together.

Tell the cutout system about your complex DOM content so it can render a correct cutout.
Create a new layer within this same system, passing in the appropriate color and depth buffers that represent said foreign GL content.

Ideally, the system would first perform fragment depth testing across all layers, followed by a blending pass. The end result is you'd have DOM + n GL layers all coexisting peacefully, all respecting the same cutout state, with a single Canvas element.
4. Ability to specify a custom depth buffer precision for each layer
For my use case, I require a logarithmic depth buffer.
5. Post-processing pipeline
I know this is already planned, so I'll just say I'm really looking forward to it. :-)
Conclusion
If nothing else, having the first item would make it such that those of us wishing to implement our own cutout system wouldn't have to rewrite most of WebGLRenderer to do so.
Thanks for reading! Looking forward to any input.

Thanks for the detailed "wish list" :smile:

Current system is messed up. Cutouts should be completely decoupled. First step is probably to make them components that send draw commands to the WebGLRenderer (just like Mesh). This allows a more flexible approach towards cutouts.

Maybe. Not sure yet.

I'm not sure if we want to introduce the complexity of having an arbitrary number of canvas layers (or number of WebGLRenderers). I would say it's rather unlikely that we're going to implement this in the near future.

Nothing planed in that direction, but I would welcome any PRs in that direction.

This is a separate feature. I'm unsure about the current priority of the PPP.

Conclusion
The cutout system will be decoupled by making the cutouts components. We're currently refactoring parts of the WebGLRenderer, changes like mentioned above will be implemented during this refactor.

I'm leaving this open in case anyone wants to add anything....

the user is free to define the cutout representing their DOM content.

But that's what I love about "mixed mode", is not having to worry about that. That's what makes Engine special. I don't want to think about that. I just want it to work. But if it's as easy as
let el = new DOMElement(node)
let cutout = new DOMElementCutout(node)

then I can live with that.

We are working on an implementation for a node to know the drawn size of the DOM and creating a cutout based on the border-radius so the user doesn't need to self define.

Eschew vertex-based cutouts

Does not a cutout also have a fragment shader? Farhad was telling me we could put shadows on the cutouts, implying fragment shaders.

@michaelobriena Awesome.

having the first item would make it such that those of us wishing to implement our own cutout system wouldn't have to rewrite most of WebGLRenderer to do so.

I could see that being really powerful in cases when a DOM element has a border radius or is clipped into arbitrary shapes using the CSS clip property. I would agree that some control is needed in those scenarios, but at the top level there should be no control required for it to work with normal rectangular DOM elements (the default).

@alexanderGugel

Good to know. Any ideas about how user-defined cutouts might work? I intentionally left that part vague since I figured it could probably go many different ways. Perhaps something as low-level as dealing with raw buffers, to as high-level as simply supplying an image.

Not sure how DOM images with alpha opacity would be handled otherwise.

Understandable. Put another way, I was basically looking for a way to have Famous accept foreign color/depth buffers and reconcile them with its cutout system. If this isn't going to happen, then at least the converse would be nice - the ability to retrieve both cutout state and Famous' GL buffers so that they could be integrated into a custom blending/compositing solution. That would probably mean having to support render targets, though.

This shouldn't be that hard, though I may as well wait until the refactor has shaken out before submitting PRs.

A couple extra thoughts:
a) I'm thinking #1 and #2 may not be as clear-cut as we'd like; in order to have pixel-perfect cutouts you'd need not just the image data (for alpha opacity), but to also have the correct filtering and anti-aliasing applied to the cutout. I imagine this is very difficult, especially when CSS matrix transformations are involved.
b) If a fragment-based cutout system isn't preferred due to performance considerations, then I suppose a hybrid system might work: fall back to vertex-based where fragment isn't required. This would have complexity implications though.
Thanks again, appreciate the feedback.


OOM in pretraining

I tried to pretrain HERO mode from scratch in HowTo100M and TV datasets, and the code worked well at the begining, but crashed after thousands of iterations. I found that the memory usage was growing in training and finally out of memory. Have you met this problem?

I also encounter the same problem. @linjieli222

@Liu0329 @hgzjy25
I have received similar reports about this issue. However, we did not met the same issue during our experiments. You may need to search online for potential solutions, sorry for any inconvenience. If you do find a solution, please also come back and post it here helping other people in need.

One potential direction, check if the memory increasing is due to caching. If so, you can force to clean the cache periodically.


feat: emit error to the webworker

Issue - https://github.com/tscircuit/snippets-cli/issues/7

Yeah correct, this thing is covered by web worker


Is there any way to tell paranoidHttp to allow localhost?

I'm not a go programmer, so pardon my ignorance - but I have everything running on my internal network - no outside IP addresses are used, and the rewriter here is the only thing that appears to work properly with my matrix install - pure proxying with nginx does not work right for me. All clients operate on the network directly or via VPN.
Looking at the docs for paranoidhttp, it does seem to exemplify on the main readme how to allow <IP_ADDRESS> - which is what I'd love to be able to do...
In my testing, other than having up-rewrite-linux-amd64 reject the IP, everything else is working.
Thank you!

This is a configuration option. See the docs for allowed gateway host
https://github.com/UnifiedPush/common-proxies/blob/main/docs/config.md

This doesn't seem to work for me as it appears that without using an explicit allow when creating the phttp client,, it still is not allowed.
My Config:
listenAddr = "<IP_ADDRESS>:5000"
verbose = true

[gateway]
        AllowedHosts = ["<IP_ADDRESS>"]
        [gateway.matrix]
                enabled = true
[rewrite]
        [rewrite.fcm]
                enabled = false
                # key = ""
        [rewrite.gotify]
                enabled = true
                address = "push.internal.lan:8081"
                scheme = "http"

Example of allowing localhost in the paranoidhttp readme:
// Add an permitted ipnets with functional option
ipNet, _ := net.ParseCIDR("<IP_ADDRESS>/32")
client, _, _ := paranoidhttp.New(
    paranoidhttp.PermittedIPNets(ipNet))

AllowedHosts = ["<IP_ADDRESS>"]

Is your Gotify (or other push provider?) reverse proxy listening on ports 80 and 443? because as https://github.com/UnifiedPush/common-proxies/blob/main/docs/config.md says

The port only needs to be included if it's something other than 80 or 443, but if so, entries for both HTTP and HTTPS should be included.

Also,

Example of allowing localhost in the paranoidhttp readme:

common-proxies does the same thing, just differently, bypassing pHTTP altogether.
https://github.com/UnifiedPush/common-proxies/blob/bc7e94a3604d74001f1070b8bf90f3e91af3d75b/handler.go#L68-L74

Gotify is what I use, it listens internally on 8081 but is proxied on port 80 and 443 currently.
Your code is good for bypassing pHTTP, however I think that because you're already initializing pHTTP in init() that localhost is rejected, preventing it from executing anyway.

What error log do you receive?

Also, are you making using Gotify at http://localhost, because that is distinct from http://<IP_ADDRESS>

Gotify is listening on <IP_ADDRESS> as well as externally on <IP_ADDRESS> on the LAN it sits on - I was just using localhost here as an alias, apologies.
It may be matrix-synapse being troublesome, I forget how it is I reconfigured the push location a couple days ago, however now it is coming from an internal <IP_ADDRESS> IP (which is local to the machine) and I've added to allowed hosts, both with and without the port 81 part - I've tried previously with port 80 with similar results.
2021/11/17 17:25:44 Loading new config
2021/11/17 17:25:44 Server is ready to handle requests at <IP_ADDRESS>:5000
2021/11/17 17:25:50 Post "http://push.internal.lan:81/UP?token=xxxxxxxxxxxx": bad ip is detected: <IP_ADDRESS>
2021/11/17 17:25:50 POST /_matrix/push/v1/notify <IP_ADDRESS>:59958 0 bytes read; 0 bytes written; Synapse/1.46.0 forward

Gotify is listening on <IP_ADDRESS> as well as externally on <IP_ADDRESS> on the LAN it sits on - I was just using localhost here as an alias, apologies.
It may be matrix-synapse being troublesome, I forget how it is I reconfigured the push location a couple days ago, however now it is coming from an internal <IP_ADDRESS> IP (which is local to the machine) and I've added to allowed hosts, both with and without the port 81 part - I've tried previously with port 80 with similar results.
2021/11/17 17:25:44 Loading new config
2021/11/17 17:25:44 Server is ready to handle requests at <IP_ADDRESS>:5000
2021/11/17 17:25:50 Post "http://push.internal.lan:81/UP?token=xxxxxxxxxxxx": bad ip is detected: <IP_ADDRESS>
2021/11/17 17:25:50 POST /_matrix/push/v1/notify <IP_ADDRESS>:59958 0 bytes read; 0 bytes written; Synapse/1.46.0 forward

I recall - I have gotify listening through nginx on port 81, and haproxy on 80 - in both cases the above log results.

bad ip is detected: <IP_ADDRESS>

You need to add push.internal.lan:81 (and the https version if there is any) to the allowed hosts lists since that's the host that's being blocked.

Ah - that got it!
I suppose hostname here matters more than an IP, then. Good to know and thank you!


Loading a directory of VCF files fails

There's a inputPath.getName().endsWith(".vcf") check that needs changing.

Not obvious to me that the current behavior is incorrect.  If the directory contains multiple VCF files with different headers, the variant loader will probably break.

You're right. I think ADAM only allows one VCF to be loaded at at time. Quince is consistent with that, so this isn't an issue. Closing.


Enhance Cedar Mapper to parse Policy Template

Currently generates an error when parsing:
permit(
    principal == ?principal,
    action in [hexa_avp::Action::"ReadAccount"],
    resource == ?resource
);

? is unexpected

Completed and added to 0.6.0-dev


Write tests

Especially integration tests. Clicking around manually to test is annoying and error-prone.
Also: hook up a CI system.

I'm definitely interested in doing this.


Fix: Flatten directory structure

This PR

[x] flattens the directory structure

Follows #344.

Going to look at this one more closely when I get home. Looks good but I'm on mobile at the moment and want to make sure this passes when integrated with your other PRs.

Actually, just saw conflict. It's probably trivial but could you go ahead and rebase this on master so we can see integrated build?

@mdwheele
Totally, will do as soon as I'm out of the :bath:!

TMI!!!

@mdwheele
Hehe, sorry - rebased!

Looking at this now. Just got internet back last night after the east coast Snowpocalypse

Looks good. Thanks @localheinz!

@mdwheele
:snowman:


Typescript declaration file (.d.ts)

Issue:
The declaration file is missing to work properly with typescript.  (= typescript support ?)
Which version are you using?
v1.0.7
Are you using storybook-router with a react based project or a vue one?
vue
Please describe the problem:
I have the following error on serve:
Could not find a declaration file for module 'storybook-vue-router'. 'd:/path/node_modules/storybook-vue-router/dist/vue.js' implicitly has an 'any' type.
Please explain how to reproduce the issue or (better) provide an example to do it.
a simple import and use decorator trigger this error
// myComponent.stories.ts
import StoryRouter from "storybook-vue-router";

export default {
  title: "aStory/Title",
  component: MyComponent,
  decorators: [StoryRouter()]
};

You can create a storybook-vue-router.d.ts file with this declaration:
declare module 'storybook-vue-router' {
  export default function storyRouterDecorator(): Function
}

I'm not sure it's 100% correct, but it will fix the "missing declaration file" error


fix(table-loading): client-side table

dont show the blankslate if not defined as prop
Proposed Changes

Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1429 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1429   +/-   ##
=======================================
  Coverage   99.15%   99.15%
=======================================
  Files         484      484
  Lines       10084    10084
  Branches     2329     2329
=======================================
  Hits         9999     9999
  Misses         85       85

Impacted Files
Coverage Δ

...vapor/src/components/table-hoc/TableWithFilter.tsx
100% <100%> (ø)
:arrow_up:

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 700bed1...3b10d50. Read the comment docs.


Bug: GoRename does not work consistently

Hi, I'm not sure, if I'm missing anything, but I ran into issues when using GoRename (using vim.lsp.buf.rename works). These are the steps to reproduce:
Create a main function calling a function Foo() in some package:

Renaming the function to FooBar work out pretty well:

Renaming the function again results in an error:

Neovim: 0.9.5
go.nvim: current master

It works fine for me

Also could you update to neovim 0.10 as 0.9 is no longer supported.

Having the same issue with 0.10:

If I can provide anymore information, please let me know.

Can this issue be reproduced with the following config
init_lazy.lua
with
nvim -u init_lazy.lua main.go

GoRename shells out to an external command, and should probably be avoided if you have a LSP client attached.
My intuition is that the file you are editing isn't being written after performing the first rename, so the file that is passed to the command on the second rename differs from what is shown in the Neovim buffer.

Actually it seems like I was wrong about this:

GoRename shells out to an external command
-- https://github.com/ray-x/go.nvim/issues/482#issuecomment-2247743882

The documentation might be a bit outdated (?). GoRename seems to consistently call lsprename():
https://github.com/ray-x/go.nvim/blob/033344ddfa3cd5cfd55037903264b2bb86691619/lua/go/commands.lua#L378-L380
Which itself calls vim.lsp.buf.rename():
https://github.com/ray-x/go.nvim/blob/033344ddfa3cd5cfd55037903264b2bb86691619/lua/go/rename.lua#L6-L10
I just tried performing :GoRename twice in a row on the same symbol, without writing the buffer between the two actions, and everything worked as expected across multiple project packages.

I did find some issue with the LSP key bindings provided by the plugin though.
If you have lsp_keymaps set to true (the default), <space>rn will call the old implementation (which I suspect to be broken):
https://github.com/ray-x/go.nvim/blob/033344ddfa3cd5cfd55037903264b2bb86691619/lua/go/lsp.lua#L66
This does seem like a bug, but you didn't mention anything about using this key binding, so I assume the problem is unrelated.

Thanks @antoineco for your explanation. I do have this keymapping in my init.lua
nmap('<leader>rn', vim.lsp.buf.rename, '[R]e[n]ame')

When  I use this keymapping, it does not open the popup prompts for the new name on the bottom of the screen. This way, the renaming works, if I however call :GoRename it does not.
I didn't mention the binding, because I could reproduce the issue with @ray-x's init_lazy.lua as well.


Issues with Bazel & ccache

If, like me, you're using ccache to significantly speed up clean rebuilds and other edge cases, you'll probably run into this problem: https://stackoverflow.com/questions/52370202/how-to-get-bazel-ccache-and-sandboxing-to-work-together-ccache-read-only-file
For me, in fact this, required also adding another directory, which you can get from either the error messages you'll, or by running ccache -p and looking what is set as temporary_dir.
For me, I then also had issues with bazel crashing because $HOME failed it's internal "must be an absolute path" which made bazel throw an exception and dump the stack 🤪
This is what my .bazelrc ended up looking like
common --sandbox_writable_path=/run/user/1000/ccache-tmp --sandbox_writable_path=/home/vianda/.cache

I don't use ccache (but maybe I should). Is this something that you think we should add generically to the project config? Getting the shell interpolation for ccache --get-config temporary_dir and $HOME/cache would require a slight layer of indirection, but it's definitely doable.

I don't think use of ccache is widespread enough to warrant including this, this issue is more so that people with similar issues can find the solution/workaround.

More reasons not to try and integrate this workaround into the repo: when (for whatever reason) the directories don't exist yet (e.g., ccache hasn't created them yet), bazel will actually complain about the directory not existing and fail when trying to create the sandbox....


Running issue

Dear Friends,
I'm new in using this kind of systems.
I'm running Orthanc with plugins server on docker and clone Viewers.git repo. After run bin/orthancDICOMWeb.sh everythings works except when i open localhost:3000 there are no matches found every time. When I open Docker orhtanc sever via 8042 port and select the image I am able to open it with embedded  viewer. Could you please help me with setup on OS X 10.12 and Windows 10 as well.
Thank you very much!

hi, rangata
i have got the same problem.
could you please tell me how you fix this?
any help will be greatly appreciated.


How to specify distinct in specification query?

Hi @nagtilaklaxman,
What is your use case? Distinct makes sense only in the projections when you're fetching a single column. You can possibly just add DictinctList repository method.

Closed as inactive.

I have problem with this query I have multiple repeated data from query!
`public class DataSetGetByFilterProjectsResultSpec : Specification<DataSet, ProjectViewDto>
{
public DataSetGetByFilterProjectsResultSpec(DataSetFilter filter)
{
Query
.ApplyFiltering(filter);
Query
  .Where(w => w.ProjectId.HasValue)
  .Include(c => c.Project)
  ;

Query
  .Select(ds => new ProjectViewDto(ds.ProjectId!.Value, ds.Project!.Name));

}
}`


Installation file should accept paths including ~ on Linux

If you try to install at ~ it says the path doesn't exist.

This is now fixed and a PR is existing for it.


Create 3-5 generic timelines with milestones

If we provide generic sets of milestones for the 3 to 5 common project types up front this will give them and idea of how to layout their schedule.

Closing in favor of #95.


Columns are not properly sized

In the message view, columns are not properly sized. The Username and the Message are equally wide.
It would be preferrable to have the username in a "slimmer" column

Fix is kinda interim. Columns are properly sized, but the minimum size for the username column seems to get disrespected randomly :( for now this is fixed, further changes should be done via new bug tickets


Expand I2D2 code in new folder structure

I've skipped 2016 due to the recoding and value label issue described in #16 and #17 and #18 , so I will continue with 2018

pausing here, continue with 19, working on issue #19

fixed industry codes, now continue expanding more difficult years to harmonize/append using iecodebook. I'm keeping in the back of my mind what I've learned in feature-data-labels branch from the retroharmonize package -- it may be that the right way forward is actually not to use iecodebook for but actually retroharmonize once I figure out how to do it properly. For the time being, moving along with iecodebook.

returning from diversion in #25:

I know that for now the best thing to do is to use the imperfect workflow of iecodebook, script with dplyr and openxlsx to export to the output from iecodebook, then find and replace with the NA values.

Maybe I should just make a metadata file for all rounds/years and that might help address many issues such at once...

after trying to recode the education codes for the 2017 file, I noticed that it's such a headache that it really would be good to have in the very least a metadata file. Even if I do nothing more than just browse/create label objects with dplyr it would be very helpful for documentation purposes to show how coding schemas of key variables change over time. Branching...

@buscandoaverroes  can the feature-expand-i2d2 branch be removed?


[Request]: elucidisk - disk space visualizer

Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[X] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
elucidisk
Description
Disk space usage visualizer
Homepage
https://github.com/chrisant996/elucidisk
Download Link(s)
https://github.com/chrisant996/elucidisk/releases
Some Indication of Popularity/Repute
Nice compact (both in file size and appearance,) graphical disk space utility. Actively maintained.
Not popular, yet. :grin:

accidentally blew up my repo - might try this at a later date.


docs: link pluto-encrypted as reference implementation

Description:
The current README gives very little info on what's needed for a Pluto implementation
Changes:

mentioned that it's Pluto.Store you need to implement (not Pluto any more)
started a "community implementations" section, and linked pluto-encrypted work

Checklist:

[x] My PR follows the contribution guidelines of this project
[x] My PR is free of third-party dependencies that don't comply with the Allowlist
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked the PR title to follow the conventional commit specification

@mixmix would u also add the storage interface that storages need to comply with ? maybe not in the main readme though

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2722

Relevant Lines:
3613

💛  - Coveralls

@mixmix please re-write your commits by adding a valid GPG signature, all our commits must be signed with a valid GPG key. This PR won't go through if not.
If i re-write it will be as if i did the changes, but you did so...

Hi @mixmix, thanks for the PR, you raise a very good point that Pluto is not documented well enough. I've created an issue here and will try to address this shortly.
I'm not sure if there are rules / restrictions about linking to external repos, I've asked in the hyperledger discord to ensure it's fine :)


Could you tell us how long it took you to train the models, please?

How long did it take to train 256 * 256 a and 1024*1024 CelebA models. How many GPUs and what model are used？

Not an author, but takes me about 6 days on 8 A100s for FFHQ 256


Support `Options' not receive parameters

Codecov Report

Merging #23 (56157ff) into master (c166439) will increase coverage by 0.02%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   89.73%   89.76%   +0.02%
==========================================
  Files          11       11
  Lines         692      694       +2
==========================================
+ Hits          621      623       +2
  Misses         71       71

Impacted Files
Coverage Δ

likeshell/options.py
86.38% <90.90%> (+0.12%)
:arrow_up:

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 c166439...56157ff. Read the comment docs.


Request for Shidoka 'Tabs' component

Is your feature request related to a problem? Please describe.
We have a new UX/UI Design wireframes provided by our UX Designer. As the designer kit has components that are forthcoming in the developer kit, the designer has put forth a wireframe design based on Tabs components.
Describe the solution you'd like
Need Shidoka Tabs component
Describe alternatives you've considered
I haven't worked on a alternative yet but might have to use a tab component from some other lib

React Tabs Component - https://www.npmjs.com/package/react-tabs
Carbon Tabs component - https://carbondesignsystem.com/components/tabs/usage/

Additional context
Sample screenshot of Wireframe

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: adding metadata to org

Adds metadata field to org

addressed comments
metadata fields will be prefixed with meta_


Limit delimiter options

Rather than allowing any input, the delimiter setting should be limited to those supported by the engine. Refer to this documentation.
https://docs.godotengine.org/en/stable/tutorials/assets_pipeline/importing_translations.html

Copying the image for easier reference.


vscode: how to disable fa4/fa5/fa5p/@fa5p snnipet suggestion?

I just want a code suggestion for b4, No code hints for font ICONS. Thanks for a great god

Yes. These compete with more relevant snippets very frequently as they are many and are at top of list.

unfortunately, I stopped to use this plugin because this icon feature :\

Disable bootstrap extension


Optional Icons added to DialogModal title

Closes D2IQ-80933
Icon will be inserted to the left of the title, you can customize the icon size, color, and type.
Note: I am not updating the story here to Controls, as there will be an update coming out soon on Storybook 6.4 that reforms the way to write CSF, will include these updates in the ui-kit chores ticket for updating the stories and controls. This is a rather large story that will have some more complex Controls to add and would be best to be separated out anyway.

Testing

Trade-offs

Dependencies

Screenshots

Checklist for PR creator

[ ] If any new components were added, there are exported from packages/index.ts
[x] If this PR is associated with a JIRA, it is mentioned in commit message footer ("Closes …")
[ ] If this PR contains breaking changes, is stated in commit message body ("BREAKING CHANGE: …")
[x] Info for applicable sections above is provided

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Move ExpressionProperty from botbuilder-dialogs-adaptive to adaptive-expressions package

close: #2089

Move ExpressionProperty from botbuilder-dialogs-adaptive to adaptive-expressions package
Use Expression string interpolation instead of LG template evaluation
Simultaneously modify the references

Pull Request Test Coverage Report for Build 124881

85 of 88 (96.59%)  changed or added relevant lines in 48 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 79.246%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/adaptive-expressions/src/expressionProperties/expressionProperty.ts
2
5
40.0%

Totals

Change from base Build 124759:
0.005%

Covered Lines:
12230

Relevant Lines:
14715

💛  - Coveralls


How to setup minimal app using react-toolbox

Hi, I'm struggling with preparing env (webpack, ...) to make react-toolbox work with minimal app.
I've cloned React Hot Webpack Boilerplate repo.
Installed react-toolbox via npm.
Added resolve into webpack.config.js.
var path = require('path');
var webpack = require('webpack');

module.exports = {
  devtool: 'eval',
  entry: [
    'webpack-dev-server/client?http://localhost:3000',
    'webpack/hot/only-dev-server',
    './src/index'
  ],
  output: {
    path: path.join(__dirname, 'dist'),
    filename: 'bundle.js',
    publicPath: '/static/'
  },
  plugins: [
    new webpack.HotModuleReplacementPlugin()
  ],
  resolve: {
    extensions: ['', '.jsx', '.scss', '.js', '.json', '.md'],
    alias: {
      'react-toolbox': path.resolve(__dirname, './node_modules/react-toolbox/components')
    },
    modulesDirectories: [
      'node_modules',
      path.resolve(__dirname, './node_modules'),
      path.resolve(__dirname, './../node_modules'),
      path.resolve(__dirname, './../components')
    ]
  },
  module: {
    loaders: [{
      test: /(\.js|\.jsx)$/,
      loaders: ['babel'],
      include: path.join(__dirname, 'src')
    }]
  }
};

My App.js
import React, { Component } from 'react';
import Button from 'react-toolbox/button';

export default class App extends Component {
  render() {
    return (

Hello, world.

      <Button label="Hello world" kind="raised" accent />
    );
  }
}

But webpack dev server is reporting error:
ERROR in ./~/react-toolbox/components/button/index.jsx
Module parse failed: /Users/finch/dev/react-toolbox-sample/node_modules/react-toolbox/components/button/index.jsx Line 1: Unexpected token
You may need an appropriate loader to handle this file type.
| import React from 'react';
| import FontIcon from '../font_icon';
| import Ripple from '../ripple';
 @ ./src/App.js 13:14-45
webpack: bundle is now VALID.

What I'm missing here?

Hi @mauron85 , I appreaciate your interest in react-toolbox ... 2 days ago I created a repo with a complete example https://github.com/react-toolbox/react-toolbox-example

@soyjavi Thank you. It is exactly what I was looking for.

Your welcome ;)


Missing BodyType type on matterImage / GameObject for Typescript

Phaser v3.22.0
Excellent work with the typescript types update for matter;  I have run into a type that still seems to be missing.   Accessing the body property of a matterImage object doesn't have any awareness of matterjs BodyType properties, and the compiler throws an error if you try to access the speed property.
Property 'speed' does not exist on type 'object | BodyType | Body | Body'.
  Property 'speed' does not exist on type 'object'.ts(2339)

I tried to add a MatterJS.BodyType type to the phaser body property on the GameObject type manually but it still didnt work.
body: object | Phaser.Physics.Arcade.Body | Phaser.Physics.Impact.Body | MatterJS.BodyType;
I don't really have much experience with manipulating the d.ts files of types, so I could very well be doing something wrong.
line 14834 body: object | Phaser.Physics.Arcade.Body | Phaser.Physics.Impact.Body
I suspect that line needs to be modified with something?

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Bug: Provided .Table() not respected

Hi there, using
Go: go1.12.3 linux/amd64
Postgres: docker image postgres:10.2
Given the following code below, the .Table("features") setting is respected in the Create method, but not in the Read method. To make the Read method work, I had to implement the TableName() method on FeatureFlagModel. The error reported by the postgres database is:
pq: missing FROM-clause entry for table "feature_flag_models"

The code:
type Repository struct {
	db *gorm.DB
}

type FeatureFlagModel struct {
	ID    string `gorm:"column:id;type:uuid;primary_key"`
	Name  string `gorm:"column:name"`
	Value bool   `gorm:"column:value"`
}

func NewRepository(config *postgres.ClientConfig) (*Repository, error) {
	db, err := postgres.ConnectClientWithRetry(config, dbTimeout, dbRetryAttempts)
	if err != nil {
		return nil, err
	}

	return &Repository{db: db}, nil
}

func (r Repository) Create(name string, value bool) (id string, err error) {
	id = uuid.NewV4().String()
	err = r.db.Table("features").Create(&FeatureFlagModel{ID: id, Name: name, Value: value}).Error
	if err != nil {
		return "", err
	}
	return id, nil
}

func (r Repository) Read(id string) (*FeatureFlagModel, error) {
	featureFlag := &FeatureFlagModel{}
	// ERROR! this next line throws error described above
	err := r.db.Table("features").Where(&FeatureFlagModel{ID: id}).First(featureFlag).Error
	if err != nil {
		return nil, err
	}
	return featureFlag, nil
}

I think this is valid processing.
SELECT * FROM `features`  WHERE (`feature_flag_models`.`id` = 'f877b279-2d17-422f-80e0-480e9c36113b') ORDER BY `features`.`id` ASC LIMIT 1

SELECT * FROM features
respected .Table("features").
WHERE (feature_flag_models.id = '...')
not respected .Table("features"), but this query made by .Where(&FeatureFlagModel{ID: id}), it is unrelated to .Table("features").
ORDER BY features.id
respected .Table("features").

I think you might as well implement the TableName(), or .Where("id = ?", id).


NGAME: Negative Mining-aware Mini-batching for Extreme Classification

I am very interested in a lot of work you do in XML, will the code for NGAME be open source

Hi
Thanks for your interest. Yes, we'll be releasing the code for NGAME.

Is there a time point of open source NGAME code?

Yes. We plan to release the code alongside the WSDM conference - early next month.


Stabilize patch-in-config (and prefer config over manifest)

Tracking issue: https://github.com/rust-lang/cargo/issues/9269

This stabilizes the patch-in-config feature (unstable entry) following the discussion in https://github.com/rust-lang/cargo/issues/9269#issuecomment-904913263.
As requested, this PR also changes the precedence behavior such that a [patch] for the same dependency in both .cargo/config.toml and Cargo.toml prefers the patch from the configuration file over the one from the manifest, which matches the behavior of other overlapping configuration options. The corresponding test has also been updated to reflect this change in behavior.

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks @jonhoo!
@rfcbot fcp merge

🎉
@bors r+

:pushpin: Commit e2777ee6aa0c92ac5893e39a01177053bd86b39f has been approved by ehuss

:hourglass: Testing commit e2777ee6aa0c92ac5893e39a01177053bd86b39f with merge 72aee9e8152268d54bb6e23c03d6f2148c5747d1...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 72aee9e8152268d54bb6e23c03d6f2148c5747d1 to master...