PasswordFlow with Cookie-Storage does not work each time

Hey,
first of all, thank you for your create work and my bad english :).
I tried different storage solutions but i need to use the cookie storage for login on different subdomains.
My Angular Version is 7 and had the same on 6.
Sometimes after calling the login with PasswordFlow:
 fetchTokenUsingPasswordFlowAndLoadUserProfile(email, password)  .then((user) => { // OK    })       .catch((err) => {          // ERROR       });
i will get an error (but not in chrome, only in Safari or Firefox). As an cookie storage i used this one:
https://www.npmjs.com/package/cookie-storage
I checked what could be the problem. Login is successfull and the access_token is stored as an cookie but i think not fast enough.
After checking the code, i found one part that could be a problem:
https://github.com/manfredsteyer/angular-oauth2-oidc/blob/0274cf5c03080e2a139abbf84d2bb9dd2480f3c2/projects/lib/src/oauth-service.ts#L688
storeAccessTokenResponse is called on token response but async. it resolves the token response but the caller in line 562 https://github.com/manfredsteyer/angular-oauth2-oidc/blob/0274cf5c03080e2a139abbf84d2bb9dd2480f3c2/projects/lib/src/oauth-service.ts#L562
doesnt work with the response and calls the
https://github.com/manfredsteyer/angular-oauth2-oidc/blob/0274cf5c03080e2a139abbf84d2bb9dd2480f3c2/projects/lib/src/oauth-service.ts#L574
method wich checks the access token first.
if the browser or the storage is not fast enough, the access token check says that the token is not valid because it doesnt exists.
Is it possible to return the token response and work with it instead checking the storage?
Thanks
David

Hmm, I don't see why you'd assume storing the value would be "too late"? The call to this.storeAccessTokenResponse(...) you mention is done syncrhonously, and it also calls this._storage.setItem(...) synchronously.
In addition, the examples on cookie-storage's GitHub and NPM pages suggest that the storage itself is also a synchronous operation.
Could you create a minimal example, e.g. using StackBlitz, so we can further assist?
Either way, if it really is an issue with sync/async, you could implement some kind of spin lock in your promise chain, something like this:
fetchTokenUsingPasswordFlowAndLoadUserProfile(email, password)
  .then(user => {
    return new Promise((resolve, reject) => {
      const rejection = setTimeout(() => reject('too long!'), 5000);
      function resolveIfDone() {
        if (cookieStorage.getItem('access_token')) {
          clearTimeout(rejection);
          resolve(user);
        } else {
          setTimeout(() => resolveIfDone(), 100);
        }
      }
      resolveIfDone();
    });
  })
  .then(user => { /* OK */ });

I think i found the error.
Setting the cookie options domain cause that the cookie is not written in ie.
// Use default cookie options
const storage = new CookieStorage({
  path: '/',
  secure: false,
  sameSite: 'Strict' // Can be 'Strict' or 'Lax'.
});

Good to hear that you've found the error. I think it is then in (the usage of) the cookie storage lib, right? This issue can then probably be closed.

Yes, your right. Thanks for your support.

What was the exact solution to this? Did you have to change something on your authentication service (specifically this 'CookieStorage' part by @davidk85), or did you do some magic within your Angular project?


Apparent race condition

I found about this issue while debugging the new implementation of MQTT. At first I though my code was buggy, then I found out that the problem occurs while using same old HTTP transport as well.
Error message:
2015-09-16 13:48:15,448:ERROR:cohorte.forker.basic: Error waiting for isolate c9755a6c-183a-475d-903a-caac5b6ffae1 (toto-2) to be loaded: ('Unknown UID after timeout: %s', 'c9755a6c-183a-475d-903a-caac5b6ffae1')
2015-09-16 13:48:15,888:ERROR:cohorte.composer.node.commander: Error calling composer on isolate toto-2: Peer toto-2 (c9755a6c-183a-475d-903a-caac5b6ffae1) has been lost
Traceback (most recent call last):
  File "/Users/ahmad/opt/cohorte/repo/cohorte/composer/node/commander.py", line 228, in __push_orders
    composer.instantiate(components)
  File "/Users/ahmad/opt/cohorte/repo/herald/remote/herald_jabsorbrpc.py", line 242, in __call__
    reply_message = self.__send(self.__peer, self.__subject, request)
  File "/Users/ahmad/opt/cohorte/repo/herald/remote/herald_jabsorbrpc.py", line 277, in __call
    return self._herald.send(peer, beans.Message(subject, content))
  File "/Users/ahmad/opt/cohorte/repo/herald/core.py", line 793, in send
    if event.wait(timeout):
  File "/Users/ahmad/opt/cohorte/repo/pelix/utilities.py", line 524, in wait
    raise self.__exception
herald.exceptions.PeerLost: Peer toto-2 (c9755a6c-183a-475d-903a-caac5b6ffae1) has been lost

Reproduction
To reproduce, run Cohorte with more than one isolat. Once started, issue peers on the console, until one peers is discover, but not all of them. As fast as possible, issue shells. If shells show all isolate, it's already too late for the error to be reproduced; retry.
More info
Issuing threads give the following output:
Thread ID:<PHONE_NUMBER> - Name: Pelix-Shell-TextConsole
Stack Trace:
  File "/Users/ahmad/opt/cohorte/repo/pelix/shell/core.py", line 549, in ShellService::threads_list
    - self = <pelix.shell.core.ShellService object at 0x10493c5f8>
    - io_handler = <pelix.shell.beans.ShellSession object at 0x102511978>
    - max_depth = 1

         while frame is not None \
                 and (max_depth is None or depth < max_depth):
             # Store the line information
      >>     trace_lines.append(self.__format_frame_info(frame))

             # Previous frame...
             frame = frame.f_back

Thread ID:<PHONE_NUMBER> - Name: IOWatcher-5271c666-036d-41eb-a457-a959ba464f01
Stack Trace:
  File "/Users/ahmad/opt/cohorte/repo/cohorte/forker/watchers.py", line 199, in IsolateWatcher::__io_thread
    - self = <cohorte.forker.watchers.IsolateWatcher object at 0x1049b1160>
    - uid = '5271c666-036d-41eb-a457-a959ba464f01'
    - process = <subprocess.Popen object at 0x1066d2320>
    - event = <threading.Event object at 0x1066d2160>

         logger = logging.getLogger("io_watcher.{0}".format(uid))

         while not event.is_set():
      >>     line = process.stdout.readline()
             if not line:
                 break

Thread ID:<PHONE_NUMBER> - Name: Herald-GC
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 294, in Condition::wait
    - self = <Condition(<_thread.lock object at 0x1037092b0>, 1)>
    - timeout = 30

             gotit = True
         else:
             if timeout > 0:
      >>         gotit = waiter.acquire(True, timeout)
             else:
                 gotit = waiter.acquire(False)
         return gotit

Thread ID:<PHONE_NUMBER> - Name: Thread-1
Stack Trace:
  File "/Users/ahmad/Lab/cohorte_test/kweso-master/repo/paho/mqtt/client.py", line 799, in Client::loop
    - self = <paho.mqtt.client.Client object at 0x10370aba8>
    - timeout = 1.0
    - max_packets = 1

         # call to publish() etc.
         rlist = [self.socket(), self._sockpairR]
         try:
      >>     socklist = select.select(rlist, wlist, [], timeout)
         except TypeError:
             # Socket isn't correct type, in likelihood connection is lost
             return MQTT_ERR_CONN_LOST

Thread ID:<PHONE_NUMBER> - Name: IOWatcher-Notify-0
Stack Trace:
  File "/Users/ahmad/opt/cohorte/repo/cohorte/composer/node/composer.py", line 309, in NodeComposer::instantiate
    - self = <cohorte.composer.node.composer.NodeComposer object at 0x1025e1668>
    - components = {RawComponent(toto, toto2)}

         # Stop the running timer, if any
         self.__stop_timer()
         # wait already created isolates to get up. TODO: should enhance this solution!
      >> time.sleep(5)
         try:
             # Compute the implementation language of the components
             bundles = self._compute_bundles(components)

Thread ID:<PHONE_NUMBER> - Name: Isolate-Waiter
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 294, in Condition::wait
    - self = <Condition(<_thread.lock object at 0x10497e8c8>, 1)>
    - timeout = 0.5

             gotit = True
         else:
             if timeout > 0:
      >>         gotit = waiter.acquire(True, timeout)
             else:
                 gotit = waiter.acquire(False)
         return gotit

Thread ID:<PHONE_NUMBER> - Name: HttpService-52632-Server
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/socketserver.py", line 154, in _eintr_retry
    - func = <built-in function select>
    - *args = ([<pelix.http.basic._HttpServerFamily object at 0x1055776a0>], [], [], 0.5)

         """restart a system call interrupted by EINTR"""
         while True:
             try:
      >>         return func(*args)
             except OSError as e:
                 if e.errno != errno.EINTR:
                     raise

Thread ID:<PHONE_NUMBER> - Name: RemoteShell-0
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/socketserver.py", line 154, in _eintr_retry
    - func = <built-in function select>
    - *args = ([<pelix.shell.remote.ThreadingTCPServerFamily object at 0x105577a20>], [], [], 0.5)

         """restart a system call interrupted by EINTR"""
         while True:
             try:
      >>         return func(*args)
             except OSError as e:
                 if e.errno != errno.EINTR:
                     raise

Thread ID:<PHONE_NUMBER> - Name: IOWatcher-4f6cf583-bb1d-4624-b183-56a0185bbaf4
Stack Trace:
  File "/Users/ahmad/opt/cohorte/repo/cohorte/forker/watchers.py", line 199, in IsolateWatcher::__io_thread
    - self = <cohorte.forker.watchers.IsolateWatcher object at 0x1049b1160>
    - uid = '4f6cf583-bb1d-4624-b183-56a0185bbaf4'
    - process = <subprocess.Popen object at 0x106295048>
    - event = <threading.Event object at 0x1065f5898>

         logger = logging.getLogger("io_watcher.{0}".format(uid))

         while not event.is_set():
      >>     line = process.stdout.readline()
             if not line:
                 break

Thread ID:<PHONE_NUMBER> - Name: NodeComposer-Starter-0
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 294, in Condition::wait
    - self = <Condition(<_thread.lock object at 0x10252c2b0>, 1)>
    - timeout = 59.99999814400144

             gotit = True
         else:
             if timeout > 0:
      >>         gotit = waiter.acquire(True, timeout)
             else:
                 gotit = waiter.acquire(False)
         return gotit

Thread ID:<PHONE_NUMBER> - Name: HeraldNotify-3
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 294, in Condition::wait
    - self = <Condition(<_thread.lock object at 0x103655c38>, 1)>
    - timeout = 59.999998245000825

             gotit = True
         else:
             if timeout > 0:
      >>         gotit = waiter.acquire(True, timeout)
             else:
                 gotit = waiter.acquire(False)
         return gotit

Thread ID:<PHONE_NUMBER>57472 - Name: MainThread
Stack Trace:
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 290, in Condition::wait
    - self = <Condition(<_thread.lock object at 0x1023f5508>, 1)>
    - timeout = None

         gotit = False
         try:    # restore state no matter what (e.g., KeyboardInterrupt)
             if timeout is None:
      >>         waiter.acquire()
                 gotit = True
             else:
                 if timeout > 0:

Content of composition.js:
{
    "name": "kweso-ibm-demo-device",
    "root": {
        "name": "kweso-ibm-demo-device-composition",
        "components": [

            {
	"name": "toto1",
	"factory": "toto",
	"node": "kweso-master",
	"isolate" : "toto-1",
	"properties": {
		"herald.rpc.target.group": "monitors"
 	}
},
{
	"name": "toto2",
	"factory": "toto",
	"node": "kweso-master",
	"isolate" : "toto-2"
},
{
	"name": "toto3",
	"factory": "toto",
	"node": "kweso-master",
	"isolate" : "toto-3",
	"properties" : {
		"herald.rpc.target.group": "toto-1"
	}
}
         ]
    }
}

To me, Cohorte seems to deal-lock somewhere, considering the number of threads waiting.
This is quit tricky to reproduce, and unlikely to happen in "normal" operation.

False alarm!
This issue only concerns MQTT indeed. It is virtually not producible on HTTP, as peers are created faster than in MQTT (in order to reproduce the rase condition, a shells must be issued while peers are being created).
This remains an issue for MQTT transport.


Willian Lauber - MongoDB -Aula5 Exercícios 1 à 6 - Resolvidos

Cuidado com as formatações! https://github.com/WillianLauber/be-mean-instagram-mongodb-exercises/blob/e740d08b1e1a46544e0ce7d93bca5abaf7a34a8c/class-05/class-05-resolved-willianlauber-Willian-Alberto-Lauber.md


Bug - Create a Block Element Bootstrap Button - Won't finish submitting.

Challenge Name: Create a Block Element Bootstrap Button
Link: https://goo.gl/7IEJ4j
Issue:
Challenge wants you to bootstrap the btn-block class within the btn class. After entering in the code specified you can see the changes occur on the right side, with the button expanding across the screen. How ever the check off area it remains red, and when trying to hit submit it does not proceed. Here is the code that it wants you to submit -
<button class="btn btn-block">Submit</button>
and here is a screenshot of the code in action along with the red check-off area.

Troubleshooting:
Closed and reopened browser
Cleared Cache/Cookies/Temp Files.
Attempted to use another browser (iexplore)
Reset code.
Deleted and re-wrote code.
Moved the positioning of the code, ie <button class="btn-block btn">Like</button>
Swapping the code around resolved the issue.  After changing the location of the class elements and hitting submit it proceeded as normal. I believe this is likely a bug

Browser Name, Version:  Firefox 47.0
Operating System:  Windows 10
Mobile, Desktop, or Tablet:  Desktop

@g3istb0t Forgive me if this sounds obtuse, but are you clicking the "Run Tests" button or the "Submit" button because the latter is an example in the instructions and you need to click "Run Tests" to proceed. I ask because I replicated your steps and it worked for me (i.e. the tests passed) though I am on a diff. machine.

The tests need to be updated to allow any order for the classes.
<button class="btn btn-block"> and <button class="btn-block btn"> both are correct.

@raisedadead Both pass fine for me?

Oh, bummer. Yes. IDK the first instant I could not pass.
@g3istb0t Can you copy and share your original code in here?

I can't reproduce

Same here, can't reproduce this.
@g3istb0t Just to be absolutely clear, when you say

How ever the check off area it remains red, and when trying to hit submit it does not proceed

and in general when you refer to pressing the submit button, you do mean the "Run tests" button and not any of the buttons actually labeled as Submit, correct?

Closing as stale. Please reopen if this issue still occurs!

This is still an issue. Using Safari 9.1.2 to do the course and can't get past this section. I have also tried re-starting browser without luck and resetting the level without luck.

Like
Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Crazy

Submit

`  Like
Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Crazy

Submit`

This happened to me today, but changing the order of the classes didn't fix it.
Safari 9.1.2
OS X El Capitan 10.11.6
Desktop

Same issue
Chrome
Linux

@monstermeat @XenoDelMoto
To be sure, you are still hitting the Run Tests (ctrl+enter) button to complete the challenge?
I still can't reproduce.
@XenoDelMoto it would be great if you could paste your code here for us to see

CatPhotoApp
Click here for cat photos.

Like
Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Crazy

Submit

On Tue, Sep 27, 2016 at 10:20 AM, Stuart Taylor<EMAIL_ADDRESS>wrote:

@monstermeat https://github.com/monstermeat @XenoDelMoto
https://github.com/XenoDelMoto
To be sure, you are still hitting the Run Tests (ctrl+enter) button to
complete the challenge?
I still can't reproduce.
@XenoDelMoto https://github.com/XenoDelMoto it would be great if you
could paste your code here for us to see
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10025#issuecomment-249933779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGsWm5NanKU5mXQFuXg_ejpDTdqNv4Ouks5quVB7gaJpZM4JZMOy
.

@xenodelmoto there is an issue with your code, please reread the instructions. If you are still stuck, please click the 'Help' button and you will be taken to a dedicated chat room where people can advise you on you code

@Bouncey Yes, I was hitting the "Run tests" button like in every other assignment. However, the problem seems to have been corrected. I went again just now to the only assignment in the first four sections that was marked unfinished, didn't change a thing and tried again. This time, it worked.


Question: Implementing custom connection scheme

Hello,
I need to find a way to connect a redis-py client application to a Redis Server via a HTTP/2 connection. My initial idea was this:
App -> gRPC -> redis-py -> Redis Server
Where the gRPC service would terminate a custom protocol and convert that to Redis commands via redis-py.
However I wondered if implementing a custom connection might be an alternative. In that case the Socket based Connection classes would be augmented by a Class which can send socket data-streams via a gRPC service.
App -> redis-py -[http/2]-> gRPC -[socket]-> Redis Server
In that case life gets a bit easier for the application as I only need to change the URL to select a different connection scheme. And actually the whole thing is transparent to redis-py and the App.
Is that an approach which might be viable?

OK, it looks like I can implement a class based on Connection(object) and then pass that class as a parameter to ConnectionPool.init().
It might also be necessary to create a dervied class from ConectionPool to take care of the url scheme ... which would need to be different for reaching the HTTP/2 + gRPC based service.
In any case, seems possible.


Address sanitizer complains about "Heap buffer overflow" (MVSC)

Using Visual Studio community, raylib 4.2 and c++. Rres file was created using the web version of the app, no encryption or compression was used.
On my resource manager, on the first call to rresLoadCentralDirectory throws and exception if I use address sanitizer.

Without address sanitizer, the program runs ok, without any noticeable issues, resources are loaded correctly and displayed/played correctly.
The exception is thrown at line 1075 on rres.h (on the if block).

This is the code used on "ResourceManager.h" to call  rresLoadCentralDirectory:
template<>
inline void ResourceManager::Add<Texture2D>(const std::string& name, const char* path)
{
	if (textures.find(name) == textures.end())
	{
		std::string temp = "Could'nt add the texture ";
		temp += path;

		rresCentralDir texDir = rresLoadCentralDirectory(textureResource.c_str());
		int id = rresGetResourceId(texDir, path);
		rresResourceChunk chunk = rresLoadResourceChunk(textureResource.c_str(), id);

		int result = UnpackResourceChunk(&chunk);

		if (result == RRES_SUCCESS)
		{
			Image img = LoadImageFromResource(chunk);
			bool added = textures.emplace(name, LoadTextureFromImage(img)).second;

			if (!added)
				throw std::runtime_error(temp.c_str());

			UnloadImage(img);
		}

		rresUnloadResourceChunk(chunk);
		rresUnloadCentralDirectory(texDir);
	}
}

Code to load the resource:
data->res.Add<Texture2D>("animation", "newlogo.png");
Additional info can be found on the attached text file, this was the output generated by address sanitizer log
Thank you Ray!
rress_ex.txt

Hi, I have also encountered this issue on Linux using Clang address sanitizer.
The culprit should be here:
https://github.com/raysan5/rres/blob/536eb301e05e2fafd631e766baeb92d76e6c8f81/src/rres.h#L1073-L1074
From what I understand, info.baseSize is the size of both the properties count, properties array, and the raw data itself; while the input data variable contains all those above. The issue is in the memcpy, where it tries to copy a size more than just the raw data, using the info.baseSize, while the source pointer has been adjusted to the raw data location, causing it to copy beyond the raw data.
I managed to get rid of the issue by computing the raw size before memcpy, modifying the above code like so:
int rawSize = info.baseSize - sizeof(int) - (chunkData.propCount*sizeof(int));
chunkData.raw = RRES_MALLOC(rawSize);
memcpy(chunkData.raw, ((unsigned char *)data) + sizeof(int) + (chunkData.propCount*sizeof(int)), rawSize);

@BeardedBread Good catch! Your approach looks correct to me! Feel free to send a PR with the fix!

Perfect! Thank you!


Add SSR support to msal-browser

This PR allows for PublicClientApplication to be instantiated in an environment where window is not available (e.g. server-side rendering for React, Angular, etc.). Note, this will not allow for login or token operations to happen server-side, as those APIs will now throw specific errors if they are invoked in such an environment.
TODO:

[x] Test in real React SSR environment
[x] Test in real Angular SSR environment
[x] Ensure all code that could be run during module resolution is SSR-safe.

Coverage increased (+0.04%) to 81.765% when pulling 5646014f807520d23744e8db65bc683fffcede31 on msal-browser-ssr-support into cb65fd979779c7eaeb6153661a0140e33383db06 on dev.

One comment: How is SSR supported when we are blocking non-browser environments? The instantiation of PublicClientApplication is done with an empty object. So does this mean SSR has a PCA instance -> does not access window -> When client component is mounted, the window is initialized?
may be a  naive q from me, just curious how this PR helps SSR and for what operations!

seems so

I think the main issue here is you are declaring a configuration setting at global scope in your config: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/84a5f5aed0ad1626e6d918fb2ef0393837aac3c6/lib/msal-browser/src/config/Configuration.ts#L100-L108. This causes browser dependent code to fire. For my use case, it would be better to delay the evaluation of the window conditioned code until I actually invoke some msal code. I can conditionally stub a component during SSR rendering if this was the case.

I think the main issue here is you are declaring a configuration setting at global scope in your config:
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/84a5f5aed0ad1626e6d918fb2ef0393837aac3c6/lib/msal-browser/src/config/Configuration.ts#L100-L108
This causes browser dependent code to fire when the module is resolving. For my use case, it would be better to delay the evaluation of the window conditioned code until I actually invoke some msal code. I can conditionally stub a component during SSR rendering if this was the case.

Good callout, I'll make sure that gets addressed.

Was this merged?

Oh yeah, I need this one!

I also need this fixed. Is there any work the community can help with to move this along?

This was closed unintentionally by our bot. Apologies for the delay, we do intend to pick this back up soon.

@mlynam if it's any help, I mocked out msal during ssr pending this PR

@mbrevda thanks man I did too. Hopefully it lands soon!

@mbrevda @mlynam  did you mock out the types and using CDN to get it to work?

There is no need to have MSAL enabled during SSR. Here is how I stubed it:
// temporoarly override module laoding logic to mock @azure/msal-browser
// workaround for https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/2052
var Module = require('module')
var originalLoad = Module._load
Module._load = (request, parent, isMain) => {
  if (request === '@azure/msal-browser') {
    return {
      PublicClientApplication: class PublicClientApplication {
        getAllAccounts() {}
      }
    }
  }
  delete global.window
  return originalLoad(request, parent, isMain)
}

// SSR code...

Looks good, thanks for taking over @tnorling!

Will there be much change in the implementation on the consumers end?

@jrmcdona No, once the next release goes out, importing msal-browser and initializing PublicClientApplication should no longer throw an error in SSR environments, everything else should stay the same. You will just need to ensure that your app is not calling any of the token acquisition APIs (i.e. loginRedirect, loginPopup, etc.) in your server render path. If, for some reason, that's not possible, you can catch the new error that this PR introduces.

Will this be released into @azure/msal-browser soon? SSR is a deal breaker for our app and having to edit the code manually on every update has been tedious...

@kostenickj Yes, this will be included in version 2.2.1 which will be released very soon.

Thanks!

hello @tnorling, you indicate the 2.2.1 version of msal-browser for this fix, but it's already release one year ago, you talk to 2.21.0 version, when will it release ?


Azure AD B2C being part of Microsoft Entra External ID or not

This page, https://learn.microsoft.com/en-us/entra/external-id/external-identities-overview, shows and describes Azure AD B2C under "Microsoft Entra External Identities"
This FAQ, https://learn.microsoft.com/en-us/entra/external-id/customers/faq-customers, has a question and answer as follows:
Why is Azure AD B2C not part of Microsoft Entra External ID?
Microsoft Entra External ID and Azure AD B2C are two separate platforms powered by ESTS and IEF respectively. Microsoft Entra External ID is our new converged platform which is future proof and developer friendly to meet all your identity needs – B2E, B2B and B2C. At the same time, we will still continue to support Azure AD B2C as a separate product offering with no change in SLA, and we’ll continue investments in the product to ensure security, availability, and reliability.
Even though Microsoft Entra External ID (for customers) is in preview, the messaging is confusing.

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

ID: ed572d43-0c0e-4b91-7297-9d2158830575
Version Independent ID: 6920706a-8de6-af57-7457-451171d53522
Content: Microsoft Entra External ID overview - Microsoft Entra External ID
Content Source: docs/external-id/external-identities-overview.md
Service: entra-external-id
GitHub Login: @msmimart
Microsoft Alias: mimart

@smtl
Thanks for your feedback! We will investigate and update as appropriate.

Hi @smtl, you're right, this messaging is being clarified and the Microsoft Entra External ID overview is undergoing updates. Thanks so much for your feedback!

@smtl
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Documentation for using ChartJS plugins with different module loaders

I've been using ChartJS in my ES6 (Aurelia) project for a few days, and today I wanted to install my first plugin.
I npm installed chartjs-plugin-deferred and import it in my project but for some reason I get errors in the console (the plugin still works however, which is a little puzzling): Uncaught TypeError: Cannot read property 'helpers' of undefined
(The code which throws the error is just var helpers = Chart.helpers).
I figured the Deferred plugin wasn't built with module loaders in mind so I tried with another plugin (Annotation and Draggable).
With those added to my package.json and build file the entire build fails completely.
Is there any documentation on how to include ChartJS and plugins using different loaders? Importing ChartJS itself works just fine (with import Chart from 'chart.js') but nothing else can even be added to the build it seems...

I think the plugin still works because we always export the Chart global variable. Maybe some of the auto-run stuff doesn't work (not sure what uses Chart.helpers off the top of my head).
Both the package.json main files are the same for the plugins so I'm kinda surprised it doesn't work. Maybe there is some dependency that causes a problem. Are there any errors when importing the plugins?

Yes,
When trying with draggable, for some reason the build system looks for its dependencies in my app root instead of inside node_modules;
Error: ENOENT: no such file or directory, open '/Users/powerbuoy/project-folder/src/d3-dispatch.js'
This is using Aurelia CLI's build system so it may be because of that, but I haven't faced this issue before and I use plenty of node modules.

That's confusing. I'm kinda surprised that just changing the main file works. 😕 I can see how it would cause the console errors though since the built version of the plugin will have no require or import statements so it never gets the version that was built into your app.
Is it possible to share the code from your project (or a similar sample) so that I can have a look at how the build is working?

I think it would be easier to try with a completely new Aurelia CLI project;

npm install -g aurelia-cli

au new

Use the defaults

cd project-folder

Make sure the project works: au run --watch

Check localhost:9000

npm install -S chart.js

npm install -S chartjs-plugin-deferred

Now you need to add them to Aurelia's bundle as well;

Open aurelia_project/aurelia.json

Find the build.dependencies array

Add both ChartJS and the plugin:
// Note that because ChartJS contains a "." we need to rename it for the build to work (this is another Aurelia CLI thing)
{
"name": "chartjs",
"path": "../node_modules/chart.js/dist/Chart.min"
},
// The plugin does not contain a "."  I am currently using v3.0.0-beta.3 in a react app without trouble.


Fix errors when making the package

1.make: unit_test is created by default.
2.make package: unit_test is not included.
3.Set -DWITH_TESTS=OFF to compile without unit_test.

Fix #346


Not working + Error window on WIN11 (ReadProcessMemory failed with code: 299)

If you keep clicking OK, the window just shows up again, the unlocker is broken with the 4.7 update

https://github.com/34736384/genshin-fps-unlock/issues?q=is%3Aissue+is%3Aclosed

my bad, updated


Updating how to cite information

[Current coverage][cc-pull] is 90.68%

Merging [#376][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #376   diff @@
==========================================
  Files            29         29
  Lines          1976       1976
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1792       1792
  Misses          184        184
  Partials          0          0

Powered by Codecov. Last updated by [9917cb1...93dc7cb][cc-compare]
[cc-base-branch]: https://codecov.io/gh/D-PLACE/dplace/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/D-PLACE/dplace/compare/9917cb10f17e8a6ce1f4103b65814aa110733deb...93dc7cba68cfd5b52a3d49f8be827ffb26e36c5e
[cc-pull]: https://codecov.io/gh/D-PLACE/dplace/pull/376?src=pr


[AppService] az functionapp create: Support container app deployments

Related command
az functionapp create
Description
As part of the Centauri effort, we need to add the --environment parameter to the az functionapp create command so that customers can pass the managed AKS environment where the function app will be deployed.
We will not consume the --plan and --consumption-plan-location parameters since plans are not relevant for function apps deployed in managed environments. This is why server_farm_id is not set for Centauri.
We will also not consume the --os-type since it will need to be Linux.
The location is set to be the container app environment location.
Testing Guide

Create an App Service plan (az functionapp plan create -g  -n  --sku S1)

Create an managed AKS environment(az containerapp env create -n  -g )

Create a function app in the managed AKS environment (az functionapp create -g  -n  --plan  -s  --environment )

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

AppService


[WIP] Have descriptions when you open stream popover.

Fixes #6032 .

@rishig  I am still getting this as the output

Can you plzz help me to reproduce your output as mentioned by you


Blockchain Pending Status

After successful payment via Coinbase few days ago, my Woocommerce store is still showing "Blockchain Pending" status.
Does it get auto updated or do i need to do something and what is the turnaround time for status update?
Thanks.

It should get updated automatically once the payment confirms. Do you have the order code from Coinbase Commerce by chance? We can take a look on our end.

No it won't update automatically.
It only goes as far as noting payment completed.
In the file class-wc-gateway-coinbase.php, you can see this on line 365:
$order->payment_complete();
Instead, to update the status, it should go something like:
                $order->update_status( 'processing', __( 'Coinbase payment completed', 'coinbase' ) ); $order->payment_complete();
This will note that the payment was completed, and the status will change to Processing.

@abhiagrawal87, this is fixed in the latest code, please stay tuned for the newest release.

Hello...
i download the latest changes from github and installed the plugin with this changes and the "processing" status update works now BUT it will not trigger the mail for "new order" for the site admin or the "processing order" for the customer...
is there also a fix avilable somehwere?


Update for new getrusage

This fixes the crash, but I still have one test failing (on Linux). No idea if it was working in the past.
$ prove6l
./t/01-basic.t ..
1..10
ok 1 - is &times imported?
ok 2 - is &times externally NOT accessible?
ok 3 - did we get some user CPU
ok 4 - did we get some system CPU
ok 5 - did we get no child user CPU
ok 6 - did we get no child system CPU
ok 7 - did we get more user CPU
not ok 8 - did we get more system CPU
# Failed test 'did we get more system CPU'
# at ./t/01-basic.t line 18
ok 9 - did we get no child user CPU
ok 10 - did we get no child system CPU
# Looks like you failed 1 test of 10
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/10 subtests

Test Summary Report
-------------------
./t/01-basic.t (Wstat: 256 Tests: 10 Failed: 1)
  Failed test:  8
  Non-zero exit status: 1
Files=1, Tests=10,  1 wallclock secs ( 0.03 usr  0.00 sys +  0.96 cusr  0.14 csys =  1.13 CPU)
Result: FAIL

Thanks!


[PubSub] Dapr generates invalid CE envelope

⚠  This ticket is entirely predicated on the assumption that my analysis in https://github.com/dapr/docs/issues/3677 is correct - please do not make a decision on this issue until the linked docs issue has been confirmed to be correct.

Given the following request :
> POST /v1.0/publish/redis-pubsub/orders HTTP/1.1
> Host: localhost:3501
> User-Agent: insomnia/2023.4.0
> dapr-app-id: myapp3
> Content-Type: application/cloudevents+json
> Accept: */*
> Content-Length: 34

{
	"data": {
		"some": "body"
	}
}

Produces the following CE envelope :
{
  "data": {
    "some": "body"
  },
  "pubsubname": "redis-pubsub",
  "source": "Dapr",
  "specversion": "1.0",
  "time": "2023-08-13T19:40:24Z",
  "topic": "orders",
  "traceid": "00-00000000000000000000000000000000-0000000000000000-00",
  "traceparent": "00-00000000000000000000000000000000-0000000000000000-00",
  "tracestate": "",
  "type": "com.dapr.event.sent"
}

Actual behaviour
The resulting CE envelope (despite Daprs augmentation) is technically invalid according to the CE spec, as the Id property is not present (It wasn't provided by me, and it wasn't provided by Dapr)
Expected behaviour
The resulting CE envelope should also include a Dapr provided Id field, to make the CE envelope valid.

Dapr does not provide CE envelope validation in case of application/cloudevents+json payload. Hence cannot reject an event.
Generating an ID could be risky because there is a tiny chance it could clash with IDs provided by the user - Dapr has no knowledge of the user provided ID space.
I suggest cloud event validation is performed prior to sending the request to the Dapr sidecar. Preferably in the SDKs - and ideally by using the Cloud Event SDKs not the Dapr SDKs.

Generating an ID could be risky because there is a tiny chance it could clash with IDs provided by the user - Dapr has no knowledge of the user provided ID space.

A simple GUID/UUID should be sufficient to mitigate this (same as what Dapr does with the auto-generated CE envelop)

Pls reopen

Pls reopen

/keep-alive

/keep-alive


Index should check that there are no duplicate read names

My feeling is that if we see two sequences with the same name we should raise an error. A lot of other bioinf tools also exhibit this behaviour. I think it also saves bigger unexplained issues (like this one) arising further down the analysis road.
Originally posted by @mbhall88 in https://github.com/rmcolq/pandora/issues/126#issuecomment-492161063
From the discussion in #126 it would save users a lot of heartache further down the analysis pipeline if we catch any entries in the PRG which have the same name and exit with an error saying all names should be unique. This is effectively a proxy for ensuring that the same PRG is not in a PRG file twice.

@leoisl given you are working on index at the moment maybe it makes sense for you to add this in?

Yeah, I will take care of it after #23

Update: I am not taking care of this right after #23
This has low priority, moving towards finishing learning pandora index, map and compare to try to make the most of the fact that Rachel is still here (without her, I am sure it would be way slower for me to understand index).


out of memory error trying to install devbio-napari on a machine with 8GB ram

conda install -c conda-forge devbio-napari fails with an out of memory error on a ubuntu 20.04 machine (a VM) with 8GB RAM.
maybe, the memory requirement should be documented somewhere?
NB: the large memory requirement happens in the "solving environment" step of the installation. Would it help to install the dependencies in smaller batches?

update: installing napari seems to require ~10-12GB RAM. Should we update the requirements in the documentation accordingly?

switching to mamba solved the issue for me


Use it with real data

Hi, I would like to test this with a 3D lidar installed on a mobile robot.
How can I do so? Could you please provide the "entry point" for the real time data?
and the output as well?
Thank you

hi, same question. How can I do so?


docs: displaying function sizes

After discussing https://github.com/formkit/tempo/pull/28 I prepared mr showing the function sizes in documentation. @justin-schroeder talked about table with sizes at the end of docs, but I think it would be helpful for users to see the sizes right next to the description. Also I think a github link for each function would be useful.
I made a few variants with different sizes:
Icon size - 24px, font-size - 14px (sm), padding - 8px

Icon size - 24px, font-size - 14px (sm), padding - 4px

Icon size - 16px, font-size - 12px (xs), padding - 4px

Usage of sizes from the file is not ready yet, I'm still waiting for comments on the design
What do you think? Should I choose one of the variants or should I still make a table?

Awesome! I like the smaller ones @aleksandrjet — and if we need to tweak them we can. I way prefer the small ones over the table. When you get it to a good place where you are wanting me to merge I can do that. If we need to do any further polishing we can. Great work!

I added the npm run post:build script to create a docs/assets/func-sizes.json file with function sizes. This file is then used in each top-level component (Parse, Helpers, etc.). I did not find info about deployment, but I think that this script should be added to CI.
Also you need to pay attention to displaying sizes of format and parse functions, because their style is different from the other functions. Perhaps these styles should be changed

p.s. The yearEnd and yearStart functions were not exported and I added re-export to index.ts
p.p.s. I see that there is no link to Helpers in the table of contents, although this is a large section, perhaps it should be added

Awesome! great work @aleksandrjet. Ive merged this and iterated a bit more on the styles.
One thought I had on this all — there is an initial size in brotli and gzip which is included in each of those. if you sum the total of all the functions its way more than the total size of the package because each contains quite a bit of compression information. addDays for example, is 524Bytes on the page, but its incremental size is tiny. I dont know exactly how we could make that more clear on the site though


Specify thread group dimensions for compute shaders

This modifies the DirectX compute shaders (accumulation and post-processing) to specify thread group dimensions. This is done to improve GPU occupancy, and thus performance. Specifically, this reduces the time required to run those compute shaders, which improves overall rendering performance. For simple scenarios, this can actually double performance as measured in Plasma. In more typical scenarios, this will increase performance by around 10%.
This change was suggested by Neil Bickford, a developer technology engineer from NVIDIA.

Please use "squash and merge" (this should be the default/only option, if I've setup the settings correctly)


Add a Disk - button says "Saving" when modal appears

This is the display when clicking  "Add a Disk," before doing anything else

Thanks! Should be dealt with in https://github.com/linode/manager/pull/1698

Closed in #1698


ports documentation is incomplete

File: compose/compose-file/index.md
The short syntax includes examples like the below, where an ip address is specified, however the documentation does not indicate what the ip address is for, nor does it indicate in the long format how one would accomplish the same thing, IE would it be added to published?
ports:
 - "3000"
 - "3000-3005"
 - "8000:8000"
 - "9090-9091:8080-8081"
 - "49100:22"
 **- "<IP_ADDRESS>:8001:8001"
 - "<IP_ADDRESS>:5000-5010:5000-5010"**
 - "6060:6060/udp"

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!

I just wanted to open the same issue. The documentation is really missing this information and therefore, this issue should be re-opened.


[Candidate 3v3 Garden of Terror ZH v1] Rotations of mirrored Oil Derricks are not properly setup

Observed behavior
Rotations of mirrored Oil Derricks are not properly setup.
Expected behavior
Mirrored Tech Buildings have mirrored rotations or somewhat consistent rotations for a consistent look.
Screenshots
Sample. I suggest to double check all.

fixed


Detailed instructions for PCB ordering from jlcpcb

Where can I find the detailed instructions to order the v4.1 Choc hotswap PCB from JLCPCB? What additional services should I add to the order?

There is no document for ordering now. There is just data for the keyboard here. Please check the data and learn how to order at jlcpcb.com etc.

@virajkanwade  hey did you manage to build this keyboard? If so please share what worked


switchAll Operator is not documented

The SwitchAll operator is visible in the new Docs, but it is more or less empty. A detailed explanation how this operator is supposed to work and what it is useful for, would be really nice.
see #4004

@JWO719 I'll take care of it.


Environment.yml fails to build development environment

Bug:

When I try conda env create --file environment.yml I get several installations failing:
Solving environment: failed

ResolvePackageNotFound:
  - openssl==1.1.1h=haf1e3a3_0
  - certifi==2020.6.20=py37h2987424_2
  - libcxx==11.0.0=h439d374_0
  - setuptools==49.6.0=py37h2987424_2
  - xz==5.2.5=haf1e3a3_1
  - tk==8.6.10=hb0a8c7a_1
  - prodigal==2.6.3=h01d97ff_2
  - python==3.7.8=hc9dea61_1_cpython
  - zlib==1.2.11=h7795811_1010
  - sqlite==3.33.0=h960bd1c_1
  - libffi==3.2.1=hb1e8313_1007
  - readline==8.0=h0678c8f_2
  - ncurses==6.2=hb1e8313_2

Specs:
OS: Linux Debian 10 (Buster)
conda version: 4.8.5

Apparently this is a common issue with conda (Unfortunately), here's an example.
I've found that removing the version hashes in the environment file (i.e. prodigal=2.6.3=h516909a_2 -> prodigal=2.6.3) fixes it and gets me an environment with the same software and versions. Though this is a very weird and not very practical solution, I can submit a pull request for it.

Found out the problem: The build hashes are platform-specific, something I didn't know.
Generating the environment via conda env export -f environment.yml --no-builds should make it so that any platform can download it.


Force get_thread_count() to 1 if single threaded

This PR makes sure that WorkerThreadPool::get_singleton()->get_thread_count() returns 1 when threads=no. Otherwise, a lot of WorkerThreadPool calls will not run as threads.size() would return 0 in single-thread builds, as jobs are set as done if no elements/threads are available.
Alternative to #98121
Fixes #88192

Thanks!


Add a cli option to choose the bind ip for outgoing connections

Provide us a use case of the feature
I would like to use instaloader to be able to grab posts without an account. This should be doable on a box with multiple ips.
Describe the solution you'd like
I would like some parameters for:
debug - to see what http/https connection the application tries to do... maybe even some levels to see the parameters/headers, or even a stacktrace
ipv6 - to enable/disable the use of ipv6 ( I don't know if ipv6 is used right now anyway )
bindip - ability to bind to a specified address
Describe alternatives you've considered
I was thinking about writing my own code but I'm not that good.
If the feature request is accepted, would you be willing to submit a pull request?
No... because I have no idea what implications there are with the various modules used.

I would like something like this but perhaps for different accounts. So one account downloads via one ip, another via a different one. In order to circumvent Instagram's rate limits. I'm getting a lot of bad requests now, even when rotating between 10 different accounts.


tested with 192 cores, performance is not very good, suggestion on how to tweak for 192 or 448 cores server

wanted to go extreme, how to make 192 cores faster? same pastebin info on server, doesnt really scale that well. can suggest how to make it multicore shard better?

@phuslu cant believe u found that out and yes i just read it. it's very interesting.
i tried thinking in numa actually but found extremely little support. i thought maybe your lru can implement the numa instead

i thought maybe your lru can implement the numa instead

I doubt it because it subject to go scheduler


Fix remnants from SelinuxProcessLabel to SelinuxLabel rename

These were missed when we moved the field to Process struct.
Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

@crosbymichael @vbatts PTAL

LGTM

On Fri, Apr 22, 2016 at 01:52:56PM -0700, Mrunal Patel:
“M specs-go/config.go (2)”
Hmm.  Not sure how that got around #339…

LGTM


Fix incorrectly reported deprecated setting

The javax.persistence.sharedCache.mode setting reported here is not set by the user but by Hibernate itself . User settings would already be reported here.

This was fixed via https://github.com/hibernate/hibernate-orm/pull/5752


rancher reset-password instructions are wrong for HA

In page https://rancher.com/docs/rancher/v2.x/en/faq/technical/
reset-password command is not working because jq command is not getting anything but empty string.
fix the instructions.
kubectl exec -n cattle-system $(kubectl --kubeconfig $KUBECONFIG get pods -n cattle-system -o json | jq -r '.items[] | select(.spec.containers[].name=="cattle-server") | .metadata.name') -- reset-password

error: expected 'exec POD_NAME COMMAND [ARG1] [ARG2] ... [ARGN]'.
POD_NAME and COMMAND are required arguments for the exec command
See 'kubectl exec -h' for help and examples.

Can you tell me what instructions you used to set up the cluster?


Breaking API change without deprecation warning

Problem
The name_scope API has changed between last week and last night's latest changes in the 0.10 branch. The previous API was:
name_scope(*args, **kwds)

The current API is:
name_scope(name)

The API was changed in a breaking way, rather than the new API being added and the use of the old API triggering a warning.
I was wondering what the TensorFlow gatekeepers' policy is on breaking API changes. Is there an official policy? Is the breaking change without a deprecation warning and phase-out intentional?

This is a deliberate change. Before 1.0, we have very few guarantees for the API itself. In fact, we're trying to make breaking changes before 1.0, so their frequency may increase in the coming months.
Please take a look at our policy regarding code.
For stored graphs, the guarantees are stronger.
After 1.0, there should not be breaking changes in minor releases, only for major ones (i.e., the next ones at 2.0).
Note that regardless of version, development at head (or branch head) is inherently dangerous when it comes to stability due to bugs.

So TF shouldn't be used in production until 1.0?  Is there documented rough schedule/timeline for 1.0?

There's no problem with using TF in production (we use it plenty). But I would advise against using TF at head using continuous updates in a production environment.
To use TF in production, I would use a release (or, if you must, a specific hash), and update TF only in a controlled manner. In other words, treat it the same as other libraries you use in production: when you update the library to a new version, verify that everything works afterwards.
Even updating more mature libraries like numpy to a new version is not something you can safely do in production.
TF is no different in principle, although because of various factors (cuda, rapid development) people are much more likely to use TF at head, and so the pressure up update is much higher.

update TF only in a controlled manner.
Seems like the problem here is that there's no warnings or release notes when breaking changes occur. How should someone like me (external to Google, using TF) know which versions are safe and which ones are unsafe to depend upon? What strategy do you recommend, @martinwicke ?

There are release notes when breaking changes occur: https://github.com/tensorflow/tensorflow/blob/master/RELEASE.md, though it's clear it isn't always filled in with all changes :(.


Add ability to delete datasets via CLI, API

Actually, perhaps ability to have status for datasets, for example retired, so that they don't show up unless you specifically ask that they be listed in queries.  I think files have something like this.  Would be nice to retire  all my old test datasets.  I think the mechanisms are already in place for files.

What is the use case for this functionality ?

The 70,000 mostly obsolete definitions in sam is an example.
In my case, as I have been testing creation of tagged datasets, I have produced multiple copies.  So the gui is clogged with multiple useless old versions.  in the end we want to present users with clarity, so the ability to remove obsolete datasets from view would be very useful.
Files already have this functionality, having it for datasets would be really useful.

70000 definitions in SAM do not belong in MetaCat. Those are project definitions and they are part of the workflow management and out of scope of MetaCat. Workflow management is moved to Data Dispatcher and (I assume) to justIN. MetaCat datasets are not the same as SAM project definitions and are not supposed to be temporary objects.
As for testing/development, please use the demo instance.
You can delete unneeded test datasets yourself using GUI. Or if you give me the list of datasets you want to delete, I can do it for you going directly to the database.

Those are not projects, those are the equivalent of  dataset definitions!  And thus fully within metacat.  The existence/location of files is not part of them.   And I agree  that 70,000 is too many - but my suggestion is that we retire bad ones.  As otherwise finding the good ones becomes hard.
I am trying to make metacat user-friendly - the use of namespaces makes it much nicer to isolate user activity but users/groups still need this kind of tool so that they can retire mistakes.  Which happen a lot.
I will try using the gui, but having a CLI would be extremely useful. Anything the gui can do should have an API in the end.
On Aug 28, 2023, at 6:23 AM, Igor Mandrichenko<EMAIL_ADDRESS>wrote:
[This email originated from outside of OSU. Use caution with links and attachments.]
70000 definitions in SAM do not belong in MetaCat. Those are project definitions and they are part of the workflow management and out of scope of MetaCat. Workflow management is moved to Data Dispatcher and (I assume) to justIN. MetaCat datasets are not the same as SAM project definitions and are not supposed to be temporary objects.
As for testing/development, please use the demo instance.
You can delete unneeded test datasets yourself using GUI. Or if you give me the list of datasets you want to delete, I can do it for you going directly to the database.
—
Reply to this email directly, view it on GitHubhttps://github.com/ivmfnal/metacat/issues/29#issuecomment-1695694990, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIA37DJDAPFKSFLJFKMGLALXXSLWLANCNFSM6AAAAAAZR4H22Q.
You are receiving this because you commented.Message ID: @.***>

Done. Please upgrade your client to 3.38.0


Sorting: Implement Quicksort in Python

https://en.wikipedia.org/wiki/Quicksort

How would you like this issue to be fixed

How would you like this issue to be fixed

An implementation of Timsort would do just fine with good comments

How would you like this issue to be fixed

Assigned it to you

Is this issue open to contribute to?


Adds command spo listitem batch remove

Closes #5688

Thank you @MathijsVerbeeck . We'll try to review it ASAP!

Thanks for the great review. It had some great suggestions and I think that I've resolved all your feedback.

Merged manually, thank you! 🎉


+n: New Window, focus location bar

Tracking for MVP milestone.
I believe these are mostly working, but should not be considered complete without tests. I do not think the location bar is focusing at the moment.

Tests added in #403 -- still need focus location portion

Location is now focusing in the new frontend.


bind_sequence requires two presses to trigger

editor.bind_sequence(KeyPress::Esc, Cmd::Interrupt);
I expect it to stop the input (Currently that works with ctrl+c). But it requires Esc to be pressed twice in order to work. Seems it's only an issue with Esc. Changing it to Home for instance works on a single press. This is on Terminator -> Linux
I don't know if this is intended.

It's because the Esc key is used to trigger Meta- sequence.
For example, Esc + b gives Meta-b (move cursor to previous word).
But you can try to use keyseq_timeout.
https://github.com/kkawakam/rustyline/blob/master/examples/example.rs#L73
    let config = Config::builder()
        .history_ignore_space(true)
        .completion_type(CompletionType::List)
        .edit_mode(EditMode::Emacs)
        .keyseq_timeout(250) // wait 250ms to make the distinction between a single escape and a sequence
        .output_stream(OutputStreamType::Stdout)
        .build();


bug fix # 312 SofaTracerFeignClient is unsupported when accessing directly using feign url and using link tracking times ...

…OperationException
Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
遇到UnsupportedOperationException重新构造一个header可编辑的request
Result:
Fixes #312

@99246255 please fix ci


GRID-455-459-475-470: Resize last col & Drag Cell Selectionm & Dragging Last Column

@joneit I intend to push an alpha tonight that won't be a 1.3.0 master target

@joneit  the only issue now is that the cursor state is wrong but the events work immediately
thats a separate ticket I saw u just assign to yourself

@joneit


FR: Improve tests by swtiching from travis -> Jenkins

Is your feature request related to a problem? Please describe.
Currently only the DAG is tested, whilst ideally we would run each rule whether it works as expected.
Describe the solution you'd like
Travis has incredibly slow computers. We can move to for instance Amazon cloud, or host Jenkins ourselves.

https://www.ensembl.org/Takifugu_rubripes/Info/Index
Fugu has very small genome, so should be good for auto tests


Hiding an element using jQuery does not respect :visible selector

I am trying to hide an element using jQuery, and then want to be able to check it is not visible using the ':visible' selector, but it does not see to work. Is this a bug?
Here is my test:
it('should hide elements', function() {
    document.body.innerHTML = '

hello

' +
                '

hello

';

    $('.one').hide();
    expect($('div.one').css('display')).to.equal('none');
    expect($('div.two').css('display')).to.equal('');
    expect($('div:visible').length).to.equal(2); // fails, reports 4
});

jquery uses offsetWidth and offsetHeight for that. If both of those are 0, an element is :hidden and not :visible.
So #135 would have to be resolved

Rolling into #1048.


Required options do not throw error

I'm using Vorpal with the reduced test case below:
'use strict';

let cli = require('vorpal')();

cli
  .command('test')
  .description('reduced test case')
  .option('--required <val>', 'Should be a required value')
  .action(function(c) {
    console.log(c.options.required);
  });

cli.parse(process.argv);

node index test

This command's action should not execute using the given command, as no --required value was specified. According to the current source in lib/util.js:199, missing required options shows a message in the console along with help. This does not happen.

This is broken again on 1.12.0

Up for this issue. I have the same problem.


spew for NOTEs regarding Swift and .xcworkspace

Now that j2objcXcode runs much more frequently, these notes appear more often.

What I'll do is move the notice to the "pod init" message, where people will see it during initial setup rather than continuously.


Don't log old addon deprecations by default

Example: https://github.com/PolyhedralDev/Terra/blob/b3f80dcb648ecf7367a4120472aaf6b0a1b921a1/common/addons/biome-provider-image/src/main/java/com/dfsek/terra/addons/biome/image/ImageBiomeProviderAddon.java#L53
The debug.log plugin/mod config option would probably be the option that enables these messages

Correct me if I'm wrong, but this seems to be completed already with commit e80e998.

That is correct.

Correct me if I'm wrong, but this seems to be completed already with commit e80e998.

Are you on our discord or matrix?

Discord; username pumk3n.


Switch icon hover circle to border from box-shadow and make service-i…

…tem display:block to enable word-break
Also

img max height and width needed to be set
 .services-item-icon flex removed and height set. flex was preventing word-break from functioning

Fixes https://github.com/openshift/origin-web-catalog/issues/201

@rhamilto PTAL
cc @spadgett

It would be great to see a screenshot :-)

@spadgett @rhamilto Updated pr for review
Added gradient fade and overview hidden to handle long names.

pr rebased

@sg00dwin @rhamilto I think we should use an ellipses when text gets cut off. In this case we would not use the fade, we would just show the ... after three lines of text and then show the full name in a tooltip on hover. This would be in line with PF and we should also be consistent with what we decide here: https://github.com/openshift/origin-web-catalog/pull/214 @serenamarie125

@beanh66 We can only do this if we decide to truncate to a certain number of characters instead of cleanly at the end of the 3rd line. There's no good way that I know in CSS to use an ellipsis to truncate a multiline string. @rhamilto keep me honest here
This will be a bigger problem for #214 where we can't truncate by characters since it will break any links that get truncated the middle. We'd have to remove the link highlighting, and it also means we might truncate in the middle of a line.

@beanh66 We can only do this if we decide to truncate to a certain number of characters instead of cleanly at the end of the 3rd line. There's no good way that I know in CSS to use an ellipsis to truncate a multiline string. @rhamilto keep me honest here

To my knowledge, there is no cross-browser compatible solution as described, which is why we went with the fade out effect here and in in #214.  :(

@spadgett @rhamilto Can we cut these titles off at 40 characters then show the ellipses? We would still allow users to see the full title on hover. Thoughts @ncameronbritt @serenamarie125?

@spadgett updated with overflow gradient removed.

@serenamarie125 @beanh66 @rhamilto I asked @sg00dwin to remove the gradient for now and add the ellipsis in a follow-on. We need to find a good way to do that both here and the overlay panel, but unfortunately it's a little tricky.

@rhamilto Can you take another look?

[merge]

agreed @spadgett I think it's also worth us discussing best practices, and making sure our recommendations are documented appropriately.  I know that we will still need to address, but if we document best practices, hopefully these edge cases will be minimized.


in create record, how to save struct likely gorm?

in gorm have this create operate:
user := User{Name: "Jinzhu", Age: 18, Birthday: time.Now()}

result := db.Create(&user)

have the  same operate in ent ? current i only see the following operate.
client.User.Create().SetName().SetAge().Save(ctx)

Yes, by design on mutations fields needs to be explicitly set. You can write a template transforming one to the other though dealing with zero values might be problematic (i.e. does zero values means update or don't touch).

These links will be useful for you, if you want to implement this:
https://entgo.io/docs/faq#how-to-create-an-entity-from-a-struct-t
https://entgo.io/docs/tutorial-todo-gql-mutation-input

@alexsn @a8m thanks a lot.


Add an option to Flip the Board for diagrams created with Insert Diagram function

Main idea is that I have an Exercise where Black is on the move and I set the board with Black at the bottom. Solution is long and I want to insert diagram within solution text but it can only be done with White at the bottom.

So I can see that now the diagram in exercise solution is inserted depending who is on the move. I cannot see an option to specify it.
Just wondering if we should worry about the situation when exercise is for Black on the move, solution is long and at some stage user wants to insert a diagram on the White Move.
Also I took one exercise for Black with the solution and within a solution inserted a diagram with Black at the bottom. Then I modified an exercise and decided that White is on the move. System left part of the solution that was still valid and got rid a few moves at the end. The part of the solution that was left had a diagram inside. So we have a diagram within a solution with Black at the bottom even though the exercise is now for White to Move. Just wandering if we should leave it there or auto flip the board if the side on the move is flipped. You decide, either way is fine with
me


Patcher diagnostic mapper

This change is

crates/defs/src/db.rs line 203 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
Add comment.
None means that the file wasn't generated.

or no remapping?


Visually distinguish alertname and summary

Put the alert title in bold to make it visually more pleasing

:+1: Thanks!


HttpClientModule is not supressing basic authentication popup dialog

I'm using HttpClientModule to invoke my rest service.  The Interceptors are not suppress the basic authentication pop-up dialog.  we need authentication interceptors for basic and digest auth like okHttpClient or  we need a callback for "WWW-Authenticate challenge".
export class AuthInterceptor implements HttpInterceptor {
    constructor(private router: Router) { }

    intercept(req: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {

        return next.handle(req).do((event: HttpEvent<any>) => {
            console.log("AuthInterceptor error"+ JSON.stringify(event))
            if (event instanceof HttpResponse) {
                let response = event;
                console.log("AuthInterceptor Authenticate challenge"+ event.headers.has("WWW-Authenticate"));

            }else{
                console.log("AuthInterceptor No Authenticate challenge");
            }
        }, (err: any) => {
            console.log("AuthInterceptor error"+ JSON.stringify(err))
            if (err instanceof HttpErrorResponse) {
                if (err.status === 401) {
                    // redirect to the login route
                    // or show a modal
                    this.router.navigate(['login']);
                    //return Observable.throw(err.message);
                }
            }
        });
    }

Refer : https://stackoverflow.com/questions/24130308/preventing-http-basic-auth-dialog-using-angularjs-interceptors

On the one hand, questions are better for SO as you did, we ask not to open issues for them, sorry.
On the other hand, this repo is deprecated, no one pays attention anymore.


Train without eos token?

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Add feature to train without eos tokens
✔ Solution
I try to train raw corpus but i can't find a way to disable eos token.
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

So, when i set the type 'completion' it works as intended. (no eos)
But when with custom prompt template there's no option to disable eos.
The reason why i use custom template is to mask some tokens.

@StableFluffy , I believe there is eos for completion?
You may need to tokenize it yourself otherwise like said in #990 ?

Hi, I tried to tokenize on my own.
It works well. I'll close this issue.


ResourceWarning: unclosed file

Instrument
NIRSpec (Stages 1-3)
What happened?
I got this error in stage 2:
ResourceWarning: unclosed file <_io.TextIOWrapper name='/home/asemon/src/eureka/Trappist1-NIRspec/Stage2/S2_2023-01-09_trappist1_run1/S2_trappist1.log' mode='w' encoding='UTF-8'>
how can I solve it?
Thanks
Error traceback output
No response
What operating system are you using?
Ubuntu (Linux-based)
What version of Python are you running?
Python 3.9.7
What Python packages do you have installed?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi, sorry for the very slow response - can you confirm that you are still having this issue? If so, can you copy-paste the whole traceback of the error message (and not just the last line of the error message) so I can understand where in the code this happened and why? My best guess at the moment is that the code was killed because the code tried to exceed your RAM size (jwst's Stage 1 and 2 pipelines are very RAM intensive unfortunately). Without more contextual information though I can't confirm that hypothesis or help you resolve the issue

Thanks for your message and for your time. Now, I don't have this problem. I close that.


Refactoring: use namedJdbcTemplate bean instead of internal new object

Allow to mock the objects in future
e.g. if jdbcTemplate is a mock then sometimes such pattern new NamedParameterJdbcTemplate(jdbcTemplate) causes a not testable code block like in JdbcScimUserProvisioning
For jdbcTemplate we create one instance and re-use it, therefore if NamedParameterJdbcTemplate is used in several beans, then we should do the same

Allow to mock the objects in future
e.g. if jdbcTemplate is a mock then sometimes such pattern new NamedParameterJdbcTemplate(jdbcTemplate) causes a not testable code block like in JdbcScimUserProvisioning

Hi! I don't think I understand this point. Could you provide elaborate? What code block is not testable under JdbcTemplate and how does NamedParameterJdbcTemplate solve that problem? It looks like the main advantage of NamedParameterJdbcTemplate is that you could perform param substitution on the SQL string in a stricter manner (but I don't see this change be leveraged in this PR?).

Allow to mock the objects in future
e.g. if jdbcTemplate is a mock then sometimes such pattern new NamedParameterJdbcTemplate(jdbcTemplate) causes a not testable code block like in JdbcScimUserProvisioning

Hi! I don't think I understand this point. Could you elaborate? What code block is not testable under JdbcTemplate and how does NamedParameterJdbcTemplate solve that problem?
It looks like the main advantage of NamedParameterJdbcTemplate is that you could perform param substitution on the SQL string in a stricter manner (but I don't see this feature being used in this PR?).

The feature it, that classes which use this can now be mocked, as done with https://github.com/cloudfoundry/uaa/pull/2863
Another reason why I wanted this refactoring is that I was wondering for a while, why we create a datasource and jdbcTemplate in a single place -> data-source.xml and re-use this reference in all needed places but the enhance jdbc class NamedParameterJdbcTemplate is always newly generated with each bean creating, see
https://github.com/search?q=repo%3Acloudfoundry%2Fuaa+"new+NamedParameterJdbcTemplate"&type=code
I think we will re-use NamedParameterJdbcTemplate more often therefore good to use it in same way we do with underlying jdbcTemplate/datasource


Disambiguate uc/lc filenames in conformance tests.

PR #6720 added a couple of new tests for the concrete syntax of bool, including a check for true and false where True or False is expected.  This led to the tests containing directories called True, true, False, and false`, but unfortunately it turns out that some filesystems are still case-insensitive, meaning that the new tests lead to apparently duplicated paths in the filesystem.  This PR prevents that by renaming the new files.

Seems to work for me. On the one hand, the fact that new Macs are case insensitive by default is silly, on the other hand we probably shouldn't break on them?

This ought to fix the immediate problem, although some of the CI with "darwin" in the name is failing?...

I think that's because of the issue that #6729's supposed to fix.

Yeah, those CI errors went away after rebasing on #6729.


[Bug] FileNotFoundError: [WinError 3]

Description
FileNotFoundError: [WinError 3] The system cannot find the path specified: '' when trying to train on custom dataset
Step to reproduce

Installed using pip
Run spleeter train -p configs\hi_config.json -d D:/hi
Got FileNotFoundError: [WinError 3]
or

Output
INFO:spleeter:Start model training
Traceback (most recent call last):
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\wesle\AppData\Local\Programs\Python\Python36\Scripts\spleeter.exe\__main__.py", line 9, in <module>
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\__main__.py", line 54, in entrypoint
    main(sys.argv)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\__main__.py", line 46, in main
    entrypoint(arguments, params)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\commands\train.py", line 98, in entrypoint
    evaluation_spec)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\training.py", line 473, in train_and_evaluate
    return executor.run()
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\training.py", line 613, in run
    return self.run_local()
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\training.py", line 714, in run_local
    saving_listeners=saving_listeners)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 370, in train
    loss = self._train_model(input_fn, hooks, saving_listeners)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 1161, in _train_model
    return self._train_model_default(input_fn, hooks, saving_listeners)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 1188, in _train_model_default
    input_fn, ModeKeys.TRAIN))
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 1025, in _get_features_and_labels_from_input_fn
    self._call_input_fn(input_fn, mode))
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 1116, in _call_input_fn
    return input_fn(**kwargs)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\dataset.py", line 78, in get_training_dataset
    wait_for_cache=False)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\dataset.py", line 406, in build
    dataset = self.cache(dataset, cache_directory, wait_for_cache)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\site-packages\spleeter\dataset.py", line 367, in cache
    os.makedirs(cache_path, exist_ok=True)
  File "c:\users\wesle\appdata\local\programs\python\python36\lib\os.py", line 220, in makedirs
    mkdir(name, mode)
FileNotFoundError: [WinError 3] The system cannot find the path specified: ''

Environment

OS
Windows 10 Pro

Installation type
pip

RAM available
32GB

Hardware spec
AMD Ryzen 5 3600X

hi_config
{
"train_csv": "configs/hi_train.csv",
"validation_csv": "configs/hi_validation.csv",
"model_dir": "hi",
"mix_name": "mix",
"instrument_list": ["vocals", "piano", "drums", "bass", "other", "guitar"],
"sample_rate":44100,
"frame_length":4096,
"frame_step":1024,
"T":512,
"F":1024,
"n_channels":2,
"separation_exponent":2,
"mask_extension":"zeros",
"learning_rate": 1e-4,
"batch_size":4,
"training_cache":"training_cache",
"validation_cache":"validation_cache",
"train_max_steps": 2500000,
"throttle_secs":600,
"random_seed":8,
"save_checkpoints_steps":300,
"save_summary_steps":5,
"model":{
"type":"unet.softmax_unet",
"params":{
"conv_activation":"ELU",
"deconv_activation":"ELU"
}
}
}
train and validation .cvs

mix_path
vocals_path
drums_path
bass_path
piano_path
guitar_path
other_path
duration

train/Queen - Bicycle Race/mixture.wav
train/Queen - Bicycle   Race/vocals.wav
train/Queen - Bicycle   Race/drums.wav
train/Queen - Bicycle   Race/bass.wav
train/Queen - Bicycle   Race/piano.wav
train/Queen - Bicycle   Race/guitar.wav
train/Queen - Bicycle   Race/guitar.wav
180.697

train/Queen - Death On Two Legs/mixture.wav
train/Queen - Death On Two   Legs/vocals.wav
train/Queen - Death On Two   Legs/drums.wav
train/Queen - Death On Two   Legs/bass.wav
train/Queen - Death On Two   Legs/piano.wav
train/Queen - Death On Two   Legs/guitar.wav
train/Queen - Death On Two   Legs/guitar.wav
223.033

train/Queen - Dont Stop Me Now/mixture.wav
train/Queen - Dont Stop Me   Now/vocals.wav
train/Queen - Dont Stop Me   Now/drums.wav
train/Queen - Dont Stop Me   Now/bass.wav
train/Queen - Dont Stop Me   Now/piano.wav
train/Queen - Dont Stop Me   Now/guitar.wav
train/Queen - Dont Stop Me   Now/other.wav
215.347

train/Queen - Get Down Make Love/mixture.wav
train/Queen - Get Down Make   Love/vocals.wav
train/Queen - Get Down Make   Love/drums.wav
train/Queen - Get Down Make   Love/bass.wav
train/Queen - Get Down Make   Love/piano.wav
train/Queen - Get Down Make   Love/guitar.wav
train/Queen - Get Down Make   Love/other.wav
259.382

train/Queen - Hammer To Fall/mixture.wav
train/Queen - Hammer To   Fall/vocals.wav
train/Queen - Hammer To   Fall/drums.wav
train/Queen - Hammer To   Fall/bass.wav
train/Queen - Hammer To   Fall/piano.wav
train/Queen - Hammer To   Fall/guitar.wav
train/Queen - Hammer To   Fall/other.wav
221.024

train/Queen - I Want To Break   Free/mixture.wav
train/Queen - I Want To Break   Free/vocals.wav
train/Queen - I Want To Break   Free/drums.wav
train/Queen - I Want To Break   Free/bass.wav
train/Queen - I Want To Break   Free/piano.wav
train/Queen - I Want To Break   Free/guitar.wav
train/Queen - I Want To Break   Free/other.wav
256.171

train/Queen - Killer Queen/mixture.wav
train/Queen - Killer   Queen/vocals.wav
train/Queen - Killer   Queen/drums.wav
train/Queen - Killer   Queen/bass.wav
train/Queen - Killer   Queen/piano.wav
train/Queen - Killer   Queen/guitar.wav
train/Queen - Killer   Queen/other.wav
180.582

train/Queen - Keep Yourself   Alive/mixture.wav
train/Queen - Keep Yourself   Alive/vocals.wav
train/Queen - Keep Yourself   Alive/drums.wav
train/Queen - Keep Yourself   Alive/bass.wav
train/Queen - Keep Yourself   Alive/piano.wav
train/Queen - Keep Yourself   Alive/guitar.wav
train/Queen - Keep Yourself   Alive/other.wav
229.102

train/Queen - Stone Cold   Crazy/mixture.wav
train/Queen - Stone Cold   Crazy/vocals.wav
train/Queen - Stone Cold   Crazy/drums.wav
train/Queen - Stone Cold   Crazy/bass.wav
train/Queen - Stone Cold   Crazy/piano.wav
train/Queen - Stone Cold   Crazy/guitar.wav
train/Queen - Stone Cold   Crazy/other.wav
146.607

Hopefully i gave all that was needed

This is happening to me as well. I installed the spleeter-gpu through conda and I have set the model_dir to point to the 2stems-finetune model.

Hi @wesleyr36, I was able to fix the problem by editing the config file.
Instead of having:
"training_cache":"training_cache",
"validation_cache":"validation_cache",
Try this:
"training_cache":"cache/training",
"validation_cache":"cache/validation",
If that doesn't work, then you can try to make sure your dependencies are the right version. I noticed that even though I installed the latest spleeter through Conda, it installed tensorflow 1.14.0.
If your tensorflow or any other dependency doesn't match what is in the requirements.txt file, then install it by running a command like this:
pip install tensorflow==1.5.2
more generally:
pip install ==<version#>

Thank you for the comment I will try it when I can

I have tried your solution and it seems to to be working Thank you for your help


Tests skipped with optional dependencies installed

There are a few tests being skipped with pytest.importorskip in the CI even though it appears the corresponding dependency is installed in the CI environment. See https://travis-ci.org/dask/dask/jobs/520461060#L2631 for a snappy example and https://ci.appveyor.com/project/daskdev/dask/builds/23870810#L1112 for a moto example. It's not immediately clear to me why this is happening, so I'm opening up this issue to document and track the problem.

Perhaps imports are failing due to incompatible/missing deps of those packages?

Looks like unpinning the conda version in #4720 fixed this issue for zarr-related tests on Appveyor.
I plan to look into this more sometime next week

@jrbourbeau does the issue persist?

This appears to be stale.  Closing.  (Though please reopen if the problem persists)


Valais/Wallis Bezirke data wrong

I just noted, Valais data is wrong since several weeks.
I checked with data for Brig, Week 46/21 is correct, Week 1/22 is a well, but the others after 46/21 are wrong (didn't check all of the though).

Hey @balzamas
We are considering, archiving this repo, because there is no activity anymore and it seems, there will be no activity in the future 🤞. Otherwise it can be unarchived if necessary.
Therefore we're checking all open issues. What about this one? Is it still important or can you either close this one as completed or close as not planned? Thank you!


Image display: not sure if this is a a bug?

I did my first page with an image; no problem uploading image and configuring alt-text but the image is not displaying as expected. I want the image to display at the full size (no resizing needed), and it looks good in the editing window, but then in the browser it is distorted. Not sure if if this is a bug, or if there is something I need to do for an image to display just as-is without any resizing?
Page:
https://hax.lauragibbs.net/sites/story-folder/s000014
Image URL:
https://hax.lauragibbs.net/sites/story-folder/files/scale-50/Screen Shot 2019-06-18 at 3.30.55 PM.png
Screenshots: editing window; looks good!

Display; image is being resized...?

I tried playing with the image and it still wasn't working; I deleted the image and uploaded a new version, 600 pixels wide. I think the problem might have something to do with this "scale-50" that is showing up in the URL. That scale-50 thing shows up automatically when I upload the image as the URL, and it's also in the HTML view of the page:

image file that I uploaded, 600 pixels wide (352 pixels height):

image as shown in the editor does look like it might be 50% version of my original?

here's how published page looks: I'm really not sure at all what's going on here...?

For a default, it seems like it would make sense to default to displaying 100% my original image, and then I would adjust that using the slider and/or by entering a fixed number for width or height...?

ahhh I see. If you take the scale-50 out of that URL you'll have your original image. We've got it in there currently as a way of hoping people don't drag and drop upload full resolution images and tick off their users.

oh yay, that's great: I'll go see if that works. I prefer to resize myself rather than resizing in the browser.
it's still kind of buggy in the sense that it was going to that scale-50 by default. I'll manually take it out and see if that does it. thank you!!! :-)

PERFECT!!! I just removed that scale-50/

Yeah, @nikkimk might have an idea for how we want to handle this. I know it came up in discussion w/ @heyMP the other day (notion of scaling / having something process media).
@nikkimk for context here, when user's upload images we default compress a thumbnail as well as do a scale-50 transformation to them. All 3 (original, and these two derivatives) are stored when you upload something but we default to the scale-50. How do you think we should handle this? Not do any of them and then create a way on the front-end to like do a drop down and pick defaults like this or something else where we can process things in bulk? Not sure the UX pattern for this as @lauragibbs is certainly not our typical IDs who will upload a 5 meg image uncompressed at times.

@nikkimk do you think some kind of file variants array could be used in what your currently doing w/ the HAX schema + eco json forms UX improvements?
This seems like it's almost a flavor of an upload... not sure. It's like we want the user to upload something and then make a decision / let them easily toggle how they handle the variants of that file. Another possible related example in the future is @heyMP has talked about upload and pandoc microservice saying "TXT file... cool, how do you want to use this though?" and allow click and convert it on the fly from whatever to whatever. Almost like "I gave you a word doc but I want to present it as a PDF" and having that be a toggle.
Does this seem like a UX convention possibly required of the uploader element or something else?

fixed. now we just default to the thing that's expected


Constant discord crash on some servers

Appears when you link something to a server you are no longer in or never was in.

We handle request errors, not sure why this is happening to you, I cant reproduce


Do not merge: Plain text description of the API

The intent of this is not to be merged. The intent was to do two things:

Pull in some of the very recent API changes or proposals (#145 and #140) into the rest of the API description.
Provide a (perhaps) easier format for discussing the API that is completely separate from implementation.

But... but... our code is self-documenting...

Agreed -- that is why this isn't intended to land in the repository ever.  I thought it would be helpful to talk about any remaining API issues/questions before deciding how we move forward with that API (ABC or protocol or...).
Put differently, I wasn't sure whether to update #126 or #142 so I copy-pasted into a separate doc 😬

Both @gpdf and @astrofrog did not like my approach with #126 , so maybe #142 is the better way to go. 🤷


throwable light source

some sort of throwable weapon akin to flares, could piggy back off of VE tribal, will investigate

Resolved with #32 and added bioferrite toolbelt mod to pack


Navigation Titile on Android is in additional View instead of just being Text

https://github.com/aksonov/react-native-router-flux/issues/98
https://github.com/exponentjs/ex-navigator/issues/24

I have the same problem with the link.
The solution is https://github.com/delta1/react-native-router-flux/commit/fd9812c06b6339e0844f5f9795276ebff83d85dd (ExRouter.js)
Please add features. ^^

thank you.

Check 3.0 release, now the component uses new React Native Navigation API. However it has a little bit different API comparing with 2.x release. Anyway I'm closing this ticket because 2.x release will not be supported except critical bugs.


Multi item lib is not complete

I'm trying to use this lib to make the bow like the vanilla one (that makes the zoom effect).
Repo (Fabric 1.18.2, using multi item lib 1.5.2)
I've added

maven {
name = "OroArmor"
url = "https://maven.oroarmor.com"
}

and

include(modImplementation("com.oroarmor:multi-item-lib:1.5.2"))

to build.gradle and it builds fine, then I added

UniqueItemRegistry.BOW.addItemToRegistry(ModItems.POWER_BOW);
UniqueItemRegistry.BOW.addItemToRegistry(ModItems.ELITE_POWER_BOW);

to both the client and server initializers (though I think only the client is needed, and I've tried also that) but It doesn't work, am I missing something?

Maybe this lib is only for proper rendering of the item, but doesn't add the zoom?

Well, I tried it with the crossbow and the change is significant, it works, so I guess there is no way to make them zoom?

For bows and crossbows I wrote https://github.com/DaFuqs/Arrowhead/blob/master/src/main/java/de/dafuqs/arrowhead/api/ArrowheadBow.java once. Just include the lib & implement that interface on your bows. There also is a crossbow one.

The zoom is not part of this mod. As I slowly transition features from this to QuiltMC/quilt-standard-libraries, such as the bow stuff, i wont be going back to fix it


log level as (number)enum

for filtering with logLevel >= LogLevel.DEBUG

:tada: This issue has been resolved in version 4.1.0-beta.31 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 5.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[azopenai] A rather large, but needed, cleanup of the tests.

It was getting difficult to tell what was and wasn't covered and also correlate tests to models, which can sometimes come with differing behavior.
So in this PR I:

Moved things that don't need to be env vars (ie: non-secrets) to just be constants in the code - this includes model names and generic server identifiers for regions that we use to coordinate new feature development.
Consolidated all of the setting of endpoint and models into one spot to make it simpler to double-check.
Consolidated tests that tested the same thing into sub-tests with OpenAI or AzureOpenAI names.

If a function was only called by one test moved it into the test as an anonymous func

Also, I added in a test for logit_probs and logprobs/toplogprobs.

/azp run go - azopenai

(There's an issue with the live pipelines right now that's unrelated to my change. Here's the manually queued pipeline, which should get around it: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3703444&view=results)


cmd/operator-sdk/internal/genutil: relax version dir check in parseGroupVersions()

Description of the change: remove requirement that the API dir structure pkg/apis/<group>/<version> have <version> match something like v1alpha1, and instead check if <version> is a dir (or dir hierarchy) that contains files.
Motivation for the change: having a API dir shared by multiple versions, ex pkg/apis/operators/shared, is valid and should be considered when running a code generator.

Before merging, we should double check with @awgreene that this solves his problem.

Before merging, we should double check with @awgreene that this solves his problem.

Thanks for working on this @joelanford and @estroz!
I pulled the relax-fqapis-check branch and ran the sdk after updating the phase types to include the deepcopy gen code:
// Phase describes the phase the object is in
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
type Phase struct {
	// Name of the phase
	Name string `json:"name,omitempty"`

	// A human readable message indicating why the object is in this phase
	Message string `json:"message,omitempty"`
}

// ObjectPhase describes the phase of a Marketplace object is in along with the
// last time a phase transition occurred and when the object was last updated
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
type ObjectPhase struct {
	// Current phase of the object
	Phase `json:"phase,omitempty"`

	// Last time the object has transitioned from one phase to another
	LastTransitionTime metav1.Time `json:"lastTransitionTime,omitempty"`

	// Last time the status of the object was updated
	LastUpdateTime metav1.Time `json:"lastUpdateTime,omitempty"`
}

I then ran the following command:
$ operator-sdk generate k8s
INFO[0000] Running deepcopy code-generation for Custom Resource group versions: [operators:[shared v1 v2], ]
INFO[0001] Code-generation complete.

It looks like the shared dir was inspected, but there was no code generated. Any thoughts?

@awgreene do you have doc.go and register.go in pkg/apis/operator/shared?
Also you might have to have Phase look like the following:
type Phase struct {
	metav1.TypeMeta   `json:",inline"`
	metav1.ObjectMeta `json:"metadata,omitempty"`

	...
}

While testing with your code I got compilation errors because Phase didn't implement the k8s.io/apimachinery/pkg/runtime.Object method, which requires a GetObjectKind() method.

type Phase struct {
	metav1.TypeMeta   `json:",inline"`
	metav1.ObjectMeta `json:"metadata,omitempty"`

Thanks for looking into this @estroz, is there a reason why this needs to be added if it's moved into it's own package? When phase was in the v1 package it did not need the field.

@awgreene can you link the rest of your code?

@awgreene can you link the rest of your code?

https://github.com/awgreene/operator-marketplace/commit/7d6b64f9dc094661e54dcca3f6b5bb68a183410f
I copied the old deepcopy code into this commit earlier but the only change other change involved moving the phase_types.go file.

Ah I see the issue. My test code had // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object above ObjectPhase and Phase which forces DeepCopyObject() to be generated, but that is not necessary. Disregard my field addition comment.

Ah I see the issue. My test code had // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object above ObjectPhase and Phase which forces DeepCopyObject() to be generated, but that is not necessary. Disregard my field addition comment.

Great - adding the doc file seems to have fixed everything! Thanks @estroz


Fix #1386: Add metadata version

Describe your changes :
#1386

Type of change :

[x] Improvement

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers


Issue with Colab Pytnotebook and Tensorflow hub

I was trying the Colab example with Tensorflow hub here
# Install the latest Tensorflow version.
!pip install --quiet --upgrade --pre tensorflow
# Install TF-Hub.
!pip install tensorflow-hub
!pip install tf-nightly

to have the nightly build, but when I try to install
import tensorflow as tf
import tensorflow_hub as hub
import matplotlib.pyplot as plt
import numpy as np
import os
import pandas as pd
import re
import seaborn as sns

I get this version error
RuntimeErrorTraceback (most recent call last)
<ipython-input-6-48e1bdaa8642> in <module>()
      1 import tensorflow as tf
----> 2 import tensorflow_hub as hub
      3 import matplotlib.pyplot as plt
      4 import numpy as np
      5 import os

/usr/local/lib/python2.7/dist-packages/tensorflow_hub/__init__.py in <module>()
     63
     64 # Comment/uncomment to skip checking the TensorFlow version.
---> 65 _check_tensorflow_version(tf.VERSION)
     66
     67 # Used by doc generation script.

/usr/local/lib/python2.7/dist-packages/tensorflow_hub/__init__.py in _check_tensorflow_version(version)
     60       "TensorFlow Hub depends on 'tf-nightly' build after %s or "
     61       "'tensorflow~=%s'. Found tf.VERSION = %s" % (
---> 62           _NIGHTLY_VERSION, _MAIN_VERSION, version))
     63
     64 # Comment/uncomment to skip checking the TensorFlow version.

RuntimeError: TensorFlow Hub depends on 'tf-nightly' build after 20180308 or 'tensorflow~=1.7'. Found tf.VERSION = 1.6.0

But I have already installed
Collecting tf-nightly
  Downloading tf_nightly-1.8.0.dev20180331-cp36-cp36m-manylinux1_x86_64.whl (48.6MB)
    100% |████████████████████████████████| 48.6MB 28kB/s
Requirement already satisfied: astor>=0.6.0 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: absl-py>=0.1.6 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: wheel>=0.26 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: grpcio>=1.8.6 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: gast>=0.2.0 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: termcolor>=1.1.0 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: protobuf>=3.4.0 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: numpy>=1.13.3 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Collecting tb-nightly<1.9.0a0,>=1.8.0a0 (from tf-nightly)
  Downloading tb_nightly-1.8.0a20180409-py3-none-any.whl (3.1MB)
    100% |████████████████████████████████| 3.1MB 382kB/s
Requirement already satisfied: six>=1.10.0 in /usr/local/lib/python3.6/dist-packages (from tf-nightly)
Requirement already satisfied: setuptools in /usr/local/lib/python3.6/dist-packages (from protobuf>=3.4.0->tf-nightly)
Requirement already satisfied: html5lib==0.9999999 in /usr/local/lib/python3.6/dist-packages (from tb-nightly<1.9.0a0,>=1.8.0a0->tf-nightly)
Requirement already satisfied: bleach==1.5.0 in /usr/local/lib/python3.6/dist-packages (from tb-nightly<1.9.0a0,>=1.8.0a0->tf-nightly)
Requirement already satisfied: markdown>=2.6.8 in /usr/local/lib/python3.6/dist-packages (from tb-nightly<1.9.0a0,>=1.8.0a0->tf-nightly)
Requirement already satisfied: werkzeug>=0.11.10 in /usr/local/lib/python3.6/dist-packages (from tb-nightly<1.9.0a0,>=1.8.0a0->tf-nightly)
Installing collected packages: tb-nightly, tf-nightly
Successfully installed tb-nightly-1.8.0a20180409 tf-nightly-1.8.0.dev20180331

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

@tensorflowbutler updated.

I solved the same issue by restarting the environment and then using:
!pip install tensorflow==1.7
!pip install tensorflow_hub

Hope that helps!

@jure it works!

Nagging Assignee @rohan100jain: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.