Deprecated package

Greetings gentlemen,
Atom Version: 0.207.0
System: Windows 8 x64
Thrown From: gitter package, v0.6.2

I am no longer maintaining Atom Gitter. I would be open to reviewing a pull request to fix these deprecation warnings.


Parameter Not Used reported, although its being used in an Assertion Rule

cfn-lint version: (cfn-lint --version)
0.44.5 (also already ran, cfn-lint -u)
Description of issue.
Using the AWS Quick Start ad-2.template, cfn-lint is reporting VPCID Parameter not used.  Although it's actually being used in the SubnetsInVPC rule.

We currently don't have a lot of logic around the rules section.  We will need to build that out so we can handle this scenario.

This issue is no longer appearing


expose I_default and TwelveHourClock

Needed to customize rendering of "AM" and "PM"

Well spotted, so easy to forget to expose something as I've discovered in past.
Ill look into publishing an update in the next few days.

Thanks and published in 9.2.0.

thank you!


Every time I load into a world the loading screen freezes and the only way to get out of it is to restart my computer

is anyone else having this issue?

It would be useful if you provided your log file and list of mods (with their version)

I have basically the same issue, I tested it 3 or 4 different times and all the same thing, it seems to stop loading near the end of the percentage range of 80% to 91% but all seem to freeze up at those points, I've removed two mods total (Haema(1.7.6+1.17) & campanion(2.0.0-beta.1)) and downgraded origins from 1.0.3 to 1.0.2 to try to figure it out. Still the same results. How to reproduce it: Create a world, load in, play on it for a bit, then exit and try to rejoin it.. I have no idea if this is a minecraft ram issue (which I can more than likely easily fix that), A Fabric issue or fabric api, or a minecraft bug. Perhaps I am doing something wrong as I have just basically swapped over from forge so I am new to fabric..
Here are my lists of mods
-Caelus-Fabric-0.0.17-1.17
-Cloth Config-5.0.38-Fabric
-Environmental Armor-1.17.1-1.4.1
-Extra Origins-1.17-6
-Fabric Api-0.40.1+1.17
-Fabric Language Kotlin-1.6.4+kotlin.1.5.30
-Voxelmap-1.10.15 for 1.17.0
-Icarus-1.3
-MCDW(minecraft dungeon weapons)-3.4.2
-Medieval Origins-1.2.0
-Mer mod-1.8
-Origins 1.17-102
-Origins Classes-1.17-1.2.2
-Pehkui-2.4.0+1.14.4-1.17.1
-Repurposed Structures-Fabric-2.3.3+1.17.1
-Terra-fabric-5.4.1-BETA+40e95073-shaded-mapped
-Trinkets-3.0.4
Here are my logs
latest.log
2021-09-23-4.log

It seems this isn't the only thread with this issue.. Looking at issue#432


(PUP-7179) Resolve Pcore types using the same loader that loaded the type

When resolving a type (Object, Alias, or TypeSet), it is essential that
the loader that initially loaded the type is used. This means that
if a type A::X in module a references type B::Y i module b, which
in in turn references a type C::Z in module c, then the loader for
module a must be used when resolving A::X, the loader for module b
must resovle B::Y and finally, the loader for module c resolves
C::Z.
In essence, this means that all loaders must be found by the loader names
adapted to the source of the AST that represents the type.
Fixing this resulted in cleaner code. It is now never necessary to pass
a scope to the TypeParser and only tests that lacks a fully configured
Loaders instance needs to pass a loader.

This is good to merge once appveyor is green.


Extending Observable

I think we're mature enough to start looking at this closer. As requested before (#1124, #1207, #1353, #1757), extending Observable should be seamless. Most of the time extending Observable and overriding lift does the job, but there are a few cases this breaks down.
If this sounds reasonable, I'll start work on a PR in the next few days.
Operators that call static methods
These are combineLatest, concat, merge, race, and zip. Implementing these with lift might be tedious, but doable.
Multicast/ConnectableObservable, and operators that emit inner Observables

We should implement multicast with selector as an operator instead of a custom Observable subclass.
ConnectableObservable, window, groupBy, and eventually join/groupJoin, should all use the prototype from the current this binding, and we can mixin their respective prototypes with Object.create. Fortunately, the ConnectableObservable implementation is already written with Operators and Subscribers, so aside from the public multicast method, this shouldn't be a lot of rework:

class Observable {
  multicast(subjectFactory) {
    const connectable = Object.create(this, connectableObservableDescriptor);
    connectable.subjectFactory = subjectFactory;
    return connectable;
  }
}
const connectableObservableDescriptor = {
  connect: { value: function () { /* ... */} },
  refCount: { value: function () { /* ... */ } },
  _subscribe: { value: function _subscribe(subscriber) {
      return this.getSubject().subscribe(subscriber);
    }
  },
  getSubject: { value: function getSubject() {
      const subject = this._subject;
      if (!subject || subject.isStopped) {
        this._subject = this.subjectFactory();
      }
      return this._subject;
    }
  }
}

regarding the static methods..are you referring to this kind of stuff:
return new ArrayObservable(observables, scheduler).lift(new CombineLatestOperator<T, R>(project));

If so, couldn't this be solved by deferring to the lift from the current prototype instead?
return this.lift.call(new ArrayObservable(observables, scheduler), new CombineLatestOperator<T, R>(project));

@jayphelps That looks useful if you define Observable.prototype.lift to default to this:
lift(o, f) {
  return o.lift(f)
}

But there's a catch: how do you link it with the parent observable, so it can be updated when the parent updates? You still need some sort of API that lets you push data into it, because otherwise, things like .map won't work otherwise.

@isiahmeadows perhaps there's some misunderstanding. I'm not recommending changing the lift method signature.
I was saying that static methods could use the lift method from the current prototype instead of using the one from the ArrayObservable because the current prototype lift might be one that was custom.
It might not have been obvious that I was using fn.call(context, ...args)...so this.lift.call(context, operator) or:
this.lift.call(new ArrayObservable(observables, scheduler), new CombineLatestOperator<T, R>(project));

This is Function.protototype.call not the RxJS internal call() method on operator classes. 😉

Operators that call static methods
These are combineLatest, concat, merge, race, and zip. Implementing these with lift might be tedious, but doable.

I think it's important to move these to use lift... specifically because it will enable us to do more with tooling around RxJS.

Regarding this.lift.call() that I proposed, that obviously wouldn't solve the static operator issue, i.e. Observable.merge, etc where this is no instance prototype to look up this.lift()
What about making lift a static method, so static operators can access it and instance operators can access it as well via this.constructor.lift()?
class Observable {
  static lift() {
    ...
  }

  map(...) {
    this.constructor.lift(...);
  }
}

@jayphelps ... that's an interesting idea... if we took the current incarnation of lift and changed it to be static, it might look like...
static lift<R>(operator: Operator<T, R>, source: Observable<T>): Observable<R> {
    const observable = new this();
    observable.source = source;
    observable.operator = operator;
    return observable;
 }

lift<R>(operator: Operator<T, R>): Observable<R> {
  return this.constructor.lift(operator, this);
}

Then I'm not sure what static operators would look like... what would be the source? etc.
static combineLatest(observables, project) {
  return this.lift(new CombineLatestOperator(project), new ArrayObservable(observables));
}

I'm not sure what that buys us over what we're currently doing.
My spider-sense is tingling about it. I feel like moving to a static lift might be coding ourselves into a corner somehow. But I could be wrong. It just seems odd.

@blesh the example you showed seems legit. I'm confused though about it not seemingly buying us anything? It buys us all that lift provides us..so custom wrapping Observable with our own operators, adding debugging/logging, etc.

@jayphelps sorry, I needed to be more specific... I'm trying to think of the use case where having lift be static is more useful than what we're currently doing.
In both cases, we're able to get the source and the operator and hook them (if we need too) for debugging, etc.
I'm mildly concerned about static lift in the wild. Using it directly seems like an anti-pattern. But hooking it seems like something useful for tooling, etc. ... I'm just leery of the change I think.

@blesh hehe that didn't clear it up for me. How can you easily hook into the existing static methods and return your own observable wrapper? You would have to override every static method individually, wouldn't you?

How can you easily hook into the existing static methods and return your own observable wrapper? You would have to override every static method individually, wouldn't you?

Yes, I see what you're saying.
Okay, okay, I'm warming up to it.

@jayphelps I've got this ready on my local machine, would you like it broken up into single/multiple commits, and single/multiple PRs, or some permutation of the two?

@trxcllnt without knowing what it all entails, hard to say. Throw it all into a single PR with how ever many commits you think it should have and if anyone thinks it's a problem we'll discuss there after seeing it.
If you think there some controversial elements not discussed here yet, those might be good candidates for separate commits so you can more easily tease them out if we wait on those but merge the rest.

@jayphelps I've only touched the prototype methods (combineLatest, concat, merge, multicast, race, and zip + tests) that weren't using lift yet, and each change is relatively minor. I'm worried multiple commits would just be noisy, but I'm happy to do it if you don't think so.

@trxcllnt is this work complete now? I haven't been keeping track. Either way, I'm closing this issue due to inactivity. Feel free to reopen if necessary.

I'd like to leave my 2c.
This is a feature we highly desire at my company. We're working various Observable wrappers, such as AngularFire2, and its a bit of nightmare to code around this limitation; as we might have a FirebaseListObservable, which is a standard Observable with a few extra functions on it.
We can't, however, use any operators on it, because then it gets transformed into a regular Observable, and the extra functions are lost.
So I hope you haven't given up on this! 😄
(I hope I'm not misunderstanding what this topic is about)

@larssn certainly not! Since the beginning I've advocated flowing Observable types through operators, perhaps more than any other issue. Along with perf, it was one of the driving forces behind implementing operators in terms of lift.
I may be the only person in the community who relies on it, but it's a really useful feature for hooking in and building custom async DSLs on top of Observable. My public rxjs-fs, rxjs-mapd, rxjs-gestures, and @graphistry/client-api projects are all built this way, as well as a number of internal APIs.
From my understanding there are just two minor issues we haven't addressed:

Calling the static Observable methods on an Observable subclass won't return an Observable of the subtype, since they're all implemented as Observable subclasses themselves. In practice we can get around this by wrapping the statics in functions that do return the subtype, but it's not as ergonomic as I'd like.
Even though subclassing works at runtime, TypeScript doesn't know how to flow the subtype through Rx operator calls, leading to type errors if you rely on subclassing in TS.

@trxcllnt

Even though subclassing works at runtime, TypeScript doesn't know how to flow the subtype through Rx operator calls, leading to type errors if you rely on subclassing in TS.

Based on reading that, you would probably need a couple specific things:

A way in TS to require and use a parametric constructor in subclasses.
Modifying this to use this.constructor rather than Observable directly.

@isiahmeadows ah, I should have been more clear. TypeScript could infer the subtype through Rx operator calls, but we'd have to remove the explicit return type declarations from all the operators (as well as lift). We could start that discussion, but I'm not sure @benlesh would go for it:
Type error (explicit types):

No type error (inferred types):

To your second point, we can't use the current this.constructor to build the out Observable for two reasons. Subclasses of Observable have their own constructor implementations that we don't want to invoke when an operator is called (e.g. operators called on fromArray() would invoke the ArrayObservable constructor each time). Additionally, we've implemented the instance versions of some static methods in terms of the static methods, so we do some funny this re-binding for convenience.

If the explicit type is not actually the type that is being returned, then it seems like the correct approach would be to let typescript infer the the type.  The current situation makes it impossible to strongly type subjects and then use any operators that are inherited from Observable.


Masks like the animals or the smiley face are more grey examples, but I think the clown and mime masks blocking identity have already set the precedent
Also looking for input regarding the cl wording, I don't want to trigger unmasking checkpoint abuse. Should I just keep this cl boring and mechanical?
Media

[X] This PR does not require an ingame showcase

Changelog

:cl: Errant

tweak: The latest security audit found that a number of novelty masks previously rated unobtrusive are in fact blocking facial features, preventing identification without relying on name tags. NT reminds all employees that they should not, under any circumstances, wear stolen or falsified ID cards.

I had a burst of inspiration and updated the cl so it still communicates the purpose of this, without sounding like instructions to security
I considered moving the component up the tree, but since it was just 1 line per entry, I did not feel very compelled to tidy it. But now the more I look at it, and with the realisation that IngestionBlocker would necessarily apply to every face blocking mask too, I probably will


Prepare release r1.1

What type of PR is this?

subproject management

What this PR does / why we need it:
Make final updates required for r1.1

Adjust versions in yaml/feature for -rc.1
Update API readiness templates
README
CHANGELOG

Which issue(s) this PR fixes:
Fixes #
Special notes for reviewers:

For location-verification some final updates to user stories and external documentation (implementation guidelines) is tbd prior to M4
Test result statement  also tbd (not clear what to provide in link)

@jlurien Ready to go 🚀


Use of third library inside <% template language %>

In my case i'd like to format the initial value of scaleLabel option with moment.js, something like:
<%= moment(value).format('HH:mm:ss') %>

But when I use this, all instancied charts failed to display and nothing prints inside the console (error, warning etc)
Any help on this ? :)

can you post a jsfiddle?

I can't actually, because I need CommonJS modules. I tried http://requirebin.com/, but it doesn't seem to work.
To be more precise, when I create a Line/BarChart, I'd like to modify the way scaleLabel displays the value.
Right now, default value of scaleLabel is:
<%= value %>

In my case, I want this value to be formatted by momentjs as durations (see my code above).
But it seems that we have no ways to inject others variables inside the micro templating engine used in ChartJS, other than value. Correct me if i'm wrong.
The only workaround I found is to simply add momentjs as global variable by importing it with a <script> tag :)
Another little experiment shows that it works if I do this right into the source code of Chart.Core.js:
https://github.com/nnnick/Chart.js/blob/master/src/Chart.Core.js#L1559
// from:
this.yLabels.push(template(this.templateString,{
    value:(this.min + (i * this.stepValue)).toFixed(stepDecimalPlaces)
}));

// to:
this.yLabels.push(template(this.templateString,{
    value:(this.min + (i * this.stepValue)).toFixed(stepDecimalPlaces),
    moment: require('moment')
}));

Here is a requirebin: http://requirebin.com/embed?gist=6264704 :)
console: Uncaught ReferenceError: moment is not defined
The only solution I found is to put momentjs library inside a <script> tag.. But i'm looking for a more elegant way to inject moment inside the templating engine as I want to keep my dependencies clean with webpack.

I personally have been looking for a better way to do the "micro-templating" your describing.  It might be something as simple as providing a hook to expose more variables like you've done in your quick fix.  HTML strings/templates -> canvas has always been my least favorite part of all this.

@etimberg Do you have any higher level thoughts on this one?

I've been thinking about this. I'm not sure what the best solution would be.
@mr-wildcard by changing the template to the following, I got it to work. I remember @nnnick mentioning this somewhere, but it's not in the documentation. Passing a function as the template string simply executes the function.
Chart.defaults.global.scaleLabel = function() {return moment().add(this.value, 'days').format('YYYY/DD/MM')};

Thanks @etimberg !
It works like a charm, and even if its undocumented I feel like its the normal way to achieve what I was trying to do. :smiley:


Update plugin version maven-project-info-reports-plugin to 3.1.0

I tested this already using the "mvn site". This afternoon I tested with the same command as used by Jenkins, except that I left out the release:perform goal. The goal release:prepare was in my test. The build was successful and I had two commits changing the POMs. The first commit introduced the timestamp in the artifact names. The second added commit added "SNAPSHOT". A tag was added to the first new commit. The build was successful.


[1.11.2] Connector won't connect to Adv Generators Flux Generator

Xnet: xnet-1.1.5
McJty: mcjtylib-1.1x-2.3.11
Compat: compatlayer-1.11.2-0.2.5
Adv Gens: generators-<IP_ADDRESS>-mc1.11.2
The Xnet connector will not connect to Advanced Generator's Flux Generator. It will however work with an RF tools Powercell.
Steps to reproduce:

Setup an Adv Generators multiblock with Flux Generator
Try to connect a Xnet connector to the Flux Generator

This is most likely caused by the flux generator not supporting 'null' for the energy side of the energy handler. You should report this to them

I don't support null since each output face gets a color assigned and can be controlled separately... So i need to know which side it's being outputted to.
Forge Energy Emitter module does work, so it can be used instead.

Ok, In that case I guess this will remain an incompatibility as I have to use null in this case since I don't know the side from which I will access it at that point

Hm, so i looked around in my code, and for Forge Energy i do provide a capability on null side (but don't allow power output from it).
I think i could do the same for RF, however i don't see where your code checks for RF connections? In ConnectorBlock.isConnectable it only checks for capabilities, IInventory and your own blocks... Am i missing something?
I also noticed that my fluid outputs don't work, likely because of the null thing as well.

Hmm seems that I forgot to check for old style RF in hy isConnectable method. I will fix that.


disk.list should return only block devices of type 'disk'

Verified on Version: 1.1.0-alpha @Revision: d1b3e6658c6805786abe795a080745c4aba39f0c


Cannot use yield in a foreach loop header

Using yield inside a header of a foreach loop generates a syntax error:
$a = function () { foreach (yield as $magic) {} };

Result:
Fatal error: Uncaught ParseError: Syntax error, unexpected 'as' in res://a.php on line 2, position 35

Same thing happens with yield from.

How can I use yield in foreach, can you add some example?
$a = function () {
    foreach (yield as $magic) {
        var_dump($magic);
    }
};

// $a is not generator in PHP 7.4, but no syntax errors

Hmm... How can I use yield in foreach? PHP 7.4 shows warnings when I try to use generator with foreach.

Works fine on my machine with 7.4:
[daniil@daniil-arch jav]$ cat src/a.php
<?php
$a = function () { foreach (yield as $key => $magic) { var_dump($key, $magic); } };
$a()->send(['a' => 'b']);
[daniil@daniil-arch jav]$ php src/a.php
string(1) "a"
string(1) "b"

Nice, recent bugfixes solved this issue.


Can not execute test

@targos Please could you tell me why npm run test does not work in this project ?

if you run asynchronous tests, you must put the it() outside of the asynchronous part and use the done() callback. See https://mochajs.org/#asynchronous-code


runtime error when starting wal-e due to new pbr library

The pbr library was updated recently, and in our CI system this seems to cause a runtime error starting wal-e. From our functional tests:
database: performing an initial backup...
Traceback (most recent call last):
  File "/usr/bin/wal-e", line 5, in <module>
    from pkg_resources import load_entry_point
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3074, in <module>
    @_call_aside
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3060, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3087, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 647, in _build_master
    return cls._build_from_requirements(__requires__)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 660, in _build_from_requirements
    dists = ws.resolve(reqs, Environment())
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 838, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (pbr 1.0.1 (/usr/lib/python2.7/site-packages), Requirement.parse('pbr!=0.7,<1.0,>=0.6'), set(['oslo.config', 'oslo.i18n', 'stevedore', 'oslo.serialization', 'oslo.utils']))

At least that is my assessment of what is probably happening--I haven't been able to reproduce this locally yet. If so, pinning pbr==0.11.0 might fix it, because that's what I see in the most recent test runs where this was passing.
Here is where the requirement is introduced:
Collecting pbr<2.0,>=0.11 (from python-keystoneclient>=0.4.2->wal-e==0.8.0)
  Downloading pbr-1.0.1-py2.py3-none-any.whl (83kB)

Reverting pbr to 0.11.0 didn't fix this for me:
postgres:~$ envdir /etc/wal-e.d/env wal-e backup-push
Traceback (most recent call last):
File "/usr/local/bin/wal-e", line 5, in
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2749, in
working_set = WorkingSet._build_master()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 446, in _build_master
return cls._build_from_requirements(requires)
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 459, in _build_from_requirements
dists = ws.resolve(reqs, Environment())
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 628, in resolve
raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: requests>=2.5.2

It's kind of a blunt workaround, but it did work for us in the PR linked to above. This is building wal-e inside a Docker container using Alpine Linux and Python 2.7.9.
Maybe one of oslo.config or stevedore will update, since they seem to be constraining the version to between 0.6 and 1.0 (minus version 0.7). Somehow those constraints are ignored until runtime, and it's only python-keystoneclient that's in requirements.txt, and it's perfectly happy downloading pbr 1.0.1...

Well, here's my pip freeze:
apt-xapian-index==0.45
azure==0.11.0
Babel==1.3
boto==2.38.0
chardet==2.0.1
colorama==0.2.5
configobj==4.7.2
envdir==0.7
futures==3.0.2
gevent==1.0.2
greenlet==0.4.7
html5lib==0.999
iso8601==0.1.10
Landscape-Client==14.12
msgpack-python==0.4.6
netaddr==0.7.14
netifaces==0.10.4
oslo.config==1.11.0
oslo.i18n==1.6.0
oslo.serialization==1.5.0
oslo.utils==1.5.0
PAM==0.4.2
pbr==0.11.0
prettytable==0.7.2
pycurl==7.19.3
pyOpenSSL==0.13
pyserial==2.6
python-apt===<IP_ADDRESS>ubuntu1
python-dateutil==2.4.2
python-debian===0.1.21-nmu2ubuntu2
python-keystoneclient==1.5.0
python-swiftclient==2.4.0
pytz==2015.4
requests==2.2.1
simplejson==3.7.2
six==1.5.2
ssh-import-id==3.21
stevedore==1.4.0
Twisted-Core==13.2.0
urllib3==1.7.1
virtualenv==1.11.4
wal-e==0.8.0
wheel==0.24.0
zope.interface==4.0.5

Oh, I see the problem.
Apparently the wal-e pip package does not, in fact, install the requirements.    Fixing, but shouldn't the pip package take care of this?

So for the pbr problem, it appears this specific problem has been fixed in oslo.config. I'll make a PR to have wal-e require oslo.config 1.12.0, which is a better fix than pinning pbr.

I don't understand the current state of this bug. Is it broken as of now?  A fresh virtualenv worked for me.
The problem with throwing in some requires in a point release is it can break upgraders if there was some past version that worked fine.

I've resolved my version of the problem by adding extra requirements, per the other issue I filed.

Pinning oslo-config to a version at or after 1.12.0 removes its incorrect requirement for pbr, which fixes the issue we saw. It would be preferable to have python-keystoneclient pin this in its requirements, but currently it only asks for oslo-config>=1.11.0. See also deis/deis#3786.
I didn't address the need to include requests or six in requirements in #183 because those weren't relevant to the problem we saw running wal-e. @fdr I'm not sure why it isn't reproducible for you in a fresh virtualenv. The conflicting requirements for pbr are there however, until oslo-config was fixed recently.

I'm not going to include transitive dependencies to work around a bug I can't reproduce and could break existing users.  Here's a transcript of things working fine:
$ virtualenv --no-site-packages /tmp/test; source /tmp/test/bin/activate
Running virtualenv with interpreter /usr/bin/python2
New python executable in /tmp/test/bin/python2
Also creating executable in /tmp/test/bin/python
Installing setuptools, pip...done.
(test)fdr@sputnik:~/codes/shogun$ pip install wal-e
[lots of output, mostly from gevent]
Successfully installed wal-e gevent boto azure python-swiftclient python-keystoneclient greenlet python-dateutil pyopenssl futures six requests simplejson oslo.serialization stevedore oslo.i18n pbr netaddr PrettyTable iso8601 Babel oslo.utils oslo.config cryptography msgpack-python pytz netifaces debtcollector idna pyasn1 enum34 ipaddress cffi wrapt pycparser
Cleaning up...
(test)fdr@sputnik:~/codes/shogun$ wal-e
usage: wal-e [-h] [-k AWS_ACCESS_KEY_ID | --aws-instance-profile]
             [-a WABS_ACCOUNT_NAME] [--s3-prefix S3_PREFIX]
             [--wabs-prefix WABS_PREFIX] [--gpg-key-id GPG_KEY_ID] [--terse]
             {version,backup-fetch,backup-list,backup-push,wal-push,wal-fetch,wal-prefetch,delete}
             ...
wal-e: error: too few arguments

what versions of six and requests do you have installed?

(test)$ pip freeze
Babel==1.3
argparse==1.2.1
azure==0.11.0
boto==2.38.0
cffi==1.1.2
cryptography==0.9.1
debtcollector==0.5.0
enum34==1.0.4
futures==3.0.2
gevent==1.0.2
greenlet==0.4.7
idna==2.0
ipaddress==1.0.7
iso8601==0.1.10
msgpack-python==0.4.6
netaddr==0.7.14
netifaces==0.10.4
oslo.config==1.12.1
oslo.i18n==1.7.0
oslo.serialization==1.6.0
oslo.utils==1.6.0
pbr==1.1.1
prettytable==0.7.2
pyOpenSSL==0.15.1
pyasn1==0.1.7
pycparser==2.13
python-dateutil==2.4.2
python-keystoneclient==1.6.0
python-swiftclient==2.4.0
pytz==2015.4
requests==2.7.0
simplejson==3.7.3
six==1.9.0
stevedore==1.5.0
wal-e==0.8.1
wrapt==1.10.4
wsgiref==0.1.2

Yeah, so you already have modern versions of Six and Requests installed, which is why you didn't notice them as requirements.  Try installing on a RHEL5 container, and you'll see the problem.

Is this possibly Python 2.6 related, if we're talking something so old?

Are you installing via pip into a --no-site-packages virtualenv?  I can't figure why a fresh install would Do The Wrong Thing as-is.  Or is this some conflict with libraries installed on that platform?

It is python 2.6-related.  That is, the target machine was upgraded to python 2.7.  But in that case, apparently it doesn't upgrade six and requests.

Are you using Openstack, per chance, or is it just an unwelcome hanger-on here? Because that's where the bulk of problems in dependencies come from.

The pbr issue is still there; you only have to read the requirements.txt files from python-keystoneclient and earlier versions of oslo.config to see that they specify mutually incompatible requirements for pbr. This issue did happen with pip install as I cited and linked to above, and it stopped our CI system dead for a few days.
It isn't happening to you because a fresh install today finds newer versions of oslo.config that realized they had no dependency on pbr, and removed it:
$ pip install wal-e
Collecting oslo.config>=1.11.0 (from python-keystoneclient>=0.4.2->wal-e==0.8.0)
  Downloading oslo.config-1.12.1-py2.py3-none-any.whl (68kB)
    100% |████████████████████████████████| 69kB 825kB/s

This will probably work, but it's still potentially dangerous that python-keystoneclient only requires oslo.config>=1.11.0. If it actually installs 1.11.0, things will break. I suppose this isn't wal-e's issue specifically, we were just hoping for an upstream fix rather than hacking around it ourselves, as we have done.
I'll leave this issue open as it is apparently focusing on an unrelated problem now.

Nope.  Standard RHEL system.

Are either of you using OpenStack?
The OpenStack client dependencies have caused a towering amount of complaint (which is to say, several, vs the others, which are none). I'm thinking that I should implement optionality of OpenStack and this will ablate most issues.
I was already careful to only dynamically load the libraries I need for a backend, so adding some error messages might handle the rest.

Are either of you using OpenStack?

Not us.

I'm not encountering this anymore.  The underlying pip libraries seem to have straightened out.
Close?

Things have been sorted out in the pip ecosystem, and AFAICT this isn't relevant now. I'll close it.


Sync timer always results in 0ms

When an upload is made, the timer is always off:
> Synced 1 file (968.69KB) [0ms]

Very aspirational but incorrect for now! 😅


Arbitrary Data Transaction Field

The eWallet team and @kfichter would like to have a metadata field as part of the transaction format. This would be an arbitrary byte field. It opens up multiple use cases:

allows multiple eWallets to coordinate by internal wallet ids
can run game actions on-chain (e.g. recording chess moves)
storing immutable data alongside a transaction

The field does not have to be very wide. Please discuss concerns here and come to a decision.
@T-Dnzt @robinclart @whoisjeremylam @kevsul @paulperegud @pdobacz

My feeling is that this gets us closer to needing a bitcoin-like fee model where users can pay to include more transaction data. There should definitely be a fee for using the field to disincentize wasting space (even if it's fixed size).

For context, here is the equivalent in Bitcoin https://en.bitcoin.it/wiki/OP_RETURN. The size for OP_RETURN was 40 bytes and was increased to 80 bytes (http://www.talkcrypto.org/blog/2016/12/30/op_return-40-to-80-bytes/). OP_RETURN is designated as prunable because nodes can choose to delete or not store this data.
Use of the data field is very useful for co-coordinating payments. The data field could contain a correlation_id that relates to instructions on where a payment should be directed.
Would the introduction of this field add any complications to checkpointing or to other designs that reduce the history? It seems that applications that require this data must keep the full transaction history.

@whoisjeremylam @T-Dnzt @kasima would a hash-worth of bytes (32) be enough for all use cases?

32 bytes seems like plenty of space. I'm going to move this into the Tracker board.


Update jawn-fs2 to 0.15.0

Updates org.http4s:jawn-fs2 from 0.15.0-M1 to 0.15.0.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "jawn-fs2" } ]

labels: semver-pre-release

Depends on fs2-2.0.0.


Support laravel 7

Sorry, I didn't notice deprecation array_get in Laravel 6 and removing in Laravel 7.

Fixed

@hernandev pew pew


Customise default settings

Hey, where can I change the default settings? I'm usually using the same configuration as stated in the "Usage Tips", so it feels cumbersome to adjust the settings every time. I've tried poking dynamic_thresholding.py and changed the values there (I have no coding experience) and it didn't work.
threshold_percentile = gr.Slider(minimum=90.0, value=95, maximum=100.0, step=0.05, label='Top percentile of latents to clamp')
mimic_mode = gr.Dropdown(VALID_MODES, value="Half Cosine Up", label="Mimic Scale Scheduler")
mimic_scale_min = gr.Slider(minimum=0.0, maximum=30.0, step=0.5, value=3, label="Minimum value of the Mimic Scale Scheduler")
cfg_mode = gr.Dropdown(VALID_MODES, value="Half Cosine Up", label="CFG Scale Scheduler")
cfg_scale_min = gr.Slider(minimum=0.0, maximum=30.0, step=0.5, value=3, label="Minimum value of the CFG Scale Scheduler")

edit ui-config.json in webui root. See also #32


First half of dates are deactivated

I created my first event and specified 8 available dates, but when I get taken to the grid, the first 4 dates show up as gray and I'm not able to click and drag to set availability during those days:

There seems to be a style applied disabling the cursor and setting the color, but I'm not sure why

Whereas a working element looks like:

I've tried creating multiple events using different accounts, and always see the same result.
Right now I'm on Chrome 58.0.3029.110 (64-bit) on a Mac.

hey @yesthatguy. the fix is on line.  Can you please test and give me a feedback ?

Working for me now, thanks!!


Change text summarization example with faster remote evaluator

Remote evaluator is slow due to recomputing test sample metrics. Rather than recomputing, the script was changed to pull test sample metrics from CSVs.

Changed pyproject.toml since torch download was failing
Refactored helper functions into utils
Made evaluator_fast to feed into the remote evaluator
Cleaned up workflow.py

Remote evaluator works: link

This doesn't put our best foot forward as a publicly-visible example — hesitant to merge because it significantly impacts the readability of this example.


[Enhancement]: Update the default themes in app theming

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
The default themes could be improved in app theming as the colors are not either appealing or typically used in apps.
Why should this be worked on?
By default apps built in appsmith should look amazing

Update:
We have defined new color palette along with color palette name
The information can be found here

New themes are documented above, @jsartisan please check

Scope:

Remove sharp, rounded and modern themes
Add the new themes listed in the document above
Whenever a new app is created automatically apply the earth theme


Initial configuration for Docker

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/gothinkster/realworld/blob/master/CONTRIBUTING.md#commit

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Welcome to this project @helumalai.
Thanks for contributing.
Please fill in the original message and let me know when the contribution is ready to be reviewed.
Mind the repository is using a monorepo structure, not being exclusively related to express but 5 apps.

Closed due to inactivity


Showing interactive plot: Solved

If you encountered the problem of getting the interactive plot in the Jupyter notebook, you could replace the first line %matplotlib notebook with %matplotlib inline as shown in the figure.

That's nice. However in VS code environment, we don't need that line at all.

Nice, it works @Mosaddeq107

This is a good fix, but can you interact with the plot like this? I don't think I can in colab

Yes it becomes messy.


package com.byteowls.capacitor.filesharer.FileSharerPlugin does not exist

Capacitor version:
Latest Dependencies:

  @capacitor/cli: 2.4.5
  @capacitor/core: 2.4.5
  @capacitor/android: 2.4.5
  @capacitor/electron: 2.4.5
  @capacitor/ios: 2.4.5

Installed Dependencies:

  @capacitor/ios not installed
  @capacitor/cli 2.4.2
  @capacitor/core 2.4.2
  @capacitor/android 2.4.4
  @capacitor/electron not installed

Affected Platform(s):

Android

Current Behavior
On trying to register plugin in Android at com.companyname.appname.MainActivity#onCreate, importing the plugin says that the plugin is not found
Expected Behavior
I would expect the plugin to register in MainActivity
Sample Code or Sample Application Repo
...
import com.byteowls.capacitor.filesharer.FileSharerPlugin;

public class MainActivity extends BridgeActivity {
  @Override
  public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    // Initializes the Bridge
    this.init(savedInstanceState, new ArrayList<Class<? extends Plugin>>() {{
      // Additional plugins you've installed go here
      // Ex: add(TotallyAwesomePlugin.class);
      add(FileSharerPlugin.class);
    }});
  }
}

Other Information
I do not have any expirence with Android Studio, any help with getting this plugin to work would be appreciated

Hey,
can you check if you have done these steps:

Install the plugin npm i ...
npx cap sync android
Open Android studio
If the Plugin is still not found. Find the Gradle section. Right click on the first entry and "Reimport Gradle Project"

See the Capacitor Android Docs for further information.
did you run npx cap sync android and then

Hey!
Thanks a ton for your help. Turns out all I needed to do was npx cap sync android.


pass location.state in server rendering redirectLocation?

Following auth-flow example and Server Rendering guide I am wondering how to combine those ways and keep location.state for nextPathname.
If protected url accessed directly, redirectLocation in match contains state at first then redirect happens
res.redirect(302, redirectLocation.pathname + redirectLocation.search)
and state is reset, so after auth there is no redirect back to the protected page.

You should store your state in some other form, such as Redux.

@timdorr this is a helpful suggestion. However, would you be able to elaborate on a standard scheme for doing this? It seems like a fairly ordinary concern that should perhaps be documented or included in an example. In the meantime, I'd love to hear your advice.

Pending any response I've chosen to include the name of the destination path after auth as part of the query string in the redirect request on the server. That way the client knows a redirect took place, and will know where to transition after auth. I considered using redux-router, but this seems like a much cleaner solution that can be easily compatible with react-router-redux.


Multiply rows for event rows

For request of this kind:
"UPDATE table_name SET field="foo" LIMIT 10"

Thanks @perepechaev


Deprecated cryptography method usage

In ecdsa.py signer method is used and it prints out a warning:
graphenelib-0.6.6-py3.5.egg/graphenebase/ecdsa.py:187: CryptographyDeprecationWarning: signer and verifier have been deprecated. Please use sign and verify instead.
  signer = private_key.signer(ec.ECDSA(hashes.SHA256()))

fixed


col-major matrices

Awesome work on the lib! I was wondering if you had any plans to support col-major matrices for those of use who aren't on Windows and are stuck without DirectX.

Thanks!
Note that you can already use the lib with other APIs, for example when using OpenGL below code should work jus fine:
const mvp = mul(proj, modelview);
// transpose(mvp) should be passed to GLSL

There are also right-handed version of some functions for OpenGL and Vulkan - perspectiveRh, lookAtRh.

Closing for now - feel free to re-open if you want to discuss it further.


PowerShell Scripts require .ps1 extension

Scripts (on linux) don't function unless they have the extension .ps1 (generally, languages do not require specific extensions in Linux when using shebang to start them #! /usr/bin/powershell
Steps to reproduce
File: myscript
#!/usr/bin/powershell
echo hi

chmod a+x ./myscript

# hangs!
./myscript

mv ./myscript ./myscript.ps1

# works
./myscript.ps1
hi

Expected behavior
On linux, this should work even if the extension isn't .ps1
Actual behavior
EDIT
it turns out it didn't hang, it fails:
System.TypeInitializationException: The type initializer for 'System.Management.Automation.PSVersionInfo' threw an exception. ---> System.IO.FileLoadException: Could not load file or assembly 'System.Reflection.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. An internal error occurred.
 (Exception from HRESULT: 0x8007054F)
   at System.Reflection.Internal.LightUpHelper.GetMethod(Type type, String name, Type[] parameterTypes)
   at System.Reflection.Internal.MemoryMapLightUp.TryLoadMembers()
   at System.Reflection.Internal.MemoryMapLightUp.get_IsAvailable()
   at System.Reflection.Internal.StreamMemoryBlockProvider..ctor(Stream stream, Int64 imageStart, Int32 imageSize, Boolean isFileStream, Boolean leaveOpen)
   at System.Reflection.PortableExecutable.PEReader..ctor(Stream peStream, PEStreamOptions options, Nullable`1 sizeOpt)
   at System.Diagnostics.FileVersionInfo.TryLoadManagedAssemblyMetadata()
   at System.Diagnostics.FileVersionInfo.GetVersionInfo(String fileName)
   at System.Management.Automation.PSVersionInfo.GetBuildVersion()
   at System.Management.Automation.PSVersionInfo..cctor()
   --- End of inner exception stack trace ---
   at System.Management.Automation.PSVersionInfo.get_PSVersion()
   at System.Management.Automation.PSSnapInReader.ReadRegistryInfo(Version& assemblyVersion, String& publicKeyToken, String& culture, String& architecture, String& applicationBase, Version& psVersion)
   at System.Management.Automation.PSSnapInReader.ReadCoreEngineSnapIn()
   at System.Management.Automation.Runspaces.InitialSessionState.ImportCorePSSnapIn()
   at System.Management.Automation.Runspaces.InitialSessionState.CreateDefault2()
   at Microsoft.PowerShell.UnmanagedPSEntry.Start(String consoleFilePath, String[] args, Int32 argc)
   at Microsoft.PowerShell.ManagedPSEntry.Main(String[] args)

Environment data
v0.4.0

This is really important. Without this we also can't use scripts as executables, as adding a shebang to the start of a file without a .ps1 extension will fail.
This completely breaks the way we work on Linux. On Windows it is fine, we are used to it, on Linux it is just wrong to require the extension.

I completely understand that the extension is not how Linux works, but wouldn't this encourage non-cross platform PS Scripts?

I don't think that's necessarily a bad thing - sure, functions and modules should make considerations for cross platform compatibility, but some of these, and some scripts that leverage them, will inherently target specific platforms.
IMHO, you might find the folks writing PowerShell scripts on *nix, or at least a decent proportion of them, recognize the importance of cross-platform support, where it makes sense.
That said, assuming I'm wrong, wouldn't it be better to have a few folks end up writing non-cross-platform scripts (assumption), rather than walking away from the language given this limitation (assumption, perhaps just as likely)?
Cheers!

@RamblingCookieMonster I'm personally where you landed on this which is to lower the barrier to entry on Linux to get adoption and where cross platform is needed, scriptanalyzer can help with that and those users will learn very quickly it won't work on Windows.

@PowerShell/powershell-committee discussed this.  There may be impact to get-command as now it would have to read the shebang to determine if it's a PowerShell script so we can run it within current runspace.  It may be worth experimenting to compare perf impact.
We should fix the hang so that scripts without extensions works as "executables" (new instance of powershell will execute it) as long as shebang is there.


Does this project follow semver?

What is better way to specify this module in requirements? Can it be used with ~= operator (does it follow Semantic versioning?)
aiohttp-cors~=0.7.0

Found information that module supports semver.


[API] When backup_name is invalid, the message of response  for get_backup is still success

{
    "requestId": "37c177b8-6fcd-11ed-9185-acde48001122",
    "msg": "success",
    "data": {}
}

fix in #40


Exclude issue with post message.

Issue description:
Post message exclude seem to be a little glitchy.
For example:
Kirki::add_field( $opt_name, array(
	'label'     => esc_html__('Form input and select style', 'opty'),
	'section'   => 'forms_section',
	'settings'  => 'form_field_style',
	'type'      => 'radio-buttonset',
	'default'   => 'square',
	'transport' => 'auto',
	'choices'   => array(
		'square'  => esc_html__('Square', 'opty'),
		'rounded' => esc_html__('Rounded', 'opty'),
		'pill'    => esc_html__('Pill', 'opty')
	),
	'output'    => array(
		array(
			'element'       => 'input, textarea, select',
			'property'      => 'border-radius',
			'value_pattern' => '0',
			'exclude'       => array( 'rounded', 'pill' ),
		),
		array(
			'element'       => 'input, textarea, select',
			'property'      => 'border-radius',
			'value_pattern' => '4px',
			'exclude'       => array( 'square', 'pill' ),
		),
		array(
			'element'       => 'input, textarea, select',
			'property'      => 'border-radius',
			'value_pattern' => '20px',
			'exclude'       => array( 'square', 'rounded' ),
		),
	),
) );

This works perfectly fine with CSS output, but with post message output, the first two options are blank and the final option will output. Eg:
Square and Rounced output:
<style id="kirki-postmessage-form_field_style"></style>

While Pill outputs:
<style id="kirki-postmessage-form_field_style">input, textarea, select{border-radius:0;}input, textarea, select{border-radius:4px;}input, textarea, select{border-radius:20px;}</style>

This is only with postmessage. Refresh/CSS output is fine.
Now I'm working on extending Kirki with several new controls in a fork ( color gradient, slider-advanced, typography-advanced, border, sortable w/ text or checkbox and spacing-advanced, toggle-tabs (grouping) ), so I thought I may of broken something, but I uploaded the current kirki/develop to the test project and ran into the same issue.
Any insight on the fix?
Also on another note, once I feel the controls are completed, I'll do a pull request.

@aristath Is this something that you could look into today?

Can you please try with this branch? https://github.com/aristath/kirki/tree/postMessage-refactor
It's a complete refactor of the postMessage module that I'm working on...

I tried the refactor and it seems to be working fine. I'm working on rewriting the PHP postmessage processors for the custom controls to work with the refactor.
This is quite cleaner too, btw. Nice.
Now question. Since I've been working on the develop branch, should I make a new fork for the postMessage refactor, or is that the only piece of the code that is being worked on? I remember you mentioning it will be merged into Kikri once you feel its right, so should I stay working on develop and collaborate with you once I feel the controls are ready for merging?
Thanks.


Build and Test Windows Wheels in a Docker Image

It should help to check whether Windows wheels actually work after it is built.

I am going to merge this because this is just the CI changes.


Shrink Fighter_CompanionMount timer to 1 rl hour

The skill computes the required number of ticks based on fixed mud time periods. this increased drastically due to configured number of days per month and hours per day.
Switching the logic from a fixed mud time period to a fixed rl period


Config Files

M Configuration files: figure out how hotpants/OIS/sep are configured and create a YAML config file

Configurable values like threshold amounts, file directories, etc. should go in a yaml file rather than directly in python. This is currently in config.__init__.


Use HMR host when set

This PR updates the hot file generation to use Vite's server.hmr.host config option when specified by the user.
This configuration option is intended to tell the Vite client where to find the HMR server for cases where it is different from server.host. I believe it makes sense for us to use it as well.
This probably helps a range of use cases, but specifically, it allows users to run Vite inside a Sail container that is running inside WSL, where the Vite dev server needs to be configured on the container's public address, but where the host machine cannot connect to this same address.
Users running Vite inside Sail inside WSL will need to add the following to the defineConfig section of their vite.config.js file:
server: {
    hmr: {
        host: 'localhost',
    },
},

Closes #28

can you please tag this PR?

As a workaround for anyone tackling a side project this weekend, open public/hot after running dev and just replace the <IP_ADDRESS> with localhost

As a workaround for anyone tackling a side project this weekend, open public/hot after running dev and just replace the <IP_ADDRESS> with localhost

It works for CSS files and js files but the static assets won't load

Releases will be tagged every Tuesday as per Laravel's standard release schedule. If you need this now, you can always build from source by cloning the repository and using npm link.

I am trying to run Vite inside Sail inside WSL on Windows 10 and as instructed, I tried adding the following code snippet to defineConfig section of vite.config.js file and then started sail followed by running npm run dev but now both localhost and localhost:5173 returns localhost didn’t send any data. :
server: {
    hmr: {
        host: 'localhost',
    },
},

What am I doing wrong?

Hey @andrewl64,
Are you running the HMR server within Sail. You will need to use ./vendor/bin/sail npm run dev.
Let us know how ya go.

@timacdonald Yes, I ran it using sail. For some reason, accessing Vite using http://<IP_ADDRESS>:5173/ works however (but http://<IP_ADDRESS> doesn't).

@andrewl64
this is how i use vite inside sail in wsl2 + windows 11 :
first add the 5173:5173 to your laravel service ports in docker-compose file.
then add this code to your vite config
    server: {
        hmr: {
            host: 'localhost'
        }
    },

now run your containers and use sail npm run dev to run the vite hot reloading.
and the last thing u must check is that your assets loading using @vite directive.
if this setups didnt fix your problem please create a repo with minimal code that reproduce the problem and mention me here so i check it on windows + wsl setup

hi @ariaieboy,
i am having the same problem (Win11 / WSL2 / Laravel / Sail / Vite).
I am using a local domain instead of localhost, that points to <IP_ADDRESS> (hosts file).
my ports definition in docker-compose is:
        ports:
            - '${APP_PORT:-80}:80'
            - '${VITE_PORT:-5173}:${VITE_PORT:-5173}'

i opened a ticket where i describe the details: https://github.com/laravel/vite-plugin/issues/127
also tried additionally server.hmr.host additionally, but no luck (snippet):
server: {
        hmr: {
            host: 'server.apps.local'
        },
        host: 'server.apps.local'/*,
        origin: 'http://server.apps.local:5173'*/
    }

maybe you have an idea, what the cause could be?

@ariaieboy Here's a LINK to a test repo with the vite.config.js  file and the docker-compose.yml file. All other files are using default config.
With the above config,

http://localhost and http://localhost:5173 both return:

localhost didn’t send any data.
ERR_EMPTY_RESPONSE

http://<IP_ADDRESS> returns a white screen with the following errors in the console:

GET http://localhost:5173/@vite/client net::ERR_EMPTY_RESPONSE                    <IP_ADDRESS>/:24
GET http://localhost:5173/resources/js/app.jsx net::ERR_EMPTY_RESPONSE        <IP_ADDRESS>/:24
GET http://localhost:5173/@react-refresh net::ERR_EMPTY_RESPONSE                <IP_ADDRESS>/:19

And http://<IP_ADDRESS>:5173 returns the Vite page correctly but with the following error  in the console:

GET http://<IP_ADDRESS>:5173/ 404 (Not Found)                                                          :5173/:1

I think it could be an ipv6-related issue with localhost maybe because http://<IP_ADDRESS>:5173/resources/js/app.jsx returns the file content but http://localhost:5173/resources/js/app.jsx returns Failed to load resource: net::ERR_EMPTY_RESPONSE.

If push comes to shove, I might just go with forwarding ports as shown here in this StackOverflow thread but I'm not sure if that's a good idea though.

Do not forget to add the --host parameter to the vite call inside the dev script within your package.json file.
For me it fixed my problem with HMR.
see https://github.com/laravel/vite-plugin/issues/127#issuecomment-1252541895

I got it to work in sail with the following:
//vite.config.ts
export default defineConfig({
    server: {
        host: '<IP_ADDRESS>',
    },
	plugins: [
		inertia(),
		vue(),
		laravel({
			postcss: [
				tailwindcss(),
				autoprefixer(),
			],
		}),
	],
})

//docker-compose.yml
ports:
       - '${APP_PORT:-80}:80'
       - 3000:3000
       - '${VITE_PORT:-5173}:${VITE_PORT:-5173}'

Then run: sail build --no-cache sail up sail npm run dev

This works for me too.

@jordanbaindev 's solution works for me but it does not load the environment variable for VITE_PORT which I have set in the .env file.

After a long troubleshooting I found that the previous project I hosted didn't have the hot file inside public folder and was working normally. But newer version of project somehow created this file and started making the same error showing the page not loading resources correctly unless I turn off Brave security and sometimes again it breaks.
SOLUTION worked for me:
DELETE file public/hot
It fixed the problem instantly without any other changes necessary.

After a long troubleshooting I found that the previous project I hosted didn't have the hot file inside public folder and was working normally. But newer version of project somehow created this file and started making the same error showing the page not loading resources correctly unless I turn off Brave security and sometimes again it breaks.
SOLUTION worked for me:
DELETE file public/hot
It fixed the problem instantly without any other changes necessary.


Run plugin commands from console

Allow to run plugin commands from console (instead of clicking in UI)

Feature request raised to Remix: https://github.com/ethereum/remix-project/issues/4729


Update dotfinder.py

Note sure if this is the most efficient solution but it solved the int to float issue for chrom positions
@sergpolly @nvictus
Warning:
/home/spracklin/devel/cooltools/cooltools/dotfinder.py:1776: FutureWarning:
Passing list-likes to .loc or [] with any missing label will raise
KeyError in the future, you can use .reindex() as an alternative.
Solution found in:
https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#deprecate-loc-reindex-listlike

This must be it then ...
But ... the error itaself does not make sense to me still:
in your pandas doc link see this section:

Selection with all keys found is unchanged.

that implies that if we are .loc[]-selecting a bunch of rows with existing indexes it should still work:
chrom_clust_group = df.groupby(["chrom1", "chrom2", "c_label"])
centroids = df.loc[chrom_clust_group["count"].idxmax()]

idxmax returns a subset of indices from df itself - it's a subset of indices that correspondond to the "brightest" pixels in each cluster of pre-filtered dots ... - this MUST work unless there is something "bad" (e.g. non-unique) with indices in the scores_df (i.e. filtered_pixels_qvals in the call dots CLI)
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.core.groupby.DataFrameGroupBy.idxmax.html
this might be relevant https://stackoverflow.com/questions/32459325/python-pandas-dataframe-select-row-by-max-value-in-group

this is to https://github.com/mirnylab/cooltools/issues/125

@gspracklin could you post the content of df after the merge (with indices) (from inside of the CLI version that is failing):
df = pd.merge(  scores_df, pixel_clust_df, how="left", left_index=True, right_index=True )

along with whatever idxmax returns:
chrom_clust_group = df.groupby(["chrom1", "chrom2", "c_label"])
chrom_clust_group["count"].idxmax()

I just checked indices in Unsync_10kb.bedpe.filteredpixelsqvals.txt, which is supposedly scores_df that goes into clustering_step, - everything looks OK - unique looking indices running from 0 to whatever ...

df after merge should be of the same length as scores_df and have THE SAME indices
idxmax should return a subset of those OK-looking indices

@sergpolly
Debugged more with Nezar...
Looks like score_df has all chromosomes (including chrY and chrM) as categorical values. During the groupby step, all possible chrom combinations are made for the number of c_labels (see attached).
seems related to this:
https://github.com/pandas-dev/pandas/issues/17594
idxmax_count_groupby.txt

This doesn't explain why I am specifically seeing this issue (pandas v0.25.3)

Hi guys,
Is this problem fixed? I am getting this error when I am trying to run call-dots on CLI.


feat: make upcoming events dynamic

Description

upcoming events part of the website is now generated dynamically from community/issues meetings.
(in case it doesn't find meetings on community repo, it will fall back to the current upcoming events)
only the next 2 events are shown for the design sake.
data building scripts now run concurrently which decreases the build time.

How it looks

Related issue(s)
Related to #429

@derberg made some changes and It's easier to review now. also, it will fall back to the previous calendar if no data is found on the community repo. currently, it shows the previous calendar since it doesn't have access to the GITHUB_TOKEN? (I can be wrong though)

@KhudaDad414 yes, GITHUB_TOKEN is not accessible on a preview build for security reason, as anyone could easily console.log it I guess.

please share a screen shot how it looks like
please adjust the description of the PR to not use word Resolves is it will close the linked issue. Unless we actually solve it? I actually think it is solving current problems. The only thing that we could improve is current link to the Google Calendar, it is focused on users that already have a Google account....but not everyone has one, so people without google account will not see the calendar. There could be another, alternative button that links to<EMAIL_ADDRESS>a button for people that do not have google account

@derberg added a button to show the calendar. (updated the photo on PR description).
sorry for being late. I was watching India humiliate us at Cricket World Cup. 🔢

@KhudaDad414 should this go to calendar button be duplicated in code?

I was watching India humiliate us at Cricket World Cup

on the bright side, it was a Cricket game so you should be happy that you managed to watch it in one day and not one week 😆

@derberg refactored the whole implementation. (please look at Calendar.js in view mode it looks too messy in diff mode. )

As @derberg and @fmvilas is already well invested in this PR, I am gonna leave it up to them to review it @KhudaDad414 🙂


Add documentation of alpha status of gRPC

Add a note in the LoggingClient and PubSubClient documentation that gRPC support is at Alpha quality level.
Note: I have added this for pubsub as well as logging, even though pubsub is not going beta. I can remove this if you think it is not relevant.
cc @omaray

LGTM


imported bookmarks.html and now no custom bookmarks

Describe the bug
I ha a lot of custom bookkmars and it would be great idea if before the bookmarks were imported - that a forced bookmark backup was done.  Now i have lost all my custom ones...
To Reproduce
Steps to reproduce the behavior:

Go to import as HTML
select "Other Bookmarks"
select bookmarks.html

Expected behavior
In caase of error - have a forced bookmarks backup done so they could be reimported by "Manage Bookkmarks" thne restore  or hwatver the top menu selection is.
Desktop (please complete the following information):

OS:  Windows 10
Browser FF
Version 115

Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Now i have lost all my custom ones...

Sorry if you experienced something unpleasent, because of my addon and thanks for taking the time to help improve it.

it would be great idea if before the bookmarks were imported - that a forced bookmark backup was don

Well ... i could also just not remove existing bookmarks in the import folder.
Not 100% sure if that would align with user expectations, but i guess this would at least prevent accidental removal/loose of data. - Which i understand is pretty important.

What did you end up doing pick us? I also received. I guess the newest source code for your ad on.Sent via the Samsung Galaxy S23 Ultra 5G, an AT&T 5G smartphone
-------- Original message --------From: igorlogius @.> Date: 7/7/23  5:04 AM  (GMT-05:00) To: igorlogius/bookmark-branch-porter @.> Cc: CSGalloway @.>, Author @.> Subject: Re: [igorlogius/bookmark-branch-porter] imported bookmrks.html and
noew no custom bookmarks (Issue #5)
Now i have lost all my custom ones...
Sorry if you experienced something unpleasent, because of my addon and thanks for taking the time to help improve it.
it would be great idea if before the bookmarks were imported - that a forced bookmark backup was don
Well ... i could also just not remove existing bookmarks in the import folder.
Not 100% sure if that would align with user expectations, but i guess this would at least prevent accidental removal/loose of data. - Which i understand is pretty important.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

What did you end up doing pick us?

I am just not clearing  the import folder before you try to import something now.
That way you should not loose any data. Hope that helps.

Is there not a way to force a backup to:
%AppData%\Mozilla\Firefox\Profiles\i8mpx1mk.default-release\bookmarkbackups
folder?
Charles S Galloway
On Fri, Jul 7, 2023 at 9:38 AM igorlogius @.***> wrote:

What did you end up doing pick us?
I am just not clearing the import folder before you try to import
something now.
That way you should not loose any data. Hope that helps.
—
Reply to this email directly, view it on GitHub
https://github.com/igorlogius/bookmark-branch-porter/issues/5#issuecomment-1625430715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAX6VSWPSCEO7PRQM63OMTXPAGOHANCNFSM6AAAAAA2BKPBW4
.
You are receiving this because you authored the thread.Message ID:
@.***>

or would anytime the local bookmark file is written to, not read from, that
a copy of the bookmark file be copied to a subfolder of
%AppData%\Mozilla\Firefox\Profiles\i8mpx1mk.default-release\bookmark-branch-porter\bookmarkbackups
and maybe have it name the file YYYYMMDD_HHMMSS.html or .JSON or whatever
extension works the best?
===============
Charles S Galloway
On Fri, Jul 7, 2023 at 5:04 AM igorlogius @.***> wrote:

Now i have lost all my custom ones...
Sorry if you experienced something unpleasent, because of my addon and
thanks for taking the time to help improve it.
it would be great idea if before the bookmarks were imported - that a
forced bookmark backup was don
Well ... i could also just not remove existing bookmarks in the import
folder.
Not 100% sure if that would align with user expectations, but i guess this
would at least prevent accidental removal/loose of data. - Which i
understand is pretty important.
—
Reply to this email directly, view it on GitHub
https://github.com/igorlogius/bookmark-branch-porter/issues/5#issuecomment-1625096463,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAX6VWJAHKPAAHPVTJOBYDXO7GI5ANCNFSM6AAAAAA2BKPBW4
.
You are receiving this because you authored the thread.Message ID:
@.***>

closing since no accidnetal removal should now happend when importing something.
But if you want to further expand the issue, feel free to comment or open another ticket for it.

Hi. What does "Assigned #5 to CSGallowa" mean?
Charles S Galloway
On Sat, Jul 8, 2023 at 4:49 AM igorlogius @.***> wrote:

Assigned #5
https://github.com/igorlogius/bookmark-branch-porter/issues/5 to
@CSGalloway https://github.com/CSGalloway.
—
Reply to this email directly, view it on GitHub
https://github.com/igorlogius/bookmark-branch-porter/issues/5#event-9763322803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAX6VR57KVVCH2URCXX55TXPENLDANCNFSM6AAAAAA2BKPBW4
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Is it not possible to check if there are any bookmark backups in (
C:\Users[name]\AppData\Roaming\Mozilla\Firefox\Profiles[profile
name]\bookmarkbackups ) ?
Charles S Galloway
On Sat, Jul 8, 2023 at 9:07 AM igorlogius @.***> wrote:

closing since no accidnetal removal should now happend when importing
something.
But if you want to further expand the issue, feel free to comment or open
another ticket for it.
—
Reply to this email directly, view it on GitHub
https://github.com/igorlogius/bookmark-branch-porter/issues/5#issuecomment-1627256510,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAX6VRKISUXVBFPJCTOZJLXPFLQ3ANCNFSM6AAAAAA2BKPBW4
.
You are receiving this because you were assigned.Message ID:
@.***>

Hi. What does "Assigned #5 to CSGallowa" mean?

dont be alarmed, i just like to assign issues if there are open questions i am waiting on.
but i also unassign when questions has been answered.

Is it not possible to check if there are any bookmark backups in ( C:\Users[name]\AppData\Roaming\Mozilla\Firefox\Profiles[profile name]\bookmarkbackups ) ?

Sadly yes. It is AFAIK not possible for an addon to check this on its own.

i will reach out to Firefox support later today and see what they say.....
Charles S Galloway
On Sat, Jul 8, 2023 at 2:21 PM igorlogius @.***> wrote:

Is it not possible to check if there are any bookmark backups in (
C:\Users[name]\AppData\Roaming\Mozilla\Firefox\Profiles[profile
name]\bookmarkbackups ) ?
Sadly yes. It is AFAIK not possible for an addon to check this on its own.
—
Reply to this email directly, view it on GitHub
https://github.com/igorlogius/bookmark-branch-porter/issues/5#issuecomment-1627451656,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAX6VVU4NUBYYGY7RT2KVTXPGQLDANCNFSM6AAAAAA2BKPBW4
.
You are receiving this because you were mentioned.Message ID:
@.***>

i will reach out to Firefox support later today and see what they say.....

Ok, ... good luck i guess.
just fyi: This is the bookmark API
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/bookmarks
which is available to addons.
If you see/find anything that helps, feel free to comeback and we can discuss your findings.


Shadow ignore in  #:cleavir-bir so that cl:ignore is not fbound

thatsecond commit was wrong, aborting


The doc is deprecated

File: docker-cloud/getting-started/your_first_node.md
Im trying to create a node using the following tutorial, but it doesnt correpond with the actual version.

Thanks for sharing the issue. The issue appears to be no longer applicable to the current docs. Closing the issue. If the issue still exists, please let us know.


Report problems applying changes

At present various code keeps track of the return value of functions
that are called to apply changes ($retval). Then calls:
get_std_save_message($retval)
But get_std_save_message() always returns the same text "The changes
have been applied successfully."
So why is there are parameter to get_std_save_message()?
If we fix this up, returning different text to indicate that there is a
problem, then the subsequent print_info_box() that actually puts up the
message needs to pass the relevant severity also (success, warning,...).
So to minimize code that needs to be changed/duplicated in many places,
I have made print_apply_result_box() which takes $retval and sorts out
both the message text and severity all in 1 go.
system_routes has the typical changes that would be needed in the
top-level code to implement this.
Without changing top-level code, the changes in guiconfig.inc would
result in the problem text being reported when there is a problem, but
in the "success" color.

This is up for discussion, including:

The exact text of the message when there is a problem - is there always some evidence of the problem in System Logs somewhere?
Should the severity for the message box be "warning" or "danger" or?
Anything else?

The changes for interfaces.php are an example of a place where there is currently no "applied successfully" message displayed after pressing "Apply".
The changes in this example make the message happen and in the way that is integrated with the guiconfig.inc changes in this PR.

If the underlying infrastructure for this is agreed, I am happy to go through the GUI PHP files and make the small changes needed, like in the interfaces.php and system_routes.php examples.

I like the idea. What do you think @sbeaver-netgate and @jim-p ?

OK, I will look through the top-level UI code that uses get_std_save_message(), make the needed changes and push it...

Closing in favor of #3343 which has the (current) best/full solution.


Please give me some suggestion

Hi I'm a student and I'm new at Python Language.I'm interested in this project and I don't How it work after fingerprinting the audio. Can you suggest me some How It work  and testing code? Thank you for suggestion

Go through the README, you'll find examples there.

Hi, @Chococobee if you want dejavu to listen and identify a song try these steps:

Open your terminal and navigate to the dejavu directory.
Once inside the dejavu directory type:  python dejavu.py -r mic 5

python dejavu.py will start the program.
-r specifies the action for dejavu to perform. In this case -r means recognize. You are telling dejavu to listen and identify an audio.
mic specifies the source of the audio.
5 specifies the number of seconds dejavu should listen to the audio. You can set this to a value of your choice

Play a song into the microphone of your machine. NB: The song you play should be a song that you have already fingerprinted and stored in the database.
After some seconds, dejavu should output the song or audio file it recognized.

I hope this helps you.


Publish 07/30/2021, 10:30 AM

Docs Build status updates of commit 55d2dcc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


New Research: Finding Beacons in the Dark

Synopsis

Finding Beacons in the Dark: A Guide to Cyber Threat Intelligence is the most comprehensive collection of cyber threat intelligence (CTI) focused on Cobalt Strike team servers ever produced.
Learn what you can do to proactively protect your organization from the growing threat of Cobalt Strike beacons and team servers from our cybersecurity experts. You’ll learn key CTI concepts and how to build a CTI program that can effectively find beacons in the dark.

Highlights include tips for:
Building out detailed profiles of threat actors
Broadening your knowledge of existing threat groups
Tracking both ongoing and new threat actor campaigns
Providing intelligence for SOC analysis and incident responders
Fine-tuning security solutions
Fine-tuning IDS/IPS solutions
Ascertaining campaign timelines for future attacks and incident response engagements

Blog: https://blogs.blackberry.com/en/2021/11/all-your-beacon-are-belong-to-us-new-blackberry-book-cracks-code-of-cobalt-strike-threat-actors
Direct Link: https://www.blackberry.com/content/dam/blackberry-com/asset/enterprise/pdf/direct/bb-ebook-finding-beacons-in-the-dark.pdf

Edit: just realised this is already on the list, but embeds the PDF in a viewer. The direct link might be better (or maybe link to the blog, with a seperate direct link to the PDF?).


Add support for conversations/models APIs

Change Summary

PR Checklist

[x] I have read and signed the Contributor License Agreement.

@kishorenc, Kishore, hi! Could you please take a look?


fix: DnsHole controls timer

Homarr

Thank you for your contribution. Please ensure that your pull request meets the following pull request:

[x] Builds without warnings or errors (pnpm buid, autofix with pnpm format:fix)
[x] Pull request targets dev branch
[x] Commits follow the conventional commits guideline
[x] No shorthand variable names are used (eg. x, y, i or any abbrevation)

I think it's can be merge


5.3.x X509 Rest Authentication Fixes

I initially did the work for PR #3478 in my 5.3.x branch, as I don't have a JDK 10 environment to test 6.0 yet (it's a WIP).  So the 5.3.x has been validated to work with my cas-overlay and configurations.

I don't know how to address the issue that showed up in the CI build, https://travis-ci.org/apereo/cas/jobs/417679833

The error in tests likely is because your new configuration class is not included in the test suite; thus your extractor is never picked up by the test execution and the ifAvailable() method that is supposed to pick it up returns null.

I don't know how to get the tests to pick it up.  When i run "gradlew test coveralls" locally it works...

@Import(value = {X509AuthenticationWebflowConfiguration.class, X509AuthenticationConfiguration.class})
public class X509CertificateCredentialsRequestHeaderActionTests extends AbstractX509CertificateTests {
...

See the list of classes that are imported by the class. Include yours.
And, you need to run the same command as Travis CI:
./gradlew --stacktrace --build-cache --configure-on-demand --no-daemon -DtestCategoryType=SIMPLE test coveralls --parallel \
-x javadoc -x check -DskipNpmLint=true -DskipGradleLint=true -DskipSass=true -DskipNpmLint=true --parallel \
-DskipNodeModulesCleanUp=true -DskipNpmCache=true -DskipNestedConfigMetadataGen=true

You can see the command in the build output for Travis CI.

alright, i'll get it working then submit next commit.  then i'll update the master branch's PR for these changes.

somehow the CI job got cancelled, and in my attempt to play nice, i don't want to just make a commit to trigger the CI job.
I believe this should work.  Running the CI tests locally failed utterly at
java.lang.AssertionError
        at org.junit.Assert.fail(Assert.java:86)
        at org.junit.Assert.assertTrue(Assert.java:41)
        at org.junit.Assert.assertFalse(Assert.java:64)
        at org.junit.Assert.assertFalse(Assert.java:74)
        at org.apereo.cas.audit.CasSupportJdbcAuditConfigurationTests.verifyAuditManager(CasSupportJdbcAuditConfigurationTests.java:59)

I restarted the job for you.

as far as I can tell this is fully ready, minus the lack of the Codacy check.


Add BEYLA_CONFIG_PATH env var

Adding @grafsean as a reviewer, as this PR affects documentation.
As an addition to the -config CLI argument, the introduced BEYLA_CONFIG_PATH env var would allow cleaner configuration in container-based environments, not forcing the users to override the /beyla command invocation (which should be kept as an internal detail).
The good-old -config CLI arg will keep working as usual.

@grcevski very good point. I'll change it to BEYLA_CONFIG_PATH

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

Comparison is base (99088fd) 67.64% compared to head (2a3f712) 44.91%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #549       +/-   ##
===========================================
- Coverage   67.64%   44.91%   -22.73%
===========================================
  Files          66       67        +1
  Lines        5687     5628       -59
===========================================
- Hits         3847     2528     -1319
- Misses       1553     2946     +1393
+ Partials      287      154      -133

Flag
Coverage Δ

integration-test
?

unittests
44.91% <ø> (?)

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

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

Looks good, little copy touched!


pull request for e2e test at 12/06/2016 21:56:05

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


fixed The jar type of java task shell script error

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Could you raise an issue and discribe the bug, thanks.

Could you raise an issue and discribe the bug, thanks.

you can read from :https://github.com/apache/dolphinscheduler/issues/15641

If you use -jar, you need to specify the entry class, but the page cannot specify the entry class on the configuration page when you select the jar type
------------------ 原始邮件 ------------------
发件人: "Wenjun @.>;
发送时间: 2024年3月5日(星期二) 上午10:33
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [apache/dolphinscheduler] fixed The jar type of java task shell script error (PR #15645)
@ruanwenjun requested changes on this pull request.
In dolphinscheduler-task-plugin/dolphinscheduler-task-java/src/main/java/org/apache/dolphinscheduler/plugin/task/java/JavaTask.java:
> @@ -183,9 +184,8 @@ protected String buildJarCommand() {          StringBuilder builder = new StringBuilder();          builder.append(getJavaCommandPath())                  .append("java").append(" ") -                .append(buildResourcePath()).append(" ") +                .append(buildExtDirs()).append(" ")
Can we refactor this, use -cp rather than -jar.
$JAVA_HOME/bin/java $JVM_ARGS \   -cp "$taskInstanceWorkingPath" \   Main.class
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


show outdated extension for Check for Extension updates action

This fixes #55114

Thanks for the patience 😄. Hope it works now as expected.

I tried out the changes. Works very well when I run the Check for Extensions Updates from the command pallet and the extensions view is hidden to begin with.
But when the extensions view is open, and I then run the Check for Extension Updates either from command pallet or from the menu, the extensions get the Update to ... label in existing view (which is for enabled extensions) and then the @outdated view is shown. This causes a flicker like perception which is not very nice.
How about the below?

Instead of using @outdated as the search term, lets use empty string. That will show the default view which will contain the enabled extensions and the outdated ones will have the Update to ... label.
We shouldnt do the above if there are no updates available.
We should show the number of extension updates in a notification.

I agree with your suggestion.
But it would be better if we also show a notification if no updates are available, otherwise, there would be no feedback to the end user on the action as the default view would also be not shown as per your suggestion.

Also, with what you've said, we need to show the number of extension updates.
But the extensionsWorkbenchService.checkForUpdates() (calls syncWithGallery()) returns a void promise; do we need to count the available extension updates and return as Promise<int> then make use of it?
In order to show the notification, I believe that we might need to add @IDialogService param (hope this would be served automatically) to this ExtensionAction and import vs/base/common/severity. Correct me if I am wrong.

But it would be better if we also show a notification if no updates are available,

Yes definitely, my third point (We should show the number of extension updates in a notification) holds even if the number is 0 :)

do we need to count the available extension updates

Yes, once extensionsWorkbenchService.checkForUpdates is done, you can use extensionsWorkbenchService.queryLocal() which will return a list of installed extensions. These extensions have a boolean property called outdated that you can use.

In order to show the notification, I believe that we might need to add @IDialogService param(hope this would be served automatically)

Use the INotificationService instead. Yes, it will be available automatically.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ParkourKarthik sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@ramya-rao-a
I've made the changes as you requested. Though I couldn't test the outdated extensions in real-time, I believe it should work fine.
I've shown a different message when no extensions are outdated - All Extensions are up to date. Let me know if you still think this needs to show the same outdated message with 0 Extensions.

I got carried away with all the features we kept adding to this PR :)
Below is the case where our idea of having the button to update in the notification fails to be a good experience

User clicks on the Check extension updates from the menu in the extensions view
The outdated extensions get the Update to .. label, and the notification shows up as well
If the user clicks on the Update to .. label on the extension
At this point our notification is still open asking user to update the extension

I'll push a commit to remove the option. Apologies for the churn.

Also, since you now know everything about outdated extensions, #22461 would be a good next item if you are interested in picking another item in VS Code :)

I'm ok with your changes, you can merge it :)


Create custom game icon

It's time for us to make our own custom icon for the game instead of stealing it from some website

please assign me this issue


[Question] How do we print chinese characters

Hi @NielsLeenheer
Thank you for building this encoder.
I've a doubt though,
In the example you've mentioned setting codepage cp936 for printing Chinese text, but this codepage is not included in the supported list of codepages.
We need to print Chinese characters in the receipt. Any idea on how do we achieve this?

I've removed the Chinese example from the README, as Star printers do not support Chinese codepages. I'll investigate if there is an alternative way for the printer to support Chinese text, but unfortunately currently the library only supports non-latin characters using codepages.


Displays the old page twice

Hi there,
i would love to use this great plugin for a drupal installation. The transition itself works fine but the old page displays twice. After one click, the animation begins. After fading out the old content, the page displays again followed by the url request. The new page loads followed by the choosen transition.
Are there ways to solve this?
Thank you very much

Hi
Sir, Could you please tell me how this can be used in div transition. I want to keep the header and footer only the central div should move. Is this possible with this.


Fix client upload code not working for WithToken option

Pelican Version:
found in v7.8.3
Pelican Service:

[X] Client

Ismael discovered a bug in the client code that shows it does not work with the WithToken client transfer option. Here is the snippet from slack describing the bug and its fix:
I think I found a bug causing your issue: https://github.com/PelicanPlatform/pelican/blob/3edb26b657f0e3dff91ee437f60af105ddcc7bc7/client/handle_http.go#L1091-L1096
We're missing some parentheses in that if statement. It seems like it should be: if (upload || ns.UseTokenOnRead) && tj.token == "" . Basically, what is happening is tj.token should be set to whatever the client (tc)'s token is, but is overwritten in this block here meaning that the token you are sending is getting erased in our code. I'll work in a patch for you so this can be fixed.

If we got a user report of a regression / bug, I'd really like to see an accompanying test.  Doesn't look like it'd be that hard to implement here...


System.Threading.Thread.Abort changes

A note about System.Threading.Thread.Abort is missing. On .NET Core it throws a "System.PlatformNotSupportedException: Thread abort is not supported on this platform." exception, while on .NET Framework it does not.
What is the equivalent on .NET Core for Thread.Abort ?!

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

@PriyaPurkayastha Should this be added as a breaking change?

@gewarren we will figure out next steps once I get some information back about existing documentation for .NET Core releases prior to .NET Core 3.0.

Related issue: https://github.com/dotnet/corefx/issues/35229

Is there any recent development on this? Thread.Abort() can still be typed without any kind of intellisense from the editor that it is obsolete (.NET Core 3.1 here). Souldn't the Abort becoming obsolete and/or moving out of .Net Core be mentioned in the documentation?

@PhilipHassialis The .NET Core behavior is mentioned in the API documentation here, but I don't think this surfaces in IntelliSense. We also have a new page in the works that lists all APIs that throw PlatformNotSupportedException on .NET Core.

Thank you so much for the prompt reply, unfortunately most of the courses/tutorials/etc out there do not adhere to Core and keep on mentioning stuff that is already deprecated. Imho, a little mention of "this doesn't work in Core" in intellisense would be a godsend - along with an immediate mention in the title of the page or in the first paragraph of the documentation that directly indicates that a method/API will not work in Core.
As a bonus, the documentation team could have separate pages that instead of having an example would simply mention that "....this method is deprecated and is supported only in Framework and not in Core", no examples, nothing the user can copy paste to their IDE only to see it fail, only the exceptions and a "do not use obsolete APIs" warning.
Once again, thanks so much for responding, happy new year and a million apologies for my rushed questioning.


feat: improve UX on macOS

Hide titlebar and integrate traffic-light controls into app. This looks cooler. FACT.
Show IPFS icon in dock and app switcher when webui window open. You can now tab back into the webui window after jumping to a finder window to get your files.

Integrated menubar controls

App icon in dock and app switcher when webui window open

License: MIT
Signed-off-by: Oli Evans<EMAIL_ADDRESS>

I'm not getting the IPFS logo in the dock nor on the app switcher:

Is this a bug or am I doing something wrong?

That is a quirk of Dev mode. If you npm run build and install from the .dmg in  dist and run the packaged app, you should get an IPFS logo


Extensions-api, new nav-functions, route-api

#74 - add nav functions that operates with NavEntry (navigate, replace etc)
#72 - add Route-api to be able to run complex navigation configured in 1 place (eg: Route)
add new Extnesions-api to be able to attach some logic to nav destinations and work with them (similar how plugins works in gradle)

Too complex builder, will rework it base on #88 idea


Usage in React

I would like to import this extension into React but have continually been getting TypeErrors. Is it possible to import this and if so what is the correct syntax?

@kristnr  What errors are you getting? Can you put some? Also, note that this extension has some dependencies like
Cytoscape.js ^3.2.0
jquery ^1.7.0 || ^2.0.0 || ^3.0.0
konva ^7.0.3
cytoscape-undo-redo ^1.0.10 (optional)

@canbax My errors are pretty simple. Mainly it has been "TypeError: cy.nodeEditing is not a function" or "TypeError: ext.apply is not a function", but as I try different ways of importing, the error fluctuates a little.
Currently, I have:
import $ from 'jquery';
import Konva from 'konva';
import cytoscape from 'cytoscape';
import CytoscapeComponent from 'react-cytoscapejs';
import nodeEditing from 'cytoscape-node-editing';
cytoscape.use(nodeEditing);
I thought about importing react-konva as well, but didn't think it was needed.

Revisiting this, I found that the import instructions on the page were correct. I may have had a typo or was missing a requirement at some point prior. Here is what I have now:
import $ from 'jquery';
import cytoscape from 'cytoscape';
const konva = require('konva');
const nodeEditing = require('cytoscape-node-editing');
nodeEditing(cytoscape, $, konva);


Cleanup duplicate station metrics

Combine "user" and "guest" station metrics into a single labeled metric
to avoid duplicate storage of the same data.
Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

Technically, I don't think we need unifipoller_device_stations for UAPs, as unifipoller_device_radio_stations covers the same data. But it may be different if there are non-UAP devices, which I don't have any of to test with.

Updated, PTAL.

This is why we tag latest from the release branch, rather than on master for Prometheus stuff. :-)
We should put the dashboard json in the repo so we can sync changes.

I'll let you merge when you're ready.

I'm generally fine with latest having "problems" - I recommend people use the stable tags for production. But 1.6 broke ENV variables in Docker that are now fixed in master, so I'm at a bit of a bridge I need to find a way to cross safely.
Since this requires updating the dashboards, I'm going to cut a new release (1.6.1). If you want to stab in some more changes, please do so, or let me know so I can get to them! I'll work on all of this later this evening (it's only 2:30, and you know I'm up late).

Prometheus latest doesn't have problems. :stuck_out_tongue_winking_eye:


[FR] add support for cross-compiling of CPython and Cython modules using setuptools

What's the problem this feature will solve?
I'm trying to cross compile CPython and Cython modules to a different platform (like Android) using setuptools
Describe the solution you'd like
Any solution that supports cross compiling, plus some documentation
Some possible approaches:

emit a CMake or Meson file that supports cross-compiling

Alternative Solutions
I tried "crossenv" but it doesn't work with Android ndk / clang.
Currently the other approach I use is to emit the build commands and manually write a makefile
Additional context
No response
Code of Conduct

[X] I agree to follow the PSF Code of Conduct

In Nixpkgs we are able to cross-compile Python packages (including cython). Note you may want to use meson (through mesonpep517) instead of setuptools directly. With meson 0.59.0 cython has first class support.


[WIP] Allow selection of AWS profile during initalization

Currently, there is no way to specify a credentials profile from ~/.aws/credentials to use. This PR adds --profile to moactl init to allow for that selection
Usage:
moactl init --profile my-moa-creds

WIP since I'm getting a null pointer after creds validation
 ./moactl init --profile osd-staging-1
I: Logged in as 'tfahlman.openshift' on 'https://api.openshift.com'
I: Validating AWS credentials...
I: AWS credentials are valid!
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xc467c8]

Closing until I have something working. Someone else can also take a run at it


Provide access to the Oculus mobile plugin init and runtime parameters.

This PR reworks the initial implementation based on the discussion in issue #35.
It also addresses issue #43 by updating the vr mode lifecycle of the oculus mobile plugin based on the updates from godotengine/godot/PR#32064

should this be a (initialized || enter_vr_mode() instead of a && ?

@NeoSpark314  It's intended to be a && since at this point of the plugin lifecycle, it should have already been initialized. If we're in an invalid state and the plugin is not initialized, we want to fail early.


[replace] sections

Using cargo vendor on packages with [replace] sections seems to fail with a warning about duplicate sources. Specifically I had that problem with rustup: NixOS/nixpkgs/issues/30742. I could work around that (rust-lang-nursery/rustup.rs#1276), but there should probably be a general solution.
I made a naive attempt to solve this by only adding a pkg to the ids map, if it is not replaced. cargo vendor succeeds then, but cargo build says with the printed .cargo/config:
   error: checksum for `mio v0.6.11` could not be calculated, but a checksum is listed in the existing lock file

this could be indicative of a few possible situations:

    * the source `registry https://github.com/rust-lang/crates.io-index` supports checksums,

      but was replaced with one that doesn't
    * the lock file is corrupt

unable to verify that `mio v0.6.11` is the same as when the lockfile was generated

Yeah I think to handle replace/patch this'll need some intrusive management of the lock file, or at least it's not clear immediately how it'd be handled :(

This is a very old bug at this point and I think it's fixed with [patch] which has replaced [replace], so I'm going to close this.


db-sync stale snapshots

On https://update-cardano-mainnet.iohk.io/cardano-db-sync/index.html#13.1 page there are only stale snapshots (11 days ago) but according the docs the snapshots should be published every epoch ~ every 5 days
I would expect 2 more snapshots during that period of time

@jbgi Can you please look into that ?

snapshots are back now. Thank you for reporting the issue.


Fix function signatures for getAllAccounts and getAccountByUsername

Please follow the issue template below.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Upcoming Library

[ ]<EMAIL_ADDRESS>
Source

[x] Internal (MSAL.js team)
[ ] Customer request

Description
Today, the signatures for PublicClientApplication.getAllAccounts and PublicClientApplication.getAccountByUsername do not indicate that these methods can return null.
Priority
Medium
Acceptance Criteria

PublicClientApplication.getAllAccounts and PublicClientApplication.getAccountByUsername return types are updated to indicate they can return null.

Wow! I just hit this 1 hour ago but you already had this issue created. Thanks!


Feature: About the application

1. Description
User should be provided with general information of the application.
This should contain at least application version, CE-marking, company and copyright information.
2. Requirements and dependencies

Should be accessible via Help menu.
Should not require login.
Should not be restricted to any specific application state.
Should not be easily alterable when application is installed.

Dependencies on other issues:
Required (at least) for #24, #25, #26.
3. Implementation details
Implementation plan:

Simple html-page included in resources.
Some data is gathered from package.json in build-time when compiling the template to the actual page.
Short BSD-2 license initially hidden, can be shown (and hidden again) with a button press.

Target schedule:
3.1 OWASP top 10
https://github.com/OWASP/www-project-top-ten/blob/master/index.md

1 Injection - No user input.
2 Broken Authentication - No authentication.
3 Sensitive Data Exposure - No server communication. No IPC communication.
4 XML External Entities (XXE) - No user-entered data. "Browsed" by Chrome-based solution, Electron.
5 Broken Access Control - Public feature.
6 Security Misconfiguration - N/A: No server communication.
7 Cross-Site Scripting XSS - No user-entered data.
8 Insecure Deserialization - No deserialization by our code. Electron/ASAR deserializes the page from archive within app installation.
9 Using Components with Known Vulnerabilities - No change (on runtime).
10 Insufficient Logging & Monitoring - N/A: No server communication.

4. Verification
Verification plan:
To be verified by following steps:

See that about window is shown from Help -> About application (or similar) menu item.
Content of the about window is correct. CE-mark and manufacturer icon are visible.

Verification results:

[ ] 1. Pass/Fail (SW version, date)
[ ] 2. Pass/Fail (SW version, date)

Implemented in f7e64b62f88913bccb01e720dfa5e40166bcd3b8 with static html generator. Not localized.