'width' attribute NaN

Hello,
Using your basic example I got the following error :

Warning: Received NaN for the width attribute. If this is expected, cast the value to a string.

Here is what I did :

change the children (I did not used createChildren(20) where createChildren is using n => range(n).map(....)
I removed all the placeholder configurations cause I don't need them

Thank you for your help.

I have some news ;
This time I used the same example hat you used in your demo and :

I have the same error
when I remove the placeholder when I access the page where I have the carousel there is no errors but when I ctrl-F5 or F5, the error is displaying.

By the way I'm using
"react": "^16.3.1",
"react-dom": "^16.3.1",

where details.js is my component

PR #24 is related

Should be fixed by #24 , check new version 2.3.7


Consider using yarn instead of npm

I've been consistently frustrated by npm since the beginning of this project due to its obtuse operation, lack of offline support and mysterious failure conditions. Yarn is in my experience much better. It has a --offline flag which is pretty nice. It's meant to build from a cache and be as fast as possible. Perhaps most importantly it has reasonably decent error reporting when it fails.
I'm not sure how much work would be required to get it to work with nix, but I think it should be doable. It has no dependencies, so it should be easy to make a custom derivation for it without needing to hack together a zipped up autogenerated nix expression like back in the npm3 days.
I might take a stab at this and at least get a sense of how feasible it would be.

I think this is a good idea and was thinking it last night. I've used the yarn2nix project in the past and my experience with it was:

yarn is much better suited for use with nix
generating a package set from the yarn.lock file is far more reliable than the package.json
yarn2nix had immature nix build features compared to this project

Ultimately, I could build a large, non-trivial node project involving a giant package set, purescript code, etc. with nixfromnpm and not yarn2nix. So the flexibility you have built into this project is important to maintain.
So, yes, I think using yarn is a good idea and adding a feature to generate packages based on the lock file would be useful too.

 I couldn't figure out how to construct a faux yarn cache directory which yarn would discover packages in, which seems to be the only way to get yarn to use the --offline option, as even if a dependency is satisfied in node_modules it will still look in its cache directory. Not sure if there's a combination of options which would disable this behavior.
I put a pin in that project and instead went another direction, which I just made a PR for...

Closing for now in light of #115


Release/0.32.2

@anuragagarwal561994 could you write changelog for us with changie?

@zhilyaev I have added a pull request to add the changelog

I need to fix the tests , and release will ready

@zhilyaev I have already added MRs for fixing these tests

https://github.com/helmwave/helmwave/pull/762
I think if possible we can also include:
https://github.com/helmwave/helmwave/pull/757
https://github.com/helmwave/helmwave/pull/755
in this release, everything from side seems ready here


django-configuration conflicts with pycharm python console

In pycharm, when run python console:
ImproperlyConfigured: django-configurations settings importer wasn't correctly installed. Please use one of the starter functions to install it as mentioned in the docs: http://django-configurations.readthedocs.org/

May be we should remove this package .
I think django-environ is better, lighter then django-configuration

Hi @daimon99,
Thanks for opening up this ticket. Does this issue occur when you run fab serve or only with pycharm?

With pycharm. When run python-console with pycharm. It is a known issue in django-configuration.
Ref to: "improperly installed" error when run from PyCharm, yet works on command line

Thanks for reporting @daimon99.
Since this is an issue with django-configuration, I'm going to close this ticket. I'm planing to remove that package in #324.


-w parameter

There is something weird about the results here. When I am reading the resulted bigwig file (i2bw file) into my R session, I am getting non-overlapping bins with one position skipped each time.
GRanges object with 5451 ranges and 1 metadata column:
         seqnames             ranges strand |              score
            <Rle>          <IRanges>  <Rle> |          <numeric>
     [1]       XV         [  1, 199]      * |  0.244747966527939
     [2]       XV         [201, 399]      * |  0.857724845409393
     [3]       XV         [401, 599]      * | -0.441242098808289
     [4]       XV         [601, 799]      * | -0.302922338247299
     [5]       XV         [801, 999]      * |  -1.30326700210571

It looks line each time one base is skipped from the counting (200,400,600,etc.)
It is even worse when the -w parameter is set to 150.
GRanges object with 60747 ranges and 1 metadata column:
          seqnames           ranges strand |              score
             <Rle>        <IRanges>  <Rle> |          <numeric>
      [1]        I       [  1, 149]      * |  0.566456496715546
      [2]        I       [201, 349]      * |   0.72018426656723
      [3]        I       [401, 549]      * |  0.222972229123116
      [4]        I       [601, 749]      * | -0.661386787891388
      [5]        I       [801, 949]      * | -0.159120514988899

Here each time 51 bases are skipped.
Is this deliberate?
I hope it is not to confusing, that I show the R output here, as this is how I can visualize my bigwig results.

I have never looked closely at the bigwigs. It is not intentional; I'll try to figure out why this happens, but I do not have the time within the next two weeks. However, the results are still based on data that is overlapping, without any gaps.
Can you show me how you read the data (which libraries and example code) and I'll get around to debugging this sometime in the future.
Thanks for reporting 👍

reading the file with epic
epic --control sizKO.input.0h.1.reads.bedpe --treatment sizKO.ip.0h.1.reads.bedpe -cpu 10 --genome sacCer3 -egf 0.961651807729 -cs Sc.R64.1.genome -b bedFiles/sizKO.InputIP.0h.1.bed -sm matrixFiles/sizKO.InputIP.0h.1.matrix -bw bigwigFiles/sizKO.InputIP.0h.1 -cbw ChipBigwigFiles/sizKO.InputIP.0h.1 --outfile OutputFiles/sizKO.InputIP.0h.1.Output --log logFiles/sizKO.InputIP.0h.1.log -fdr 0.1 -w 150

reading it into R
library(rtracklayer)
ip.file <- import.bw(con = "../EpicResults/sizKO.ip.0h.1.reads_log2fc.bw")

I have never looked closely at the bigwigs. It is not intentional; I'll try to figure out why this happens, but I do not have the time within the next two weeks. However, the results are still based on data that is overlapping, without any gaps.

I am not sure what this means regarding the results i have. If they are overlapping, the positions are completely wrong. if the first bin ends at the 149 and the second begins at 200, I have a gap of 50 bases, if the second bin begins at 150 (or 151 if the structure is consistent), the given positions are shifted 50 bases for each bin. This would cause a huge shift at the end of each chromosome. It still looks like it is based on a 200 bases windows, but calculate the coverage for only 150 bases within this 200 bases each time.

Great catch, I did not see that part! For the -w 150 the results are probably very wrong!

I'll try to find some time to start looking at this tomorrow. It is a serious error.
Endre

I can reproduce the 150-error for paired-end data, but not single-end beds. This error also shows up in the matrix creation (--sm).

I thanked you in the CHANGELOG. Should be fixed in 0.2.5. The problem was simply a hardcoded bin-size. Thanks again for reporting 👍


fix: Daylight: tabs: re-add d2l-tab-panel-text-changed event description

I got the same failed visual diff test on my branch


feat(RingTheory/TensorProduct): Add Algebra.baseChange_lmul

I am not really thinking while writing, but do we have a version with rmul? Anyway thanks!
bors d+

I believe the l in lmul means linear and I don't think there is a rmul.
bors merge


GPON-ONU-34-20BI set PLOAM password QA42 firmware

Hi, many thanks for this useful resource
I'm trying to set the PLOAM/IDONT password, which on my o2/Movistar device reads as 20 characters beginning F4 and ending in 000000.
I suspect this is really hex encoding.
However, for this firmware version I only see a way to input via sip_i2c -i11 -s  10 ASCII characters. How do I input if many characters are non-printeables?
Regards and thanks again

Try to convert hex characters to ascii and those not convertible put in hex like \FF
or try this firmware, https://hack-gpon.org/ont-huawei-ma5671a-fs-mod/


Update LSP6 Permission Guide

Related code snippet:

[ ] https://github.com/lukso-network/lukso-playground/pull/43

Thanks for the feedback. I should not have missed the L16 appearance. Will remind myself to do better self-reviews.
Applied all suggestions, changed naming, and included small outputs into the code 👍🏻
Also updated the related playground code ✅


Maybe the condition is reversed?

rotatelogs.go:332
if rl.maxAge <= 0 && rl.rotationCount <= 0 {
return errors.New("panic: maxAge and rotationCount are both set")
}
Maybe the condition is reversed?

Maybe, but please produce a test.


cant install crouton, new to all this

Please paste the output of the following command here: sudo edit-chroot -all

sudo install -Dt /usr/local/bin -m 755 ~/Downloads/crouton

sudo crouton -t xfce

#### Please describe your issue:

Compiling CRAS (i386)...
In file included from libcras/cras_client.c:47:0:
libcras/cras_client.h:1439:1: error: 'no_sanitize' attribute directive ignored [-Werror=attributes]

cc1: error: unrecognized command line option '-Wno-int-in-bool-context' [-Werror]
cc1: all warnings being treated as errors
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

#### If known, describe the steps to reproduce the issue:

Dup of issue:

Linux not installing saying something like CRAS ·
Issue #4450 · dnschneid/crouton

Closing, please track this issue at the link above.
-DennisLfromGA


Re-word: only "most" ABIs have -unwind

@tmandry How's this? (Sorry, I couldn't find a way to use Git's "suggestion" feature to make this entire change on your existing PR.)

Looks good, thanks!


Getting KeyError: u'CollectionType' in remote.py

Hi,
Here are my version details:

Jellyfin 10.3.7
Mopidy 2.2.3
Mopidy-Jellyfin 0.5.3

my mopidy.conf
[jellyfin]
enabled = true
hostname = 192.168.1.x
port = 8096
username = admin
password = password
libraries = Music

I can play music files just fine from the Music library via Jellyfin web. However in M.A.L.P. nothing shows up upon clicking Files > Jellyfin
Restarting the mopidy service I can see in Active Devices on Jellyfin dashboard the admin user logging in via Mopidy-Jellyfin.
Checking the service log with journalctl -u mopidy show the following error:
Aug 14 18:16:41 qtpi mopidy[30943]:   File "/usr/local/lib/python2.7/dist-packages/mopidy_jellyfin/remote.py", line 243, in get_music_root
Aug 14 18:16:41 qtpi mopidy[30943]:     if library['CollectionType'] == 'music'
Aug 14 18:16:41 qtpi mopidy[30943]: KeyError: u'CollectionType'

Cheers

I've noticed some occasional weirdness in M.A.L.P. where once you get into an artist/album it shows as empty. If you touch the 3 dots in the upper left corner and choose "Show all tracks", they should start showing up in the that folder.  I haven't had time to track down if it's a mopidy-jellyfin issue or a M.A.L.P. issue.
I'll have to look into the KeyError sometime in the next few days.

Sorry for the huge delay, I've been working on the Kodi addon a lot lately.  With the new release I just pushed out yesterday it should fix the KeyError messages in the logs.  If you keep having issues then it's something else we'll have to track down, but closing for now since the primary issue is fixed.


Rename silently fails for renaming a module that's a file name

cc @amiralies
This should either work, or we should put up an error message. Whichever one's safer and simpler.

WorkspaceEditsupports file renames, I think we should use that if that's trivial. I'll take a look proabably this week.

first we need to fix references for file/top-level modules.
https://github.com/rescript-lang/rescript-vscode/blob/98ec36bc0d52dfbaa9ae1d4e32b3e1df7bf29691/analysis/src/References.ml#L467-L468

@amiralies
In examples, references to file-level modules seem to work.
Is there an example showing cases where they don't work?

This is what a reference to a file looks like:
{"uri": "Foo.res", "range": {"start": {"line": 0, "character": 0}, "end": {"line": 0, "character": 0}}}

Perhaps the reference should have a different format?

Trying to get references to a file level module results in an empty array for me.

What does the example look like?

Main.res :
let x = AnotherModule.b

AnotherModule.res:
let b = 3

cmd:
rescript-editor-analysis.exe references src/Main.res 0 15

perhaps you are running definition command?

Thanks you're right.
Added that as an example, to start taking a look:
https://github.com/rescript-lang/rescript-vscode/pull/248

@amiralies in the PR https://github.com/rescript-lang/rescript-vscode/pull/248/files, it finds:
[{"uri": "References.res", "range": {"start": {"line": 0, "character": 0}, "end": {"line": 0, "character": 0}}}]

which is something, but it would try to go rename at position (0,0).
Any idea of what format should represent the entire file?

I guess there isn't really a way to reference an entire file, as that's a peculiarity of ocaml's module system.
And perhaps the best option is to issue an error message.

aside from reference to the file-module itself we should find references in other files to that file, if that's not possible I think it's okay for now to show error message.

aside from reference to the file-module itself we should find references in other files to that file, if that's not possible I think it's okay for now to show error message.

I think the best one can hope for is: in the presence of File.res. Given a use File.foo, when finding all references to File, try to find all the other references.

Done in the updated https://github.com/rescript-lang/rescript-vscode/pull/248.

So if there are several references to the same file, the instances are renamed (but the file is not).


SOLR-16405: Remove unneeded errorprone suppressions

https://issues.apache.org/jira/browse/SOLR-16405
these were introduced in https://github.com/apache/solr/pull/1000

My methodology is super suspect, I checked out main, and looked the java errors in intelliJ, then I checked out this branch, and looked at the java errors in intelliJ.   We went from 27,544 warnings to 27,448 warnings.  So this has my +1.

I think this found a potentially real bug in:
TestDistributedStatsComponentCardinality

   >     java.lang.AssertionError: int_i: relativeErr=0.09722718241315029, estimate=4746, real=5337, p=q=id_i1:[4807+TO+10143]&rows=0&stats=true&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8}int_i&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8+hllPreHashed%3Dtrue}int_i_prehashed_l&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8}long_l&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8+hllPreHashed%3Dtrue}long_l_prehashed_l&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8}string_s&stats.field={!cardinality%3Dtrue+hllLog2m%3D7+hllRegwidth%3D8+hllPreHashed%3Dtrue}string_s_prehashed_l
   >         at __randomizedtesting.SeedInfo.seed([FEAEB9079F935BD9:76FA86DD316F3621]:0)
   >         at org.junit.Assert.fail(Assert.java:89)
   >         at org.junit.Assert.assertTrue(Assert.java:42)
   >         at org.apache.solr.handler.component.TestDistributedStatsComponentCardinality.test(TestDistributedStatsComponentCardinality.java:179)

I think the above error is actually https://issues.apache.org/jira/browse/SOLR-10918

So in commit 02cecd6731874465537e5b688cfcf1d0e60f5503, I finally found a log2m value that gives some accurate enough results for testing. I am beasting this test to make sure all is good.


Support updating lists in different accounts

Given that the Cloudflare tools are moving away from global account IDs, it's probably better to specify account IDs along with list names anyway. If we do specify account IDs, it's then natural to support lists in different accounts.

I would opt for the account_id/list_name format. It provides a clear structure that is easy to understand and use.
The list_name@account_id format is also an option, but account_id/list_name seems more intuitive to me and fits better with common practices.
If you opt to allow using just list_name, it could be a nice feature, provided the necessary permissions are granted and there’s a reliable way to ensure that list names are unique or at least easily identifiable.

@Gamerou Hi, I just want you to know that, after implementing the Happy Eyeballs algorithm for something unrelated (see #883), 1.14.0 will be released. Once it's released, I will be much more reluctant to change the interface due to backward compatibility. Let me know if you want more time to test it, or I will ship it very soon!
To test the development version, use the edge Docker tag.

@Gamerou Hi, I just want you to know that, after implementing the Happy Eyeballs algorithm for something unrelated (see #883), 1.14.0 will be released. Once it's released, I will be much more reluctant to change the interface due to backward compatibility. Let me know if you want more time to test it, or I will ship it very soon!
To test the development version, use the edge Docker tag.

Thanks for the heads-up! I'm really excited about the upcoming 1.14.0 release. No need to wait on my end—I’ll find time to upgrade when I can. Go ahead and ship it!

@Gamerou FYI: it's released! There are still a few rough edges (e.g., #893) but I think the WAF list updating feature is quite usable now.


calibration of sensitivity factor should be forced to 0

The current calibration doesn't go to 0, if use it to get the transient faradic efficiency, which always assume it goes to 0,  it doesn't match. Should force the linear fit to go through zero.

@Caiwu-L , finally solved this, 8 months later (lots of distractions in the meanwhile!). I don't think it should always be forced through zero, but I've now implemented the option. Take a look at the code here (feel free to comment!): https://github.com/ixdat/ixdat/pull/135


Hooked text transformer

This PR faciliates the loading of the text encoder of open clip models into a hooked version, similar to the existing HookedViT class.
The new class is HookedTextTransformer which inherits from HookedTransformer (along with HookedViT). It also required adding attention mask to the attention, mlp, and transformer block modules.
At the moment the way to run this is:
text_model = HookedTextTransformer.from_pretrained(
      "open-clip:laion/CLIP-ViT-B-32-DataComp.XL-s13B-b90K", is_timm=False, is_clip=True, model_type=ModelType.TEXT
).to(DEVICE)
text = tokenizer(["a diagram", "a dog", "a cat"])
text_features = text_model(text)
final_language_output_hooked, language_cache = text_model.run_with_cache(text)

There is an integration test at load_text_model.py in a similar style as the existing test_loading_CLIP-ViT-B-32-DataComp-XL-s13B-b90K.py.
Also added is:

A cosine annealing lr scheduler with warmup in schedulers.py
Changes the test_loading_CLIP-ViT-B-32-DataComp-XL-s13B-b90K.py test file not to have a period in the file name

excellent!


Using absolute positioning, when the parent element has padding, the width height of the current element is incorrectly laid out using a percentage layout

Description
/**
 * Sample React Native App
 * https://github.com/facebook/react-native
 *
 * @format
 */

import React from 'react';
import {
  SafeAreaView,
  ScrollView,
  StatusBar,
  useColorScheme,
  View,
} from 'react-native';

function App(): React.JSX.Element {
  const isDarkMode = useColorScheme() === 'dark';

  const backgroundStyle = {
    backgroundColor: '#eee',
  };

  return (
    <SafeAreaView style={backgroundStyle}>
      <StatusBar
        barStyle={isDarkMode ? 'light-content' : 'dark-content'}
        backgroundColor={backgroundStyle.backgroundColor}
      />
      <ScrollView>
        <View
          style={{
            backgroundColor: '#000',
            height: 300,
            paddingTop: 200,
            paddingLeft: 200
          }}>
          <View style={{
            position: 'absolute',
            // width height 100%会排除父元素的padding
            width: '100%', height: '100%', left: 0, top: 0,
            backgroundColor: '#666'
          }} />
        </View>
      </ScrollView>
    </SafeAreaView>
  );
}

export default App;

In the new architecture, I set the height to 100%, but its height is not equal to the height of the parent container, but will subtract the padding of the parent container
Steps to reproduce
1
React Native Version
0.75.2
Affected Platforms
Runtime - Android, Runtime - iOS
Areas
Fabric - The New Renderer
Output of npx react-native info
System:
  OS: macOS 14.5
  CPU: (16) x64 Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz
  Memory: 35.26 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.20.3
    path: /private/var/folders/rf/gd60z12164z72bny7z_0d4f80000gn/T/xfs-de932bea/node
  Yarn:
    version: 3.6.4
    path: /private/var/folders/rf/gd60z12164z72bny7z_0d4f80000gn/T/xfs-de932bea/yarn
  npm:
    version: 10.7.0
    path: ~/.nvm/versions/node/v18.20.3/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.18034.62.2412.12266719
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.11
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.75.1
    wanted: 0.75.1
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: Not found
  newArchEnabled: false

Stacktrace or Logs
empty

Reproducer
https://github.com/ShaoGongBra/rn-test
Screenshots and Videos
No response

The same code appears inconsistent on the web side and can occupy all the width and height of the parent element

I just closed the new architecture with 0.75.2 and got the same result, which cannot occupy all the height. However, in the previous version, I used 0.71, which was consistent with web performance at that time

hello
i think i can solve this issue but i an new to opensource. Can i start forking this project?

hello i think i can solve this issue but i an new to opensource. Can i start forking this project?

yes

is it available inside the helloworld directory?

is it available inside the helloworld directory?
The reproduced code is just these two sentences

<View
          style={{
            backgroundColor: '#000',
            height: 300,
            paddingTop: 200,
            paddingLeft: 200
          }}>
          <View style={{
            position: 'absolute',
            // width height 100%会排除父元素的padding
            width: '100%', height: '100%', left: 0, top: 0,
            backgroundColor: '#666'
          }} />
        </View>

@ShaoGongBra this doesn't look New Architecture related (at least from your reproducer), right? Like the same behavior happens on both Old and New Architecture

这看起来与 New Architecture 无关（至少从您的复制器来看），对吧？就像旧架构和新架构上都会发生相同的行为一样

yes

I just ran into this issue. The only work around is to wrap all content inside the container with another view and set the padding on that one.
<View style={{ backgroundColor: '#000', height: 300 }}>
  <View style={{ position: 'absolute', backgroundColor: '#666', width: '100%', height: '100%' }} />

  <View style={{ paddingTop: 200, paddingLeft: 200 }}>
    {/* your content here */}
  </View>
</View>


Avoid to crash on outputting log at the early stage config parser

Which issue(s) this PR fixes:
Fixes https://github.com/fluent/fluentd-kubernetes-daemonset/issues/583
What this PR does / why we need it:
By #3352, an early stage config parser is introduced to enable setting
log rotation by system config. While this stage, the global logger
doesn't exist yet, so that it causes crash when the config parser try to
output log. This commit avoid the crash by checking $log. Same logs will
be output at the later stage so that it's not needed in the early stage.
See also: https://github.com/fluent/fluentd-kubernetes-daemonset/issues/583
Docs Changes:
None
Release Note:
Same with the title

I've fount one more case that $log is used while parsing a config file, it's rarely used though.


[task_manager] Schedule task for storage collection

schedule periodic task for storage device collection

/assign Pravin

/move sim


[Snyk] Security upgrade node from 18.16.0-bullseye-slim to 18.17-bullseye-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to node:18.17-bullseye-slim, as this image has only 50 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

621
HTTP Request Smuggling SNYK-UPSTREAM-NODE-5741793
Proof of Concept

614
Prototype Pollution SNYK-UPSTREAM-NODE-5741794
No Known Exploit

614
Denial of Service (DoS) SNYK-UPSTREAM-NODE-5741889
No Known Exploit

514
Buffer Over-read SNYK-UPSTREAM-NODE-5741894
No Known Exploit

514
Denial of Service (DoS) SNYK-UPSTREAM-NODE-5741895
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution
🦉 Denial of Service (DoS)

Codecov Report
Patch and project coverage have no change.

Comparison is base (cb2a5d0) 78.26% compared to head (7315617) 78.26%.

:exclamation: Current head 7315617 differs from pull request most recent head 99a0e3b. Consider uploading reports for the commit 99a0e3b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1584   +/-   ##
=======================================
  Coverage   78.26%   78.26%
=======================================
  Files          32       32
  Lines        2627     2627
  Branches      545      545
=======================================
  Hits         2056     2056
  Misses        405      405
  Partials      166      166

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


gocd-server, gocd-agent: 19.3.0 -> 22.2.0

Description of changes
https://nvd.nist.gov/vuln/detail/CVE-2022-39308
Also added sourceProvenance information as this is a binary package.
I don't really know how to test this beyond the nixosTests.
22.05 only needs to be bumped to 19.11.0 to address the issue.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg test gocd-agent gocd-server

nixosTests.gocd-server appears already-broken:
gocd-server-start[817]: Running GoCD requires Java version >= VERSION_1_8 and <= 12. You are currently running with Java version 13. GoCD will now exit!

Without this I have no idea what I can do to test this package.

@ofborg test gocd-server


Added Graphics.Wpf samples

Added 3 projects

Primitive, tested and running for proving the underlying drawing to a display.
SimpleWPF - original netmf demo program imported. Still to be tested, requires hardware buttons.
Tetris - written by Pavel Bansky, again needs some hardware buttons.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Persian translation to package

First and foremost, I would like to express our gratitude for this amazing package and the valuable contribution it makes to the community.
This commit adds a Persian translation to the package

Thanks!


Belated duplicate messages

The other day a message I sent from the app (v26.0.123, I think, on Android) was sent promptly... and then nearly a week later it got sent again. (This was in PMs with @jainkuniya -- here's a link which he and I can see to look back at any details.)
Similarly, a user reports:

we've had issues where someone will hit their mobile app a couple hours later and all of a sudden it will "resend" the messages it sent before. This is not a frequent problem, but creates some confusion when it occurs.

Some details on the instance I saw:

I believe I was using the app mostly in a test org in the intervening week, and not on chat.zulip.org where this message was. The second send happened shortly after I'd done a "switch account" back to chat.zulip.org.
I wasn't looking at that PM thread at the time. Fortunately the message was harmless (and @jainkuniya immediately guessed that it was a double-post); but if it had caused confusion, I wouldn't necessarily have had any idea it had happened in order to try to clear things up.
The original message is dated (by the server) a little under four minutes after the message it replies to, so that's an upper bound on how long it took the app to successfully send it the first time. IIRC the previous message was already a few minutes old when I sent my reply, so it may in fact have sent immediately.

One thing we should probably do in any case:

[ ] After some time (an hour or two?) that a to-be-sent message has stuck around in the outbox, stop trying to send it. Sometimes late is worse than never (especially for very short messages like "Sure", "no", "OK" that can fit with different meanings into lots of contexts).

An ideal version of this would then have some clear indication to the user that it hadn't sent. But pending that, at some point it's better to silently drop the message than to silently finally send it.

Then in both the case I saw, and the user's report, there's the interesting fact that the message actually had gotten sent in the first place. That may point to a bug we can fix.

See also the earlier issue #2374, which is very similar but without the long intervening delay.


cmd,pkg,test: Move the deploy package down a directory level.

This has always been referred to as pkg/deploy in PR's, and the deploy package makes more sense to me living outside of the operator-specific package code.
This presumably would also require updating any existing deploy-metering local binaries via make bin/deploy-metering.

I don't full understand the PR comment: This presumably would also require updating any existing deploy-metering binaries via make bin/deploy-metering. Does this mean work that should be done in some other PR? It almost sounds like if we merge this PR it does not have enough to allow said binaries to continue to be built correctly?

No, the make bin/deploy-metering would need to be run locally. The deploy-metering CLI tool is also not the preferred way of installing metering over the hack install scripts at the moment (and the CLI option is also not currently documented either), so I'm not too worried about making this change.

/lgtm

Just to be clear, we don't manage any CLI binaries in the operator-metering repo but build them locally when necessary, e.g. make run-reporting-operator-local will build the bin/reporting-operator-local binary.

/retest

/retest


Solve WiFi.status() not updating bug

Added line 162 delay(1000);
Details
The previous bug was caused by the high priority of the RTOS main() task blocking the WiFi status from updating.
Adding a delay could avoid this task.
Upon Wi-Fi is disconnected, a ROM message: RTL8721D[Driver]: no beacon for a long time, disconnect or roaming will be shown in the log.
Attached is the debugging log and returned WiFi.status() details for your reference: https://paste.ofcode.org/rnyEnrhcDSPk9NEu2a274e
typedef enum {
    WL_NO_SHIELD = 255,
    WL_IDLE_STATUS = 0,
    WL_NO_SSID_AVAIL,     // 1
    WL_SCAN_COMPLETED,    // 2
    WL_CONNECTED,         // 3
    WL_CONNECT_FAILED,    // 4
    WL_CONNECTION_LOST,   // 5
    WL_DISCONNECTED       // 6
} wl_status_t;

Thank you.

Hi @S10143806H
I'm sorry I'm out of the country right now and have no access to any board to verify.
Just a concern that this is just a kludge, not final fix because adding the delay(1000) in the loop will slowdown any operation in the future for customer code.
My thinking is that we have to fix in the core, by modifying the tasks' priority or by modifying the delay() to use yield(). I have no way now to verify if yield() will work for the core.
Please investigate and test..
Thanks.

Hi @khoih-prog,
Yes, I do agree that actually decreasing the osPriority in core/main.cpp will also fix the issue of Wifi.status() not updating, for example:
osThreadDef(main_task, osPriorityHigh, 1, MAIN_THREAD_STACK_SIZE);       // osPriorityRealtime -> osPriorityHigh
Attached is the log here: https://paste.ofcode.org/d84b8JneskGWBSagMuVuFW
Besides, I have also tested that using yield() will not fix this issue.
However, the fix mentioned above requires internal discussion since it might affect other codes. Therefore, a by-pass solution was provided yesterday.

HI @S10143806H
I think I'd better to wait until the core is fixed than to merge this temporary PR.
The task priority assignment must be correct and is a precise science, not arbitrary liberal art.
IMO, the WiFi priority must be equal or even higher than normal tasks. People is using RTL8720DN because of 2,4GHz + 5GHz WiFi, and without this best WiFi working normally, I'm afraid people will select other boards, such as ESP32, etc.
Also check https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/main.cpp#L23-L32 to see if you can use something similar to ESP32 in cores/arduino/main.cpp
#if CONFIG_FREERTOS_UNICORE
void yieldIfNecessary(void){
    static uint64_t lastYield = 0;
    uint64_t now = millis();
    if((now - lastYield) > 2000) {
        lastYield = now;
        vTaskDelay(5); //delay 1 RTOS tick
    }
}
#endif

I had to use the similar quick-and-dirty fix before, and really don't like it as I haven't spent time enough then
Check these kludges (there are many more, only necessary for RTL8720DN !!):
https://github.com/khoih-prog/WiFiWebServer/blob/master/examples/WiFiMulti/AdvancedWebServer_WiFiMulti/AdvancedWebServer_WiFiMulti.ino#L386-L389
#if defined(CONFIG_PLATFORM_8721D)
  // Important delay() for RTL8720DN
  delay(200);
#endif

Hi @khoih-prog ,
Thanks for your information.
Upon internal discussion, we agreed to modify the priority of the main task to a lower priority osPriorityNormal
typedef enum  {
  osPriorityIdle          = -3,          ///< priority: idle (lowest)
  osPriorityLow           = -2,          ///< priority: low
  osPriorityBelowNormal   = -1,          ///< priority: below normal
  osPriorityNormal        =  0,          ///< priority: normal (default)
  osPriorityAboveNormal   = +1,          ///< priority: above normal
  osPriorityHigh          = +2,          ///< priority: high
  osPriorityRealtime      = +3,          ///< priority: realtime (highest)
  osPriorityError         =  0x84        ///< system cannot determine priority or thread has illegal priority
} osPriority;

Meanwhile, could you kindly please raise an issue in ambd_arduino so that we could officially fix it? I am doing some tests to ensure other examples might not be affected by this bug fix.
Thanks again.

Done. The bug report is at WiFi.status() not updating bug #87


Symbol for :: (cons)?

I have the blank square issue, but I'm using the recommended font (DejaVu Sans). Which fonts have the "cons" symbol? Is it possible to disable this symbol only?
Thanks,
Jonathan

Mine comes from Symbola. You can selectively disable symbols using M-x customize-variable RET fstar-symbols-alist RET. Maybe I should just remove that symbol.

This Symbola? https://github.com/zhm/symbola/tree/master/fonts

This one: http://shapecatcher.com/downloads/Symbola706.zip

Thanks. I've edited the README.md to point to the right symbola font (there were quite a few different results from a search engine).


[flash_ctrl] HWIP Technical Specification - Idle indication to pwrmgr

The idle indication to pwrmgr should be described. Interface is missing as well.

thanks for the issue! i will address this today!
I know we discussed this in a PR the other day, but I had already forgotten!


Support multiple matches in Match() function

Hello everyone,
I´m new to google refine and regex, but I´ve tried them a couple of times and I introduced a question to the open refine group yesterday that seems to have no solution. So, Tom Morris suggested that I could open a new issue here and here we go.
The problem was (as always lol ) simple: I developed a regex to mach proper names in brasil. In a regex tester, it matches almost everything. So, I thought, is just a matter of putting the regex inside a value.match and put it all inside a forEach statement. As the value.match is already a function that generates an array, the forEach is unnecessary (thanks again Tom Morris). So, I ended with this:
value.match(/.(([A-Z][a-z]\s[A-Z][a-z][áãâéêíóôõú][a-z])+).*/).join("|")
So, in the phrase: "The ex-president Luiz Inácio is not Marcos Valério friend, senator."
It was expected as a result: Luiz Inácio|Marcos Valério
But it just gives me: Marcos Valério|Marcos Valério
So, I think we have a big issue here.
I don´t know if I can give suggestions here, but here they go (humble):
A button "Extract the matches" inside the Edit Column > Split into several columns... dialog would do the entire magic. Instead of creating columns with the content on left and right my matches, refine would create the columns with the matches. For example:
"The ex-president  Luiz Inácio is not Marcos Valério friend, senator."
Using Split into, with regex .(([A-Z][a-z]\s[A-Z][a-z][áãâéêíóôõú][a-z])+).*
It returns three columns:

The ex-president
is not
, senator

With a possible new feature in the future:

Luiz Inácio
Marcos Valério

I would consider myself as a truly Gandalf with this...lol
So, this is it. Next I explain the motivation for this regex, but maybe not everyone will be interested so, if this is the case, I appreciate your time and willingness to help and improve this wonderful software.
MOTIVATION:
Since I started in a new project to scrape political news from some sites for text analysis purposes, I realized that if I simply split the text on periods (to form phrases) and then split again using the blanks to take only the words and perform a basic count (faceted text), I would end with many words representing first and last names that were supposed to be together.
So, I thought that identifying proper names with a regex would allow me to identify most names automatically.
With the names of each news in a new column separate by a pipe, it would be possible to split into several columns with the pipe separator. After that, I would transpose from columns to rows and then and then create a text facet to obtain a list of names. Associating a single word code to each name is easy  (Luiz Inácio --> LI0001; Marcos Valério --> MV0001, and so on), and with all names coded, I would mass edit the original column transforming all names into single word codes, split into several columns by the period (".") character, transpose and than split it all again with the "space/blanks". Transpose again and voilá! Text faceting would reveal the most common words in the news and open the way for a more complex text analysis.
Thank you very much for reading the motivation, I think it can be the issue of many people. The link for the original Open Refine list is this:
https://groups.google.com/forum/?fromgroups=&hl=pt-BR#!topic/openrefine/JemOUierAPs
Leonardo Paris.

I think the new find function solves this problem. E.g. you can extract the two names using:
value.find(/[A-Z][a-z]+\s[A-Z][a-z]+[áãâéêíóôõú][a-z]+/)
@thadguidry @ettorerizza do you agree?

@ostephens Yeap !  That use case is one of the reasons for us adding the new find() !
I'll add this example to our Recipes.
To find Proper Names using the full Latin Character set would be:
value.find(/([A-Z]([a-z]|[\u00C0-\u00FF])+)\s([A-Z]([a-z]|[\u00C0-\u00FF])+)/).join(" | ")

Recipe added:  https://github.com/OpenRefine/OpenRefine/wiki/Recipes#finding-proper-names-or-people-names-in-a-string


potentially missing functionality: title, config, and filtering-expressions

Here's a couple of examples from the VegaLite gallery that I couldn't encode (I'm reasonably sure these are missing features, but perhaps I've missed something).  I can split this into 3 tickets if you'd prefer.
*) Line Charts Showing Ranks Over Time
I couldn't set title to an object, just a string.
  "title": {
    "text": "World Cup 2018: Group F Rankings",
    "frame": "bounds"
  },

*) Mosaic Chart with Labels
I think ConfigurationProperty is missing a concat option:
  "config": {
    "view": {
      "stroke": ""
    },
    "concat": {"spacing": 10},
    "axis": {
      "domain": false,
      "ticks": false,
      "labels": false,
      "grid": false
    }
  }

*) Table binned Heatmap
This one I can get to work, by manually implementing a "valid" filter, but it is messier than having support in the API. However, this looks like a potentially-invasive change.
The boolean operators expect strings, so we can't combine fiValid with and
  "transform": [{
    "filter": {"and": [
      {"field": "IMDB_Rating", "valid": true},
      {"field": "Rotten_Tomatoes_Rating", "valid": true}
    ]}
  }],

I forgot the following, but it's perhaps less-important (since you can always provide a data source as a JSON object rather than with dataRow etc):
*) Line Chart with Markers and Invalid Values
I experimentally added a NullValue constructor to DataValue, but I am not 100% happy with this choice, to be able to represent the following data:
    "values": [
      {
        "x": 1,
        "y": 10
      },
      {
        "x": 2,
        "y": 30
      },
      {
        "x": 3,
        "y": null
      },
...

Thanks very much for these (v. helpful). Would you mind splitting these into 4 separate issues as I think the solutions/discussion around each may be different?

I've separated them out (linking each back to this for context), so am going to close this ticket.


Build failed

When I try to build with make in macOS 10.14.4, got following errors:

cc src/lib/firmware/fbsd.c
src/lib/firmware/fbsd.c:690:7: error: implicit conversion changes signedness: 'unsigned int' to 'int' [-Werror,-Wsign-conversion]
case MSR_EFER:
^~~~~~~~
In file included from src/lib/firmware/fbsd.c:80:
src/include/xhyve/support/specialreg.h:771:18: note: expanded from macro 'MSR_EFER'
#define MSR_EFER        0xc0000080      /* extended features */
^~~~~~~~~~
1 error generated.
make: *** [build/lib/firmware/fbsd.o] Error 1

getting the same error with my mac that is also on 10.14.4

I think this is related to #246 which has a PR attached to it that supposedly fixes this issue.  I'm anxiously awaiting for that PR to be merged :)

#248 has been merged and I think fixed this, thanks,


[Due 2023-11-01] Deprecate altool – use notarytool instead

Coming from https://expensify.slack.com/archives/C07J32337/p1686756113697079
Problem
We are using a deprecated tool for code-signing.
Why this is important
If we don't stop doing this by November 1st then app deploy will stop working.
Solution

Figure out where exactly we're still using altool (it's inside some tool we're using, I don't think we're using it directly)
Use notarytool instead

No update

I've looked through briefly and I think this is only used in Fastlane (for signing the iOS app)? It shouldn't be used for electron: https://github.com/Expensify/App/blob/442ed0c511acb8ab6340374a7a2f4430ba36084f/desktop/notarize.js#L13

No update. I spent a bit of time and am still not confident where or how we are using altool.

No update

No update

No update

No update, no update planned soon. I can't find where this is used so I'm going to close this until it actually affects us. AFAIK there's no problem here.


Default yarn command to install types doesn't seem to work with yarn 3.2.0

Hi frens :)
First timer with Thin, really looking forward to testing it!
Running into some difficulties to run the following command in my NextJS project:
yarn add --dev "https://thin.dev/types/....../......."
Getting:
Usage Error: It seems you are trying to add a package using a https:... url; we now require package names to be explicitly specified.
I haven't found the right yarn command yet. I'm using yarn 3.2.0.
Can anybody help?
Thanks and cheers!

How did you solve this?


Uncaught error during the computation of ComputedObservable

Using ReactNative 19.0
My store:
import {observable} from 'mobservable';

const counterStore = observable({
  counter: 0,
  increment: function () {
    this.counter++;
  },
  decrement: function() {
    this.counter--;
  }

});
module.exports = counterStore;

My JSX Render
import {observer} from 'mobservable-react/native';
////
let CounterUI = observer(React.createClass({
  _increment() {
    this.props.store.increment();
  },
  render() {
    return (
        <View style={styles.container}>

        <TouchableHighlight onPress={this._increment}>
           <Text style={styles.text}>|   +   | </Text>
        </TouchableHighlight>

        <Text style={styles.text}>Clicked: {this.props.store.counter}  times</Text>
        </View>
    );
  }

}));

The exception

[mobservable.view '.increment'] There was an uncaught error during the computation of ComputedObservable[.increment (current value:'undefined')] function (){
this.counter++;}reactConsoleError @ ExceptionsManager.js:76console.error @ YellowBox.js:49ViewNode.computeNextState @ dnode.js:204ViewNode.wakeUp @ dnode.js:168ObservableView.get @ observableview.js:37Object.defineProperty.get @ observableobject.js:57_reactNative2.default.createClass._increment @ CounterUI.js:33React.createClass.touchableHandlePress @ TouchableHighlight.js:168TouchableMixin._performSideEffectsForTransition @ Touchable.js:689TouchableMixin._receiveSignal @ Touchable.js:607TouchableMixin.touchableHandleResponderRelease @ Touchable.js:395invokeGuardedCallback @ ReactErrorUtils.js:27executeDispatch @ EventPluginUtils.js:79executeDispatchesInOrder @ EventPluginUtils.js:102executeDispatchesAndRelease @ EventPluginHub.js:43executeDispatchesAndReleaseTopLevel @ EventPluginHub.js:54forEachAccumulated @ forEachAccumulated.js:23EventPluginHub.processEventQueue @ EventPluginHub.js:259runEventQueueInBatch @ ReactEventEmitterMixin.js:18ReactEventEmitterMixin.handleTopLevel @ ReactEventEmitterMixin.js:34merge._receiveRootNodeIDEvent @ ReactNativeEventEmitter.js:120merge.receiveTouches @ ReactNativeEventEmitter.js:205__callFunction @ MessageQueue.js:183(anonymous function) @ MessageQueue.js:87guard @ MessageQueue.js:41callFunctionReturnFlushedQueue @ MessageQueue.js:86onmessage @ debuggerWorker.js:39

Hi,
You are trying to make an observable from a function that doesn't really return a value but modifies the state instead. observable should only be used to create observable values or derivable values, but not to change values. So your store should be like:
import {observable} from 'mobservable';

const counterStore = observable({
  counter: 0
});
counterStore.increment = function () {
    this.counter++;
}
counterStore.decrement = function() {
    this.counter--;
};

module.exports = counterStore;

Note btw that the _increment handler should be bound to the component instance in your JSX component (by using this._increment.bind(this) or better, define _increment as follows: _increment = () => { .. }

Note btw that the _increment handler should be bound to the component instance in your JSX component (by using this._increment.bind(this) or better, define _increment as follows: _increment = () => { .. }

I don't follow....like this:
  <TouchableHighlight onPress={this._increment.bind(this)}>
           <Text style={styles.text}>|   +   | </Text>
        </TouchableHighlight>

Apologies, that is not needed indeed, since you are using React.createClass. Binding is only needed when using ES6 component classes.

So I think this is what you wanted me to do?
 <TouchableHighlight onPress={() => this.props.store.increment()}>
          <Text style={styles.text}>|   +   | </Text>
  </TouchableHighlight>

Thanks!  works great...love it.

@bartonhammond the onPress was fine just as it was :) I assumed to quickly that it was an ES6 React.Component based class which would need the binding.


An error occurred at 1:36 a.m., which I couldn't correct

Hi, there was a crash at 1:36 AM, and PM2 restarted everything right after. Here's the error:
TypeError: Cannot create property 'mobile' on boolean 'false'
    at ClientReconnect.init (/home/debian/node/jbnc/lib/ClientReconnect.js:22:54)
    at new ClientReconnect (/home/debian/node/jbnc/lib/ClientReconnect.js:8:10)
    at Server.clientReconnect (/home/debian/node/jbnc/lib/Server.js:734:16)
    at TLSSocket.<anonymous> (/home/debian/node/jbnc/lib/Server.js:331:58)
    at TLSSocket.emit (node:events:518:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23)

dragon||AJFiej2fn2345/irc.dal.net:6667||somepassword/mobile
or
dragon||AJFiej2fn2345/irc.dal.net:6667/mobile
In the entire code, I can't find any occurrence of the word 'mobile'; it seems to be related to the PASS command.
If I can't find it, I'll have to uncomment 'uncaughtException,' which is a shame because it's the first time it has crashed and there's only one mobile error. Rare bug.

That's very confusing. Is the client connecting somehow with 'mobile' in their connection?

I would try to reproduce the bug by replacing "dragon||AJFiej2fn2345/irc.dal.net:6667||somepassword/mobile" with "mobile" and making other changes to test it. Otherwise, I don't know how to reproduce the bug. Maybe I need to bring the computer and then connect with the mobile; I will also try that.

I manage to reproduce the bug. Just do this:

Open 2 different JBNC servers, one with a connected nickname (example: Testor on the dedicated server)
Then turn on your JBNC on the PC and also put "Testor" it will say that your nickname is already taken, do not disconnect, it will put this:

[03:37:45] * Connecting to <IP_ADDRESS> (8888)
[03:37:45] -irc.d.com- *** Looking up your hostname...
[03:37:45] -irc.d.com- *** Found your hostname (cached)
[03:37:45] Le nickname (Carly) est déjà utilisé.
[03:37:45] Le nickname (.) est erroné.
[03:37:45] You are now logged in as Carly.
[03:37:45] SASL authentication successful
[03:37:45] -*jbnc- *** No CAPabilities available. ***
[03:37:45] -*jbnc- *** No CAPabilities available. ***

Wait a few seconds for it to make this error:
[03:38:16] Closing Link: [<IP_ADDRESS>] (Registration Timeout)
[03:38:16] * Disconnected
[03:38:17] * Connect cancelled

Immediately log in, and there is this error:
[03:39:09] * Connecting to <IP_ADDRESS> (8888)
[03:39:09] * [10053] Software caused connection abort
[03:39:09] * Disconnected

and :
C:\Users\Mcdo\Documents\GitHub\jbncdev\lib\ClientReconnect.js:22
        this.connection.buffers[socket.clientbuffer] = { data: '', connected: true, privmsgnotice: [] };
                                                     ^

TypeError: Cannot create property 'mobile' on boolean 'false'

The error seems to be fixed. I've deployed it to production for tomorrow. I'll check if everything's okay : https://github.com/freenode/jbnc/pull/79/commits/a550d20ec9f3b70e5c3c07cdec44b909d0c3a025

The error recurred last night, 3 times in 5 minutes, presumably by the same user. The error even triggered again this morning at 6:05 AM, where it seems to have occurred 5 times in 20 minutes, likely by the same user, but there's no certainty if it's the same user as yesterday.
The error resembles what I fixed here https://github.com/freenode/jbnc/pull/79 for the events 433 ERR_NICKNAMEINUSE and 451 ERR_NOTREGISTERED. It just requires placing a "connection end()" in the right place.
I'm not familiar with many events where "end()" would need to be placed, except for:

At the time of quit (/quit)
ERROR (kill etc.)
433 ERR_NICKNAMEINUSE
451 ERR_NOTREGISTERED
The "gone" with the settimeout()

It seems like "end()" should be placed in a location where the connection has been registered in the array, but in reality, the client disconnected before or couldn't establish the connection, yet it remains registered in the "array" without being able to connect to it.
The error could also occur due to something like:

Forbidden nickname
Error in /PASS (incorrect or missing)

It could also be a missing raw in jbnc.
Here's the current error:
1714449963 # Serious problem (uncaughtException) - this should not happen but the JBNC is still running. TypeError: Cannot create property 'mobile' on boolean 'false'
    at ClientReconnect.init (/home/debian/node/jbnc/lib/ClientReconnect.js:24:54)
    at new ClientReconnect (/home/debian/node/jbnc/lib/ClientReconnect.js:8:10)
    at Server.clientReconnect (/home/debian/node/jbnc/lib/Server.js:739:16)
    at TLSSocket.<anonymous> (/home/debian/node/jbnc/lib/Server.js:335:58)
    at TLSSocket.emit (node:events:518:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23)

The bug occurs when reproducing the following steps:

Change the nickname (for example, Nick|AFK).
Then, the user gets g-lined. The error occurs as soon as they attempt to reconnect. Furthermore, when trying to reconnect, it doesn't work immediately; they have to refresh the web page with the applet.

The bug does not occur if the person has reverted to their initial nickname, which is Nick before getting gline.


[hotfix] Block button clicks when loading or disabled

Is there any case I'm missing where we'd want to register clicks even when a button is Loading or Disabled?

This looks good to me; I can't imagine why we would want to register the button click if a button is disabled or in the loading state.


Introduce AssertionsEngine's formatter to BrowserLibrary (#1645)

Closes #1645

Failures in CI are not related to your changes.

Let me know if there is anything I can do to help
As for the code itself, is everything looking ok?

Closed due #1663


Fix GH-41226

When not drawing a border, the mGapBetweenPaths adjustment can create noticable pixelation when drawing curves through a low number of pixels.  This is noticable mostly on buttons and such on low-dpi devices.  This fix only applies the fix if clipping for the border radius is done.

Summary:
When drawing small radius rounded backgrounds (e.g. to draw a circle or button) we see visible pixelation. This is particularly noticable on low DPI devices.
Changelog:
[ANDROID] [FIXED] - Don't use mGapBetweenPaths if not drawing a border
Test Plan:
Built an android app that directly uses CSSBackgroundDrawable to draw a background and verified repro of this issue.

Then modified the code according to this PR and verified that anti-aliasing is appropriately applied

@knappam Can you link the issue on your summary and make the title be descriptive of what your change is?

@knappam Can you link the issue and make the title be descriptive of what your change is?

done, thanks!


No .txt file is created

I annotate an object and and I press submit, after that the program prints the path of the image.txt. I close the program, and there is no text file whatsoever. Not sure what's going on?

Make sure you can write in that folder of images, it was a permission problem.


spawns

there aren't any pokemon spawns that are visible to me anymore i cant find any pokemon .

Hello! This project doesn't deal with any spawn data directly, it's only the codebase. You'll need to take up your grievance with your local admin. Thanks!


Document downgrade compatibility and instructions

We should first document the current steps, and then decide if this requires additional development work to get to a process we are happy with.
We should also document how to know if it is "safe" to do a particular downgrade/upgrade".

We need it to add to CI as well


Improve Dockerfile

Union argument layers and copy source code.
Changed layers order final image for grow cache hit

Please, rebase branch on top of main branch for api/ and internal/controller directories to appear, so that build won't fail on not found error


Add custom parser to DragValue and Slider, with additional support for binary, octal, and hexadecimal numbers.

Closes https://github.com/emilk/egui/issues/1953.
Adds a custom_parser method to DragValue and Slider, along with extra binary_u64, octal_u64, and hexadecimal_u64 which overwrite the custom_formatter and custom_parser.
The purpose of custom_parser is to convert the text input from both widgets to a f64. A parser function returns an Option<f64> set to Some when parsing succeeds or None when it fails.
Additional methods for different number systems all take in a min_width parameter that specifies the minimum number of displayed digits. If the number is shorter than min_width, it will be prefixed with additional 0s to match it. hexadecimal_u64 also takes in a bool which specifies if in the output text alphabetic digits should be uppercase or not.
Demo (apologies for quality):

Code for the demo:
ui.add(
    egui::Slider::new(&mut self.age, 0..=1023)
        .text("bin")
        .binary_u64(10),
);
ui.add(
    egui::Slider::new(&mut self.age, 0..=1023)
        .text("oct")
        .octal_u64(4),
);
ui.add(
    egui::Slider::new(&mut self.age, 0..=1023)
        .text("hex")
        .hexadecimal_u64(3, false),
);
ui.add(
    egui::Slider::new(&mut self.age, 0..=1023)
        .text("HEX")
        .hexadecimal_u64(3, true),
);
ui.add(
    egui::Slider::new(&mut self.age, 0..=((60 * 60 * 24) - 1))
        .text("time")
        .custom_formatter(|n, _| {
            let n = n as i32;
            let hours = n / (60 * 60);
            let mins = (n / 60) % 60;
            let secs = n % 60;
            format!("{hours:02}:{mins:02}:{secs:02}")
        })
        .custom_parser(|s| {
            let parts: Vec<&str> = s.split(':').collect();
            if parts.len() == 3 {
                parts[0]
                    .parse::<i32>()
                    .and_then(|h| {
                        parts[1].parse::<i32>().and_then(|m| {
                            parts[2]
                                .parse::<i32>()
                                .map(|s| ((h * 60 * 60) + (m * 60) + s) as f64)
                        })
                    })
                    .ok()
            } else {
                None
            }
        }),
);

May be return a Result<T, String> where the Err variant will represent a text that can be shown to the user why his input is incorrect? For now egui can just use .ok() and not shown anything, but in the future it could.

May be return a Result<T, String> where the Err variant will represent a text that can be shown to the user why his input is incorrect? For now egui can just use .ok() and not shown anything, but in the future it could.

If we do want this, I think Result<T, Option<String>> would be better as it'd give you the option not to have an error message.

Applied suggested changes, also added a twos_complement flag to give the choice whether you want to display negative numbers with a - sign or in the 2's complement representation.

Would be awesome to add thousands separator,  e.g. 1,000,000.0

Would be awesome to add thousands separator, e.g. 1,000,000.0

@enomado Right now you can do it with custom_formatter I added in #1851, it's available in v0.19 :smile:
use num_format::{Locale, ToFormattedString};
ui.add(DragValue::new(&mut n).custom_formatter(|n, _| n.to_formatted_string(&Locale::en)));


Inconsistent detection with or without protocol

I have noticed that the link detection is somehow different when the link starts with https:// or sth like that, is it done on purpose?  Like the image below, when the link starts with https://, the numbers at the end are also considered part of the link.

Hi @TongLing916, yes this is by design. Linkify considers everything following a protocol http://... to be part of the URL (except for whitespace and a few other special characters). This is to support exotic custom or domain-specific URLs - there could conceivably be an environment where http://www.google.com123 resolves to a valid URL.
However, when a protocol is not included, Linkify can only detect more conventional URL styles such as those that end in .com in order to avoid false positives.
Hope that clarifies it.


added practical tasks blogpost using loops, arrays and objects

👷 Deploy Preview for cg-eleventy-blog processing.

Name
Link

🔨 Latest commit
20be78eadf89cc4935d9a6d8e4569a1a5b39d0ac

🔍 Latest deploy log
https://app.netlify.com/sites/cg-eleventy-blog/deploys/65f6509f8e808e00080e0fa6


Go SDK fails to install with dependency error

Install Go SDK fails with the following error:

go get -u github.com/Azure/azure-sdk-for-go/...
package go.opencensus.io/exemplar: cannot find package "go.opencensus.io/exemplar" in any of:
C:\Go\src\go.opencensus.io\exemplar (from $GOROOT)
C:\Users<user>\go\src\go.opencensus.io\exemplar (from $GOPATH)

This is a known upstream issue, please see https://github.com/census-instrumentation/opencensus-go/pull/1067#issuecomment-474279057.
Is there a reason you're not using dep for package management?

Thanks for your quick response @jhendrixMSFT
There is no technical reason. Go SDK document gives two options to install Go SDK and at the time we wrote our tests we chose to use Git instead of dep to install the SDK (mainly because we have Git installed on our environments and didn't want to install additional tools).
I can see the upstream fix is being merged 6 hours ago. But issue still repros on our end. Any ideas why?

@seyadava, the PR with a fix was merged 20 minutes ago (or 6 minutes before you posted your comment).

https://github.com/census-instrumentation/opencensus-go/pull/1071 should temporarily fix this. Sorry for the inconvenience.
If you're using Go1.11+, maybe consider pinning a released version of go.opencensus.io, for example https://github.com/census-ecosystem/opencensus-go-exporter-stackdriver/blob/ab5a58af316a529613aadf9f50eeed1b6f044b2f/go.mod#L11.

@songy23 thanks for the update.
@seyadava I highly recommend using dep to manage your dependencies to avoid such issues.  It's what we test with so everything should always just work.

Closing as the fix has been made upstream.


Encode h.265: Multiple sessions in a serie

In a .cpp project I wrote a class based on simple_encode_hevc10.cpp to encode images to a video stream.
In the first movie all frames are written as expected to the video stream.
Once the HW is initialized the system should write other videos in a sequence.
For doing that I use mfxENC again. But this fails with errors from the Hardware.
For me it is important to initialize the GPU (Kaby Lake) only once because of the 1s it uses to initialize.
My application is implemented this way:

init(): - to initialize the HW
start(fileName): - to start a new stream
writeImg(img): - to encode new frames
stop(): - to flush buffers and cleanup

The sequence would be: init, start, writeImg (100x), stop, start, writeImg (100x), stop, start, writeImg (100x), stop, ...
System information

CPU i7, Debian x64
GPU Kaby Lake /dev/dri/render128

Issue behavior
Describe the current behavior

Using existing handler mfxENC for the next move ends with errors from HW.
Allocating another mfxENC not work neither.

Describe the expected behavior

After cleaning up the session correctly whith mfxENC.Close() it should be possible to reuse the interface.

I report this to be a bug because the system should be able to write several videos after initialization. But it can also seen as a new feature.
I would be very glad to get some support from the experts. Best regards, Chris

It would be great to see source code or at least have API traces (https://github.com/Intel-Media-SDK/MediaSDK/tree/master/tools/tracer).
But generally I guess the issue is when you create the other mfxENC you need the other mfxSession at first. One session can have one decoder, one VPP and one encoder.

Thank you. Please find attached the trace log and class I wrote. In the stop() routine the cleanup is inactive because the handler will be used for the next file initialized with start().
traceAndSrc.zip
Issue behavior

At the 2nd call of sts = m_mfxENC->Init(&m_mfxEncParams); the method returns MFX_ERR_UNDEFINED_BEHAVIOR.

I looked at the code and logs.

It seems logs don't match to the cpp file. It's because I code I don't see Encode::Close but it's present in logs.
In logs Close returned MFX_ERR_DEVICE_FAILED but you're writing about MFX_ERR_UNDEFINED_BEHAVIOR
Leaving aside above, no clue of the issue looking at logs. You're doing 2nd Init with exactly same params at in 1st run.

Could you add #define MFX_DEBUG_TRACE at line https://github.com/Intel-Media-SDK/MediaSDK/blob/0db73aa1b2dec5b61900ee54db3fa574de6d1752/_studio/shared/include/mfx_utils_defs.h#L28
rebuild mskd library and re-run the use case? With such define you'll get logs where an error was generated in msdk soure code. Let's take a  look.

I went trough your points:

The Method MFXClose is called in the destructor of the class MFXVideoSession.
The 2nd call of m_mfxENC->Init() returns MFX_ERR_UNDEFINED_BEHAVIOR.
Yes, you are right, I use the same parameters. I assume that Init() is required to initialize a new video file. I run a test that calls Init() only once. See Behavior with single Init()

New trace with code from 23.12.21
I added the MFX_DEBUG_TRACE and got the following output:
libva info: VA-API version 1.14.0
libva info: Trying to open /usr/local/lib/x86_64-linux-gnu/dri/iHD_drv_video.so
libva info: Found init function __vaDriverInit_1_6
libva info: va_openDriver() returns 0
 Undefined behavior. ../imageProcessor/videoio/videoEncoderHvec10.cpp 265
../imageProcessor/test/videoWriter_test.cpp:95: Failure

mfxtracer_32274.log
Behavior with single Init()
singleInit.zip

I don't get any error
The 2nd video file is smaller and cannot be extracted by ffmpeg. Maybe the header is missing...

I wondered what steps are required to start another video file.
I wondered if should be used for every file CloneSession() - but the method does not handle a MFXVideoSession

The Method MFXClose is called in the destructor of the class MFXVideoSession.

I meant MFXVideoENCODE::Close(). I don't see you call it (at least looking at videoEncoderHvec10.cpp you shared before). It confuses me and makes me think you shared not exactly the same cpp/h file on which you experiment.

The 2nd call of m_mfxENC->Init() returns MFX_ERR_UNDEFINED_BEHAVIOR

Yes, now in log file mfxtracer_32274.log it's present. But in this log file there's no MFXVideoENCODE::Close call :) . Now it matches the code but seems the app has an issue: Instead of making Init/Close/Encode../Init/Close, app is doing  Init/Encode../Init

I added the MFX_DEBUG_TRACE and got the following output:

Thanks. But my point was you would rebuild libmfxhw64.so.1.35 and replace it on the system. Lack of console logs means you probably didn't replace the library.

The 2nd video file is smaller and cannot be extracted by ffmpeg. Maybe the header is missing...

Yes, no headers. It's because a valid stream shall start from IDR frame and has SPS/PPS in front of it. It's called Random Access Point.
To avoid confusion I mentioned above it would be great if you can share code which I can successfully compile and run on my side. If possible please upload to some sandbox github project. In this case, it would be simpler to point to exact lines of code which confuses me.
If you can't share it's also okay. In this case please just make sure your app is doing these sequence:

create vaDisplay
Create MFXVideoSession
Call SetHandle(session, vaDisplay)
Allocate surfaces
Create MFXVideoENCODE object
Init MFXVideoENCODE object
Encode several frames
When doing stop, drain encoder by calling EncodeFrameAsync with nullptr
Close MFXVideoENCODE object,
Repeat 6-9
...
Destroy MFXVideoENCODE, MFXVideoSession objects and surfaces.

Discussed offline. Fixed on app side.


something wrong start fabric-docker-compose-svt

git clone https://github.com/onechain/fabric-docker-compose-svt.git
mv fabric-docker-compose-svt $GOPATH/src/github.com/hyperledger/fabric/examples/
cd $GOPATH/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt
./download_images.sh
./start.sh
2017-08-04 01:53:19.078 UTC [msp] getMspConfig -> INFO 001 intermediate certs folder not found at<EMAIL_ADDRESS>Skipping.: [stat<EMAIL_ADDRESS>no such file or directory]
2017-08-04 01:53:19.078 UTC [msp] getMspConfig -> INFO 002 crls folder not found at<EMAIL_ADDRESS>Skipping.: [stat<EMAIL_ADDRESS>no such file or directory]
2017-08-04 01:53:19.078 UTC [msp] getMspConfig -> INFO 003 MSP configuration file not found at<EMAIL_ADDRESS>[stat<EMAIL_ADDRESS>no such file or directory]
2017-08-04 01:53:19.129 UTC [msp] GetLocalMSP -> DEBU 004 Returning existing local MSP
2017-08-04 01:53:19.129 UTC [msp] GetDefaultSigningIdentity -> DEBU 005 Obtaining default signing identity
2017-08-04 01:53:19.131 UTC [channelCmd] InitCmdFactory -> INFO 006 Endorser and orderer connections initialized
2017-08-04 01:53:19.132 UTC [msp] GetLocalMSP -> DEBU 007 Returning existing local MSP
2017-08-04 01:53:19.132 UTC [msp] GetDefaultSigningIdentity -> DEBU 008 Obtaining default signing identity
2017-08-04 01:53:19.132 UTC [msp] GetLocalMSP -> DEBU 009 Returning existing local MSP
2017-08-04 01:53:19.132 UTC [msp] GetDefaultSigningIdentity -> DEBU 00a Obtaining default signing identity
2017-08-04 01:53:19.132 UTC [msp/identity] Sign -> DEBU 00b Sign: plaintext: 0A8C060A074F7267314D53501280062D...53616D706C65436F6E736F727469756D
2017-08-04 01:53:19.132 UTC [msp/identity] Sign -> DEBU 00c Sign: digest: E23F3F8AF872F375A3892F1E80246F26C3B269F7BFDA0E20CF5AD985BF290FDB
2017-08-04 01:53:19.132 UTC [msp] GetLocalMSP -> DEBU 00d Returning existing local MSP
2017-08-04 01:53:19.132 UTC [msp] GetDefaultSigningIdentity -> DEBU 00e Obtaining default signing identity
2017-08-04 01:53:19.133 UTC [msp] GetLocalMSP -> DEBU 00f Returning existing local MSP
2017-08-04 01:53:19.133 UTC [msp] GetDefaultSigningIdentity -> DEBU 010 Obtaining default signing identity
2017-08-04 01:53:19.133 UTC [msp/identity] Sign -> DEBU 011 Sign: plaintext: 0AC4060A1608021A06088FA78FCC0522...1548167A1D7B35DE8FC365A9BA90247A
2017-08-04 01:53:19.133 UTC [msp/identity] Sign -> DEBU 012 Sign: digest: 147310DA6A88B60D64DF001479100BF96BA2620C0ED04038F59136F76913F0A4
Error: Got unexpected status: SERVICE_UNAVAILABLE
Usage:
peer channel create [flags]
Global Flags:
-b, --blockpath string           Path to file containing genesis block
  --cafile string              Path to file containing PEM-encoded trusted certificate(s) for the ordering endpoint

-c, --chain string               In case of a newChain command, the chain ID to create.
-f, --file string                Configuration transaction file generated by a tool such as configtxgen for submitting to orderer
  --logging-level string       Default logging level and overrides, see core.yaml for full syntax

-o, --orderer string             Ordering service endpoint
  --test.coverprofile string   Done (default "coverage.cov")

-t, --timeout int                Channel creation timeout (default 5)
  --tls                        Use TLS when communicating with the orderer endpoint

-v, --version                    Display current version of fabric peer server
!!!!!!!!!!!!!!! Channel creation failed !!!!!!!!!!!!!!!!
================== ERROR !!! FAILED to execute End-2-End Scenario ==================

you can try this project .https://github.com/wushouhe/fabric-docker-compose-svt  ,in the project ,i removed kafka and zookeeper. mybe this can help you

I got the same exception

I got the same exception

I got the same exception

I got the same exception


Set up MkDocs

MkDocs has been configured, and the "About Us" information has been extracted from the OSGeo Nepal wiki page.

@HEM2058  Can you also provide github.io for testing from your local fork ?

@kshitijrajsharma https://hem2058.github.io/RollingConvo/

Thank you for @HEM2058  for this PR . Excellent work ! Cheers !
Some Suggestion and Enhancements from my side  :

I have added auto deployment of docs through github actions
I have also installed several extensions such as dynamic navigation based on docs change , fonts , github linking , search bar etc ! Checkout here
I have disabled readthe docs format and migrated to this new layout for easecase !


Avoid long-lived references for RTFM compatibility

https://github.com/nickray/lpc55s6x-hal/blob/ebbb8cf1d41c508ae983459560f3be63bc9adbdf/examples/rtfm_led.rs#L17 doesn't build.
cf. https://github.com/lpc-rs/lpc8xx-hal/issues/159#issuecomment-526952307

RTFM blinky works now: https://github.com/nickray/lpc55s6x-hal/blob/2f0b924dfc691798b8eca8268bfe39e1ad8f74fc/examples/rtfm_led.rs
removed 'gpio-themed references
we keep the reference in .into_gpio_pin(&gpio) to ensure GPIO peripheral is enabled
but since don't intend to let the user access to dirset/set/clr
the svd2rust-generated API is horrible for stealing, would like to put (stolen) owned registers in Pin<T, Gpio>.state to remove all the explicit unsafe calls

Generally:

prevent the user from making mistakes (if a peripheral needs to be enabled, enforce that)
trust ourselves as HAL writer to uphold invariants
don't let the user directly access registers that could break our invariants

I added a RegClusterProxy that allows us to embed the necessary GPIO registers in GPIO pins: https://github.com/nickray/lpc55s6x-hal/blob/2fdc72a20045e4b51dcecb3c4c115b8a18f45485/src/iocon.rs#L62-L66
See also: https://github.com/rust-embedded/svd2rust/issues/213

v0.0.1 is released: https://crates.io/crates/lpc55s6x-hal


[WIP] Default maxSize to 100KB for SplitChunks

What kind of change does this PR introduce?
feature
Did you add tests for your changes?
already exist
Does this PR introduce a breaking change?
yes
What needs to be documented once your changes are merged?
Set optimization.splitChunks.cacheGroups.defaultVendors.maxSize to 100KB

@sokra Currently these changes do nothing. I've greatly improved the plugin typings but I have no clue on what I should do… My guess is that this block needs to be updated: https://github.com/webpack/webpack/blob/next/lib/optimize/SplitChunksPlugin.js#L851-L875
Could you give me some hints/advices?

Thanks


New Integrationen with api key

I cannot install this integration because the Steam integration in Home Assistant already requires an API key and I cannot create another one. If you enter the existing API key, it will not work in your Steam Wishlist integration.
Unexpected error fetching steam_wishlist data
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 379, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 280, in _async_update_data
return await self.update_method()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/steam_wishlist/sensor_manager.py", line 57, in _async_fetch_data
item["appid"] for item in wishlist_data["response"]["items"]
~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
KeyError: 'items'

I don't believe it's related to having setup the steam integration.  I have both the steam integration and this custom integration setup at the same time.
This is a little tricky to debug, could you confirm that you get a successful response with this url:
https://api.steampowered.com/ISteamUser/ResolveVanityURL/v1/?key=<your-api-key>&vanityurl=<your-account-name>

If it's successful it should return something like:
{
  "response": {
    "steamid": "123456789",
    "success": 1
  }
}

If that returns a successful response, could you also make sure this URL returns data as expected:
https://api.steampowered.com/IWishlistService/GetWishlist/v1?key=<your-api-key>&steamid=<your-steamid-from-the-response-of-the-previous-url>

If successful it should return something like:
{
  "response": {
    "items": [
      {
        "appid": 305050,
        "priority": 0,
        "date_added":<PHONE_NUMBER>
      },
      {
        "appid": 347800,
        "priority": 0,
        "date_added":<PHONE_NUMBER>
      }
    ]
  }
}

ttps://api.steampowered.com/ISteamUser/ResolveVanityURL/v1/?key=&vanityurl=<your-account-name

If I enter the existing API key, this appears in Home Assistant:

ttps://api.steampowered.com/ISteamUser/ResolveVanityURL/v1/?key=&vanityurl=<your-account-name

Based on your 2nd screenshot it appears that the first url returns a successful response since it shows your steam id in parenthesis.
So help debug, I'd need the response of this url (replace <your-api-key> with your real api key):
https://api.steampowered.com/IWishlistService/GetWishlist/v1?key=<your-api-key>&steamid=76561197960522903

https://api.steampowered.com/IWishlistService/GetWishlist/v1?key=&steamid=76561197960522903

I also get the same exact empty response.
{"response":{}}

Thanks for the response.  I thought maybe it had to do with privacy settings as previously the Game Details setting had to be public in order to retrieve the wish list data.  I wouldn't think that would be required with an api key, and in my testing when I made my profile private it still pulled data as expected.
But you could try setting "Game details" to "Public" under the Privacy Settings when editing your profile and report back if that changed anything.
Somewhat at a loss at the moment as to why it can't fetch your wish list.

I changed something else in Steam, but unfortunately it didn't result in it retrieving the wishlist.

I changed something else in Steam, but unfortunately it didn't result in it retrieving the wishlist.

Thanks for trying.  I unfortunately don't have any other suggestions.  It works for some folks and not others for some reason.  There really isn't another way to get the wishlist data besides the endpoint I"m currently using: https://steamapi.xpaw.me/#IWishlistService/GetWishlist

V3.01 fixed it for me.

Thank you very much for your efforts to solve the problem that not everyone has. With your new fix it works straight away. Thank you again. I wish you a wonderful Christmas. Best wishes, Johnyboy1984


Update home page to explain what is it about, how can people use the app and how they can contribute

Currently we have a beautiful welcome message and when logged in as an admin there is a dashboard with some metrics, we should update this to explain a bit what this site is about and how can people use the application and contribute to it:

as users of the app;
as collaborators for data additions and editing;
as store owners managing their stores and reporting on their stores queues;

There are no designs for this, so whoever picks it up can come up with something. We are using TwitterBootstrap so maybe using some cards and some images or just some cards with text might be good. Don't forget that it needs to be localised and we can get translations from some of the other volunteers.

gonna grab this one fyi.


Stop using Hash#stringify_keys method

#29 has removed the runtime dependency on the activesupport gem and removed require 'active_support/all' but the Hash#stringify_keys method added by Active Support core extensions is still used, thus causing a NoMethodError error if the extension is not loaded.
$ cat Gemfile
source 'https://rubygems.org'
gem 'garage_client', git: 'https://github.com/cookpad/garage_client'

$ bundle info garage_client
  * garage_client (2.4.4 1bdc412)
...

$ bundle exec irb -r garage_client
irb(main):001:0> GarageClient.headers = {}
=> {}
irb(main):002:0> GarageClient::Client.new(endpoint: 'foo').headers
...
        1: from /path/to/garage_client/lib/garage_client/client.rb:30:in `headers'
NoMethodError (undefined method `stringify_keys' for {}:Hash)

Thank you!


No way to apply GOGS_CUSTOM for SSH

Bug Description:
There seems to be no way to apply GOGS_CUSTOM for SSH, which is making GOGS_CUSTOM nearly useless, as you can't get SSH to use the same config..
Gogs Version: v<IP_ADDRESS>6 Beta
Git Version: 2.2.1
System Type: Arch Linux
Error Log: Nothing
Other information: #486 have some useful information (bottom)

A solution for running multiple Gogs daemons with the same unix user would probably be a PAM module, but i am not that familiar with it, so please correct me if i’m wrong. For instance, looking at /etc/pam.d/ssh shows that one of the first strategy to be loaded when authenticating SSH is pam_loginuid.so. Putting a custom compilable module before this step would make this possible, but isn’t it really too much for what we want? IMO, having N users for N instances is very clean, and is the “unix” way to do things.
The binary argument seems the best option to me too.
As @phsmit  stated, there could be a number of possible arguments to override.
By default, Gogs could be looking with priority:

Hard-coded defaults when initializing the configuration structures,
Read defaults from /etc/gogs if they exist,
Read overrides (Or full config if all parameters are in the file) from a location pointed by -config “/my/custom”
Also as pointed by @phsmit, this could also work with the templates, and the variable data (Logs / PIDs). System defaults overridable by executable flag.

A solution for running multiple Gogs daemons with the same unix user would probably be a PAM module, but i am not that familiar with it, so please correct me if i’m wrong. For instance, looking at /etc/pam.d/ssh shows that one of the first strategy to be loaded when authenticating SSH is pam_loginuid.so. Putting a custom compilable module before this step would make this possible, but isn’t it really too much for what we want? IMO, having N users for N instances is very clean, and is the “unix” way to do things.
The binary argument seems the best option to me too.
As @phsmit  stated, there could be a number of possible arguments to override.
By default, Gogs could be looking with priority:

Hard-coded defaults when initializing the configuration structures,
Read defaults from /etc/gogs if they exist,
Read overrides (Or full config if all parameters are in the file) from a location pointed by -config “/my/custom”
Also as pointed by @phsmit, this could also work with the templates, and the variable data (Logs / PIDs). System defaults overridable by executable flag.

Instead, why not search for the current directory (The directory where the user running the daemon was positioned when starting Gogs)?

This way we can apply your custom config when ./gogs serve been called when you push.

Instead, why not search for the current directory (The directory where the user running the daemon was positioned when starting Gogs)?

This way we can apply your custom config when ./gogs serve been called when you push.

@Unknwon Yes :) But again, @phsmit pointed a more reliable solution IMO, the flags parsing option.
Please see my edited post for more soutions about the SSH keys for multiple instances.

@Unknwon Yes :) But again, @phsmit pointed a more reliable solution IMO, the flags parsing option.
Please see my edited post for more soutions about the SSH keys for multiple instances.

I suggest making the configlocation an argument of gogs web, with as default the current situation as default. The same for a sharelocation (where the templates are stored). If you like I can either give a detailed description or implement it in pull request and discuss that further ( @Unknwon which do you prefer?)

This is problem we have right now, ./gogs serve is called by system(actually git itself), the argument you pass into gogs web has no way to tell when gogs serve is executed.

I suggest making the configlocation an argument of gogs web, with as default the current situation as default. The same for a sharelocation (where the templates are stored). If you like I can either give a detailed description or implement it in pull request and discuss that further ( @Unknwon which do you prefer?)

This is problem we have right now, ./gogs serve is called by system(actually git itself), the argument you pass into gogs web has no way to tell when gogs serve is executed.

PS:
Unless every time you run Gogs with option(where is the custom path), Gogs rewrites entire authorized_keys file.

PS:
Unless every time you run Gogs with option(where is the custom path), Gogs rewrites entire authorized_keys file.

@Unknwon Understood. I really think multi-instances installs can stay with as many unix users right now. But it's worth investigation, either with the UUID system or with the PAM module whenever people really want this feature.

@Unknwon Understood. I really think multi-instances installs can stay with as many unix users right now. But it's worth investigation, either with the UUID system or with the PAM module whenever people really want this feature.

One bummer, use case:

Two gogs running in the same user.
One user being registered with the same SSH key in the two instances.

Right now, stick with the user-separation :)

One bummer, use case:

Two gogs running in the same user.
One user being registered with the same SSH key in the two instances.

Right now, stick with the user-separation :)

@hickscorp
I would prefer one user one instance.
But a thought on one user multiple instances, Gogs can do a SSH forward, which handles AUTH on Gogs, and forward operations to system(real) SSH server. This way, we do not need to care about default authorized_keys file any more, but just Gogs database check.

@hickscorp
I would prefer one user one instance.
But a thought on one user multiple instances, Gogs can do a SSH forward, which handles AUTH on Gogs, and forward operations to system(real) SSH server. This way, we do not need to care about default authorized_keys file any more, but just Gogs database check.

Any update? I need to edit authorized_keys manually, for every new ssh key..

Hi @klausenbusk , why would this issue be matter with "I need to edit authorized_keys manually, for every new ssh key"? Sounds like it's about fail to add SSH key?

Hi @klausenbusk , why would this issue be matter with "I need to edit authorized_keys manually, for every new ssh key"? Sounds like it's about fail to add SSH key?

I need to change it from:
command="/usr/share/gogs/gogs serv key-5"
to:
command="GOGS_CUSTOM="/srv/gogs" /usr/share/gogs/gogs serv key-5"
So it can find the config file..

An update on:
dev branch only saves partial configuration when install, so it's possible to write config options(that not conflict with) in other places.

https://github.com/gogits/gogs/pull/905 this PR will help on this

By PR https://github.com/gogits/gogs/pull/905 and other code I've pushed to dev branch, you are now able to fix this issue.
PS: you can now also do rewrite SSH key file and all repository update hooks in admin dashboard with one click.

@Unknwon I'm trying to compile the dev branch, but I get this error
WORK=/tmp/go-build952790733
../../macaron-contrib/cache/memcache/memcache.go:22:2: cannot find package "github.com/bradfitz/gomemcache/memcache" in any of:
	/mnt/gogs/src/build/go/src/github.com/bradfitz/gomemcache/memcache (from $GOROOT)
	/mnt/gogs/src/build/src/github.com/bradfitz/gomemcache/memcache (from $GOPATH)
../../microcosm-cc/bluemonday/sanitize.go:38:2: cannot find package "golang.org/x/net/html" in any of:
	/mnt/gogs/src/build/go/src/golang.org/x/net/html (from $GOROOT)
	/mnt/gogs/src/build/src/golang.org/x/net/html (from $GOPATH)
modules/base/template.go:16:2: cannot find package "golang.org/x/net/html/charset" in any of:
	/mnt/gogs/src/build/go/src/golang.org/x/net/html/charset (from $GOROOT)
	/mnt/gogs/src/build/src/golang.org/x/net/html/charset (from $GOPATH)
modules/base/template.go:17:2: cannot find package "golang.org/x/text/transform" in any of:
	/mnt/gogs/src/build/go/src/golang.org/x/text/transform (from $GOROOT)
	/mnt/gogs/src/build/src/golang.org/x/text/transform (from $GOPATH)

Don't know if it is something in your code or.. Not a Go expert :)

@klausenbusk this means you need to download some of packages that are newly added in dev branch.
 Shouldn't they be in .gopmfile or is that for "other stuff"?

Hmm... yeah, gopmfile updates before every release. Are you using gopm right? Or just use the gopmfile?

Hmm... yeah, gopmfile updates before every release. Are you using gopm right? Or just use the gopmfile?

At the momemt, I'm just patching the ArchLinux PKGBUILD file.. Which seems to parse the .gopmfile and use go get  : https://github.com/fanningert/PKGBUILDs/blob/master/aur/gogs/helper.sh#L52

I see, I've pushed fix on dev, please help try again!

I see, I've pushed fix on dev, please help try again!

package golang.org/x/net
	imports golang.org/x/net
	imports golang.org/x/net: no buildable Go source files in /mnt/gogs/src/build/src/golang.org/x/net

also got that error, when I tried go get in the PKGBUILD..

New compile error http://dpaste.com/14W5J1B (5 last line)

Fine... I'm nor familiar with that, how about test this on newer release which comes soon?

Fine... I'm nor familiar with that, how about test this on newer release which comes soon?

okay :) I don't have any problems waiting, there is only 2 users on my gogs "instance", so I can handle it manually..

@klausenbusk sorry for that, this issue actually improves a lot things in Gogs.

@klausenbusk sorry for that, this issue actually improves a lot things in Gogs.

 I did recompile 0.5.11, after I was finish testing..

Try update all dependencies,,,

Try update all dependencies,,,

You mean everything in gopmfile? I fetch that everytime it compile..
It still don't work, just did a recompile now (from a clean "build" environment)..

Just do go get -u on the root directory of source code of Gogs.

Just do go get -u on the root directory of source code of Gogs.

So I should switch to master?

 If I remember correctly.

 If I remember correctly.

At the momemt, I just fetch it with git and the 0.5.11 tag

I think I just wait for a release (hopeful not too long), master and dev branch won't compile (http://dpaste.com/2KXNEM1)

Sure, planned this week.

Fixed with #905.. Kind of..

:beer::beer::beer:


Adding alert to Change Existing User Account form

Adding an alert to the form, advising users trying to use this form
of circumstances when they do not need to raise a request.
As per Zendesk ticket: https://govuk.zendesk.com/agent/tickets/5908958

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Looks like this PR includes changed from your other one, which need to be removed. The change itself looks good to me.


No Auth token on authentication resource

I just generated the JWT authentication strategy via CLI and when I try to authenticate with the following request:
curl -X POST \
  http://localhost:3030/authentication \
  -H 'cache-control: no-cache' \
  -H 'content-type: application/json' \
  -d '{
"email"<EMAIL_ADDRESS>"password": "password"
}'

The answer is:
{
  "name": "NotAuthenticated",
  "message": "No auth token",
  "code": 401,
  "className": "not-authenticated",
  "data": {},
  "errors": {}
}

And the debug log says:
Fri, 21 Apr 2017 21:28:49 GMT feathers-authentication:passport:authenticate Authentication strategy 'jwt' failed Error: No auth token
    at JwtStrategy.authenticate (/home/app/tagger/node_modules/passport-jwt/lib/strategy.js:87:26)
    at /home/app/tagger/node_modules/feathers-authentication/lib/passport/authenticate.js:131:18
    at /home/app/tagger/node_modules/feathers-authentication/lib/passport/authenticate.js:38:14
    at Object.<anonymous> (/home/app/tagger/node_modules/feathers-authentication/lib/hooks/authenticate.js:80:55)
    at process._tickCallback (internal/process/next_tick.js:103:7) undefined

This is my actual configuration
Fri, 21 Apr 2017 21:28:49 GMT feathers-authentication:hooks:authenticate Attempting to authenticate using jwt strategy with options { name: 'jwt',
  bodyKey: 'accessToken',
  secret: 'a big secret',
  header: 'Authorization',
  entity: 'user',
  service: 'users',
  passReqToCallback: true,
  session: false,
  jwt:
   { header: { typ: 'access', type: 'access' },
     audience: 'localhost:3030',
     subject: 'anonymous',
     issuer: 'feathers',
     algorithm: 'HS256',
     expiresIn: '1d' } }

Have someone else faced this issue? Seems that the /authentication route is under protection.

Fixed, I had to comment this line:
app.service('authentication').hooks({
  before: {
    create: [
      // You can chain multiple strategies
      auth.hooks.authenticate(['jwt', 'local']),
    ],
    remove: [
      auth.hooks.authenticate('jwt')
    ]
  }
});

I don't understand why this was generated :)

@waldemarnt that was not a fix.  You need to leave that line in place or you basically enable anonymous authentication.  We have an open issue to remove the implicit anonymous auth.  You've generated a token with no user attached.  The real solution would be to make sure you pass a strategy in the payload, as mentioned in the API docs.

@marshallswain thank you, I trying to figure out how those strategies work, I couldn't found on the documentation :(

https://docs.feathersjs.com/api/authentication/server.html#the-authentication-service
https://docs.feathersjs.com/api/authentication/server.html#the-authenticate-hook <-- Just added
https://docs.feathersjs.com/api/authentication/client.html#additional-feathersclient-methods
https://github.com/feathersjs/feathers-authentication/blob/master/docs/migrating.md#setting-up-authentication-on-the-client
Where else do you think it should go?


(error) MOVED 15495 <IP_ADDRESS>:7002

<IP_ADDRESS>:7000> set a 1
(error) MOVED 15495 <IP_ADDRESS>:7002
How to solve this problem?

@wormding Use the following
$ redis-cli -c -p 7000
redis <IP_ADDRESS>:7000> set foo bar
-> Redirected to slot [12182] located at <IP_ADDRESS>:7002
OK
redis <IP_ADDRESS>:7002> set hello world

you are not using the cluster mode on the redis-cli tool.

redis-cli -c  -p 7000
<IP_ADDRESS>:7000> set foo bar
-> Redirected to slot [12182] located at <IP_ADDRESS>:7002
why will I got <IP_ADDRESS>:7002 this? Thanks, buddy!!

I'm facing the same issue on docker for mac. Is this docker's network problem in mac?

I cleared with the similar product, that assigned fixed IP value as `<IP_ADDRESS>'.

https://github.com/tommy351/docker-redis-cluster/blob/master/3.2/start.sh#L12

@Grokzen How about IP could be assigned by outside parameter?

https://github.com/Grokzen/docker-redis-cluster/blob/master/docker-data/docker-entrypoint.sh#L24

@wormding You should really read up on the redis-cluster spec and basic docs before you continue. You can find it here https://redis.io/topics/cluster-tutorial .
I can't help everyone who has basic redis cluster problems that is in no way related to this client lib. For that kind of help you have to ask in other forums/boards.

@smdmts No there is no relation between the problem wormding has and the problem you mentioned. Basically there is a difference between the IP address you connect to the cluster with, and the IP address that redis internally tracks each node with.

I have already fixed it out. @Grokzen  thanks a  lot~


Align documentation with new manifest type "Microsoft Graph App Manifest (New)"

Align documentation with new manifest "type" Microsoft Graph App Manifest (New)

@garrodonnell
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Fix tool naming convention

As far as I can tell, the convention for the tool executable name is dotnet-<command> (e.g. dotnet-clean as is shown further in the article), not dotnet <command>.

@svick Thanks for your contribution.  This looks great to me.
:shipit:


Recommendation regarding documentation

Great tool! Just successful used it!
I would have been much faster when I knew to points so please add them to the readme.md!

Dependency to PSPKI module and how to install it Install-Module -Name PSPKI
How to set script execution Set-ExecutionPolicy Unrestricted -Scope CurrentUser (The info is in the .ps1 file but not the readme.md()
RDP can't deal with certificate chains I have to use the plain cert and not the chain. Please add a note. (Actually I have no glue why RDP does not ask for the intermediate cert)

Thank you for the feedback.
All of your points are worth documenting. I'll improve the docs per your suggestion with the mentioned content.

Fixed in https://github.com/HQJaTu/RDP-cert-tools/commit/1b9a9bd279b6c19dace40c0510b757c0a3e17eb0.
Should further requirements covering this issue raise, please, re-open. For any new ones, file a new issue.