Start dragging and resizing in the DnD calendar immediately if the event is already selected

Do you want to request a feature or report a bug?
I guess a feature, I view this as a bug, but it's really anyone's guess.
What's the current behavior?
When using the DnD calendar, for an event to start being dragged or resized when the user drags it with the mouse or drags on its resize handles, the event has to be clicked first. So every time the user wants to drag or resize, they have to go: click, drag/resize, click, drag/resize. This is pretty bad in terms of UX as far as I am concerned, especially because the resize handles show up on hover, not on focus. I assumed the event will start being actioned immediately if it is selected, went to some lengths trying to preselect an event on hover in #538 only to find the event doesn't care at all if its selected or not, DnD will take two clicks no matter what.
What's the expected behavior?
I would like to suggest that a selected event starts being dragged/resized immediately and not after a focusing click.

I think @bs85 is the person to ping here as you seem to be the developer on the drag and drop addon.

Okay, this is hella confusing. On the demo page at http://intljusticemission.github.io/react-big-calendar/examples/index.html#dnd, and even locally when checking out the project and running npm run examples, the DnD demo does not need the click-then-action dance. No matter what item was last selected, you can just grab any item and drag it or resize it.
But when using RBC in my own project, even though the use is completely identical to the DnD demo code, I do have to first click and only then can I action a slot. I used to think I had to first click to select and then action unless previously selected, but turns out the behavior actually is it will immediately action on every other click.
I figured it may be that this was fixed, just not released, but I was not able to build the project. I am using Windows so I fixed a few things to work with it and I was able to run npm run build, but I am not sure how to get to a state where I can just npm install ../react-big-calendar in my other project to be able to run against my local fork. When I try to do that with successful npm run build, I am greeted with errors like Can't resolve BakcgroundWrapper.js in ../react-big-calendar/dist, which makes sense, because the files are in lib`.
I am not sure how to proceed from here, the problem is not present in this repository or on the demo site, but I can reliably reproduce it in any app which uses RBC at its latest version. I am hoping I could get some support on this issue as it makes the DnD experience significantly unpleasant.

This is my repro of the problem I describe above:
https://github.com/TomasHubelbauer/rbc-dnd-auto-select
Just npm install and npm start.

Not sure, but maybe it's same issue as I had in #1105

This issues does indeed sound very similar to #1105. I will wait until yours has been resolved and see if that fix makes my repro work as well, if yes, I will close my issue.

I'm running into the same issue, any updates on this?

I haven't tried this because in my project the drag requirement was dropped. I think this project has been abandoned so I think you're best off debugging this for yourself and PR-ing just in case, if this is a blocker for you and you're on a deadline.

Hey @TomasHubelbauer  thank you for the quick reply. I have no deadline pressure, it's just a side project, but I would really like to have this feature before I present it. I will tinker with it for a bit to see if I can fix it. I'm curious to know what you ended up implementing to change the event time range? Did you use a modal?

I am not sure actually, unfortunately. I did a handful of experiments with RBC:
https://github.com/TomasHubelbauer?utf8=✓&tab=repositories&q=rbc&type=&language=
I think either https://github.com/TomasHubelbauer/rbc-resizable-event-wrapper or https://github.com/TomasHubelbauer/rbc-dnd-auto-select may be of interest to you, but I am not sure if they contain a solution, it's been a while since I focused on RBC and I mostly worked around all its weirdness by wrapping it in my own React component and forgot the specifics :-/.

@TomasHubelbauer thank you for sharing! I'm looking at it rw. Love you sense of humor, lol

Hello, I want to know if the problem can be solved in any way

Not sure, but in my case this workaround with initial data (in initial view) helped me last time I was working with RBC.
https://github.com/jquense/react-big-calendar/issues/1105#issuecomment-459563886

Not sure, but in my case this workaround with initial data (in initial view) helped me last time I was working with RBC.
#1105 (comment)

that's the workaround that i implemented, but is weird that the in the DnD example page works very fine. Maybe the creators use the same   workaround in that example. I dont know. Thanks for the reply.


Update from 2.6.2 to 3.0.0-alpha1 with kubernetes datastore broke outgoing nat

I try update calico 2.6.2 to 3.0.0-alpha1 and outgoing nat stop work.
kubernetes version: 1.7
calico datastore: kubernetes
calico in policy-only mode
It was fixed by updating ippool with calicoctl.
Maybe, there is a result of renaming nat-outgoing parameter to natOutgoing.

This is expected; there is no upgrade support from v2 to v3.0.0-alpha (or beta).  Our GA release will include upgrade support.


adding spinner page

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@250d02f). Click here to learn what that means.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #125   +/-   ##
=========================================
  Coverage          ?   79.27%
=========================================
  Files             ?      123
  Lines             ?    11391
  Branches          ?     1231
=========================================
  Hits              ?     9030
  Misses            ?     2361
  Partials          ?        0

Flag
Coverage Δ

unittests
79.27% <100.00%> (?)

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

Impacted Files
Coverage Δ

packages/lib/components/transfer/transfer.tsx
84.90% <100.00%> (ø)

Continue to review full report at Codecov.

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


FEAT: Initial RDS Schema and associated Tooling

This change contains the initial attempt at RDS schema mapping based on CUBIC spec 1098-00057.K.
All schema files are located in /schema folder, open to other suggestions on organization for these files, but this is my initial attempt.
Also included in this change are the following:

alembic configuration files
utility functions for RDS interaction
utility functions for logging
additional libraries required for RDS utils (boto3,  psycopg2)
updates to Dockerfile to optimize re-builds

Asana Task: https://app.asana.com/0/1205827492903547/1205824362474708

LGTM 🍰


remove "default" CaseFormat for stringConstants!

this solves the following problem:
when having similar names for attributes and classes, such as

ssn:FeatureOfInterest (denoting a class)
ssn:featureOfInterest (denoting a property)

having UPPER_UNDERSCORE as the default stringConstantCase will cause duplicate fields ...and there is no possibility to omit reformatting to any of the CaseFormat options.

Are you able to modify the maven property to do what you need without changing the default?
The reason that the default case format exists is to make it work for the majority of cases that we need it to work for, while making sure that others are able to reconfigure it themselves in their pom.xml files.

Hello Peter, Thomas,
the problem I see, that there is no valid option in com.google.common.base.CaseFormat which says "leave as it is". In fact, for now, there are the following options:

omitting the tag : this will do the default of UPPER_UNDERSCORE
provide the tag but with no value: this will do the default of UPPER_UNDERSCORE
use another valid CaseFormat option (LOWER_HYPHEN, UPPER_HYPHEN etc.): this will do the requested case adjustments but result in a duplicate field (at least for the SSN ontology as mentioned in my request and also for the FOAF ontology where for example foaf:familyName and foaf:family_name cause a conflict)
use an invalid option such as DO_NOTHING: this will result in an error since the configuration cannot be parsed.

For me, the simplest solution was to remove the default, because this results in a "no case format adjustment" when omitting the  tag.
However, a more sophisticated solution could allow

specify the default generation options for URI's and/or strings including case format, prefix, suffix
allow overriding the default settings for each configured vocabulary (partially already done with vocabulary-specific caseFormat)

Best
Dietmar

Thanks, that explains it quite well I think. In that case, I would definitely support offering either a dummy value or default to nothing and the current users update our pom.xml files with our desired cases.

there was some time left so I modified the functionality quite a bit in order to provide more (generation) flexibility:
(1) allow to specify prefix & suffix for the URI constants as well.
either provide the tags in the plugin as
<createUriConstants>true</createUriConstants>    (defaults to true)
<uriConstantPrefix></uriConstantPrefix> (defaults to "")
<uriConstantSuffix></uriConstantSuffix> (defaults to "")
<createStringConstants>true</createStringConstants>    (defaults to true)
<stringConstantPrefix></stringConstantPrefix> (defaults to "")
<stringConstantSuffix></stringConstantSuffix> (defaults to "_STRING")

or in an special "setting" structure (which may also be used for each vocabulary)
<uriGeneration>   (or <stringGeneration>)
...<caseFormat></caseFormat>
...<constantPrefix></constantPrefix>
...<constantSuffix></constantSuffix>
</uriGeneration>

this for example gives you the possibility to configure a "_URI" suffix for URI's (if desired) . in addition, vocabulary specific settings are also possible.
(2) default case format will be used for both URI & String generation...
I personally found it not logical to have URI constants adjusted differentlty than String constants by default. So I'd propose to have only one "caseFormat" element which is used for both generation options by default. now, no caseFormat adjustment for URI's takes place by default (as it was before). the caseFormat setting is then used for string generation as well, unless it is further specified in a <stringGeneration> element.
(3) no changes in pom.xml of your users required, command line parameters are still the same
the changes are just additions, nothing has been removed.
finally, there were quite some adjustments and changes in the maven plugin and in the VocabBuilder class as well. Maybe you find the changes useful.

I'd suggest to switch to the new configuration format @dglachs suggests and deprecate the old way (to be removed in one the next versions). Keeping both is too much overhead and will confuse users.
For the defaults: I'd add a special handling for AS_IS or NONE which then should be also used as default/not explicitly set...
And finally: After merging these changes we should have new release!

I totally agree with @ja-fra . @dglachs could you add the 'NONE' case? Afterwards we can merge and release.

Added the required functionality ...
currently the following tags in the  section are possible:
<caseFormat>NONE</caseFormat>

this is the default and specifies NO caseFormat-Adjustments for both URI & String-Constant generation. (the distinct default generation options are still specified with the "old" parameters, see my previous comment). For having more fine grained generation options - the GenerationSetting class is used. One may specify
<uriGeneration>   (or <stringGeneration>)
    <caseFormat></caseFormat>
    <constantPrefix></constantPrefix>
    <constantSuffix></constantSuffix>
</uriGeneration>

which then override the default-settings. to have it more sophisticated it is also possible to have the following structure along with a vocabulary (under )
<vocabulary>
    <className>FOAF</className>
    <prefix>http://xmlns.com/foaf/0.1/</prefix>
    <file>${project.build.directory}/../ontologies/foaf.ttl</file>
    <uriGeneration>   (or <stringGeneration>)
        <caseFormat></caseFormat>
        <constantPrefix></constantPrefix>
        <constantSuffix></constantSuffix>
    </uriGeneration>
</vocabulary>

for specifying specific prefix/suffix for a particular vocabulary (overriding the general settings from above).
As noted - there are still the "duplicated" or "old" parameters (, ) - I've marked them as deprecated. Anyways, as @ja-fra suggested, these "old" parameters should be removed, for this, it is recommended to remove only the Parameter-Annotation and just leave the variables (including) their default as constants.

Overall, the code also needs to be reformatted to use spaces instead of tabs.

reverted the NONE option commit since it created more problems than it solved ...
in fact the NONE option was present before because simply omitting the caseFormat-tag (at any level) was the NONE option.
anyways the proposed configuration structure is still valid

The code seems to be okay now. Should we merge the request and release a new version?

Yes, the functionality looks good, just some style issues remaining that we can automatically reformat after merging and delete some of the commented code that is still present.
Can we switch on TravisCI to do automated tests btw? I thought it was on because we have the .travis.yml file, but it doesn't seem to be enabled at the other end right now:
https://travis-ci.org/tkurz/sesame-vocab-builder


Add policy check to Vm retirement.

Vm retirement is handled as a request per https://github.com/ManageIQ/manageiq/pull/16933.
Depends on https://github.com/ManageIQ/manageiq-content/pull/552.
https://bugzilla.redhat.com/show_bug.cgi?id=1702018
@miq-bot add_label bug, retirement, hammer/yes, ivanchuk/yes, changelog/yes
@miq-bot assign @tinaafitz

@lfu
Looks good waiting for the conflicts to be resolved.

@d-m-u Please review.

@gmcculloug Please review.

Hi @gmcculloug Please review. This is needed for the 5.10 build.

@lfu Please review test failures.

Method make_retire_request overrides method with same name in RetirementMixin and it causes error in API repo
@lfu  Do you have idea how to fix it ?
thanks!!

@tinaafitz This PR requires API code change.

Yeah, I'm just here to say that Libor's right and the API spec failure's due to this PR

thanks @d-m-u
fyi: we are working on fix with @lfu and work is in progress

@lfu There are multiple conflicts in the spec file for backporting to hammer branch. Can you please create a separate PR for hammer?

@lfu This PR conflicts backporting to invahcuk branch as well. If the conflict is same between hammer/ivanchuk, just 1 PR is needed (against ivanchuk) and that can be marked as hammer/yes.
If they're different, please create another PR for ivanchuk.

Backported to hammer via https://github.com/ManageIQ/manageiq/pull/19643
Backported to ivanchuk via https://github.com/ManageIQ/manageiq/pull/19647


Add a DebugPrint() macro

Description
Fixes #43

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

Project coverage is 88.36%. Comparing base (c8e128f) to head (b59adde).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #234   +/-   ##
=======================================
  Coverage   88.36%   88.36%
=======================================
  Files          20       20
  Lines         576      576
=======================================
  Hits          509      509
  Misses         67       67

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


[Parse] Eliminate backtracking in collection expression parsing

Parsing collection literal expression used to take exponential time depending on the nesting level of the first element.
Stop using parseList() because using it complicates libSyntax parsing.
SR-9220 / rdar://problem/45221238

@swift-ci Please smoke test

Cool, I wonder if your PR fixes this bug too: https://github.com/apple/swift/pull/15619/files#diff-1a9565d4ad49da3f2dcb708fa95d21a3

@slavapestov Yes it fixes that. Thank you for the link. I added a test case from #15619 .

@swift-ci Please smoke test


JU-431  java21 migration -> main account-validator-api

Dev and Platform code changes for service: account-validator-api

CI: Security warnings found!

CI: No security warnings found


setting z-order for vertical span

Question
Any way to control the z-order of the vertical_span? so that we can keep the chart in focus but only affect the background color?
alpha set to 0.4 in rgba

alpha set to 1 in rgba

Code example
No response

Unfortunately Lightweight Charts offers no implementation for z index order, but the feature has been requested so it will potentially be possible in the future.
Louis


🦺[Tech Debt] App must target Android 14 (API level 34) or higher

Goal
Describe the work to be done

Upgrade to API 34
Definition of Done

[x] Upgrade
[x] Test

Done


Remove oss-kits as SCXcore submodule (already added to superprojects

(Difficult to manage tips properly; will be tracked by superproject)
This PR will relocate installer/oss-kits to /opsmgr-kits. Note that this new super project is already committed to both Build-SCXcore and Build-OMS-Agent-for-Linux projects. Once this is committed, the change will actually take effect.
I'm uncertain how to get the new subproject without a new clone. For simplicity, I'll just suggest that people clone the repo again (trivial with pbuild, just throw on --clone), which absolutely works.
@Microsoft/ostc-devs @Microsoft/omsagent-devs

Signed off


Added support for function deployment from gzip/bzip2/xz compressed tar files

Issue Ref: #1167
Description:
Added support for deploying and updating functions from gzip/bzip2/xz compressed tar files.
Supports the following standard file extensions - .tar.gz, .taz, .tgz, .tar.bz2, .tb2, .tbz, .tbz2, .tz2, .tar.xz
TODOs:

[x] Ready to review
[x] Automated Tests
[x] Docs

Thanks @andresmgot :) In order to run the url+compressedtar tests from examples/Makefile, I also need the compressed tar's to be present in the examples folder already. Do you want me to create a separate PR for just those 3 files?


Adds SHORT_USER/HOSTNAME support to powerline themes

Brings the short user and hostname variables into the powerline themes.
I use these so I can explicitly set the values in my prompt just as in my terminal title.
-DF

Thanks!


Cargo.toml for miniz-sys should document Unlicense

The Cargo.toml file for miniz-sys lists the license as MIT/Apache-2.0; however, miniz.c itself uses the Unlicense.
Since the Unlicense puts the code in the public domain, if you intended to relicense it, you can do so; however, if you didn't intend to relicense it, you should document the original license.

I don't really know what to do when it comes to license, but I'd love to ensure that all licenses are abided and what not. What specifically needs to be done here?

The easiest fix, if you're fine with putting the rest of the packaging for miniz-sys under the same license as miniz.c itself, would be to change the license field in Cargo.toml to just say "Unlicense" (and drop the LICENSE-* files from the repository).  Alternatively, you could change it to "MIT/Apache-2.0/Unlicense", if you prefer.

But all the Rust code is under the "standard Rust license", not the unlicense?

@alexcrichton If you want to keep the Rust code under a different license than the C code, then that'd be (MIT OR Apache-2.0) AND Unlicense, in SPDX syntax.
Note: I'm not suggesting changing the license expression of flate2; I'm suggesting changing the license expression of miniz-sys.  (That's why one reasonable possibility seemed like putting miniz-sys under the same license as miniz.c.)

Ok cool, sounds good to me!

Thanks!


Added support for "missing" and "exists" queries

@michaelbudnik thanks so much for this but I've already implemented this on my local 2.0 branch. Will push today.

Great, thanks


zed query: support HTTP, HTTPS URLs as input files

This makes the following command work.
zq -z https://github.com/brimdata/zed/raw/main/testdata/babble.zson

Closes #734.

I thought you were going to unify proc.DataAdaptor.Get and Open...?

I thought you were going to unify proc.DataAdaptor.Get and Open...?

I am, but in a separate PR.


Set checkbox and radio right padding to 1rem

Will resolve #207

 Issue #207: When radios or checkboxes are greater than the standard length the right padding should be 1rem from the label.


Fix responsive design

The responsive design looks terrible.

Yeah, this cannot stay like this

Fixed with https://github.com/LucaSchwan/schwan-outdoor_shop/commit/1c09687ef0f5fff9a92447291db504c13f6158be


Scan wildcard directory recursively

Recursive wildcard scanning
Resolves #12
The primary intent of this PR is to allow the script to recursively scan the wildcards directory, which would allow users to organize their wildcards with file structures of arbitrary depth.
To ensure the listed wildcards continue to represent full paths of now arbitrary depth, we also change the return value from path.name to path.relative_to(WILDCARD_DIR)
This should not effect existing wildcards, as we previously didn't support any depth.
New entries will only having leading path elements if they exist in a subdirectory of any depth under WILDCARD_DIR

Controversial additions
I wanted to mention these items before this is merged in case we want to omit or break them out.
Wildcard globbing as explained here: https://github.com/evanjs/sd-dynamic-prompting/tree/extended-glob#fuzzy-globrecursive-wildcard-filedirectory-matching
This is not crucial to the recursive scanning which is core to this pull request, and we can probably omit without much issue.
Source: https://github.com/evanjs/sd-dynamic-prompting/blob/7e9f7d1edce0cd38b8effee0a4b44b9220188466/dynamic_prompting.py#L80-L87
Ignoring lines that start with a leading #
I found this a useful way to persist wildcards while being able to easily disable anything that doesn't work well until it can later be addressed. At the same time, I can see how it might cause issues if there exists wildcards that start with #, such as hashtags, etc.
Source: https://github.com/evanjs/sd-dynamic-prompting/blob/7e9f7d1edce0cd38b8effee0a4b44b9220188466/dynamic_prompting.py#L94-L95.
Replace underscore option for danbooru tag compatiblity
Tags from danbooru use underscores rather than spaces as a word separator.
This option provides an easy way to source native Danbooru tags without first manually replacing every underscore with a space.

Awesome - I've merged the rglob bits but have temporarily removed the wildcard globbing and the danbooru tag compatibility.
Wildcard globbing is a great idea, but I'm sure about the implementation. It didn't work as I expected globbing would work. I had two files:

wildcard_dir/keywords1.txt
wildcard_dir/keywords2.txt

I tried __keywords*__
but this line split my glob into ["keywords", ""]. An empty string matches everything causing all of my wildcards to be included. It would be better to use python's native globbing functionality rather than manually resolving it.
Regarding replaced underscores for danbooru - what is the rationale? This line would break recursive resolution of wildcards, i.e., imagine that I have keywords.txt, it contains two lines

__keywords1__
- __keywords2__

after that line runs, the underscores become spaces and then keywords1 and keywords2 become literal strings.
Maybe there's another way to achieve want you want?

Awesome - I've merged the rglob bits but have temporarily removed the wildcard globbing and the danbooru tag compatibility.
Wildcard globbing is a great idea, but I'm not sure about the implementation. It didn't work as I expected globbing would work. I had two files:

wildcard_dir/keywords1.txt
wildcard_dir/keywords2.txt

I tried __keywords*__
but this line split my glob into ["keywords", ""]. An empty string matches everything causing all of my wildcards to be included. It would be better to use python's native globbing functionality rather than manually resolving it.
Regarding replaced underscores for danbooru - what is the rationale? This line would break recursive resolution of wildcards, i.e., imagine that I have keywords.txt, it contains two lines __keywords1__  __keywords2__
after that line runs, the underscores become spaces and then keywords1 and keywords2 become literal strings.
Maybe there's another way to achieve want you want?

Thanks!
Yeah, I don't really have the issue, as boorucomplete seems to have it's own option which handles the problem.
cc @Vetchems who originally implemented the feature.
As for "globbing", that should be fine; we can consider it again if we find a cleaner implementation.
As mentioned in #11, we probably want to be careful and deliberate about any modifications to syntax as is.

Hi,
The reason I added the replace underscore feature was to be able to have a random booru tag selected from a text file (red_dress) but then expressed as red dress for use with models not trained on booru tags.
The results from individual models varies greatly depending on the underscores.
Boorucomplete will replace the underscores if you use it while writing the prompt but it will not replace them if the tag is inside a text file.
Also to my knowledge the replacing of underscores was done at the last step of the script before processing, I would assume that would allow the script to select its random line (keywords1) recursively do its thing until the loop has finishedand then finally replace remaining underscores in the sting?
I could be wrong in my understanding there, I didnt realise it would potentially break so much.

@evanjs I have written an implementation of the globbing functionality. Could you give it a spin to make sure that it doesn't break anything on your end? https://github.com/adieyal/sd-dynamic-prompting/tree/feature/path-globbing

Okay, I think I have a better understanding of your initial feedback now, and can better respond to what I was doing and why in the context of my workflow.
My initial implementation was indeed not strict globbing, as that quickly became somewhat cumbersome for me.
I was doing a more "fuzzy glob", where * is more akin to a regexp like .*.
This way, I could do something like clothing*modifier and settings*modifier, and it would pick up anything with modifier in the path under clothing or settings.
[Strict] Path globbing seems to work with my current workflow, but does requires a tiny bit more thought.
i.e. chars/favorites/**/*.txt can be reduced to something like  __*/*fav*/*__, but with my ~lazy~ "fuzzy globbing", I could do something like __fav__, etc.

gotcha - the problem with the fuzzy approach was that it resulted in undesired matches, e.g. https://github.com/adieyal/sd-dynamic-prompting/issues/18. I think strict globbing is quite useful because it also providers character classes. Most importantly, we pathlib handles it for us .


Delete + Add document on import

Right now you forced to delete imported library to avoid duplicates on re-import. That's definetly not the way to go.
Records should be updated on unique index (LibraryId + File + Ext + Archive) instead.

Having a 'Date Added' would assist in scanning for changed/new documents/books? The Created Date in FileInfo would be the source.
The same scan could be used to delete books/documents that are no longer present.

This is one option i'm considering, but not created date, rather 'import
date'.
Going that way we can update that field on import and delete all
non-updated documents in library namespace after.
Trying to remember why i didn't do that early :)
2017-02-03 18:02 GMT+03:00 Gerrit Visser<EMAIL_ADDRESS>
Having a 'Date Added' would assist in scanning for changed/new
documents/books? The Created Date in FileInfo would be the source.
The same scan could be used to delete books/documents that are no longer
present.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/DeniSix/DotOPDS/issues/10#issuecomment-277269223, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAm6ZiirqW4jsKhRX5t3adFTyaggmviks5rY0GWgaJpZM4L2Gno
.

--
Yours sincerely,
Denis Shemanaev
Skype: denis.shemanaev

That would work, any file with a Creation Date after the Date Added would be ripe for Add or Update depending on whether it was already in the Library/Index. I expect to be updating the info in my PDF collection over time, lots are missing Title, Author and Keywords :-(

I tried this out, not sure it is working as expected. Re-importing a library does not result in Deleted books >0.
When Insert a book, it has GUID.Empty. So the original will never be found using that. Then on Add, MapBook sets the GUID. I suspect that we end up with 2 Documents in the Index.

There is the catch: records delete not based on GUID, but on
libraryid+file+ext+archive. Right now cleanup deletes only records that not
being updated more than 6 hours ago. I'll fix this a bit later (remember
time of import start and delete records updated later than that should be
ok I assume).
2017-02-05 17:06 GMT+03:00 Gerrit Visser<EMAIL_ADDRESS>
I tried this out, not sure it is working as expected. Re-importing a
library does not result in Deleted books >0.
When Insert a book, it has GUID.Empty. So the original will never be found
using that. Then on Add, MapBook sets the GUID. I suspect that we end up
with 2 Documents in the Index.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/DeniSix/DotOPDS/issues/10#issuecomment-277521907, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAm6UvbYu8zeIqO7DhR1xL4FUwZnUE8ks5rZddSgaJpZM4L2Gno
.

--
Yours sincerely,
Denis Shemanaev
Skype: denis.shemanaev

Also you can open and explore Lucene.Net index with Luke.
2017-02-05 17:13 GMT+03:00 Denis Shemanaev<EMAIL_ADDRESS>
There is the catch: records delete not based on GUID, but on
libraryid+file+ext+archive. Right now cleanup deletes only records that not
being updated more than 6 hours ago. I'll fix this a bit later (remember
time of import start and delete records updated later than that should be
ok I assume).
2017-02-05 17:06 GMT+03:00 Gerrit Visser<EMAIL_ADDRESS>
I tried this out, not sure it is working as expected. Re-importing a
library does not result in Deleted books >0.
When Insert a book, it has GUID.Empty. So the original will never be
found using that. Then on Add, MapBook sets the GUID. I suspect that we end
up with 2 Documents in the Index.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/DeniSix/DotOPDS/issues/10#issuecomment-277521907,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAm6UvbYu8zeIqO7DhR1xL4FUwZnUE8ks5rZddSgaJpZM4L2Gno
.

--
Yours sincerely,
Denis Shemanaev
Skype: denis.shemanaev

--
Yours sincerely,
Denis Shemanaev
Skype: denis.shemanaev

Doh, I should have my cappuccino before reading and testing :-)
I also found https://luke.codeplex.com/. Both very useful and educational. Thank you


Convert wasmdome's engine to lattice mode

Check the engine-provider 's README for more information on the new NATS subjects being used and how to manually initiate a game run without using the wasmdome CLI.

The changes to the wasmdome binary and the offline demo environment will be covered by additional issues, starting with #8


Enhancement: Add lyrics view

I looked at the Apple's API and they don't provide lyrics, but we have enough data from their API to fetch lyrics from another source. What do you think about this?

Hello @swapnilraj, if you (or someone else) is able to implement this, I am all for adding this (assuming I'm comfortable with the external service selected).


pull request for e2e test at 11/29/2016 05:26:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


updated W2012R2 DC to new 64GB base size

updated W2012R2 DC to new 64GB base size

This may be a bug, closing until confirmed


Update array.hpp

Error: This PR changes the Ginkgo ABI:
For details check the full ABI diff under Artifacts here

Error: This PR changes the Ginkgo ABI:
bla
bla

For details check the full ABI diff under Artifacts here

Error: This PR changes the Ginkgo ABI:
file build-old/lib/libginkgod.so does not exist

For details check the full ABI diff under Artifacts here

Error: This PR changes the Ginkgo ABI:
Functions changes summary: 0 Removed, 1 Changed (28 filtered out), 0 Added functions
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable

For details check the full ABI diff under Artifacts here

Error: This PR changes the Ginkgo ABI:
Functions changes summary: 0 Removed, 1 Changed (28 filtered out), 0 Added functions
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable

For details check the full ABI diff under Artifacts here

Note: This PR changes the Ginkgo ABI blablabla

Note: This PR changes the Ginkgo ABI:
Functions changes summary: 0 Removed, 1 Changed (28 filtered out), 0 Added functions
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable

For details check the full ABI diff under Artifacts here

Error: The following files need to be formatted:
core/base/allocator.hpp

You can find a formatting patch under Artifacts here or run format! if you have write access to Ginkgo

Error: The following files need to be formatted:
core/base/allocator.hpp

You can find a formatting patch under Artifacts here or run format! if you have write access to Ginkgo

Error: The following files need to be formatted:
core/base/allocator.hpp

You can find a formatting patch under Artifacts here or run format! if you have write access to Ginkgo

Error: The following files need to be formatted:
core/base/allocator.hpp

You can find a formatting patch under Artifacts here or run format! if you have write access to Ginkgo


new: added 2 new root commands: configs and s3.

Implemented stats and cleanup commands for s3 too. Updated readme consequently.

TODO:

we miss s3 tests
of course s3 cleanup require proper auth (load from config?)
implement an S3 test endpoint to be used by tests


circt: 1.84.0 -> 1.85.0

circt: 1.83 -> 1.84
circt: add sequencer to maintainers

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks~

Result of nixpkgs-review pr 338312 run on x86_64-linux 1

3 packages built:

circt
circt.dev
circt.lib

result of nixpkgs-review pr 338312 (build on aarch64)
$ git -c fetch.prune=false fetch --no-tags --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/338312/head:refs/nixpkgs-review/1
remote: Enumerating objects: 2132, done.
remote: Counting objects: 100% (1028/1028), done.
remote: Compressing objects: 100% (395/395), done.
Receiving objects: 100% (691/691), 211.05 KiB | 783.00 KiB/s, done.
remote: Total 691 (delta 475), reused 442 (delta 253), pack-reused 0 (from 0)
Resolving deltas: 100% (475/475), completed with 143 local objects.
From https://github.com/NixOS/nixpkgs
 * [new branch]                master                -> refs/nixpkgs-review/0
 * [new ref]                   refs/pull/338312/head -> refs/nixpkgs-review/1
$ git worktree add /home/sequencer/.cache/nixpkgs-review/pr-338312/nixpkgs b956c2f42a5d0c9fbbfd8b2a9cf3bb5f0a6fcf41
Preparing worktree (detached HEAD b956c2f42a5d)
Updating files: 100% (42976/42976), done.
HEAD is now at b956c2f42a5d ocamlPackages.lockfree: 0.4.0 → 0.5.0
$ git merge --no-commit --no-ff 21baffd4052beea8fc8b78415b24cac5528ac94e
Auto-merging maintainers/maintainer-list.nix
Automatic merge went well; stopped before committing as requested
┏━ Dependency Graph:
┃ ✔ review-shell
┣━━━ Builds
┗━ ∑ ⏵ 0 │ ✔ 1 │ ⏸ 0 │ Finished at 20:16:49 after 0s

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/338312

3 packages built:
circt circt.dev circt.lib

$ /etc/profiles/per-user/sequencer/bin/nom-shell --argstr system aarch64-linux --argstr nixpkgs-path /home/sequencer/.cache/nixpkgs-review/pr-338312/nixpkgs --argstr nixpkgs-config-path /tmp/tmp39xsx_dn.nix --argstr attrs-path /home/sequencer/.cache/nixpkgs-review/pr-338312/attrs.nix --nix-path 'nixpkgs=/home/sequencer/.cache/nixpkgs-review/pr-338312/nixpkgs nixpkgs-overlays=/tmp/tmp2gias7bm' /nix/store/31zva50bkbk1cihpzd66jy7aahm6521z-nixpkgs-review-2.10.5/lib/python3.12/site-packages/nixpkgs_review/nix/review-shell.nix


golang: added Session.ListDevices method

Implemented Session.ListDevices() method

Can one of the admins verify this patch?

My use case was I was trying to balance session creation manually across multiple GPUs so I needed to fetch a number of available GPUs together with available memory. So I decided to create an empty session at the start and then fetch all info via ListDevices().

Jenkins, test this please

Jenkins, test this please

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please

windows rerun at http://ci.tensorflow.org/job/tensorflow-pr-win-cmake-py/5945/


WIP: Built out framework for functional tests

I've started working on adding some functional tests to this repository. I'm going to backfill several tests (and will need to figure out how to get the Android SDK in using travis, for example). Once this is in a good state, I'll work with @jeremymailen and try to apply these changes to #55.
For now, this is just a WIP showing you what I'm working on.

Thanks for taking the initiative here @mikeholler! Sorry about the delay -- I've been on vacation. I'll give it a review.


added field for the state applied in

I added method to BIACase to return the U.S city and state of the original application.

Uses regex to find city, state abbreviation 5-digit zip code pattern. Not sure if it's the most effective way to find it, I found the lawyer's mailing address is usually the same as the applicant's location.
Lawyer mailing address format is the same for application decisions and appeals. The method should safeguard against Falls Church, Virginia
Edge case: Lawyer being located outside the state of application

This finds the first state abbreviation that it finds. Is this ALWAYS the right state? What about city?

This pull request has been updated.

I scraped the locations of all the immigration courts (there's 69 courts, file has 61, some cities have more than one court) and converted to JSON
The file reads in the JSON file as a dictionary
Checks the Spacy GPE label to match a city in the dictionary
Returns the state and city in separate fields.

Can someone remind me what gpe is?

If I'm not mistaken, it means Geo Political Entity @KWeatherwalks


Upgrade Linux Kernel for flatcar-3510 from 5.15.122 to 5.15.124

Subject says it all.

We need to first release the next Stable with Kernel 5.15.122, https://github.com/flatcar/scripts/pull/1009, not this one.
Added on-hold.

Now that Stable 3510.2.6 was released with 5.15.122, we can merge this PR.
Though we probably need to soon update Stable Kernel to 5.15.125+ to address Kernel security issues.


Error installing pg gem on ubuntu 16.04

I ran into the following error while installing pg gem:
find_header: checking for libpq-fe.h... -------------------- no

The libpq-dev package needed to be installed on Ubuntu.


Update third_party/protobuf to v3.15.2

@donnadionne

Let's wait for @jtattermusch's approval and passing tests before merging. LGTM from my side.

for some reason the src/objective-c/BoringSSL-GRPC.podspec was being updated as well (dirty boringssl version locally?) so I pushed a commit to revert that change.

for some reason the src/objective-c/BoringSSL-GRPC.podspec was being updated as well (dirty boringssl version locally?) so I pushed a commit to revert that change.

Apologies, I was updating boringssl and changes from that branch seeped in here. Thanks for fixing this, Jan!

iOS Binary Size Diff (internal CI) — https://source.cloud.google.com/results/invocations/c73d3cf4-3a8f-4f0a-a43b-277372c8271f/log -  No significant differences in binary sizes, but some infra errors.
Basic Tests Python MacOS — https://source.cloud.google.com/results/invocations/c1ae0202-dac2-43a6-ad71-c5d52c36a576/log - test timeout, unrelated to this PR.


Tool to auto-sort attribute lists

Describe the solution you'd like

A tool that can take a multiline string of example attributes (copy pasted and piped through) and then sort them.
Describe alternatives you've considered

Doing it manually
Additional context

Assign me to this issue because I think I know how to make such a tool.

Thanks for the idea. However, I don't think this is something we need in this project.


Tab improvements

Hello! Thanks so much for the sweet Hyperterm theme. Love the concept :)
One thing I figured could use a bit of polish: the tabs! Not sure if this is something you're interested in, but spent a little time playing with the tab styles for fun :) If you like the improvement, please feel free to merge / ask for updates!
Before:

After:

Before:

After:

Thanks! This looks like a good addition. I'll add a few comments inline

Version 1.1.0 released with this. Thanks!

Awesome, thanks again for the sweet theme :)


Fix up unit tests so the automated builds pass

The automated build has been failing for a long time. The breaks represent bugs in the program that probably came from changes to the websites since the automation was added. The broken build represents bugs that need fixing.
Once it's green again, more tests can be added to ensure the program continues to work.
Having the automated build in a working state also gives a first look at the quality of PRs, although right now the coverage is pretty much useless because all PRs are failing due to the currently-broken nature of the automated check.

It turns out there are more failures in Travis CI than there are when running tests locally. I'll have to figure that out later. For now, fixing the test pass for local testing is still a useful exercise and a step in a right direction. The issues I've opened are for the test failures I've seen locally.


Class-level @see tag not rendered

I am experiencing a problem where @see annotations on classes are not visible in the final output.
For instance, you can see how this
https://github.com/contentful/contentful-management.php/blob/master/src/Management/Resource/Space.php
is rendered like this
https://contentful.github.io/contentful-management.php/api/master/Contentful/Management/Resource/Space.html
According to phpDoc's specs, @see can be applied to structural elements, thus including classes. Am I missing something, or is Sami ignoring the annotation? On methods the annotation is rendered correctly (though it'd be nice if it were displayed as an actual link instead of simple text).

It's a code thread it need to be recorded with .Com a internet Link
On Aug 28, 2017 5:31 AM, "Davide Borsatto"<EMAIL_ADDRESS>wrote:

I am experiencing a problem where @see annotations on classes are not
visible in the final output.
For instance, you can see how this
https://github.com/contentful/contentful-management.php/
blob/master/src/Management/Resource/Space.php
is rendered like this
https://contentful.github.io/contentful-management.php/api/
master/Contentful/Management/Resource/Space.html
According to phpDoc's specs
https://docs.phpdoc.org/references/phpdoc/tags/see.html, @see can be
applied to structural elements, thus including classes. Am I missing
something, or is Sami ignoring the annotation? On methods the annotation is
rendered correctly (though it'd be nice if it were displayed as an actual
link instead of simple text).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FriendsOfPHP/Sami/issues/301, or mute the thread
https://github.com/notifications/unsubscribe-auth/Abst5LUnIRstM485Ypy1_wNiM_tm9FOuks5scoj9gaJpZM4PESno
.

There looks to be a working pull request (#289) that includes this functionality.


Go through all Redux resources

Course: https://www.youtube.com/watch?v=hmwBow1PUuo
Project Generator: https://github.com/davezuko/react-redux-starter-kit
https://github.com/cloudmu/react-redux-starter-kit/blob/master/src/reducers/users.js
https://github.com/reactjs/react-redux/issues/361#issuecomment-212491715

duplicate of https://github.com/GoTeamEpsilon/angular-to-react/issues/45


da.scaleValue is always 0

import { usePinch } from "@use-gesture/react";
import React, { useRef, useState } from "react";
import { useSpring, animated } from "react-spring";

function ScalableFrame({ children }: { children?: React.ReactNode }) {
  const [{ xyzs }, set] = useSpring(() => ({ xyzs: [0, 0, 0, 100] }));
  const ref = useRef();

  usePinch(
    (state) => {
      const { da, offset } = state;
      const [scale] = offset;
      console.log("scaleValue", scale, da);
      set({ xyzs: [0, 0, 0, scale] });
    },
    { target: ref }
  );

  return (
    <div
      ref={ref}
      style={{
        width: "100%",
        height: "100%",
        transform: xyzs
          .to((x, y, z, s): string => {
            return `translate3D(${x}px, ${y}px, 0) scale(${s / 100})`;
          })
          .get(),
      }}
    >
      {children}
    </div>
  );
}

The value of console.log("scaleValue", scaleValue, da); is always 0.
Code referenced https://codesandbox.io/s/219jx?file=/src/index.js:396-809

What device are you using? It would be great if you'd fill the information at the bottom of the issue template, this makes things easier for me.

I made it work using
offset#scale
  usePinch(
    (state) => {
      const { offset } = state;
      const scale = offset[0];
      onRescale(scale);
    },
    { target: ref }
  );

Related - https://github.com/gridaco/designto-code/pull/97

I'm closing since this is a discussion and you're being super vague.

That's bit rude but ok


feat(cmd/log): Implementation of the kubectl trace logs command

TODO:

[x] basic end to end functionalities to get the actual logs of a bpftrace program
[ ] make the ability to follow logs optional
[ ] write down examples and long description
[ ] update readme

Closes #4
Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>

@fntlnz I rebased this on top of the master.


Add roll out strategy for Shanghai

Add roll out strategy for Shanghai

Good catch! Rephrased a bit to emphasize that there's no plan to move away from timestamps for upgrade scheduling.


print tendermint log to stdout only if ANOMA_TM_STDOUT=true

trying to hide the TM stdout to see if it maybe helps a bit with flakiness in our e2e tests.
Repeatedly running the CI, we got 19 e2e successes and 1 failure.

merged via https://github.com/anoma/namada/pull/222


ERROR: Invalid requirement: 'gdsfactory gplugins[femwell,gmsh,meow,sax,tidy3d]'

Hello
I am working on Windows 11 and I have encountered the following problem:

I have installed the mentioned two packages and Anaconda.

may you help me

But it was fixed with the following command that I took from this link; Is there a problem?
pip install "gdsfactory[full]" --upgrade


Correctness of Hessians and Gradients

From my understanding, for binary objective without sample weight and class weight, we should have (like in xgboost) : gradient = (p - y), hessian = p * (1 - p)
But, in light-gbm we have : gradient = x, hessian = abs(x) * (1 - abs(x)), where x = -y/(1+e^(y*score))
Can someone please help me figure out how these two are similar or work similarly?

y is 1 or -1, so x equals to p.


protected_attributes is not capatible with rails 5.2.3 with ruby 2..6.3

Bundler could not find compatible versions for gem "activemodel":
In snapshot (Gemfile.lock):
activemodel (= 5.2.3)
In Gemfile:
draper was resolved to 3.1.0, which depends on
activemodel-serializers-xml (>= 1.0) was resolved to 1.0.2, which depends on
activemodel (> 5.x)
protected_attributes was resolved to 1.0.2, which depends on
  activemodel (>= 4.0.0.beta, < 5.0)

rails (~> 5.2.3) was resolved to 5.2.3, which depends on
  activemodel (= 5.2.3)

web-console (>= 3.3.0) was resolved to 3.7.0, which depends on
  activemodel (>= 5.0)

is there any other way to make compatible it with rails 5.2.3 to access attrtibutes

No. This gem doesn't support Rails 5 and will not support.


AF-2140: Fix restoring a custom editor

Also on this PR:

List of Known Issues on README.md
IT for restoring a custom editor

@tiagobento We should also report these issues in JIRA or Issues section here on github.

@domhanak I think we can do that, but I think having the known issues on the README is also valuable for people who don't know about our JIRA.

@tiagobento Yea I agree, we can link JIRA there :)

@domhanak Done. Are we good to merge?


G2 Radar chart using javascript - reverse plotting of values

G2 Version: 4.1.42
Platform: windows / mac
Mini Showcase(like screenshots):

Plotting values in reverse is not working in Antv G2 Radar chart.
I am trying to plot chart.scale as min:5, max:1 - so the values in the middle should be 5 and should reduce to 1 when scaling outside. But, trying this in antv g2 chart, the chart is not responding properly and taking random axis values irrespective of given values(only if min value is greater than max value).
Much appreciable, if anyone has any idea about this to share.!
Thanks!

Very special situation.
A solution, the labels whose from 1.0 to 5.0 are just text label.

process the demo, (x) => 5 - x to reverse the data
show yaxis label 1.0 with 5.0, and so on

chart.axis('score', {
  label: {
    formatter: (v) => 5 - v, // here
  }
});


[Windows 10]Profile Name Error and White Screen Issue After Seedvault/Seed Use

Issue description
At first when you enter a profile name you get an error then if you click continue with the same name you can proceed. After attempting to use the seedvault created from Trinity or 81 character long seed, we get an empty unresponsive whitescreen.
Edit: Profile name issue might be due to the directory Firefly installed includes user name having "ğ" letter.
Wallet version
Desktop 1.0.0
System specification
Windows 10
Is it a repeatable issue?
Y
Steps To reproduce the issue
Explain how the maintainer can reproduce the issue.

Enter your seed or use seedvault.
Press continue.
See whitescreen.

Expected behaviour
We expected to see the amount to be migrated.
Actual behaviour
Whitescreen occurs.
Errors

This happend to me too, I use a user name with a special character 'ä', the user directory where the files are located contain this special character too.
See user path from Diagnostic:
User Path: C:\Users\Bänni\AppData\Roaming\Firefly

To be honest I'd rather not go through the hoops of changing the username folder in Windows 10. And it appears many people are encountering this issue so far, any news from the IF side?

@Grachas we're working on a patch for the issue

Closing as duplicate of #1015


Strong naming

We've had a long standing request to strongly name the assemblies. Thanks to @Sicos1977 this happened.
Sorry
This is replacing PR #68 with this PR. Maybe there was a better way to push further changes but it is late. I am totally giving @Sicos1977 credit in the release notes.
Closes #58

CI looks like it is not Building the SNK from the environment. Well it worked on my box. 😕 🤔
I'll look at this later.


[makeMarkdown] Must we use jsdom?

Some environments don't support dom manipulations. Is there any other implements instead of jsdom? Like regex or something to parse it? I don't know if that's too hard to implement, but it would be more compatible.

Hope this could be a good reference: https://github.com/Jxck/html2json

@Songkeys Which environments are incompatible with JSDOM?


Implement a VobSub decoder

Around https://docs.rs/vobsub/0.2.3/vobsub/, probably best as GstVideoDecoder subclass once we have it... or GstElement.
Needs some changes to expose the &[u8] based API that is currently internally, and accepts elementary stream packets.
The repository also has some SRT code, might be worth adding something around that too.
CC @emk who brought this to my attention on reddit

Sounds like a fun project!
There's a couple of us working on Rust subtitle support. I have an *.srt decoder in substudy, and there's also a list of other people working on textual subtitle decoding in Rust. I also remember there being a few crates on crates.io for related things.
(My very long-term goal for the subtitles-rs is to improve the current state of Vobsub OCR significantly, allowing automatic conversion from VobSub to SRT for major languages. Existing solutions rely on either print OCR engines (which are limited to a few languages, and which are messy to set up), or manual letter identification (which is tedious). But that's not going to be finished soon.)

FWIW, I started working on this.

This is now tracked in https://gitlab.freedesktop.org/gstreamer/gst-plugin-rs/issues/21


Large result set

Description
Taking in offsets and limits. If neither offsets nor limits are passed, the client makes as many requests as needed to get the data.
GitHub Issues

#39

Coverage decreased (-1.08%) to 83.059% when pulling 096f1dc698d91bdd2c58c119e7cfcea75263d972 on mcrowson:large_result_set into 9ee85c1ba08ca066d0c78e06b1da5e6c9a4cfffe on refindlyllc:development.

also #13


feat: add expiration time, print sig-msg tuple for manual verification

Add extra default field to expand test use-cases
console.log tuple so that user can generate and test manually on their platform

The notepad example was moved out of this repo, please check https://github.com/spruceid/siwe-notepad


fastmri requirement

Hi deepinvs,
if I understand correctly, you "only" need fastmri in this demo here.
I suppose that the core (and all unit tests) of deepinv also works without fastmri.
If that is the case, I think it would be better to remove fastmri as an explicit dependency
to potentially avoid the "dependency hell"
Georg
PS: I am asking since I am trying to get deepinv on conda-forge which needs fastmri on conda-forge first,
which is harder than I tought ;)

Hi Georg, thanks a lot for the feedback! You are completely correct, we could actually remove the fastmri dependency. I'll close this issue when we remove it.

Great. That will facilitate the conda-forge build a lot :)
Would it be also possible to be a bit more specific (restrictive) in the required versions of the dependencies in pyproject.toml?
I can e.g. imaging that you need a torch version that is not "too old".

Sure! From memory we tested in torch>1.9 but for other dependencies I don't remember, I'll have a check.

Hi Georg, we've finally removed the fastmri dependency - which beared several spurious dependencies like pytorch-llightning... Closing this issue for now, don't hesitate to re-open if it didn't fix for you!


Remove [CoinGecko] prints, move ticker update back to backround queue #5010

Closes #5010

Why remove the logging?

it quite oversized, and it's hard to figure out the other one logs


Fixed #8571 (Required Sign Asterisk)

Issue Reference
#8571
Description
Just Changed the Field-type blade file, used existing rendered validation in file for required sign setup
How To Test This?
Go to 'Admin panel'
Click on 'Configuration'
Scroll down to 'Email setting'
It showing the asterisk sign for the required fields.
Documentation

[ ] My pull request requires an update on the documentation repository.

Hi @JunaidJaved248,
We have other plans to solve this and we are thinking of a long-term perspective so we need a permanent fix. Not a quick fix.
Hint:
Just go through the system.php file and connect all validations with the frontend and backend as well.

Hi @JunaidJaved248,
We have other plans to solve this and we are thinking of a long-term perspective so we need a permanent fix. Not a quick fix.
Hint: Just go through the system.php file and connect all validations with the frontend and backend as well.

@devansh-webkul , I've employed the system.php validation and examined other files where similar validation checks are applied. Specifically, in the system.php file related to email configuration, I noticed the validation rule 'required|max:50' being used.
I've replicated this approach in the frontend to determine if a field's validation includes the 'required' rule. If it does, I display an asterisk next to that field.
Alternatively, I'm considering introducing a new key called 'is_required' with values 'true' or 'false'. This would allow us to easily check this requirement in the frontend.
What would be your recommendation regarding this approach?

For the front end, we are using vee validate 4, my approach would be to connect with the vee validate 4 so that if any validation comes it will automatically attached to vee validate 4. Rest all validation will get handled by vee validate 4. If there is some customized validation then I can think of that.
For backend it is already Laravel stuff so easy one.

Right now, you are using is_required. If tomorrow there are some more validation. We need to handle it as per that perspective.

@devansh-webkul I have reviewed again with others like create customer,
This Page Validation process is same as other form like customer,
As per reference, you can see that for showing asterisk with label of field, Just added class="required",
I just used same scenario for it, For fixing this issue,
I just shown asterisk on based on $validation from system.php file. It seems to fine to me. If still something there that i am skipping now.

@devansh-webkul Can you update on it?

@devansh-webkul Its Done


Review and update SEO/OG tags

SEO Elements to be optimized

Title Tag
Meta Description
H1
Canonical URL

Open Graph Tags

OG:URL
OG:TItle
OG:Description
OG: Image

@10thfloor please take a look! we'd also like to understand the page types we have on the portal, so that there could be tags optimized for there. right now, it seems like there are at least two:

hard-coded landing pages
markdown-base / internal pages

But, I assume we implemented a few more page types, so input here would be useful 🙏

These are the only 2 pages types.

SEO Recommendations Spreadsheet
The spreadsheet above includes recommendations for the following

Title Tag
Meta Description
H1 Title
Canonical URL

Note: Duplicate URLs were discovered upon my initial page crawl (designated on the spreadsheet in red). We'll want to 301 redirect those pages to the original URL.
Open Graph Tags will use elements from SEO elements
OG:URL = Canonical URL
OG:TItle = Title Tag
OG:Description = Meta Description
OG: Image = TBD (Option: Designate one image (ex. Flow Logo) for all Developer pages)

This is currently blocked by resolving the route refactor changes.

https://github.com/onflow/next-docs-v1/issues/487
PR:

Unblocked!

Items to fix:

sitemap.xml - change [onflow.org](http://onflow.org) to [flow.com](http://flow.com) ✅
canonical url is not defined ✅
optimize title tag and description (Conway to create an excel - separate item in the table)
stretch goal - crosslinking (currently not required)

https://github.com/onflow/docs/issues/255


Mobile Primary Navigation Icons

@plhnk brought up the idea of having icons associated with our primary navigation at the bottom of the viewport on mobile -- this will look awesome and very app-like. Need an icon for Articles, Careers, Events, Team, and Handbook. Keep them simple and amazing. They should feel connected to the branding on liferay.design

see the Gatsby site for one example:


integration-cli: use cmd.Stdin instead of cat/tee for TestExportContainerAndImportImage

TestExportContainerAndImportImage:
os.Exec("bash", "-c", dockerBinary) ends up making a call like bash -c "c:\...\docker.exe" on windows which doesn't work because bash won't accept windows paths and there's no unix path available for dockerBinary.
This test makes use of exec.Command.Stdin to pass the docker exported image back to docker import using strings.Reader.

Upside: fixes the test on windows
Downside: cat/tee compatibility of the commands is no longer tested in this test case

Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>Label: #windows
cc: @tianon @unclejack @jfrazelle @icecrime

https://jenkins.dockerproject.com/job/Windows-PRs/35/console
=== RUN TestExportContainerAndImportImage
[PASSED]: export - export a container
[PASSED]: import - import an image
--- PASS: TestExportContainerAndImportImage (3.64s) :+1:

Nice! LGTM

+100
LGTM
IMO, this is how this test (and similar tests) should've worked in the first place. :+1:

LGTM
I am curious though about keeping everything in a buffer. At some point, wouldn't using a pipe between the two commands be a more appropriate solution?

LGTM

IMO, this is how this test (and similar tests) should've worked in the first place.

@tianon I thought it was intentional to test how unix tools work with cli. :open_mouth:
@duglin valid concern. in this case image is small, so I ignored it, also in this case I'd prefer runCommandWithOutput to return []byte; not string.

LGTM


Tree sticky scroll doesn't work as expected in searxch view

Testing #199153
The file name isn't covered, probably due to how the search tree is implemented.

Do you mean it's not detecting the tree node with the file name as sticky? Not able to reproduce that. Did you maybe set the workbench.tree.stickyScrollMaxItemCount value to 1?

I also couldn't repro this in the tree view for search.

Oh, I guess it was due to the tiny height of the tree then. This is actually working fine then 👌


fix(cypress): disable flaky tests

SUMMARY
The recent PR #22302 broke a few tests that were assuming certain examples were created by the admin user. For some reason CI didn't catch this, and gave the PR a green checkmark Since this is an incorrect assumption (they should not be created/owned by anyone), we need to refactor the tests so that they aren't testing incorrect behavior. In addition, the tests should be made less flaky to ensure they fail correctly.
This is just a hotfix to unblock master branch, but I will follow up with a proper fix later.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Ping @codyml and @EugeneTorap .


clmagma: init at 1.3

Motivation for this change
CLMagma is introduced, as magma is only available for CUDA.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip" doesn't apply
[x] Tested execution of all binary files (usually in ./result/bin/) doesn't apply
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build clmagma

On a NixOS machine, I ran
nix-review pr 54967

and got this build failure
builder for '/nix/store/aa88m794ffdnhi3xnqhzi5wjxlry7k4i-clblas-2.12.drv' failed with exit code 2; last 10 log lines:
                                                 ~~~~~~~^
       ValuesIn(orderSet), ValuesIn(uploSet), ValuesIn(transSet),
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       Values(QUEUES_TEST_MATRIX_SIZES), Values(QUEUES_TEST_MATRIX_SIZES),
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       Values(clMath::ExtraTestSizes()), ValuesIn(numQueues)));
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  make[2]: *** [tests/CMakeFiles/test-medium.dir/build.make:700: tests/CMakeFiles/test-medium.dir/correctness/test-correctness.cpp.o] Error 1
  make[1]: *** [CMakeFiles/Makefile2:400: tests/CMakeFiles/test-medium.dir/all] Error 2
  make: *** [Makefile:152: all] Error 2
cannot build derivation '/nix/store/il4ixcf3l4hbqfmq3s1m8wa2gq5xi8jx-clmagma-1.3.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/zjnr063zj7l275chdmn98b7dls5gm831-env.drv': 1 dependencies couldn't be built

It looks like there is some issue with clblas

@ryantm Yes, clblas is broken on my machine too. It has some issues with the test suite and I have just disabled them in order to compile clmagma. I'm not sure I have time to debug clblas for now (that's why I didn't include it in the PR at all), but adding "-DBUILD_TEST=OFF" to the clblas's cmakeFlags is enough to test that clmagma builds properly.

Are there any updates on this pull request, please?

Since 4c74ee948ed6b709f79fa7af809ebba7e308504b fixed clblas (by disabling tests), this pull request can be now merged. Library builds on my computer without any extra patches. Sorry for the delay.


Inconsistent connection state reported by nodes

Inconsistent connection state:

NodeCA (<IP_ADDRESS>) "thinks" it is connected to NodeAA
NodeAA (<IP_ADDRESS>) is not showing connection to NodeCA

Logs
NodeAA-CA Logs.zip
Node connections

Code version
9535e61e (06 Dec @ 15:32); Patches: Kevin

*** Build date: 06/12/2018 12:34:06.68
*** Specific commit:

*** Recent commits
*    9535e61e - (HEAD -> master, origin/master, origin/HEAD) syncing speedup (#2901) (Thu, 6 Dec 2018 15:32:25 +0300) <noescape0>
*    63b750a3 - Fix DoS vector (#2904) (Thu, 6 Dec 2018 12:53:36 +0300) <noescape0>
*    3719dab9 - Set the transaction fee to the network defined minimum fee (#2907) (Thu, 6 Dec 2018 08:05:27 +0000) <Jeremy Bokobza>
*    e2af0c02 - (origin/sc/v0.13.0-beta) Add version to network folder name (#2915) (Thu, 6 Dec 2018 15:11:18 +1100) <Rowan de Haas>
*    f4a8ec92 - Increment magic and nonce (#2914) (Thu, 6 Dec 2018 15:05:31 +1100) <Rowan de Haas>
*    0c313825 - Increment SC version (#2911) (Thu, 6 Dec 2018 12:20:01 +1100) <Rowan de Haas>
*    0f93ca37 - RuntimeObserver -> Stratis.SmartContracts.RuntimeObserver (#2912) (Thu, 6 Dec 2018 12:13:24 +1100) <Jordan Andrews>
*    5c8c08f9 - Powershell NuGet fix + version updates (#2900) (Thu, 6 Dec 2018 10:41:10 +1100) <Jordan Andrews>
*    a6a8d2b7 - Added custom path for KeyTool (#2902) (Wed, 5 Dec 2018 17:05:43 +0000) <Jeremy Bokobza>
*    528337cc - Missing inputs error was not reporting correctly the error (#2906) (Wed, 5 Dec 2018 16:53:23 +0000) <Dan Gershony>
******************

*** Repackaged: 06/12/2018 12:51:28.90

I've observed this issue now as well.
I had a C# node running, connected to a my GW which was connected to X.
I shutdown the GW node but my C# is still reporting a connection:
MempoolSize: 7    DynamicSize: 830 kb OrphanSize: 0

[2018-12-09 10:29:00.3299 39] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__76_0 ======Node stats====== 12/09/2018 10:29:00
Headers.Height:      673764   Headers.Hash:     bfac1009cd99d2dc76aab13b4b21787893896e43865e7e5a4e8b6d89915b148a
Consensus.Height:    673764   Consensus.Hash:   bfac1009cd99d2dc76aab13b4b21787893896e43865e7e5a4e8b6d89915b148a
BlockStore.Height:   673764   BlockStore.Hash:  bfac1009cd99d2dc76aab13b4b21787893896e43865e7e5a4e8b6d89915b148a
Wallet.Height:       No Wallet

======Connection====== agent fassa-StratisNode:1.2.6 (70012) [in:0 out:15]
...
Peer:[::ffff:<IP_ADDRESS>]:26178,      connected:outbound,       (r/s):673600/673764         agent:/Stratis:<IP_ADDRESS>/
...

GW node:
[2018-12-09 10:29:45.2110 12] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__76_0 ======Node stats====== 12/09/2018 10:29:45
Headers.Height:      673766   Headers.Hash:     59f2ccb962ec37d81e29c8e0052085c9b7ba3661f214967ce734101cef53376e
Consensus.Height:    673766   Consensus.Hash:   59f2ccb962ec37d81e29c8e0052085c9b7ba3661f214967ce734101cef53376e
BlockStore.Height:   673766   BlockStore.Hash:  59f2ccb962ec37d81e29c8e0052085c9b7ba3661f214967ce734101cef53376e
Wallet.Height:       No Wallet

======Connection====== agent fassadlr-StratisNode:1.2.6 (70012) [in:0 out:1]
Peer:[::ffff:<IP_ADDRESS>]:26178,           connected:outbound,       (r/s):673766/673766         agent:/Stratis:<IP_ADDRESS>/

I will investigate this one a bit.

Did you have anything to report @fassadlr about this?

watching at NodeAA.tcpdump I see 3 connections from <IP_ADDRESS> to <IP_ADDRESS>:

at 14:58:35 (from port 40971)
at 14:58:45 (from port 40485)
at 15:18:37 (from port 43239)

I see 2 FIN (connection end):

at 14:58:50 (from port 40485)
at 15:18:36 (from port 40971)

This mean that:
first connection starts at 14:58:35 and end at 15:18:36
-duration ~ 20 minutes
second connection starts at 14:58:45 and end at 15:18:50
-duration 5 seconds
third connection starts at 15:18:36 and doesn't close (may be because of end of tracing)
So it seems actually these peers are connected. Invastigating other logs.

in NodeCA log (<IP_ADDRESS>) I see it went offline

[2018-12-06 14:18:36.5264 39] INFO: Stratis.Bitcoin.Connection.ConnectionManagerBehavior+d__15.MoveNext Peer '[::ffff:<IP_ADDRESS>]:16178' offline, reason: 'Pong timeout for<PHONE_NUMBER>1652718343'.
and get reconnected later

Object reference not set to an instance of an object. ??
[2018-12-06 13:58:35.1674 8] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeerConnection.ConnectAsync (endPoint:[::ffff:<IP_ADDRESS>]:16178,)
[2018-12-06 13:58:35.3101 7] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeerConnectionParameters.CreateVersion (-)$exception:System.NullReferenceException: Object reference not set to an instance of an object.
   at NBitcoin.Protocol.NetworkAddress.set_Endpoint(IPEndPoint value) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\NBitcoin\Protocol\NetworkAddress.cs:line 37
   at Stratis.Bitcoin.P2P.Protocol.Payloads.VersionPayload.set_AddressFrom(IPEndPoint value) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Protocol\Payloads\VersionPayload.cs:line 126
   at Stratis.Bitcoin.P2P.Peer.NetworkPeerConnectionParameters.CreateVersion(IPEndPoint externalAddressEndPoint, IPEndPoint peerAddress, Network network, DateTimeOffset timeStamp) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Peer\NetworkPeerConnectionParameters.cs:line 95
[2018-12-06 13:58:35.3101 7] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeerFactory+<CreateConnectedNetworkPeerAsync>d__13.MoveNext (-)$exception:System.NullReferenceException: Object reference not set to an instance of an object.
   at NBitcoin.Protocol.NetworkAddress.set_Endpoint(IPEndPoint value) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\NBitcoin\Protocol\NetworkAddress.cs:line 37
   at Stratis.Bitcoin.P2P.Protocol.Payloads.VersionPayload.set_AddressFrom(IPEndPoint value) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Protocol\Payloads\VersionPayload.cs:line 126
   at Stratis.Bitcoin.P2P.Peer.NetworkPeerConnectionParameters.CreateVersion(IPEndPoint externalAddressEndPoint, IPEndPoint peerAddress, Network network, DateTimeOffset timeStamp) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Peer\NetworkPeerConnectionParameters.cs:line 95
   at Stratis.Bitcoin.P2P.Peer.NetworkPeer..ctor(Boolean inbound, IPEndPoint peerEndPoint, Network network, NetworkPeerConnectionParameters parameters, IDateTimeProvider dateTimeProvider, ILoggerFactory loggerFactory, ISelfEndpointTracker selfEndpointTracker, Action`1 onDisconnected, Action`2 onSendingMessage) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Peer\NetworkPeer.cs:line 292
   at Stratis.Bitcoin.P2P.Peer.NetworkPeer..ctor(IPEndPoint peerEndPoint, Network network, NetworkPeerConnectionParameters parameters, INetworkPeerFactory networkPeerFactory, IDateTimeProvider dateTimeProvider, ILoggerFactory loggerFactory, ISelfEndpointTracker selfEndpointTracker, Action`1 onDisconnected, Action`2 onSendingMessage) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Peer\NetworkPeer.cs:line 320
   at Stratis.Bitcoin.P2P.Peer.NetworkPeerFactory.CreateConnectedNetworkPeerAsync(IPEndPoint peerEndPoint, NetworkPeerConnectionParameters parameters, NetworkPeerDisposer networkPeerDisposer) in C:\Code\InternalTestnet_Data\Sources\StratisBitcoinFullNode\src\Stratis.Bitcoin\P2P\Peer\NetworkPeerFactory.cs:line 196
[2018-12-06 13:58:35.3101 7] TRACE: Stratis.Bitcoin.P2P.PeerDiscovery+<<DiscoverPeersAsync>b__15_2>d.MoveNext Discovery from '[::ffff:<IP_ADDRESS>]:16178' finished

[2018-12-06 13:58:45.3088 15] TRACE: Stratis.Bitcoin.P2P.PeerDiscovery+<b__15_2>d.MoveNext Attempting to discover from : '[::ffff:<IP_ADDRESS>]:16178'

this explain the connection from port 40485 was a PeerDiscovery connection, this is why got disconnected fast

@maciejzaleski
I see that NodeCA has connection to  <IP_ADDRESS>, so same range?. Range filter was off?
beside this, NodeCA received a ping from NodeAA

[2018-12-06 13:58:45.3354 15] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeerConnection+d__29.MoveNext [2-[::ffff:<IP_ADDRESS>]:16178] Received message: 'ping: PingPayload :<PHONE_NUMBER>4287925274'

but it seems it didn't replied back because after 20 minutes, NodeAA disconnected to it because of pong timeout

[2018-12-06 14:18:45.3657 38] TRACE: Stratis.Bitcoin.P2P.Protocol.Payloads.PingPayload.ToString (-):'PingPayload :<PHONE_NUMBER>4287925274'
[2018-12-06 14:18:45.3657 38] TRACE: Stratis.Bitcoin.P2P.Protocol.Behaviors.PingPongBehavior.PingTimeout (ping:PingPayload :<PHONE_NUMBER>4287925274)
[2018-12-06 14:18:45.3657 38] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeer.Disconnect (reason:'Pong timeout for<PHONE_NUMBER>4287925274',exception:)
[2018-12-06 14:18:45.2756 42] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeerCollection.Match (ip:::ffff:<IP_ADDRESS>,port:16178,)
[2018-12-06 14:18:45.3657 38] TRACE: Stratis.Bitcoin.P2P.Peer.NetworkPeer.Disconnect [28-[::ffff:<IP_ADDRESS>]:40485] (-)[DISCONNECTED]

@maciejzaleski
Understanding what happened, without having contextual thread information in logs is a pain (asyn code may mix log output) but what I can say is taht NodeCA seems to be disconnected from NodeAA because another peer with IP <IP_ADDRESS> was already connected and so NodeAC(<IP_ADDRESS>) is RANGE_FILTERED
this is the output of NodeAA and you can see that before NodeAC connection it has already a connected peer with same group
[2018-12-06 13:58:14.3954 19] INFO: Stratis.Bitcoin.Connection.ConnectionManagerBehavior+<OnStateChangedAsync>d__15.MoveNext Peer '[::ffff:<IP_ADDRESS>]:33763' connected (inbound), agent 'StratisNode:1.2.6 (70012)', height 0
[2018-12-06 13:58:14.9182 13] INFO: Stratis.Bitcoin.Utilities.AsyncLoop+<>c__DisplayClass16_0+<<StartAsync>b__0>d.MoveNext WalletManager.DownloadChain starting.
[2018-12-06 13:58:14.9182 19] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__76_0 ======Node stats====== 12/06/2018 13:58:14
Headers.Height:      0        Headers.Hash:     0000066e91e46e5a264d42c89e1204963b2ee6be230b443e9159020539d972af
Consensus.Height:    0        Consensus.Hash:   0000066e91e46e5a264d42c89e1204963b2ee6be230b443e9159020539d972af
BlockStore.Height:   0        BlockStore.Hash:  0000066e91e46e5a264d42c89e1204963b2ee6be230b443e9159020539d972af
Wallet.Height:       0        Wallet.Hash:      0000066e91e46e5a264d42c89e1204963b2ee6be230b443e9159020539d972af

======Connection====== agent StratisNode:1.2.6 (70012) [in:2 out:1]
Peer:[::ffff:<IP_ADDRESS>]:33763,          connected:inbound,        (r/s):0/0                   agent:StratisNode:1.2.6 (70012)
Peer:[::ffff:<IP_ADDRESS>]:34671,          connected:inbound,        (r/s):0/0                   agent:StratisNode:1.2.6 (70012)

It seems to me a race condition, we need better logging (#2969) and maybe some more statistical data on connections (#2968)

without having contextual thread information in logs is a pain (asyn code may mix log output)

logs produced by nlog config in documentation folder should contain thread number in them for each line.

I can say is taht NodeCA seems to be disconnected from NodeAA because another peer with IP <IP_ADDRESS> was already connected and so NodeAC(<IP_ADDRESS>) is RANGE_FILTERED

That seems like that case. Though I thought that @maciejzaleski recently updated the network so all of the peers are in the different IP groups.

Closed


Locale_id

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

Addon version used
beta23
Description
Pluging cant open after update to beta23. It says locale_id. My lang at netflix is Turkish.
Installation

[x] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[ ] I'm using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
LibreElec 9.0.1 on RPi3
Debug log
https://paste.ee/p/hZOAD
Other information
[e.g. detailed explanation, related issues, suggestions how to fix, links for us to have context, etc.]

my mistake, fixed, later you can download the dailybuild


CardinalSpline rendering has extra line

The following sample code (inside of an object that implements CCDrawNode:
var splinePoints = new List<CCPoint> ();
splinePoints.Add (new CCPoint (0, 0));
splinePoints.Add (new CCPoint (50, 70));
splinePoints.Add (new CCPoint (0, 140));
splinePoints.Add (new CCPoint (100, 210));
splinePoints.Add (new CCPoint (0, 280));
splinePoints.Add (new CCPoint (150, 350));

int numberOfSegments = 64;
float tension = .05f;
base.DrawCardinalSpline (splinePoints, tension, numberOfSegments);

This results in the following rendered spline:

Screenshot was taken using Genymotion emulator, Nexus 7.
This was performed on CocosSharp build <IP_ADDRESS>
I did not test this on other platforms, just Android.

Weird, I can not recreate this in master source but it does show up when using the package.

This should not be happening in source base.  See DrawPrimitiveTest that was added in commit above.


Missing header

The USB_PORTATTR_xxx flags for USB_NODE_CONNECTION_ATTRIBUTES are defined in USB.h, so that should also be a requirement of this IOCTL

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

ID: 0a90b50c-655d-039e-086f-480d8a3350ae
Version Independent ID: b3ff6e4a-1ef9-3741-aef5-15255398eee0
Content: IOCTL_USB_GET_NODE_CONNECTION_ATTRIBUTES
Content Source: wdk-ddi-src/content/usbioctl/ni-usbioctl-ioctl_usb_get_node_connection_attributes.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Hi @BrianCatlin,
I’ve created an internal work item for myself to investigate this with the USB team, and I’ll make a note here when I complete the changes. In the meantime, I’m closing the GitHub issue while it’s tracked in the bug. Thanks again for helping to improve the docs.


Add RedHat/CentOS 9 support

can you add rocky and alma linux support too? you can do so in another PR if you prefer :)


Multiple Values for List type Slot

Rasa Core version: 0.8.5
Python version: 2.7
Operating system (windows, osx, ...): Ubuntu 17.10
Issue:
When should we expect this addition , Allowing multiple values for list type slot? If its not coming soon then Any quick solution?
Content of domain file (if used & relevant):

This is a duplicate of #307 so i will answer there


Bugfix for radix rule and property access

PR checklist

[x] Addresses an existing issue: #2281
[x] New feature, bugfix, or enhancement

[x] Includes tests

[ ] Documentation update

Overview of change:
[bugfix] don't warn for missing radix on method calls
Fixes: #2281
[enhancement] added check for global.parseInt and window.parseInt

I wouldn't mark this as fixing #2281 since fixing that would require making this an OptionallyTypedRule.

You are right, that it does not fix all cases.
I wouldn't make it an OptionallyTypedRule, because it would not fail in CLI but fail in vscode, which is really annoying.
Also other rules like this (no-eval, no-construct, array-type, ...) don't check if the identifier is shadowed. That's what line switches are made for.
If the current behavior is really a problem (awating more feedback on this), we could use some code from no-shadowed-variable (I'm currently working on a reusable utility)


Manage DLL Heck in PowerShell

SQL Server is made up of a vast number of components, many managed by different teams.
Some components are being updated while others are not which leads us to DLL heck.
The way that .NET handles this is through app.config files but I don't know how to beautifully load an app.config, which looks like this
    <dependentAssembly>
        <assemblyIdentity name="System.Runtime.CompilerServices.Unsafe" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>"/>
    </dependentAssembly>

And then this:

That basically says "if any components call for old versions of this dll, just give them the new version instead".
You can read more here, which I found right after running into my first error after mixing and matching SMO, SqlClient and DacFx: https://stackoverflow.com/questions/62764744/could-not-load-file-or-assembly-system-runtime-compilerservices-unsafe
Right now, I solve it with this code, which is working. It had to be written in C# otherwise, I'd get a StackOverflow error in Appveyor.
if ($PSVersionTable.PSEdition -ne "Core") {
    $dir = (Join-Path $script:libraryroot "lib\net462\").Replace('\','\\')

    if (-not ("Redirector" -as [type])) {
        $source = @"
            using System;
            using System.Linq;
            using System.Reflection;
            using System.Text.RegularExpressions;

            public class Redirector
            {
                public Redirector()
                {
                    this.EventHandler = new ResolveEventHandler(AssemblyResolve);
                }

                public readonly ResolveEventHandler EventHandler;

                protected Assembly AssemblyResolve(object sender, ResolveEventArgs e)
                {
                    string[] dlls = {
                        "System.Runtime.CompilerServices.Unsafe",
                        "System.Resources.Extensions",
                        "Microsoft.SqlServer.ConnectionInfo",
                        "Microsoft.SqlServer.Smo",
                        "Microsoft.Identity.Client",
                        "System.Diagnostics.DiagnosticSource",
                        "Microsoft.IdentityModel.Abstractions",
                        "Microsoft.Data.SqlClient",
                        "System.Configuration.ConfigurationManager",
                        "Microsoft.SqlServer.Replication",
                        "Microsoft.SqlServer.Rmo"
                    };

                    var name = new AssemblyName(e.Name);
                    var assemblyName = name.Name.ToString();
                    foreach (string dll in dlls)
                    {
                        if (assemblyName == dll)
                        {
                            string filelocation = "$dir" + dll + ".dll";
                            //Console.WriteLine(filelocation);
                            return Assembly.LoadFrom(filelocation);
                        }
                    }

                    foreach (var assembly in AppDomain.CurrentDomain.GetAssemblies())
                    {
                        // maybe this needs to change?
                        var info = assembly.GetName();
                        if (info.FullName == e.Name) {
                            return assembly;
                        }
                    }
                    return null;
                }
            }
"@

        $null = Add-Type -TypeDefinition $source
    }

    try {
        $redirector = New-Object Redirector
        [System.AppDomain]::CurrentDomain.add_AssemblyResolve($redirector.EventHandler)
    } catch {
        # unsure
    }
}

$dll =
if ($PSVersionTable.PSVersion.Major -ge 6) {
    Join-Path $script:libraryroot "lib\net6.0\dbatools.dll"
} else {
    Join-Path $script:libraryroot "lib\net462\dbatools.dll"
}

try {
    Import-Module "$dll"
} catch {
    throw "Couldn't import dbatools.dll | $PSItem"
}

if ($PSVersionTable.PSEdition -ne "Core") {
    [System.AppDomain]::CurrentDomain.remove_AssemblyResolve($onAssemblyResolveEventHandler)
}

I would also like a way to run commands (such as a disconnect) before and after specific commands without actually having to code that into there. When we talked, you mentioned this syntax which I liked.
[Reflection.AssemblyMetadata("TypeConflict", "t")]
[Reflection.AssemblyMetadata("TypeConflict", "t2")]
[Reflection.AssemblyMetadata("TypeConflict", "t3")]
param()

Thank you!

@potatoqualitee I think I finally found a good way to represent dependencies (which ironically is how the xml you mentioned at the top of this thread)
[runtime.CompilerServices.dependency("AnythingWeNeedIncludingADLL", "Always")]
[runtime.CompilerServices.dependency("AnythingWeNeedIncludingADLL", "Sometimes")]

Essentially, we should be able to use the presence of a [Runtime.CompilerServices.Dependency] attribute on a [ScriptBlock] to  specify that a dependency exists.
If that dependency is an easily resolvable assembly, a given script would be golden.
If that dependency is, say, a path, it would be about as easy to manage.
If that dependency were something more complex (like, say, application X must be installed), we could represent that as a [ScriptBlock], which would cleanly fit into the attribute:
[Runtime.CompilerServices.Dependency({<# whatever we want #>}, "Always")]
param()

What do you think?


Meta data readable by Facebook/Twitter?

Can facebook/twitter bots read meta data as of now with latest angular i see the meta service introduced.

Yeah I've also noticed that meta service has been introduced by Angular. However, it only contains a small subset of ng2-metadata utility.
About facebook/twitter bots, there's still no remarkable update.


feat: (IAC-284) Request to include AWS FSx for Lustre

Customers looking to modernize already have existing infra with FSx for Lustre and it would be useful for Viya to include this FSx for Lustre. [Please take a look at SAS Grid architecture](https://docs.aws.amazon.com/eks/latest/userguide/fsx-csi.html

Closing this as we are not pursuing FSx for Lustre in the IAC side of things at this time.  However, we added support for FSx for NetApp ONTAP since the iac-aws 7.2.0 release.

@sayeun does it mean that SAS does not recommend FSX for Lustre with SAS Viya and NetApp ONTAP is recommended ?


select() returns error when no value/text selected

Mostly an enhancement, but I'm currently having to work around this:
Assume you have a select multiple element but with no value selected:
<select multiple id="my_id">
<option value="volvo">Volvo</option>
<option value="saab">Saab</option>
<option value="mercedes">Mercedes</option>
<option value="audi">Audi</option>
</select>
and
select(:nothing_selected_box, id: 'my_id')
then
on(MyPage).nothing_selected_box
yields the error "undefined method 'text' for nil:NilClass"
I suggest that upon encountering a select box with nothing selected, the method return an empty string "" rather than error.

The fix for this bug is in a branch to get merged and will be in the 2.2.0 release.
https://github.com/cheezy/page-object/pull/432


a bit complex demo, failed....

\begin{array}{*{20}{c}}
1&3&\Theta \\
{\frac{{dy}}{{dx}}}&{\mathop {\lim }\limits_{\delta x \to 0} }& \pm \\
{\sum\limits_1^3 2 }&x& \ne
\end{array}\theta

above code copy from MathType, its cannot render....

KaTeX doesn't yet support the repeated column description *{20}{c}. It's one of the issues tracked by #269. As you are only using three columns, writing ccc instead would work well enough. Support for \mathop is a recent addition; in case you're still using KaTeX 0.6.0 (as we have CDN issues with 0.7.0), you might want to omit that, too. The following works on https://khan.github.io/KaTeX/:
\begin{array}{ccc}
1&3&\Theta \\
{\frac{{dy}}{{dx}}}&\lim_{\delta x \to 0}& \pm \\
{\sum\limits_1^3 2 }&x& \ne
\end{array}\theta

I think I'll consider this a duplicate of #269.


Expand .gitignore

Brief Description
In this PR, we add additional files to ignore on the Github repository

.DS_Store
demos/ folder: this way, we can easily experiment locally with scenes
test-scene/ folder: this way, we can keep a local copy of the test
scene.

Pull Request Guidelines

[x] I have added pull requests labels which describe my contribution.
[ ] All existing tests passed.

[ ] I have added tests to cover my changes, of which pass.

[ ] I have compared the render output of my branch to master.
[ ] My code has passed the ESLint configuration for this project.
[ ] My change requires modifications to the documentation.

[ ] I have updated the documentation accordingly.

Just an idea:
Would it be useful to have a single "local" folder at the root with all of this kind of files that are only meant to be kept locally? For example:
local/test-scene
local/demos
local/tmp/orders
With only ignoring "local/*" we would ignore this and future local files.

Interesting idea Santiago. I agree there's something we can do here to handle these demo files better. I'll think about these options.
It would be nice if we had an automated way of installing all these local files. Putting the files in the repo officially is the easiest thing to do, but the problem with that is that when we npm install this repo as a dependency, the dependency will include the megabytes of examples scenes that won't be needed as a dependency.
Do you have any ideas how we can track these files while at the same time excluding them from the renderer when used as a dependency?

I did not fully understand the implication of the problem. I wouldn't want to delay a change of a .gitignore for such a small suggestion so feel free to disregard this!
Two things come to my mind and those might be extreme over-engineer this simple task:

Create a separate repo for all these additional files and add them as devDependency. Each time you npm install on the repo, it will fetch the newest scenes
An alternative might be, instead of using the http-server module to start the application, create a small static-server that also works a proxy, so each time one of these scenes is "fetched" if not present, it'd be downloaded to a tmp folder. Machete uses this same approach for orders.

Just an idea:
Would it be useful to have a single "local" folder at the root with all of this kind of files that are only meant to be kept locally? For example:
local/test-scene
local/demos
local/tmp/orders
With only ignoring "local/*" we would ignore this and future local files.

Yeah, I thought about something like that but the "future" needs are not clear. Keeping it simple for now.

Interesting idea Santiago. I agree there's something we can do here to handle these demo files better. I'll think about these options.
It would be nice if we had an automated way of installing all these local files. Putting the files in the repo officially is the easiest thing to do, but the problem with that is that when we npm install this repo as a dependency, the install will include the megabytes of examples scenes that aren't needed as a dependency.
Do you have any ideas how we can track these files while at the same time excluding them from the renderer when used as a dependency?

We could add a bootstrap.sh shell script, where we "setup" the repo. We could include fetching the test-scene, demos and other relevant files. Something to think about.


docs: revisit install process

Closes #4156
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (change restructuring the codebase without changing functionality)
[ ] Improvement (change adding some improvement to an existing functionality)
[X] Documentation update

Checklist

[X] I added relevant documentation
[X] follows the style guidelines of this project
[ ] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@davidberenstein1957 Shall we merge and start the process of maybe using a tab structure in another issue?


Picture library

Please upload your picture here

Circuit diagram about the working mechanism of H bridge
H bridge driver

H bridge, M1 and M4 conducting

H bridge, M2 and M3 conducting

H bridge, STOP

Building blocks of the drone: