Distributed learning with Async update does not work.

Hi All,
I am running currently distributed learning using the launch.py tool. My plan is to run the distributed learning by using Sync and Async distributed modes. For the Sync PS, the training works fine and I got the number. However, when I switch the kvstore type to "dist_async", the following error shows up:
terminate called after throwing an instance of 'dmlc::Error'   what():  [11:58:12] src/kvstore/././kvstore_dist_server.h:294: Check failed: sync_mode_ Updater needs to be set for async mode Stack trace returned 10 entries: [bt] (0) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x30cbe2) [0x2aedf8ebdbe2] [bt] (1) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x30d1b8) [0x2aedf8ebe1b8] [bt] (2) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x2ab1f11) [0x2aedfb662f11] [bt] (3) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x2ab6f10) [0x2aedfb667f10] [bt] (4) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x2ab74d4) [0x2aedfb6684d4] [bt] (5) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x2a97b06) [0x2aedfb648b06] [bt] (6) /work/projects/Project00755/envs/mxnet_gpu/lib/python3.5/site-packages/mxnet/libmxnet.so(+0x2f640d5) [0x2aedfbb150d5] [bt] (7) /shared/apps/gcc/4.9.4/lib64/libstdc++.so.6(+0xc9090) [0x2aee34139090] [bt] (8) /lib64/libpthread.so.0(+0x7e25) [0x2aedc9953e25] [bt] (9) /lib64/libc.so.6(clone+0x6d) [0x2aedca36934d]
Any idea why this error happens?
Thanks :)

Are you using Gluon? You would have to pass update_on_kvstore=True to trainer if you want to do async updates in Gluon.

@rahul003
yes I am using Gluon. The trainer is ImageClassification, would it work if export  update_on_kvstore=True?

@sandeep-krishnamurthy Improved this here https://github.com/apache/incubator-mxnet/pull/11910
@absalama You need to pass update_on_kvstore=True when creating the Trainer. Refer the test above.

Hi all,
Thank you all guys for you help. However, I am still seeing the issue even after passing "update_on_kvstore=True".
I am using example/gluon/ImageClassification with slurm tracker from dmlc. Please could you reference to the error trace on the top since the error is still generated from the same corner.
Thanks

Are you sure you changed set this variable on each of your machines, and can you post a snippet

In trainer.py I changed the default value of update_on_kvstore from None to True in the init method. The imageclassification.py initialises the trainer object so I assume that if the update_on_kvstore is True in the init method then it should be set?
We are using slurm cluster, and all nodes are sharing the same folder where mxnet source resides. So the any change should be seen by all worker.
Do I need to set something extra for slurm configuration?

What optimizer are you using?

Closing the issue here in favor of https://github.com/apache/incubator-mxnet/issues/11965
Thanks @absalama for reporting the issue.
Please reopen if closed in error


Adding soumit's key

please test this

please test this

please test this

failures are not related to this patch. Merging.


Add feature - across concerns

$ dolittle add feature <Name Of Feature>

Adds it to all the different concerns (Domain, Events, Read ---- ALL).
If modules are enabled, you need provide the module.

What should this actually do? Without any artifacts a feature is just a folder

I think that the newest changes I've made to the cli kinda does what you want. Check it out and merge it inn so I can publish the package.

Its just painful adding folders on every project. Should just be a simple ; add feature. Is this what is in the latest stuff?

It does not just add folders. You can do:
$ dolittle add blah Feature.SubFeature.SubFeatureN.ArtifactName
And it will create all the folders for you

I don't think that we need this

Maybe we're talking past each other here. This feature is about adding feature folders automatically unto each project with one command.
Si from anywhere type "dolittle add feature blah" and a folder called "blah" will be in Concepts, Domain, Events, Read and so forth. If using modules it would honor that and you'd have to specify which module - unless you're in a module doing it - then the module is implicit.
If I understood the rc file thing - that's where the path for the different concerns are. Which is where we'd go to figure out where to put things.


Plugin Crashed my Grafana

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I had added in the data source for my Powerwall Dashboard so I could see both systems from the same Grafana viewer. I added the  Influxdb and pointed to my server, added Sun and Moon (for weather info), then imported the "Powerwall Dashboard". It had been running just fine for several months, up until 2 days ago. After a reboot from a windows update the Grafana module in my Teslamate Docker, kept restarting and then failed. I tried to "Re-install" the Teslamate system, but it didn't correct the Grafana issue. I ran a backup on my original Teslamate, to recover my data. I then created a new Teslamate install folder called "Teslamate2". I ran the install and it created a Telsamate2 in docker. I had to stop my original Teslamate 1st.
It started just fine, so I imported my data.
I started to add in my Powerwall Dashboard info again, and it crashed again. I tried a 3rd, then 4th time, not adding the Powerwall the 4th time so I could keep my system running.
On another PC, I created a Teslamate, and a Powerwall Dashboard., and started again to find out the actual cause.
I added the Influx database - No Issue after restarting Docker.
I imported and added the Sun and Moon database - No Issue after restarting Docker.
I imported the dashboard - No Issue after restarting Docker (although one of the panels was missing it's plugin).
I imported the missing plugin: "Boom Table" - Grafana crashed!
I don't know why it crashed now, but didn't when I did this the first time months ago, but I tracked it down to when I added that plugin. I don't know how to remove that plugin from my Grafana, since it won't restart.
These are installed on Windows machines, and I'm not very savy at which command line to use for what (e.g. Windows cmd, git bash, wsl cmd). so any help would be greatly appreciated. I Loved having my Powerwall & Teslamate data on the same screen, and not having to "login" to which ever Grafana system I wanted to see.
Hopefully someone else has ran into this problem and can help. I saved the error log from Grafan if needed.
Please help...
Expected Behavior
System was running with this setup prior to 2 days ago. I'm not sure why this happened now???
Steps To Reproduce

Added Plugin: Boom Table
Crash!!!

Relevant log output
2023-01-11 18:23:13 logger=settings t=2023-01-11T23:23:13.72+0000 lvl=info msg="Starting Grafana" version=8.5.6 commit=623255cf2d branch=HEAD compiled=2022-06-14T09:25:38Z
2023-01-11 18:23:13 logger=settings t=2023-01-11T23:23:13.72+0000 lvl=info msg="The state of unified alerting is still not defined.

Screenshots
No response
Additional data
No response
Type of installation
Docker
Version
Not sure

An update to above...
I managed to get the plugin's installed, and setup the databases. The problem seems to start when I import a dashboard, any dashboard. Since the original dashboard I was importing had a lot of my own changes, I tried to import the original Powerwall-dashboard json file. Same crash!
Once it crashes, I don't know how to recover other than deleting the Grafana container, and the Grafana volume, then re-running the install script.


Download firmware

Need help to download the SM-A9080B firmware and not with a zip file please

Sorry, but I'm not sure what you're asking. There is no SM-A9080B, only an SM-A9080 and an SM-N908B. All firmware comes in ZIP files.

A9080 firmware please
On Thu, 21 Oct 2021, 21:20 Zachary Wander, @.***> wrote:

Sorry, but I'm not sure what you're asking. There is no SM-A9080B, only an
SM-A9080 and an SM-N908B. All firmware comes in ZIP files.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zacharee/SamloaderKotlin/issues/41#issuecomment-948973508,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWFFMYUMRFWEAPFT244MOYTUIBYZNANCNFSM5GO7K3PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hello, i tried to download firmware for S7 Edge and it downloaded a encrypted file with extention enc4, i tried to decrypt it with your tool in the tab decryption and nothing happened, what am i doing wrong? Kind regards


Review the introduction and methods once more

I'm going over the entire text once more to smooth out any rough spots. Some very minor changes so far.

@santisoler I'm done with the intro and methods. I'll get started on another PR for the results (based on this branch). Feel free to make any changes or merge when you're ready (I'll rebase the new PR).


Fast-Plast finished with only two directories (1_Trimmed_Reads and 2_Bowtie_Mapping)

Now, I'm trying to assemble the chloroplast genome of a diatom with Fast-Plast (v.1.2.8).
This is the command that I tried.
perl fast-plast.pl -1 L2_1.fq -2 L2_2.fq -n FastPlast --bowtie_index all --coverage_analysis --clean light --threads 8
But, at the end of this command, i got only two directories (1_Trimmed_Reads and 2_Bowtie_Mapping).
Other files including log file were removed.
I can not understand what is happen...

Can you send the log, error, and other output files that track progress?

Thank you for replying this issue.
I would like to send some files, but I don't have those files (They were removed at the end of above command).
Sorry, I'm very new for bioinfomatics, so if I have something to do, please let me know.

There should be some files produced with this naming format:
name_Fast-Plast_Progress.log
name_result_out.log
name__results_error.log
These are made early on and there is not automatic way that Fast-Plast removes them. They will say what errors there might be. Based on what you have described, my guess is either it is not installed properly (I would need some sort of error information to help with this) or there is a problem with your data. You should check to see if files are in the trimming directory. If so, move on to the bowtie2 directory to see what was produced.
If you are running this on a scheduler, then there should be some report that comes back if there were problems from the scheduler. It is difficult to help without more information.

I just did a clean install to see if there were issues. I found one in the way bowtie2 was installed by the installation script. If you redownload and try again, I think it will work. At least, that will get you over a hurdle.
Best,
Michael

Thank you for your kindness.
Following your advice, I redownloaded and set bowtie2 path, then it worked.
Thank you so much.
On Jul 22, 2020 06:55 +0900, Michael McKain<EMAIL_ADDRESS>wrote:
I just did a clean install to see if there were issues. I found one in the way bowtie2 was installed by the installation script. If you redownload and try again, I think it will work. At least, that will get you over a hurdle.
Best,
Michael
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mrmckain/Fast-Plast/issues/46#issuecomment-662126762, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQJTJ3Z33ZRLSPZQRJDM5KTR4YFGVANCNFSM4O3TKFNA.

Hi!
I have the same issue. I downloaded the installation script today, and there's maybe something wrong with SPADES. The file names in the Bowtie folder do have the sample name (and they look ok), but SPADES is not recognizing them in the next step.
This is the _Progress_log:
Mon Aug 10 15:15:09 2020        Starting Fast-Plast v.1.2.8.
Assemblying plastome with 0 single end libraries and 1 paired end libraries.
Mon Aug 10 15:15:09 2020        Determining best kmer sizes.
K-mer sizes for SPAdes set at 55,87,121.
Mon Aug 10 15:15:09 2020        Starting read trimming with Trimmomatic.
Using /scratch/.../Fast-Plast/bin/Trimmomatic-0.39//trimmomatic-0.39.jar.
Mon Aug 10 15:15:23 2020        Starting read mapping with bowtie2.
Using /scratch/.../Fast-Plast/bin/bowtie2-<IP_ADDRESS>-linux-x86_64/bowtie2.
Samples for Asparagales used to make bowtie2 indices.
ERROR: No trimmed read files were identified to run SPAdes.  Please check 1_Trimmed_Reads.
For clarity, these are the files in 1_Trimmed_Reads:
subset_file1.fq      subset_file2.fq
And these are the files in 2_Bowtie_Mapping:
Asparagales.fsa     V_lillacina_W3730_bowtie.2.bt2       V_lillacina_W3730_bowtie.4.bt2      V_lillacina_W3730_bowtie.rev.2.bt2
V_lillacina_W3730_bowtie.1.bt2        V_lillacina_W3730_bowtie.3.bt2        V_lillacina_W3730_bowtie.rev.1.bt2
Any clues of what to do?
Thank you.
Janice

Hi Janice,
Are you running this on a linux machine? The installation is for linux only.
There should be trimmed files in the 1_Trimmed_Reads directory. Since there isn't, that suggests Trimmomatic isn't running properly. Do you have java installed?
Can you give me the log, out, and error files?
Best,
Michael

Hi Michael
I'm running it in a Linux machine, a HPC.  I don't have java installed there... I have a conda with java and Trimmomatic in my HPC account, so I can install all other FastPlast dependencies there... I'm just not quite sure how to connect the Trimmomatic with the java in the pipeline... The other option is to provide the trimmed files (from the Trimmomatic in conda) to the pipeline and start in the Bowtie step... but I'm not quite sure how to do it either. Any advice?
I'm attaching the log files, JIC
Thank you
Janice
Vanda_lillacina_W3730_Fast-Plast_Progress.log
Vanda_lillacina_W3730_results_error.log
Vanda_lillacina_W3730_results_out.log

Prior to running Fast-Plast, load conda with Trimmomatic (assuming it is a module). You can go to the fast-plast.pl script and change line 25 to give the path to wherever Trimmomatic is installed on your HPC.
To skip trimming, provide the trimmed R1 and R2 files to -1 and -2. Add the --skip trim flag when you run Fast-Plast.

Thank you, Michael.
The second option sounds easier for me :)
j.


TabBar cannot work.

015-10-28 21:59:30.439 [error][tid:com.facebook.React.RNVectorIconsManagerQueue][RCTBatchedBridge.m:815] Exception thrown while invoking getImageForFont on target RNVectorIconsManager with params (
    Ionicons,
    "\Uf448",
    30,
<PHONE_NUMBER>,
    11
): *** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]

What version of RN are you using?

It's OK. My setting is wrong.

@yangco I get the same error, what was the problem for you ?

@JeremDsgn: Did you follow all installation steps? Is the example working?

I did install a custom font and it works I can used it by <Icon> but it didn't works with Icon.TabBarItem. I also follow those installation steps http://facebook.github.io/react-native/docs/linking-libraries-ios.html#content.

And is the TabBarExample working? What's your tabbar code looking like?

I basically just make something like :
import React from 'react-native';

let {
  View,
  Text,
  TabBarIOS,
} = React;
var Icon = require('react-native-vector-icons/Ionicons');

var TabBarView = React.createClass({
  render: function() {
    return (
      <TabBarIOS>
        <Icon.TabBarItem
          title="Home"
          iconName="ios-home-outline"
          selectedIconName="ios-home"
          }}>
          <View style={styles.tabContent}><Text>Home Tab</Text></View>
        </Icon.TabBarItem>
      </TabBarIOS>
    );
  }
};

You have a syntax error in your example: }}, but I assume it's an artifact from cleaning up your code.

Check that you added your custom font to your project (not in a folder, but in a group).
Check that it's referred to in your Info.plist.
Try without the selectedIconName prop.
Try to pass a second fontFamily argument when creating the icon.

Can you help?
Attempting to run the tabbarexample and followed manual setup instructions.  React Native V32.
2016-09-05 15:41:11.917 [fatal][tid:com.facebook.react.RNVectorIconsManagerQueue] Exception '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]' was thrown while invoking getImageForFont on target RNVectorIconsManager with params (
Ionicons,
"\Uf4a7",
30,
4278190233,
17
)

@JoeArnone i had your same error and fix it following step 2 of this manual http://codewithchris.com/common-mistakes-with-adding-custom-fonts-to-your-ios-app/

@joseph-adam any idea on how to solve this? it's throwing it now for me after updating react & react-native.

@msqar I can't remember what the solution was looking back.  I posted this question back in 2016 when the react native framework was in a lot of upheaval.  I wish I could help you.

@joseph-adam thanks anyway man :) already solved it!

@msqar I am experiencing a similar issue. How did you solve it?

I had the same issue and spent half of the day solving it.
Drag the folder Fonts from node_modules/react-native-icons to your project in Xcode. Make that "Create groups" is checked and that every font file is added to targets (if you click on the file you can see Target Membership on the right side, make sure it is checked).


[Ameba] Update docker image

Rework DCT to fix bugs
Move __dso_handle from Matter sdk to Ameba SDK

Hi @andy31415 @electrocucaracha
We have committed another PR #24273 which update the docker image to the latest one.
The CI error should able be fixed after PR #24273 merge.

@pankore #24273  is now merged.
For correctness however, this PR should not contain the change in third_party/ameba/mbedtls.cmake

I did a conflict resolution ... now only change is the mode to not be executable which seems correct.

Rebase repo and update docker version to the latest.


Video Timeline: video with no audio

Want to annotate a video which doesn’t have audio. When we open the task a pop-up error appears with below description .
Error while loading audio. Check $videoSource field in task.
Technical description: Error decoding audiobuffer

The config we are using is...

Is there any way to remove the pop-up error.

@NitinKhandelwal1947 It's not possible to do segmentation without audio, because this type of labeling is based on the audiowave. The fastest way to fix it is to convert your videos and include silence inside:
ffmpeg -f lavfi -i anullsrc=channel_layout=stereo:sample_rate=44100 -i video.mov -c:v copy -c:a aac -shortest output.mov

btw: we are releasing video labeling soon, so there you will be able to this without audio track.

btw: we are releasing video labeling soon, so there you will be able to this without audio track.

Hi, @makseq has this been merged? I can't find any documentation regarding this
I'm looking to use Label Studio to annotate actions in a video. For example:
filename  label(Action)  start_frame  end_frame
vid1.mp4              1                   0           100
vid1.mp4              2                 100         150
vid2.mp4              3                   0           50

Hi, please check this post: https://labelstud.io/blog/label-studio-1-6-video-object-tracking-and-new-annotation-ui/


Reset disables the ability to pass through C1 control characters

Windows Terminal version
1.15.1862.0
Windows build number
10.0.19044.1889
Other Software
No response
Steps to reproduce

Open a WSL bash shell in Windows Terminal.
Execute the following statement: printf "\e 7\u009B?5h"
Note that the screen attributes have reversed.
Execute reset to get the screen back to normal.
Execute printf "\e 7\u009B?5h" again.

Expected Behavior
Both step 2 and step 5 should cause the screen attributes to be reversed.
Actual Behavior
Only the first attempt works. The second attempt appears to output ?5h instead.
What's happening is we've got a DECAC1 sequence (ESC SP 7) which is supposed to enable C1 support, followed by DECSCNM (reverse screen mode), using a C1 control sequence (CSI ? 5 h).
That works the first time, but after the reset, the C1 support is permanently disabled on the conpty client side, so C1 control sequences can no longer be passed through.

I think this may be another regression from #13024.
When we first added support for DECAC1 in #11690, the assumption was that it would be enabled permanently on the conpty client side, and there was nothing in the terminal code that could turn that off. However, when TerminalDispatch was replaced with AdaptDispatch, it inherited the conhost SoftReset implementation, and that included turning off DECAC1.
I haven't tried this yet, but one simple fix could be to add another parser mode - something like AlwaysAcceptC1 - which is set at startup in the terminal, and never reset anywhere. Then the test in the state machine just becomes:
_parserMode.any(Mode::AcceptC1, Mode::AlwaysAcceptC1)

It's a bit hacky, but that should add no significant overhead, and we won't then have to care what AcceptC1 might be set to.

Ah, this is a weird case. Great catch! Eventually we would want to turn that new "always" flag off by default, because there may not always be a guarantee that Terminal is connected to ConPTY.
I'm cool with this fix, if we file a followup/todo somewhere.
Quick question: does this mean a hard reset breaks Terminal in general? Should I crack the seal on the coming 1.15 and 1.16 releases to slip in a fix for this? 🙂

Eventually we would want to turn that new "always" flag off by default, because there may not always be a guarantee that Terminal is connected to ConPTY.

Yeah. We've actually already got a comment in the code at that point.
https://github.com/microsoft/terminal/blob/d11ea7263916c52106c91e57bf44398430664b9a/src/cascadia/TerminalCore/Terminal.cpp#L88-L89

Quick question: does this mean a hard reset breaks Terminal in general?

Only if someone is actually using C1 controls, which requires actively enabling them with DECAC1. So I don't it's that big a deal, but I do know at least one application that is doing that (see https://github.com/microsoft/DbgShell/commit/26a40a1a457eb25be55ad9a29e19bdc1ee06d5d4).


remove blacklist option

according to https://github.com/mehcode/atom-project-plus/issues/29

Great work on the project @shemerey -- you're now a contributor. I look forward to working more with you on this.

@mehcode I glad to help with really nice project :)


Merge masterthesis into main

Merge first files from dev branch of Alfouly's master thesis into main branch

weather data: changed directory / file names


Atoms

I think we should take a step back and first outline all the atoms on the design. From there we can start creating the actual atoms in pattern lab. So the first task is to break apart the design. Then the other changes that we will be making are in two different sections, the "scss" folder and the "_patterns" folder. all other areas are just helper areas that we may or may not touch. I am going to close this PR and we will have start again on the atoms of the site.


Feature/refactor tests

see closed https://github.com/blainesch/alfred-chrome-bookmarks/pull/13

@afaur this does work on 5.6, which is great. It's not backwards compatible though. Theoretically this just needs to work with the php system default in macs. Yosemite ships with 5.5, and I'll guess Captain has 5.6 since you used that. I think Mavericks would be the lowest we might need to support, and that's 5.4. Thoughts?

@blainesch Use 5.6 but have a branch that supports the older versions where features get added after they are implemented in 5.6?  Find out if we get issues regarding implementing the newer features in the older php version branch?

@afaur that's a little bit too bleeding edge. To put it in perspective, everybody at my office is still using Yosemite and because we don't use PHP nobody is going to upgrade for a plugin. Also, you want to upgrade to 5.6 to use 1 feature that is only included in your tests.

@blainesch didn't mean it for this PR.  I was meaning for future work.  If they don't want the latest features there will be a build release for them.

@blainesch also doesn't mean we don't get features to them, just means they don't get the new features when they first ship.

So maintain a legacy app that lives on like a 1.* versioning system and a edge system that lives on a 2.* system?

@blainesch That's the idea, but its ultimately up to you.

@blainesch I removed the splat and used the func_get_args instead.  Probably better to just keep it this way and avoid complexity.


Release v0.10.1

Version bump

Coverage remained the same at 86.116% when pulling bc85b67a97210581aa4ed424a9f50dbe6c39d7a2 on fenekku:release_v0.10.1 into 958570ed1c275992656284dfc5613859a12ad8b4 on inveniosoftware:master.


Connecting to non-neptune TinkerPop db

Trying to connect to a TinkerPop database, I get the error "Synchronization Required
It is necessary to synchronize the connection to be able to work with the database."
Looking into network calls, I see it tries to query g.V().groupCount().by(label)
which is refused with the error message:
Could not find a suitable index to answer graph query and graph scans are disabled
Seems like an index on label is required to run graph-explorer

What TinkerPop database are you running against?

JanusGraph
It has a specific setting to disable queries that do not use an index

Are you able to enable full graph scans for the near term? The GraphExplorer does not try to interrogate the back end currently to see what indexes exist. Also, as far as I am aware, JanusGraph does not support indexing a label (or at least it did not used to when I used it more regularly).

The same error happened to me as well when connecting to a gremlin server.

@otnt Which database backend were you using, was it also JanusGraph? Are you still seeing the issue with the latest version of Graph Explorer? @yuvallb were you ever able to get it working with JanusGraph? Just checking for any more information before taking a deeper look at this one. Several fixes and improvements have been made in the meantime that may also help with this issue.

I was able to reproduce the issue pretty much as described. Using:

JanusGraph 0.6.3 running with a Gremlin Server using the WsAndHttpsChannelizer and an in-memory graph. Everything else pretty much defaults.
Graph Explorer built to use HTTP rather than HTTPS (just to make connecting easier)
The schema of my small test graph was correctly fetched
The minute I clicked in the search bar of the Explorer an exception was thrown and the browser window went to just a white background with nothing else.

Issue #55 is potentially related to this issue. It may not be the root cause but given the ID scheme used by JanusGraph the fix for that issue will also be required before JanusGraph fully works with GraphExplorer.

I built a version of Graph Explorer using the current PR #60 and was able to get data to display using Janus Graph. Once PR #60 merges, this issue can hopefully be resolved/closed.

Looks like issue has been resolved. please re-open if it's not the case.


Issue in ObstacleController.cpp for counting tags

Issue inside setTagData where a double for loop creates a tag^n problem.  Does not impact current code but will impact anybody attempting to count total tags seen inside this controller.

Erroneously counts cubes as well as collection zone tags.

@wfvining Requesting update


Update JXL to v0.7.0 (Failing build)

Version is up to 0.10.2 ATT
https://github.com/libjxl/libjxl/releases


ANDROID: Add syncfs API in liblibc

This is required to sync everything in a single filesystem. Other solutions like sync() flushes all filesystems which is unnecessary, it is also impractical to call fsync on all files of the filesystem.

Thanks! @bors r+

:pushpin: Commit c04938525ba0395013ea37939c552e8ae5b28cf7 has been approved by JohnTitor
It is now in the queue for this repository.

Hi, what are the next steps to get this merged and released? It looks like it's been in the queue for a while now with no change.

Seems bors didn't anything to merge, @bors retry

@bors r+

:pushpin: Commit c04938525ba0395013ea37939c552e8ae5b28cf7 has been approved by JohnTitor
It is now in the queue for this repository.

:broken_heart: Test failed - checks-actions

The test Docker Linux Tier2 (arm-linux-androideabi) fails with error:
CANNOT LINK EXECUTABLE "/data/local/tmp/main-d701c417e8b08db6": cannot locate symbol "syncfs" referenced by "/data/local/tmp/main-d701c417e8b08db6"...
Any ideas where else I need to add the symbol or what tests to modify?

syncfs is added in API 28 but our ARM targets on CI use API 24: https://github.com/rust-lang/libc/blob/216a428bfe9129472e0a88aac8dc331beffc8882/ci/android-install-sdk.sh#L18-L24
Let's ignore on ARM by

adding an arm var like let arm = target.contains("arm") || target.contains("aarch64");:
https://github.com/rust-lang/libc/blob/216a428bfe9129472e0a88aac8dc331beffc8882/libc-test/build.rs#L1510
adding a new arm like the below here:
https://github.com/rust-lang/libc/blob/216a428bfe9129472e0a88aac8dc331beffc8882/libc-test/build.rs#L1779-L1815

// Added in API level 28:
"syncfs" if arm => true

Thanks @JohnTitor ! I made the changes and all checks now pass (:
Now, presumably, another @bors r+ would be required.

@shikhapanwar: :key: Insufficient privileges: Not in reviewers

Thanks! Let's check again, @bors r+

:pushpin: Commit fde1ddf9e7904b5ada40723c90717d822eebe1fa has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit fde1ddf9e7904b5ada40723c90717d822eebe1fa with merge ad916458e7f948a685d22492df1f5b4564971aba...

:broken_heart: Test failed - checks-actions

 CANNOT LINK EXECUTABLE "/checkout/target/x86_64-linux-android/debug/deps/main-d677790ea659c418": cannot locate symbol "syncfs" referenced by "/checkout/target/x86_64-linux-android/debug/deps/main-d677790ea659c418"...
libc: CANNOT LINK EXECUTABLE "/checkout/target/x86_64-linux-android/debug/deps/main-d677790ea659c418": cannot locate symbol "syncfs" referenced by "/checkout/target/x86_64-linux-android/debug/deps/main-d677790ea659c418"...

Uhm, it's unexpected but something on x86_64 emulator is old, I guess. Let's ignore the test unconditionally.

Done. Thanks. Will require attempt at @bors r+ I presume

@shikhapanwar: :key: Insufficient privileges: Not in reviewers

@bors r+

:pushpin: Commit ca8f0726644d786a9addfb241af4a8a9af3f35d5 has been approved by JohnTitor
It is now in the queue for this repository.

:broken_heart: Test failed - checks-cirrus-freebsd-14

I cannot possibly explain where ld-elf.so.1: /usr/local/lib/libcurl.so.4: Undefined symbol "nghttp2_option_set_no_rfc9113_leading_and_trailing_ws_validation"  is coming from for nightly x86_64-unknown-freebsd-14. Have you seen it before?

That's unrelated to this PR and I've fixed the issue on master, let's retry
@bors retry

:hourglass: Testing commit ca8f0726644d786a9addfb241af4a8a9af3f35d5 with merge 106b57a796145c42f9a385a2a8cf764abbc66624...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 106b57a796145c42f9a385a2a8cf764abbc66624 to master...


As a prototyper, I want to be able to use Prefer header to choose one of the examples in a callback

User Story Description

As a prototyper,
I want to be able to write multiple examples in a callback,
so I can use Prefer headers with prism to trigger specific content in a callback.
Acceptance Criteria

[ ]

Sprint Ready Checklist

[ ] Acceptance criteria defined
[ ] Team understands acceptance criteria
[ ] Team has defined solution / steps to satisfy acceptance criteria
[ ] Acceptance criteria is verifiable / testable
[ ] External / 3rd Party dependencies identified

Hey @jayvdb is this what you are looking for? https://docs.stoplight.io/docs/prism/beeaad4dc0227-prism-cli#request-specific-examples

@daniel-white, I believe that functionality does not select examples from within callbacks.  It seems to only work within responses.


feat: statsperform parser internal format

Expand the StatsPerform parser with functions to read the internal format for position and event data.

Hi @hbiermann95,
thanks for the contribution, I think this will be a good addition with some changes. Some initial thoughts:

I don't know how the statsperform Web API looks like, but it feels to me that API access and parsing should not be dealt with in the same module. Especially as we do rely on the former in many cases, it's probably better to have some basic but dedicated functionality for working with URLs, temporary file saves etc. @tkgroot is currently building something in this fashion to load the eigd-dataset. Maybe this is something that could do the trick for this parser, as well.
Is closed really the optimal format name? I'd think it's quite open for anybody having access, so I'd reconsider this choice
As the file is getting quite long, maybe a refactor into multiple files would make the code more accessible. You could use some private modules with leading underscores to outsource functionality where it makes sense, and maybe also break it down further into more functions? At this point, it's really hard to review as I can't really see which part is necessary for parsing which format.


Method to disable end turn, ko Buttons on Heroes

For campaign stages.

Disabled when UI items are present.


Document the convertToJson function

Turns out there is a convertToJson function that converts a yaml object to a json string
parameters:
  - name: complex
    type: object
    default:
      this_is:
        a_complex: object
        with:
          - complex
          - properties
stages:
  - stage: stage1
    jobs:
      - job: job1
        steps:
          - pwsh: |
              Write-Host "${{ convertToJson(parameters.complex) }}"

This should be documented (along with any other hidden functions)

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

ID: 77c58a78-a567-e99a-9eb7-62dddd1b90b6
Version Independent ID: 680a79bc-11de-39fc-43e3-e07dc762db18
Content: Expressions - Azure Pipelines
Content Source: docs/pipelines/process/expressions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks for flagging this! From the code, it looks like you can use convertToJson with parameters in your YAML file but not a template. I'm reaching out to the product team for more info.

I'm working on adding convertToJson to the docs now.


Implement authentication for callout executor

Allow different authentication types. Set the Authorization header according authentication type and authentication value loaded from callout definition parameters

Done


Support migration between tunnel mode and direct routing mode

Ask from Cilium Slack:

Any thoughts on how to do a migration from vxlan tunneling to direct L2 routing without segmenting during the migration.  I have a demo Kubernetes cluster up with half the nodes on the old config (tunnel=vxlan), and the other half on with the new config (auto-direct-node-routes=true,tunnel=disabled).  Packets from the new nodes to the old are routed and received by containers on the old nodes, but replies come back vxlan encapsulated and subsequently dropped by the new nodes which don’t have a vxlan interface.
I think this could work if a) I could setup a receive only vxlan interface on the new nodes, or b) I could convince the old nodes to send unencapsulated packets to new nodes.

I propose that this could be achieved through a two-phase process:

Initially all nodes are configured in tunnel mode
(new mode) Configure all nodes with tunnel receive but will use direct routing to send.
Remove tunnel configuration

At the core the new mode would be very similar to tunnel mode (for device configuration purposes etc.), but the difference would be that when learning about remote nodes, the tunnel map entries would not be pushed into the cilium_tunnel_map.
In this configuration, the tunnels would be configured and as nodes are migrated to the intermediate new mode, they will transition from sending traffic over the tunnel to sending traffic directly. The entire time, the receive side should remain configured to allow nodes that have not yet upgraded to continue to transmit traffic via the tunnel, and any arbitrary node will be able to receive that tunneled traffic.
Once the entire cluster is migrated to the new mode, no traffic should be transmitted via the tunnels any more. After rolling the whole cluster to this mode, then tunnel could be entirely disabled. (edited)
I expect that given the new mode, the reverse migration could also be achieved (DR -> tunnel).
Tasks:

[ ] Add a new mode (daemon flag) for tunnelling which configures the daemon to only establish tunnels for receiving packets, not transmitting

[ ] When in this mode, skip tunnel map updates

[ ] Add a transition guide to the documentation
[ ] Add CI tests to validate the transition path

I've been thinking about how to support something like this as well. Are you suggesting adding a third mode so that we have tunnel, direct, and "hybrid" mode (I don't have a better term)? That's my understanding of the proposal. I assume the idea is to minimize packet loss across the cluster when switching modes.
I'm brainstorming here, so bear with me. I'd love to explore this in more detail, I see toggling from tunnel to DR mode as something people will want to do. Strategies that eliminate (or at least mitigate) packet loss during this transition are needed IMO. The strategy you're describing seems to assume that cilium is working with a route table that is populated rather quickly so that pods sending traffic have it routed to the right destination. Convergence time with something like BGP isn't always fast, so unless cilium has hooks into a routing daemon you might still end up with an interim period of packet loss as your routing protocol converges. At that point, you might as well bite the bullet and just jump straight from tunnel->DR modes and accept a transient outage.
It may come across as a bit of a tangent, but hear me out. If cilium supported a native routing daemon instead relying on something out-of-band like kube-router, you would have a place for hooks that could be used for toggling when to tunnel a packet on egress. In addition, that has the benefit of ensuring a route has been received for the destination which will keep packets from either black-holing or going to the default route (which you may not want) while your routing protocol converges.
Again, I'm just brainstorming here. I'm thinking there may be a way to support this migration without a third mode of operation if you had hooks into a routing daemon. Thoughts?

I assume the idea is to minimize packet loss across the cluster when switching modes.

That was the idea behind this issue, yeah.

I see toggling from tunnel to DR mode as something people will want to do.

I'm curious if you have more details on this. To my knowledge, most users will already have a cluster topology in place, be it on-prem or via a managed service which provides strong direction as to which of the modes they will select. There can be a subsequent architecture decision that could drive desire to switch (potentially implying need for migration path), do you have another case in mind for such a "hybrid" mode?

If cilium supported a native routing daemon instead relying on something out-of-band like kube-router, you would have a place for hooks that could be used for toggling when to tunnel a packet on egress.

I agree that to seamlessly transition, some co-ordination between Cilium and the routing agent needs to be performed. Ie Cilium would need to be react to routing being ready, to trigger the switch of the mode. There's multiple ways to implement this, whether by extending the Cilium API so that external routing daemons can call into Cilium; whether you have Cilium subscribe to some external notifications to reach; Cilium running some routing functionality internally; or driven by the user where they deploy the routing daemon side, then after convergence, trigger the change of mode in Cilium.

This issue is not particularly difficult to address but still a bit too complex for a good-first-issue IMO.

We currently run kube-router + kube-proxy in one of our main k8s clusters, all of our other clusters are running cilium 1.15 in direct routing / no tunnel mode - this is working nicely for us, we now wish to migrate the remaining main cluster over but we can't suffer downtime like the other clusters did when we switched from kube-router/proxy to cilium.
I have been running through the migration guide which works ok when using tunnel mode but when I change the migration config to use direct routing mode, connectivity begins to fall apart in strange (probably expected) ways.
So rather than migrating straight from kube-router/proxy to cilium in DR mode, I was hoping to first migrate to cilium in tunnel mode, then over to cilium DR mode - when I noticed this github issue I wondered if there had been any movement on a 'hybrid' solution? Otherwise it seems like I have to either live with cilium in tunnel mode or do a full stop migration with downtime, neither of which are desirable...

Hi!
We have a use case  (https://github.com/cilium/cilium/issues/32810) when the ControlPlane (CP) and Worker (WK) nodes are on different L3 subnets but the underlying network infrastructure does not support routing between them. In this case, the encapsulated tunnel is mandatory for CP-to-WK connections but the CP-to-CP and WK-to-WK can use the native routing for better performance. So, a "hybrid-routing" mode will help to resolve this.


[Feature Request] set time/date for giveaways

Title kind of says it all, I would like to be able to set a time/date for giveaways instead of setting a timer for them.

If i understand correctly, you want to set a date/time when the giveaway automatically gets started right?
For example:
if its a time, then it gets started everyday on that time
if its a date, it gets executed on that date

No I'm looking for an set end time

Forgot about this, but this has been added in the following commit: d2360050ed38ee97dfa2512e0a906f29282761c5


Problem about the released checkpoint

Hi, Thanks for your excellent work!
I want to ask which datasets are the released checkpoints pretrained on, 4m dataset or 4 + 10m dataset? Since there are two pretrained dataset in your paper.
Thanks!

Hi, the checkpoints are pretrained on 14M dataset. Thanks!

Thanks!


Use the Drive folder ID for storing puzzle files instead of the same directory as the template

Fixes #541
This lets us keep the template in a different location from the puzzle files, which is useful
if we're potentially going to make a bunch of copies of the template in advance and need to
distinguish those from puzzle sheets.

Thanks for fixing this.
Just curious, if the template happens to be in the same folder as the Drive folder, does the code still work (no error returned by the Google APIs)?

Just curious, if the template happens to be in the same folder as the Drive folder, does the code still work (no error returned by the Google APIs)?

Yep, still works. This task is also async from the sheet creation one so even if something does go wrong for some reason, it won't interfere with sheet creation.

Thanks for confirming!


Add comment toggling for RPGLE free and CL

Add comment toggling to RPGLE free and CL. I doubt I can toggle an asterisk at column 7, but I'll try to look
https://code.visualstudio.com/api/language-extensions/language-configuration-guide#comment-toggling

@worksofliam added RPGLE, CL, and MI quick comments in release v0.5.6 . Unfortunately, something fancier will have to be used if any fixed-format code needs this feature


Fix setStyle method for Circle class

The Circle class inherits from CircleMarker class, which has a setStyle method which sets the radius. There was no equivalent in Circle class (the setStyle method was the one from the Path class), so we couldn't set the Circle radius with the setStyle method

Why is this needed, when there is already a setRadius() method?

I'm currently working on https://github.com/jupyter-widgets/ipyleaflet, we use python models on the backend for defining Leaflet widgets on the frontend. Every time the model changes on the backend, we update the frontend with a simple setStyle call passing the options defined in the model. This works fine for each attribute of the Circle class (those defined in the Path class: color, opacity, fillColor...), except for the radius.

setStyle appears to take the options dictionary and radius is listed in the options for the circle marker.

BTW, this could be a much cleaner change if you just replace setStyle: Path.prototype.setStyle, with setStyle: CircleMarker.prototype.setStyle,.

On second thought, it seems that the behaviour of setStyle is an oversight dating back to https://github.com/Leaflet/Leaflet/commit/508a75f7a836a0a5bd1e9472cb09434183046860. I'm partial to accepting this, but I'll wait for other maintainers to chip in.
Maybe we need to add some docstrings to make it clearer that setStyle on both Circle and CircleMarker do set the radius too.

I don't think this is an oversight. Radius not being a part of Circle "style" is intentional, because unlike CircleMarker, it'a physical property of the circle, just like geographical location. It's not a visual thing.

@mourner
Should we close this?


Cannot add text to projection axes

Currently, the text wrapper for proplot cannot handle projecting to geoaxes, making it impossible to accurately place text on a map projection. For matplotlib one just passes a transform=ccrs.PlateCarree() for example to the ax.text() to interpret the values as coordinates on the map:
import matplotlib.pyplot as plt
import cartopy.crs as ccrs
import cartopy.feature as cfeature

f, ax = plt.subplots(subplot_kw=dict(projection=ccrs.SouthPolarStereo(central_longitude=-70)))
ax.set_extent([-180, 180, -90, -40], ccrs.PlateCarree())
ax.add_feature(cfeature.LAND, color='#d3d3d3')

ax.text(120, -83, 'test', transform=ccrs.PlateCarree())

proplot breaks when one tries to place ccrs.PlateCarree() into the transform keyword for the text wrapper. And the options of data, axes, figure don't fix this problem. Perhaps an extra keyword such as geoaxes could handle this?
import proplot as plot

f, ax = plot.subplots(proj='spstere', proj_kw={'central_longitude': -70})

ax.text(120, -83, 'test', color='w', fontname='Helvetica')

ax.format(land=True, boundinglat=-40)

Thanks for the report, this was a really simple fix: 29b7d6df. For cartopy axes, text(..., transform='data') (the default behavior) now corresponds to longitude-latitude coordinates. You can use transform='map' or transform=ax.transData to use map projection coordinates.
Tried to also fix this for basemap, but apparently the Basemap class doesn't support positioning text in longitude-latitude coordiantes.


added raise_on_error option to load_maf and load_maf_dataframe

Fixes https://github.com/hammerlab/varcode/issues/210 for @tuomastik

This change is

Coverage decreased (-0.9%) to 87.294% when pulling 67e72dd9aef3651373bd1ebf96a2be0fb5a4f709 on load-maf-raise-on-error into 45575fb71efe2d18efd21b28639670d806e8ea4f on master.

Coverage decreased (-0.9%) to 87.294% when pulling 6437ebc9be5eb334e61f59ecf0b853313a7220c5 on load-maf-raise-on-error into 45575fb71efe2d18efd21b28639670d806e8ea4f on master.

@tavinathanson The trouble with adding it to load_vcf is that the code there is significantly more complicated with many more opportunities for errors. I think I'd rather hold off on that until there's a user request.


fasole: init at 1.2.3

Description of changes
Add fasole - minimalist's todo-list with md file format
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

Your commits are breaking Commit Conventions please rebase.
In your case reword 2cc2f6cd6dd72451d53c00b35e6ec1584bda9ddd to maintainers: add proggerx and squash dee9d9cd47314141814d38b36b0c7fa1c0e7f202 into 71ab48f123d0053986281027b809b004a2bcbaad

@DontEatOreo is it better now?

Please change title to fasole: init at 1.2.3

oh f***

wait, i'll fix it

@DontEatOreo @AndersonTorres Is it correct now?

Result of nixpkgs-review pr 309366 run on aarch64-darwin 1

1 package built:

fasole

Result of nixpkgs-review pr 309366 run on aarch64-darwin 1

1 package built:

fasole

Who can merge it?

yeeey


parent collective id is not related to the host

when wwcode creates a new collective, this collective is not added to the correct supercollective (id51) which is not their host, but the wwcode collective.
We need to do this manually every time.

We should not do that anymore, closing.


differentDomains (true) conflict with Nuxt routeRules (ssr: false)

Environment
Nuxi 3.1.2

Operating System: Linux
Node Version:     v18.16.0
Nuxt Version:     3.1.2
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, routeRules, app, css, plugins, build, modules, components, postcss, content
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Add config routeRules to nuxt.config.ts:
routeRules: {
    '/auth/**': { ssr: false },
},

Config i18n module to have differentDomains = true in nuxt.config.ts:
modules: [
    [
      '@nuxtjs/i18n',
      {
        strategy: 'no_prefix',
        defaultLocale: 'en',
        fallbackLocale: 'en',
        langDir: './locales',
        differentDomains: true,
        locales: [
          { code: 'en', iso: 'en-US', file: 'en.yml', domain: 'http://localhost:8800' },
          { code: 'vi', iso: 'vi-VN', file: 'vi.yml', domain: 'http://localhost:8801' },
        ],
      }
    ]
],

Then go to any page that match routeRules like /auth/login to see it keep infinite refresh loop
Describe the bug
Browser refresh loop when going to route match routeRules (no SSR) with config differentDomains = true
Additional context
No response
Logs
No response

Thank you for your reporting!
I've backed from shot vacation.
Unfortunately, We could not your issue.
Could you give your minimal reproduction us with stackbliz (nuxt starter) or github repo please? 🙏

Thank you for your reporting! I've backed from shot vacation.
Unfortunately, We could not your issue. Could you give your minimal reproduction us with stackbliz (nuxt starter) or github repo please? 🙏

Here is it: https://stackblitz.com/edit/github-fjfcqz
Go to "infinite loop refresh page" in the demo to see the problem

Thank you for your reproduction!
I've faced the warning output in my env
 WARN  [@nuxtjs/i18n]: The differentDomains option and no_prefix strategy are not compatible. Change strategy or disable differentDomains option.

no_prefix strategy isn't supported with differentDomains: treu
you need to change other strategy.

Thank you for your reproduction!
I've faced the warning output in my env
 WARN  [@nuxtjs/i18n]: The differentDomains option and no_prefix strategy are not compatible. Change strategy or disable differentDomains option.

no_prefix strategy isn't supported with differentDomains: treu you need to change other strategy.

Thank you for clear that to me

Hello @kazupon,
We have the same issue, event if we use another strategy than 'no_prefix'.
Here is a link to the stackblitz to reproduce :
https://stackblitz.com/edit/nuxt-starter-iceuqr?file=nuxt.config.ts


/usr/bin/ld: cannot find -lcuda

Hi all,
I don't know if it is the issue but I don't have root access so I install CUDA in my home directory. Then I added those paths to PATH and LD_LIBRARY_PATH:
export PATH="/my_local_path/cuda/bin:$PATH"
export LD_LIBRARY_PATH="/my_local_path/cuda/lib64:/my_local_path/cuda/lib:$LD_LIBRARY_PATH"
export CFLAGS=-I/my_local_path/cudnn/include
export LDFLAGS=-L/my_local_path/cudnn/lib64

I tried to install chainer from source and by pip install chainer --no-cache-dir -vvvv but they all failed to build with cuda:
Bellow are logs when I built chainer from source. Pip install outputs similar logs.
Options: {'profile': False, 'annotate': False, 'linetrace': False, 'no_cuda': False}
Include directories: ['/users/ud2017/hoavt/cuda/include']
Library directories: ['/users/ud2017/hoavt/cuda/lib64', '/users/ud2017/hoavt/cuda/lib']
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
/usr/bin/ld: cannot find -lcuda
collect2: error: ld returned 1 exit status
Cannot build a stub file.
Original error: command 'c++' failed with exit status 1
**************************************************
*** WARNING: Cannot link libraries: ['cublas', 'cuda', 'cudart', 'curand', 'nvToolsExt']
*** WARNING: Skip installing cuda support
*** WARNING: Check your LDFLAGS environment variable
**************************************************
Include directories: ['/users/ud2017/hoavt/cuda/include']
Library directories: ['/users/ud2017/hoavt/cuda/lib64', '/users/ud2017/hoavt/cuda/lib']
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
/tmp/tmpHcwk2p/a.cpp:1:19: fatal error: cudnn.h: No such file or directory
 #include <cudnn.h>
                   ^
compilation terminated.
command 'x86_64-linux-gnu-gcc' failed with exit status 1
**************************************************
*** WARNING: Include files not found: ['cudnn.h']
*** WARNING: Skip installing cudnn support
*** WARNING: Check your CFLAGS environment variable
**************************************************
Include directories: ['/users/ud2017/hoavt/cuda/include']
Library directories: ['/users/ud2017/hoavt/cuda/lib64', '/users/ud2017/hoavt/cuda/lib']
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
/tmp/tmpAvadWz/a.cpp:1:18: fatal error: nccl.h: No such file or directory
 #include <nccl.h>
                  ^
compilation terminated.
command 'x86_64-linux-gnu-gcc' failed with exit status 1
**************************************************
*** WARNING: Include files not found: ['nccl.h']
*** WARNING: Skip installing nccl support
*** WARNING: Check your CFLAGS environment variable
**************************************************
Include directories: ['/users/ud2017/hoavt/cuda/include']
Library directories: ['/users/ud2017/hoavt/cuda/lib64', '/users/ud2017/hoavt/cuda/lib']
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
running instal

It took me two days to fix this issue without any success. I would really appreciate any suggestion.
I want to use chainer for my project but without cuda support, it is quite slow.
OS: Ubuntu 14.04
gcc 4.8.4
python 2.7.6
Cuda compilation tools, release 7.0, V7.0.27
cudnn:

define CUDNN_MAJOR      4
define CUDNN_MINOR      0
define CUDNN_PATCHLEVEL 7
define CUDNN_VERSION    (CUDNN_MAJOR * 1000 + CUDNN_MINOR * 100 + CUDNN_PATCHLEVEL)

Did you set FLAGS environment variables?
See : https://github.com/pfnet/chainer#installation-with-cuda

You can ignore nccl.h not found error if you're going to use only one gpu.

Thank you @Paosder yes I did. I updated my first comment.

I also tried Cuda compilation tools, release 7.0, V7.0.27 and it output the same errors.

@hoavt-54, did you put -I or -L characters in front of your environment variables?
(LD_LIBRARY_PATH, CFLAGS...)
i.g.
export CFLAGS=-I/Developer/NVIDIA/CUDA-8.0/include
export LDFLAGS=-L/Developer/NVIDIA/CUDA-8.0/lib

And you should also check that your cudnn path has -L characters(for lib directory).
LD_LIBRARY_PATH=-L/Developer/NVIDIA/CUDA-8.0/lib:$LD_LIBRARY_PATH
In my case, in MacOS, I put these characters so that I could fixed these link errors.

Hi @Paosder Thank you. I did put those characters in the flags, also tried to put in the LD_LIBRARY_PATH but it doesn't work.
The thing is that the installer already see my cuda( as you can see from the logs):
Include directories: ['/opt/cuda/include']
Library directories: ['/opt/cuda/lib64', '/opt/cuda/lib']

I really don't know why it doesn't work.

@hoavt-54 , could you show your environment variables?
And did you checked your CUDA installed successfully?
(use nvcc --version to check your cuda)

Most important error message is this:
/usr/bin/ld: cannot find -lcuda
collect2: error: ld returned 1 exit status
Cannot build a stub file.

It suggests that C compiler (ld command) cannot find libcuda.so. This is critical problem. libcuda.so is part of nvidia driver and is usually located at /usr/lib/x86_64-linux-gnu. For example in my ubuntu14.04 it is here:
% ls -la /usr/lib/x86_64-linux-gnu/libcuda.so
lrwxrwxrwx 1 root root 12  5月  9 21:08 /usr/lib/x86_64-linux-gnu/libcuda.so -> libcuda.so.1

How did you install NVIDIA driver and CUDA toolkit? I highly recommend to use deb package which you can download NVIDIA's site, and do not use .run file.

@Paosder Thank you so much. Here is my environment variable
$ echo $LD_LIBRARY_PATH
/users/ud2017/hoa/cudnn/lib64:/users/ud2017/hoa/cudnn:/opt/cuda/lib64:/opt/cuda/lib

It will be the same whether I put -L at the beginning or not
@unnonouno You're right but I thought putting instalation path to $LD_LIBRARY_PATH would help ld?
I ran ld -lcuda --verbose and ld said he couldn't find cuda in usual places, but he did not look into the places in $LD_LIBRARY_PATH :(
attempt to open /usr/x86_64-linux-gnu/lib64/libcuda.so failed
attempt to open /usr/x86_64-linux-gnu/lib64/libcuda.a failed
attempt to open //usr/local/lib/x86_64-linux-gnu/libcuda.so failed
attempt to open //usr/local/lib/x86_64-linux-gnu/libcuda.a failed
attempt to open //usr/local/lib64/libcuda.so failed
attempt to open //usr/local/lib64/libcuda.a failed
attempt to open //lib/x86_64-linux-gnu/libcuda.so failed
attempt to open //lib/x86_64-linux-gnu/libcuda.a failed
attempt to open //lib64/libcuda.so failed
attempt to open //lib64/libcuda.a failed
attempt to open //usr/lib/x86_64-linux-gnu/libcuda.so failed
attempt to open //usr/lib/x86_64-linux-gnu/libcuda.a failed
attempt to open //usr/lib64/libcuda.so failed
attempt to open //usr/lib64/libcuda.a failed
attempt to open //usr/local/lib/libcuda.so failed
attempt to open //usr/local/lib/libcuda.a failed
attempt to open //lib/libcuda.so failed
attempt to open //lib/libcuda.a failed
attempt to open //usr/lib/libcuda.so failed
attempt to open //usr/lib/libcuda.a failed
ld: cannot find -lcuda

Anyway, the problem is I don't have root privileges and cannot link the lib to those usual places.
I'm gonna close this. Thank you guys so much for your support.

No, usually a liker command uses $LIBRARY_PATH. $LD_LIBRARY_PATH is used only on runtime by dlopen system call. See http://man7.org/linux/man-pages/man3/dlopen.3.html


Survey return fail - empty block

There are many possibilities for a survey to fail. it turns out that if a block is EMPTY (which it can be) our class broke. I think this is fixed... but still testing. opening this issue as a reminder to test on several surveys.

This was fixed using this commit... but leaving open for more testing
0f507b998bdb0be1112b87c643385e79cf9dee2c

this seems to be stable now! closing


not working for net48

Hi
I can see it should support net48 but when i install it , i get this error

Make an effort and you will get there :)


feat(DataBrowser): add form to load from dicomWeb server

Just a sketchy proof of concept...
closes #109

Nothing to see here...


Add examples for using parameters as conditionals and setting parameters

Add examples for using parameters as conditionals and setting parameters
PR for issue
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] Added or updated documentation
[x] All your commits have a Signed-off-by line in the message. (more info)

Ah - sorry, code looks good but you'll need to fix the DCO check.
git rebase HEAD~1 --signoff and force push the branch aught to fix it! :pray:

Thanks for this! Great stuff!


docs: we support scopes now

Now that we support scopes, we can remove the warning, but since it still is not a token endpoint, a new warning should be added.
Is this really a warning or should we add it somewhere else?
I used 'we recommend', because I think we can recommend tokens.indieauth.com, while pointing at the selfhosted version too. Is everyone on board with that recommendation?

Yeah, this isn't a warning anymore, should certainly document it.  And honestly any token endpoint is going to be now tied in to whatever micropub endpoint.  I don't see any need to recommend any specific software.

New commit addresses @dissolve’s comment and links to a list of IndieWeb implementations instead of IndieAuth.com’s token endpoint.
Also move it out of warnings and into the general information section.


Apply ABL for multiple classes

Thanks, @wangchi95 for the great work. I want to apply ABL for multiple class segmentation problems. Is it only the C in line 45 (bellow) to be changed to "total number of classes" in this case?
           partial(class2one_hot, C=1),

Thanks for your interest in our work. Our loss is natively designed for multiple classes, so you can directly feed the target with shape [N,H,W] and logits with shape [N,C,H,W]  to ABL, where the C here is the number of classes. In more detail, we obtain GTB (function in line #89) and PDB (function in line  #65) by comparison and measuring multi-class probability differences with KL divergence, respectively.
The code around line #45 is used to generate a distance map via GTB, and the C here is not the classes number of logits.

Thank you @wangchi95 for the reply.

On the other hand, are there any changes needed to apply this loss to a binary output?

On the other hand, are there any changes needed to apply this loss to a binary output?

No more changes to loss are required for binary output. But it should be noted that the data should be provided in one-hot format. Assuming the shape of the binary probability output is [Batch, H, W], you can use the concatenation of [1-prob, prob] as the prediction input (the shape should be [Batch,2,H,W]) for loss.

On the other hand, are there any changes needed to apply this loss to a binary output?

No more changes to loss are required for binary output. But it should be noted that the data should be provided in one-hot format. Assuming the shape of the binary probability output is [Batch, H, W], you can use the concatenation of [1-prob, prob] as the prediction input (the shape should be [Batch,2,H,W]) for loss.

On the other hand, are there any changes needed to apply this loss to a binary output?

No more changes to loss are required for binary output. But it should be noted that the data should be provided in one-hot format. Assuming the shape of the binary probability output is [Batch, H, W], you can use the concatenation of [1-prob, prob] as the prediction input (the shape should be [Batch,2,H,W]) for loss.

Is there any problem with binary segmentation where the output is [batch_size, 2, H, W] and the label is [batch_size, 1, H, W]?


@Stroke/@SkillStroke triggers not working with Lumberjacking skill

Tested X Revs: 3720 and 3772
When it comes time for the additional strokes marked with local.craftstrokecnt in @Start while using the skill Lumberjacking, the @Stroke trigger does not apply any effect. It is easy to verify:
[SKILL 44]
DEFNAME=Skill_Lumberjack
KEY=Lumberjacking
TITLE=Lumberjack
FLAGS=skf_gather
DELAY=1.0
STAT_STR=85
STAT_DEX=45
STAT_INT=30
BONUS_STR=90
BONUS_DEX=10
BONUS_INT=0
BONUS_STATS=20
//using custom skillchance and skillgain
ADV_RATE=0,0,0
VALUES=1,1,1
RANGE=2
ON=@Stroke
src.say test test
And the character does not say the phrase. The same goes for return 1 or any other command. In crafting skills like blacksmith, mining, or bowcraft, trigger works without any issues.

Tested with vanilla script from scriptpack and it works, so the problem is on my side with scp file


Chaos;Head Dual [PCSG00411]

Game summary

Game name: Chaos;Head Dual
Game serial: PCSG00411
Game version: 1.00

Vita3K summary

Version: v0.1
Build number: 1276-3ee33b5e
Commit hash: https://github.com/Vita3K/Vita3K/commit/3ee33b5e40861a8d1ed7673b86cad5ec398724cf

Test environment summary

Tested by: chnark
OS: Windows 10
CPU: Intel i7-6700
GPU: NVIDIA GTX 1070
RAM: 16 GB

Issues
Nothing past initial game selection screen (Noah or Love Chu Chu)
Screenshots
None
Log
vita3k.log

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

So... this "game" is actually just a launcher for two other games(Chaos;Head Noah and Chaos;Head LoveChuChu). The menu "game" loads fine. I can't recall if there was actual music for the launch screen but besides that it's basically fully working(looking at the files, no sound... so minus sceAppMgrLoadExec not being impl'd it works 100%)
The thing is... it's really three games. The two packaged titles released for the Vita digital only later on under their own serial numbers(Just USERDIR_ and the eboot renamed). Should these be split with this as a hub page or...?

Launcher - Everything works minus... actually launching the two other games. It also requires hardware flipping
ChaosHeadDual_launcher.log
Noah - Unicorn error out of the gate. It seems it was doing some folder IO (and failing), called sceKernelRegisterThreadEventHandler(unimpled), and then unicorn'd.
ChaosHeadDual_Noah.log
Love Chu Chu - Goes through some of the loading menus, then crashes from NGS. Flipping required.
ChaosHeadDual_LCC.log

as of https://github.com/Vita3K/Vita3K/commit/b5aa1ec5dd373ed7f8f65b1b7593392d4b145477, both Noah and Love: Chu Chu is ingame. Noah has some graphical issues and Chu Chu has a slight audio bug and when i pressed the L button, while moving to the dialogue, it froze.
Screenshots

Logs
vita3k (CHL;CC).log
vita3k (CHN).log
vita3k (menu).log
Recommended Labels

Ingame +
graphics bug
audio bug

Recommended Labels
Ingame +
graphics bug
audio bug

Recommended labels
ngame +
graphics bug
audio bug

Recommended labels
Ingame +
graphics bug
audio bug


New publishing with dev versions on bintray

The plan is to add an interactive script/task for the version update and publishing process in this PR before merging it.


Consider more transforms to address lint warnings

Warning
Count
Github Issue

camelcase
1267

eol-last
819
Autofix by eslint #68

sort-class-members/sort-class-members
739

no-unused-vars
654

consistent-return
633

babel/arrow-parens
529

prefer-const
382
#137

dot-notation
312
#111

shopify/jquery-dollar-sign-reference
196
#107 (partial fix)

no-empty-function
178
#110 (partial fix)

shopify/prefer-class-properties
166

newline-per-chained-call
152

id-length
141

block-scoped-var
124

no-unused-expressions
95

lodash/prop-shorthand
71

func-style
62

lodash/preferred-alias
54

shopify/binary-assignment-parens
48

no-implicit-coercion
44
#105

consistent-this
34

shopify/prefer-early-return
32

quotes
27

no-extra-semi
21

lodash/collection-method-value
20

no-console
16

no-empty
14
#109 (partial fix)

lodash/chain-style
13

prefer-template
9

no-sequences
8

shopify/sinon-prefer-meaningful-assertions
8
#81

lodash/no-single-chain
6

yoda
6

comma-dangle
5

no-new
5

promise/catch-or-return
5

babel/object-shorthand
3

no-nested-ternary
3

no-alert
2

no-extra-boolean-cast
2

no-inner-declarations
2

no-throw-literal
2

operator-assignment
2

lodash/matches-shorthand
1

lodash/prefer-flat-map
1

space-infix-ops
1

I definitely want to solve the prefer const, particularly for for of loops, which appear to consistently use let.

I can work on a prefer const in for of when no reassignment in scope transform.

@fandy no need, I corrected it in the jscodemod/no-vars transform, it should be available for us to use.

I'm going to close this because I think we've addressed about as many of these as we can.


How to support distributed transaction?

Hi,I‘m a programmer from China,so I'm sorry for my bad English.
Now I built my application with .net core 2.2,it consists of multiple webapi services.If I have a distributed transaction,and each activity is in its own service, how should I use Route slip and saga?Is there an example for reference?
Thank you!

There are examples of using routing slips:
https://github.com/phatboyg/demo-registration
There is also documentation:
http://masstransit-project.com/MassTransit/advanced/courier/


pdf/A

With unoconv it was possible to make different kind of pdf/A by passing -eUseTaggedPDF=1 -eSelectPdfVersion=1
Would this be possible with unoserver/unoconvert?

The next release will have a --filter-option parameter where you can add SelectPdfVersion=1. I haven't tried TaggedPDF=1, but I think that should work as well.


fmCheckMate: Chart kills Performance

On the fmCheckMate text result view there is a FileMaker chart lying off-layout.
It was deactivated for the runtime solution, but now we are back to g  FileMaker files it can be reactivated.
In the meantime it is causing the text view to be very slow with large texts.
It should be removed from the text view and implemented on a separate layout.￼￼￼

I think this is already fixed.


APIM Fix deploys stv2 compute platform now

By introducing an explicit public IP APIM deploys in stv2 compute model
Needed because stv1 is almost EOL
You will have to reprovision APIM - so new AZD UP in a fresh environment

You are very right @punitkshah !
I'll change the bicep. In either case (Internal or External) a public IP should be provided to force stv2.
I wasn't aware, thanks for pointing this out.


[RFC][diem adapter] introduce VMAdapter trait

Motivation
This PR introduces VMAdapter trait and refactors some diem_vm code, making it easier for other instantiations of Diem to reuse Diem VM code. Changes include:

adapter_common.rs file is added and some of the code in diem_transaction_validator is moved there and made generic
DiemVMValidator is combined into DiemVM
VMAdapter is introduced and DiemVM implements it

TODOs:

More refactoring, especially code in diem_transaction_executor
Figure out what methods to add to/remove from the trait

Have you read the Contributing Guidelines on pull requests?
Yes

My biggest surprise was to see currency related information here as that doesn't feel really like a general concept for Move.

Refactoring out a common subset of the Diem adapter does not necessarily mean that those common things are general concepts for Move. I know we've talked about having Move provide a general adapter skeleton, but I didn't know that was the intention here. Maybe this could be a first step in that direction, but I can also see it being a totally separate thing that is only used for Diem and derivatives of Diem.

Bob's way of reasoning makes sense to me and I think we should further clarify these:

Do we want it to be something general for common clients/blockchains using the Move VM?
Do we even consider Trove a derivative of Diem?

I know we've talked about having Move provide a general adapter skeleton, but I didn't know that was the intention here.

My intention with the trait and adapter_common code is to provide some common functionalities that adapters for other blockchains or distributed ledgers can use, and the trait implementation should contain the customizations for the specific blockchain. So ideally code in adapter_common should not mention any Diem-specific concepts. Diem-specific concepts should be hidden inside DiemVM's implementation for the trait.
I guess the question here is: which are Diem-specific concepts? Paying for gas? Currency? Session? Session is actually a Move runtime concept, so maybe it's fine to include session?

Session? Session is actually a Move runtime concept, so maybe it's fine to include session?

Your reasoning (xxx is a Move concept => fine to include xxx in the adapter trait) totally makes sense, but I'd like to mention that we are looking into to getting rid of Session, so it may not be a good idea to expose it in this particular case. Instead, maybe expose interfaces that would allow you to execute different types of transactions? (fn execute_script, fn execute_function, fn publish module or maybe even a single fn execute_transaction that takes an associated transaction type?)

My biggest surprise was to see currency related information here as that doesn't feel really like a general concept for Move. However, I could see it being useful for several different applications... so thats a tough one. Is there a way to have some shared code for it without making it a part of the trait?

Currency related content has been removed from adapter_common. Gas price is still present though because of limitations I mentioned in the comments in the code.


Add two nos in Python

This Pull Request resolves Issue #429

Any updates or should we close the PR?

PR closed due to inactivity!


add s1meta.geoloc

add s1meta.geoloc (second try because first PR #45 ) was linked to a branch heading on a branch with lots of changes and not the latest develop.
PR meant to address issue #42 .

polygon of the footprint given by s1meta.geoloc.footprint

with last commit we temporarily remove incidence and elevation from s1meta.geoloc.


Don't require GITHUB_TOKEN when it is not needed

From what I can see, the GITHUB_TOKEN is not needed for discovery (it would only have access to the current repo anyway), so we can remove it from this check. I ran into it during debugging (output tested in a ACTIONS_STEP_SUMMARY) and since it is not needed....

I think we need it for the discovery mode because in order to use the octokit instance (See) you first need to provide the token (See)

For the call to get public repos you don't need to be authenticate, so that was my thinking. I now remember that this will work on github.com, but not on Enterprise server (where I eventually want to use this as well for my customers), since even for a public repo on that, you will need to be authenticated.
Even the Octokit toolkit makes the token required, so it is needed.
Good to double check this indeed 😄. Closing this issue.


Bug: If RFI files requested = null then do not show upload files button on applicant dashboard

Describe the bug
Applicants can still see the upload files button when there are no files being asked for (but there used to be).
Applicant still has the ability to upload a file that was unselected (when at least one other file is being asked for)
To Reproduce
Steps to reproduce the behavior:
Part 1

Go to RFI screen on analyst portal
Find an existing RFI that asks for files to be uploaded and the due date hasn't passed
Uncheck all of the boxes for the files being requested
On applicant dashboard the upload files button still appears
Part 2:
Go to RFI screen on analyst portal
Find an existing RFI that asks for MORE THAN ONE file to be uploaded and the due date hasn't passed
Uncheck ONE of the boxes for the files being requested
On applicant dashboard click the upload files button
On the upload page it still allows the user to upload the file unselected in step 3.

Expected behaviour
Part 1: If no files are selected then applicant should not see the upload documents button
Part 2:  If analyst unselects one file to be uploaded but they are still asking for one or more files then applicant should not have the ability to upload the file that was unselected

Part 1 is not quite right yet.

Note above that I don't have any files selected for upload but in the below it still offers me an upload documents link on the applicant dashboard.

Part 2 works as expected! :)

@MarsRomer2 Can you please give me more information on this? I have tested this some more and I cannot reproduce it. I have gone back and forth checking and unchecking files for the RFI and the Upload files button appears when it should for me. Thanks!


nvalt crashes when trying to print to pdf

Hi,
I've recently started using nvALT, and I wanted to report a bug. When trying to save a previewed note to PDF, nvALT crashes. This happened every times I tried to do so. I'm working on MBA 2014, El Capitan 10.11.3, nvALT 2.2b.
I'm attaching the crash report in case it helps...
Adam
nvalt crash report.txt

Duplicate with #399


rootless.md: add systemd unit example

Fix #3866 .
Only problem here is that what I am saying is perhaps wrong :) . I can't manage to get BIND capability to work with this example (simplified, cf commit for the rest):
[Service]
AmbientCapabilities=CAP_NET_BIND_SERVICE
ExecStart=/usr/bin/podman run --name=nginx --net=host --add-cap=CAP_NET_BIND_SERVICE nginx

I am getting a permission denied on binding.
I don't know if I am missing something, there is a bug or if it is a limitation.

as rootless you won't be able to use privileged ports (< 1024).
If you wish to do so, you'll need as root to tweak net.ipv4.ip_unprivileged_port_start

/ok-to-test
/approve

/lgtm
Thanks @MatMaul

@giuseppe thanks for the answer, something is still unclear to me: will other capabilities be "forwarded" to the child process running in the user namespace if I use AmbientCapabilities and --add-cap, BIND being a special case ? Otherwise we perhaps want to remove "capabilities" from my last sentence.


Added link to Quobyte's volume plugin.

Signed-off-by: Felix Hupfeld<EMAIL_ADDRESS>

LGTM, thanks!

LGTM 🐵


Drop support for CloudABI

Since https://github.com/rust-lang/rust/pull/78439, Rust doesn't support the CloudABI targets anymore. We dropped CI for them (https://github.com/rust-lang/libc/pull/1981) but we still have the CloudABI module (https://github.com/rust-lang/libc/tree/02fe5de20b9d24db107932985aa3b4dc11fe408c/src/cloudabi). We will deprecate it and remove in the future release.

If the cloudabi targets no longer exist in Rust, then it's no longer possible to do a build that could access these modules, so I don't think it's a breaking change to remove them.
In theory, someone could use a custom target, or an older version of Rust, but we already don't guarantee that arbitrarily old versions of Rust can build arbitrarily new crates.

If the cloudabi targets no longer exist in Rust, then it's no longer possible to do a build that could access these modules, so I don't think it's a breaking change to remove them.
In theory, someone could use a custom target, or an older version of Rust, but we already don't guarantee that arbitrarily old versions of Rust can build arbitrarily new crates.

I am also inclined to drop CloudABI support. I don't think this is a breaking change since the target has already been removed from rustc. And honestly nobody is using CloudABI any more, they've all moved to WASI.


[2.0-wip] Waiting/Hour glass

Perhaps add a Waiting/Hour glass functionality. Progress bar works great for really long tasks, but for not so long tasks I guess hour-glass is better.

i don't get it, can you explain it more precisely or draw it?

Something like this (wait cursor) - http://www.sapdesignguild.org/resources/images/busy/loading_anim.gif

So you want that to include an image or css in bt?
existing site for image spinner: http://www.ajaxload.info/
existing site for css spinner: http://davidwalsh.name/dw-content/spin-js.php

Yea making it part of BT library.

We won't be adding a spinner at this time.


Add new extensions

Lists some new extensions I found, so that I (and others) can find them again in the future :)
@dimitrov-adrian & @georgexchelebiev hopefully you are okay with adding these to the list?

@Jonesus yes, of course

@Jonesus yes <3 In the next couple of weeks I'll dedicate some time to cleanup the code and set up a pipeline for releases


Json validation part 2

In this PR we're adding validation that checks if json dictionaries are all sorted, unit test for that validation and integration with ledger.
If invalid message is sent to be signed, ledger will parse it and reject it returning an error code.
Also, in this PR, we've added 2 new messages that have invalid formats to goclient code.
Known issue:
It seems that a bog standard message serialized by cosmos is already invalid because its dictionaries are not sorted. If you run goclient.go code, the very first message fails validation. @cwgoes, @jleni have I misunderstood the "dictionary in sorted order" validation or is there a problem in cosmos serialization?

Ref https://github.com/tendermint/go-amino/issues/171

The PR has some compilation error. But still, merging this will result in rejected txs until tendermint/go-amino#171 is solved.

Blocked on https://github.com/cosmos/cosmos-sdk/issues/1550, we should also sort the top-level fields (chain ID, sequence, ...) for consistency.


Travis CI is no longer supported

Travis CI has changed how open source projects are supported, and Hyperledger can no longer build there. I suggest Azure Pipelines or GitHub Actions as a replacement.

Thanks, @ryjones. We have been moving to github actions for all of our projects.

Moved to Github Action.


Add type for DeploymentExtend and ResourceGroup

We add Type in ARM API, accordingly we also add Type for DeploymentExtended and ResourceGroup for consistency

Can one of the admins verify this patch?

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node


Multistage Docker build

We can reduce the size of our docker images by using multistage builds. Let's do that.

@cbizon should this move to jira?

Yes


fix - typo in contribute.md

"your" -> "you"
one thing... I couldn't find the staging branch... is that supposed to be "next"? I'm a little confused.

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[ ] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #XXXXX
Description

This typo is fixed in at least one pre-existing PR (#18251 )


require exampleUrl and urlPattern if using staticExample

Adds a couple of sanity checks. All our current badges pass them
closes #1994

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS


fix: Validate UUID tokens

fixes https://github.com/codecov/codecov-action/issues/436

Closes #222
Closes #228


Omit "saga has been cancelled" printout in browser console

Is there any way to disable this? I'm triggering it manually with 'yield cancel(saga)' so I don't see the point of it being printed by default...
Thanks for any help!

It's console.info, so you can easily filter it out, also its dev only

IMO it's still not very convenient. Cancellation is a regular situation, nothing like uncaught exceptions, so I don't think it should be logged. Also you can actually see it in Saga Monitor if you need.

@Andarist I understand that, but I think there should be an option to not log cancellations, since it's a common practice as @aikoven suggests. Especially since the biggest(?) react boilerplate project (https://github.com/mxstbr/react-boilerplate) uses saga injections which means that sagas have to be cancelled on every route change.
Wouldn't it be better to f.e. send in an extra param to the cancel like so:

yield cancel(sagaName, {log: true});

I agree that in my project it creates some useless logging statements.
However they are still useful when something is going wrong and you want to debug sagas without plugging the monitor :)

should we make the logging opt-in?

It would be nice if this could also be configured per generator.
Also, not sure if it was proposed already, it'd be great if we could supply a custom displayName for generators, different from generator.name, to be shown in these log messages as well as saga monitor. Would be useful for higher order generators. Setting Function.name doesn't work on some browsers.

Sorry that it took so long to act on this in any way.
What I would propose for now is - console.log only non-cancel cancellations (what a mumbo jumbo). What I mean is that we can distinguish why the cancelation has happened - is it user induced via cancel effect (no logging should take place) or is it cancel caused by cancel propagation when something throws (logging should take place in dev mode).
@aikoven
I like your displayName idea, will try to explore this. Would be also useful for overriding display name for helper effects like takeEvery as its implemented as mere fork(takeEveryHelper), would be cool to omit fork part out of this.

or is it cancel caused by cancel propagation when something throws

for this, we could also group these messages to more clearly indicate where the error originated and to which cancellations it lead.

Would be also useful for overriding display name for helper effects like takeEvery

Actually I have a lot of higher order sagas / custom effects, both could benefit from this. For now I'm using this:
function setFunctionName(func, name) {
  try {
    Object.defineProperty(func, 'name', {
      value: name,
      configurable: true,
    });
  } catch (e) {
    // ignore
  }

  return func;
}

But it only works on Chrome AFAIK.

for this, we could also group these messages to more clearly indicate where the error originated and to which cancellations it lead.

More trickier, but may be doable, gonna try this when I have time!

Actually I have a lot of higher order sagas / custom effects, both could benefit from this.

Could you share some of your ideas for higher order sagas / custom effects? Just names, not necessarily the code. I think it would be cool to expose the idea more to the audience by making a section of the docs about those 2.
As to higher order sagas I didnt do much of them in my app but like 1 or 2 became really handy, especially trailingThrottle and takeEveryBetween. Didnt have chance (or need) to write custom effects though other than some specified higher order functions returning regular take, like takeResponse (from websocket connection).

Is there any particular reason you are using defineProperty here instead of just assigning?

If I remember correctly, assignment didn't have any effect at all.

Could you share some of your ideas for higher order sagas / custom effects?

Sure! The one higher order saga I use most is bindAsyncAction, where async action is actually a set of three actions: started, done and failed. The returned generator first puts started, then calls the source generator, then done or failed depending on the result. Useful when you want to track the execution of your worker in reducers.
Another one, runBetween, runs given generator between two given actions, i.e. starts with the first action and cancels with the second. I use it for processes that should be run only when user is logged in. Of course I could just start these processes from the saga that controls log-in and user session, but there are a lot of them and I wanted to decouple.
I use custom effect for subscriptions to remote data updates, works like this:
const task = yield subscribe(modelName, filters, worker)

Worker gets updates through a channel that it receives as argument. The task has a special cancellation logic that sends unsubscribe request to the server.
One more custom effect / helper is listenState(selector, worker), which invokes worker once selected state slice changes.
As we discussed before, I think that custom effects and helpers like takeEvery are basically the same concept. I can't think of anything that couldn't be expressed as a composition of built-in effects. Therefore each of them is just a convenience function returning call, fork or whatever else effect.

Yeah, ive kinda have ended yesterday with the same conclusion - there is not really that much gain of custom effects as they tend to be achievable with higher order sagas in most cases.
There are those rare occasions when its hard to implement things with higher order sagas alone. For example your context feature. Although the easier version without dynamic scoping could be also easy implemented with a module keeping your context in a closure.
Also its not possible for now to have a forking worker (started due to call effect) which would return its result immediately when ending itself.
Although those custom effects are rather rarily needed it would still be cool to have a way of extending the core and built-ins.
Im now also wondering if moving helpers to the effects was a good idea (cc @yelouafi). It might be a good idea to move them out of there (again... :/) and create a separate module called 'helpers' (higher-order sagas is really a long name). Plus obviously spend more time in the docs about the idea of higher order sagas, so people understand the concept and dont be confused about separation of those and effects.
Nice ideas with your examples! Is listenState implemented using take('*')?

Is listenState implemented using take('*')?

Yep:
function* listenState(selector, worker) {
  let prevValue = undefined;

  while (true) {
    yield take('*');

    const value = yield select(selector);

    if (value !== prevValue)
      yield spawn(worker, value, prevValue);

    prevValue = value;
  }
}

I agree, that printing out should be optional. Would love to see my console less polluted :)

Same here. When developing, i have the browser devtools open most of the time. When switching from one route in my app to another, i tend to see a few hundred cancelled logs of websocket/firebase subscirptions, which really slows navigation down.

That's certainlt something that should be done. Ive added help wanted label. Maybe somebody would like tackle this? I can help with explaining the source code, but hadnt time to sit down and fix this yet (and probably wont in the nearest future).

@Andarist I've started this with #945 but could use some help with determining when a cancel is caused by error propagation (if I understand the new approach correctly)

Any update on this?  When I hot reload during development this gets pretty obnoxious.

@bradennapier it is definitely annoying. i started a pr here: redux-saga/redux-saga/pull/945
unfortunately, i am not working on this at the moment. would you like to pick up where i left off?

As a work-around for this in Chrome devtools you can filter your logs to not include these logs. Filter for /^((?!has been cancelled).)*$/ which is a regex that excludes lines with has been cancelled.

https://github.com/redux-saga/redux-saga/commit/f4b52b04eb6bb9094ab07661c98178b55b401a85
Gonna fo a release in a moment.


Add target to build controller for custom bundle

Description of changes
Allows to
make bundles-override-local-e2e-tests IMAGE_REPO=public.ecr.aws/xxx

This will:

Build the controller image
Push it to your registry
Reference that image in a custom bundle in local disk

The bundle will be in bin/local-bundle-release.yaml and the custom eksa manifest in bin/local-eksa-components.yaml. This is compatible with the e2e test suite, you just need to export T_BUNDLES_OVERRIDE=true.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


TransmogrifAI - GSoC

Hello GSoC mentors,
My name is Abhi Jadhav, and I am an undergraduate student at Duke University studying biomedical engineering and computer science. Throughout my studies, I have developed a passion for machine learning and data analysis. What intrigued me about the TransmogrifAI project(s), is the attention toward model interpretability - something I think is lacking in many approaches to ML (very black box prone). Additionally, the hyperparameter optimization theme reminds me of my own struggles with painstakingly trying to fine-tune parameters. I truly believe both of these factors are what contribute to ML being perhaps a bias in the age of the technological elite, something which the TransmogrifAI API can serve to work against. I would love to learn how I can contribute to this open source project and/or chat with a mentor!
Thanks for your time,
Abhi

Bump?

Adding on to a1pha, I took would like to know the way forward.


Need clean way to share data between parameterized tests

MSTest v2 needs a clean way to share test-data between tests. It should support objects.
Current approaches fall short:

DataRow can't be shared and doesn't support objects (just compile-time values).
DataSource doesn't support objects, and requires a database/csv.
Looping through an array/collection lumps the cases into a single test.

Suggest adding the equivalent of NUnit's TestCaseSource, which takes the name of a field, property, or method that returns an IEnumerable (which can handle objects):
static object[] DivideCases =
{
    new object[] { 12, 3, 4 },
    new object[] { 12, 2, 6 },
    new object[] { 12, 4, 3 }
};

[Test, TestCaseSource("DivideCases")]
public void DivideTest(int n, int d, int q)
{
    Assert.AreEqual( q, n / d );
}

Absolutely. Thanks for filing this @JVimes . This should also help drive a structure around #141.

This is being discussed in: Microsoft/testfx-docs#12

This has been addressed in 1.2.0-beta release of the Framework/Adapter pair.

Thanks! For others who find this, here's how to try it out:
Make sure you have MSTest v1.2.0 or later. At time of writing it's in beta, so need to install yourself using NuGet's "Include prerelease" checkbox:

Then, add a static, object[][] property to your test class. Fill in data. Add[DynamicData("ThePropertyName")] to your method, and make sure it has the right number/type of parameters:
static object[] Data
{
    get => new[]
    {
        new object[] {1, 2, 3},
        new object[] {4, 5, 6}
    };
}

[TestMethod]
[DynamicData("Data")]
public void Test1(int a, int b, int c)
{
    Assert.AreEqual(1, a % 3);
    Assert.AreEqual(2, b % 3);
    Assert.AreEqual(0, c % 3);
}

You can alternatively use a method that returns an IEnumerable<object[]>, instead of a property. I'm going off this doc.


Sources, not field

I feel that current setting field does not reflect the context.

Changing the setting parameter from field to source
Rationalizing default value to all
Pluralize sources
Tidying array build using concat

@forsigner any inputs? :)


wrapperClass only set by fragment, no other value will set it

Would it be possible to allow a default for wrapperClass to be passed in as WrapperClass the same way we do ClassName?
I'm using images/image as a partial in my WIP imagegrid partial, and there is no way to default the wrapperClass to, for example, #float-left. We only allow a user to pass down the class of the , not the  or .

I think we can pass the alignment via Filename, e.g. example.jpg#float-left.

That's what I'm doing, fine. No worries

PR #39 adds more parameters for the image partial, you should be able to use the Alignment for this purpose since v0.9.0.


AMQ-6327 - getNextScheduledTime() returns incorrect time when working…

… with day of month

'working' is not a very expressive variable name, but good patch. Many thanks!

Not sure why github didn't close this automatically. Please feel free to close.


feat: initial token for swap widget

This allows to set an initial token in , token out in widget through passing a token object. (Similar as in tokenList)

Should i add a simple utility that parses query params (or any string as if it was query param string ) for token symbols and finds them inside a token list (provided as an argument). We will use this stuff in defuse-frontend anyway can save ppl a bit of time .
@cawabunga-bytes

I don't think we need it rn.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Adding lbc big number

@brekk @sheeley @rcarnahan @bgphipps Please review.

Don't have a deep feel for it, but seems good to me. :+1:

LGTM

CONFIDENT MERGE


No background color

I'm new to vim as well as extending it. I've installed airline via vundle and added the custom fonts but airline doesn't seem to load any background color for the status bar. As a result the big triangles look a bit odd. What did I miss? Here is a screenshot and my config:
set laststatus=2
let g:airline_powerline_fonts = 1
set ttimeoutlen=50
set guifont=Source\ Code\ Pro\ for\ Powerline
let g:airline#extensions#tabline#enabled = 1

Thanks!

Add set t_Co to ~/.vimrc would solve this problem for me. But this would change my SolarizedDark colorscheme.

I have the same question.

I have the same issue using Terminator on Xubuntu 15.10
Any ideas? setting t_Co didn't work for me.

It turns out, in my case, the terminal emulator was not letting vim know it could display 256 colours.
The was confirmed by echo $TERM returning xterm.
The quick fix is to either  add set t_Co=256 in the .vimrc or to configure the .bashrc to return the correct $TERM value. This article http://vim.wikia.com/wiki/256_colors_setup_for_console_Vim provides all the necessary information.
Terminator on Xubuntu 15.10
before fix

after Fix