Make golden tests easier to update

Make the golden tests easier to update. Currently, you can replace the expected list of CBOR terms with an empty list and run the test again; it will print the expected list.
However, using files would be even easier. The problem with files, which we had before, is the relative paths to the files, which gave problems when you run the tests from different directories and/or in Nix CI. See #1351.

This was done in #2362.

This was done in #2362.


API check failed with new key (but not really?)

Using fresh clone of master and new key purchased today, I received this error on all ptc and google account:
[2017-05-31 16:29:09] Login procedure started. [2017-05-31 16:29:17] Login successful. [2017-05-31 16:29:17] Niantic Official API Version: 0.63.1 [2017-05-31 16:29:17] Latest Bossland Hashing API Version: 0.63.1 [2017-05-31 16:29:17] We have detected a Pokemon API Change. Latest Niantic Version is: 0.63.1. Program Exiting...
If I tun off api check in config the bot runs fine and I do not appear to be shadowbanned. All captcha checks passed as well.
OS: Windows 8.1

Do me a favor, at line 1064 add the following
print(format(officialAPI_int))
print(format(hashingAPI_int))

Let me know what are their values and are they different

[2017-06-01 11:16:39] Login successful. [2017-06-01 11:16:40] Niantic Official API Version: 0.63.1 [2017-06-01 11:16:41] Latest Bossland Hashing API Version: 0.63.1 631 610 [2017-06-01 11:16:41] We have detected a Pokemon API Change. Latest Niantic Version is: 0.63.1. Program Exiting...

610?????!
You are using 0.61.0 Hashing.... in your requirement.txt, do you have this line?
git+https://github.com/pogodevorg/pgoapi.git@develop#egg=pgoapi

I would advise that you stop your bots until you are sure you are using latest hashing instead of old one. I have no idea why are you using 0.61.0 Hash

I cloned fresh, but I double-checked requirements and it's exactly as it appears on the current commit.

question- do we need to clone recursively or something? when i clone i do:
git clone -b master https://github.com/PokemonGoF/PokemonGo-Bot
then do: pip install -r requirements
but that should have nothing to do with the install correct? api is on bossland's end, right?

We are using API from https://github.com/pogodevorg/pgoapi/tree/develop
What is werid is that you are using a old api. There have to be something that went wrong or your pervious installation of pogo API wasn't overwrite by new one. @pogarek You able to give suggestion?

Have you tried running PokemonGo-Bot-Updater.bat?

I tried that but it seems to hang will installing the web submodule.

OK I see what's happening- somehow the pip install is install to my Python3.5 installation!

So problem solved? :) Can this issue be closed? :)

Uninstalled 3.5 and removed from PATH, reboot, FIXED!

:)


Texture.flip_horizontal() and flip_vertical() does not trigger a redraw

Versions

Python: 3.6.1
OS: Windows
Kivy: 1.10.0
Kivy installation method: pip

Description
It's not possible to flip texture at runtime as shown in example below. I haven't researched this, but on the surface it seems like graphics instructions should bind to tex_coords. Not sure about difficulty, maybe the correct fix is to document it.
edit: correction, seems like dispatching a property event for "texture" on flip may be enough
Code and Logs
from kivy.base import runTouchApp
from kivy.lang import Builder

runTouchApp(Builder.load_string('''
BoxLayout:
    Image:
        id: img
        source: 'data/logo/kivy-icon-512.png'
    Button:
        on_press: img.texture.flip_horizontal()
'''))

I expect this to flip the texture, but it doesn't. Behavior is the same when using a Rectangle with texture directly. One workaround is to explicitly replace the .texture of a Rectangle after flipping it.

I've run into a similar issue and have a few suggestions:
We should add a note to the Image module documentation: "Note: Class objects using Image and sharing the same source share a texture by default. Set nocache=True when instantiating the object to create a unique texture."
It would also make sense to implement a flip_horizontal() and flip_vertical() within the Image class and include as part of that implementation a Class.texture.dispatch(self) event which is required in order to actually flip the texture.


Array vs. Object for lists of links

I'd like to re-raise this for discussion, per https://github.com/radiantearth/stac-spec/issues/127#issuecomment-405300991

Sprint 3 Static Group:
We recommend arrays.  This is largely in line with these comments #127 (comment)

Fixed with #207.


adjtime implements olddelta incorrectly

Module or chip used:  ESP32-WROOM-32
IDF version (run git describe --tags to find it): v4.0 through v4.2

Problem Description
The POSIX and GNU libc definition of adjtime specifies:

If the olddelta argument is not a null pointer, the adjtime function returns information about any previous time adjustment that has not yet completed.

Note the word "previous". The ESP-IDF implementation incorrectly returns
the current time adjustment that has not yet completed, which is rather useless.
Fix
Swap the two outer if blocks in the adjtime function https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L193.

@tve Thanks for reporting, we will look into.

Hi @tve!
Thanks for your report. I agree, now we need to call it twice to get the same result. I will fix it and provide backports up to 4.0.


As a pathway owner, I need to get email notifications when an issue is reported for an activity

Already some work done towards this.

This still suffers from the fact that the email gets sent, but then it gets quarantined by the outgoing SMTP server. I can manually release the emails, but it takes days for them show up in the quarantine release area so it's useless.


Mod over const

Related RFC
To get proper GHA testing going, and to showcase current state of mod_const

bors merge


Install glasskube-autoupdater if a package with auto update enabled gets installed

Is your feature request related to a problem? Please describe.
We released the glasskube-autoupdater package. But it does not get automatically installed if a package with automatic updates is installed or automatic updates get enabled later on.
Describe the solution you'd like
Glasskube auto updater should get installed if a package gets installed with auto update enabled.
Describe alternatives you've considered
Additional context

I can try this


Fix missing semicolon in nginx configuration snippet

Thanks!

I'm really confused by why the Travis build is failing. It's clearly not the fault of this commit, though, so I'm going to merge it anyway, then assume fetal position and cry until the build gets better.

@taion I've checked it. Some issues with browserify, probably will be fixed in a bit.


Testsuite improvements

Core - Full Integration Build 6089 outcome was FAILURE using a merge of 6b762e15c9e94fe9e51d2b54b8ccecab44d43acd
Summary: Exit code 1 (new) Build time: 00:01:09

Full integration - Windows Build 4528 outcome was FAILURE using a merge of 6b762e15c9e94fe9e51d2b54b8ccecab44d43acd
Summary: Exit code 1 (new) Build time: 00:03:24


On month change cancel and apply buttons are out of screen

On opening this picker, it is located in the bottom left corner with a margin of ~3px. When chaning the  month to a "longer one" (additional row of dates) the close and apply button are pushed down and then are out of screen.
How can i apply my styling to the overlay of this picker to relocate it?
My idea was to
::ng-deep .mat-datepicker-popup {
  top: 50%;
  left: 50%;
  transform: translate(-50%, -50%);
}

but the styling doesn't get applied. The CSS itself is fine. If i manually add it in de dev tools, i get the desired behaviour to locate the popoup in the middle of the screen.
Also is there a way to add a black background with opacity to the popup? Like the normal mat datepicker, filling the whole screen.

Hi,
This repo is no longer supported, please consider using the repository angular-material-components instead.
npm install --save  @angular-material-components/datetime-picker

Free free to open issue here.
Thank you very much.


[FINE] Set default value of dropdown list to be nothing

Fix for https://bugzilla.redhat.com/show_bug.cgi?id=1555487.
We're setting dropdown defaults if they're multiselect and dynamic to be the first value in the array. Prior to multiselect, this made sense, because the single select gets the options of  or  but the use case for wanting a multiselect that loaded without an option hadn't been considered. The problem with the workaround of creating an option that is nothing and have it say "Nothing selected" or something is that that option still gets picked. If the multiselect flag is set it might make more sense to set the default to have nothing selected which this does. Incidentally this works fine on master as of the time of this PR thanks to the work done in ui-components.

@eclarizio can you 👀 please?

@miq-bot add_label bug
@miq-bot assign @gmcculloug

@bdunne any chance you could 👀 for me please?

@miq-bot add_label blocker


make throttle threadsafe

This is from elasticsearch/logstash#1469
It makes throttle theadsafe

+1 Strongly needed :)

+1

+1

@suyograo can someone check this out please?

Hi @addisonj. I've run this in production but unfortunately I ran into an issue. I'll try and explain below.
For the purpose of this test, lets assume this is my config:
throttle {
  before_count => 0
  after_count => 10
  period => 60
  key => "%{UUID}"
  max_counters => 100000
}

I then ran a load test of around 500 req/s. This should result in ~30,000 UUIDs per 60 seconds. However, when I run this patch, after 100,000 requests it starts spewing log messages like:
{
  :timestamp=>"2015-08-31T11:19:16.913000+0000",
  :message=>"filters/LogStash::Filters::Throttle: Purging oldest counter because max_counters exceeded. Use a better key to prevent too many unique event counters.",
  :key=>"43b4dd82-4fd0-11e5-8b98-02fc3a693d29",
  :expiration=>2015-08-31 11:15:21 +0000,
  :level=>:warn
}

Using the original version of this filter I don't receive a single warning of this kind.
I think the keys aren't being expired after 60 seconds with this version, which means eventually you start churning keys. What I think should happen is that after 60 seconds old keys are removed so as long as I'm not running > 100,000 req per minute I should never see that warning.
Hope that makes sense.

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

+1

+1

I am not using logstash anymore these days, so its pretty low priority for me to dig in and get this fixed, but looking at the code (its been a loooong time), I could see a few places where it might be. Ideally, if someone wants to take this over that would be great, otherwise, if someone just wants to test a few changes for me, that would make it much easier to get this working.

This PR appears to be superseded by #5. Can someone confirm?

@timbunce comfirmed. logstash-filter-throttle is multithreaded since v4.0.0 (pull request: https://github.com/logstash-plugins/logstash-filter-throttle/pull/5). Issue can be closed.

@addisonj your pull request has been superseded by https://github.com/logstash-plugins/logstash-filter-throttle/pull/5. Can you close the pull request?


Running backstage-backend with readOnlyRootFilesystem=true in kube

Expected Behavior
There should be a elegant solution to running backstage backend in an environment with readOnlyRootFilesystem.
Current Behavior
When deploying the backend docker image in Kubernetes, I keep getting the following error when the startup script first tries to inject the config into module-backstage.hash.js:
2021-01-18T22:27:51.341Z backstage info Loaded config from app-config.yaml, app-config.production.yaml, k8s-config.yaml
2021-01-18T22:27:51.345Z backstage info Created UrlReader predicateMux{readers=azure{host=dev.azure.com,authed=false},bitbucket{host=bitbucket.org,authed=false},github{host=github.com,authed=true},gi
tlab{host=gitlab.com,authed=false},fallback=fetch{}}
2021-01-18T22:27:51.468Z catalog info Locations Refresh: Beginning locations refresh
2021-01-18T22:27:51.485Z catalog info Locations Refresh: Visiting 1 locations
2021-01-18T22:27:51.485Z catalog info Locations Refresh: Refreshing location bootstrap:bootstrap
2021-01-18T22:27:51.516Z techdocs info Creating Local publisher for TechDocs
2021-01-18T22:27:51.521Z proxy info [HPM] Proxy created: [Function: filter]  ->  https://example.com
2021-01-18T22:27:51.521Z proxy info [HPM] Proxy rewrite rule created: "^/api/proxy/test/" ~> "/"
2021-01-18T22:27:51.526Z app info Serving static app content from /usr/src/app/packages/app/dist
2021-01-18T22:27:51.629Z app info Injecting env config into module-backstage.e812caed.js
Backend failed to start up, Error: EACCES: permission denied, open '/usr/src/app/packages/app/dist/static/module-backstage.e812caed.js'

My organization has a global PodSecurityPolicy set with readOnlyRootFilesystem=true which is causing this to happen. Running docker locally or in minikube works with no issue. I have brainstormed many solutions, such as mounting a volume and copying the /usr/src/app files in, but it is not very simple and I would rather do something more easily maintainable.
Possible Solution
Modify whichever files need to be written to in a TMPDIR of some sort which can be mounted as a persistent volume from the kube platform.
Steps to Reproduce

Set readOnlyRootFilesystem: true in the PodSecurityPolicy in Kubernetes
Run a pod with the default docker image from the included Dockerfile.

Context
I am deploying this project as a replacement for an old internal infrastructure toolset on my team, and so I do not have fine-grained control over our Kube permissions. I imagine this will be a more common issue as more companies begin picking up this project, so having a good out of the box solution for this issue seems very desirable.
Your Environment

FROM node:14-buster
ArgoCD deployment
Kubernetes Server v1.17.14

Dockerfile (one of many I have I tried)
FROM node:14-buster

WORKDIR /usr/src/app

ADD yarn.lock package.json skeleton.tar ./
RUN yarn install --frozen-lockfile --production --network-timeout 300000 && rm -rf "$(yarn cache dir)"

COPY . .

# I have tried this, does not work with Kubernetes
# VOLUME /usr/src/app

CMD ["node", "packages/backend", "--config", "app-config.yaml", "--config", "app-config.production.yaml"]

Thanks everyone! I'm new with Backstage and a lot of the stuff it runs on, so please forgive me if I messed something simple up.

@rmanny This is caused by the config injection of the @backstage/app-backend plugin. You'd really need to remove that and choose a different way of serving the frontend, such as https://github.com/backstage/backstage/tree/master/contrib/docker/frontend-with-nginx.
That's unless we think it is useful to add an option that disables the config injection, would be happy to review a PR :grin:

@rmanny This is caused by the config injection of the @backstage/app-backend plugin. You'd really need to remove that and choose a different way of serving the frontend, such as https://github.com/backstage/backstage/tree/master/contrib/docker/frontend-with-nginx.
That's unless we think it is useful to add an option that disables the config injection, would be happy to review a PR :grin:

@rmanny This is caused by the config injection of the @backstage/app-backend plugin. You'd really need to remove that and choose a different way of serving the frontend, such as https://github.com/backstage/backstage/tree/master/contrib/docker/frontend-with-nginx.
That's unless we think it is useful to add an option that disables the config injection, would be happy to review a PR 😁

Thank you! That was my hunch after looking through the source code, glad to know an nginx container should fix the issue. Might take you up on that PR 😉

@rmanny This is caused by the config injection of the @backstage/app-backend plugin. You'd really need to remove that and choose a different way of serving the frontend, such as https://github.com/backstage/backstage/tree/master/contrib/docker/frontend-with-nginx.
That's unless we think it is useful to add an option that disables the config injection, would be happy to review a PR 😁

Thank you! That was my hunch after looking through the source code, glad to know an nginx container should fix the issue. Might take you up on that PR 😉

Hi @rmanny , have you got it resolved by following https://github.com/backstage/backstage/tree/master/contrib/docker/frontend-with-nginx?
I'm stuck with it now, as the nginx seems to be not working with readOnlyRootFilesystem=true in kube.
The inject-config.sh still wrote files to root file system. Did you get it sort out? How? Thanks in advance!

Yeah I am just using a basic nginx container for my frontend. To completely disable the frontend trying to host the backend, you will need to remove this line in your router, index.ts (https://github.com/backstage/backstage/blob/master/packages/backend/src/index.ts#L128) or add a boolean to disable it to your config.
Then make sure to remove inject-config.sh from your Dockerfile entrypoint, that is what is trying to write to the FS. For production you can apply the config during the build step of the frontend instead using --config flag.


Fix install-binary.sh to work with Helm v3

In the Helm V3 installation, the home command for helm is not available anymore. Adding a way to detect the helm version and execute the corresponding command to install it.
Also, when wget is used: Install failed on a system without curl, since the wget command used to retrieve latest version doesn't grep for the detected OS, and so returns a URL for every available OS, which subsequently causes wget to fail.
Fixes #87

HI, is there any plans to merge this PR? Im just wating for it to clean a bunch of CI steps on some repos.

HI, is there any plans to merge this PR? Im just wating for it to clean a bunch of CI steps on some repos.

Hi @gcavalcante8808,
There isn't any progression on this repo for some time (see #102 ).
I have created a fork of the helm-unittest, which include some new features including full support for Helm 3 (https://github.com/quintush/helm-unittest). The fork is completely backwards compatible with the existing test files.
Since a few days there is also a docker image available which contain helm and the unittest plugin (https://hub.docker.com/repository/docker/quintush/helm-unittest), this may help speedup you CI steps.
Greetings,
@quintush

@quintush, nice to hear!
I'll check it out. Thank you!

Close due to archiving repository.
Sorry for not presenting so long. I've been working on another project and don't have time to for helm-unittest.
Please consider other working forks like quintush/helm-unittest.


fix(rego): metadata for rego results

Fixes #1148 by adding the Parent and SourcePrefix fields to the generated rego input/metadata, and using those fields to reconstruct the original metadata when results are matched.

Thanks for the PR, will take a look.

hi @lyoung-confluent - are you still looking into this? Would be nice to have it in.


How to isolate or namespace third party library styles in one app?

Hi All,
We have a project which contains two apps in one page, one is for navigation which can be plugged to many other apps. Currently the navigation app is using styled components, but both apps may use the same third party libraries, such as react-bootstrap which may be in different version. Styled components can make self created components to be isolated from overriding or override other apps, but third party styles can't. My question is that is there a way to isolate or add namespace third party libraries' style to the navigation app only? Thanks.

Hi there - please post usage questions in a help forum like StackOverflow or our Spectrum community. This issue tracker is only meant for bug reports and feature requests.


⚠ Bofrost.at has degraded performance

In 8afea8e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1420 ms

Resolved: Bofrost.at performance has improved in 3fd9154 after 27 minutes.


Flutter: Character reference "&#x1B" is an invalid XML character.

On my flutter project by using junitreport I am making junit report but it is XML,
 flutter test --machine | tojunit --output test.xml

Now by Ant, I want to make Html from it. test.xml is at the root of my project, here I made build.xml:
<project name="genTestReport" default="gen" basedir=".">
  <description>
    Generate the HTML report from JUnit XML files
  </description>

  <target name="gen">
    <property name="genReportDir" location="${basedir}/unitTestReports"/>
    <delete dir="${genReportDir}"/>
    <mkdir dir="${genReportDir}"/>
    <junitreport todir="${basedir}/unitTestReports">
      <fileset dir="${basedir}">
        <include name="test.xml"/>
      </fileset>
      <report format="frames" todir="${genReportDir}/html"/>
    </junitreport>
  </target>
</project>

Now I run this command in root folder:
alt@Alis-MBP rr-front % ant -buildfile build.xml
Buildfile: /Users/alt/Projects/rr-front/build.xml

gen:
   [delete] Deleting directory /Users/alt/Projects/rr-front/unitTestReports
    [mkdir] Created dir: /Users/alt/Projects/rr-front/unitTestReports
[junitreport] [Fatal Error] test.xml:264:23: Character reference "&#x1B" is an invalid XML character.
[junitreport] The file /Users/alt/Projects/rr-front/test.xml is not a valid XML document. It is possibly corrupted.

Does it seem I need some configuration ?
Inside test.xml I have some testcase like below:
<testcase
classname=".Users.alt.Projects.rr_front.test.widgets.top3_pro.pro_container" name="overlay tap tap work" time="0.312">
      <system-out>[34m[🌎 Easy Localization] [WARNING] Localization key [widgets.pro_container.selected_a_pro] not found[0m</system-out>

Can you test what happens if you manually change the first line in the xml document to version="1.1"?

Can you test what happens if you manually change the first line in the xml document to version="1.1"?

I changed and it seems  Character reference "&#x1B" is an invalid XML character problem is solved but I have this error:
gen:
[delete] Deleting directory /Users/alt/Projects/rr-front/unitTestReports
[mkdir] Created dir: /Users/alt/Projects/rr-front/unitTestReports
[junitreport] the file /Users/alt/Projects/rr-front/test.xml is not a valid testsuite XML document
[junitreport] Processing/Users/alt/Projects/rr-front/unitTestReports/TESTS-TestSuites.xml to /var/folders/t0/4hlhr8115vs6d6v9zz701dl40000gn/T/null977921457
[junitreport] Loading stylesheet jar:file:/opt/homebrew/Cellar/ant/1.10.12/libexec/lib/ant-junit.jar!/org/apache/tools/ant/taskdefs/optional/junit/xsl/junit-frames.xsl
[junitreport] Transform time: 212ms
[junitreport] Deleting: /var/folders/t0/4hlhr8115vs6d6v9zz701dl40000gn/T/null977921457
BUILD SUCCESSFUL
Total time: 0 seconds
error -> test.xml is not a valid testsuite XML document

Hmm, possibly the character problem disappeared because the ant junit task doesn't allow version 1.1 files at all.
Can you modify any working test suite xml to 1.1 to see if those then also break?

I changed like this:

No difference, I still have a `not a valid testsuite XML document` error.

I have the same issue with GitLab CI XML parser - which doesn't support XML version 1.1:
Base report parsing error: JUnit XML parsing failed: 129:25: FATAL: xmlParseCharRef: invalid xmlChar value 27

@lukaszdudek-trueenergy As a temporary solution you can remove unsupported char sequence from generated XML file at GitLab CI level.
For example:
script:
  - flutter test --machine --coverage ${TEST_DIR} | tojunit --output ${OUTPUT_XML_NAME}
  - sed -i 's///g' ${OUTPUT_XML_NAME}

Where  is char sequence that broke XML structure.

The flutter test command exited prematurely, so the file with invalid characters was still uploaded to the artifacts in Gitlab CI/CD. In my case, the following helped:
variables:
   J_UNIT_REPORT_FILE_NAME: junit-report.xml
script:
   - flutter test --machine --coverage | sed 's/\\u001b//g' | tojunit --output $J_UNIT_REPORT_FILE_NAME || true
   - errors=$(echo 'cat //testsuites/testsuite/@errors' | xmllint --shell $J_UNIT_REPORT_FILE_NAME | awk -F'[="]' '!/>/{print $(NF-1)}')
   - failures=$(echo 'cat //testsuites/testsuite/@failures' | xmllint --shell $J_UNIT_REPORT_FILE_NAME | awk -F'[="]' '!/>/{print $(NF-1)}')
   - if [ "$errors" == "0" && "$failures" == "0" ]; then echo "All Tests Passed"; else echo "Some Test Failed"; exit 1; fi


[FLINK-15802][table] Support new type inference for table functions

What is the purpose of the change
Enables the new type inference for table functions. The PR conists of two commits.
The first commit refactors the code generation around table functions such that return data type and argument data type are read from the function at the same location instead of reading each in a separate code generator. This adds an additional runtime collector that is only responsible for result conversion. All existing tests pass after this change.
The second commit further develops the stack based on BridgingSqlFunction and the new type inference. Similar to FLINK-15487, we don't test every data type combination yet.
Brief change log
See commit messages.
Verifying this change
This change is already covered by existing tests, such as CorrelateITCase.
Additional tests added in FunctionITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): yes
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

CC @JingsongLi

Thanks for the feedback @JingsongLi and @dawidwys. I addressed your comments. I could further improve the runtime code in the collector. I also documented the behavior for collect(null).

@flinkbot run travis

@flinkbot run travis

@flinkbot run travis

Travis gave green light. Merging...


GitBlog

The current blog mechanism we are using in the NodeOS web page based on Jekyll
although flexible, is tedious to write and maintain (compared to Blogger or
WordPress) specially to include images, don't allow comments and it's not
integrated with our current workflow based on GitHub issues. That's why I've
though we could use the GitHub API to create a blog system that use the issues
as engine. This way, we allow to readers to left comments but also we get
notified about them, would not require any maintenance, and making a new blog
entry would be as simple as create a new issue and add a 'blog' tag :-) I think
it would be done as a pure client-side Javascript webapp that does requests to
the GitHub API, but it would be a plus if it would be integrated in Jekyll (at
least by using CSS) so it could be published in GitHub pages :-)

Ok, although it's not supported natively, it's also possible to automatically get a RSS feed... :-P
Definitely, this would be a really interesting project... :-)

https://gist.github.com/formula1/e4be0a9263da28c59d5a
Heres a very simple gist using angular for a blog based off of github. I was going to do more things, such as using markdown-js to display the markdown as well as use async to display eachone one at a time. However, I realized that I needed to host the clientside scripts which I decided against. That being said, built in about an hour and I have to get back to work

This is hilarious! :-P Yeah, I supossed it would be an easy and funny hack, but not just one hour! A little bit of CSS and it's ready ;-)
I had in mind other features like a main and a per-post page with comments, but showing only the list and pointing directly to the GitHub issue is a good solution, too. The markdown can be generated by GitHub itself, and in fact it can render the full posts with internal links to users and other issues, so there's no need to include external libraries :-)
It can be easily hosted at GitHub pages, that's in fact how current NodeOS webpage is hosted ;-) I've tried to publish it with http://gist.io/ but it's broken for new ones like this :-( So in resume: if you convert your gist to a proper GitHub project, anything that goes to a 'gh-pages' branch, or if you create a project with a special name (like the one we are using for NodeOS webpage project), it get's automatically published ;-) There's a lot of projects that host their webpages this way... :-)
I don't know how Jekyll plugins works, too, but I think you are correct. Also, they have the premise that "html is valid Jekyll code" so we could just drop your gist directly on the NodeOS webpage code, but I believe it would be better as a reusable project (at least as an independent project only for the blog engine) so other could fork it and create their own GitHub based blog... :-)
Good job! :-D

I've improved the RSS pipe to include the label, the new RSS feed is http://pipes.yahoo.com/pipes/pipe.run?_id=d98a965ba7d3f6555b939d707b18dc39&_render=rss&label=blog&owner=NodeOS&repo=NodeOS :-) I've also seen that the 'sort' query argument is not neccesary, so I'm not using ig

Changing the file name to index.html and with some quick tune-ups... here you have it online! :-D

I'm getting a 403 with pipes when debbuging : / And now I'm getting 403's with my own page. At some point we're probably going to need to create an API key to avoid this problem.
https://github.com/formula1/NodeOS-Blog
Just created the repo. Also made it look like your website, also handling errors and directing them to submit them to My repo to create an issue. If you run into 403 at some point, you'll know what it looks like.
I'm questioning just hosting the scripts on my server and allowing a Cors request. Of course that can be faked, but Its a bit frustrating how I have no access to browserified modules. Though, It's not 100% necessary.
Amoungst the issues that I have yet to solve are

Markdown to HTML - Github's rate limit causes 403's which is not a happy place to be. By avoiding the markdown parsing, each client may be able to avoid using up more bandwith than needed. Optionally, I can also cache everything on the clients computer and just request things one at a time, though it feels a bit invasive.
Highlighting Code-This is again something simple, however it is code that needs to be hosted somewhere

Something Else I can do is running the requests serverside (again on a different server) and making the Angular base its model off of what that gives back. This will be a slower then just directly getting it from github though.

I think the main problem with this is that google won't find it. So if someone searches for NodeOS blog they'll find the issue list first and not the actual blog, which makes it quite unnecessary to be honest.
Or you have to make the ?_escaped_fragment_= stuff work too, I'm not sure how much extra work is that with Angular.

I thought about the lack of SEO as an issue as well, though I figured build first figure out the rest later. Now adays google runs javascript. However, this doesn't make it more SEO friendly, only allows us to be a bit more lazy.
Should I just build a server and host it on nodejitsu/heroku? The theme isn't that complex and would be able to be converted to any template easily (though obviously jade/blade/etc would require a little more work). This would allow the github app to work without problem, rendering serverside would allow Search Engines to index as they please and etc. It probably won't take more than a few days making the github integration most likely most of the work.
Of course, an operating system's website would be using that operating system to get the feel of real world use cases.

I'm getting a 403 with pipes when debbuging : / And now I'm getting 403's with my own page. At some point we're probably going to need to create an API key to avoid this problem.

 :-( I don't know how limits could be gotten, I don't thnk we have use them so much... :-/

Just created the repo. Also made it look like your website, also handling errors and directing them to submit them to My repo to create an issue. If you run into 403 at some point, you'll know what it looks like.

Cool :-) We can work there or we can move it to NodeOS organization, what do you think?

I'm questioning just hosting the scripts on my server and allowing a Cors request. Of course that can be faked, but Its a bit frustrating how I have no access to browserified modules. Though, It's not 100% necessary.

This should not be necesary at all using GitHub Pages, that I believe it's the best option for this (and free!!! :-D ), and also probably being served by GitHub itself there's no so restrictive limits or could be found a workaround (they have some repo APIs to have better integration on organization pages...), and if you are thinking about hosting the scripts only for the browserified files, maybe your google-fu is not as good as my google-fu... ;-)

Markdown to HTML - Github's rate limit causes 403's which is not a happy place to be. By avoiding the markdown parsing, each client may be able to avoid using up more bandwith than needed.

Great :-)

Optionally, I can also cache everything on the clients computer and just request things one at a time, though it feels a bit invasive.

I don't think is a bad idea, in fact both localStorage and more specially IndexedDB were designed for this purposses :-) Problem is that we are using directly the GitHub API and don't think there's a way to be notified of changes on an issue... :-/ I think that just by rendering markdown on the client (although loosing links to other issues and users) would drop drastically the API usage since will only be needed a request per view.

Highlighting Code-This is again something simple, however it is code that needs to be hosted somewhere

The same, browserify as a service :-) Other option is to commit the node_modules folder...

Something Else I can do is running the requests serverside (again on a different server) and making the Angular base its model off of what that gives back. This will be a slower then just directly getting it from github though.

I think a pure client-side solution is possible and just enough, we don't need to deploy (and maintain) our own servers here. Static content (HTML, Javascript...) can be hosted on GitHub Pages or Dropbox, or as I showed you yesterday, as a gist + bl.ocks.io or gist.io to work as proxies, and probably rawgit.com too.

I think the main problem with this is that google won't find it. So if someone searches for NodeOS blog they'll find the issue list first and not the actual blog, which makes it quite unnecessary to be honest.

I don't think SEO is so important here, since there's only one NodeOS blog and just by the HTML content should be enough to be identified, specially if we link to it from the main page and/or is hosted in the same domain integrated with the current Jekyll web page project (that after @formula1 work, now I think the integration is not need anymore). Only problem would be with the blog post since they disappear and they will be available only as GitHub issues, but since we are mainly moving here at the issues tracker, I think it makes sense (and in fact was one of the points that I though about doing it this way :-P ).

Of course, an operating system's website would be using that operating system to get the feel of real world use cases. But we're probably not there yet

Yeah, this would be a good way to eat our own dog food :-D But we would need to re-enable Docker and network support first, and later find a (free) place where to upload it... :-/

Moving it to NodeOS

I would be honoured for it to be there. Everyone I've met so far here seems like a rockstar and this is a project I earnestly believe in. So if I can stamp my name here I would be glad.

Just precompiling it

Your google-fu is indeed superior. You win this one.

gist.io / github pages / hosting it at a proxy

Thinking about it, I could technically avoid gist.io and just grab the raw content. This would would send content-type: text/plain as a header. But most modern browsers don't care so long as you set <script type="text/javascript" .... And even if you don't it will probably interpret it as javascript anyway at least in Chrome/Firefox. But you are correct, gist.io flew by my head as a something much more complex than it had to be. Also, for github pages, if static html is considered a valid jekyll page. Will a text document? Granted, it may send the wrong headers (Most likely text/html for jekyll). I suppose, I'm getting snobby about headers when I'm getting stuff for free.

Moving current blog posts

One thing I considered was sorting the posts by update date. In this manner, you'll be able to choose which is considered "featured". In addition, some blog posts may get updated with new information/etc
I gotta get some sleep, tomarrow.

Moving it to NodeOS
I would be honoured for it to be there. Everyone I've met so far here seems like a rockstar and this is a project I earnestly believe in. So if I can stamp my name here I would be glad.

Set me as collaborator (or is it owner?) so I have permissions and I'll move it ;-)

Just precompiling it
Your google-fu is indeed superior. You win this one.

Not so, just that I remembered about seeing it sometime ago due to work :-P

gist.io / github pages / hosting it at a proxy
Thinking about it, I could technically avoid gist.io and just grab the raw content. This would would send content-type: text/plain as a header. But most modern browsers don't care so long as you set <script type="text/javascript" .... And even if you don't it will probably interpret it as javascript anyway at least in Chrome/Firefox. But you are correct, gist.io flew by my head as a something much more complex than it had to be. Also, for github pages, if static html is considered a valid jekyll page. Will a text document? Granted, it may send the wrong headers (Most likely text/html for jekyll). I suppose, I'm getting snobby about headers when I'm getting stuff for free.

Take a look at GitHub Pages. Honestly, for static content is one of the best hosting services you would want: cached, unlimited size, fast delivering, no transfer limits, SSL... and free!!!! :-P NodeOS web page is running there, and my P2P WebRTC serverless webapp was doing it too ;-) Dropbox is a good alternative too but it's more complex to manage for public sites, it's better to see it as an alternative for AWS.

Moving current blog posts
One thing I considered was sorting the posts by update date. In this manner, you'll be able to choose which is considered "featured". In addition, some blog posts may get updated with new information/etc

I would like a more traditional publish-date sort, but this could be configurable by user (it's just a parameter on the request). Main problem I have is with the publish date or authoship of old post, but maybe would be a good moment to do a clean-up of outdated info :-)

I just updated the blog to make it a little easier for me to work in because a single page started feeling a bit sloppy. https://github.com/formula1/NodeOS-Blog/tree/master
Things Working

Hello, highlight, markdown are all being browserified (though they were already before) then minified
I'm compiling the views instead of creating one big html page
everything gets created when calling npm start in /dist

Things Untested

Everything.
Blog List -> Blog Article
403 -> Login

I basically did this just so that I have reasonable files to work within. From here, I'll probably be making my own gitpage for testing purposes. Once everything feels correct. I also still need to incorperate historyjs but I didn't want to over complicate things right away.

Quick Update: https://github.com/formula1/NodeOS-Blog/tree/master
I have it working as intended. The markdown is getting parsed, the code is getting highlighted (I think?) and I am able to get issues via #issueid
From here...

Make pictures not look gigantic
403 -> Login
Start testing on github pages + gist

I just updated the blog to make it a little easier for me to work in because a single page started feeling a bit sloppy. https://github.com/formula1/NodeOS-Blog/tree/master

Great :+1:
I've cloned the repo on NodeOS organization and added you to the webpage team, now you have permissions to push there :-) I've also creted a gh-pages branch from master and renamed blog.html to index.html and it's automagically available at http://node-os.com/NodeOS-Blog (yes, domain too! :-D I think is a feature of GitHub organizations so it's possible to have a main organization page and per-project ones... :-) ). It looks good, so we could start moving the blog entries to GitHub issues and when it's ready just rename the project and have it available as the NodeOS blog (the current one is still available at http://node-os.com/blog, so we could add a redirect).
I think a good idea would be to just rename the project to GitBlog as I proposed, so it can became a generic tool that other can fork and use, while at the same time we can have NodeOS specific things on the gh-pages branch and use it as NodeOS oficial blog, what do you think?
By the way, when doing scroll the post are shown under the top traslucent menu I think it's a bug... but I love it!!! It's cool!!! :-D
P.D.: you are doing an awesome work, thank you! :-)

I've just updated project page to use GitBlog, good job @formula1! :-)

Thanks @piranna :)
Btw, you have posted in almost every issue I've looked up recently. Including a firefox bug! I thought you were badass before, You are at a new level badass in my eyes now. It's cool to think I'm following in your "bug awareness" footsteps as I enhance my google fu.
I'll tear down my repository so that theres no conflict. It was only up for you to see how I implemented it

Well, you should say this to my boss... :-P Also, don't having a life nor a girlfriend helps to achieve this... :-/

Aw..... Well, I don't want to hold you back from having a life. We need piranna's in the genepool! Practice makes perfect, understanding the problem well will make things more basic, Fearlessly doing something even when you aren't sure you will succeed also helps. Theres plenty of documentation on how to find a girlfriend/etc. It's all about whether or not thats the path you want to take now. : P


Submitting and agent overview in repo docs

Just so folks do not need to refer back to the website each time they want to know whats up.

Thanks. Rebased.


go-sciond: Handle zero up/down segments in DB lookup logic

https://github.com/scionproto/scion/blob/1a068d8ec9cf34b5748ce66fafe33f1501488a28/go/sciond/internal/fetcher/fetcher.go#L336 if getStartIAs(downs) or getStartIAs(ups) is a zero length list, the query we should no longer query the database and instead return 0 segments.

Maybe it's simplest/possible to handle this inside getSegmentsFromDB?


How print specific area in image file

I want to get the specific content of an image. How do I do that?
Something like that?
$ img = Image::make ($ file);
$ img->getContent (400,400) // x, y coordinates
It's possible?
example.png

I guess crop() is what you are looking for.


Application Load Balancer, LogAccessLogs doesnt follow best practise

Describe the bug
The policy generated by the LogAccessLogs method allows too wide of a permission on a prefix of loadbalancer
${log-bucket.Arn}/loadbalancer*
vs
${log-bucket.Arn}/arn:aws:s3:::${log-bucket}/loadbalancer/AWSLogs/${AWS::AccountId}/*
To Reproduce
Create a ALB and call LogAccessLogs on it, with a bucket
Generates a bucket policy with ${log-bucket.Arn}/loadbalancer*
Expected behavior
Generate a bucket policy with ${log-bucket.Arn}/arn:aws:s3:::${log-bucket}/loadbalancer/AWSLogs/${AWS::AccountId}/*
https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-access-logs.html
Version:

OS - Win 10
Programming Language - CSharp
CDK Version - 0.28

I would be happy to work on this if it's ok for you!

Feel free, havnt digged down anything in it yet

I made the change, but I was thinking if it is maybe too much restrictive then... is there any situation in which you would like to have access to multiple accountId under this?

I think that can be up to the usre to add a less restrictive policy for those rare cases?
AWS writes it like this:

For Amazon Resource Name (ARN), type the ARN of your S3 bucket in the following format. For aws-account-id, specify the ID of the AWS account that owns the load balancer (for example,<PHONE_NUMBER>12). Do not specify a wildcard for the account ID, as this would allow any other account to write access logs to your bucket. To use a single bucket to store access logs from load balancers in multiple accounts, specify one ARN per account in the bucket policy, using the corresponding AWS account ID in each ARN.
arn:aws:s3:::bucket/prefix/AWSLogs/aws-account-id/*

Otherwise anyone can send data to the bucket

Yes, you're right. I will open a pull request for this


Action is being dispatched twice

Hey Guys,
First of all thanks for this great side-effects library \m/.
Secondly I got one small issue with the implementation described in the doc.
I have used async example to implement one todo app and the issue is actions are being dispatched twice once directly without going through sagaMiddleware and another one through sagaMiddleware.
So each and every Action which I am listening for using take side-effect is being dispatched to reducer directly as well as via saga.
Tried to go through stack trace as well but it just give me all this info. I can't figure out what's the issue in here.
Let me know if this is due to some configuration switch or what could cause this issue.
Cheers,
Suraj

Could you post here code of your saga?
On Sun, 4 Sep 2016 at 13:58, Suraj Savita<EMAIL_ADDRESS>wrote:

Hey Guys,
First of all thanks for this great side-effects library \m/.
Secondly I got one small issue with the implementation described in the
doc.
I have used async example to implement one todo app and the issue is
actions are being dispatched twice once directly without going through
sagaMiddleware and another one through sagaMiddleware.
So each and every Action which I am listening for using take side-effect
is being dispatched to reducer directly as well as via saga.
Tried to go through stack trace as well but it just give me all this info.
I can't figure out what's the issue in here.
Let me know if this via some configuration switch or something else.
Cheers,
Suraj
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yelouafi/redux-saga/issues/527, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJWMknu24vW0i6vALq9Vn2TaO9Gnv14gks5qmrJYgaJpZM4J0hN2
.

Here is the code:
//configurStore.js:
export default function configureStore(preloadedState) {
const sagaMiddleware = createSagaMiddleware()
sagaMiddleware.run(rootSaga)
return {
...createStore(rootReducer, applyMiddleware(sagaMiddleware)),
runSaga: sagaMiddleware.run
}
}
//saga/Index.js
export function FetchAPIActor(url = config.api_root, method = 'GET', payload = {}) {
let request_map = {
method: method,
mode: 'cors',
redirect: 'follow',
credentials: 'same-origin',
headers: new Headers({
'Content-Type': 'application/json',
}),
}
if (payload && ['PUT', 'POST'].indexOf(method) >= 0) {
request_map.body = JSON.stringify(payload)
}
var request = new Request(url, request_map);
console.log("REQUEST : ", request)
return fetch(request)
.then(response => response.json())
.then(json => {
console.log(json.data)
return json.data
})
.catch(error => console.log("Fetch Error ", url, method, payload, error))
}
function* patchTodo() {
while (true) {
const {id, text} = yield take(actionTypes.EDIT_TODO)
try {
//yield call(patchFetchApi, config.api_root + /mytodos/ + id, {todo: text})
yield call(FetchAPIActor, config.api_root + /mytodos/ + id, 'PATCH', {todo: text})
yield put(Actions.editTodo(id, text))
}
catch (error) {
console.log("PATCH TODO ERROR", error)
}
}
}
export default function* root() {
yield [
//    startup(),
patchTodo(),
]
}
//actionTypes.js
export function editTodo(id, text) {
console.log("PATCH_TODO")
return {
type: types.EDIT_TODO,
id,
text
}
}
//reducers/index.js
const initialState = []
export default function todos(state = initialState, action) {
switch (action.type) {
case EDIT_TODO:
return state.map(todo =>
todo.id === action.id ?
Object.assign({}, todo, {text: action.text}) :
todo
)
default:
return state
}
}
// So here I can see in console log "PATCH_TODO" is getting printed 2 times.

That's cause sagas are not for intercepting actions, but for acting whenever 1 happens. So you are both listening for EDIT_TODO and dispatching EDIT_TODO in response for that. For it to work properly, you need to distinguish those 2.
For better understanding look into the source code here - https://github.com/yelouafi/redux-saga/blob/7b30115cc75ed121e3308fa8d5b274b5346120b0/src/internal/middleware.js#L50
As you can see the action is hitting reducers first and then emitted to the redux-saga

To me middleware seems more of a decorators wrapping the actual code like an onion layers so that you can run code before and after actual action being performed by reducers.
I have seen few examples too in redux-saga like async example which uses SELECT_REDDIT action in reducer as well as in sagas but runs only once on dropdown selection change.
It looks like I am missing some major point in here.
Ref: https://www.codementor.io/reactjs/tutorial/intro-to-react-redux-pros

You can dispatch action from the saga and run code in reducers at the same
action. Just do not listen for action A in the saga (with take) and then
dispatch action A, cause it will end up running ur reducers twice
On Sun, 4 Sep 2016 at 17:59, Suraj Savita<EMAIL_ADDRESS>wrote:

[image: image]
https://cloud.githubusercontent.com/assets/5236276/18232085/0de978cc-72e6-11e6-83e2-f4f542c68558.png
To me middleware seems more of a decorators wrapping the actual code like
an onion layers so that you can run code before and after actual action
being performed by reducers.
I have seen few examples too in redux-saga like async example which uses
SELECT_REDDIT action in reducer as well as in sagas but runs only once on
dropdown selection change.
It looks like I am missing some major point in here.
Ref: https://www.codementor.io/reactjs/tutorial/intro-to-react-redux-pros
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yelouafi/redux-saga/issues/527#issuecomment-244611454,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJWMks5mrxmplPUbHC6SeqzA00aT8ioOks5qmur5gaJpZM4J0hN2
.

Looks like I have to follow Database transaction like actions here: BEGIN --- Saga --- [COMMIT/ROLLBACK]
Thanks, It would do the job.


tests, network: XFail IPv6 connectivity check to an external address

What this PR does / why we need it:
IPv6 connectivity check to the statically set address fails on CI.
The same tests pass locally.
Marking the assertion with xfail in order to allow investigation.
Ref: https://github.com/kubevirt/kubevirt/issues/5113
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Seen on https://testgrid.k8s.io/kubevirt-presubmits#pull-kubevirt-e2e-k8s-cnao-1.19
Originally raised by @enp0s3 on slack.
Release note:
NONE

/cc @enp0s3 , @AlonaKaplan

/cc @enp0s3

/cc @AlonaKaplan

If I understand correctly, this has been now addressed by https://github.com/kubevirt/project-infra/pull/987 and https://github.com/kubevirt/project-infra/pull/989 .
@fgimenez , @enp0s3 , @rmohr I'm placing this on hold for now.
/hold

If I understand correctly, this has been now addressed by kubevirt/project-infra#987 and kubevirt/project-infra#989 .
@fgimenez , @enp0s3 , @rmohr I'm placing this on hold for now.
/hold

Based on #5126 run, the IPv6 issue is confirmed to be fixed by the bootstrap update.
Closing this PR.

/hold cancel
PR 5130 was closed -- this PR is still necessary short term.

@stu-gott I understood the lanes are good now with the new bootstrap image, therefore this PR is not a must.
Do you see a specific problem that occurs now?

@EdDev I was just following up with the conversation in https://github.com/kubevirt/kubevirt/pull/5130 -- I understood it to mean this PR was still needed. You have more situational awareness on this topic than I do atm, so do what you think is right.

@EdDev I was just following up with the conversation in #5130 -- I understood it to mean this PR was still needed. You have more situational awareness on this topic than I do atm, so do what you think is right.

Ahh yes, it is a bit confusing.
Well, it goes like this: The latest bootstrap bump fixed the docker config and therefore allowed the tests in question to pass.
What was raised in that conversation was that the test itself (and actually another one) is problematic as it expects external cluster configuration (docker config in IPv6 case, internet access in IPv4 case), so we need to handle this to avoid failures when running on non-kubevirtci setups.
To summarize, we do not need to xfail/skip these now, we need to handle them in a different way later.
Said that, I do not mind if this is getting in until that happens.
I suggest to leave it to @AlonaKaplan to decide here.

@EdDev I was just following up with the conversation in #5130 -- I understood it to mean this PR was still needed. You have more situational awareness on this topic than I do atm, so do what you think is right.

Ahh yes, it is a bit confusing.
Well, it goes like this: The latest bootstrap bump fixed the docker config and therefore allowed the tests in question to pass.
What was raised in that conversation was that the test itself (and actually another one) is problematic as it expects external cluster configuration (docker config in IPv6 case, internet access in IPv4 case), so we need to handle this to avoid failures when running on non-kubevirtci setups.
To summarize, we do not need to xfail/skip these now, we need to handle them in a different way later.
Said that, I do not mind if this is getting in until that happens.
I suggest to leave it to @AlonaKaplan to decide here.

Lets keep the XFail here and add one to the test expects ipv4 internet access till we will introduce flags as suggested here.


150+ switches and Internet Explorer = Bug

When you have a lot of switches on the same page in Internet Explorer, you get this warning 😟

:

When you have a lot of switches on the same page in Internet Explorer, you get this warning 😟

:

Any updates regarding this ?


GPU sync failed and CUDA_ERROR_ILLEGAL_ADDRESS error

System information

Have I written custom code: yes
OS Platform and Distribution: ubuntu 20.04
Mobile device if the issue happens on mobile device: No
TensorFlow installed from (source or binary): binary
TensorFlow version: Tensorflow-1.4.0
Python version: Python 3.5.2
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA Version 8.0.61, cuDNN 7.0.5
GPU model and memory: GeForce GTX 1080 Ti with 11G memory

Describe the current behavior
The baseline model is implemented under TensorFlow 1.4.0, so I built my model based on this model. Then I made a docker and ran it in docker. When I train my model, it occasionally shows this error:
tensorflow.python.framework.errors_impl.InternalError: GPU sync failed
and
2017-11-08 12:24:52.838106: E tensorflow/stream_executor/cuda/cuda_timer.cc:59] Internal: error destroying CUDA event in context 0x51f18f0: CUDA_ERROR_ILLEGAL_ADDRESS
When using fewer training samples, it seems that there is no problem when training the model once (epoch=1). This problem usually occurs when I train the model 5 times (epoch=5).
When more training samples are used, even if the model is only trained once, this problem will occur more frequently.
I downloaded cudnn7.0.5 according to the this, but it did not alleviate this problem.
I had tried to restart the docker but it didn't help.
The GPU memory is not exceeded. In the case of successful training, it only takes up 30% of the memory.
At the same time, I found the following error in the system log. (by sudo journalctl -f)
Dec 29 15:02:52 zpp-lab kernel: NVRM: Xid (PCI:0000:17:00): 31, pid=1342831, Ch 00000012, intr 30000000. MMU Fault: ENGINE GRAPHICS GPCCLIENT_T1_9 faulted @ 0x7f96_54002000. Fault is of type FAULT_PDE ACCESS_TYPE_ATOMIC
Then I run my code with cuda-memcheck, here is the error info.
Describe the expected behavior
Contributing

Do you want to contribute a PR? (yes/no): no
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
https://github.com/fakeppz/demo
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@Jeaninezpp ,
We see that you are using tf version 1.4, 1.x is not actively supported, please update to latest stable v2.7 and let us know if you are using same issue.Please take a look at this link for tested build configurations.Thanks


docs: update broken link in metadata-modelling

This PR corrects incorrect link in metadata modelling documentation.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@yoonhyejin Seems like CI checks are not triggered.

@sachinsaju Yep seems like it - @hsheth2 could you force-merge this PR? Thanks!


Set positionalParams as a static property on the class.

Ember 1.13.6 will be deprecating usage of positionalParams set within .extend in favor of it being set as a static property on the class itself (.reopenClass).
This makes both 1.13.0 - 1.13.5 and 1.13.6+ work without deprecation.

See the following issues/PR's for details:

https://github.com/emberjs/ember.js/issues/11686
https://github.com/emberjs/ember.js/pull/11934

The beta and canary builds failed due to needing an updated version of ember-qunit, I submitted a separate PR fixing those failures (https://github.com/dockyard/ember-cli-async-button/pull/54) I'll rebase this once those fixes are in so we can see Travis :green_book:.

Cool, looks like #54 fixed the build.  This is green now.


Can not navigate out of neovim using keymap but works using Cmdline?

I have copied the config from the README to .config/nvim/lua/plugins/nvim-tmux-navigator.lua
I am using LazyVim.
I can navigate

between split windows in neovim
between tmux panes
from tmux to neovim

Navigating from neovim to tmux does not work using <C-hjkl> but it works using NvimTmuxNavigate*.
How can I fix that?

I had to move the keymap config to ~/.config/nvim/lua/config/keymaps.lua to make it work in LazyVim

I'd say this is an issue with the README, which should be improved. I tried placing both of the suggested code suggestions in nvim/lua/plugins/tmux-navigation.lua, but neither one worked.
I solved it the same way as mrt181, but I'll provide the solution as well for anyone else who stumbles upon this thread:
nvim/lua/plugins/tmux-navigation.lua (or whatever you've named your plugin file):
return {
  "alexghergh/nvim-tmux-navigation",
  config = function()
    require("nvim-tmux-navigation").setup({})
  end,
}

nvim/lua/config/keymaps.lua:
vim.keymap.set("n", "<C-h>", require("nvim-tmux-navigation").NvimTmuxNavigateLeft)
vim.keymap.set("n", "<C-j>", require("nvim-tmux-navigation").NvimTmuxNavigateDown)
vim.keymap.set("n", "<C-k>", require("nvim-tmux-navigation").NvimTmuxNavigateUp)
vim.keymap.set("n", "<C-l>", require("nvim-tmux-navigation").NvimTmuxNavigateRight)
vim.keymap.set("n", "<C-\\>", require("nvim-tmux-navigation").NvimTmuxNavigateLastActive)

Hey,
Thanks for opening the issue and thanks for chiming in @Elyviere.
I agree that the README is not completely transparent for the beginner user (either with Neovim or with installing plugins in general). It's not as straightforward as literally "copy this piece of code into this file and it will work". It does involve placing into it's corresponding section inside the lazy.nvim setup.
Moving onto the issue with LazyVim (just making sure, you mean the distribution on top of Neovim, not the plugin manager lazy.nvim, correct?), I am not quite sure how it handles custom plugin installs. Do you mind pointing to the relevant part of the documentation where you found the solution? I could consider changing / improving the README here to reflect potentially different configs for LazyVim.
Please confirm, though, that this is not an issue with the plugin itself, but rather just a "where do I place this piece of code" kind of issue.
Thanks!

Hey alex.
In my case I am indeed using LazyVim, but since it in turn uses lazy.nvim I'm not sure there's actually a difference. As far as I know, lazy.nvim is the one defining that all plugin specs in the lua/plugins/ folder will be automatically read, not LazyVim (lazy.nvim#structuring-your-plugins).
I cannot tell you why the code provided in the readme for lazy.nvim didn't work for me. To my eyes, it looks like it should work since it's making use of the config property of lazy.nvim.
The documentation I was following was from the LazyVim distribution: lazyvim#adding&disabling-plugin-keymaps
And yes, it's not an issue with this plugin itself, only potentially an issue with the readme provided for this plugin, although I'm not myself sure what the issue is.


[core] Make ObjectsCache thread safe

Purpose
Make ObjectsCache thread safe
Tests

API and Format

Documentation

+1


v4l/libv4l: allow opening device by name

Allows using non-default device names such as /dev/v4l/by-path/pci-0000:00:1d.0-usb-0:1.1:1.0-video-index0
Resolves #6390

Tested using a python script on Fedora 22. It's a pretty straightforward change, and is really useful when you have multiple video devices and you cannot depend on the ordering of the /dev/videoX devices.

:+1:


I want to analyze a WKT that contains 3D coordinates.

I want to be able to analyze WKT where Point Z with 3D information is used.
However, I'm not familiar with Swift or its library extensions, and I'm not sure if this is the right way to do it.
Could you please kindly help me?

I'm looking for a GeoJSON library that supports 3d coords, this would be very useful

Any plans to implement Z coordinates? I'm using GEOSwift at the moment but will need elevation support soon.

Closing this PR in favor of tracking this feature request in issue #252


close -> asyncio.gather(return_exeptions=True)

Should be closed by https://github.com/aio-libs/aioslacker/commit/824f299f806f4506468fd8f3638cdebba326067a


Implement github_repo_pr_reviews table

For listing the reviews of a GitHub pull request (PullRequestReview in the GraphQL API)

Should be handled by https://github.com/mergestat/mergestat/pull/279


showStepNumbers option to show  steps number in a red circle not working

i'm using intorjs version 4.3.0 and intro.js-react  version 0.5.0. when I add option showStepNumbers, the red cicle is not showing.
in my code i use options={{
showStepNumbers:true,
showBullets: true
}}
FYI, this has worked in the older version: 3.0.1

I have same problem with showStepNumbers.please give  solution

the problem is related to intro.js so i will close the issue.

@ahoubouby @rajat-jain-au5 do you found any solutions?
I have the same problem now - https://take.ms/3XbBV


corrupted wallet

Hello
The reason I am putting this issue up here is because I have been told my wallet isn't valid.
How can you evaluate if my wallet is good or not? One of the people on the devs team sent me 1000 coins that they mined. Those were not on a fork. what I am told today is the reason my wallet doesn't have any coins in it was because I was going down the wrong block chain. All I want to know is what happened to the legitimate coins that were sent to me? Why don't I have those coins now? Why does me going down the wrong block chain overwrite the coins that were sent to me?

thx for the feedback


#13941 Added validation for minionInstaceTag while creating a new TableTask

When creating a new Task with in TableTaskConfig, validate that a minion tenant with the same tag exists.
If a minion with same minionInstaceTag doesn't exist, we will fail the table creation.

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

Project coverage is 55.34%. Comparing base (59551e4) to head (f9e611f).
Report is 1177 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (f9e611f). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (f9e611f)

integration
7
0

integration2
3
0

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
0

unittests
5
1

unittests1
2
1

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #14228      +/-   ##
============================================
- Coverage     61.75%   55.34%   -6.41%
- Complexity      207      696     +489
============================================
  Files          2436     2068     -368
  Lines        133233   109222   -24011
  Branches      20636    17293    -3343
============================================
- Hits          82274    60454   -21820
+ Misses        44911    43934     -977
+ Partials       6048     4834    -1214

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
55.34% <ø> (-6.28%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
55.34% <ø> (+27.62%)
:arrow_up:

temurin
55.34% <ø> (-6.41%)
:arrow_down:

unittests
55.34% <ø> (-6.40%)
:arrow_down:

unittests1
55.34% <ø> (+8.45%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

cc @tibrewalpratik17

cc @swaminathanmanish

cc @swaminathanmanish

Taking a look .


Rework Unit Test Format

1 huge test for a method doesn't work out well. Need to look into an alternative way of organizing the unit tests. Possibly with contexts of some sort. Look at different unit tests other projects have written in Swift.

Fixing some in the implement-ties PR #1. Will fix all other tests in a separate commit.

Need to fix MeasureTests still.

Fixed by #14

Somehow #14 was never merged. Need to figure out how to merge it now even though it has been closed and the branch deleted.

Fixed by #14


Electron Builder Script not working

Hi. I am getting this error in the workflow
Building and releasing the Electron app…
not found: electron-builder
/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:144
				throw err;
				^

Error: Command failed: npx --no-install electron-builder --linux --publish always
    at checkExecSyncError (child_process.js:621:11)
    at execSync (child_process.js:657:15)
    at run (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:21:27)
    at runAction (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:132:4)
    at Object.<anonymous> (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:150:1)
    at Module._compile (internal/modules/cjs/loader.js:959:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:995:10)
    at Module.load (internal/modules/cjs/loader.js:815:32)
    at Function.Module._load (internal/modules/cjs/loader.js:727:14)
    at Function.Module.runMain (internal/modules/cjs/loader.js:1047:10) {
  status: 127,
  signal: null,
  output: [ null, null, null ],
  pid: 2511,
  stdout: null,
  stderr: null
}

Should I add it in my dependencies?

same error, anyone advice?

same error, anyone advice?

Same for me 😞

Same for me too

i add

run: npm install electron-builder --save-dev
in build.yml it's work
but error from postInstall -_-

I fixed it by moving the electron dependency to devDeps (I still think that it is not this library to decide where the dep. should be..) And adding an extra env option:
env:
  NODE_OPTIONS: '--max_old_space_size=4096'

I fixed it by moving the electron dependency to devDeps (I still think that it is not this library to decide where the dep. should be..) And adding an extra env option:
env:
  NODE_OPTIONS: '--max_old_space_size=4096'

I did not understand very well, I have the same problem but I do not have much knowledge of Actions. 😢

i add
- run: npm install electron-builder --save-dev

in build.yml it's work
but error from postInstall -_-

Having the same error, but this did not solve the problem. Did anybody find another approach?

You could add the environment variables in the workflow yaml file. This is an example:
      - name: Build/release Electron app
        uses: samuelmeuli/action-electron-builder@v1
        with:
          # GitHub token, automatically provided to the action
          # (No need to define this secret in the repo settings)
          github_token: ${{ secrets.github_token }}

          # If the commit is tagged with a version (e.g. "v1.0.0"),
          # release the app after building
          release: ${{ startsWith(github.ref, 'refs/tags/v') }}
        env:
          NODE_OPTIONS: '--max_old_space_size=4096'

I was getting a similar error, but after checking a little closer, I was trying to build pacman packages but with the current runner, it seems that Github changed the default runner to Ubuntu 20.04 a few months ago https://github.blog/changelog/2020-10-29-github-actions-ubuntu-latest-workflows-will-use-ubuntu-20-04/
This is causing some errors related with the missing binary bsdtar used by the pacman builder. I found this workaround https://github.com/electron-userland/electron-builder/issues/4181 . Just install libarchive-tools for the runner.
Can you try this? This is a reported error https://github.com/samuelmeuli/action-electron-builder/issues/42


Improve BIP

People are responding well to the ordinal theory post, and we're focusing on the sats-as-collectables use case at first, so I think the BIP should reflect this:

Remove all the speculation about future applications that use ordinals. (I still think these applications are possible, and some will come about, but I think it detracts from the initial, easy, concrete use-case.)
Add extensive comments to the specification code
Mention that dust limit isn't a problem
Mention that this is a playful art project, and that 99.99% of users don't need to know or care about ordinals.
Add degree notation.

I think this is mostly done.


Fixing links in docs

Fixing some bad links that I noticed while reading the docs.
One thing - reference/tls-auth.md is referenced twice, but the file doesn't exist. I updated it to the how-to/tls-termination.md file, but it wasn't clear to me what the here intent was.

Thanks for this. We had a more comprehensive TLS overview, but decided to delete it since it just covered the core essentials of TLS.


Retrieval intent needs to store full intent name in tracker

Description of Problem:
The intent ask_faq/python_version is grouped into a single retriever event and placed in the tracker store as ask_faq. If you are using the TwoStageFallbackPolicy and the ActionDefaultAskAffirmation to further contextualize the conversation with the user, lack of the full intent name means you cannot determine the users intent.
Overview of the Solution:
Store the full intent name in the tracker store when using retrieval intents.

it should be available in the tracker.latest_message right? that should contain the top predicted response for the retrieval action

@amn41 maybe so, but not sure that solves all the problems this issue causes:

ask_faq/python_version is grouped into the single retriever intent and placed in the tracker store as ask_faq
This seems less than ideal, i’ve also been talking to a customer who ran into this. Their use case involves lots of FAQs, but they want to know which ones are being asked the most so they can turn those into further contextual conversations to continue to help the user. as of now it is impossible when looking in the tracker store to filter by the specific faq.

right, but that isn't the use case for retrieval actions. Retrieval actions are there to cover all the things where context doesn't matter. If you need to handle it differently, it should be a separate intent

The idea here is to be able to decide, out of these questions, which ones should actually be handled differently, which they can't do without conversations with real users, to see which paths should have further information. I guess you could do that by having them all as separate intents, but it IMO feels like we're missing an easy win here by not letting you filter by the sub-intent

ok, I get it now! Yeah it would be great to have a way to find out which faqs should be "promoted" to real intents

I think that fits super well into "Moving from step 2 to step 3" dialogue as well -- first start with handling a bunch of faqs, then see what ones your users have follow-up questions for

cc @akelad ideas for moving from step 2 to 3

@rgstephens is this still happening?

not by Greg probably. Do you want to take it on instead? Or should this be on a different board?

@erohmensing do you want to do a code walkthrough on where I left this?

@melindaloubser1 is gonna look into this, so maybe you two can chat

@erohmensing should this be based off of master and go into 1.9? I've transferred the work Greg and Daksh did to a branch off of 1.8.x but I realize it will break loading old models since it will look for an extra file (response_text_key-dict) to load from and won't find it.


Publish size of cropped point cloud

This commit adds a new topic on which the planner publishes the
number of points in the cropped point cloud. This can be very
helpful when debugging on real hardware and logging capabilities are limited.

Closing in favor of #255


Add ability to modify timezone for mysql chart

I have a requirement to modify the default server timezone for a mysqld instance. The chart doesn't currently have support to do this (that I can see).
I'm looking to follow up with a PR to simply have the ability to set the system TZ.

Fixed here https://github.com/helm/charts/pull/7123

/close


Alphabetical listing of languages on Preferences page not intuitive?

This might be very subjective (: but I find the left-to-right listing of languages on one's preferences page not to be intuitive.
I keep reading the list column-wise, not row-wise, but the alphabetical sorting is done by rows.

I agree @kerstin , It's not the most readable.
There are two routes where the list is displayed (that come to mind!) - /preferences and /projects
Would you be interested in submitting a pull request to present them by column? There may be some responsive design considerations compared to a left-to-right list.
I'm happy to review or help with the PR if necessary.

Heya, thanks for the offer/encouragement!
I totally would, but I'm not a Ruby person and have absolutely no time to figure out new tools right now /: (setup alone, based on what I saw in the README, would take me quite a while to do on my machine). Same goes for #1056.
I can, however, let some Ruby folks looking for beginner issues on GH know about these. (:


added responsivenes,various copy

Current coverage is 47.04%

Merging #183 into master will not affect coverage as of b7ade18

@@            master    #183   diff @@
======================================
  Files           20      20
  Stmts          321     321
  Branches         0       0
  Methods          0       0
======================================
  Hit            151     151
  Partial          0       0
  Missed         170     170

Review entire Coverage Diff as of b7ade18

Powered by Codecov. Updated on successful CI builds.


Backends to implement

So, if anyone has arrived so early here in the project, help me choose the next backends:
1 - Redis
2 - Vault
3 - Consul
4 - etcd

2 - Vault


KeyError in ldpred fast

Hi,
I am getting the error below when trying to run ldpred fast. Do you have any suggestions how to troubleshoot the problem?
Thanks!
ldpred fast \
   --cf cad_coord_chr17_out \
   --ldr 2210 \
   --ldf cad_ld_chr17_out \
   --out cad_ldpred_chr17_fast_out \

=============================== LDpred v. 1.0.11 ===============================

Traceback (most recent call last):
  File "/usr/local/bin/ldpred", line 11, in <module>
    load_entry_point('LDpred==1.0.11', 'console_scripts', 'ldpred')()
  File "/usr/local/lib/python3.6/site-packages/LDpred-1.0.11-py3.6.egg/ldpred/run.py", line 380, in main
    main_with_args(sys.argv[1:])
  File "/usr/local/lib/python3.6/site-packages/LDpred-1.0.11-py3.6.egg/ldpred/run.py", line 366, in main_with_args
    LDpred_fast.main(p_dict)
  File "/usr/local/lib/python3.6/site-packages/LDpred-1.0.11-py3.6.egg/ldpred/LDpred_fast.py", line 202, in main
    eff_type = get_eff_type(p_dict['cf'])
  File "/usr/local/lib/python3.6/site-packages/LDpred-1.0.11-py3.6.egg/ldpred/LDpred_fast.py", line 190, in get_eff_type
    eff_type = df['parameters']['eff_type'][...]
  File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "/usr/local/lib/python3.6/site-packages/h5py/_hl/group.py", line 264, in __getitem__
    oid = h5o.open(self.id, self._e(name), lapl=self._lapl)
  File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py/h5o.pyx", line 190, in h5py.h5o.open
KeyError: "Unable to open object (object 'parameters' doesn't exist)"

I can second this error

I am getting the same error

I am getting this error with both ldpred fast and ldpred gibbs and I can't figure out how to fix it.  One problem may be the fact that I don't have an actual file for --ldf.  The documentation says it calculates the LD for you though so this in theory should be ok...


Add correct wiki url

The wiki currently doesn't exist. When it does, the URL needs adding.

Added the temporary wiki for now. Leaving this open as it will still need to be done once it's on the actual wiki.


duplicating node with only numbers in name got extra space in name

Related to this: It would also be nice if the duplicate function was smart enough to notice if the node being duplicated had an appended number on the name, and if there's no space, to not put one in the copy. For example: "enemy1" -> duplicate -> "enemy2" etc. I can't recall right now if it goes "enemy 2" or "enemy1 2" but I know it puts a space there that I don't want. I don't like spaces in my node names at all.

Also if you like starting your numbering from 0 the next number will be 2.

or make it configurable...
prefix = empty, _ or space

I took some time tonight and fixed all of these issues. :) Hopefully reduz will merge as soon as possible.


Build failure

Hi,
I was trying to run the project on my local jenkins
It failed in the build stage
error below:
[0mERROR: /root/.cache/bazel/_bazel_root/7d8fdfd7c15a233be456f2a7a7f5337a/external/local_jdk/BUILD.bazel:155:1: @local_jdk//:jdk-lib: invalid label 'lib/ssl/certs/NetLock_Arany_=Class_Gold=_F��tan��s��tv��ny.pem' in element 4490 of attribute 'srcs' in 'filegroup' rule: invalid target name 'lib/ssl/certs/NetLock_Arany_=Class_Gold=_F��tan��s��tv��ny.pem': target names may not contain non-printable characters: '\x91' ERROR: /root/.cache/bazel/_bazel_root/7d8fdfd7c15a233be456f2a7a7f5337a/external/local_jdk/BUILD.bazel:39:1: Target '@local_jdk//:windows' contains an error and its package is in error and referenced by '@local_jdk//:java' ERROR: /root/.cache/bazel/_bazel_root/7d8fdfd7c15a233be456f2a7a7f5337a/external/bazel_tools/tools/jdk/BUILD:72:1: Target '@local_jdk//:java' contains an error and its package is in error and referenced by '@bazel_tools//tools/jdk:java' ERROR: Analysis of target '//src/apps/backend:backend' failed; build aborted: Analysis failed INFO: Elapsed time: 37.420s INFO: 0 processes. FAILED: Build did NOT complete successfully (12 packages loaded, 1838 targets configured) FAILED: Build did NOT complete successfully (12 packages loaded, 1838 targets configured) error Command failed with exit code 1. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. The command '/bin/sh -c yarn build-all' returned a non-zero code: 1
Please let me know if you need more information
Also please excuse my ignorance if I missed something
Thanks

How are you running the build?

Hi. The Jenkinsfile in the project is only intended for internal use since it builds a docker image of the application and pushes it to dockerhub. This part will only work for the owner of the dockerhub account. You could update the Jenkinsfile to point to your own dockerhub account. I have pushed a fix to the Dockerfile, which will enable the image to be created successfully.

thank you for your response....this was helpful
closing the issue


WRN-20381: Skip scroller ui-test.

Enact-DCO-1.0-Signed-off-by: Dongsu Won<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added

Need to add delay for VirtualList ui-test(QWT-2317).
Need to skip Scroller ui-test(QWT-2662).

Resolution

Add delay for VirtualList ui-test(QWT-2137).
Skip Scroller ui-test(QWT-2662).

Additional Considerations
Links
WRN-20381
WRN-20012
QWT-2137
QWT-2662
Comments
QWT-2662 will skip test until WRN-20012 will be resolved.

Jenkins passed:
/job/enact-ui-tests/2920/


Add option to toggle toolbars visibility

Related to #2353
Description
Adds a toggle toolbars visibility button on view menu. Visibility on (action checked) visibility off (action unchecekd).
The last visible toolbars are updated only when using the toggle action.

Todo

[x] Saves last visible toolbars when using the toggle button
[x] Update config
[x] Add a shortcut (Ctrl+Shift+T)
[ ] Add docstrings

@ccordoba12, @Nodd  what would the expected behavior be after a shutdown?
I think it makes sense to "toggle" as in, if 5 toolbars were on then the toggle will make those (and only those 5) visible/invisible.
My question is... what should be done on startup if all toolbars are invisible? should I keep track of the last set of toolbars that was visible before the disable? or should we just grey out the toggle visibility option if all toolbars are off after a start?

Comments @spyder-ide/core-developers ?

I think it makes sense to "toggle" as in, if 5 toolbars were on then the toggle will make those (and only those 5) visible/invisible.

My question is... what should be done on startup if all toolbars are invisible? should I keep track of the last set of toolbars that was visible before the disable? or should we just grey out the toggle visibility option if all toolbars are off after a start?

Yeah we could keep track of the current/previously visible toolbars, and toggle would bring back the previous known state, that seems intuitive.

:+1:

Ok, first review done.

I have to add some docstrings still but I fixed all of your comments and now instead of toggling, it uses a trigger (like maximize pane) so that the text in the option changes between Show toolbars and Hide toolbars

@ccordoba12 updated docstrings!


Errors occur-ed while running your project library.

cannot resolve method with(),addTo(),removeFrom(). I had included gradle also. Can you please explain what is error.

I can run the project successfully also deleting the cache.

Can you post here the build stacktrace?
Are you compiling against localDebug?

As written in the release notes of the version 1.3.0, you must build the divider before adding it to the RecyclerView.
Example: RecyclerViewDivider.with(context).build().addTo(recyclerView)

@Puneethsai Was the issue solved?

Thank you @Fondesa. Solved!!


Confusing hint when record field type is a list

SSCCE: https://ellie-app.com/3gZsGj7pdnLa1
module Main exposing (main)

import Browser
import Html exposing (Html, button, div, text)

view : () -> Browser.Document ()
view model =
    { title = "Hello"
    , body = text "hello" }

main : Program () () ()
main =
    Browser.document
        { init = \_ -> ((), Cmd.none)
        , view = view
        , update = \_ _ -> ((), Cmd.none)
        , subscriptions = \_ -> Sub.none
        }

When the compiler expects a record with a field of type List a, but it gets a record with a field of type a, it displays the following hint:
Hint: Did you forget to add [] around it?

This is confusing, because it is unclear what “it” refers to. Something like this might be better:
Hint: Did you forget to add [] around the value of the the `body` field?

Ended up removing the hint in https://github.com/elm/compiler/commit/0cd361d54c3a56940ebf94882069adf1fbc4600e. Seems like it's actively confusing in some cases, and in general, probably very rarely the real problem. It is not possible to get the kind of specificity you are describing without rewriting certain aspects of the code that colors in the differences between types, and I think it'd be too risky to try that so close to a release.
Thank you for reporting this!


Pydantic 2.5.0's project's description exceeds Azure DevOps limits

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
We are using Azure DevOps artifacts for installing our dependencies in our CICD pipilenes and they started failing this morning with the error bellow:
ERROR: Could not install requirement pydantic>=2.0.1 from https://pkgs.dev.azure.com/transport-logistics/d078984b-b6cf-4504-8f42-3ad03340e820/_packaging/8faec573-4cef-4281-976b-bed281e3d20f/pypi/download/pydantic/2.5/pydantic-2.5.0-py3-none-any.whl#sha256=7ce6e766c456ad026fe5712f7bcf036efc34bd5d107b3e669ef7ea01b3a9050c (from prids-adx-metrics==2.0.0) because of HTTP error 400 Client Error: Bad Request - The package description exceeds the maximum length of 324608 bytes.
The project description outgrew Azure DevOps limit. Although there is not something wrong with the package, I expect it will effect other organisations as well and it is something worth bringing under your attention.
Example Code
Not relevant

Python, Pydantic & OS Version
This is not relevant for this issue

I also just hit this issue. Just to clarify, this is a 'problem' with Azure DevOps Artifacts. We are using it to vendor our packages with PyPI upstream. The issue has been brought up before on their forum: https://developercommunity.visualstudio.com/t/python-package-cannot-be-installed-from-azure-arti/1434932?from=email&viewtype=all
While it is not strictly an issue of pydantic, it would be great to update the PyPI description to remove some of the older release details since it goes back to v1.0b1 and there is already a cutoff link to the full changelog on the website.

Duplicate of https://github.com/pydantic/pydantic/issues/8096

Fixed in https://github.com/pydantic/pydantic/commit/9a9ad407a239c30e7decfde809bc8a2c7303f856


FF17 Fails to load app

STR-
Using FF 17 go to CMAPI map.
Payloads are not prepopulating and the map.status.ready is not either.

Only the test harness part is broken, in that it doesn't auto-populate. Copy/paste works fine. Since this view is not for production, fixing it for a specific version of ff doesn't matter.


Blazor got issue on Firefox 104.0

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/run
Operating System: windows

it works fine on Chrome but not working on my Firefox version 104.0

any thoughts @adityamandaleeka?

It seems working already now, no any change,

I'm glad it started working, so I'll close the issue.

ok thanks


Why not for generator-trails

Is there a reason for this ? Because when it will be updated all modifications have to be done here also :/ (except if this generator will don't do anything like the generator-trails). Just curious :)
I just start a refactoring to move all generators global code into generator-util https://github.com/trailsjs/generator-util/pull/2 all api/models/policies/controllers/services are in it.
Can be useful here too :)

Very true.  Last time I wrote a generator that did a lot of this same stuff for Angular 1, I added additional files to the generation of additional files for things like models, controllers, api etc.  It would be really cool to be able to pull in the default trails generations and just add the treefrog content. That way, we wouldn't have to update Treefrog each time generator-trails was refactored.

For example: yo treefrog:model <model-name> might add react or angular specific attributes or files.

I see... Maybe instead of forking we can only use generator-trails as a dependency of this one. Like this we can call all commands of generator-trails, it does his job and then we add specific need from this one.
But to see the best solution we need to know what this generator can do more the generator-trails exactly. Like this we will know if we can reuse it or just copy/paste it ^^
generator-trails commands are classes so maybe we can just extends them and add what we need. Unfortunately not everything is in classes...

I'm closing this as it is decided that we will include generator trails to initiate normal trails app parts and wrap them in treefrog so that we don't need worry about one or the other.


feature: support Brazilian Portuguese language 'pt-BR'

Is your feature request related to a problem? Please describe.
Brazilian Portuguese translation is not available.
Describe the solution you'd like
I would like to add Brazilian Portuguese translation support.
Additional context
I really like this project and want to extend it to Brazilian users.

Thanks for your contribution. I'll push a release later tonight :)


build failed by file system is not case-sensitive but it is

I'm at the end of the toolchain setup from the sratch for MacOS. Actually there is just an issue with the last step the build.

The file system is Case Sensitiv.
The build procedure doesn't accept it.

Unfortunately I'm stuck now.

Hi Kay,
The "ctng" filesystem in the screenshot is mounted at /Volumes/ctng but the path in the error is /Users/xxx/esp/crosstool-NG which is part of your main filesystem.
Is there a symlink from one to the other? If so, a workaround might be to change cd into the "real" mount point directory and run ct-ng from there.
Angus

Hi @iGyver , did the above suggestion help you resolve the issue?

I have the same error, I think I did @projectgus suggestion but nothing happened. (Using High Sierra 10.13.6)


Secure Endpoints Using Keycloak

Currently exposed endpoints can be used by whoever can access them. To secure them against fraudulent use we should implement an approach using keycloak as an out-of-the-box solution. Using keycloak will also align with how the frontend is already secured.

Has been addressed in #35.


Picking a default windows toolchain by detecting msvc is unpredectable

On Windows, rustup will detect whether MSVC exists, and if so will install the MSVC toolchain, otherwise GNU. That you get different software based on what happens to be preinstalled in your environment is unpredictable. As an alternative, we might instead do the detection, but always install the msvc toolchain, printing a warning with further instructions if you don't have MSVC.
cc https://github.com/rust-lang/rust-www/issues/341#issuecomment-230357757 @Diggsey @vadimcn

We could also ask the user what they want to do.

@retep998 The installer already surfaces an option to change the toolchain.

I'm in favor of changing the order the GNU/MSVC toolchains are presented and changing the final sentence of the footer to say something about why you should pick MSVC.

Might want to wait until after the 1.10 release so we're not surprising people, but most stakeholders are in this thread.

I think we always default to msvc now.


Compose function transforms values from L to R, typically this is R to L

I was using recompose the other day and I finally realized, since it had been working like this way all along, that the compose function was transforming values from L to R. Typically a compose function maps values from R to L. In this case compose is working more like a pipe function. I only bring this up because recompose is considered a functional React library and therefore carries over many of the semantics from the functional realm. In terms of aligning with these concepts it makes sense, at least to me, to update the function so that is works in the same way as a how a compose function has been defined in functional programming. This would require a small change to the actual function. The new function would look like:
export default function compose(...funcs) {
  if (funcs.length === 0) {
    return arg => arg
  }

  if (funcs.length === 1) {
    return funcs[0]
  }
 //  All I changed was reduce to reduceRight
  return funcs.reduceRight((a, b) => (...args) => a(b(...args)))
}

I already looked into the tests and it would require updating the about 30+ tests or so.
I would like to contribute and make this change if it is deemed necessary. I didn't want to go and make a PR if the team/contributors felt otherwise. Let me know what you all think!

compose in recompose defined right. Its compose and not the pipe.
read this
https://medium.com/@icelabaratory/why-recompose-uses-compose-19e6cb1430ac

I am not sure your article answers the matter. In your article you mention the function flow, a form of reduce in Lodash. I looked at Lodash's FP library and saw this:
_.pipe is an alias of _.flow and _.compose is an alias of _.flowRight
In FP terms the function signature for a compose function is (f, g)(x), which is the same as f(g(x)).
If Recompose's compose is meant to behave like a pipe function that is fine. But I have to disagree that it is defined correctly.

You can use compose function from any other library with recompose and all will work the same.
You can just take an example I provided in article
compose(x => x*2, x=> x + 2) and look at the result.

You can at least write on the paper the current definition of compose
[aF, bF, cF].reduce((a, b) => (...args) => a(b(...args)))
and write down reduce
step 1: (aF, bF) => (...args) => aF(bF(...args))
step 2: ((...args) => aF(bF(...args)), cF) => (...args) => aF(bF(cF(...args))

Now u see that it's compose?

Yep, I can see it.  I am not sure the article is as accurate as your code snippet above. Unless I am reading it wrong.

In thinking about it some more, the compose function works as expected but in the context of Recompose it does not function as expected in the sense of how components get mapped and built up. It functions more as a pipe operator in how it is applied in Recompose. This speak more to what is going on under the hood and how functions/JSX are composed together in React. It's strange how the underlying logic can be correct and yet the implementation be something opposite.

it does not function as expected in the sense of how components get mapped and built up

Why? In FP languages it's not rare to see composition of higher order functions which is the same as  we do in recompose.
As an easy example:
const fn = compose(
 f => x => f(x * 2),
 f => x => f(x + 2),
)(x => x);
// fn(1) gives 4 as order of internal ops will be "1 *2 " then +2

and it's not the same as
const fn = compose(
 x => x * 2,
 x => x + 2,
);
// fn(1) === 6

So no confusion, it just depends on what you compose.

That's interesting. I haven't seen a compose function in this sense. So in this way it inverts the arguments. That's different


PartialEq of GenericByteViewArray (StringViewArray / ByteViewArray) that compares on equality rather than logical value

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

Comparing two GenericByteViewArray using PartialEq compares by structure
(the u128s) and contents of the buffers, not by logical content. As there

As @tustvold points out on https://github.com/apache/arrow-rs/pull/6673/files#r1827839121 this is inconsistent with how PartialEq is implemented for other arrays, which compare based on ArrayData which compares based on equality:
https://github.com/apache/arrow-rs/blob/ebcc4a585136cd1d9696c38c41f71c9ced181f57/arrow-data/src/data.rs#L1777-L1781
Describe the solution you'd like
Implement PartialEq for GenericByteViewArray that compares two arrays on logical value rather than physical representation
Describe alternatives you've considered
Add manual impl of PartialEq to GenericByteViewArray
Additional context

take


bug here

hi, i have tried pydbml and it has a bug here:
once you try to parse this snippet using pddbml,
enum business_type {
  Repaid [note: ""]
  Refund [note: ""]
  Rebalance [note: ""]
}

the error occus:
  File ".../lib/python3.9/site-packages/pydbml/tools.py", line 44, in strip_empty_lines
    while not lines[first_line] or lines[first_line].isspace():
IndexError: list index out of range

the reason is note: "" , the empty note was passed to strip_empty_lines function.
the related code in tools.py:
// the passed param source is ""
def strip_empty_lines(source: str) -> str:
    """Remove empty lines or lines with just spaces from beginning and end."""
    first_line = 0
    lines = source.split('\n')
    last_line = len(lines) - 1
    while not lines[first_line] or lines[first_line].isspace():
        first_line += 1
    while not lines[last_line] or lines[last_line].isspace():
        last_line -= 1
    return '\n'.join(lines[first_line: last_line + 1])

hope you can fix this issue, thx

Thanks for the report! I'll fix it

Fixed in 1.0.6