Why is the nowadays activity so quiet here?

Maybe some of your are having same question, so let me break the silence.
HaitiCSS started as a side project building a multi-brand design system, and the early idea was to be an extended version of INUIT or ITCSS that should fit our case. However, on the way, we discovered our best input on conceptual abstractions, and not on code itself. For us is even hard to see HaitiCSS as a tool/boilerplate or as a methodology. While we have different points of view and goals, the project is maturing on our usage case and not here, our apologies for that. The main reason that pointed us to do it, was the fragility and frequency of the changes on the project, too noise to follow. Concluding, the project is still alive, but unfortunately not here for now. I promise to come back with revamped code and ideas soon ;)

Not related but I just wanted to thank you for your work thus far. I've been tasked with untangling our CSS and after jumping all around the web reading multiple articles on ITCSS, discovering multiple open source repos on GitHub (include those by Harry himself) I came across haiticss just yesterday.
Of all the examples I have come across thus far it is the one I have found myself agreeing with the most.
I have to say it almost feels like the spiritual successor to inuitcss. While I understand ITCSS is just a way of thinking it helps a whole bunch to have a base to build from and solidify understanding.
I particularly like your inclusion of the foundations layer, particularly the way you have separated the spaces from being based on a single core-spacing-unit like in Harry's examples. The article linked in issue #64 was particularly interesting.
Anyway long story short. Keep up the good work all!


We're getting a warning that google maps version 3.31 is retired.

WARN: 'Google Maps JavaScript API warning: RetiredVersion https://developers.google.com/maps/documentation/javascript/error-messages#retired-version'
Happy to submit a PR with some instruction.

Hi and thanks for reporting. PR would be appreciated.
It should be enough to just change this line: https://github.com/Carrooi/Js-GoogleMapsLoader/blob/master/lib/Google.js#L21
and run npm run minify.

@davidkudera PR submitted.
... also, just setting the version like so GoogleMapsLoader.VERSION = 3.34 seems to do the trick.

hi there.. and the release in npm?

@jrgcubano just require a specific git commit for now
"dependencies": {
    "google-maps": "git@github.com:Carrooi/Js-GoogleMapsLoader.git#ff66b0645fe2e32d7bde6a272281355e10b3e0a4"
}


Error running 'git submodule update --init'

I got the following error when I ran git submodule update --init:
Cloning into '/Users/alberto/projetos/real-world-rails/apps/opensit'...
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:danbartlett/opensit.git' into submodule path '/Users/alberto/projetos/real-world-rails/apps/opensit' failed
Failed to clone 'apps/opensit' a second time, aborting

👍 Thanks for reporting @betogrun. Did/does it complete successfully now?

Hi @eliotsykes
It is working fine now!
Thanks!


http test requires Internet Explorer

Running http tests from a headless PowerShell session will fail due to a lack of Internet Explorer.
E.g. running
Describe 'Website' {
    Http "http://localhost" StatusCode { Should Be 200 }
}

From Octopus deploy as a job step results in:
NotSupportedException: The response content cannot be parsed because the Internet Explorer engine is not available, or Internet Explorer's first-launch configuration is not complete. Specify the UseBasicParsing parameter and try again.
14:52:45Info
   at <ScriptBlock>, <No file>: line 1
14:52:45Info
   at <ScriptBlock>, C:\Program Files\WindowsPowerShell\Modules\poshspec\Private\Invoke-PoshspecExpression.ps1: line 12

As far as I can tell none of the Properties supported by this test require complex parsing in IE, so I suggest just adding the -usebasicparsing flag.

@cdhunt Any chance of doing a release to the PSGallery so we can get this out in the wild pretty please? :)

v2.1.10 has been published to GitHub and the Gallery.


Tutorials nighly fix

Description
Fixing tutorials test.
Disabling some tutorials test until we get a fix for them. Explicitly disabling gluon perf test, profiler test and mkldnn tutorial test
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@anirudh2290

github issue created - https://github.com/apache/incubator-mxnet/issues/16181 for reenabling  tests once we have fixes.

one nitpicky comment rest LGTM!


hmr update not working with beforeRouteEnter

Describe the bug
<template>
  <h2>{{ msg }}</h2>
</template>
<script>
export default {
  name: 'Home',
  data() {
    return {
      msg: '',
    }
  },
  beforeRouteEnter(to, from, next) {
    return next(vm => {
      vm.msg = 'hello111'
    })
  },
}
</script>

This is minial code for repdocution.
When we change code in , beforeRouteEnter section doesn't run with hmr.
Is it suppoed to work like that or bug..?
There is no similar issue so I post it
Reproduction
https://stackblitz.com/edit/vue3-vite-router-starter-qerzmy?file=src%2Fviews%2FHome.vue,src%2Frouter.js,src%2FApp.vue,src%2Fmain.js
Change any code in views/Home.vue beforeRouteEnter section, Then you can check hmr not working with beforeRouteEnter.
System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (12) x64 AMD Ryzen 5 2600X Six-Core Processor
    Memory: 21.39 GB / 31.93 GB
  Binaries:
    Node: 16.13.1 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.7 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 102.0.5005.115
    Edge: Spartan (44.19041.1266.0), Chromium (102.0.1245.44)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-vue: ^2.3.3 => 2.3.3
    vite: ^2.9.9 => 2.9.12

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

IMO this is intended. HMR reloads the component. So route change will not happen and then beforeRouteEnter will not happen.

Thanks for comment.
Then, is there alternative way to work with beforeRouteEnter? any options?

I cannot come up with a solution.
Closing this as IIUC this needs to be implemented by vue-router if it is going to be imlpemented.

Thanks I need to make code in created or some another section , not in related to vue-router.


Fixed incorrect links for YAML files under Run Application section

The current links for YAML files throws 404. Updated the links from tutorials to tasks. Also did a random check for YAML links in other pages under Run Applications.

This change is

I have signed the CLA


sitemap and rss 404 errors

Hi, any reasons to why there are sitemap and RSS console errors? I've seen this issue also happening on the example website.

thx

hey nice find!
it looks like the issue is because the Next.js Link component will try to prefetch information on a page and I was using that <Link component to link to those 2 files, with the thought that they were still technically available on the same host
seems they should be treated as external links instead to avoid this lookup, about to push up a fix


How to check if a tensor is empty in TensorFlow

Hi there,
I am just using TensorFlow to train a CNN model. I used a command:
t_iteration = tf.placeholder('int32', [], name='t_iteration') to initialize a tensor with shape of (0,).

I would want to know how to judge if the tensor t_iteration is empty? I have tried to use
x = tf.constant(3)
y = tf.constant(5)
 idx0 = tf.shape(t_iteration)
        A = tf.cond(tf.cast(idx0 == 0, tf.bool),
                                    lambda: tf.multiply(x, 17),
                                    lambda: tf.add(y, 23))

But I found that this method didn't work-as the `idx0' always equals to 0 even the tensor has not been assigned any value.
How to solve this tricky issue?
Thanks!

Hi there,
I am just using TensorFlow to train a CNN model. I used a command:
t_iteration = tf.placeholder('int32', [], name='t_iteration') to initialize a tensor with shape of (0,).

I would want to know how to judge if the tensor t_iteration is empty? I have tried to use
x = tf.constant(3)
y = tf.constant(5)
 idx0 = tf.shape(t_iteration)
        A = tf.cond(tf.cast(idx0 == 0, tf.bool),
                                    lambda: tf.multiply(x, 17),
                                    lambda: tf.add(y, 23))

But I found that this method didn't work-as the `idx0' always equals to 0 even the tensor has not been assigned any value.
How to solve this tricky issue?
Thanks!

Refer to stackoverflow here:
https://stackoverflow.com/questions/50529309/how-to-check-if-a-tensor-is-empty-in-tensorflow
Quick Note:
placeholder is a tf1.x module.  if you are using tf2.x, you have to use tf v1 compatibility, refer to doc here:
https://www.tensorflow.org/api_docs/python/tf/compat/v1/placeholder

@Dee-Ma
This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
please check to link for reference

Tensorflow requires a function to check if a tensor is empty


Syntax error in input(3) of itkVectorContainer.i

Description
I want to build insighttoolkit for openSUSE Tumbleweed on s390x and x86.
It is failing with the following error message:
[ 1239s] /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/itkVectorContainer.i:177: Error: Syntax error in input(3).
[ 1239s] ninja: build stopped: subcommand failed.

Actual behavior
Build of the openSUSE package is failing because of a syntax error.
Reproducibility
100% on s390x and x86
Versions
5.2.1
Environment
openSUSE Tumbleweed
architecture: s390x or x86
cpp-12-2.2
cmake-3.23.2-2.1
gcc-12-2.2
gtk2-devel-2.24.33-2.5
Additional Information
(see build log: https://build.opensuse.org/public/build/Application:Geo/openSUSE_Factory_zSystems/s390x/insighttoolkit/_log )

Does the error persist when you use master instead of 5.2.1?

I have tested the master branch in my home repo now.
It does not come so far because of this new error message:
[  146s] + ninja -v -j6
[  146s] ninja: error: 'Wrapping/Modules/ITKCommon/castxml', needed by 'Wrapping/castxml_inputs/itkFixedArray.xml', missing and no known rule to make it

It is worthwhile to start a clean build (build in a new directory, to avoid possible interference of files from the previous build).

I got the same errors when building itk for ArchLinux with gcc 12.1.0, see the complete build log here

class itkVectorContainerUCD : public itkObject, private std::vector< double > {
  public:
    static itkVectorContainerUCD_Pointer New();
    virtual itkLightObject_Pointer CreateAnother() const;
    itkVectorContainerUCD_Pointer Clone() const;
    virtual char const * GetNameOfClass() const;
    std::vector< double > & CastToSTLContainer();
    std::vector< double > const & CastToSTLConstContainer() const;
    ?unknown? ElementAt(unsigned char arg0);
    ?unknown? ElementAt(unsigned char arg0) const;
    ?unknown? CreateElementAt(unsigned char arg0);
    double GetElement(unsigned char arg0) const;
    void SetElement(unsigned char arg0, double arg1);
    void InsertElement(unsigned char arg0, double arg1);
    bool IndexExists(unsigned char arg0) const;
    bool GetElementIfIndexExists(unsigned char arg0, double * arg1) const;
    void CreateIndex(unsigned char arg0);
    void DeleteIndex(unsigned char arg0);
    unsigned char Size() const;
    void Reserve(unsigned char arg0);
    void Squeeze();
    void Initialize();
    ~itkVectorContainerUCD();
  private:
  protected:
    itkVectorContainerUCD();
    itkVectorContainerUCD(unsigned long n);
    itkVectorContainerUCD(unsigned long n, double const & x);
    itkVectorContainerUCD(itkVectorContainerUCD const & r);
};

see also the complete itkVectorContainer.i

It built cleanly for me with gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1).
For you, it chokes at ?unknown? ElementAt(unsigned char arg0);, which is invalid C++ syntax.
The corresponding part of itkVectorContainer.i looks like this for me:
class itkVectorContainerUCD : public itkObject, private std::vector< double > {
  public:
    static itkVectorContainerUCD_Pointer New();
    virtual itkLightObject_Pointer CreateAnother() const;
    itkVectorContainerUCD_Pointer Clone() const;
    virtual char const * GetNameOfClass() const;
    std::vector< double > & CastToSTLContainer();
    std::vector< double > const & CastToSTLConstContainer() const;
    double & ElementAt(unsigned char arg0);
    double const & ElementAt(unsigned char arg0) const;
    double & CreateElementAt(unsigned char arg0);
    double GetElement(unsigned char arg0) const;
    void SetElement(unsigned char arg0, double arg1);
    void InsertElement(unsigned char arg0, double arg1);
    bool IndexExists(unsigned char arg0) const;
    bool GetElementIfIndexExists(unsigned char arg0, double * arg1) const;
    void CreateIndex(unsigned char arg0);
    void DeleteIndex(unsigned char arg0);
    unsigned char Size() const;
    void Reserve(unsigned char arg0);
    void Squeeze();
    void Initialize();
    ~itkVectorContainerUCD();
  private:
  protected:
    itkVectorContainerUCD();
    itkVectorContainerUCD(unsigned long n);
    itkVectorContainerUCD(unsigned long n, double const & x);
    itkVectorContainerUCD(itkVectorContainerUCD const & r);
};

It might be due to different versions of SWIG, CastXML or maybe even CMake. On my box: cmake version 3.23.2, SWG and CastXML bundled with ITK.

ArchLinux environment:
swig 4.0.2
cmake 3.23.2
I'll try using different castxml release, the failed one uses castxml 0.4.5.

openSUSE is using also CastXML-devel-0.4.5-1.2 and swig-4.0.2-6.3 .
I found this page about the changelog of the ongoing release: https://pyup.io/changelogs/itk/
I compare the new features and their requirements with our build environment.

/usr/include/wchar.h:155:24: error: use of undeclared identifier '__builtin_free'; did you mean '__builtin_frexp'?
  __attribute_malloc__ __attr_dealloc_free;
                       ^
/usr/include/sys/cdefs.h:691:46: note: expanded from macro '__attr_dealloc_free'
# define __attr_dealloc_free __attr_dealloc (__builtin_free, 1)
                                             ^
/usr/include/wchar.h:155:24: note: '__builtin_frexp' declared here
/usr/include/sys/cdefs.h:691:46: note: expanded from macro '__attr_dealloc_free'
# define __attr_dealloc_free __attr_dealloc (__builtin_free, 1)
                                             ^
In file included from /build/itk/src/build/Wrapping/ITKCommonBase.cxx:1:
In file included from /build/itk/src/ITK-5.2.1/Modules/Core/Common/include/itkCommand.h:21:
In file included from /build/itk/src/ITK-5.2.1/Modules/Core/Common/include/itkObject.h:31:
In file included from /build/itk/src/ITK-5.2.1/Modules/Core/Common/include/itkLightObject.h:21:
In file included from /build/itk/src/ITK-5.2.1/Modules/Core/Common/include/itkMacro.h:47:
In file included from /usr/lib/gcc/x86_64-pc-linux-gnu/12.1.0/../../../../include/c++/12.1.0/string:40:
In file included from /usr/lib/gcc/x86_64-pc-linux-gnu/12.1.0/../../../../include/c++/12.1.0/bits/char_traits.h:39:
In file included from /usr/lib/gcc/x86_64-pc-linux-gnu/12.1.0/../../../../include/c++/12.1.0/bits/postypes.h:40:
In file included from /usr/lib/gcc/x86_64-pc-linux-gnu/12.1.0/../../../../include/c++/12.1.0/cwchar:44:
/usr/include/wchar.h:155:24: error: '__malloc__' attribute takes no arguments
  __attribute_malloc__ __attr_dealloc_free;
                       ^
/usr/include/sys/cdefs.h:691:30: note: expanded from macro '__attr_dealloc_free'
# define __attr_dealloc_free __attr_dealloc (__builtin_free, 1)
                             ^
/usr/include/sys/cdefs.h:690:21: note: expanded from macro '__attr_dealloc'
    __attribute__ ((__malloc__ (dealloc, argno)))

building with -DITK_USE_SYSTEM_CASTXML:BOOL=ON still not working. check the complete error log itk.bundled-castxml.txt

On my computer, itkMacro.h:47 has following content #include <string>. That does not compile for you? That might be architecture-specific weirdness. That might come from misdetecting compiler. Check content of vcl_compiler_detection.h in itk-build-dir\Modules\ThirdParty\VNL\src\vxl\vcl\.

I could build itk master branch with this PKGBUILD, with -DITK_USE_SYSTEM_CASTXML=OFF, -DCMAKE_CXX_FLAGS='-fno-sized-deallocation' as menthioned in #3452.

I have figured out, why my build is failing so early with the master branch:
The test VXL_HAS_SSE2_HARDWARE_SUPPORT has been failing.

https://github.com/InsightSoftwareConsortium/ITK/issues/3506#issuecomment-1200167635

I found a similar issue https://github.com/CastXML/CastXML/issues/208. It seems related to GCC version. I could build it with clang 14.0.6.

I could build itk master branch with this PKGBUILD, with -DITK_USE_SYSTEM_CASTXML=OFF, -DCMAKE_CXX_FLAGS='-fno-sized-deallocation' as menthioned in #3452. I'm ok to wait for release 5.3.
These options have helped me for coming further. The build is failing because of this error message now:

[  463s] In file included from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkImageRegion.h:33,
[  463s]                  from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkImage.h:21,
[  463s]                  from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Transform/include/itkBSplineBaseTransform.h:23,
[  463s]                  from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Transform/include/itkBSplineTransform.h:21,
[  463s]                  from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/IO/TransformFactory/src/itkTransformFactoryBaseRegister.hxx:44,
[  463s]                  from /home/abuild/rpmbuild/BUILD/ITK-master/Modules/IO/TransformFactory/src/itkTransformFactoryBaseRegisterDoubleParameters.cxx:18:
[  463s] In member function 'constexpr const itk::Size<VDimension>::value_type& itk::Size<VDimension>::operator[](size_type) const [with unsigned int VDimension = 2]',
[  463s]     inlined from 'itk::Neighborhood<TPixel, VDimension, TAllocator>::SizeValueType itk::Neighborhood<TPixel, VDimension, TAllocator>::GetSize(DimensionValueType) const [with TPixel = double; unsigned int VDimension = 2; TAllocator = itk::NeighborhoodAllocator<double>]' at /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkNeighborhood.h:147:18,
[  463s]     inlined from 'void itk::NeighborhoodOperator<TPixel, VDimension, TAllocator>::FillCenteredDirectional(const CoefficientVector&) [with TPixel = double; unsigned int VDimension = 2; TAllocator = itk::NeighborhoodAllocator<double>]' at /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkNeighborhoodOperator.hxx:112:43:
[  463s] /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkSize.h:343:85: warning: array subscript 2 is above array bounds of 'const itk::Size<2>::SizeValueType [2]' {aka 'const long unsigned int [2]'} [-Warray-bounds]
[  463s]   343 |   constexpr const_reference operator[](size_type pos) const { return m_InternalArray[pos]; }
[  463s]       |                                                                      ~~~~~~~~~~~~~~~^
[  463s] /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkSize.h: In member function 'void itk::NeighborhoodOperator<TPixel, VDimension, TAllocator>::FillCenteredDirectional(const CoefficientVector&) [with TPixel = double; unsigned int VDimension = 2; TAllocator = itk::NeighborhoodAllocator<double>]':
[  463s] /home/abuild/rpmbuild/BUILD/ITK-master/Modules/Core/Common/include/itkSize.h:229:40: note: while referencing 'itk::Size<2>::m_InternalArray'
[  463s]   229 |   alignas(SizeValueType) SizeValueType m_InternalArray[VDimension];
[  463s]       |                                        ^~~~~~~~~~~~~~~

What source file/test is being compiled? And this looks like a warning, not an error message.

In the first step, I also thought it would only be a warning.
But this warning exists multiple times. If a warning is happening in an endless loop, then the build is not successful:
Here is the full build log.

In the first step, I also thought it would only be a warning. But this warning exists multiple times. If a warning is happening in an endless loop, then the build is not successful: Here is the full build log.

It seems the build server fail to download castxml,
[  419s] CMake Error at castxml-stamp/download-castxml.cmake:170 (message):

You could check also this PKGBUILD I used for building itk master for ArchLinux. It works as expected, and here is the full build log.

It seems this issue can be caused by castxml being built with llvm 14. If built with llvm 13, I don't have the issue.

ITK git master now has https://github.com/InsightSoftwareConsortium/ITK/pull/3541 -- as suggested by @hubutui this may help with this issue.

@thewtex Passed -fno-sized-deallocation(just used your patch from PR#3541) but we get the same error:
[  566s] FAILED: Wrapping/Modules/ITKCommon/itkVectorContainerPython.cpp Wrapping/Generators/Python/itk/itkVectorContainerPython.py /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Modules/ITKCommon/itkVectorContainerPython.cpp /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Generators/Python/itk/itkVectorContainerPython.py
[  566s] cd /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/python && /usr/bin/swig -c++ -python -O -features autodoc=2 -py3 -doxygen -Werror -w302 -w303 -w312 -w314 -w361 -w362 -w350 -w383 -w384 -w389 -w394 -w395 -w467 -w508 -w509 -o /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Modules/ITKCommon/itkVectorContainerPython.cpp -I/usr/share/swig/4.0.2/python -I/usr/share/swig/4.0.2 -I/home/abuild/rpmbuild/BUILD/ITK-5.2.1/Wrapping/Generators -I/home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/python -I/home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs -outdir /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Generators/Python/itk /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/itkVectorContainer.i
[  566s] /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/itkVectorContainer.i:177: Error: Syntax error in input(3).

Full log attached here _log.zip

ITK-5.2.1

@badshah400 thanks for testing. Is it possible to test Git master?

@thewtex Git master does not build due to something else:
[   78s] -- Build files have been written to: /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build
[   79s] + ninja -v -j8
[   79s] ninja: error: 'Wrapping/Modules/ITKCommon/castxml', needed by 'Wrapping/castxml_inputs/itkFixedArray.xml', missing and no known rule to make it
[   79s] error: Bad exit status from /var/tmp/rpm-tmp.KL7ywe (%build)

Odd considering I am using -DITK_USE_SYSTEM_CASTXML:BOOL=ON.
Ditto with v5.3.0rc04; trying to figure out the last commit that proceeds with the build at all.

Need to revert commit ee989dc89afa3487bf0ec1e730e0bdb8ebea38a2 to get it building at all without the previous error. After reverting this commit, git master (as commit b9005b83b315cdf61a345037c49dfd84f7bde386) still fails to build with the originally reported problem:
[  590s] /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/ITKCommonBase.i:1042: Error: Syntax error in input(3).
[  593s] /home/abuild/rpmbuild/BUILD/ITK-5.2.1/build/Wrapping/Typedefs/ITKCommonBase.i:1042: Error: Syntax error in input(3).
[  595s] error: Bad exit status from /var/tmp/rpm-tmp.42ma7B (%build)

@hjmjohnson what was the use case where https://github.com/InsightSoftwareConsortium/ITK/commit/ee989dc89afa3487bf0ec1e730e0bdb8ebea38a2 was needed?

#3729 addresses the ITK_USE_SYSTEM_CASTXML build. I get a successful local build with:
CC=gcc-12 CXX=g++-12 cmake -DITK_WRAP_PYTHON=1 -DITK_USE_SYSTEM_CASTXML=1 ~/src/ITK

gcc-12.1.0

hmmm, I just test with gcc 12.2.0, castxml 0.4.6, still the same error.
make[2]: Entering directory '/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build'
[ 75%] Generating ITKCommonBasePython.cpp, ../../Generators/Python/itk/ITKCommonBasePython.py
cd /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs/python && /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Generators/SwigInterface/swig/bin/swig -c++ -python -O -features autodoc=2 -py3 -doxygen -Werror -w302 -w303 -w312 -w314 -w361 -w362 -w350 -w383 -w384 -w389 -w394 -w395 -w467 -w508 -w509 -o /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Modules/ITKCommon/ITKCommonBasePython.cpp -I/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Generators/SwigInterface/swig/share/swig/4.0.2/python -I/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Generators/SwigInterface/swig/share/swig/4.0.2 -I/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/itk-src/Wrapping/Generators -I/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs/python -I/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs -outdir /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Generators/Python/itk /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs/ITKCommonBase.i
/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs/ITKCommonBase.i:1042: Error: Syntax error in input(3).
make[2]: *** [Wrapping/Modules/ITKCommon/CMakeFiles/ITKCommonPython.dir/build.make:441: Wrapping/Modules/ITKCommon/ITKCommonBasePython.cpp] Error 1
make[2]: Leaving directory '/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build'
make[1]: *** [CMakeFiles/Makefile2:39410: Wrapping/Modules/ITKCommon/CMakeFiles/ITKCommonPython.dir/all] Error 2
make[1]: Leaving directory '/home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build'
make: *** [Makefile:166: all] Error 2

and /home/hubutui/Projects/archlinuxcn-repo/archlinuxcn/itk-git/build/Wrapping/Typedefs/ITKCommonBase.i:1042:
    static ?unknown? progressFloatToFixed(float f);

According to:
https://github.com/InsightSoftwareConsortium/ITK/blob/85d3745371e7eda6ccff5b5266849bda308a6a65/Modules/Core/Common/include/itkProcessObject.h#L913-L926
We should get
    static unsigned int progressFloatToFixed(float f);

for build/Wrapping/Typedefs/ITKCommonBase.i:1042.
Maybe we need a manual typemaps as the swig doc mentioned?
BTW, I'm using swig 4.0.2. What's your swig version? @thewtex

@thewtex Yes, this patch https://github.com/InsightSoftwareConsortium/ITK/pull/3736 fix the issue. I could now build itk with system's swig and castxml.
@skriesch Hi, you could if this patch work on openSUSE.

BTW, I also rebuild itk 5.2.1 with this patch and it works great.

Yes. Thank you! It is fixing also our issue.

@hubutui @skriesch excellent, thanks for testing!


Article Says Task is for "Classic Release pipelines" but example is in YAML

Probably just a relic of pre YAML docs but it would be awesome to clear up any confusion!

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

ID: d5db2c74-c06f-475b-0b97-dd8901d5ff63
Version Independent ID: d5db2c74-c06f-475b-0b97-dd8901d5ff63
Content: ManualIntervention@8 - Manual intervention v8 task
Content Source: task-reference/manual-intervention-v8.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

@SamGrantham Thank you for the feedback. I agree - this is a classic release only task and so it does not need a YAML representation. They are autogenerated from the inputs of the task, let me think about how to remove it. This should be live in a few days.

I am going through my backlog, and I am making some updates to this area. The pipelines team has tested a few of the older classic release pipeline tasks and found that they can work in YAML, although in this case, ManualValidation@0 is the better task. I have adjusted the wording in ManualIntervention@8 to clarify this.


[enhancement] Set data storage location

Hi.
I love tdo and it would be great if I can set location where are data stored. Gist storing is cool, but I use owncloud and it would be great to sync my notes via that like file.
Thanks a lot,
Vojtěch Vladyka

I agree, that would be great. Dropbox and Google Drive are also very interesting options.
It seems that Google supports implicit auth flow
which could be used for this, although I'm not sure if it would work with our hash based routing.
Unfortunately, GitHub doesn't support this auth flow and it would require some kind of server-side middleware.
This is a big task so let's see if there are more people who would like the same feature.

And where are data stored now when I haven't Gist connected?
How about just give option to set location of that file?

By default, the data is stored in browser's localStorage.

I see... I am thinking if is there possibility to store data in file somewhere else, but I don't think so...

Hi,
I would be very much interested in integrating / handling lists stored locally (at least as an alternative to Gist storage). I would offer my support for implementing / testing this kind of feature (if support is needed / welcome).
At first, one question about the application structure. The browser address link always says something like this:
https:// <server_name> / <path_to_indexfile> / #<identifier>
(1) where is the <identifier> locally pointing to?
(2) how is that (temporary local) file structured?
In a first step, an additional tool could grab local portions from the browser's storage and synchronize it with a remot (e.g. GIT) repository on user demand. Likewise updates could be fetched from the same remote repo. Authentification could be organized in the standard GIT way?

The identifier is just an id of the active board. By putting it in the url, boards become linkable.
Regarding storing lists locally, I'm not sure if that is the right path for tdo. I would be more interested in cloud features offered by Firebase, such as cloud login, real-time database and hosting. It would be great if someone is willing to help me integrate that.
If you really need a local tdo server, feel free to fork it and adapt it to your needs. That should be rather simple to implement.

Do you have an "outline" of the path you'd like to go with tdo? Would you like to share it?
I will try to sketch an outline about what I would have in mind regarding a further development of tdo (in one of my next posts). Maybe (hopefully) a combination would be a possible solution? 1+1=3 ;-)
Some small requirements (all points to make the utilisation more "crisp" for real projects):
USER INTERFACE

operation on mobile devices (currently, on a tablet, todo items can not be re-opened)
...

DATA MODEL

integration of multiple "responsibles", i.e. one person per task responsible for its completion
integration of optional "deadlines" to specify the expected / required completion date
integration of "scores" to provide a transparent completion status (and its project-specific meaning)
auto-eMail-reminders based on scores, deadlines and the current date
...

RELIABILITY / QUALITY

interface for (self hosted) version control of list (using GIT)
...

Anyway, I could imagine supporting you with integrating cloud operation, real-time database and hosting. But I have to say that I am a "generalist". I know lots of programming languages as I (in the past) always adapted to existing project environments and picked any language necessary to contribute to the task at hand (ASSEMBLER, C/C++, JAVA, HTML, CSS, JavaScript, PHP, Python,...). But I am not a real "specialist" in any of them. Rather in the field of "development environments". The same with other technologies (microcontrollers, hardware design, content management, ...) . So, I might be rather a help regarding INFRASTRUCTURE, QUALITY and HELPFUL DOCUMENTATION. If that's ok for you, we can give it a try.

@nnako Sorry for a late reply. I don't have an outline, so I wrote my ideas in a new issue #18. Please comment there what would you like to see in tdo and why.

@mstijak Just a short question about the storage of tdo's data (when not using a Ghist account). Where exactly do I find the user's tasks / lists / boards in the browser's local storage? And what data structure is used for this? I could figure it out, I guess, if I had at least a starting point within the source code. Any help?

@nnako You can find it here: https://github.com/mstijak/tdo/blob/master/app/data/middleware/persist.js.  You can simply type localStorage.tdo in the console and you'll see all data (json encoded).

@nnako any chrome like browser

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @smttl, @guilhermebkel, @108abhishek, @gnrbrhm, @michaelvils, @Rzbck, @unitycoder

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @RickWong, @zofe, @muratyilmaz, @baontq, @suttonj, @maxoja, @divanoli

Exciting Job Opportunity at Microsoft!
Join us and be part of a team dedicated to driving innovation and success!
How to Apply:
To apply for this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.white-duke3, @gauravagrwal, @Skydenden, @rhijjawi, @rohanrhu, @Loopshape, @youfoundron, @helloworld9912, @xlBuckExpresslx, @karstenjakobsen, @crazyants, @ffrmns, @lorenzodisidoro, @bliotti, @h0tcatSub, @mitzampt, @fastnodclub, @PlayerOfGames797, @1levi1, @mayaleh

Yeah how about you don't click that link.


Fix creation of remote control processor

Please consider merging this fix for remote debugging.
Cheers,
Ivan.

@izaera Totally up for keeping more explicit dependencies, specially when not doing it leads to errors!
Just one tiny change, can you use regular require instead of require_relative, so we stay consistent with the rest of the code?

OK. I’ll redo and resend.
Thx.


Add virtual directory for redirection

https://review.docs.microsoft.com/en-us/powershell/azure/azurerm-ps-in-docker?branch=qinezh%2Fredirection
will be redirected to
https://review.docs.microsoft.com/en-us/powershell/azure/install-azurerm-ps?branch=qinezh%2Fredirection&view=azurermps-6.12.0

@qinezh - Not sure how this actually addresses the problem, because the OPS warnings still indicate that there are redirect issues. Please fix.


Consider a more 'swifty' API

How about a more 'swifty' API, e.g.
pipe(incr, into: square)

instead of
pipe(incr, square)

Ha, we actually made concat a lil more "swifty" in order to support trailing closures.
concat(buttonStyle) {
  $0.backgroundColor = .red
}

// vs.

concat(buttonStyle, and: {
  $0.backgroundColor = .red
})

This is a bit of a cheat code, but it works with concat as it scales to its variadic number of arguments.
concat(a, b, c, d, e, f, and: g)

The pipe and chain functions also are overloaded to create larger pipelines. I'm not sure what to do about scaling them, though:
pipe(a, into: b, then: c, before: d, andThen: e)

I'm also not sure if these kinds of labels are readable together. E.g., using with makes the meaning less clear:
with(x, pipe(incr, into: square))

I think in general the goal of these functions is to reduce visual noise and focus on the pipeline and composition. It should also be easy to manipulate existing pipelines and plug a new function into them anywhere, which is a flexibility that seems at odds with adding argument labels.
We're up to discuss ideas that address these problems, though!

Heh, you got me confused. I am also not sure what is the best API after all ;-)

I'm using it just for say that's a huge achievement and work, I'm reading it and impress, do you had a Medium article about it?
That's a game change for Swift. I'm pretty sure that work will have recognition in the future when people would be aware of these workings.


[Suggestion] Ability to customize url format

Instead of /blog/{slug} I'd like to use /{year}/{month}/{day}/{slug}. As far as I can see, I can change the path in the routes/web.php file but that doesn't persist through the site because you're not using Laravel's named routes.
I can come up with a PR for this if you'd like. Basically my approach would be this:

Name all the routes instead of using the hardcoded slugs.
In Frontend\BlogController@showPost, accept any of the following as parameters (slug, year, month, day, post id, etc). When present, all will be used as where() conditions when finding the post.

From there all someone has to do is customize the routes file and it should just work. There might be a few more things to this later down the road related to #95 and upgrading but I don't think it's something that would be too hard to work past.
Thoughts?

@tomschlick I like that idea a lot. Would users still retain the ability to edit their post slugs? That came into play as users in other countries had language difficulties that necessitated the request for editing slugs. If that will still be the same, then I see no reason why we can't merge in a PR doing what you propose.

Yeah I don't see any reason why that wouldn't work. The route wouldn't care what the slug was, it just passes it back to the controller to parse & query the database.
I'll probably create this as two PRs. One for the named routes and another for the url params.

What is the status for this?
I would really like to be able to have the url like:
www.myblog.com/my-post1
www.myblog.com/my-post2
etc..
i have seen that there has been some discussion about it, but is it still ongoing?

@joelstahre Yes, this is still a feature being developed on. Will let you know more as soon as possible.


Relay servers for Wallet Connect

Description
These are the servers that facilitate communication between a wallet (ie. Valora) and a dApp. In the future these servers will only handle the handshake between walletdapp before upgrading to peer-to-peer WebRTC.
WalletConnect represents the premium wallet experience for our end users and we don't want to be relying on a third party to facilitate that connection.
The actual server we need to run is built in typescript and can be autoscaled as long as it's backed by a Redis cluster. We only need to run two instances, staging and production, because WC is independent of the blockchain network used.
The code is here:
https://github.com/WalletConnect-Labs/walletconnect-v2-monorepo/tree/master/servers/relay
Acceptance Criteria

There is a pre-built image of the relay server distributed by the WalletConnect team or our custom built image.
There is a new command in celotool that allows deploying a new instance of the relay server.
The command also creates a Redis (memorystore) instance (see blockscout cloudsql example).
Relay server is being deployed with a horizontal pod autoscaler enabled with the max of desired replicas set to 1.

This seems pretty straightforward, wrap the service in a helm chart and add the deployment to celotool


SSE event stream ends without 'end' event being triggered

I currently have the problem that my particle events stream socket seems to be closed after a while without any notification if no events are happening.
I use an implementation similar to this one:
https://github.com/spark/sparkjs/blob/master/examples/node/get-event-stream-forever.js
New events will just not be sent to the client. Meanwhile, the client's 'end' event is not triggered, so there is currently no way for the client to know and no reconnection happens. This can be an issue with a user's network infrastructure, too. I need to find a way to ensure the channel still works after keeping it open for a longer while.
Is there already heartbeat/keepalive mechanism that allows the client to find out if the connection is still open? Or do I have to send events from my node client periodically to keep the subscription channel open and find out if I have to reconnect?

Since 5.1.0 the event stream should reconnect automatically. If it cannot, it will throw an error event. Some example code is below:
api.getEventStream({ deviceId: 'aaa', auth: '111' }).then(function (req) {
  req.on('event', function(e) {
    console.log(e);
  });
  req.on('error', function(err) {
    console.log(err);
    // trigger full reconnection here
  });
}, function(err) {
  // initial connection error
});

Does this help?

Nope, this does not seem to work. When a disconnect occurs, neither reconnection is triggered nor is there any error event emitted.

I tracked this down and the problem surfaces when the events stream contains a extra newline character.
I went to https://api.particle.io/v1/events?access_token=[token] and collected the public sample below:

...
event: DoorMsg
data: {"data":"Door closing or opening","ttl":"60","published_at":"2017-01-27T21:51:28.335Z","coreid":"51ff6e065082554906370887"}
event: Motion
data: {"data":"2","ttl":"60","published_at":"2017-01-27T21:51:29.392Z","coreid":"35003d000447343337373738"}
event: 15.734930C, 41.187805%, 29.667389in.Hg
data: {"data":"null","ttl":"60","published_at":"2017-01-27T21:51:28.262Z","coreid":"370036000547343233323032"}
...

The module throws a squashed SyntaxException at EventStream.parseEventStreamLine after the event "event: 15.734930C, 41.187805%, 29.667389in.Hg" is parsed and doesn't recover.

This might be a separate problem. IMHO we have here:

event stream parsing fails due to improper handling of newlines in payload
event stream interruption/disconnection is not handled as expected and is not signaled to listeners

Has any progress been made on this issue? After several hours I stop getting events from a getEventStream listener and I am not receiving any 'end' or 'error' events indicating that I should restart the stream. Interestingly, another getEventStream listener that is running in the same process but listening for different events continues working as expected.
I have resorted to restarting the stream periodically, however that's not a very principled solution given that I don't know what's causing the disconnection in the first place.

I have the exact same problem.

when will this issue get solve ? any workarounds ?
Thanks

@rlaurente it's possible your event handler is throwing an exception and crashing the event processing loop.
Add a try {} catch inside your event handler:
  stream.on('event', event => {
	try {
	  console.log(`received event`, event);
	  crash;
	} catch (error) {
	  console.error('crash in event handler', error);
	}
  });


Checkt twitter omniauth

Currently it is not possible to sign in via twitter.
If you try, you get redirected to the homepage.
Output from the logs:
2023-04-28T15:40:22.440413+00:00 app[web.1]: D, [2023-04-28T15:40:22.440342 #45] DEBUG -- omniauth: (twitter) Request phase initiated.
2023-04-28T15:40:22.498987+00:00 app[web.1]: E, [2023-04-28T15:40:22.498925 #45] ERROR -- omniauth: (twitter) Authentication failure! 401 Unauthorized: OAuth::Unauthorized, 401 Unauthorized

we deleted the login via twitter.


destroy test repository

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7c5274b) 98.47% compared to head (524cf31) 98.47%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4053   +/-   ##
=======================================
  Coverage   98.47%   98.47%
=======================================
  Files          43       43
  Lines        4068     4068
=======================================
  Hits         4006     4006
  Misses         62       62

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


Partial assignability error where T extends a compatible type

TypeScript Version:  2.5.3 / playground
Code
interface Vertex {
    name: string;
}

function create<T extends Vertex>(): void {
    let x: Partial<T> = {
        name: "A"
    };
}

Expected behavior:
The code should compile, since every T will have name: string in its definition.
Actual behavior:
error TS2322: Type '{ name: "A"; }' is not assignable to type 'Partial<T>'.

This is a correct error. create<{ name: "B" }>() is a legal call that the function body fails to fulfill correctly

The same error occurs if I change the Vertex definition to:
interface Vertex {
    name: "A";
}

In which case would the function fail with this definition?

Found it: create<{ name: null }>(). Thanks, I'll close this issue.


Skip brew typecheck if running on stable

Since brew typecheck <tap> is not in a brew release yet, we need to skip this for stable tap_syntax checks otherwise: https://github.com/Homebrew/homebrew-core/actions/runs/10516750573/job/29139564452?pr=182076

Style/NegatedIf: Favor unless over if for negative conditions.

Keeping up with when unless is preferred or not is really difficult.


Feature idea: alert modals

What about an alert functionality without any buttons, just a text showing up, to create simple notification modals?

It is coming in the new version ;)


ExpressionFuzzer: Add support for COALESCE expressions

Allow expression fuzzer to generate and test random expressions that have coalesce expressions in them.

@mbasmanova  this is done right?

Added in #3070


Executor never shows openapi-generator-cli output

When running generate-api-lib-sources executor, the openapi-generator-cli output is never shown to the user, even when an error happens. This makes it very hard to debug issues with the OpenAPI schema or even with the executor configs.
It would be useful to show the output on error or even consume the verbose flag from executor context.
Note: the storybook plugin for nx has an silent option which hides the build logs output, maybe this could be used instead of the verbose flag

We were running into this as well, and we frequently had to run openapi-generator-cli directly just for debugging purposes.
Here's a PR that logs the command used to invoke openapi-generator-cli, as well as forwards the stdout to the nx logger: https://github.com/trumbitta/nx-trumbitta/pull/71

@all-contributors please add @JulioC for ideas


Uncaught Invariant Violation: Expected to find a valid target.

Uncaught Invariant Violation: Expected to find a valid target.
I'm facing this issue when I drag a tag.

Envs:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
A related tag here: Uncaught Error: Invariant Violation: Expected to find a valid target

DND and a little buggy.
It rarely works and most of the times i'v that error.
Envs:

"react": "^16.9.0"
"react-dom": "^16.9.0"
"react-dnd": "^5.0.0" (as recommended)
"react-dnd-html5-backend": "^3.0.2", (as recommended)

I'm getting the same error. I think it's because the key prop isn't stable
Currently it's set to key={${tag.id}-${index}} in ReactTags.js:389
Changing it to key={tag.id} fixed the problem for me!

I'm having the same issue with dragging a tag and throwing the invariant error about not being able to find a target.
Same as above, except
"react": "^16.8.0"
"@hot-loader/react-dom": "^16.8.0"

Some potential causes is inconsistent ids (maybe based on their position on the array) and keys. We'll see if I can make any progress on my own error.


Having some troubles... and documentation seems inaccurate to current version (v1.7.0)

First let me preface this with the comment that I really like the potential of this library, and love the fluent interface approach. I am sorry if I sound frustrated, but with Elasticsearch being a steep learning curve anyway, it is tough if the documentation for the library is incomplete or inaccurate. I can't say for sure if I am running into bugs or just undocumented "features" :)
I am using documentation provided at http://nest.azurewebsites.net/
Starting at the beginning. I need to create a mapping.

Starting with attribute mapping on my type, the documentation shows many ElasticType attribute properties, but on mine I only see two: Name and important one for me IdProperty which I'm guessing (and hoping) is to tell Nest what property should be mapped to the _id for the type. It is not shown nor discussed in the documentation.

Documentation for Map<T>() shows an empty method signature:

var response = this.ConnectedClient.Map<ElasticSearchProject>();

However in code, it takes a Func<PutMappingDescriptor<T>,PutMappingDescriptor<T>>
While I have seen references in other places in the documentation of use of this with m => m.MapFromAttributes() I'm not 100% certain what this does (though I guessed that it should map my type from the attributes I specified.
I have (I believe) successfully created my mapping (though I don't see any reference to the id field when I get the mapping back using a raw REST call)

Attempting to use the BulkAsync call was failing for me using almost exactly the code example in the Bulk documentation. Documentation shows:

var descriptor = new BulkDescriptor();

foreach (var i in Enumerable.Range(0, 1000))
{
    descriptor.Index<ElasticSearchProject>(op => op
        .Document(new ElasticSearchProject {Id = i})
    );
}
var result = client.Bulk(descriptor);

With that code, and my attribute annotated type, the result from the Bulk call were validation errors on every operation saying "missing index, missing type". SO I then found undocumented... an option for "Index" and "Type" on the descriptor.Index call:
bulk.Index<MyType>(op => op.Index("myindex").Type("mytype").Document(objMyType));

This didn't help :(
Only when I finally tried the also undocumented FixedPath option on the BulkDescriptor object did it actually specify the index and type in the request (and I'm guessing it would have to be the same index and type for all documents in the BulkDescriptor request..?
bulk.FixedPath("myindex","mytype");

After finally getting a bunch of documents indexed, I notice it is NOT using my id field as specified in the ElasticType(IdProperty="id") attribute. Instead I get internally generated IDs for all of my documents.

Am I just missing some more documentation somewhere? Is there a bug with the code? I can't tell for sure. Thanks in advance.

Well to follow up a little more, I think I have indeed run into at least one bug with the Bulk API. I have attempted to explicitly set an Id field, and the request still doesn't specify anything in the index action json (I was able to locate the request sent in debugger)
bulk.Index<GeoLocation>(op => op
   .Document(location)
   .Id(location.id)
   .Index("geo")
   .Type("geolocation"));

As mentioned earlier, if I don't use "FixedPath" I get the errors about "missing index, missing type".
Here is the request generated from above code:
`http://es:9200/geolocation/geolocation/_bulk,
Request:
{ "index" :  {} }
{"id":"kec.1234","location":{"lat":41.75979,"lon":69.38798},...}

If however I call directly the Index method, I DO get back the proper id, and it generates the proper request (inferring the index, type, and id all correctly)
var res = elasticClient.Index(location);
results in:
Url: http://es:9200/geolocation/geolocation/kec.1234
Request:
{
  "id": "kec.1234",  "location": {    "lat": 41.75979,    "lon": 69.38798  },...
}

Hey @thunderstumpges
I can only echo your frustration with the out of date documentation, we are taking a radical different approach for NEST 2.0 combining unit/integration tests and roslyn to create something we call literate tests to drive our documentation. This means that any code in the documentation will be code thats always compiled and run on each commit.
Our .Map() method translates to the elasticsearch Put Mapping API which is used to update an existing index's mapping. You can only introduce new fields and not update existing mappings. Its usually a better approach to describe the mapping when you create the index using the CreateIndex() api.
var x = this.Client.CreateIndex(index, s => s
	.AddMapping<ElasticsearchProject>(m => m
		.MapFromAttributes()
		.Properties(pp=>pp
			.String(pps=>pps.Name(p=>p.Country).Index(FieldIndexOption.Analyzed))
		)
	)
);

Here we create index and add a mapping for ElasticsearchProject and explicitly tell it to infer the mapping from the CLR properties and associated attributes using MapFromAttributes() using .Properties() we can override what it infers with a more explicit mapping for the Country property.
On your bulk problem, this seems strange and would love to get to the bottom of whats going on. Our unit test that test .Document() on an unbounded bulk request (e.g not calling Fixed*) still translates properly to an operation with an inferred index, type and id, see here

I have some thoughts on what goes on, have a look at the tests trying to reproduce this:
https://github.com/elastic/elasticsearch-net/blob/develop/src/Tests/Nest.Tests.Unit/Reproduce/Reproduce1528Tests.cs#L26
When no default index is set on index settings the bulk throws an NRE which is bad.
But when it is the json that is generated: https://github.com/elastic/elasticsearch-net/blob/develop/src/Tests/Nest.Tests.Unit/Reproduce/Issue1528.json does include index and type and id.

Still trying to capture the behavior you are seeing of completely empty bulk operation headers:
{ "index" :  {} }

Can you share a bit more how your POCO looks like?

Thanks for the quick turn-around. Your tests look just about like my setup. I am setting the default index on connection settings, and it is the index for this operation. A couple of differences I see are that I am not specifying ElasticProperty.Name but rather letting the inference determine it. Also I have a significantly more complicated structure with a couple of Nested objects. Guessing the difference must be somewhere in there.
Here is my complete poco. The fields are organized and named this way as it is the format of an existing json data source, so while I can change the attributes, I cannot change the structure or property names.
    // ReSharper disable InconsistentNaming
    [ElasticType(IdProperty = "id")]
    public class GeoLocation
    {
        [ElasticProperty(Type = FieldType.Nested)]
        public GeoLocationCountry country { get; set; }

        [ElasticProperty(Index = FieldIndexOption.NotAnalyzed)]
        public string id { get; set; }

        [ElasticProperty(Type = FieldType.GeoPoint)]
        public GeoLocationLatLon location { get; set; }

        [ElasticProperty(Type = FieldType.Nested)]
        public List<GeoLocationName> names { get; set; }

        public string type { get; set; }

        public class GeoLocationCountry
        {
            [ElasticProperty(Index = FieldIndexOption.NotAnalyzed)]
            public string code { get; set; }
            [ElasticProperty(Index = FieldIndexOption.NotAnalyzed)]
            public string id { get; set; }

            public string name { get; set; }
        }

        public class GeoLocationLatLon
        {
            public double lat { get; set; }
            public double lon { get; set; }
        }

        public class GeoLocationName
        {
            [ElasticProperty(Index = FieldIndexOption.NotAnalyzed)]
            public string language { get; set; }

            public string name { get; set; }

            [ElasticProperty(Index = FieldIndexOption.NotAnalyzed)]
            public string script { get; set; }
        }
    }
    // ReSharper restore InconsistentNaming

Our .Map() method translates to the elasticsearch Put Mapping API which is used to update an existing index's mapping. You can only introduce new fields and not update existing mappings. Its usually a better approach to describe the mapping when you create the index using the CreateIndex() api.

 Are you saying I should not do that? The intent was to create it if it did not exist, and update it should there be an additional field down the road.
Another thought, is that I don't get any metadata in my mapping definition when echoing it back form elasticsearch. According to the book I'm reading (Elasticsearch: The Definitive Guide chapter 10), after putting the mapping (especially after explicitly setting the id "Path")
// update the mapping.
var mapRes = await elasticClient.MapAsync<GeoLocation>(m => m.MapFromAttributes().IdField(md=>md.Path("id")));

I should have an _id metadata field: (excerpt below is from the book example)
PUT /my_index
{
    "mappings": {
        "my_type": {
            "_id": {
                "path": "doc_id" 1
            },
            "properties": {
                "doc_id": {
                    "type":   "string",
                    "index":  "not_analyzed"
}  }  }  }}

Thanks for setting up that unit test. I have cloned the repo, built the test, and I will work on updating it to reproduce the issue I have. Likely won't get around to it again until Monday morning though.

Umm... so actually two of the four tests in that file fail on tip. Both of the "Explicit" tests fail for me:
Issue1528_ExplicitTypeAndIndex Failed
  { "index" :  {"_index":"myindex","_type":"sometype","_id":"my_id_0"} }
  Expected: True
  But was:  False

I think the assertion json file is incorrect for the "explicit" methods, it still has 'default-index':
{ "index" :  {"_index":"default-index", "_type":"mytype","_id":"my_id_0"} }
{"name":"my_id_0"}

I am modifying the json file locally to continue testing.

 Are you saying I should not do that? The intent was to create it if
it did not exist, and update it should there be an additional field down the road.

This is fine as long as you are aware of its append only nature :+1: Changing properties won't be updated. I personally typically go down the road of reindexing into a time based index e.g myindex-[timestamp] and using aliases to point myindex to the latest version. If your dataset is too big for this pattern and your domain is not volatile put mapping with its caveats can work just as fine for you :)
This is related to the id path behaviour you've witnessed, given a completely empty index with no mappings in place.
var mapRes = await client.MapAsync<GeoLocation>(m => m.MapFromAttributes());
var mapRes2 = await client.MapAsync<GeoLocation>(m => m.MapFromAttributes().IdField(md=>md.Path("id")));

The call to IdField() will only succeed if its done as part of mapRes as part of mapRes2 it will fail since a mapping is already in place for GeoLocation.
side note: id field paths support has been completely removed from Elasticsearch 2.0.

On the topic of the weird bulk output, your POCO does not seem weird at all, will try to make the tests fail too later today.

Umm... so actually two of the four tests in that file fail on tip. Both of the "Explicit" tests fail for me:

argh I thought I pushed those up before running out Sunday, sorry for that noise. Just pushed them back up.

Ok, I think I figured it out. Based on documentation on connecting it said we could use SetJsonSerializerSettingsModifier to:

update the internal Json.NET serializer settings to your liking
I used this to change the ContractResolver to our own internal contract resolver which globally ignores some property names (from Thrift and Avro serialilzation of objects). I didn't really need it in this scenario, but looking at the code now that I have it, you guys have a very important ContractResolver, and it seems that we should really not change it out. (I may have to inherit another one from yours to ignore our properties sometime later, but not needed for this particular case).

After removing the setting of the ContractResolver, everything worked great.
Maybe good to mention the pitfalls of changing different things in SetJsonSerializerSettingsModifier. what things are actually alright to change? Our internal ContractResolver is actually inherited from CamelCasePropertyNamesContractResolver... any chance we could use something like this for Pocos that don't use camel-case property names? Is that already handled by the ElasticContractResolver ?
By the way, thanks so much for the very quick and thorough follow up. I appreciate the time taken to explain reasoning and caveats to different things. Everything is working great now!

Thank you @thunderstumpges for not rage-quiting :smile: your feedback is very very valuable.
You are right if you set your own ContractResolver through SetJsonSerializerSettingsModifier it needs to inherit from NestContractResolver, i will make it throw a hard exception if it does not!
If all you want to do is change how property names are handled globally you can also modify this by calling:
DefaultPropertyNameInferrer(p=>p.ToLowerInvariant()) on ConnectionSettings if you want to leave the property names verbatim call DefaultPropertyNameInferrer(p=>p)


Download data link colour contrast

Under the download data tab, the buttons indicating download actions are green with a white link. When focus is on them, this link colour turns to navy blue which is too close to the green

@lizscuffell Colour has been updated

Colours altered and contrast ratio fixed. Closing issue


Error creating commit: Invalid request

[INFO] --- site-maven-plugin:0.9:site (execution1) @ lolchatapi ---
[INFO] Creating 18 blobs
[INFO] Creating tree with 19 blob entries
[INFO] Merging with tree 9a11f1f5c5d4e3441cb3949dcf9e907fa6c1f1d0
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 20.494 s
[INFO] Finished at: 2014-08-29T12:09:59+02:00
[INFO] Final Memory: 31M/267M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.github.github:site-maven-plugin:0.9:site (execution1) on project lolchatapi: Error creating commit: Invalid request.
[ERROR]
[ERROR] Expected data to be of type "object"; value was: nil.
[ERROR] Expected data to be of type "object"; value was: nil. (422)

Expected data to be of type "object"; value was: nil.
Expected data to be of type "object"; value was: nil. (422)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:216)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:116)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:80)
	at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
	at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:120)
	at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:347)
	at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:154)
	at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
	at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
	at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
	at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
	at org.codehaus.classworlds.Launcher.main(Launcher.java:46)
Caused by: org.apache.maven.plugin.MojoExecutionException: Error creating commit: Invalid request.

Expected data to be of type "object"; value was: nil.
Expected data to be of type "object"; value was: nil. (422)
	at com.github.maven.plugins.site.SiteMojo.execute(SiteMojo.java:440)
	at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:132)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208)
	... 20 more
Caused by: org.eclipse.egit.github.core.client.RequestException: Invalid request.

Expected data to be of type "object"; value was: nil.
Expected data to be of type "object"; value was: nil. (422)
	at org.eclipse.egit.github.core.client.GitHubClient.createException(GitHubClient.java:552)
	at org.eclipse.egit.github.core.client.GitHubClient.sendJson(GitHubClient.java:643)
	at org.eclipse.egit.github.core.client.GitHubClient.post(GitHubClient.java:757)
	at org.eclipse.egit.github.core.service.DataService.createCommit(DataService.java:397)
	at com.github.maven.plugins.site.SiteMojo.execute(SiteMojo.java:434)
	... 22 more

Everything worked great 5 days ago, but today I started getting this error. Is there anything on my end that I can do?

hi guys, I am passing for this problem. I have updated my profile with email. But I am not getting to deploy mvn deploy.
Any idea?

hi guys, I am passing for this problem. I have updated my profile with email. But I am not getting to deploy mvn deploy.
Any idea?

Still with the same issue in 0.10. This was released with the PR submitted above?

I gave up on it being fixed in github.   I created a branch locally and published a "fixed build" which we use for all publishing.  One day when it's resolved we'll use the "fixed" version, but until then, we'll carry on without it.

This is still failing on my en pinging to 0.11 :/, getting:
[ERROR] Failed to execute goal com.github.github:site-maven-plugin:0.11:site (default) on project MY-PROJECT: Error creating commit: Invalid request.
[ERROR]
[ERROR] Expected data to be of type "string"; value was: nil.
[ERROR] Expected data to be of type "string"; value was: nil. (422)
What I did is to put in the settings.xml username and password that are different from the one set to git but that are allowed to push to the gh-pages branch.

I had the same problem with the version 0.11, I solved it by setting the real name on my github profile (https://github.com/github/maven-plugins/issues/85)

Similar issue.

org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.github.github:site-maven-plugin:0.13-SNAPSHOT:site (default) on project threads: Error creating c
ommit: Invalid request.
nil is not a string.
nil is not a string. (422)
Not able to push artifacts: http://stackoverflow.com/questions/14013644/hosting-a-maven-repository-on-github/14013645#14013645

Getting the same issue as @zkejid anyone know which string is nil?


have name and value attributes and still cannot pass Survey Form

Describe your problem and - if possible - how to reproduce
I'm working in the Responsive Web Design Examples Section
And I'm working on the Survey form.
In my code I have a "name" attribute as well as a "value" attribute. I also have a "unique ID" for each radio button. Please refer to my screenshot below.
Yet I still cannot pass this Survey Form as per the forked copy with the JavaScript checker.
It reports an error that says that I have not grouped my checkboxes with the name attribute.
Yet, my checkboxes are grouped, and surrounded with a border as if they are grouped.
I've used the Read-Search-Ask and have yet found another way to group a bunch of checkboxes.
I've even placed the "name" attribute so that it appears first - to no avail.
Add a Link to the page with the problem
https://codepen.io/joemakestheweb/pen/VGrNJw?editors=1000
Tell us about your browser and operating system

Browser Name:  Chrome
Browser Version:  68
Operating System: Windows 10

If possible, add a screenshot here

@theonlyjoeyouneedtoknow According to the challenge an instruction says

User Story #13: Inside the form element, I can select a field from one or more groups of radio buttons. Each group should be grouped using the name attribute.

In your code you are missing radio buttons.

Thank you so much. I'm sorry. I feel silly. Maybe, I'm taking Free Code Camp too seriously? Still, no excuses. My apologies. I passed! I'm so happy, thank you for pointing out my error. Because of you, I passed!

@theonlyjoeyouneedtoknow could you close this issue as it has been solved?


Multithreaded option for VariantQC

Hello,
This is a very interesting tool that I would like to try out. I'm wondering if there is a multithreaded option for VariantQC or if there is a recommended way of performing QC for whole genome sequencing data on ~2000 individuals.

Split by individual?
Split by chromosome? or smaller genomic regions?
How to combine results from various runs?

Thanks,
-Jon

Hi,
Thank you for this tool. On the same note is there an option similar to "--merge-evals" from here to merge per chromosome output?
Thanks

Hello - in its current form, VariantQC accepts one VCF as input and therefore doesnt support anything like --merge-evals.  There is some complexity associated with converting to a MultiVariantWalker (what i think would be needed to make this work).  This might not be ideal, but as you probably know you could do a VCF merge upfront, and then run VariantQC.

@jhchung This took much longer than hoped, but we recently significantly reworked how we iterate the VCF which should make it considerably faster, even if not truly multithreaded. please try version 1.26 or higher to get this change.


[Docs] - Customise the Readme file

A fix to issue #252 ,

I added the Chimoney logo to the readme file

Add the Twitter link 🖇 to the mode of Communication in the Readme file

Enhanced the readme file

changes made @phyleria

I noticed that the updated file documentation is not written in Markdown
I'm linking a proper md update below

I noticed that the updated file documentation is not written in Markdown I'm linking a proper md update below

It was written in Markdown

This i not proper MD

This i not proper MD

I think you are referring to the way I used HTML tags in the readme file,
HTML tags can be used in markdown file as well


Inference on trained model

Hi,
I've gotten my model trained, however, i'm unable to run inference on it.
The original data I have is laid out as such:
-- train/
-- images/
--- img0.tif
--- img1.tif
...
-- masks/
--- img0_mask0.tif
--- img0_mask1.tif
--- img1_mask0.tif
--- img1_mask1.tif
--- img1_mask2.tif
...
This layout is used in a train and test (valid) folder. The aim is a segmentation aim with no classification.
When i look at the data generated by run_train.py i see it created two folders in logs:
-- logs/
--- 00/
---- events.out.tfevents
---- net_epoch=1.tar
...
---- stats.json
--- 01/
---- events.out.tfevents
---- net_epoch=1.tar
...
---- stats.json
When i want to run run_infer.py i don't know what to select as input dir or model paths and looking at the notebook usage too. I don't understand where to start from for the inference wsi or tile and how to get it running.
If someone can help me on that one it would be amazing,
Thank you
Best,
Leonardo

Hi,
We have provided two scripts (run_tile.sh and run_wsi.sh) that can be used to guide you how to perform inference. We have also provided detailed instructions on the README page. Please go through that and let us know in particular what you are struggling with.

Hi @LeoMira-1999 - the trained models are the .tar files with the epoch at which they were generated in their title. The stats.json file shows the evaluation metrics at each epoch so you can use this to decide which of the trained model .tar files to keep for use in inference.
As @simongraham pointed out in the above comment, you can use the run_tile.sh / run_wsi.sh files as scaffolds to make an inference script - the README goes into more detail about what each of the arguments are in inference.
For inference, you would select one model (.tar file) and use its path for the --model_path argument in inference.
The input_dir is the location of the images you want to conduct inference on.
The wsi / tile methods are a matter of choice and resources - if you have whole slide images (+ good hardware with >48GB RAM (in my experience) and a good GPU ), you can use the wsi option to run inference on tissue in a whole slide image. If you want to infer on smaller .tif patches, use the tile option.
Does this make sense?


Make sure the stsci x1dints reader is robust to different file structures.

Lots of people use this FITS file structure, but they don't always have the same extension/header structure. Check the x1dints we have access to through the submissions, and make whatever checks we need to make sure they work reasonably well. (We should probably add some warnings in places where they break.)

I tested our all the x1dints-ish files that were submitted to the Data Challenge, and so far it seems to work everywhere.
Once we get real data, I suspect there'll be some changes to the pipeline outputs that we'll need to account for yet again, so I think we should probably make an official "JWST x1dints" reader and move the current reader to a legacy "kludgy x1dints" reader.


Pass extra configuration keys for feature flags

This PR extends the feature flag functionality by allowing users to pass extra keys as configuration.
@mistercrunch, @xtinec should we add a dummy KEYS_CONF to config.py?

Codecov Report

Merging #6943 into master will decrease coverage by <.01%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master    #6943      +/-   ##
==========================================
- Coverage   63.98%   63.98%   -0.01%
==========================================
  Files         422      422
  Lines       20537    20539       +2
  Branches     2230     2231       +1
==========================================
+ Hits        13141    13142       +1
- Misses       7264     7265       +1
  Partials      132      132

Impacted Files
Coverage Δ

superset/views/core.py
74.91% <ø> (ø)
:arrow_up:

superset/assets/src/SqlLab/App.jsx
0% <0%> (ø)
:arrow_up:

superset/assets/src/dashboard/App.jsx
0% <0%> (ø)
:arrow_up:

superset/assets/src/explore/App.jsx
0% <0%> (ø)
:arrow_up:

superset/__init__.py
74.21% <100%> (+0.2%)
:arrow_up:

superset/views/base.py
69.1% <100%> (ø)
:arrow_up:

superset/assets/src/featureFlags.ts
62.5% <50%> (-8.93%)
:arrow_down:

...uperset/assets/src/datasource/DatasourceEditor.jsx
61.84% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Can I merge https://github.com/apache/incubator-superset/pull/6808 first?

@mistercrunch sure, go ahead!

I'd like to make the feature conf dynamic too, much like the feature flags.
One thing to note here is that the feature flag and feature conf could be one and only thing. I'm not sure how other feature flag frameworks do this. I don't have a strong opinion on how it should be other than it should be possible to make this dynamic and we should follow standards. Now I wish I knew more about how Gatekeeper worked :)

@mistercrunch cool, I'll update it after your PR. I think the function approach should allow us to reproduce GKs:
def GET_FEATURE_FLAGS_FUNC(feature_flags_dict):
    if (
        is_admin(g.user) or
        is_data_scientist(g.user) or
        g.user.id % 10 == 0
    ):
        feature_flags_dict['some_feature'] = True
    return feature_flags_dict

My first implementation of this extended the feature flag from a boolean to a dict, but @xtinec suggested separating them. I also have no strong opinions on one vs. the other approach.

@mistercrunch this is ready for review.

I finally took the time to do a bit more research. I had heard "LaunchDarkly" is an up-and-coming feature flag as a service company. I hear you, originally I also thought "really?!", but thinking about things like Gatekeeper and multi-language support and AB testing I thought "hmm well yeah probably makes sense"
Here's the way they to it (cmd-f to multivariate)
https://docs.launchdarkly.com/docs/creating-a-feature-flag
They just have one namespace of features, and the feature can be boolean, but also other types, including nested/JSON.

@mistercrunch I liked the idea, and I've actually did that for lightweight pipelines. :)
Closing this.


Don't use
 in alert block

Follows https://github.com/go-gitea/gitea/pull/29121

When I implemented alert blocks I was always testing the markdown in issue comments. I used
 for line breaks and it looked good. I have since learned that the markdown on README files doesn't allow these tags. So a comment with
> [!NOTE]
> If you're interested in using our APIs, we have experimental support with [documentation](https://try.gitea.io/api/swagger).

looked like this in a comment

but looked like this in a README

So I changed how we render the alert block by having the alert itself have a dedicated paragraph, so line breaks happen naturally between paragraphs.
Before

After

New alert blocks are only available in 1.22 so no need for backport

Does this new behaviour match GitHub rendering in both comments and .md files?

Yes, this brings us closer to GitHub


[wip] Combine .css.proxy.js files into one

Changes
Background
Background: https://github.com/snowpackjs/snowpack/discussions/1334
For every .css file imported into JS, Snowpack creates one .css.proxy.js file. This requires an additional network request as well as a tiny amount of main thread blocking time to add the styles to the page.

As you can see, navigating through a Snowpack site can mean extra network requests, as well as delay in rendering while styles download.
Preloading CSS
Instead, a user can opt to concatenate all the CSS imported by JS, and only that, into one file. This can be achieved by passing { preloadCSS: true } to '@snowpack/plugin-optimize'. This serves all the CSS as one request, and injects it into the <head> element.
In this way, you get the best of both worlds: all the CSS-in-JS features you’d expect with a modern setup, but compiled away into zero runtime, and in a way optimized for browsers.
Tested on skypack.dev:

~48 kB saved between no longer needing to inline <style> tags in the payload + no more wrapper code needed around all the JS files
26 network requests saved (and no waterfall)
No runtime/no dynamic <style> injection/no main thread blocking (though main thread time was only ~4ms on a fast machine)

Why is this PR so big?
This does incorporate #1406, which is part of the length. Merging that will reduce this.
But the rest of the length comes from a few other changes which made the work of preloading CSS a heck of a lot easier. This is marked a draft PR for review because all 4 of the following could be split out as one or multiple prereqs if needed:

chore: Organized HTML, JS, and CSS logic into separate files
feat: A new __snowpack__/optimize-manifest.json was created so that this plugin can better optimize, as well as giving people generating their own HTML a hook to see generated files from Snowpack (inspired by a discussion from  https://github.com/snowpackjs/snowpack/discussions/1334)
chore: hypertag was added, which is a much better way of scanning for <script type="module"> tags than we were using (and 0 deps! 🎉 )
chore: node-html-inject (created by yours truly to solve this exact problem) was added as a better way to inject arbitrary HTML inside<head> and <body>. While I found many HTML parsers and HTML AST libraries for Node, I couldn’t find any that easily let you mutate the HTML or AST. But even of the few that could, they were either very heavy and cumbersome or they mangled the original HTML in some way (because if you mutate something, everything has to be rebuilt). So I came up with a lightweight solution that’s basically just pure string injection, but in a better way than we were doing.

And of course, all these made the process of preloading CSS easier, which is the final change on top of these.

Testing
Tests were added to test this behavior.

Docs
Plugin docs were updated

Don't forget the TODO about the debug logs

I’m not sure if I can fix that. By default, console.debug all pipes through, and I want it to only show when we specify the --verbose command. I think we may have to leave that until we provide a better log API for plugins, or we set a LOG_LEVEL=VERBOSE env var or something


Accessing cell during cell import returns 500 error.

Depending on the timing of accessing cell, the .pmeta file under cell may not exist.
When importing cells, modify the implementation to overwrite the .pmeta file under cell without deleting it.

Released in v1.6.6


[Bug]: Vercel 빌드 오류

Describe the bug
Deploy 과정에서 빌드 오류가 뜨네요.
[12:36:17.521] Running build in San Francisco, USA (West) – sfo1
[12:36:17.567] Cloning github.com/JioChoi/JinseoBlog (Branch: main, Commit: 9be1ee2)
[12:36:17.706] Previous build cache not available
[12:36:18.289] Cloning completed: 720.733ms
[12:36:18.426] Running "vercel build"
[12:36:19.001] Vercel CLI 28.16.13
[12:36:19.315] Installing dependencies...
[12:36:19.680] yarn install v1.22.17
[12:36:19.779] [1/4] Resolving packages...
[12:36:20.023] [2/4] Fetching packages...
[12:36:41.313] [3/4] Linking dependencies...
[12:36:41.317] warning "react-notion-x ><EMAIL_ADDRESS>has unmet peer dependency "@babel/runtime@>=7".
[12:36:41.317] warning "react-notion-x ><EMAIL_ADDRESS>has incorrect peer dependency "react@^15 || ^16".
[12:36:41.317] warning "react-notion-x ><EMAIL_ADDRESS>has incorrect peer dependency "react-dom@^15 || ^16".
[12:36:41.319] warning "react-notion-x > react-pdf ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^5.0.0".
[12:36:41.321] warning " ><EMAIL_ADDRESS>has unmet peer dependency "@next/env@*".
[12:36:41.356] warning Workspaces can only be enabled in private projects.
[12:37:00.179] [4/4] Building fresh packages...
[12:37:00.665] Done in 40.99s.
[12:37:00.698] Detected Next.js version: 12.1.6
[12:37:00.700] Running "yarn run build"
[12:37:00.916] yarn run v1.22.17
[12:37:00.982] $ next build
[12:37:01.772] Attention: Next.js now collects completely anonymous telemetry regarding usage.
[12:37:01.773] This information is used to shape Next.js' roadmap and prioritize features.
[12:37:01.773] You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
[12:37:01.773] https://nextjs.org/telemetry
[12:37:01.773]
[12:37:01.860] info  - Checking validity of types...
[12:37:08.032] info  - Creating an optimized production build...
[12:37:47.357] info  - Compiled successfully
[12:37:47.358] info  - Collecting page data...
[12:37:49.712]
[12:37:49.712] > Build error occurred
[12:37:49.714] TypeError: Cannot read properties of undefined (reading 'value')
[12:37:49.715]     at getPosts (/vercel/path0/.next/server/chunks/89.js:301:35)
[12:37:49.715]     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
[12:37:49.715]     at async getStaticPaths (/vercel/path0/.next/server/pages/[slug].js:387:19)
[12:37:49.715]     at async buildStaticPaths (/vercel/path0/node_modules/next/dist/build/utils.js:491:31)
[12:37:49.716]     at async /vercel/path0/node_modules/next/dist/build/utils.js:632:119
[12:37:49.716]     at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/trace/trace.js:79:20) {
[12:37:49.716]   type: 'TypeError'
[12:37:49.716] }
[12:37:49.780] error Command failed with exit code 1.
[12:37:49.806] Error: Command "yarn run build" exited with 1

어떻게 해야할까요...?
To reproduce
https://github.com/JioChoi/JinseoBlog#getting-started
대로 따라했습니다.
Browser
Google Chrome
Validations

[x] Follow our Code of Conduct
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

안녕하세요 :) 아래 사진처럼 vercel에 환경변수가 설정되어 있을까요?

Can you redeploy? I used to forgot adding environment and push the branch first and it build from the time I don't have environment.


Switch the current implementation to use btree_cursors

The current implementation is very much a bodge of using the range method largely in a unintended fashion.
Instead I propose we switch to using btree_cursors, which is a new API that was proposed with the express intention of improving the implementations of interval collections such as this ones this project provides.
The btree_cursors feature is currently a nightly unstable feature so this would more concretely lock us onto the nightly version of rust than the currently used, but more work-a-roundable features  we are using at the moment.
Overall I think this will greatly improve the simplicity of the code, and will also greatly improve the performance due to using more fit-for-purpose methods on the raw BTreeMap.
Related btree_cursors Issues and PRs
Tracking Issue: rust-lang/rust#107540
Proposal: rust-lang/libs-team#141
Implementation: rust-lang/rust#105641
Related Entry Proposal: rust-lang/libs-team#73

I should probably report on my finished usage and experience after using the new feature on the tracking issue.

closing in favor of #25

I should probably report on my finished usage and experience after using the new feature on the tracking issue, if that's the right place for feedback I'm not sure.

Probably.

@workingjubilee I ended up not really using btree_cursors as much as I planned in favor of ripytide/btree_monstrousity . Hence I don't really have much feedback other than cursors seem useful it certain situations.


Initially page load showing all Images in the slider. It has to show only one Image at a time.

Initial Page load all the Images in the slider are displayed. I need help to show only one Image to be shown on page load. Tried all the ways that are shared my forum members but no help. Please help. Please find more details below.

====================================================================
Issue facing:

Expected Output: One Image at one Time:

[  paste your jsfiddle link here  ]
use this jsfiddle to reproduce your bug:
http://jsfiddle.net/simeydotme/fmo50w7n/
we will likely close your issue without it.
====================================================================
Steps to reproduce the problem

...
...

====================================================================
What is the expected behaviour?
...  Only one Image should be displayed when page is loaded.
====================================================================
What is observed behaviour?
...
====================================================================
More Details

Which browsers/versions does it happen on? All Browsers
Which jQuery/Slick version are you using? New Version
Did this work before? No

Hej vikaspaturi!
You've forgot to add a jsfiddle link where you reproduce your described bug. But a link is not necessary at all, because the in the issue template referenced jsfiddle is showing exactly what you want: a slider where only one image is shown.
So just open the jsfiddle and you find all needed information.

No Fiddle Please.

If you won't post a fiddle, we can't directly help you. But the default configuration of slick is to only show one picture configured by the parameter slidesToShow with a default value of 1


Use the same version of marked in our build and our plugins

Re-opened from https://github.com/alphagov/govuk-design-system/pull/3130 because @dependabot closed the PR it was based on, which closed our PR too

Unblocks<EMAIL_ADDRESS>via https://github.com/alphagov/govuk-design-system/pull/3157 and closes:

https://github.com/alphagov/govuk-design-system/issues/2767

HTML changes include:

Automatic "smart" punctuation is unescaped ‘ → ‘ everywhere
Automatic "smart" punctuation is now applied to Nunjucks rendered HTML
Indentation is better preserved around code blocks
Macro options Markdown now shares the same renderer

Copy pasting from the previous PR:
Diff appears to mostly be:

formatting changes to the code blocks,
nice pick ups from smartypants that we've missed

We should edit this example code, since there's a LOT of instances of it that smartpants has to transform. There's a few others in the diff, but none of them appear 312 times 😆  Quite a few of them come from the govuk-frontend YAML properties, so we could potentially grep and replace there to reduce transformations further.

However, there is some weirdness, like:
Replacing -- with – in some examples
diff --color -r .cache/public/get-started/updating-your-code/index.html deploy/public/get-started/updating-your-code/index.html
1146c1153
<       govuk-section-break
govuk-section-break--visible
govuk-section-break--xl
govuk-section-break--l
govuk-section-break--m
---
>       govuk-section-break
govuk-section-break–visible
govuk-section-break–xl
govuk-section-break–l
govuk-section-break–m
1167c1174
<       govuk-list
govuk-list--bullet
---
>       govuk-list
govuk-list–bullet
1171c1178
<       govuk-list
govuk-list--number
---
>       govuk-list
govuk-list–number
1242c1249
<       govuk-label--s
---
>       govuk-label–s

Suspect we just want to wrap these in
Some component options tables are weird?
For some reason, the Notification Banner and Summary List component Nunjucks options tables are showing up as big diffs (though they look like they're just getting a bit of formatting). The Summary List options tables are about 5000+ lines of the diff file.
Is there something funky with the page code that's affecting these tables versus other components where the tables are not getting reformatted?

Ahh well spotted @domoscargin, default support for -- → – is too risky for CSS classes
I've pushed a commit https://github.com/alphagov/govuk-design-system/pull/3134/commits/5208150d6291c37df5dc72d30325780feacdd12a to configure quotes and ellipses only
Diffs wise, I've got expected changes only for Notification banner and Summary list. Do you mind checking if it's not one of the other PRs that merged recently? This PR is based on https://github.com/alphagov/govuk-design-system/pull/3133 now

Thanks @domoscargin


Remove extra space after topic lables

Remove extra spaces after the topic labels before commas in list

fixed in front-end-refactor


Config frequency of recording

Currently, temperatures are uploaded every 30 seconds.
Allow this to be configurable.
Eventually, I'd prefer to have the temperature file on the OS be read continually and 'streamed' to the client. Until then, make it configurable as it's pretty quick to do.

Wont fix, side stepped with #6


doc(Usage of discriminator): Encourage inheritance rather than polymorphism

What type of PR is this?

documentation

What this PR does / why we need it:

Clarify things between inheritance and polymorphism

Which issue(s) this PR fixes:

None

Special notes for reviewers:

Same as  #255 in old repo

Changelog input
Update section on "Usage of discriminator" : Encourage inheritance rather than polymorphism

Additional documentation
This section can be blank.

Hi @RubenBG7, @jordonezlucena,
Could you take a look at this PR and let us know what TEF's position is?
Thanks in advance

@patrice-conil: sorry for not providing our final feedback (I though we had done it!).  We're ok with this change -- actually, it is already actioned upon other Sub-Projects.


Feature/sftp

Hi,
referring to #110 I've created the ability to enable SFTP.
Cheers,
Jan

Hi jmara,
great work!!! Can you add some documentation lines to the README.md and please add somthing like
"This is a ChrootDirectory ownership problem. sshd will reject SFTP connections to accounts that are set to chroot into any directory that has ownership/permissions that sshd considers insecure. sshd's strict ownership/permissions requirements dictate that every directory in the chroot path must be owned by root and only writable by the owner. So, for example, if the chroot environment is /home must be owned by root. "  see https://wiki.archlinux.org/index.php/SFTP_chroot
If you just enable it and create the group with a user then it is not working, because of the wrong ownership. Just to provide the user a hint.
Thanks

Hey Patrick,
you're welcome. I've added a Section for SFTP, added your suggestion to the FAQ and described the attributes.
Cheers,
Jan

Hi,
awesome work!!! Thanks


chore(Measure.FiniteMeasure): remove an @ replace with arguments for implicit parameters

The porting note seems to have mis-diagnosed the problem here. This is more accurate and easier to read.

[ ] depends on: #10191

I haven't pinpointed the exact issue yet but this form seems an improvement.

Thanks :tada:
If CI passes, please remove the label awaiting-CI and merge this yourself, by adding a comment bors r+.
bors d+

bors merge


Make find widget toggles configurable settings

Related to https://github.com/Microsoft/vscode/issues/4309#issuecomment-225023802
Add to settings.json a way to always enable the case-sensitivity feature of Find.

Setting? I believe that's stuff we should store and retrieve using the memento stuff...

It would be the default value

@jrieken not sure what the memento stuff is. As a user, my expectations is that I'd be able to change that in my user settings (the JSON file) and then never have to think about it again.

Find across files already persists the Find options (case sensitivity, whole word and regex). Find in a single file doesn't. Using configuration setting for default value for Find in File and keeping the existing persisting logic in Find across files will make experience inconsistent.
Can we not use the same persisting logic for Find in File instead of configuration setting?

I'd love to see a setting default for Alt+C (case-sensitive search and match). I'd like to be able to configure Code such that it defaults to case-sensitive search/match, and then be able to use Alt+C to toggle it on/off (but Alt+C should not change my default setting).

Let's track the setting persistence in #6158


[Fix] Make DiagnosticsReporter check scheme before converting to File targeting 1.3.x

This is https://github.com/sbt/zinc/pull/719 but targeting 1.3.x

To validate this pull request against other sbt modules, please visit this link.


Question: How can I increase the contrast in my terminal prompt with doom-wilmersdorf and vterm

Thanks for maintaining this package!
What happened?
Low contrast in the username part of my terminal prompt when using vterm with doom-wilmersdorf:

Alacritty is on the left with a theme applied by PyWal. At the right doom-wilmersdorf theme. As you can see the username is hard to read.
What did you expect to happen?
Better contrast in the terminal prompt
Steps to reproduce

Install vterm
Use doom-wilmersdorf theme
Setup your terminal prompt PS1

System Information

Emacs version: 27.1
Emacs Distro: None
Operating System: Fedora 33
Installed commit of doom-themes: 18cddefb3c8b9da63086496142ad6756f0bc9167

It's hard to tell without knowing what ansi codes your prompt is putting out, but if you're using evil, you could try to determine what faces are involved: switch to normal mode, place your cursor over the hard-to-read bits and invoke M-x describe-char. At the bottom of the popup that appears it will list all faces at point.
I'll guess that that is vterm-color-red and=or vterm-color-magenta. You can tweak them with:
(custom-set-faces
  `(vterm-color-red ((t (:foreground "black")))))

And play around with that until you get the effect you want, but frankly, I think the best solution would be to tweak your prompt, or use a prompt theme that matches your Emacs theme as close as possible.
As our issue tracker isn't for how-to/config questions I'll close this, but feel free to follow up here.