[FEATURE] Colorblind accessibility.

So I have photophobia and need to use dark mode, but I'm also colorblind and can't distinguish the red filled dots from the black empty ones. Same issue with health/willpower boxes.
It would be really helpful to have the option to change the color of the dots and slash marks. Or if customizability isn't an option, perhaps just a static choice between red and white.

Colorblind accessibility is definitely something I want to work on soon! It'll probably come with the overall redesign of the sheets to be more accessible, which will include making some buttons more obvious and doing my best to make things easier to navigate and read.
Thank you for creating an issue about this, I'll try to work on it when I can!

Until I can add an actual option to customize dot colours, I did want your opinion if I was to push out a small update to help with this issue. Would the following be a decent change to help you see the dots and slash marks better? (Old on the left, updated on the right)

I'm not sure if having the boxes filled in helps much, but please let me know if it does! And feel free to point out any other parts of the sheet that may be hard to read in dark mode.
In the future I do want the customizability to let people change the colours of the sheets as they wish, but because that may be a ways away, I hope I can find a good short-term solution to help out people for now.

The slashes are completely distinguishable on that. Although the empty boxes should probably also be white for consistency. The dots are very slightly better as well. That's just me though. Different colorblind people will have different reds that they can and can't see.
I'll have a look through all the sheets in a bit to see if there are any other places that could be improved.
Also, many thanks for addressing this.

Good idea! I'll adjust the styling a bit more but try to keep the colour contrast to help, at least until I can give the ability to properly edit the sheet colours and all.


push-notifications: add plugin

Adds a plugin that will send notifications to your phone.
Currently only supports pushbullet because i couldn't find many widely
supported free ones. Recommendations welcome.
Closes runelite/runelite#11437

You need a license in your repo

I've licensed the repo.


Remove debugging messages

We are starting to log messages in AWS Cloudwatch and the debug SASS messages show up there. It's not necessary there.

whats the tactic for clearing up these warnings?

I think just removing the @warn from the scss files?

:thinking:

ok, for now I've commented out the @warn directives. You'll see the PR, let me know how the logs shake out afterwards.


Previous fix was good, but not perfect

As mentioned in a previous issue, I was stuck at 104 releases of ~950, now it's down to 13, so great improvement
-> https://deadsara.bandcamp.com/album/self-titled
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("data did not match any variant of untagged enum StringOrNumber", line: 1, column: 2246)', ~/.cargo/registry/src/github.com-1ecc6
299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://stuntrock.bandcamp.com/album/were-not-assholes-were-just-smarter-than-you-split-release-with-tuyo
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 3492)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://stuntrock.bandcamp.com/album/fuck-canada-fuck-america-split-release-with-venetian-snares
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 3464)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://stuntrock.bandcamp.com/album/this-is-stunt-rock-volume-one-in-memory-of-bob-seger
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 3443)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/technoptimistic
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1879)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/red-eyes-ep
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1862)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.
2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/the-cult-of-remute
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1892)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/electronic-lifestyle
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1899)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://stuntrock.bandcamp.com/album/this-is-stunt-rock-volume-two-in-memory-of-tray-loren
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 3442)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://65daysofstatic.bandcamp.com/album/utopian-frequencies-disquiet-vinyl-bundle
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 4133)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/theme-tunes-for-10-games-never-made
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1958)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://redrumrecordz.bandcamp.com/album/threnody-bewildering-thoughts-1995
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 4210)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
-> https://remute.bandcamp.com/album/the-sound-of-money-radio-edit
thread 'tokio-runtime-worker' panicked at 'called Result::unwrap() on an Err value: Error("invalid type: null, expected a string", line: 1, column: 1936)', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/bandsnatch-0.2.0/src/api/mod.rs:194:74
Because the JSON isn't anonymized, I'm not posting it here, the locations seem to be off too, so it's hard to pin point the exact offending key value pair. I apparently have a bandcamp key left for a few of the remute's, so there is a way to test it.

Should be fixed now in v0.2.1

Worked like a charm for the last 13, going to redo everything now to see if it can run in 1 session, and after that check with flac for completeness.

Regetting all the mp3-320 gave 1 error on the first try, and finished all on the second try, probably a connection dropped or something,
Getting all the flac gave 8 fails on the first try, and all finished on the second try, I'll chalk that up to connection issues too, Amazing results since the initial release I tried last week.

Awesome, glad to hear it's all working out now.


saved posts not showing

Is your feature request related to a problem? Please describe.
<-- A clear and concise description of what the problem is. -->
Describe the solution you'd like
<-- A clear and concise description of what you want to happen. -->
the new user stuff made a bug where the saved posts no longer show

Yep, simple fix tho: src/web/src/Pages/user/ProfilePage.tsx line 88 need to add "saved"


ISuiteListener.onStart method called mulitple times if i have multiple test elements in xml configuration file

Hi
I'm using testng 6.3.1. I got a weird result if i declare a @Listeners, which define my implementation of ISuiteListener, on a base class and define multiple test elements, each of which include only one test class inherited from that base class, in the xml configration file.
Here is my testng.xml file:
<suite name="suite" verbose="1">
    <test name="test1">
        <classes>
            <class name="org.beangle.webtest.temp.TestA" />
        </classes>
    </test>
    <test name="test2">
        <classes>
            <class name="org.beangle.webtest.temp.TestB" />
        </classes>
    </test>
</suite>

My implementation of ISuiteListener
public class SuiteListener implements ISuiteListener {

    @Override
    public void onFinish(ISuite suite) {
        // TODO Auto-generated method stub
    }

    @Override
    public void onStart(ISuite suite) {
        System.out.println("*******Suite Listener Start");
        System.out.println(":::::::suite detected: " + suite.getName());
        for(XmlTest xmlTest : suite.getXmlSuite().getTests()) {
            System.out.println(":::::::test  detected: " + xmlTest.getName());
            for(XmlClass xmlClass : xmlTest.getClasses()) {
                System.out.println(":::::::class detected: " + xmlClass.getName());
            }
        }
        System.out.println("*******Suite Listener End\n");
    }
}

TestBase.java
@Listeners(SuiteListener.class)
public class TestBase {
}

TestA.java
public class TestA extends TestBase {
    @Test
    public void test() {
    }
}

TestB.java
public class TestB extends TestBase {
    @Test
    public void test() {
    }
}

After I run the test, the console output is
*******Suite Listener Start
:::::::suite detected: suite
:::::::test  detected: test1
:::::::class detected: org.beangle.webtest.temp.TestA
:::::::test  detected: test2
:::::::class detected: org.beangle.webtest.temp.TestB
*******Suite Listener End

*******Suite Listener Start
:::::::suite detected: suite
:::::::test  detected: test1
:::::::class detected: org.beangle.webtest.temp.TestA
:::::::test  detected: test2
:::::::class detected: org.beangle.webtest.temp.TestB
*******Suite Listener End

I am confused about the result. SuiteListener.onStart method is called twice, is that the expected result ?
Can I make SuiteListener.onStart method called only once?
Thanks in advance.

Yes I met the same problem, I tested it in testng 6.8.8 and testng 6.8.17。
No only that.
If I have multiple suite files,I declare my custom IReporter or IExecutionListener in maven-surefire-plugin like
<artifactId>maven-surefire-plugin</artifactId>
	<configuration>
		<suiteXmlFiles>
			<suiteXmlFile>src/test/resources/testng.xml</suiteXmlFile>
			<suiteXmlFile>src/test/resources/testng2.xml</suiteXmlFile>
		</suiteXmlFiles>
		<properties>
			<property>
			<name>listener</name>
                          <value>com.ctrip.cap.reporter.MyReporter,com.ctrip.cap.reporter.MyReporter</value>
			</property>
		</properties>
       </configuration>

or I use @ Anotation Listeners in super class,
or I use testng's xml configuration in testng.xml and testng2.xml like
	<listeners>
		<listener class-name="com.ctrip.cap.internal.suite.MyExecutionListener " />
		<listener class-name="com.ctrip.cap.reporter.MyReporter" />
	</listeners>

They all called multiple times.
My Question is that why not make Listener's Class Container as a Set,and no matter how to register it,The Listener of the same Class and same type should be called once time.

I am facing the same issue . In my scenario, there is only one testcase in single class. I am using Testing version 7.1.0 and I am facing the issue. I have tried testing version below 6.10 but facing the same. Kindly help.

@Abhijitdatta can you show us a complete sample that can be used to reproduce this problem ?


reverse_named_url fails on Django 1.5 if using url arguments

While migrating an app to Django 1.5 one of my menus handled by treemenus started to fail, it was looking for an url named this way:
"my_awesome_url its_awesome_argument"
All as a single url, this commit fixes that, but i owe you a test case.

https://docs.djangoproject.com/en/1.8/ref/templates/builtins/#std:templatetag-url
try to use 'url' to replace 'reverse_named_url'   :)
just  like :
{{ menu_item.caption }}


Update Find-DbaStoredProcedure.ps1

Fixed main problem in Issue 3687 - find in TextHeader. Updated old-style join to normal join. Ran Invoke-DbaToolsFormatter.

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[X] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Address Issue 3687 - find in TextHeader
Approach

Commands to test

Screenshots

Learning

Seems the formatting is broke, that is what the test fails for on this command....not sure what editor you are using?

Also we ask that you ensure you have your development branch in sync with upstream when submitting PRs. I believe the other test failure will be resolved once you sync your branch with ours.

Thanks, @wsmelton . Let me try this again: I did a fresh fork from dbatools and a clone and then re-did my changes. I used Visual Studio Code as the editor. It formats things correctly, but I try to use Invoke-DbaToolsFormatter on the file before committing, and that messed up the formatting when I removed the unnecessary IncudeSystemObjects SMO branch at the bottom. I run Invoke-DbaToolsFormatter because the automated testing often fails for me with that suggestion. Thanks.

We have a format file for VS Code in the repository so when you clone it down and open the workspace you can just use the formatting in VS Code.
After you have forked the repository you will simply need to add an upstream on your local system (not something VS Code allows in the UI). It is git remote add upstream https://github.com/sqlcollaborative/dbatools.git (or the SSH address if that is what you prefer to use).
After you add that upstream you can go into VS Code, make sure you have development checked out and then do the command palette option to pull from another remote. Just type in git pull from, you will see the 'upstream' added above in that list and then pick the development branch. That will "sync" our changes to your local development, then you can just create a branch from there.

Thanks, @wsmelton . For git work I have been using GitHub desktop, but will give this approach in VS Code a try next time. I tried fixing this issue this morning but got a separate (unrelated?) issue: https://github.com/sqlcollaborative/dbatools/pull/5340


Finished TODOs

Commit messages are not clean

no exceptions in login
no getter for username needed for now. fixing TODO 2 error is easier than you think!

There is a "Wrong Username or password" exception in login function in line 67
From: Ali Mahdiyar
Sent: 28 October 2018 13:58
To: alimahdiyar/stackoverflow-in-cpp
Cc: s-arme; Author
Subject: Re: [alimahdiyar/stackoverflow-in-cpp] Finished TODOs (#3)
no exceptions in login
no getter for username needed for now. fixing TODO 2 error is easier than you think!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

There is a "Wrong Username or password" exception in login function in line 67 From: Ali Mahdiyar Sent: 28 October 2018 13:58 To: alimahdiyar/stackoverflow-in-cpp Cc: s-arme; Author Subject: Re: [alimahdiyar/stackoverflow-in-cpp] Finished TODOs (#3) no exceptions in login no getter for username needed for now. fixing TODO 2 error is easier than you think! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

I saw exception now. But there are still problems with username.

Do you mean easier solution instead of getter for username or …?!
From: Ali Mahdiyar
Sent: 28 October 2018 15:46
To: alimahdiyar/stackoverflow-in-cpp
Cc: s-arme; Author
Subject: Re: [alimahdiyar/stackoverflow-in-cpp] Finished TODOs (#3)
There is a "Wrong Username or password" exception in login function in line 67 From: Ali Mahdiyar Sent: 28 October 2018 13:58 To: alimahdiyar/stackoverflow-in-cpp Cc: s-arme; Author Subject: Re: [alimahdiyar/stackoverflow-in-cpp] Finished TODOs (#3) no exceptions in login no getter for username needed for now. fixing TODO 2 error is easier than you think! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
I saw exception now. But there are still problems with username.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@alimahdiyar

Try fixing TODO 2 without even touching the for loop.
The problem is now fixed with old for loop style.


parent field not accessible in arborescence class

Hi,
I am using your code to obtain max arborescence, but the field parent in your arborescence class is not public. For eg., I cannot do the following (taken from the test code for Chu-Liu-Edmonds)
final Map<Integer, Integer> maxBranching = weightedSpanningTree.val.parents;
Please provide a method to access the parents field.
Thanks

fixed in https://github.com/sammthomson/ChuLiuEdmonds/commit/8826522449d4d64edca810134979f55ead25fd08

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


Removing a  MethodEffect created dynamically ?

Let say I have this custom element, I try to "link" it with the centralized data.
  class CarCard extends AppData(Polymer.Element) {
    /* the AppData mixin only provides a global object
       centralizing data ("appdata" object) */

    static get is() { return 'car-card' }

    static get properties() { return {

      carId: {
        type: Number,
        observer: '_onCarIdChange'
      }
    }}

    static get observers() { return [
       `_onCarChange(appdata.cars.${this.carId}.*)`
       /* this won't work because this is a static function
          and properties get initialized after observers.
          Solution : Dynamic Observers (see "_onCarIdChange" below) */
    ]}

    _onCarChange(car) {
      console.log(`A property in car number #${this.carId} has just changed.`);
    }

    _onCarIdChange() {
      /* solution : */
      this._createMethodObserver(`_onCarChange(appdata.cars.${this.carId}.*)`);
      /* here we need to remove the old observer if any */
    }
  }

I don't know if this is a good idea to use _createMethodObserver directly (the leading underscore makes me think like that) but I can't think of another solution in this case.
I struggle to find the method/api to remove an observer created dynamically. Does someone know how I can do that ?

Yes you can definitely use _createMethodObserver, as this is part of our public API. The reason it starts with an underscore is to indicate it is only intended to be overridden by a subclass, but not intended to be invoked from an element user perspective.
You should be able to remove the effect with this._removePropertyEffect, however it seems the the implementation is pretty gnarly. The problem is that you have no access to the actual effect object being pushed into the internal data store. We would need to investigate how easy it would be to update this API.

Why would it not be possible for _createMethodObserver and accordingly _addPropertyEffect to just return the effect and make it removable by calling _removePropertyEffect?

@fooloomanzoo Because underlying createMethodEffect is called, which creates effects for every argument. This means that there are a lot of effects to be returned. Instead we should consolidate this data somehow in a nice manner and make _removePropertyEffect able to handle that


Map card design

Here is a map card design proposal. I have added features that are needed to accommodate layer reordering that was suggested by Ari in the demo.
The size of the card can be slightly larger without affecting the functionality too much. Here the map image is 70x70 px, it could be 80x80, and the buttons to the right could be 40x40. Current measures are based on iD design.
Basic state

Expanded metadata view

Flyout for actions

Reorder - hover over middle area

Remove map from list

Drag maps to reorder

Change map transparency

Map transparency display

Map transparency display on hover

Great set of mockups, I believe they should now be in the external front end project: https://github.com/wikimaps-dev/wikimaps-html
The mapwarper code (and thus, these issues) should be just for the existing user interface and site and the future API only proposal. mapwarper will be the API for this external project to talk to, but mapwarper won't be running these new designs, if that makes sense :)
I'll close this for now.


Fix all recurring job related test failed

https://ci.longhorn.io/job/public/job/v1.2.x/job/v1.2.x-longhorn-tests-ubuntu-amd64/2/
The recurring job spec.groups is an optional field https://github.com/longhorn/longhorn-manager/blob/3df7b91/types/resource.go#L190
Check the argument contains the groups field and then verify it's equal compares to the recurring job spec.groups.
Signed-off-by: JenTing Hsiao<EMAIL_ADDRESS>

@jenting Please backport to 1.2.x.


Tutorial [WinError 3] The system cannot find the path specified

I am trying to run the tutorial.  I get FileNotFoundError when running the below piece of code. I have installed gluonts in an Anaconda environment with Python 3.6 on Windows Home 10 version 1803 OS Build 17134.765.
`from gluonts.model.deepar import DeepAREstimator
from gluonts.trainer import Trainer
estimator = DeepAREstimator(freq="5min", prediction_length=12, trainer=Trainer(epochs=10))
predictor = estimator.train(training_data=training_data)`

FileNotFoundError                         Traceback (most recent call last)
 in
3
4 estimator = DeepAREstimator(freq="5min", prediction_length=12, trainer=Trainer(epochs=10))
----> 5 predictor = estimator.train(training_data=training_data)
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\model\estimator.py in train(self, training_data)
187     def train(self, training_data: Dataset) -> Predictor:
188
--> 189         training_transformation, trained_net = self.train_model(training_data)
190
191         # ensure that the prediction network is created within the same MXNet
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\model\estimator.py in train_model(self, training_data)
180             net=trained_net,
181             input_names=get_hybrid_forward_input_names(trained_net),
--> 182             train_iter=training_data_loader,
183         )
184
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\trainer_base.py in call(self, net, input_names, train_iter)
177
178         with tempfile.TemporaryDirectory(
--> 179             prefix='gluonts-trainer-temp-', dir='/tmp'
180         ) as gluonts_temp:
181
~\Anaconda3\envs\GluonTS\lib\tempfile.py in init(self, suffix, prefix, dir)
790
791     def init(self, suffix=None, prefix=None, dir=None):
--> 792         self.name = mkdtemp(suffix, prefix, dir)
793         self._finalizer = _weakref.finalize(
794             self, self._cleanup, self.name,
~\Anaconda3\envs\GluonTS\lib\tempfile.py in mkdtemp(suffix, prefix, dir)
368         file = _os.path.join(dir, prefix + name + suffix)
369         try:
--> 370             _os.mkdir(file, 0o700)
371         except FileExistsError:
372             continue    # try again
FileNotFoundError: [WinError 3] The system cannot find the path specified: '/tmp\gluonts-trainer-temp-nodyqafj'

This looks like a hand-written path somewhere (the “/tmp” path on Windows means nothing).
Thanks, looking into it.

#77 should have fixed the issue. Until a new release is tagged, you can try out the master branch by doing
cd gluon-ts
pip install .

and verify that the tutorial notebook works fine (fingers crossed).

Thank you for the super quick assistance!  That apparently took care of the initial issue that caused things to error out immediately, and I was now able to get to the end of model training.  However, at that point, I hit another (similar) error.
…
INFO:root:End model training
FileNotFoundError                         Traceback (most recent call last)
 in
3
4 estimator = DeepAREstimator(freq="5min", prediction_length=12, trainer=Trainer(epochs=10))
----> 5 predictor = estimator.train(training_data=training_data)
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\model\estimator.py in train(self, training_data)
192         # context as the one that was used during training
193         with self.trainer.ctx:
--> 194             return self.create_predictor(training_transformation, trained_net)
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\model\deepar_estimator.py in create_predictor(self, transformation, trained_network)
246         )
247
--> 248         copy_parameters(trained_network, prediction_network)
249
250         return RepresentableBlockPredictor(
~\Anaconda3\envs\GluonTS\lib\site-packages\gluonts\support\util.py in copy_parameters(net_source, net_dest)
126     """
127     with tempfile.TemporaryDirectory(
--> 128         prefix='gluonts-estimator-temp-'
129     ) as model_dir:
130         model_dir_path = str(Path(model_dir) / 'tmp_model')
~\Anaconda3\envs\GluonTS\lib\tempfile.py in init(self, suffix, prefix, dir)
790
791     def init(self, suffix=None, prefix=None, dir=None):
--> 792         self.name = mkdtemp(suffix, prefix, dir)
793         self._finalizer = _weakref.finalize(
794             self, self._cleanup, self.name,
~\Anaconda3\envs\GluonTS\lib\tempfile.py in mkdtemp(suffix, prefix, dir)
368         file = _os.path.join(dir, prefix + name + suffix)
369         try:
--> 370             _os.mkdir(file, 0o700)
371         except FileExistsError:
372             continue    # try again
FileNotFoundError: [WinError 3] The system cannot find the path specified: '/tmp\gluonts-estimator-temp-tth7mv0n'
Sent from Mail for Windows 10
From: Lorenzo Stella
Sent: Wednesday, June 5, 2019 5:01 PM
To: awslabs/gluon-ts
Cc: DaleThomas68; Author
Subject: Re: [awslabs/gluon-ts] Tutorial [WinError 3] The system cannot findthe path specified (#75)
#77 should have fixed the issue. Until a new release is tagged, you can try out the master branch by doing
cd gluon-ts
pip install .
and verify that the tutorial notebook works fine (fingers crossed).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That's really weird, because tempfile should now use the right default temporary directory for windows (c:\temp or something like that).
Maybe something got cached? Did you try installing the master branch on a brand new environment?

Thanks so much for the suggestions and your very timely responses!  I tried installing the master branch both by downloading the files, changing to the directory, and then using ‘pip install .’ and by pip install git+https://github.com/awslabs/gluon-ts.git.  Both errored out for me, and I could not successfully install.  So, I manually edited the two files after using pip gluonts in a new environment and also cleared my temp directory to handle the possible caching issue.  After all of that, things ran successfully. Thanks again!
Here was the error that was reported (I probably made a simple mistake, and I would love to know what it is so I will know in the future if you have time to answer):
(gluon_ts) C:\Users\dalet\Desktop\gluon-ts-master\gluon-ts-master>pip install .
Processing c:\users\dalet\desktop\gluon-ts-master\gluon-ts-master
Installing build dependencies ... done
Getting requirements to build wheel ... error
ERROR: Complete output from command 'C:\Users\dalet\Anaconda3\envs\gluon_ts\python.exe' 'C:\Users\dalet\Anaconda3\envs\gluon_ts\lib\site-packages\pip_vendor\pep517_in_process.py' get_requires_for_build_wheel 'C:\Users\dalet\AppData\Local\Temp\tmpg8xig0yc':
ERROR: Traceback (most recent call last):
File "C:\Users\dalet\Anaconda3\envs\gluon_ts\lib\site-packages\pip_vendor\pep517_in_process.py", line 207, in
main()
File "C:\Users\dalet\Anaconda3\envs\gluon_ts\lib\site-packages\pip_vendor\pep517_in_process.py", line 197, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
File "C:\Users\dalet\Anaconda3\envs\gluon_ts\lib\site-packages\pip_vendor\pep517_in_process.py", line 54, in get_requires_for_build_wheel
return hook(config_settings)
File "C:\Users\dalet\AppData\Local\Temp\pip-build-env-azzwbim1\overlay\Lib\site-packages\setuptools\build_meta.py", line 145, in get_requires_for_build_wheel
return self._get_build_requires(config_settings, requirements=['wheel'])
File "C:\Users\dalet\AppData\Local\Temp\pip-build-env-azzwbim1\overlay\Lib\site-packages\setuptools\build_meta.py", line 126, in _get_build_requires
self.run_setup()
File "C:\Users\dalet\AppData\Local\Temp\pip-build-env-azzwbim1\overlay\Lib\site-packages\setuptools\build_meta.py", line 234, in run_setup
self).run_setup(setup_script=setup_script)
File "C:\Users\dalet\AppData\Local\Temp\pip-build-env-azzwbim1\overlay\Lib\site-packages\setuptools\build_meta.py", line 141, in run_setup
exec(compile(code, file, 'exec'), locals())
File "setup.py", line 47, in
stdout=open("/dev/null", "w"),
FileNotFoundError: [Errno 2] No such file or directory: '/dev/null'
ERROR: Command "'C:\Users\dalet\Anaconda3\envs\gluon_ts\python.exe' 'C:\Users\dalet\Anaconda3\envs\gluon_ts\lib\site-packages\pip_vendor\pep517_in_process.py' get_requires_for_build_wheel 'C:\Users\dalet\AppData\Local\Temp\tmpg8xig0yc'" failed with error code 1 in C:\Users\dalet\AppData\Local\Temp\pip-req-build-aqz77ara
Sent from Mail for Windows 10
From: Lorenzo Stella
Sent: Thursday, June 6, 2019 4:23 AM
To: awslabs/gluon-ts
Cc: DaleThomas68; Author
Subject: Re: [awslabs/gluon-ts] Tutorial [WinError 3] The system cannot findthe path specified (#75)
That's really weird, because tempfile should now use the right default temporary directory for windows (c:\temp or something like that).
Maybe something got cached? Did you try installing the master branch on a brand new environment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think there is a problem in the package build process on Windows. I’ll open a dedicated issue for that.
Good to hear that the fix works though, you will be able to properly get the upgraded package with pip install —upgrade once we release a new version.

hi, Could you please help me in this error:
Traceback (most recent call last):
File "C:/Users/mohammed/PycharmProjects/TensorFlowProject/tajrebe.py", line 30, in
ds_train = tf.keras.preprocessing.image_dataset_from_directory(
File "C:\Users\mohammed\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\keras\preprocessing\image_dataset.py", line 175, in image_dataset_from_directory
image_paths, labels, class_names = dataset_utils.index_directory(
File "C:\Users\mohammed\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\keras\preprocessing\dataset_utils.py", line 65, in index_directory
for subdir in sorted(os.listdir(directory)):
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'C:/Users\mohammed\PycharmProjects\TensorFlowProject\cancer_data\Train_subfolders/'


Document Notifications - Copy Address from Sales Document

Describe the request
In codeunit 1390 "Document Notifications",would it be possible to have 2 new events added to function CopyBuyFromVendorAddressFieldsFromSalesDocument and CopyPayToVendorAddressFieldsFromSalesDocument to mirror what is in place in the equivalent customer functions. It would be useful to have
OnCopyBuyFromVendorAddressFieldsFromSalesDocumentOnBeforeModify(Vendor,PurchaseHeader)
OnCopyPayToVendorAddressFieldsFromSalesDocumentOnBeforeModify(Vendor,PurchaseHeader)
Additional context
Useful to add consistency across the documents.
Internal work item: AB#498115

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 15815.


Geojson simplify interface

This could be powered by topojson and under the Meta tab.

Closing as idle. PRs are accepted that would completed this task but it is not one of my own priorities to implement it myself.


Mapbox Vector Tiles and ol.interaction.Select

There seems to be an issue with selecting features from VectorTile layers. On first selection, a TypeError is thrown and the select event is not fired.
Uncaught TypeError: target.addEventListener is not a function
    at Object.ol.events.listen (ol-debug.js:7335)
    at ol.source.Vector.setupChangeEvents_ (ol-debug.js:64883)
    at ol.source.Vector.addFeatureInternal (ol-debug.js:64859)
    at ol.source.Vector.addFeature (ol-debug.js:64842)
    at ol.source.Vector.<anonymous> (ol-debug.js:64998)
    at ol.Collection.boundListener (ol-debug.js:7208)
    at ol.Collection.ol.events.EventTarget.dispatchEvent (ol-debug.js:7628)
    at ol.Collection.insertAt (ol-debug.js:8271)
    at ol.Collection.push (ol-debug.js:8298)
    at ol.Collection.extend (ol-debug.js:8200)

After that, no more type errors are thrown and the select event works correctly. However, features are never highlighted automatically like they would be in a regular Vector layer.
I've put together an example here: https://codepen.io/anon/pen/xLojJM

ol.interaction.Select does not work with ol.render.Feature features, which is the default feature class used by ol.format.MVT. To make ol.interaction.Select work you need to configure the MVT format with {featureClass: ol.Feature}.
The reason why you do not get highlighting is because coordinates are in pixel space, at least in all releases up to date. With the next release, this will change. I have updated your codepen on top of master, which shows highlighting working already: https://codepen.io/ahocevar/pen/GvbYQv?editors=1010. However, highlighting is probably not what you'd expect in many cases, because highlighting is bound to the tile you clicked.

Thank you, it works for me after implementing like this
var vectorSource = new ol.source.VectorTile({     format: new ol.format.MVT({ featureClass: ol.Feature }),     url: ${vectorTileServer}/mh_district/{z}/{x}/{y},     maxZoom: 14,   })


build: add tslint rule to ensure that setters are declared after getters

Based on https://github.com/angular/material2/pull/11483#discussion_r190681292. Adds a custom tslint rule to enforce that we put getters before setters.

Marking as patch since I have seen crazy closure bugs before where doing things that seem totally safe turn out not to be


Fixed platform specific issues & API changes

I apologize for this pull request addressing more than one issue - but since I was under the impression that upstream work has ceased, I started collecting my personal fixes in my own fork's master, and depended on that for https://github.com/muesli/beehive
As a result of that, I can't easily force-push feature branches or squash commits in my fork, without breaking a bunch of users' clones.
I'd very much like to switch back to depending on the upstream anaconda, though. So here goes nothing, at the very least we could start a discussion on how to best handle that situation now. Seeing how the entire PR is still fairly straight forward and small, I'm still hoping we can agree to merge it.
Fixes included in this pull request:

[x] Removed "?" and "=" chars from filenames. These are illegal filename chars in some filesystems (in particular on Windows).
[x] Tweet's Contributors are an array of integer IDs. This caused an unmarshal panic.

Changes:

[x] Updated Twitter response entities.
[x] Let Travis test anaconda against Go 1.4 through tip, on Linux and OSX.

Alright, the AppVeyor build (Windows) which I just added is still going to fail even on this branch, because it'll fail to checkout the repository entirely. Once the Travis CI tests finish (they're still running), we can merge this and the AppVeyor tests should pass.
Would it be a problem if I squash-rebased this when merging, or do you need me to keep this fast-forwardable from your feature branch?

I'm assuming AppVeyor is still checking out the entire repository (including master) which would cause this problem. In that case merging should indeed fix the tests (at least for the master branch).
I'm fine with you squash-rebasing, since I can then switch back to depending on upstream. That being said, I do plan to further contribute to the project and I just loathe force-pushing to a master branch, so I would still prefer if you don't. In the future I'll stick to feature-branches.


Allow truncate_with_folder_marker to use multiple markers

Currently I use the truncate_with_folder_marker truncation strategy for my shell, however this leaves me only able to truncate by one kind of marker.  Right now I set POWERLEVEL9K_SHORTEN_FOLDER_MARKER=.git to truncate to the closest parent git directory.  However this means I am unable to truncate to other vcs systems or other project strutcures.
I would like to be able to provide multiple markers. For example, if I set POWERLEVEL9K_SHORTEN_FOLDER_MARKER=".git .hg .terraform" I would like it to find the closest parent directory that is a git repo, a mercurial repo, or a terraform project and truncate to that.

Hi @fishpen0 . Thanks for your feature request. This will be indeed possible in the upcoming Version.
You can use this, if you update to the next branch, our development branch.
To find multiple markers, you'll need to set P9K_DIR_SHORTEN_DELIMITER=(.git|.hg|.terraform). Syntax is like that, because we use zsh's globbing mechanism here.


MM-41952 : Unread channels names and sidebar item render on top of channel options menu

Summary
Removed animation-fill-mode css property on sidebar links for them to not overflow over to side bar menus
Ticket Link
https://mattermost.atlassian.net/browse/MM-41952
Related Pull Requests
N/A
Screenshots

before
after

Release Note

Channel links on hover overlapping channels menus fixed

@michelengelen sorry just saw that you are on vacation removing,

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/157932

📂 E2E report 📂
https://automation-dashboard.vercel.app/cycles/1ced06aa-89af-4f24-9914-a12522c06fac

/cherry-pick release-6.5

Cherry pick is scheduled.

/cherry-pick release-6.3

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#9918-upstream-release-6.3-1646663662
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#9918-upstream-release-6.3-1646663662'
Branch 'automated-cherry-pick-of-mattermost-webapp-#9918-upstream-release-6.3-1646663662' set up to track remote branch 'release-6.3' from 'upstream'.

+++ About to attempt cherry pick of PR #9918 with merge commit 606d028a9a833605ed17fc4de17ca614b87e9d85.

error: could not apply 606d028a9... MM-41952 : Unread channels names and sidebar item render on top of channel options menu (#9918)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU sass/layout/_sidebar-left.scss
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

/cherry-pick release-6.3

Cherry pick is scheduled.


Support attaching delta tables as catalogs

This draft PR adds support for attaching delta tables as catalogs.
This feature relies on several recent changes made to DuckDB recently. Therefore this PR is made to the feature branch of the delta extension.
How to use
With this PR, you can now do:
ATTACH './some/delta/table' AS my_delta_table (TYPE delta)
SELECT * FROM my_delta_table;

The attach statement allows setting the PIN_SNAPSHOT option, which will enforce caching the snapshot. Caching the snapshot will result in DuckDB only reading the snapshot once, which can offer a little performance boost at the cost of potentially querying an old snapshot:
ATTACH './some/delta/table' AS my_delta_table (TYPE delta, PIN_SNAPSHOT)
SELECT * FROM my_delta_table;

Performance implications
With this PR, using ATTACH should generally be as fast as, or faster, as using the delta_scan function. Note that even without using the PIN_SNAPSHOT function, using ATTACH can result in a significant performance increase when a table is scanned more than once in the same query, for example:
ATTACH './some/delta/table' AS my_delta_table (TYPE delta)
FROM my_delta_table UNION ALL FROM my_delta_table

will only scan the delta metadata once, whereas
FROM delta_scan('./some/delta/table') UNION ALL FROM delta_scan('./some/delta/table')

will scan the metadata twice
Optimizations in this PR
Initial naive implementation resulting in a lot of snapshot metadata being read unnecessarily.
Caching the Snapshot in the Transaction
Snapshots are cached in transaction, this both ensures that every time a delta table is scanned within the same transaction, not only will they see the same version of the table, the delta metadata is reused instead of re-scanning the table.
Sharing kernel snapshots
The raw ffi::SharedSnapshot that the delta kernel returns, is wrapped in a thread-safe wrapper. This means that when making a copy of a DuckDB DeltaSnapshot object, we simply copy the reference to the thread-safe wrapper instead of reconstructing the snapshot
Not emitting a FileExpandResult
DeltaSnapshot::GetExpandResult will no longer materialize the file list to return FileExpandResult::NO_FILES, FileExpandResult::SINGLE_FILE or FileExpandResult::MULTIPLE_FILES.
TODOs
This PR is still draft because of some remaining TODOs

Fix the current regression in the parquet reader in duckdb/duckdb that is causing this CI to fail
Fix an issue in DuckDB's clients where rebinding is causing the bind of a delta snapshot to be called too many times

what's the type of this item, is it a table or a view, that's seems like an external table to me ?
ATTACH './some/delta/table' AS my_delta_table (TYPE delta)

@djouallah great question! its sortof a new thing: it's a catalog with a default schema and table:
see https://github.com/duckdb/duckdb/pull/14118 where the feature is added to duckdb and
https://github.com/duckdb/duckdb_delta/blob/1261ea985c0b9707b3924bec2c64ddecd35efbb6/test/sql/dat/attach.test#L72 for how you can actually switch the default catalog to this delta catalog.
Note that this is the first step towards write support: through the newly added DeltaCatalog we can start adding appends, updates etc.

Ci Failure is expected: it's due to a regression test comparing to a previous version that doesn't support the attach statement


Release: v0.6.3.dev0

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[3.6.1] Room Editor: improved controls

Document zoom, pan, and how to call the context menu.

Added here: https://github.com/adventuregamestudio/ags-manual/wiki/EditorRoom/_compare/62332a5^...62332a5


python310Packages.cloup: 0.15.0 -> 0.15.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.cloup is: "Click extended with option groups, constraints, aliases, help themes"
meta.homepage for python310Packages.cloup is: "https://github.com/janLuke/cloup"
meta.changelog for python310Packages.cloup is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.15.1 with grep in /nix/store/pbc0g73s90053fvcg04mb2jvyrl17ali-python3.10-cloup-0.15.1
found 0.15.1 in filename of file in /nix/store/pbc0g73s90053fvcg04mb2jvyrl17ali-python3.10-cloup-0.15.1

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
manim
python310Packages.cloup
python39Packages.cloup

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pbc0g73s90053fvcg04mb2jvyrl17ali-python3.10-cloup-0.15.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.cloup https://github.com/r-ryantm/nixpkgs/archive/36508d32d90f38d47d81271f1bec3cf9b3ba4a87.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pbc0g73s90053fvcg04mb2jvyrl17ali-python3.10-cloup-0.15.1
ls -la /nix/store/pbc0g73s90053fvcg04mb2jvyrl17ali-python3.10-cloup-0.15.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

manim

2 packages built:

python310Packages.cloup
python39Packages.cloup

Maintainer pings
cc @friedelino for testing.

build of manim, for which cloup is a dependency fails due to failing tests while building cloup-dependency jupyterlab_server after commit  b9f50b7803a0d85bea6ef51ab3a75ec43add7a57 by @dotlambda. found out about this through git bisect.


Luca's PCB DogTag Changes

PCB Dog Tag - Luca Hanni

Nice work! You prob want to change your name's text box layer to top overlay (unless you want it written backwards on the back of the board)

You might want to make it bigger, only about 3cm long right now.


Add RUNC.md file for runc supported version

Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

ping @estesp
Does this look good to you?

Can you please put a link to this in README?

Codecov Report

Merging #878 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #878   +/-   ##
=======================================
  Coverage   59.55%   59.55%
=======================================
  Files           5        5
  Lines         759      759
=======================================
  Hits          452      452
  Misses        199      199
  Partials      108      108

Continue to review full report at Codecov.

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

LGTM


Fix #6567: missing checks for RAND_bytes()

Fix #6567: missing checks for RAND_bytes(). Function RAND_bytes() may return 0 or -1 on error, simply goto end label when it fails.

Hi, @mspncp
Thanks for your attention! It's my first time to contribute for such a large project, I have to admit that I'd ignored some of your coding styling. I've fixed the code and edited the commit body as you suggested. In addition, I've sent my CLA request to<EMAIL_ADDRESS>but haven't heard from the foundation yet. Am I missing something?

but haven't heard from the foundation yet. Am I missing something?

Maybe they did not react yet, because today is sunday  🌞.  Just give them another day :-). You will notice that they received it when the [need-cla] label gets removed.

Close/re-open to kick the CLA bot.

Looks good to me now.
Editorial note: the most common practice is to place the Fixes #nnnn note on a separate line at the end of the commit message body, see e.g. fb62e47c782397cadf607b92ce50f2bbe250d12e. But you don't have to change it anymore @Jason2031, I will do it for you when merging and pushing your pr to the repository.

Merged! Thank you very much @Jason2031 and congratulations to your first successfull pull request :-).


Add Disabling of IMDSv1 to Default Security Baseline

This contains a stackset to disable IMDSv1 for all instances without a specific tag (when IMDSv1 is required) with the ability to check the tag on an instance to set the requisite # of hops. The lambda is invoked hourly via CW events.
Description of changes:
let me know know the proper place to submit this code either here when we set up our next sync.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

adding second stackset to enforce ssl on s3 buckets dynamically


[WIP] Delete evm_server_db_backup_low_space event

Followup to deleting database backups, this removes an event that should only be triggered via db backups.  Will most likely require a migration to fully remove.

this is a revival of #21430

Do we need a schema migration for this as well, or is deletion of the event handled at seeding time?

Do we need a schema migration for this as well, or is deletion of the event handled at seeding time?

I don't see that in seed.  I think we'll need to remove any MiqEventDefinition of this type and likely remove their associated:
  has_many :miq_policy_contents
  has_many :policy_events

Not sure how you handle the unlikely situation that someone has a policy using this event definition.


CLI bundle: add debug and kernel options. fixes #1447

Improve the bundle command:

add an explicit --debug option, which is otherwise false by default
add a --kernel option to pass the main kernel file and treat the directory as the kernel namespace library.

Also enforces that any library, kernel or not, has at least one export.
Note: we can still build a library w/o exports using from_dir which does not use Library::new.

Also, to get the changelog CI to pass, you need to add an entry to CHANGELOG.md

The command also includes the stdlib now (for both kernel and normal libraries). This was necessary to bundle api.masm. Tested with:
cargo run --features executable -- bundle ~/miden/miden-base/miden-lib/asm/kernels/transaction/lib --kernel ~/miden/miden-base/miden-lib/asm/kernels/transaction/api.masm

Not sure if the test could be added to miden-base.


online-deployment issue "The Managed Inference service creation is taking longer than our normal time"

az --version
azure-cli   2.29.0 *
az extension list
[
{
"experimental": false,
"extensionType": "whl",
"name": "ml",
"path": "/home/icertisadmin/.azure/cliextensions/ml",
"preview": true,
"version": "2.0.2"
}
]
Description of issue (in as much detail as possible)
Command:- az ml online-deployment create --name blue --endpoint advanced-ner-endpoint-demo -f deployment.yml --all-traffic
Exception details
Creating or updating online_deployments
Check: endpoint advanced-endpoint exists
The deployment request bhushan-workspace-advanced-ner-endpoint-demo-8612672 was accepted. ARM deployment URI for reference:
https://ms.portal.azure.com/#blade/HubsExtension/DeploymentDetailsBlade/overview/id/%2Fsubscriptions%2F2b044625-9119-453a-8f50-53426430883b%2
Registering model version (ad2f42d3-xxxx-xxxx-be9f-2e20ef135c16 1 ) Done (1s)
Registering code version (c665e43f-xxxx-xxxx-a98d-a84911248e91 1 ) Done (1s)
Registering environment version (32890045-xxxx-48d3-a43c-6f2ec0323d08 1 ) Done (6s)
Creating deployment blue ................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................Code: ResourceDeploymentFailure
Message: The resource operation completed with terminal provisioning state 'Failed'.
Exception Details: (DeploymentTimedOut) The Managed Inference service creation is taking longer than our normal time.

route to service team

Acknowledged - beginning investigation and will reach out for more information if needed.

Hi @bhushangholave! We found an error stating that the operation timed out waiting for an image build to complete. If you check the Azure portal, are you able to find an image build error under operation details?

AzureML doesn't have access to the image build log because of the privacy. Though it is available for the user from the storage associated with the workspace or from the container registry associated with user workspace (ACR task runs). Timeout issue from the image build usually means poor environment specification and pip fails to resolve the conflicts in user dependencies with a reasonable time.
Solution from the user end would be to inspect the dependencies and fix conflicts. Should be easily repro-ed locally by materializing environment from conda spec.
Temporary workaround that might work is to pin pip<=20.2.4 to use older resolver that ignores some of the conflicts

@vs-li and @vizhur  thank you for your comments.
Looks like some timeout issue to me but it reproducing every time.
Error details in ARM deployments are
{
    "status": "Failed",
    "error": {
        "code": "DeploymentTimedOut",
        "message": "The Managed Inference service creation is taking longer than our normal time.",
        "details": [],
        "additionalInfo": []
    }
}

@vs-li In Azure portal Environment tab it is still stuck at installing pip dependencies.
Same environment is working on our other workspace.

2021/11/11 10:05:42 Downloading source code...
2021/11/11 10:05:43 Finished downloading source code
2021/11/11 10:05:43 Creating Docker network: acb_default_network, driver: 'bridge'
2021/11/11 10:05:44 Successfully set up Docker network: acb_default_network
2021/11/11 10:05:44 Setting up Docker configuration...
2021/11/11 10:05:44 Successfully set up Docker configuration
2021/11/11 10:05:44 Logging in to registry: 02f210dd14eb4df1a8d3ac2f948c3ede.azurecr.io
2021/11/11 10:05:45 Successfully logged into 02f210dd14eb4df1a8d3ac2f948c3ede.azurecr.io
2021/11/11 10:05:45 Executing step ID: acb_step_0. Timeout(sec): 5400, Working directory: '', Network: 'acb_default_network'
2021/11/11 10:05:45 Scanning for dependencies...
2021/11/11 10:05:46 Successfully scanned dependencies
2021/11/11 10:05:46 Launching container with name: acb_step_0
Sending build context to Docker daemon  68.61kB

Step 1/19 : FROM mcr.microsoft.com/azureml/openmpi3.1.2-ubuntu18.04:latest@sha256:fda0c9dd1702f56aa5ce42ebeeda6af6b768d1cb18c04380af3ded4184eedb09
mcr.microsoft.com/azureml/openmpi3.1.2-ubuntu18.04:latest@sha256:fda0c9dd1702f56aa5ce42ebeeda6af6b768d1cb18c04380af3ded4184eedb09: Pulling from azureml/openmpi3.1.2-ubuntu18.04
284055322776: Already exists
4c683b87b3fa: Pulling fs layer
1577eaefa27a: Pulling fs layer
8b473b71c2c4: Pulling fs layer
7f9f8de2dc06: Pulling fs layer
f8839526f541: Pulling fs layer
f3bffb23c6c4: Pulling fs layer
ecddeb6524fc: Pulling fs layer
a417bd11e4b2: Pulling fs layer
7f9f8de2dc06: Waiting
f8839526f541: Waiting
f3bffb23c6c4: Waiting
ecddeb6524fc: Waiting
a417bd11e4b2: Waiting
8b473b71c2c4: Verifying Checksum
8b473b71c2c4: Download complete
1577eaefa27a: Verifying Checksum
1577eaefa27a: Download complete
f8839526f541: Verifying Checksum
f8839526f541: Download complete
f3bffb23c6c4: Verifying Checksum
f3bffb23c6c4: Download complete
ecddeb6524fc: Verifying Checksum
ecddeb6524fc: Download complete
a417bd11e4b2: Verifying Checksum
a417bd11e4b2: Download complete
4c683b87b3fa: Verifying Checksum
4c683b87b3fa: Download complete
7f9f8de2dc06: Verifying Checksum
7f9f8de2dc06: Download complete
4c683b87b3fa: Pull complete
1577eaefa27a: Pull complete
8b473b71c2c4: Pull complete
7f9f8de2dc06: Pull complete
f8839526f541: Pull complete
f3bffb23c6c4: Pull complete
ecddeb6524fc: Pull complete
a417bd11e4b2: Pull complete
Digest: sha256:fda0c9dd1702f56aa5ce42ebeeda6af6b768d1cb18c04380af3ded4184eedb09
Status: Downloaded newer image for mcr.microsoft.com/azureml/openmpi3.1.2-ubuntu18.04:latest@sha256:fda0c9dd1702f56aa5ce42ebeeda6af6b768d1cb18c04380af3ded4184eedb09
---> 02c62b48927a
Step 2/19 : USER root
---> Running in 9dacaa1d14ad
Removing intermediate container 9dacaa1d14ad
---> 6fde3ca80c55
Step 3/19 : RUN mkdir -p $HOME/.cache
---> Running in 55edcd44085d
Removing intermediate container 55edcd44085d
---> f8c35f9a9cb8
Step 4/19 : WORKDIR /
---> Running in 2e91d46f7cc4
Removing intermediate container 2e91d46f7cc4
---> 907a8123f6b3
Step 5/19 : COPY azureml-environment-setup/99brokenproxy /etc/apt/apt.conf.d/
---> df36fc224f0c
Step 6/19 : RUN if dpkg --compare-versions conda --version | grep -oE '[^ ]+$' lt 4.4.11; then conda install conda==4.4.11; fi
---> Running in b09508eaf72e
Removing intermediate container b09508eaf72e
---> 43c921cbf924
Step 7/19 : COPY azureml-environment-setup/mutated_conda_dependencies.yml azureml-environment-setup/mutated_conda_dependencies.yml
---> c14f668617bf
Step 8/19 : RUN ldconfig /usr/local/cuda/lib64/stubs && conda env create -p /azureml-envs/azureml_996e6ced9a072187393391a560ac11f8 -f azureml-environment-setup/mutated_conda_dependencies.yml && rm -rf "$HOME/.cache/pip" && conda clean -aqy && CONDA_ROOT_DIR=$(conda info --root) && rm -rf "$CONDA_ROOT_DIR/pkgs" && find "$CONDA_ROOT_DIR" -type d -name pycache -exec rm -rf {} + && ldconfig
---> Running in 7c1f90ce87ae
Collecting package metadata (repodata.json): ...working... done
Solving environment: ...working... done
Downloading and Extracting Packages
readline-8.1         | 362 KB    |            |   0%
readline-8.1         | 362 KB    | ########## | 100%
_libgcc_mutex-0.1    | 3 KB      |            |   0%
_libgcc_mutex-0.1    | 3 KB      | ########## | 100%
ca-certificates-2021 | 115 KB    |            |   0%
ca-certificates-2021 | 115 KB    | ########## | 100%
ncurses-6.3          | 782 KB    |            |   0%
ncurses-6.3          | 782 KB    | ########## | 100%
ncurses-6.3          | 782 KB    | ########## | 100%
six-1.16.0           | 18 KB     |            |   0%
six-1.16.0           | 18 KB     | ########## | 100%
tk-8.6.11            | 3.0 MB    |            |   0%
tk-8.6.11            | 3.0 MB    | ########## | 100%
tk-8.6.11            | 3.0 MB    | ########## | 100%
scikit-learn-1.0.1   | 5.5 MB    |            |   0%
scikit-learn-1.0.1   | 5.5 MB    | ########## | 100%
scikit-learn-1.0.1   | 5.5 MB    | ########## | 100%
ld_impl_linux-64-2.3 | 586 KB    |            |   0%
ld_impl_linux-64-2.3 | 586 KB    | ########## | 100%
intel-openmp-2021.4. | 4.2 MB    |            |   0%
intel-openmp-2021.4. | 4.2 MB    | ########## | 100%
intel-openmp-2021.4. | 4.2 MB    | ########## | 100%
numpy-base-1.21.2    | 4.8 MB    |            |   0%
numpy-base-1.21.2    | 4.8 MB    | ########## | 100%
numpy-base-1.21.2    | 4.8 MB    | ########## | 100%
threadpoolctl-2.2.0  | 16 KB     |            |   0%
threadpoolctl-2.2.0  | 16 KB     | ########## | 100%
joblib-1.1.0         | 211 KB    |            |   0%
joblib-1.1.0         | 211 KB    | ########## | 100%
blas-1.0             | 6 KB      |            |   0%
blas-1.0             | 6 KB      | ########## | 100%
libgfortran-ng-7.5.0 | 22 KB     |            |   0%
libgfortran-ng-7.5.0 | 22 KB     | ########## | 100%
setuptools-58.0.4    | 775 KB    |            |   0%
setuptools-58.0.4    | 775 KB    | ########## | 100%
mkl-2021.4.0         | 142.6 MB  |            |   0%
mkl-2021.4.0         | 142.6 MB  | 6          |   6%
mkl-2021.4.0         | 142.6 MB  | #          |  10%
mkl-2021.4.0         | 142.6 MB  | #8         |  18%
mkl-2021.4.0         | 142.6 MB  | ##2        |  22%
mkl-2021.4.0         | 142.6 MB  | ###1       |  31%
mkl-2021.4.0         | 142.6 MB  | ####       |  40%
mkl-2021.4.0         | 142.6 MB  | ####8      |  49%
mkl-2021.4.0         | 142.6 MB  | #####7     |  58%
mkl-2021.4.0         | 142.6 MB  | ######6    |  66%
mkl-2021.4.0         | 142.6 MB  | #######4   |  75%
mkl-2021.4.0         | 142.6 MB  | ########3  |  84%
mkl-2021.4.0         | 142.6 MB  | #########2 |  93%
mkl-2021.4.0         | 142.6 MB  | ########## | 100%
zlib-1.2.11          | 103 KB    |            |   0%
zlib-1.2.11          | 103 KB    | ########## | 100%
libffi-3.3           | 50 KB     |            |   0%
libffi-3.3           | 50 KB     | ########## | 100%
wheel-0.37.0         | 33 KB     |            |   0%
wheel-0.37.0         | 33 KB     | ########## | 100%
libgcc-ng-9.3.0      | 4.8 MB    |            |   0%
libgcc-ng-9.3.0      | 4.8 MB    | ########## | 100%
libgcc-ng-9.3.0      | 4.8 MB    | ########## | 100%
sqlite-3.36.0        | 990 KB    |            |   0%
sqlite-3.36.0        | 990 KB    | ########## | 100%
libgfortran4-7.5.0   | 995 KB    |            |   0%
libgfortran4-7.5.0   | 995 KB    | ########## | 100%
_openmp_mutex-4.5    | 22 KB     |            |   0%
_openmp_mutex-4.5    | 22 KB     | ########## | 100%
mkl_fft-1.3.1        | 172 KB    |            |   0%
mkl_fft-1.3.1        | 172 KB    | ########## | 100%
libgomp-9.3.0        | 311 KB    |            |   0%
libgomp-9.3.0        | 311 KB    | ########## | 100%
xz-5.2.5             | 341 KB    |            |   0%
xz-5.2.5             | 341 KB    | ########## | 100%
python-3.7.11        | 45.3 MB   |            |   0%
python-3.7.11        | 45.3 MB   | ##2        |  22%
python-3.7.11        | 45.3 MB   | ####9      |  49%
python-3.7.11        | 45.3 MB   | #######6   |  76%
python-3.7.11        | 45.3 MB   | #########8 |  98%
python-3.7.11        | 45.3 MB   | ########## | 100%
mkl_random-1.2.2     | 287 KB    |            |   0%
mkl_random-1.2.2     | 287 KB    | ########## | 100%
mkl-service-2.4.0    | 56 KB     |            |   0%
mkl-service-2.4.0    | 56 KB     | ########## | 100%
pip-21.2.2           | 1.8 MB    |            |   0%
pip-21.2.2           | 1.8 MB    | ########## | 100%
pip-21.2.2           | 1.8 MB    | ########## | 100%
libstdcxx-ng-9.3.0   | 3.1 MB    |            |   0%
libstdcxx-ng-9.3.0   | 3.1 MB    | ########## | 100%
libstdcxx-ng-9.3.0   | 3.1 MB    | ########## | 100%
scipy-1.7.1          | 16.4 MB   |            |   0%
scipy-1.7.1          | 16.4 MB   | ######7    |  67%
scipy-1.7.1          | 16.4 MB   | ########## | 100%
scipy-1.7.1          | 16.4 MB   | ########## | 100%
openssl-1.1.1l       | 2.5 MB    |            |   0%
openssl-1.1.1l       | 2.5 MB    | ########## | 100%
openssl-1.1.1l       | 2.5 MB    | ########## | 100%
numpy-1.21.2         | 23 KB     |            |   0%
numpy-1.21.2         | 23 KB     | ########## | 100%
certifi-2021.10.8    | 151 KB    |            |   0%
certifi-2021.10.8    | 151 KB    | ########## | 100%
Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working...
Installed package of scikit-learn can be accelerated using scikit-learn-intelex.
More details are available here: https://intel.github.io/scikit-learn-intelex

For example:

    $ conda install scikit-learn-intelex
    $ python -m sklearnex my_application.py

done
Installing pip dependencies: ...working...

@bhushangholave  you have dependencies conflict. Unfortunately conda swallows pip's log and you cannot see pips struggles. if you pip install -r requirements.txt with those deps with the latest pip you should see where pip gets stuck. There are a bunch of tuckets for pip for those kind of issues, feel free to file one too.
just to unblock yourself you can try to pin pip<=20.2.4 to use old resolver that ignores some of the conflicts

@vizhur you are right, corrected my dependencies file and it worked. But portal or deployment creation needs to throw correct error and logs for debug.

@bhushangholave Since the work around helped. Closing this thread for now.


Online editor not working

The online editor at https://bootstrapemail.com/editor currently does not appear to be working. When clicking Rerender (even without changing the default email), the following error is returned, see also the screenshot:

The change you wanted was rejected.
Maybe you tried to change something you didn't have access to.
If you are the application owner check the logs for more information.

I haven't been able to reproduce this error. Are you still seeing the issue?

Hi,
Yes, the isuse is still there for me. I tried Chrome and Safari, both normal and incognito, with and without adblock. Same issue in all cases, as posted initially.

Update: It seems to work on another machine without any issues.
So this might be a local problem rather than something wrong with the online editor.

Ill do some investigating, thank for you the feedback Ill look into it 👍

I just had the same issue when using Safari on macOS. I then switched to Firefox and it was fine.

The root cause of the issue is that 3rd-party cookies disabled.
If one enables cookies from bootstrap-email.herokuapp.com problem disappears.


docs: add rst versions of java training slides

Follow up to #1725 for Java slides.
Key points
I've pulled out reusable content into snippets in the following places:

intro to local data flow
intro to global data flow
path query information slide

In cases where the original google slide could not be replicated in rst, I've worked around it by saving that slide as an image. It is then pulled into the slide deck using a custom rst-class directive.
I have added a short template slide deck, which contains most of the information required to create new slides.
~Note that I haven't fixed the bug in the speaker notes yet-some links remain unclickable.~
EDIT: I seem to have fixed the speaker notes bug. Links are behaving as expected at the moment.
You can view the output here

@felicity-semmle please could you review the text for typos and obvious errors if you have a moment? Don't worry about the content too much please. I'm going to make it clear that these slides are primarily used in our training sessions when I rework the training homepage on help.semmle.com.
Thanks

Thanks for the review @felicity-semmle. Comments addressed.
I'll leave the directory renaming I mentioned earlier today out of this PR in order to avoid any confusion.

@lukecartey: after felicity has approved the latest updates i think this should be pretty much ready to merge

Thanks @lukecartey. I've updated the image slides so that the correct message is displayed in the footer, as discussed offline.


improve bottom padding on page indicator in setup screens

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @tneot01 in #requests.

Done on nullsafety branch


Possible change to tag page - include description?

Hey,
I was checking out the Ghost docs and noticed that we now have a tag manager, and a way to add descriptions, images, and SEO data to tags. Do you think it's worth incorporating the tag description on the /tag/tag-name page?
Default to post count, but if there is a description that is shown instead:

tag with descritption
tag with empty description

What do you think?

Yep, I like it. Feel free to make a PR! Good catch! :)

Created!

Closed in #17


W0D2_Tutorial1, bug while defining function plot_all()

Within the third cell under the 3 Setup title, while defining function plot_all(), at line 46:
plt.scatter(t_range, v_n[j], color="k", marker=".", alpha=0.01)

here variable v_n is neither an input variable nor a variable defined in the function, therefore it must be a typo for v. It works fine in this notebook, but won't function normally if v_n doesn't exist or is not equal to the input of the function.

Excellent, thank you so much @SidongWang. That bug was spotted and corrected in an earlier version, but apparently didn't make it to the final version. Please keep us posted on any other issue you may come across.

@mpbrigham OK to close?


refactor(starknet_sequencer_node): rename component execution config

Stack:

#2578
#2577
#2576
#2575
#2574 ⬅
#2573
#2544

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #2575


[tensorrt] check CUDA compute capability.

Change-Id: I80bf42f0d28f0b651c3dfbd0a2ba89173939a534
Description
Brief description of what this PR is about

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

Codecov Report

Merging #1284 (14565a2) into master (bb5073f) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1284      +/-   ##
============================================
- Coverage     72.08%   72.05%   -0.04%
+ Complexity     5126     5125       -1
============================================
  Files           473      473
  Lines         21970    21970
  Branches       2351     2351
============================================
- Hits          15838    15830       -8
- Misses         4925     4935      +10
+ Partials       1207     1205       -2

Impacted Files
Coverage Δ

...gine/src/main/java/ai/djl/dlr/engine/DlrModel.java
51.61% <0.00%> (-3.23%)
:arrow_down:

api/src/main/java/ai/djl/inference/Predictor.java
88.46% <0.00%> (-2.89%)
:arrow_down:

api/src/main/java/ai/djl/util/Platform.java
69.81% <0.00%> (-1.89%)
:arrow_down:

...ain/java/ai/djl/tensorflow/engine/TfNDManager.java
88.14% <0.00%> (-1.49%)
:arrow_down:

...ngine/src/main/java/ai/djl/mxnet/jna/LibUtils.java
40.00% <0.00%> (-1.18%)
:arrow_down:

.../djl/training/listener/MemoryTrainingListener.java
76.62% <0.00%> (ø)

...rc/main/java/ai/djl/paddlepaddle/jni/LibUtils.java
48.27% <0.00%> (ø)

api/src/main/java/ai/djl/repository/Artifact.java
88.70% <0.00%> (+0.80%)
:arrow_up:

Continue to review full report at Codecov.

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


BUG: Inaccurate pixel ratio for small screens.

G6 Version: 3.5.3
Platform: Windows 10
Mini Showcase(like screenshots):
On small screens, the image looks blurry.
This is better seen when zooming in on the browser, see screenshot. (increase in the screenshot by 300%)

Screen resolution: 1600x1200

The situation is better when using renderer: 'svg'

Try to set the pixelRatio manually to see if it is improved:
graph.get('canvas').set('pixelRatio', x); // x is the pixelRatio number

@Yanyan-Wang this is a solution to the problem!


Readme, config file, missing config, config file

Adds to README.md:

missing config properties
*little example of a typedoc.js configuration file
reference to typedoc-plugin-markdown
adds a little description in --mode taken from above comment that  describes what mode=file or modules mean

travis was already broken right ? Because I didn't tough package.jsons files

These updates look great. Could we add the command descriptions to the code as well?

I took a look at the changes and it looks like they have been superseded by work done in other PRs, the read me has been shortened and now points users to the site for detailed descriptions.
Thanks for the contribution!


Update docs

Coverage remained the same at 99.551% when pulling 24a71ecc788cf8f372e253997498d54c5616fc22 on prtksxna:docs into be75deefdd341463a73c9b53b726d9612d319acc on winkjs:master.


Unexpected token

Vue2, Webpack2 error:
Unexpected token: operator (>) [./~/vue-strap/src/utils/utils.js:4,0]

how did you solve this?

var alert = require('vue-strap').alert;
but it will increase the code 200 kb, so i wrote own component 4 this


Add an example scenario in the Readme

Hi, not so much an issue, but a way to figure out if I'm doing something wrong
I installed autokbisw and I see no difference with vanilla. I'm not sure if it isn't working properly or if I am doing something wrong.
There are no information in the readme about what is supposed to happen and when.
I imagine it's supposed to do something like that:

start typing with a keyboard 1 to make it active
select input source 1 for that keyboard
start typing with keyboard 2
select input source 2 for that keyboard

Now, when you type with keyboard 1, input source 1 is used (you can see it change on the upper right) and when using keyboard 2, input source 2 is used.
That's what I imagine is supposed to happen (but nothing happens here for me), but I can't be sure without a description.

PR #11 fixes this Issue

Good luck with the bugfix

Hi @rdelassus, thanks for the PR, I'll look into it later today. I (hopefully) fixed the issues with the app, please update the Homebrew tap and try again (brew install ohueter/tap/autokbisw and brew services start ohueter/tap/autokbisw). Make sure Homebrew installs the version 1.4.3 release. Please let me know if it works!

Great to hear that at least something is happening now :-)
On my Mac (after enabling the control + space shortcut, didn't know about that!), it works as expected: When typing on my external US-layout keyboard, autokbisw switches to US layout. When pressing Ctrl+Space, MacOS switches to DE layout and autobisw saves DE as layout for this keyboard. Same when going back to US. There is no jumping between the layouts.
What I would recommend trying:

use the graphical switcher in the menu bar for initial setup to avoid pressing keys and triggering autokbisw
disable Karabiner temporarily to avoid any interference

Everything works as expected when Karabiner is shutdown, and stops working when karabiner is started.
Thank you for the quick fix

You're welcome! Glad that you could figure it out. I'll close this issue then.


Small tweaks to sampling from model in vae

Uses the poutine mechanism to extract the model latent, instead of actually running the decoder
it's certainly fewer lines of code, though more machinery to work with to extract something you can do with a helper function. not sure what the 'right' approach is

return the bernoulli probs as one of its return values

yeah i like that better, though it sort of violates the idea of model taking parameters and returning data

there is no such contract

it's the right way to think about generative models imo. in any case, i updated the PR to reflect that


Metrics with labelvalues are not collected

Metrics with labelvalues are not collected. In the example below, I would expect all 3 counters to be collected, but the one with labelvalues is not.
from prometheus_client import REGISTRY, Counter

c1 = Counter("c1", "c1")
c2 = Counter("c2", "c2", labelnames=["l1"])
c3 = Counter("c3", "c3", labelnames=["l1"], labelvalues=["v1"])

c1.inc()
c2.labels("v1").inc()
c3.inc()

for item in REGISTRY.collect():
    print(item)  # prints c1 and c2, but not c3

That should be producing an exception, that's invalid usage.

Where should I learn about the intended usage?
sob., 13.04.2019, 11:44 użytkownik Brian Brazil<EMAIL_ADDRESS>napisał:

That should be producing an exception, that's invalid usage.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/client_python/issues/398#issuecomment-482794379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8NdtJxKO88W88S99VSe-GTqMv2TYkFks5vgacagaJpZM4ct8rw
.

https://github.com/prometheus/client_python#labels

It does not mention labelvalues. If labelvalues are not for passing values
for labels and my example is invalid usage, the issue should be that
labelvalues are not intuitive and not documented.
sob., 13.04.2019, 12:35 użytkownik Brian Brazil<EMAIL_ADDRESS>napisał:

https://github.com/prometheus/client_python#labels
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/client_python/issues/398#issuecomment-482797607,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8Ndpazs9s3u--8jWNSBzLNbz-EAWseks5vgbMPgaJpZM4ct8rw
.

Non-existent features tend not to be documented.

I meant the labelvalues argument. It is an argument of a constructor of a
public class that is not documented. It would be fine if there was only one
reasonable behavior given its name and that was how it worked, but this is
not the case.
I have an answer now, but I won't close the issue - you may want to keep it
open as a reminder that the labelvalues parameter could use a docstring.
Otherwise close it.
sob., 13.04.2019, 14:54 użytkownik Brian Brazil<EMAIL_ADDRESS>napisał:

Non-existent features tend not to be documented.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/client_python/issues/398#issuecomment-482806660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8NdjqoHroupTSfqWuI-fTRsxNVtwmOks5vgdOEgaJpZM4ct8rw
.

It is an argument of a constructor of a public class that is not documented.
I agree with @bm371613 here. It is easily assumed that labelvalues is meant for accepting default values for labels, and there is no indication that this is wrong -- like @brian-brazil said, it doesn't raise an exception.

I'm not sure what the intended use of labelvalues is, but it would be a useful feature to have the ability to set "default" labels or a base set of labels for all child metrics. Support for chaining calls to .labels() would probably be required too.

labelvalues is for internal use only. It is used in the MetricWrapperBase.labels() method to create a new MetricWrapperBase with labelvalues set. Since the metric already was registered, there's an if statement in MetricWrapperBase's constructor that prevents registration of the metric if labelvalues is set.
So, that's why you're seeing the behaviour that you are. The user was never supposed to use the labelvalues argument of MetricWrapperBase in the first place.
This is terrible and it should be fixed! One way to do it is add an internal argument, register=True, to MetricWrapperBase. Then, MetricWrapperBase.labels() initiates the child MetricWrapperBase with _register=False. Then, labelvalues will have the obvious and assumed behaviour.
Incidentally, if this is what you want, initialise c3 like this instead:
c3 = Counter('c3', 'c3', labelnames=['l1']).labels('v1')

IMO the current situation is quite confusing and quite time consuming. What do you feel about adding a couple of lines in the documentation to warn about this argument? Or perhaps just a warning for users?

I'm not sure how changing the user docs will help if users are ignoring the docs and instead digging into the source and using undocumented internal APIs.

In this case it's not clear that it's an internal API. Labelvalues is a kwarg on a public constructor, and it looks like it fills a need that the end user often feels (setting a default value for a Gauge's labels). If it's only meant to be internal then yeah maybe you're right, the best place to make hanger might not be in the documentation.
Renaming it by prepending a "_" is a common pythonic way of marking something as internal. Maybe that?

That sounds reasonable.
I'll note that a default value for a label doesn't really make sense, you need to have code to provide the actual label value one way or the other.

@belm0 That is unrelated to this issue.

It's related because the people attempting to use labelvalues are wanting precisely this functionality, as the OP concedes.

No, they're not looking for initialisation of childs. Please don't confuse the issue.


Great work, come on!

I may use this project to try to transform a traditional desktop application. It suits me very well.Great work, come on!

Thank you for the kind words! If you have any further feedback or suggestions please let us know!


ISSUE 58: Added logo in assets folder

Describe your changes
Created the logo and added it to the assets folder in public
Issue ticket number and link
#58
Name of Contributor
@rickansen
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My code does not generate any new warnings or errors
[ ] New and existing unit tests pass locally with my changes (if relevant)

Fantastic work on that Logo Anderson. It will make our project stand out for sure!


msl-cli: in verbose mode, print the full HTTP request and response

With -v true, it prints something like:
{"headerdata":"X","signature":"Y","entityauthdata":{"scheme":"PSK","authdata":{"identity":"I"}}}{"headerdata":"X","signature":"Y","entityauthdata":{"scheme":"PSK","authdata":{"identity":"I"}}}{"payload":"P","signature":"S"}{"payload":"P","signature":"S"}
Error: Server returned HTTP response code: 400 for URL: http://xyz/uri

But the full HTTP request actually looks like this:
POST /scriptManager?hash=None&hasChronosSync=true&action=RUN&v=1.1&debug=true HTTP/1.1
User-Agent: Java/1.8.0_60
Host: <IP_ADDRESS>:7001
Accept: text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
Connection: keep-alive
Content-type: application/x-www-form-urlencoded
Content-Length: 1458

{"headerdata":"X","signature":"Y","entityauthdata":{"scheme":"PSK","authdata":{"identity":"I"}}}{"payload":"P","signature":"S"}

Missing:

HTTP verb and version
HTTP headers for the request
HTTP headers for the response
HTTP payload of the response

To clarify the request: support output of the underlying transport raw data as part of the CLI, independent of the MSL data which is currently supported by the MSL core code.


Introduce the SliceAtLeast wrapper

This PR introduces a slice wrapper that can only be constructed from a slice that is at least the specified const length. This type is used in the merging function with a length of at least 2 (i.e. SliceAtLeast<T, 2>).
It helps library users in avoiding potential issues with the number of elements to merge, it is now verified that the number of values to merge is at least 2.

Not that important in fact.


first attempt at generating json output from yotta ls

There are still some holes in the implementation, but it seems to work.

:thumbsup:
In the long run I'd like to rearrange the formatting so that it generates the JSON internally and then formats that in whatever way for display.
I might tweak some of the property names in the json before publishing.


Look at a search or filter on the main card page

this would let me show only Subtype Cards or cards with a key value of X

Searching enhancements will include:

[ ] Breadcrumb on each card
[ ] A filter on all element types
[ ] The table of contents hierarchy should be maintained when searching

@previnWong Let's update the search functionality to search the TOC on the click of a search icon and not be dynamic to make the user experience smoother.

.


WGSL: Allow imports anywhere

Permit using an import anywhere within the file.
Note that only certain locations are supported by wgsl-analyzer. However, this is better than requiring them also to be on newlines. If you need a new place, we can probably add it to wgsl-analyzer.
We intentionally only support ifdef etc. at the start of a line, to avoid having to think too hard about it.
The preprocessor also doesn't have any tests, so if something here looks wrong it probably is.

Can we expect wgsl-analyzer to also be able to handle #import in the middle of a line?

In the sense that it's theoretically possible, yes. At the moment, the actual places are quite limited, but for any other places you need, adding such support should be relatively feasible

Hmm, that means actually using imports for constants will result in a red squiggle?

Yes exactly, but if you poke me on the zulip I'd guess it should take maybe 30 minutes per syntactic location.
I can only think that we need some attributes support (potentially just top level attribute to avoid having to think about nesting). Apparently parameter declarations was also needed - that supports already exists, but I'm not sure why.
You should already be able to import global constants, which is equivalent to having it be in any other place you can refer to constants, just with slightly more indirection.
I think everything else should be equivalently constructable from these cases.
I'll merge now though


Sfm plugin

maybe wait for https://github.com/voshch/arena-rosnav/pull/2
I just merged master into this, should be merge-able to master seamlessly.

Full functionality depends on arena-simulation-setup cleanup, otherwise anything derived from RandomTask won't work

random, scenario, random scenario, staged working well, pedestrian can be detected by laser, actors moving
Problems:
add installation of lightsfm in utils into installation docs
rviz shuts down after a few episodes, gazebo continues to run
shuts down after about 5 episodes although reset count in scenario set higher


add helm feature - [closed]

In GitLab by @tmon546596046 on Feb 19, 2019, 08:26
Merges helm -> dev
add helm feature

In GitLab by @wknet123 on Nov 13, 2019, 11:37
The test coverage for backend is 70.5%  38.96%  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:37
The test coverage for backend is %  38.93%  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:37
The test coverage for backend is %  %  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:37
The test coverage for backend is %  39%  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:37
The test coverage for backend is %  39%  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:37
closed


First JOIN always fails with TTN

For some reason, the first JOIN sent by the client always fails with TTN. According to the TTN console, the JOIN is forwarded to the LNS, but there is no response. The second JOIN usually succeeds. Investigate why.

A duplicate of #25


Ci yarn

After #58

We should probably add a step which runs prettier and fails if changes are suggested.

We should probably add a step which runs prettier and fails if changes are suggested.

Total agreement, but I'd like to do that in another PR because otherwise, I have to prettify the whole codebase now, which I'd rather reserve for that next PR.