Incorrect terminal on open

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.60.0 (user setup)
OS Version: Windows 10 Pro + 21H1 + Feature Experience Pack 120.2212.3530.0

Steps to Reproduce:

Open vscode

Details:
When you open VScode the default terminal is opening as powershell and not the selected default (in my case cmd)
if  you select Terminal -> New terminal  cmd opens.
if you close the power shell terminal leaving only cmd and restart VScode powershell returns.
View -> Command palette -> Terminal: Select default profile does not correct this.
setting: "terminal.external.windowsExec" is set to "C:\Windows\System32\cmd.exe"

/duplicate of #132150
It's a bug which is already fixed in Insiders build and supposed to be in upcoming Recovery August release.


sqls formating removes all whitespaces

Expected Behavior
CREATE TABLE accounts (
  user_id SERIAL PRIMARY KEY,
  username VARCHAR (50) UNIQUE NOT NULL,
  password VARCHAR (50) NOT NULL,
  email VARCHAR (255) UNIQUE NOT NULL,
  created_at TIMESTAMP NOT NULL,
  last_login TIMESTAMP
);

Actual Behavior
CREATETABLEaccounts(
    user_id SERIALPRIMARYKEY,
    usernameVARCHAR(
        50
    )UNIQUENOTNULL,
    passwordVARCHAR(
        50
    )NOTNULL,
    emailVARCHAR(
        255
    )UNIQUENOTNULL,
    created_atTIMESTAMPNOTNULL,
    last_loginTIMESTAMP
);

I'm not sure how it started, I was using nvim and setting up  for postgres with lsqconfig,
because I have same issue,for work around I added to ~/.config/sqls/config.yml following:
# Set to true to use lowercase keywords instead of uppercase.
lowercaseKeywords: false
connections:
  - alias: psql
    driver: postgresql
    dataSourceName: 'host=<IP_ADDRESS> port=5432 user=postgres password=secret dbname=testdb sslmode=disable'

it's clearly not related to error, but after that in some point, formatter started going crazy. Now I removed all configs from every corner but this bug persists

It looks like this "only" seems to happen on non-insert and non-selects, which includes a lot of queries :thinking:

It looks like in https://github.com/sqls-server/sqls/blob/c95deb1c28b221b1c0ff8cd1fc37051b7794106a/internal/formatter/formatter.go#L135 we're not matching i.e. CREATE or VARCHAR.
I can see a diff like so:
diff --git internal/formatter/formatter.go internal/formatter/formatter.go
index 15cbf96..e17d485 100644
--- internal/formatter/formatter.go
+++ internal/formatter/formatter.go
@@ -144,6 +144,8 @@ func formatItem(node ast.Node, env *formatEnvironment) ast.Node {
                        "LIMIT",
                        "WHEN",
                        "ELSE",
+                       "CREATE",
+                       "TABLE",
                },
        }
        if whitespaceAfterMatcher.IsMatch(node) {

Will start to do this correctly, but not sure if we want to allowlist each keyword + combination? (I realise that's how it's currently already done)

Is the problem solved?

As per the unclosed issue / unmerged PRs, no, it's still present

It's also removing white spaces for insert statements for me, not sure if this is expected
Unformatted
INSERT INTO users (
  username,
  password
) VALUES ($1, $2);

Formatted
INSERT INTO users(
  username,
  password
)VALUES($1,$2);

If anybody wants to stop sqls from formatting the buffer, you can add this in the on_attach function when setting up the LSP (using lspconfig in my case).
on_attach = function(client, _)
    client.server_capabilities.documentFormattingProvider = false
    client.server_capabilities.documentRangeFormattingProvider = false
end

As a bonus, here's my solution for formatting my .sql files (usually Postgres) using sleek.

-- Pick what you need :)
if filetype == filetypes.sql then
    if vim.fn.executable("sleek") ~= 1 then
        vim.notify("sleek is not installed, skipping formatting", vim.log.levels.ERROR)
        return
    end

    local bufnr = vim.api.nvim_get_current_buf()
    local temp_file_name = vim.fn.tempname()

    vim.api.nvim_buf_call(bufnr, function()
        vim.cmd("silent noa write " .. vim.fn.fnameescape(temp_file_name))
    end)

    vim.cmd("silent !sleek " .. vim.fn.fnameescape(temp_file_name))

    local temp_file = io.open(temp_file_name, "r")

    if temp_file == nil then
        vim.notify_once("Failed to read formatted file", vim.log.levels.ERROR)

        return
    end

    local temp_content = temp_file:read("*all")
    local formatted = vim.split(temp_content, " ")
    formatted = vim.split(temp_content, "\n")

    temp_file:close()

    os.remove(temp_file_name)

    vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, formatted)

    return
end

Basically it's copying the current buffer's content to a temporary buffer, formats it, reads it, replaces the current buffer's content with the formatted output.

still exist


socketIO exception

using Tom's logistic regression script (ask him for it if you want to try), and other pyfora scripts, we've been hitting errors of the following form
2016-02-10 20:52:05,714 computing in pyfora ...
Exception in thread Thread-2:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/__init__.py", line 232, in wait
    self._process_packets()
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/__init__.py", line 254, in _process_packets
    for engineIO_packet in self._transport.recv_packet():
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/transports.py", line 155, in recv_packet
    six.b(packet_text))
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/parsers.py", line 93, in parse_packet_text
    packet_type = int(get_character(packet_text, 0))
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/symmetries.py", line 17, in get_character
    return chr(get_byte(x, index))
  File "/usr/local/lib/python2.7/dist-packages/socketIO_client/symmetries.py", line 21, in get_byte
    return six.indexbytes(x, index)
  File "/usr/local/lib/python2.7/dist-packages/six.py", line 655, in indexbytes
    return ord(buf[i])
IndexError: string index out of range

I think the problem is in transports.py, line 145, the WebsocketTransport class is getting an empty string from _connection.recv(). I think a nasty but acceptable workaround would be to patch the function recv_packet to look like
    def recv_packet(self):
        try:
            packet_text = None
            while packet_text is None:
                packet_text = self._connection.recv()
        except websocket.WebSocketTimeoutException as e:
            raise TimeoutError('recv timed out (%s)' % e)
        except websocket.SSLError as e:
            raise ConnectionError('recv disconnected by SSL (%s)' % e)
        except websocket.WebSocketConnectionClosedException as e:
            raise ConnectionError('recv disconnected (%s)' % e)
        except socket.error as e:
            raise ConnectionError('recv disconnected (%s)' % e)
        engineIO_packet_type, engineIO_packet_data = parse_packet_text(
            six.b(packet_text))
        yield engineIO_packet_type, engineIO_packet_data

with the difference being that we're looping around the _connection.recv(), filtering out the empty strings.


"externalize" laxar/laxar-widget-service-mocks

With the new browser entry in laxar's package.json (LaxarJS/laxar#425) we can externalize the dependency and have webpack resolve it automatically.

Implemented on master (v2.0.0)

After it turned out the change in laxar had to be reverted (LaxarJS/laxar#427) a small adjustment had to be made here (14a57c8), but that should work now … I guess …


chore: update pub publish scripts

Status
READY
Description
Fixes #70
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[x] ✅ Build configuration change
[ ] 📝 Documentation
[x] 🗑 Chore

Confirmed that Pub.dev settings for this is correct as well.


ScheduledTask: Add 'StopExisting' to valid values for MultipleInstances parameter

Pull Request (PR) description
Add 'StopExisting' to valid values for ScheduledTask MultipleInstances parameter - fixes Issue #333
This Pull Request (PR) fixes the following issues
Fixes Issue #333
Task list

[√] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[√] Resource documentation added/updated in README.md in resource folder.
[√] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[N/A] Comment-based help added/updated.
[N/A] Localization strings added/updated in all localization files as appropriate.
[N/A] Examples appropriately added/updated.
[√] Unit tests added/updated. See DSC Resource Testing Guidelines.
[N/A] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[√] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

The HQRM tests are failing with the error below for all the examples for every resource, saying Use Import-DSCResource to import the resource.  However, the examples do import the resources using the module name: Import-DscResource -ModuleName ComputerManagementDsc. Has something gone awry with the tests?
    Context ScheduledTask\1-ScheduledTask_CreateScheduledTaskOnce_Config.ps1
      [-] Should compile MOFs for example correctly 86ms
        Expected no exception to be thrown, but an exception "At D:\a\1\s\source\Examples\Resources\ScheduledTask\1-ScheduledTask_CreateScheduledTaskOnce_Config.ps1:35 char:9
        +         ScheduledTask ScheduledTaskOnceAdd
        +         ~~~~~~~~~~~~~
        Undefined DSC resource 'ScheduledTask'. Use Import-DSCResource to import the resource." was thrown from D:\a\1\s\source\Examples\Resources\ScheduledTask\1-ScheduledTask_CreateScheduledTaskOnce_Config.ps1:35 char:9
            +         ScheduledTask ScheduledTaskOnceAdd
            +         ~~~~~~~~~~~~~.
        141:                     } | Should -Not -Throw
        at <ScriptBlock>, D:\a\1\s\output\RequiredModules\DscResource.Test\0.13.0\Tests\QA\ExampleFiles.common.Tests.ps1: line 27

Hi @Rob-S - this is due to a change to recent ModuleBuilder change. I've got a PR #334 open that will resolve it. Once that is trough the build should work properly again.

You should be able to rebase and resolve conflicts and tests should pass.

Does anything else need done on this?

Hi @Rob-S - sorry, this is on me. I'll review this weekend (this week is a bit crazy with MS Build and so none of my evenings are free).

Hi @Rob-S - the build failure is caused by a recent release of Module Builder. I've fixed the issue in this PR https://github.com/dsccommunity/ComputerManagementDsc/pull/334 - if you rebase onto master again the build should pass - then we can take a look at the integration tests.

Hi @Rob-S - Sorry, correction, the issue you're seeing is because of the Pester v5.x release. I'm going to pin this module to remain on Pester 4.10.1.

Once this PR https://github.com/dsccommunity/ComputerManagementDsc/pull/337 is through you can rebase and the tests will hopefully pass.

@Rob-S - can you try rebasing your branch onto master again to see if that fixes the build?

Hi @Rob-S - looks like the tests are failing. I think this may be because:https://dev.azure.com/dsccommunity/ComputerManagementDsc/_build/results?buildId=2305&view=logs&j=158455ef-2db5-56fd-6022-83ded799e050&t=b737ddad-5217-50db-260a-b6e0dd548ea1&l=933

@PlagueHO Yes, I had fixed the tests around the same time you entered that comment. But the actual change I made to fix Get-CurrentResource is the one thing that is still failing the tests. When I run it myself locally, it works fine; it's just failing in the tests.
On line 157 of DSC_ScheduledTask.Tests.ps1 where it has:
$result = Get-TargetResource @getTargetResourceParameters
what we really need to do is get the parameters from the actual target resource that was affected by the previously executed Set-TargetResource on line 153 (rather than the parameters variable). I'm not familiar with the test syntax, particularly the "@" variables and don't see anywhere else in that test script where Get-TargetResource follows the Set-TargetResource -- Set-TargetResource is always after the Get-TargetResource. For the test that is failing, the Set-TargetResource sets the value in the CimInstanceProperties, and that's the value that needs to be retrieved by Get-TargetResource.
Do you know how, in the test script, to reference the resource affected by Set-TargetResource?

@PlagueHO Moving the Get-TargetResource into the "It" block didn't help. Just to prove to myself that it wasn't just the test for the change that I had made... I had noticed that the test starting on line 126 under "No scheduled task exists, but it should" did not actually test in "Should create the scheduled task in the set method" to ensure that the resource was actually created by Set-TargetResource. When I added the $result.Ensure | Should -Be 'Present' on line 129 it failed as well, indicating that the resource was still Absent after Set-TargetResource.
Do you see anything wrong with the way that test, the one starting on line 126, is coded, where the resource is still 'Absent' after it was supposedly created by Set-TargetResource?

I think what we need is a unit test that tests when MultipleInstances is set to StopExisting. I could be wrong, but I didn't see one. Is it easy to add that?

No. It's easy to add it for the other values, but not for StopExisting, since the Enum does not support that. See the comments for my commits when I attempted to do that.


FR: Sync with Google Contacts

You can sync them by adding your Google account to MacOS's system settings.

You can sync them by adding your Google account to MacOS's system settings.

Good point, but some users have reported that avatars don't sync properly. And since we have the login with Google thing, it makes it kinda easy to also grab contacts or something

Yes I would agree the pictures don't sync over from google also it would be cool If the profile photo sync across all devices and maybe grab the main Google photo profile over ?

Yes I would agree the pictures don't sync over from google also it would be cool If the profile photo sync across all devices and maybe grab the main Google photo profile over ?

yes, we have thought about pulling that info since BlueBubbles already asks for access to your google account for setting up firebase. I just need to implement it. That would likely be the best way to go, rather than using the Google sync via the Contacts app

Hi. I stumbled on this problem today and I have found that I had some pictures in the windows client if the google contact hadn't any space in the cell number. For the contacts registered with spaces, no pictures in the client. Weirdly, even if I change it and reset settings in the server and clients apps, I can't get the picture. I even thrwon the server out of applications into the bin, but when I installed it again, the settings were still there. I am a noob on macos, so I don't know how to properly ininstall, I have check in Apllication support and did a search but nothing.

I Would love to see direct google contacts integration. This would definitely end a lot of the contact issues with it syncing from the mac contacts app.

Would it be possible to set an interval when it pulls. Such as daily, weekly, etc instead of just when a user manually hits the button?

Would it be possible to set an interval when it pulls. Such as daily, weekly, etc instead of just when a user manually hits the button?

Unfortunately, no, not without a server to handle the persistent authentication, or not without including the client secret in the source code (which we don't want to do). This is because when you authenticate, the credentials are only valid for 30 minutes or so. Whatever the default time is. If I try to use the credentials 24 hrs later, it won't work until you consent again


[FEATURE REQUEST] - Support for Efficiency Loader

Summary
It would be great to see the Efficiency Loader node, part of the Efficiency Nodes suite, supported in the next version of the app.
Basic Example
Hi. You can see my scenario below:

If it helps, the full workflow is downloadable here: https://perilli.com/ai/comfyui
I understand that the whole workflow might be too complex to support but, at least, having the Efficiency Loader and Efficiency KSampler nodes supported would help.
Thank you
Reference Issues.
No response

Can you post an image generated using this workflow?

Sure, @receyuki:

I finally had some time to take a look at this workflow, but the bad news is that it's quite complex. Adding support for the Efficient loader is not difficult, but this workflow involves a large number of other custom nodes, and it's challenging for me to support such a large number of custom nodes.
I have just completed the development of the SD Prompt Reader ComfyUI node version, which includes a node similar to "Save Image w/Metadata", but fixes existing issues. I will be writing instructions within the next few days.
https://github.com/receyuki/comfyui-prompt-reader-node

@receyuki I just published version 4.0 of my workflow above, with a series of Save Image w/Metadata nodes, as you recommended.
I'll keep an eye to see when your new node pops up in ComfyUI Manager and test if I can use that one, instead.

My pull request was just merged into the ComfyUI Manager, so from now on, the ComfyUI Manager will handle the installation and update process.
If you encounter any issues or have any suggestions, please let me know.

Yes, I saw the new suite and installed it, thank you @receyuki.
My problem, which is really quite different from what I wrote in the title of this feature request, is that I cannot decouple the checkpoint name from everything else and carry it to your SD Prompt Saver node or to the Save Image w/Metadata node (which you previously suggested) via a series of reroute nodes.
This has nothing to do with your nodes, so I am sharing it just for the sake of documentation:
The only node I ever found that isolates the name of the checkpoint/modelname is the CheckPoint Selector node, in the Image Saver Tools suite.
This node connects to both your SD Prompt Saver node or to the Save Image w/Metadata node without issues.
But if I try to reach long distances with the help of reroute nodes (I mainly use rgthree Reroute nodes), the connection disconnects. Either immediately, or upon refresh, or randomly, as if there's a race condition that is not addressed.

And this thing prevents me from keeping the workflow tidy.
Again, nothing to do with your nodes. I thank you for sharing them with the community.
I think this feature request can be closed.

Firstly, thank you for your feedback.
In my nodes, there is a Parameter Generator node that can achieve the same thing as the CheckPoint Selector. The only difference is that I integrated all the selectors, generators, and literals from that suite into a single node.
Based on my guess, the reason for the disconnection might be due to different formats. My Prompt Saver node, like the official one, can only accept 'folder_paths.get_filename_list("checkpoints"),' which is a list containing all the checkpoint names. If you attempt to connect a STRING to 'model_name,' it will disconnect. If you don't mind, you can upload your workflow here, and I'll try to see if I can resolve the issue.

Thanks! The workflow is here. I left a red note for you.

Damn it, everything is working fine on my end, and I can't reproduce this issue. Another possibility might be that a custom node's JS that you've installed is causing this bug.

This thing is driving me crazy. Since you can't reproduce, I recreated all reroute nodes (just in case) and started the process of migrating from the Save Image w/ Metadata node to your SD Prompt Saver node, as you can see below:

However, as I queue a generation, an error informs me that the Efficient Loader is now missing the checkpoint name.
This makes me discover that while the connections from the Checkpoint Loader to the SD Prompt Saver node now work, the wire connecting the Checkpoint Loader to the Efficient Loader is now severed:

This is what must be happening: either I can go towards the Efficient Loader or towards the Save Image w/ Metadata // SD Prompt Saver node.
To test my theory, I try to reconnect the Checkpoint Loader to the Efficient Loader. It refuses to. The link appears possible but it refuses to maintain the connection. Which is exactly what was happening before towards the Save Image nodes.

Oh my goodness! Maybe you need to find an alternative for 'reroute.' I just took a look at the source code for 'reroute', and it seems like the entire node is written in TypeScript, so I might not be able to help you.
But wait, are all the issues occurring with things like 'model_name'? In other words, STRING won't have similar problems? If that's the case, maybe I can try creating a node that converts this type into STRING and allows Saver to input STRING.

Yes, all the issues are because of this model_name output.
Perhaps it would be easier to create a replacement node that exclusively outputs the model/checkpoint name. Like your Parameter Generator, but just for the model name / checkpoint variable.
However, it would have to be

compatible with the Efficient Loader
compatible with your SD Prompt Saver
compatible with rgthree Route

I'm not sure about 2 and 3, but it seems that the Efficient Loader doesn't accept a string input as ckpt_name.
But perhaps it's not worth your time.
Regardless, thank you for all the time you have invested in this so far.

BTW, it's not just rgthree reroute node. If I try to connect the model_name output in your SD Parameters Generator node to the ckpt_name input in the Efficiency Loader node via a default ComfyUI Reroute node, I get the same rejection.

I found a hack:
If I put a rgthree Node Collector node just before the Efficient Loader node and the SD Parameters Generator / Save Image w/ Metadata nodes, nothing gets disconnected.

If I put the Node Collector at the top of the chain, before all reroute nodes, instead, the links continue to break.
It's a terrible hack, but it seems to be working.

That would help a lot!

I've just added alternative inputs to Saver, please update and reload the nodes. Note that inputs of type STR have higher priority, so don't connect STR inputs unless necessary, as Saver will prioritize reading STR inputs over regular ones

This is great, thank you. I'll test it over the weekend and let you know.

This new SD Type Converter node is SO useful. I can now print checkpoint, sampler, and scheduler names for debug purposes. Thank you!

That's great! After a few days of use, have you encountered any other issues or do you have any other suggestions? If everything is fine for now, I plan to release the stable version 1.0.0.
BTW, are "Unstable-Reality" and "alessandroperilli" both your accounts?

I encountered no issues with your nodes or your implementation. My new AP Workflow 4.0.2 implements both your SD Parameter Generator and your SD Type Converter node.
The issue I still have is due to the way ComfyUI is implemented. I'll explain below, for transparency, but I have no expectation that you could fix this.
As I was explained by rgthree in this post (highly recommended reading), a checkpoint loader (either yours, inside the SD Parameter Generator node, or the one I was previously using), builds a list of available models which is used as output towards other nodes. A Save Image node does the exact same.
If the total number of checkpoints in your computer changes, the workflow might generate a mismatch error between these two lists and the reroute nodes that connect them will disconnect the modelname input (like the one in the Save Image w/ Metadata node).
Apparently, there is no way to solve this problem, but to delete the existing nodes with the checkpoint lists and recreate them. Occasionally, I found that stopping ComfyUI and restarting it might fix it, too, but it's not clear to me why that would work.
It's very annoying because an image generation might complete successfully but you won't have your image saved because the modelname input is missing.
Also, the sudden disconnection is very confusing for users who are not deeply familiar with how the workflow works.
Regarding your ask for suggestions, I have a few :) They are related to the SD Parameter Generator.
As you can see in this partial screenshot of AP Workflow 4.0.2, I cannot use your node in its entirety because it's missing five key features that I find in other nodes.

Here's the five features:

A button to quickly reuse the last seed. This is incredibly useful and I have to rely on rgthree Seed node for it.

A way to automatically define the ratio between base and refiner steps. As you can see I created my own automated function for that by using 2 nodes. Some nodes allow you to define at what step to finish with the Base and at what step to start with the Refiner, but I preferred to go beyond that, giving the users the chance to input a percentage (e.g., do 75% of the steps with the Base). I then pass that variable to the rgthree Big Context node.

A way to automatically set height and width based on SDXL training ratios. For that, I use a CR SDXL Aspect Ratio node.

A way to set the batch size. For that, I use the same a CR SDXL Aspect Ratio node.

A way to set independent positive and negative aesthetic scores. For that, I use simple Constant Number nodes.

I'm happy to keep using a mix of your nodes and others as is today, but I think it's a bit confusing for people to see some outputs connected and some others not.
If you'd add these capabilities, you'd have the ultimate SD Parameter Generation node IMO.
A final suggestion:

It's great that there's a way to define the model config. Definitely a plus. But it's a bit confusing to see a name that is not associated with the mode I'm using at any given point, even if the switch with_config is set to off.

It would be easier if the drop-down config menu had an option called none or built-in selected by default.
Thank you!
(Yes, Unstable Reality is my AI R&D company. I used that account by mistake at the beginning of this thread. You can force close this feature request, or I can close it with that account.)

I just read rgthree's explanation, and it's ridiculous. I also checked reroute's source code try to understand how they fixed this issue, but it seems I can't implement a similar fix on my node due to my limited JS skills.
I recently realised a new problem. Every time I update the node and add new input/output, all workflows that use this node break because comfyui seems to locate connections between output and input solely by index rather than by name. For example, in the old version of node A, there were four inputs (a, b, c, d). When I updated to the new version with inputs (a, b, b1, c, d) and reloaded the node, (b1, c, d) would break (because comfyui treats b1 as c and c as d), and I have to reconnect them. So, I think I should try to finalise all inputs and outputs before the stable release.
About the feature suggestions:

It sounds very reasonable. I checked the source code of Efficiency Loader and rgthree, and they both implemented this feature in JS. I may need to spend some time learning how they did it, and if possible, I will add this feature.

Since I initially created the Parameter Generator just to make Prompt Saver work correctly, I didn't consider anything beyond what Prompt Saver needed. However, considering that KSampler require this parameter and it's just a few lines of code, I will add this feature.
(As you mentioned: add an input named base_refiner_step_ratio and an output named STEPS_REFINER, where STEPS = steps * ratio, STEPS_REFINER = steps - STEPS, and the default value of ratio is 1.0, meaning the refiner is turned off. How does that sound?)

Thank you for letting me know about SDXL's optimal resolution and ratio. I will add the resolutions mentioned on https://platform.stability.ai/docs/features/api-parameters as optional choices.

This also makes sense and is easy to implement.

I completely forgot about aesthetic scores. Since the models I use daily are SD1.5, I have never used aesthetic scores. However, considering that CLIPTextEncodeSDXLRefiner has this parameter, I will add this input.

About final suggestion, I have already made changes in the dev branch. When I was writing it, I simply copied the original code over and didn't think of this (clearly better) method.

I really appreciate your suggestions, they have been very helpful to me.
BTW, it's His her  SD Parameter Generator node :)

Ridiculous or not, every time I add or remove a checkpoint file to my directory, the workflow breaks. It used to disconnect the wire going into the modelname input of the Save Image node. Now, instead, it raises an error and tells me that the hash of the checkpoint list has changed.
Either way, if my directory of checkpoints changes, I need to go through acrobatics to make the workflow work again. And while I am capable of doing that, it makes really hard to distribute the workflow to other members of the community.
Re 2: IMO, the easiest-to-understand implementation would be that the node asks you to input the Refiner steps as a percentage and, as you do, it automatically calculates the number of steps and displays it in parenthesis.
For example, the label could say "Refiner Steps %", you put in 20 (as in 20%), and automatically, in parenthesis, the node shows you that 20% equals 12 (as in 12 steps).
I think the ratio is less intuitive to understand, but whichever implementation you will choose, it will be welcomed. Thank you.
Re all the other points discussed: great! I look forward to using the updated node a near future.
Thank you for implementing all these features.

Finally, everything mentioned above has been completed. How do you feel about this?

I'm very excited about this. I stand by what I said before and I think you have built the definitive Parameter Generator. Can't wait to try it.
The only thing that I would recommend is rearranging the vertical order of certain elements, if possible. Not just in terms of selectors, but also in terms of output order*.
I could give you the full order I think would work best, if you'd consider the possibility.
*I wish the underlying engine had a way to arbitrarily rearrange output and selector elements without messing with Python code. It would be transformative for the cleanness of the workflows.

About rearranging elements, I think it can be done by modifying some of the comfyUI's code using JS. Perhaps someone proficient in JS could do it. I can only adjust the input and output order by editing the Python code.
Currently, the input and output order I've designed is mostly based on KSampler, as shown in the image below. Additionally, I've placed common parameters at the beginning and SDXL parameters at the end. About the bottom section of the node (textboxes and buttons), I haven't found a way to rearranging it, so I've left it at the end.
If you have any better ideas, please don't hesitate to let me know.

It's unfortunate that the textboxes and buttons cannot be placed elsewhere, in between the output dots.
And, what I'll suggest below might become useless in a few weeks if Stability AI (hypothetically) releases SDXL 3.0 that doesn't require the refiner component anymore.
That said, this is the order that I think would be most logical:
OUTPUTS

Model_name
Model
CLIP
VAE
Seed
Steps
Base_steps << consider renaming this Base_end_step to maintain coherence with various KSamplers
Refiner_steps << consider renaming this Refiner_start_step to maintain coherence with various KSamplers
CFG
Sampler << remove suffix name to maintain consistency with the Scheduler output
Scheduler
Positive_ascore
Negative_ascore
Width
Height
Batch

SELECTORS

model_version
model_name << I would call it this way rather than ckptname, to maintain consistency with the output label
model_config
seed
steps
refiner_start
CFG
sampler
scheduler
positive_ascore
negative_ascore
aspect_ratio
width
height
batch _size

INFO WINDOWS

Step calculator
Image dimensions calculator

I would invert the position as suggested to maintain consistency with both the Output section and the Selectors section, where I suggested keeping the settings about the image towards the bottom.
I understand that this re-arrangement wouldn't align perfectly with the KSampler inputs, but I believe that those inputs were not designed for a world where the Refiner exists.
Whatever you choose, IMO it's a massive improvement over the current version, and I'll 100% incorporate it in the AP Workflow.
A final suggestion:
If you could add a final output dot, at the very end, that outputs ALL of this as a series of strings label: value, that would be monumental.
At that point, we could use 3rd party nodes to print the whole list in the terminal and/or save the whole list in a file, for debug and logging purposes. Today, I have to do this via an expansive Debug section with a dozen nodes, and it could be completely avoided.
I can't think of anything else that could improve this node. Great job!

I've implemented some suggested changes (but not all of them), and I'll explain why.
About order:
After careful consideration, I find all your suggestions to be quite reasonable. As a result, I've rearranged inputs and outputs according to your recommendations. The only exception is ckpt_name, which I believe should be placed at the beginning.
About renaming:

Renaming base_steps to base_end_step makes sense, However, renaming refiner_steps to refiner_start_step or refiner_end_step could be confusing. In my understanding, refiner_start_step = base_end_step + 1 (or refiner_start_step = base_end_step in some documents) and refiner_end_step = total_step. Therefore, I choose to keep the current naming as it is.
It's necessary to keep sampler_name align with the name displayed in KSampler inputs. just like "model" and "model_name" are two different things.
About ckp_name, I've checked build-in loaders and many custom nodes, and all loaders are named as ckp_name to distinguish it from other models, such as the upscaling model.

BTW, parameters is the debug output you're looking for.
I'll merge dev into main ASAP after I finish modifying the readme.

I'm very grateful for this work. It looks great.
I'm testing it now and I have a few minor things to report:

The two windows describing what's happening and the buttons at the end don't appear in my version:

I updated the extension and restarted ComfyUI, as usual. Not sure what's the issue.

Would it be possible to be more detailed in the aspect_ratio options? For example, instead of just writing 16:9, could we see the exact dimensions chosen, like in the other implementation I was using?

I am not sure this is possible, but as the user selects a certain aspect ratio, is it possible to automatically change the values of width and height to reflect the choice?

For example, if one chooses 16:9, could width automatically become 1344 and height automatically become 768?
I always found it extremely confusing, even in the other node I was using, that one selects a certain aspect ratio but those values don't reflect the choice.

Would it be possible to have more precision in the definition of the Refiner start? Right now I can only choose between 0.8 and 0.9, but I usually like to set it at 0.75 for a 75% ratio.

Would it be possible to print additional information in the parameters output? It's particularly important to print the base ends at step X and refiner starts at step Y.

Finally: I understand the logic for not wanting to rename the base_end and refiner_start outputs, but I would politely flag the following situation I have:

Here, to make it work as expected, I have to connect the Base_Steps output of the SD Prompt Generator node to the step_refiner input of rgthree's Context Big node. I know what's going on and I can deal with it, but it might be very confusing for other people.
A person who is less familiar with ComfyUI and the intricacies of the SDXL architecture might be naturally inclined to connect your refiner_steps output with his step_refiner input, which causes the SDXL Base model to run the few steps meant for the Refiner.
It's not the end of the world, but maybe this might be worth some more design thoughts.
Sorry for the extra requests. This node is fantastic.

Just fixed.
Different model versions have different optimal resolutions for the same aspect ratio. To avoid an excessively long list, I may need to split the node into multiple versions, similar to what other custom nodes do. This is why I decided to display this information in the info window.
Great idea, just made the change.
Just fixed
Since I can't detect whether refiner_steps is connected to anything, I'm concerned that many users, especially those using SD 1.5, might simply ignore it rather than set refiner_start to 1. In such cases, output this information might seem odd. (model_version is unreliable)
Actually, the refiner_steps in the node is useless, it wont's be used in any nodes. Since base_end_step is equal to refiner_start_step, it might be better to change base_steps to refiner_start_step and remove refiner_steps.

Trying it now. It works splendidly.
I agree on 6.
Very minor final comment: you might want to remove the comma after every debug line:

I added the commas in the parameters for consistency with the format of settings, considering that "parameters" might be used for other purposes.
Just updated 6

If you have any new ideas, please feel free to contact me anytime. :)

The only thing I can think about to improve the Parameters Generator node is to generate the output of the two windows at the bottom in real time rather than after you hit Queue as it happens now. The reason is that it might be rather confusing for people to see those values when they just set up a very different thing with the sliders. But I don't know if it's technically possible.
Other than that, I have very many other ideas for completely different nodes. Nothing related to this specific one.

Yes, that node is VERY slow and I was hoping to fix the problem. But I simply followed your original recommendation and didn't have the time to review alternatives (including your Prompt Saver).
Now I did some tests and the next version of AP Workflow will have your Prompt Saver in place of the Save Image w/Metadata node. Nothing makes me happier than reducing the number of dependencies on 3rd party custom nodes, especially if they are not maintained anymore.
Plus, the extra info input is particularly useful for something I was thinking about earlier today.
The only thing that I don't fully understand is the function of these three inputs:

I thought I could use them to create custom labels for model, sampler, and scheduler values, but they don't appear in the metadata when I read the image with your SD Prompt Reader (I use the macOS version), so I simply leave them blank.
Other than that, another great node. And yes, I have a couple of suggestions about it (but it's much better if I open a separate issue for it).

I've just added alternative inputs to Saver, please update and reload the nodes. Note that inputs of type STR have higher priority, so don't connect STR inputs unless necessary, as Saver will prioritize reading STR inputs over regular ones

As I mentioned earlier, they are just some backup inputs because you said model_name, sampler_name, and scheduler would sometimes get disconnected.
I have just tested these three inputs, everything is working fine on my end.

But now, I feel they are not needed. If these three inputs are disconnected, the other connections might have already been a mess. So perhaps I can simply remove them.

Other than that, another great node. And yes, I have a couple of suggestions about it (but it's much better if I open a separate issue for it).

You can open a new issue in https://github.com/receyuki/comfyui-prompt-reader-node/issues

Ah, I see. Only the model name gets disconnected, and it happens when the list of checkpoint in the file system changes.
Those three backup inputs I cannot use anyway because they don't allow an inbound connection from rgthree Context Big node.
So I'd say it's safe to comment them out until I can get to a point where I can reproduce the bug at will. I'm amazed so few people have noticed this issue among the ones that build complex workflows.

Alright, see you in another repo


fix: typo task definition

Purpose of the pull request

Brief change log

Verify this pull request

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

Codecov Report

Merging #10958 (5a32679) into dev (b245e7c) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #10958      +/-   ##
============================================
- Coverage     40.30%   40.29%   -0.01%
+ Complexity     4858     4855       -3
============================================
  Files           943      943
  Lines         37015    37009       -6
  Branches       4061     4055       -6
============================================
- Hits          14920    14914       -6
+ Misses        20594    20592       -2
- Partials       1501     1503       +2

Impacted Files
Coverage Δ

...he/dolphinscheduler/dao/entity/TaskDefinition.java
71.65% <ø> (ø)

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.60% <0.00%> (-0.30%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
29.49% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (+0.05%)
:arrow_up:

...hinscheduler/plugin/alert/wechat/WeChatSender.java
29.78% <0.00%> (+0.20%)
:arrow_up:

...olphinscheduler/plugin/alert/email/MailSender.java
51.38% <0.00%> (+0.35%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for your contribution. This is a good beginning. Considering that it's your first contribution, I think we can get deep communication, you can contact me through mail or add WeChat(SbloodyS), when mail or added, please tell me who you are. I think I can help to familiar with the Dolphinscheduler if you meet with problems. @MichaelDeSteven


Internal server error when cast Audio using Samsung tv

Hi,
I'm trying to cast a simple mp3 audio to Samsung DLNA, but I get this error many times:
Display Audio failure: Internal Server Error
Can anyone explain and fix this for me? Thanks

hi, I wonder if you have fixed that error yet? @kun510

@theduc0x I have fixed it

Can you show me how to fix it? @kun510

@theduc0x Please show the error or problem so I can look into it

During the process of connecting to a Samsung TV, the connection will run successfully and enter onDeviceReady

Then I used NanoHttp to create a server and wanted to transfer images to the TV
And I can see that it has displayed images on the server "http://<IP_ADDRESS>:8888/"
However, when running this function, it will report an error "error when show image: com.connectsdk.service.command.ServiceCommandError: Internal Server Error"

@kun510 Do you have any way?

You run debug fun displayMedia in ConnectSDK DLNA Service, show String url, SubtitleInfo subtitle, String mimeType, String title, String description, String iconSrc, see if there are any errors, running on the server has nothing to do with DLNA Service @theduc0x

After a while of testing, I found that if it is an https url, it will be transmitted without error, but if it is http, the error will still be reported. @kun510


Deploy BC Gov LZ2 (the GC version)

Describe the issue
BC Gov wished to test the Government of Canada's (GC) Enterprise Landing Zone (LZ) with all the extra components that come with it.
BC Gov has an existing LZ called LZ1 that is the lightweight LZ we are evolving away from.  LZ2 is intended to be the GC version, deployed to a test BC Gov account.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
AWS is willing to call this work "Solutions Architecture" even though it falls on the edge of Professional Services (ProServe) because as long as we run the deployments, they classify this as just helping us.  If they perform the deployments, we must pay for this work.
Definition of done

Run the Infrastructure as Code from ideally the BC Gov private repo dedicated to LZ2
The code must be documented or simple enough that we or our successors can come back and make sense of what we did
DOD is only that initial run happen, AWS will help us after with any fine tuning

LZ2 account created and warmed up.
Service order to increase sub-account amount in order to handle GC LZ.  Likely end of next week.
Will coordinate with Ryan J end of next week for next steps.

Service order complete, org billing set up, waiting for LZ2 to finish initializing prior to running setup

We don't have code, but the LZ2 is partially up.  Keycloak integration to come later


Scroll resets to first selected element in a multipleDates calendar when state is updated

When using multiple dates functionality, the calendar resets to the zoom to focus on the first selected date on a re-render.
I think most users would expect the scroll to remain unchanged. Is there a way to use the InfiniteCalendar API to avoid this?
See this CodeSandbox

Full code
import React, { useState } from "react";
import "./styles.css";
import InfiniteCalendar, {
  Calendar,
  defaultMultipleDateInterpolation,
  withMultipleDates,
} from 'react-infinite-calendar';
import 'react-infinite-calendar/styles.css'; // only needs to be imported once

export default function App() {

  const [selectedCalendarDates, setSelectedCalendarDates] = useState([]);

  const arrayRemove = (arr, value) => {
    return arr.filter(function (ele) {
      return ele != value;
    });
  }

  // Credit: Mayura Wijewickrama https://stackoverflow.com/questions/45504601/react-infinite-calendar-set-the-date
  const updateSelectedCalendarDates = (date) => {
    var converted =  date.toLocaleString('default', { weekday: 'short' })
                  + ' ' + date.toLocaleString('default', { month: 'short' })
                  + ' ' + date.getDate() + ' ' +date.getFullYear() + ' 00:00:00';

    if (!selectedCalendarDates.includes(converted)) {
        setSelectedCalendarDates([...selectedCalendarDates, converted]);
    } else {
        var result = arrayRemove(selectedCalendarDates, converted);
        setSelectedCalendarDates(result);
    }
  }

    return (
        <InfiniteCalendar
        Component={withMultipleDates(Calendar)}
        selected={selectedCalendarDates}
        width={'100%'}
        height={600}
        interpolateSelection={defaultMultipleDateInterpolation}
        onSelect={updateSelectedCalendarDates}
        minDate={new Date()} // Minimum selectable date is today
        min={new Date()} // Minimum date the calendar can be scrolled to is today
    />
  );
}

This is a problem with the height property,
quick fix: remove the height prop
<InfiniteCalendar
        Component={withMultipleDates(Calendar)}
        selected={selectedCalendarDates}
        width={'100%'}
        interpolateSelection={defaultMultipleDateInterpolation}
        onSelect={updateSelectedCalendarDates}
        minDate={new Date()} // Minimum selectable date is today
        min={new Date()} // Minimum date the calendar can be scrolled to is today
    />

Did you guys solve this? I tracked it down to the onSelect={updateSelectedCalendarDates}. Since it updates the state it then rerenders the component. I could understand that if you have the state in selected. But I tried with different statets and it still rerenders. Any update or fix?


java.lang.NoClassDefFoundError when instantiating classes from android project

When running the tests I am getting java.lang.NoClassDefFoundError when trying to instantiate a class in the main Android Project.
I have my project setup in the exact same fashion as the sample project. My only though on this is that since I can't seem access library files inside of the specifications, is that libraries aren't being loaded into the second project, and the class I am trying to use throws this exception when the dependency can not be loaded.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Just updated to include my work around in the project. https://github.com/pieces029/robospock-test
If you remove the groovy plugin and the dependency section you will see the exception again. Still trying to figure out a way to have this be fixed in the plugin.
For whatever reason both the groovy and java plugins don't pick up any of the dependencies if you do compile project(':app')

Just updated to include my work around in the project. https://github.com/pieces029/robospock-test
If you remove the groovy plugin and the dependency section you will see the exception again. Still trying to figure out a way to have this be fixed in the plugin.
For whatever reason both the groovy and java plugins don't pick up any of the dependencies if you do compile project(':app')

Just realized Timber also has transitive dependencies so that wasn't really a good test in that sample app.

Just realized Timber also has transitive dependencies so that wasn't really a good test in that sample app.


[UX] Define missing Material 3 surface colors

 ➡
Fixes #2346
@sufyanAbbasi @rfontanarosa FYI

Thank you, @shobhitagarwal1612 !


Possibly Missing Some Sandwiches

Type of MEV
sandwich
Other MEV Type (If "Other" selected above)
It seems to me that you're missing some sandwiches.
This is based on spot checking.
I downloaded the snapshost last week and everything ran fine.  Check for example in blocks 20214616 you only find 1 sandwich but I think i see at least one other.Sorry if i messed up something. It's univ2 so I think it's supported?
Etherscan Links to Transactions
User Tx: https://etherscan.io/tx/0x12c3332d5485d7fde113433f6ef4b4c37e3530dac055e667eb5c4025ece199bb
Sandwiches immediately above and below:
https://etherscan.io/tx/0x3c042b906f16b9f23de63af06ec40b79c0745f0e59915c08d787cb907debd29b (frontrun)
https://etherscan.io/tx/0xa6d0d72cf699153e3ff8454f0e7c759c53aee3d891a31978dcdc868573d2d4da (backrun)
Protocols Interacted With
-Univ2
-Uni Universal router (for user tx)
All Protocols Supported?

[X] Yes, all protocols were supported.
[ ] No, not all protocols were supported.

Protocols Needing a Classifier (Optional)
No response
Tokens Interacted With (Optional)
-ETH
-WETH
-WEIF
Code of Conduct

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

Hey @mmp610, thank you for pointing this out. Will investigate early this week. I assume it is getting filtered out for some reason.


Unable to create DynamoDB table in function stack

Describe the bug
I'm trying to create a DynamoDB table in a cloudformation stack of a lambda function:
    "AuthTable": {
      "Type": "AWS::DynamoDB::Table",
      "Properties": {
        "BillingMode": "PAY_PER_REQUEST",
        "ProvisionedThroughput": {
          "Ref": "AWS::NoValue"
        },
        "TableName": {
          "Fn::Join": [
            "-",
            [
              {
                "Ref": "AuthTableName"
              },
              {
                "Ref": "env"
              }
            ]
          ]
        },
        "KeySchema": {
          "AttributeName": "Key",
          "KeyType": "HASH"
        }
      }
    },

This will generate the following error:
CREATE_FAILED  AuthTable  AWS::DynamoDB::Table  Tue Apr 30 2019 13:47:29 GMT+0000 (Coordinated Universal Time) Property ProvisionedThroughput cannot be empty.

The documentation says "If you set BillingMode as PAY_PER_REQUEST, you cannot specify this property."
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-dynamodb-table.html#cfn-dynamodb-table-provisionedthroughput
Is there a possibility that the CLI will change the stack before submitting it? Is seems unlikely to me, but as I copied the above table definition I'm also quite sure this should work.

Is there a possibility that the CLI will change the stack before submitting it? Is seems unlikely to me, but as I copied the above table definition I'm also quite sure this should work.

CLI doesn't change the cloud formation template before submitting it. I think the problem here is you're using AWS: : NoValue without a condtion. If use it with a condition as shown below, it should work without problem
    "AuthTable": {
        "Type": "AWS::DynamoDB::Table",
        "Properties": {
            "BillingMode": "PAY_PER_REQUEST",
            "ProvisionedThroughput": {
                "Fn: : If": [
                    {
                        "Fn::Equals": [
                            {"Ref": "BillingMode" },
                            "PAY_PER_REQUEST"
                        ]
                    },
                    5,
                    {
                        "Ref": "AWS: : NoValue"
                    }
                ]
            },
            "TableName": {
                "Fn: : Join": [
                    "-",
                    [
                        {
                            "Ref": "AuthTableName"
                        },
                        {
                            "Ref": "env"
                        }
                    ]
                ]
            },
            "KeySchema": {
                "AttributeName": "Key",
                "KeyType": "HASH"
            }
        }
    }

@thde Closing this issue for now. If you're still stuck, please feel free to comment or re-open this issue.


Is there any way to adjust the memory limit of pika?

If we have a machine with a relatively large memory, and want pika to make full use of it to improve efficiency, how can we adjust the config file? (It seems that "write-buffer-size" is related to this problem, but pika seems to store "write-buffer-size" in an integer variable, which is at most 2^31-1 (roughly 2GB), and not big enough.)

pika use rocksdb as its storage engine, it doesn't cache more data in memory except for blockcache(8M default). write-buffer-size is only used to adjust the size of memtable and memtable is not used as cache.
Anyway, you can adjust "max-cache-files"(5000 default) to enlarge the size of tablecache. To an extent it could improve performance when you have a HUGE dataset in pika


Changing operational status of a server queues more than it should

In traffic portal, when you change the operational status of a server all DS present on the server are auto-queued for updates.  This means that peer servers within cache ring are being told to update when it makes no difference to them.  It does make sense for child cache rings since you're removing the target server from their parent cache configs.  If you have a wide edge cache with lots of DS on every server this makes it difficult to know which queues are meaningful and which are just noise.

I don't believe your statement of "all DS present on the server are auto-queued for updates" is technically correct because DSs are not queued for updates. Rather, servers (caches) are queued for updates.
My understanding is when you change "the operational status of a server" (for example, from REPORTED to ADMIN_DOWN), the system (the API (https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_ops/app/lib/API/Server.pm#L880) ) auto queues updates for all "child" caches. Child caches are defined as caches that are in a child cache group.
For example:

cache group A is the parent of cache group B
if you change the status of a cache in CG A, then all caches in CG B need to have updates queued.

why is this? because all the caches in CG B (the child cache group) have a file called parent.config which is going to change because as you know, you just changed the status of one of the "parents".
^^ yes, you said this. I was just adding a little more info for others.
Anyhow, yes, if in fact "peer servers" are being queued, this seems like a bug. If I change the status of a cache in CG A, I would not expect the other caches (peers) in CG A to have updates queued....

i'm going to do a check to see if peer caches are queued in this scenario or not. I don't believe they are.

yeah, 99% sure it doesn't do this @jhg03a . maybe it got fixed at some point. Currently, if a cache group contains caches A, B, C, D and I change the status of cache A, updates are queued for cache A (and it's children) but not for it's peers (B, C, D). Let me know if i can close this.

We can go ahead and close it out.  It's something that's seemed like it's happened several times, but I don't have a solid reproduce case for it.  If I find one, I can open a new issue or reopen this one.


it will return an EOF error when run with git bash on windows

1.The go code:
	rl, err := readline.New("> ")
	if err != nil {
		fmt.Println("ERROR:",err)
		return
	}

	defer rl.Close()
	if str, err := rl.Readline(); err == nil {
		fmt.Println(str)
	}else {
		fmt.Println("err:",err)  // got a error here when program started
	}

2.The error bellow:

3. Are there some skill to fix it? thx.

In your code above, you're just reading a single line — and stopping. As a consequence, you'll get a EOF, which is exactly what the package returns that.
I'm quite sure you forgot to enclose everything inside a for {...} loop...

I have the same issue using https://github.com/manifoldco/promptui/ and it fails immediately with an EOF when using Git Bash. All other operations succeed.
Any idea how to workaround or fix it?

I have the same problem, but if I run git bash from visual studio code, it works fine :disappointed_relieved:

I have the same problem, but if I run git bash from idea/vscode, it works fine.


Create a "blank" default template

Right now you have to specify --template hugo|jekyll. I don't love this for two reasons:

That's not 0 config ;)
Pushing Hugo or Jekyll may not the be the right choice

Instead, I propose running create-static-site <dirname> creates the bare minimum of a static site -- asset processing & static files.
This could then evolve into a few other "bare-minimum" templates like a react -> toString template.
Thoughts? @ncphillips @budparr @scottgallant ?

Those tools have unique directory structures, which would mean you'd manually have to create them, correct?

@budparr The power of C-S-S is that you can provide a template with any directory structure or files.
In this scenario, I'd just imagine getting a site that looks like:
src/
  css/
  js/
  img/
site/
  *.html
dist/
.gitignore
static-scripts.config.js
README.md

And my static-scripts.config.js would look as follows:
/**
 * Configuration file for create-static-site
 * A zero configuration build tool for static sites
 * https://github.com/forestryio/create-static-site/
 */
const {resolve} = require("path")

module.exports = () => ({
    generator: "Generator",
    command: "cp",
    proxy: false,
    args: {
        default: [resolve("site/"), resolve("dist/")],
    },
    directories: {
        src: "src/",
        build: "dist/",
        static: "dist/",
        generator: "site/"
    }
})

Just an extremely slim MVP that allows me to prototype static HTML w/ asset processing.

At some point you have to specify what tool you're using, why not up front? Perhaps I'm still missing something 🦆 (duck emoji because they had no dunce)

I guess my thought is create-react-app is 0 configuration, extremely straightforward. I want to eliminate the "choice of tool" if you're absolutely brand new to static sites.
E.g, I'm using CRA because I want to build a react app. With C-S-S, I'm using it to build a website. My initial choice becomes harder if I have to choose Jekyll, or Hugo, or Hexo, etc...
I'm thinking the most basic from of C-S-S, it provides:

best-practise javascript w/ webpack
best-practise css w/ postcss
image processing
svg sprites
static HTML in the site/ folder to be used with your processed assets.

Thoughts?

I see. Bare Metal.

best-practise javascript w/ webpack

Are you thinking separate files for prod/dev or one? I went with one to keep it simple, but now you've got me studying CRA and am musing over having two.

I went with two webpack configs because they're so heavy and I have detailed comments.

Agreed. We use one webpack configuration at Forestry right now, and I deeply regret it. The duplication is worth the simplicity.


fix images

File: machine/overview.md, CC @gbarr01
The images on this page have text which is difficult to read when selecting night mode.

I'm closing this as a duplicate of #5072


chore: bdd-spring-1592965398 to 0.0.1

chore: Promote bdd-spring-1592965398 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Edit SAKE token address

update sake token address

The decimals() is added.
Please review.Thanks
@e-asia


Add default introductory code example

Instead of having logic for a fallback in the backend, we're choosing a
default for the snippet file.
For tracks that have core exercises, we pick the first core exercise.
For tracks without these, we pick the first exercise listed in the config.
Note that we're aiming for 10 lines and a max-width of 40 columns.
This solution has a max-width of 59, so we may want to adjust it.
See https://github.com/exercism/meta/issues/89

The example you have looks to be an answer to an exercise. Just want to make sure that's okay.

Yeah, that's fine. That said, it would be better to just make this a straight-up hello world without the conditional, I think.
Is this what it would look like?
class HelloWorld {
  String hello() => "Hello, World!";
}

Normally, there should be a main() function in order for the single dart file to run properly.
So depending on whether you want the main () then it could be:

class HelloWorld {
  void main() {
    print("Hello World"):
  }
}

class HelloWorld {
  String hello() => print("Hello, World!");

  void main() {
    hello();
  }
}

Based on that this will be based on the first exercise. Then I think what you did will serve that.

The goal of the snippet isn't related to the exercises at all, it's really just a way of giving people a quick visual taste of what the language looks like.
I think both of your examples do a grand job of that, however we have a 40 column limit, and example 2 has a width of 43 characters.
So either we name the function hi() instead of hello(), or we should go with example 1.
I'm in favor of using example 1.

I think example 1 will be fine.

@Stargator I've updated the example to use your first suggestion.
Also—I really appreciate your help on this one. Would you by any chance be interested in having commit on this repo, and helping review and merge the occasional pull request? The volume is quite low, and you wouldn't be required to do anything more than you're already doing unless you wanted to.

@kytrinyx I think I would be fine with that then.

I appreciate it! Welcome to Exercism :-)
We have some (possibly woefully inadequate) documentation here: https://github.com/exercism/docs/blob/master/maintaining-a-track/README.md If you ever have any questions, please ask them so we can improve the docs!

Now I have something to read this weekend :)


Randomize dad image from a pool of 5 images

DaddyLaugh
Adding a new feature to upgrade the horizon of dad images shown on our website

Hello, please assign me to this issue.

#21  Randomize dad image from a pool of 5 images :::: Please merge my Pull request


TileMap: Updating texture_offset in _tile_data_runtime_update() has no effect

Godot version
v4.0.beta3.official [01ae26d31]
System information
Windows 10
Issue description
Changing the texture offset in the runtime TileData in _tile_data_runtime_update() does nothing, since in draw_tile() the texture_offset from the runtime TileData is ignored:
https://github.com/godotengine/godot/blob/ec521a405a89a1b80a9989c09dcc2f2e7763ddcf/scene/2d/tile_map.cpp#L1369-L1375
Steps to reproduce
Create a TileMap
Update a cell's TileData texture_offset in _tile_data_runtime_update(). The tile's position will not change.
Minimal reproduction project
RuntimeTileData.zip

You example project clearly shows a mistake, assuming that _tile_data_runtime_update is ran every frame.
This method is in fact only called when you set/update a tile, otherwise it would cause huge performance issues.
This method can be useful, but it should definitely be not used to animate tiles in a complex way (like, more complex than what the built-in tileset animation system provides). Instead, you should use a TileSetSceneCollectionSource, and paint scenes on your tilemap.

The example project calls force_update() in _process(), you can add a line to change the modulate and you can see it will change while the offset does not.
I agree TileSetSceneCollectionSource is probably a better solution for animating tile movement, but I still think it should be possible to alter offsets per tile, one example (non animation related) use case being to slightly offset tiles for visual effect:

I used "tile_tex_offset" in godot v3.5 to create a tile bouncing effect when the player hits it from below like in a Mario game. But I ran into this problem when migrating my project to v4. I believe that this restriction should be lifted.

AFAIK this is the only mechanism we have (barring TileSetScenesCollectionSource) to dynamically position a tile separately from the collision shapes defined in its physics layers. Please correct me if I'm wrong (I'd really like to do this another way for my use case).
I'm making a top-down 2D game with a pseudo height dimension. "Height" is implemented as rendering the sprite at a Y offset. I want to implement physics so all collision shapes are placed at "sea level", but the corresponding sprites can be rendered at any "height". This simplifies collision logic between objects at different heights, e.g. when stairs are involved. But I also want to use tilemaps at different heights. For this to work I need some way to e.g. render a tilemap at a certain height but send the collision shapes to "sea level". In Godot 3 I used TileSet.tile_set_texture_offset(...) to achieve this, and that's the maximum amount of hacking around I'm willing to do. If there really is no other way in Godot 4, I'd like this restriction lifted.

@groud BTW I think the name of the property and the behavior are reversed. For positive texture_offset I'd expect the tile to be visually shifted right/down (also that's how it works in 3.x). Currently it would be shifted left/up instead because as mentioned it's the offset of the texture origin, hence the visible texture offset is the opposite. So I suggest either renaming this property or reversing its behavior.

In the end I think that maybe the restriction could be lifted, at the price of maybe a confusing UI when you have weird origins sets (the origins might overlap in the property painting panel. But I guess we could find a workaround.

Even without a workaround I'd say a confusing UI is better than lack of the feature many users require / rely on. UI can always be improved later (e.g. based on the feedback of some angry users :smile:).

@groud BTW I think the name of the property and the behavior are reversed. For positive texture_offset I'd expect the tile to be visually shifted right/down (also that's how it works in 3.x). Currently it would be shifted left/up instead because as mentioned it's the offset of the texture origin, hence the visible texture offset is the opposite. So I suggest either renaming this property or reversing its behavior.

Yeah, I guess it could be renamed "texture_origin" I guess. But I feel it's a bit late now.

Even without a workaround I'd say a confusing UI is better than lack of the feature many users require / rely on. UI can always be improved later (e.g. based on the feedback of some angry users smile).

Users should not rely on that though. Animating such a value is super inefficient and should be done with a scene instead. I am not strictly against it but it is not of very high priority TBH.


Projects have trouble loading and intellisense showing errors when used with Unity

Issue Description
I'm often having issues when using OmniSharp with Unity.
I get this message when opening the project in VS Code.
"Some projects have trouble loading. Please review the output for more details."
Output log with hundreds of warnings attached.
output.txt
Also Intellisense is showing lots of CS0246 errors even though there's nothing wrong with the C# code. The namespace is present but grayed out and flagged as unnecessary.

I have tried deleting the projects and solution and letting Unity recreate but it does not fix it.
Steps to Reproduce
Expected Behavior
Actual Behavior
Logs
OmniSharp log
Post the output from Output-->OmniSharp log here
C# log
Post the output from Output-->C# here
Environment information
VSCode version: 1.44.2
C# Extension: 1.21.17
Mono Information
        OmniSharp using global mono :6.6.0
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.101
 Commit:    b377529961
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.14
OS Platform: Darwin
RID:         osx.10.14-x64
Base Path:   /usr/local/share/dotnet/sdk/3.1.101/
Host (useful for support):
Version: 3.1.1
Commit:  a1388f194c
.NET Core SDKs installed:
2.1.302 [/usr/local/share/dotnet/sdk]
2.2.104 [/usr/local/share/dotnet/sdk]
3.1.101 [/usr/local/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.2 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.2 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.2 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.2 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.1 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.2 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.2 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.1 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

azure-account
ms-vscode
0.8.9

csharp
ms-dotnettools
1.21.17

playfab-explorer
Playfab
0.1.8

unity-debug
Unity
2.7.5

vscode-azurefunctions
ms-azuretools
0.22.0

Hi @andymads, I am sorry you are running into this Unity bug. I am not sure which version of the Visual Studio Code Editor package your Unity project is configured with, but this is a known issue with version 1.1.4. See this thread where they suggest downgrading to version 1.1.3 - https://forum.unity.com/threads/visual-studio-code-editor-1-1-4-error-the-referenced-project-does-not-exist.806628/. I am closing this issue as the error is caused by a bad Unity package.


NameError: uninitialized constant SimpleCov

The simplecov README states to install the gem like so:
gem 'simplecov', :require => false, :group => :test

Simplecov Teamcity Summary states to install the gem like so:
gem 'simplecov-teamcity-summary'

When require => false is set for simplecov, and not set for simplecov-teamcity-summary, the following error is raised when trying to run tests:
NameError: uninitialized constant SimpleCov
<EMAIL_ADDRESS>rmatter.rb:2:in `<top (required)>'

It'd be nice if SimpleCov Teamcity Summary explicitly required simplecov, so it would work with the default simplecov installation.

I think this got removed in https://github.com/benc/simplecov-teamcity-summary/pull/1/files


chore: Fix consistent ordering of NodePools and Provisioners (#860) into HEAD

Fixes #N/A
Description
This cherry-picks over #860 into HEAD
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 79.946%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
7658

Relevant Lines:
9579

💛  - Coveralls


Cannot flash OTA partition

I have a project where I have 2 ota partitions and no factory partition. espflash.exe fails flashing an image with the error   ╰─▶ No factory app partition was found.

Thanks for opening this.
I went in with the assumption that the factory partition was required, which does not appear to be the case. Should be a fairly simple fix I think, I'll try to take a look at it this week.

I can also try and take a look, should you be out of time :)

That would be great! :)


Apply axioms early

User-defined axioms are now applied at the start of TermLike simplification, and we alternate user axioms and internal simplification until the result stabilizes. The StabilizeT monad transformer is introduced to track when the computation stabilizes.
Other changes:

Added Control.Monad.Stabilize for the StabilizeT transformer.
Added Kore.Step.Function.Evaluator.evaluateOnce.

Reviewer checklist

[x] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[x] Style conformance: stylish-haskell

I actually now feel like I made the stabilizer code more complicated than it needs to be for a simple state machine, and I intend to rewrite that part.


Feature/unify api forms

Closes #368

@mauriciovieira and/or @frenchbread, could you please review this PR, since we were working on it together with Brylie?

Will this fix issues https://github.com/apinf/api-umbrella-dashboard/issues/602 and https://github.com/apinf/api-umbrella-dashboard/issues/633 ?

Found an issue:
If current user does not have a permission to edit an apiBackend -> page counties to load and no error (e.g. 404) is given.
If a backend, to which user is trying to navigate does not exist, error is also not given. Page infinitely continues to load.
@elnzv Could this one be fixed?

Code looks good though!

@frenchbread, same issue in develop branch.
I think we need to create a separate issue for this.

Ok then. Merging this.


data coming back different than selected item

Hi,
I am using this library to scrape a state tableau site for COVID-19 vaccine data. My goal is to eventually obtain all of the relevant county level data. However, when I select a county a different county's data comes back, with one county "Santa Cruz" not selecting anything. This may be an issue with the worksheet.
If it is an issue with the worksheet itself, is there any workaround I can use this to select the county via the map.
Thank you!
from tableauscraper import TableauScraper as TS
import pandas as pd
import time

counties = ["Apache", "Coconino", "Cochise", "Graham", "Greenlee", "Gila", "La Paz", "Maricopa", "Mohave", "Navajo", "Pinal", "Pima",  "Yavapai", "Yuma"]

for county in counties:
    print(county)

    while True:
        try:
            data_list = []

            url = "https://tableau.azdhs.gov/views/VaccineDashboard/Vaccineadministrationdata?%3Aembed=y&"

            #initialize scraper

            ts = TS()
            ts.loads(url)

            #select that value
            dashboard = ts.getWorksheet(" County using admin county").select("Admin Address County W/ State Pod", county)

            for t in dashboard.worksheets:
                data_list.append(t.data)

            res = [int(i) for i in str(data_list[0]).split() if i.isdigit()]
            one_dose = res[1]
            print(one_dose)
            print(data_list[0])
        except:
            continue
        break

    time.sleep(30)

@az-data-guru Do you use version v0.0.8 ? I've just released it today, it fixes some bug with select and dropDown

I don't. I will update now and let you know. Thanks!

It is still showing the wrong county.

@az-data-guru  I see, there is a bug related to the index of the values in the array. In all other tableau dashboard I've seen, the values were index from 1. But it seems in this case it starts from 2. With Apache being the last with value 16 while there are only 15 values.
Also you can move ts = TS() outside of the for loop (in order not to load a new session each time) and there is already a delay in the lib of 500 ms between select/dropdown calls. It can be changed with ts = TS(delayMs=500)
from tableauscraper import TableauScraper as TS

counties = ["Apache", "Coconino"]

url = "https://tableau.azdhs.gov/views/VaccineDashboard/Vaccineadministrationdata"

ts = TS()
ts.loads(url)
worksheet = ts.getWorksheet(" County using admin county")

values = ts.getWorksheet(" County using admin county").getValues("Admin Address County W/ State Pod")
print(values)

for county in counties:
    print(county)
    dashboard = worksheet.select("Admin Address County W/ State Pod", county)
    print(dashboard.getWorksheet("Number of People").data)

I will look at the raw data to see how to get that starting index issue

It seems there is an offset between the values index, it starts from 1 but from the 3rd one, the index is shifted :
"Yuma" 1
"Yavapai" 2
"Santa Cruz" 4
"Pinal" 5

Thank you for catching that. This is something I would need to inform the tableau developer in order for it to work right?
Thank you!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Bertrand Martel @.>
Sent: Friday, March 19, 2021 6:29:54 PM
To: bertrandmartel/tableau-scraping @.>
Cc: Archer, Garrett @.>; Mention @.>
Subject: Re: [bertrandmartel/tableau-scraping] data coming back different than selected item (#6)
[EXTERNAL SENDER]
It seems there is an offset between the index values, it starts from 1 but from the 3rd one, the index is shifted :
"Yuma" 1
"Yavapai" 2
"Santa Cruz" 4
"Pinal" 5
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bertrandmartel_tableau-2Dscraping_issues_6-23issuecomment-2D803216825&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=yYgQ2xaFq6SJP6MQGae1U4R-NKhfd2gMaOiGQhYdiqc&s=xNDlMZdaiVhqkNkdQ4SWb4VyxnJRF-X2RWtzO__VWO8&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APBUTG5XHEESIELQJBJS3B3TEP3BFANCNFSM4ZPYPUAQ&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=yYgQ2xaFq6SJP6MQGae1U4R-NKhfd2gMaOiGQhYdiqc&s=FB3pXrvAQSI94tKM7JmRdPAcUzdtIv_ovhAb95QJtr8&e=.
Scripps Media, Inc., certifies that its advertising sales agreements do not discriminate on the basis of race or ethnicity. All advertising sales agreements contain nondiscrimination clauses.

@az-data-guru  I don't know. This is the first time I notice an offset in the index of selection, there must be something in the data relating to those index. But maybe I'm missing something. It seems that there is no problem with the data since the tableau website can successfully get the data. Just that the Tableau website knows that SantaCruz is index 4 (an not 3) whereas I have no idea where this info is located in the json API result.
I will continue to investigate tomorrow

Thank you!
From: Bertrand Martel @.>
Sent: Friday, March 19, 2021 7:21 PM
To: bertrandmartel/tableau-scraping @.>
Cc: Archer, Garrett @.>; Mention @.>
Subject: Re: [bertrandmartel/tableau-scraping] data coming back different than selected item (#6)
[EXTERNAL SENDER]
@az-data-guru [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_az-2Ddata-2Dguru&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=WAW6anQKfyri2997uSW_mBm12xhbjIMuPdPdQ0rQsGs&s=0srax_o3y1GoZ_MbNyADX8C-5urInMFe8qh7NP2YETY&e= I don't know. This is the first time I notice an offset in the index of selection, there must be something in the data relating to those index. But maybe I'm missing something. It seems that there is no problem with the data since the tableau website can successfully get the data. Just that the Tableau website knows that SantaCruz is index 4 (an not 3) whereas I have no idea where this info is located in the json API result.
I will continue to investigate tomorrow
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bertrandmartel_tableau-2Dscraping_issues_6-23issuecomment-2D803228295&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=WAW6anQKfyri2997uSW_mBm12xhbjIMuPdPdQ0rQsGs&s=Xhypxy9M6pgKUKRmkuJqki5mBCo7GXS0zWiAkiMeK98&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APBUTG6TODV5HHCURDSEAALTEQA7TANCNFSM4ZPYPUAQ&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=WAW6anQKfyri2997uSW_mBm12xhbjIMuPdPdQ0rQsGs&s=7Rh6HYPyx4CeWHaGjVpjoUbHXXwwJCgmMLJyycBUjP8&e=.
Scripps Media, Inc., certifies that its advertising sales agreements do not discriminate on the basis of race or ethnicity. All advertising sales agreements contain nondiscrimination clauses.

@az-data-guru  I've identified that there is a column [system:visual].[tuple_id] which points to:
{
    "tupleIds": [
        1,
        2,
        4,
        5,
        6,
        7,
        8,
        9,
        10,
        11,
        12,
        13,
        14,
        15,
        16
    ],
    "valueIndices": [],
    "aliasIndices": [],
    "formatstrIndices": []
},

This seems to be the way the index is built (called tupleId)

There is another bug related to #5. I need to look further into records of data dictionary chunks between calls

This is now working in v0.0.9
The following get all the counties:
https://replit.com/@bertrandmartel/TableauCovidArizona
from tableauscraper import TableauScraper as TS

url = "https://tableau.azdhs.gov/views/VaccineDashboard/Vaccineadministrationdata"

ts = TS()
ts.loads(url)
worksheet = ts.getWorksheet(" County using admin county")

counties = ts.getWorksheet(" County using admin county").getValues(
    "Admin Address County W/ State Pod")

for county in counties:
    print(county)
    dashboard = worksheet.select("Admin Address County W/ State Pod", county)
    print(dashboard.getWorksheet("Number of People").data)

Thank you. It works now.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Bertrand Martel @.>
Sent: Saturday, March 20, 2021 8:08:29 AM
To: bertrandmartel/tableau-scraping @.>
Cc: Archer, Garrett @.>; Mention @.>
Subject: Re: [bertrandmartel/tableau-scraping] data coming back different than selected item (#6)
[EXTERNAL SENDER]
Closed #6 [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bertrandmartel_tableau-2Dscraping_issues_6&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=n-0YBpsfr4goSqe9Za00WyD8LA9fo_4vWsBu0gY2QjY&s=emEK3a9ZS_Jt49D1w-YkEqUyihdeo4jYVBDOQtR2Hoc&e=.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bertrandmartel_tableau-2Dscraping_issues_6-23event-2D4485698617&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=n-0YBpsfr4goSqe9Za00WyD8LA9fo_4vWsBu0gY2QjY&s=TKfNXO8i3NlE21M36wlC-3lbXc2dD_9F4X6zdJRy70k&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APBUTG7BEHNWVI4IF72KBFLTES263ANCNFSM4ZPYPUAQ&d=DwMCaQ&c=aLv4kG3eFBuAUFgZFQ07JQ&r=Ja4O_iWklFEbvDJXGlgTxtHuwEfIy-pTxRJxupOQHIw&m=n-0YBpsfr4goSqe9Za00WyD8LA9fo_4vWsBu0gY2QjY&s=rI1eErkc5BzC4YbaMhtENN0JGE_Uq4J6kmErbiPwNPA&e=.
Scripps Media, Inc., certifies that its advertising sales agreements do not discriminate on the basis of race or ethnicity. All advertising sales agreements contain nondiscrimination clauses.

@az-data-guru Hello, there have been some improvement on the way the tuple index is used. If you want, you can move to v0.1.0, this would be
from tableauscraper import TableauScraper as TS

url = "https://tableau.azdhs.gov/views/VaccineDashboard/Vaccineadministrationdata"

ts = TS()
ts.loads(url)
ws = ts.getWorksheet(" County using admin county")

counties = [
  t["values"]
  for t in ws.getSelectableItems()
  if t["column"] == "Admin Address County W/ State Pod"
][0]

for county in counties:
    print(county)
    wb = ws.select("Admin Address County W/ State Pod", county)
    print(wb.getWorksheet("Number of People").data)


Private message group styling changes.

This changes the color of the private message groups to a light blue
with a darker blue header.
This also applies a fix that ensures the private message floating container is the same height as the normal containers by unbinding the border-top from the header and moving to the message group container.

Added white text to private message date headers.


high cpu usage behaviour

Hi,
I am running performance tests on one of the CF applications. cf app has 4GB memory allocated. i have triggered a very slowramp tests and i see the cpu usage is increasing gradually and reached upto 300% and then app usage drops to 0 and the test execution fails with 500 Internal Server errors.
I don't see any restarts in the cf events of the applications.
as 4Gb is allocated i understand that the app has guaranteed cpu of 1c.
can you please help me understand this cpu behaviour of cf app from 300% to 0% and then again 300% to 0% of cpu.
Does the deigo restarts the container as it is consuming more cpu than guaranteed?
Thanks and Regards,
Pooja

This does not describe a cf cli issue, though please correct me or reopen if I am misunderstanding.  I would suggest opening an issue on diego-release if you have proven that your app is not the cause of the cpu usage spikes.


The behavior of this command has been altered by the following extension: aks-preview

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks create Extension Name: aks-preview. Version: 0.5.51.
Errors:
The command failed with an unexpected error. Here is the traceback:
No module named 'azure.cli.command_modules.acs.decorator'
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 663, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 697, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 333, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "C:\Users\aymarqui\.azure\cliextensions\aks-preview\azext_aks_preview\custom.py", line 766, in aks_create
    from azure.cli.command_modules.acs.decorator import AKSParamDict
ModuleNotFoundError: No module named 'azure.cli.command_modules.acs.decorator'

Expected Behavior
Environment Summary
Windows-10-10.0.22000-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.40.0

Extensions:
aks-preview 0.5.51
azure-iot 0.14.1
blockchain 0.1.1
kusto 0.5.0

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

route to CXP team

@dejimarquis Thanks for reaching out to us and reporting this issue. I see that you are using an older version of aks-preview extension. Could you please update it using the below command and then run your AzCLI command again ?

az extension update -n aks-preview

Awaiting your reply.

@dejimarquis Thanks for reaching out to us and reporting this issue. I see that you are using an older version of aks-preview extension. Could you please update it using the below command and then run your AzCLI command again ?

az extension update -n aks-preview

Awaiting your reply.

That resolved the issue. Thanks!

@dejimarquis Thanks for getting back. We will now proceed with closure of this GitHub issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


move study design domains

Rationale
The original intent of the study design tables were to scope those five domains at the project level. In reality this only partly happened, while the domain URI is scoped to the project, the domains are created in the study container, so if the study is not at the project level the domain and property descriptors are scoped to the study folder, see StudyDesignManager.ensureStudyDesignDomains which gets called at study creation time.
The referenced bug can occur if the study which owns the domains is deleted and those descriptors and properties get cleaned up by the container listener. If there is a sibling study or in another subfolder, you'll see the NPE. The fix is simple, just ensure the domains in the correct folder, the more complicated part is moving the domains in existing studies.
The upgrade script will move both DomainDescriptors and PropertyDescriptors if the domain container isn't at the project level. It's unlikely that there will be PropertyDescriptors associated with those domains as I don't recall allowing the user to mutate those domains, but since we don't shut it off in the domain kind we still should check.
I don't recall having to do this type of cleanup before, so if either of you can make sure I'm not missing anything or orphaning objects.
related issue

Probably need to merge from develop to get this building


Fix concurrency issues

Fixes #157
tl;dr We should avoid using waterline's .save() method for database models, and instead use .update() to make updates to the database. If a document is modified between the time that it is fetched and .save() is called, the changes will be overwritten by .save() but not by .update().
There is still the possibility of leaving the database in an inconsistent state if connection is randomly lost (which is sort of difficult to avoid since mongo doesn't have transactions), but there shouldn't be any more race condition issues now.

The tests are failing because travis uses an outdated version of mongodb (2.4) for some reason. I'll figure out how to upgrade that.

You'll probably need to do the thing we did with fapp. Travis is terrible
at doing these things automatically. I'll do that.
On Sat, 25 Jun 2016, 04:00 not-an-aardvark<EMAIL_ADDRESS>wrote:

The tests are failing because travis uses an outdated version of mongodb
(2.4) for some reason. I'll figure out how to upgrade that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/porygonco/porybox/pull/183#issuecomment-228504929,
or mute the thread
https://github.com/notifications/unsubscribe/ACD_4avUWWicaD3ipkguCZ6aKNYJJrHpks5qPJnhgaJpZM4I-P9N
.

I already fixed it, this can be merged whenever.


One Line / Tabs On Bottom options - changes

Exotic tab layouts will be removed in v1.6.116 but will remain in the ESR version.

ESR release generated. Already removed from v1.6.116.


Introduce a .bazeliskrc file to define an alternate github server

First idea on how to solve parts of https://github.com/bazelbuild/bazelisk/issues/72
This is a naive approach on how the basic cases of regular releases could be switched to a different github instance.

Thank you @FaBrand! I'll check this out next week and will give some feedback :)

@nevion Thanks for the Review! I agree, quite recently i was contronted with a similar issue.
e.g if github is not used and bazel can only be downloaded though a NAS or a Service like Artifactory.
The idea of a URL pattern sounds good, maybe a list of sources in yaml?
e.g.
servers:
   - github: github.com/pattern/...
   - other_service: <...>
   - disk_location: <...>

Maybe i can get something done tomorrow to get in sync with Master again.

Any update on this? Anything the the community can do to help?

Hmmm. Didn't see this before adding (the very simple) #127

Closing this because it #127 is already merged and released.
No point in completely rewriting it

Closing this because it #127 is already merged and released.
No point in completely rewriting it


improve 'gopass git' command usage text

Fix the Usage text for gopass git, Closes #1093.
Please let me know if there are any references in the docs that I missed.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1135   +/-   ##
=======================================
  Coverage   62.57%   62.57%
=======================================
  Files         184      184
  Lines       10239    10239
=======================================
  Hits         6407     6407
  Misses       3033     3033
  Partials      799      799

Impacted Files
Coverage Δ

commands.go
87.54% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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