4090 using 512 fp16 weights is still out of memory.

Error occurred when executing ToonCrafterNode:
Allocation on device 0 would exceed allowed memory. (out of memory)
Currently allocated : 22.16 GiB
Requested : 1.25 GiB
Device limit : 23.99 GiB
Free (according to CUDA): 0 bytes
PyTorch limit (set by user-supplied memory fraction)
:<PHONE_NUMBER>4.00 GiB
File "E:\soft\ComfyUI-aki-v1.2\execution.py", line 151, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "E:\soft\ComfyUI-aki-v1.2\execution.py", line 81, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "E:\soft\ComfyUI-aki-v1.2\execution.py", line 74, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "E:\soft\ComfyUI-aki-v1.2\custom_nodes\ComfyUI-ToonCrafter_init_.py", line 161, in get_image
z, hs = self.get_latent_z_with_hidden_states(model, videos)
File "E:\soft\ComfyUI-aki-v1.2\custom_nodes\ComfyUI-ToonCrafter_init_.py", line 218, in get_latent_z_with_hidden_states
encoder_posterior, hidden_states = model.first_stage_model.encode(x, return_hidden_states=True)
File "E:\soft/ComfyUI-aki-v1.2/custom_nodes/ComfyUI-ToonCrafter/ToonCrafter\lvdm\models\autoencoder.py", line 106, in encode
h, hidden = self.encoder(x, return_hidden_states)
File "E:\soft\ComfyUI-aki-v1.2\python\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "E:\soft\ComfyUI-aki-v1.2\python\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "E:\soft/ComfyUI-aki-v1.2/custom_nodes/ComfyUI-ToonCrafter/ToonCrafter\lvdm\modules\networks\ae_modules.py", line 454, in forward
h = self.down[i_level].block[i_block](hs[-1], temb)
File "E:\soft\ComfyUI-aki-v1.2\python\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "E:\soft\ComfyUI-aki-v1.2\python\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "E:\soft/ComfyUI-aki-v1.2/custom_nodes/ComfyUI-ToonCrafter/ToonCrafter\lvdm\modules\networks\ae_modules.py", line 200, in forward
h = nonlinearity(h)
File "E:\soft/ComfyUI-aki-v1.2/custom_nodes/ComfyUI-ToonCrafter/ToonCrafter\lvdm\modules\networks\ae_modules.py", line 15, in nonlinearity
return x * torch.sigmoid(x)

@LianTianNo1 @donatienLef  There's been some discussion here about hardcoding half precision in ToonCrafter's code to make it take <24GB VRAM.

@donatienLef @LianTianNo1  We impl the low vram version. Try the latest branch for testing.


Schedules without a catch up window use the min catch up window instead of the default catch up window

Expected Behavior
It's indicated in a few places that the default catch up window on a schedule is 1 year:

The documentation website
The proto definition
The server code

So I would expect that if I don't set a catch up window on my SchedulePolicy when creating the schedule it will use the default value of 1 year.
Actual Behavior
If you don't set a catch up window on the SchedulePolicy the SDK will turn that into a duration of 0 when converting it into the proto Duration object here. Then on the server the it will see that 0 is lower than the 10 second minCatchupWindow and use the min instead here.
Steps to Reproduce the Problem
Create a schedule with a policy that does not contain a catch up window:
var policy = SchedulePolicy.newBuilder()
        .setCatchupWindow(null) // Or, remove this line since null is the default
        .build();

var schedule = Schedule.newBuilder()
        /* set action and spec and whatever else you want */
        .setPolicy(policy)
        .build();

var handle = scheduleClient.createSchedule("ScheduleId", schedule, ScheduleOptions.newBuilder().build());

Specifications

Version: As far as I can tell, all versions since Schedules were released (Server - 1.17.0+, Java SDK - 1.20.0+)
Platform:

This behaviour was changed in https://github.com/temporalio/sdk-java/commit/9a856f30 which is included in Java SDK 1.24.0+

Oh cool, we have a bunch of systems using older versions so that's what I had dug through. My fault for not checking the whole data flow in the newer version.

For existing schedules can we do a no-op update to remove the 0 value from the policy?

Yes you can update the schedule with new policy to remove to remove the value


SFDX CLI: Error: Cannot find module 'lodash/core'

Since today I'm getting an error whenever executing an sfdx command, that it cannot find the module 'lodash/core'. It is used in the file 'node_modules/jsforce/lib/api/analytics.js'. It happens when installing the cli freshly or up- / downgrading. There seems to be an issue with the dependency.

I can confirm that we're also having this problem starting some time after 7/8/2020 02:00 UTC

sfdx-cli/7.63.0-c897e8bb1a linux-x64 node-v10.20.1

sfdx plugins --core

sfdx-cli 7.63.0 (core)
Run sfdx plugins --core
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.2.3 (core)
@oclif/plugin-help 3.0.0 (core)
@oclif/plugin-not-found 1.2.3 (core)
@oclif/plugin-plugins 1.7.9 (core)
@oclif/plugin-update 1.3.9 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.0.7 (core)
analytics 1.9.1 (core)
generator 1.1.2 (core)
salesforcedx 48.19.0 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.5 (core)
├─ custom-metadata 1.0.1 (core)
├─ salesforcedx-templates 48.18.0 (core)
└─ salesforce-alm 48.20.0 (core)

sfdx-cli 7.63.0 (core)

We can confirm the same as well, due to this any fresh sfdx cli installation would be unable to load any sfdx plugins.. sfdx plugins:link is working fine

For us this happens since about 14:00 (2PM) EST. So seems the lodash v4.17.17 causes this issue (https://www.npmjs.com/package/lodash). Here the error message:
Error: Cannot find module 'lodash/core'
Require stack:

/usr/local/lib/node_modules/sfdx-cli/node_modules/jsforce/lib/api/analytics.js
/usr/local/lib/node_modules/sfdx-cli/node_modules/jsforce/lib/api/index.js
/usr/local/lib/node_modules/sfdx-cli/node_modules/jsforce/lib/jsforce.js
/usr/local/lib/node_modules/sfdx-cli/node_modules/jsforce/index.js
/usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/node_modules/<EMAIL_ADDRESS>/usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/node_modules/<EMAIL_ADDRESS><EMAIL_ADDRESS>/usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/commands/force/org/list.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>/usr/local/lib/node_modules/sfdx-cli/dist/cli.js
/usr/local/lib/node_modules/sfdx-cli/bin/run
at Function._load (/usr/local/lib/node_modules/sfdx-cli/node_modules/@salesforce/lazy-require/lib/LazyLoader.js:89:24)
at require (/usr/local/lib/node_modules/sfdx-cli/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
at Object. (/usr/local/lib/node_modules/sfdx-cli/node_modules/jsforce/lib/api/analytics.js:8:9)
at Module._compile (/usr/local/lib/node_modules/sfdx-cli/node_modules/v8-compile-cache/v8-compile-cache.js:192:30)

@dsi0  Yes thats right, same behaviour

We are also hitting this issue

We have the same issue.

I believe this points out a problem with how you are creating your npm distribution. You don't have a package-lock.json that fixes dependencies at specific versions. Without this, the npm installation is going to pull in new versions of dependency packages (like lodash). [https://www.screencast.com/t/Ramob6Qhkh8M]

Hi All - Thanks for raising & reaching out on this issue.  We're actively working on a resolution

Same here!

Same here... Happened to be making changes to our CI this morning so I thought it was something I did.  Wasted my morning chasing geese 🤦
How is this happening?  My CI is targeting a docker instance (image: salesforce/salesforcedx:7.33.2-full).  Shouldn't the sfdx-cli command already be built?
Or is it a problem that gets introduced when installing plugins?

We're working to see if there's a workaround until lodash has a fix available.

Here is the lodash issue https://github.com/lodash/lodash/issues/4849 that is causing this transient dependency issue. The only known workaround right now is to use the archives but I understand that may not be desirable to update CI jobs.

@smaddox-sf can you not just target the previous version of lodash?  Shouldn't dependency like this be "locked" in the the sfdx-cli package.json and only manually upgraded?

@ChuckJonas We can't because it is jsforce that is is using a carrot dependency. Hardcoding dependencies also have a size and perf implications on something the size of the CLI. We could potentially ship with shrinkwrap, but that would only work for npm and not others like yarn. Not only that, but there are also several problems with shrinkwrap files.
This should really be done with lock files on the user running npm install. For example, if you checked in a package.json with the sfdx-cli dependency and a lock file to your repo, then ran npm install it would use the deps in the lock file and this issue wouldn't surface. You would then just need to reference ./node_modules/.bin/sfdx-cli or add that to your path.

@ChuckJonas you will also notice the docker images, quite a while ago, have moved over to use archives other than npm. Updating your docker image will also fix this.

looks like lodash released a new @latest version.
@amphro seems like if bundle size is a serious concern, you shouldn't use jsforce in the first place 😉 (11mb unpacked!).
Or maybe jsforce could use some dependency trimming.  lodash seems pretty unnecessary these days.

I just did a fresh install and the problem now appears to be resolved. The lodash team put out a new version already.

It not just about bundle size, but Ha! Ya, I agree about jsforce ;) but that is something we can't prioritize for now.

Thanks, @amphro for your prompt help. I built a SFDX plugin (https://github.com/eltoroit/ETCopyData) that is failing due to this error, can you please let me know how to fix this issue in my plugin. Thanks

Not sure if this fixed it, but I deleted node_modules, run ncu -u (to update dependencies) and deployed a beta package to pm. Now it works.

@tfuda . Yes! Its works now for me too.


[Snackbar] Add a maxWidth property

Add a maxWidth parameter to specify the maximum width of the tooltip.
Default value is 300 to enhance readability of long text by default.
Specify maxWidth={0} to suppress the width constraint.
Add demo and api docs.
Add unit tests and pass linter validation.

I'm very hesitant to have this property. [...]
why not using CSS override?

That's a fair point. I appreciate the effort that @simoami has put in to this PR, but perhaps it should just be a customisation example instead.

I think that we should sort the demos, when possible by their popularity, so overall, people scroll less in the pages to find what they are looking for.

Well, there's a certain order to them at the moment; simplistically: basic functionality, advanced functionality, customization – but within that there's certainly some room to rearrange things.
Perhaps the TOC for each page that we discussed previously is another way to reduce  the need to scroll? Even a basic TOC would be better than none, although I do like your idea of a TOC that highlights the current section.

I'm very hesitant to have this property. [...]
why not using CSS override?

That's a fair point. I appreciate the effort that @simoami has put in to this PR, but perhaps it should just be a customisation example instead.

@oliviertassinari @mbrookes
I'm perfectly fine reducing this to a simple css prop. I think wrapping text by default is helpful if you happen to feed the tooltip with long text, as opposed to having say a 1000px one line tooltip. It's harmless for normal usage.

A sensible default along with a customization example would be perfect. 👍

@mbrookes @oliviertassinari
Ok, and just to double check, is everyone ok with 300px as a good default? Feel free to see the demo again.
One more question: do you want to see a new section for max-width customization or an extra demo block under "Customized Tooltips" ?

do you want to see a new section for max-width customization

It might be worthwhile to show the default alongside the customization, which would suggest its own section. @oliviertassinari, thoughts?

As you guys see fit. I see some value in a 200px vs 300px min-width as well as two demos vs a single demo.

A default of 200px might be too restrictive. You still want to give regular sentences a chance to appear in a single line. Example:

What do you think?

@mbrookes @oliviertassinari Take 2: #12338


home-assistant: TESTING, DON'T MERGE

Motivation for this change
Getting more details for #124444
This is a test to see if we can get the full log output from a failing
test case called test_view in the prometheus component.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build home-assistant

Result of nixpkgs-review pr 124445 run on x86_64-linux 1

1 package built:

home-assistant

@GrahamcOfBorg build home-assistant

Couldn't reproduce the issue. :disappointed:


Haskell: overrideScope loses previously applied overrides

haskellPackages.git-annex.src in 9721c545927610b77a304011d4aee72e098352cb is not what you'd expect. The previously applied overrideCabal is lost after overrideScope. This works correctly only if overrideScope is applied first.

Assigning myself, just out of interest (i.e. I can't work on this at the moment).


Member only of an "archived" project can still log in : correct behavior?

Describe the bug
A member of a project that has been archived/deactivated can still log in. I wonder whether it is a correct behavior. If the user was only member of one project and if this project is deactivated, should he still be able to log in? (He couldn't do much but still, I think the question is worth asking)
To Reproduce Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

OPTIONAL: Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem (drag-and-drop the image directly here).
Desktop (please complete the following information):

OS: macOS 10.14.6 (18G103), French
Browser Firefox
Version 70.01.1

Additional context
Add any other context about the problem here.

@kilchenmann what do you think?

Yes, they can still login, because the project is archived and not the user itself. This is something that probably Knora-Api can handle?
@flavens Can you create a user story and assign to the backend-developers? Or what do you think?

This behaviour seems correct to me.  A "user" is an user of the whole platform.

@kilchenmann @subotic if the current behaviour is correct, I will close this issue without creating a YT issue. Is it ok?

@flavens I agree to close this issue without moving to YT


[BUG] Wrong SparseTensor.dense conversion

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Given a dense torch tensor, for example:
test = torch.rand(3,3,2)
The result test tensor:

Then I try to convert this test tensor to a SparseTensor by the following code:
sparse_data = test.to_sparse()
sparse_indices = sparse_data.indices().transpose(1,0).contiguous()
# add batch to indice, shape (Nx4)
sparse_indices = torch.cat((torch.zeros(sparse_indices.size(0), 1), sparse_indices), dim=1)
sparse_feature = scan_data.values().view(-1,1)
sparse_data = SparseTensor(feats=sparse_feature.cuda(), coords=sparse_indices.cuda(), spatial_range=(3,3,2))
Then when I convert it back to its dense counterpart, the result is different from the original one:
print(sparse_data.dense().cpu().squeeze())

Is my converting code wrong or there are any problems with the SparseTensor.dense() function?
Expected Behavior
No response
Environment
- GCC:
- NVCC:
- PyTorch:2.1.2
- PyTorch CUDA: 11.8
- TorchSparse: 2.1.0

Anything else?
No response

"Hello! Have you figured out how to convert a dense tensor to a sparse tensor? For instance, I'm looking to convert a dense tensor with the shape [B, T, C, H, W] to a sparse tensor. Any insights or solutions would be greatly appreciated. Thank you!"

"Hello! Have you figured out how to convert a dense tensor to a sparse tensor? For instance, I'm looking to convert a dense tensor with the shape [B, T, C, H, W] to a sparse tensor. Any insights or solutions would be greatly appreciated. Thank you!"

I use torch.tensor.to_sparse() to transform the dense tensor to the coo sparse tensor, then I use the indices, values, and size of the created sparse tensor to create Torchsparse sparse tensor ( SparseTensor(feats=values, coords=indices, spatial_range=size) ). Remember to transform the indices as specified in the Torchsparse docs and add the batch dimension if your original tensor does not have batch dim

Thank you very much, I wrote a function and it worked!
def from_dense(x: torch.Tensor):
    """create sparse tensor fron channel last dense tensor by to_sparse
    x must be BTHWC tensor, channel last
    """
    sparse_data = x.to_sparse(x.ndim-1)
    spatial_shape = sparse_data.shape[:]
    batch_size = sparse_data.shape[0]
    sparse_indices = sparse_data.indices().transpose(1, 0).contiguous().int()

    sparse_feature = sparse_data.values()

    return SparseTensor(feats=sparse_feature.cuda(), coords=sparse_indices.cuda(), spatial_range=spatial_shape)

"Hello! Have you figured out how to convert a dense tensor to a sparse tensor? For instance, I'm looking to convert a dense tensor with the shape [B, T, C, H, W] to a sparse tensor. Any insights or solutions would be greatly appreciated. Thank you!"

I use torch.tensor.to_sparse() to transform the dense tensor to the coo sparse tensor, then I use the indices, values, and size of the created sparse tensor to create Torchsparse sparse tensor ( SparseTensor(feats=values, coords=indices, spatial_range=size) ). Remember to transform the indices as specified in the Torchsparse docs and add the batch dimension if your original tensor does not have batch dim
Thank you very much, I wrote a function and it worked!

def from_dense(x: torch.Tensor):
    """create sparse tensor fron channel last dense tensor by to_sparse
    x must be BTHWC tensor, channel last
    """
    sparse_data = x.to_sparse(x.ndim-1)
    spatial_shape = sparse_data.shape[:]
    sparse_indices = sparse_data.indices().transpose(1, 0).contiguous().int()
    sparse_feature = sparse_data.values()

    return SparseTensor(feats=sparse_feature.cuda(), coords=sparse_indices.cuda(), spatial_range=spatial_shape)


[10.x] Add option to create macroable method for paginationInformation.

This pull request allows customizing pagination information without having to extend all resources to a base resource.
This way it is possible to include the paginationInformation into a mixin:
/** @mixin \Illuminate\Http\Resources\Json\ResourceCollection */
class ResourceCollectionMixin
{
    public function paginationInformation(): Closure
    {
        return fn ($request, $paginated, $default) => collect($default)->mapWithKeysRecursively(fn ($item, $key) => [Str::camel($key) => $item])->toArray();
    }
}

Who wrote this code? Lol 🤡

Very nice


Extra Balance

I was wondering if there was a feature where if you overpaid you get balance, I looked through the code a bit and didn’t see anything

You need to code it by yourself...


Dependency scanner for Gradle doesn't work

Describe the bug
Dependency scanning for the Gradle package manager doesn't work. I have completed all the necessary prerequisites written on https://fosslight.org/fosslight-guide/scanner/2_dependency.html#android-gradle. However when I try to scan a directory containing a build.gradle file, it outputs the error
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.8/logging/__init__.py", line 1085, in emit
    msg = self.format(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 929, in format
    return fmt.format(record)
  File "/usr/local/lib/python3.8/dist-packages/coloredlogs/__init__.py", line 1140, in format
    return logging.Formatter.format(self, record)
  File "/usr/lib/python3.8/logging/__init__.py", line 668, in format
    record.message = record.getMessage()
  File "/usr/lib/python3.8/logging/__init__.py", line 373, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/usr/local/bin/fosslight_dependency", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/fosslight_dependency/analyze_dependency.py", line 1098, in main
    logger.error("Fail to generate result file. msg:()", msg)
Message: 'Fail to generate result file. msg:()'
Arguments: ('* There is no item to print in FOSSLight-Report.\n',)
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.8/logging/__init__.py", line 1085, in emit
    msg = self.format(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 929, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/logging/__init__.py", line 668, in format
    record.message = record.getMessage()
  File "/usr/lib/python3.8/logging/__init__.py", line 373, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/usr/local/bin/fosslight_dependency", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/fosslight_dependency/analyze_dependency.py", line 1098, in main
    logger.error("Fail to generate result file. msg:()", msg)
Message: 'Fail to generate result file. msg:()'
Arguments: ('* There is no item to print in FOSSLight-Report.\n',)
### FINISH!! ###

To Reproduce
I'm not sure if I'm getting it right, but dependency-license.* files and license-dependency.* files are all empty in the build/reports/license directory. This seems to be causing the error.
System environment (please complete the following information):

OS: Ubuntu 20.04
Python : 3.8

@kwmyoo
Is it android application project or java project?
If you want to analyze the dependencies in java project with gradle, then you need to set tup the gradle prerequisite(https://fosslight.org/fosslight-guide/scanner/2_dependency.html#gradle) not android-gradle(https://fosslight.org/fosslight-guide-en/scanner/2_dependency.html#android-gradle).

I used the wrong package manager. It works now.


0,01 % tsäänssi, että bob reagoi johonkin viestiin emojilla

I'll start sketching this.

Implemented reply but cannot work out how to build the unit test.

Built unit testing. Pull request waiting.


JavaScript Error!

Operating system: Windows 7 Professional SP1 x86
Messenger for Desktop version: 2.0.1

This happens when trying to execute the program.
This has happened several times and the solution is to reinstall it.
Didn't happen in previous versions.

We will continue the discussion here: https://github.com/aluxian/Messenger-for-Desktop/issues/1170


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 86eb735:
:clock10: Full build: the average full build time is 5 min(s) 55 sec(s), this is based on the last 755 build(s) for this repository.

Docs Build status updates of commit 86eb735:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


fix: LEAP-1647: Fix comments when user uses OCR textarea

The problem is that OCR textarea (displayMode=region-list) doesn't render any element in annotation, only in region list. So DOMWatcher failed trying to locate the tag.

/git merge


Adding AML Policy to Disable Public Network Access

This PR fixes

Added missing policy to disable public network access for Azure Machine Learning workspaces. This policy is not existing as built-in today and is therefore included here for reference.

Deployment works as expected and effects are also working as shown here:

Did a visual check; compared it to the policies we already have in place; looks ok to me.
Approving PR.


move datasources to init container.

To close pr https://github.com/helm/charts/pull/8637.
Datasources configmap must be collected before grafana startup. So, datasources container part is moved from containers to init-containers. And a new enviroment METHOD=LIST is added for datasource init container.
BTW, a new feature supporting METHOD is added for k8s-sidecar. If env METHOD is LIST, the program will just list the configmaps needed and then exit. To see more, pls visit https://github.com/xuxinkun/k8s-sidecar/tree/0.0.7

@zanhsieh @rtluckie Pls review.

/hold cancel

@xuxinkun, can you bump the version again pleease

@tim-beatson Change the version again.

@xuxinkun, please could you check the current version of the grafana chart. It's progressed significantly whilst this PR has been open. The version in this PR is older than the current released version

/lgtm

I believe this broke actual data source reloading. Seems like this resolved the race condition but now datasources are not hot reloadable like the dashboards are.

@stevenimle-honeywell, Grafana does not allow datasources to be reloaded like dashboards, that's the whole point of the PR. If datasources could be reloaded, the race condition would not exist.

I see, then why is this still called a sidecar? It makes it sound like this is an option.

@stevenimle-honeywell, perhaps you could suggest to @xuxinkun that he make additions to his PR to make that clearer?

Fair enough, I appreciate the prompt reply. Since this PR is already merged however, would it not make more sense to raise this as a PR itself?

Ah, I didn't see that this had been merged!

@stevenimle-honeywell
I mainly wanted to quickly implement the feature that moving the datasource sidecar container to init container at the beginning. So, I just moved the code and did not modify the sidecar. Maybe additional init container value is better. You can raise another PR to format it.


Notification warning from Apple

What
ITMS-90078: Missing Push Notification Entitlement - Your app appears to register with the Apple Push Notification service, but the app signature's entitlements do not include the 'aps-environment' entitlement. If your app uses the Apple Push Notification service, make sure your App ID is enabled for Push Notification in the Provisioning Portal, and resubmit after signing your app with a Distribution provisioning profile that includes the 'aps-environment' entitlement. Xcode does not automatically copy the aps-environment entitlement from provisioning profiles at build time. This behavior is intentional. To use this entitlement, either enable Push Notifications in the project editor's Capabilities pane, or manually add the entitlement to your entitlements file. For more information, see https://developer.apple.com/library/content/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/HandlingRemoteNotifications.html#//apple_ref/doc/uid/TP40008194-CH6-SW1.

@teolemon That's standard Apple vs flutter warning.
As we don't use push notification, we can ignore that. (I do)
cf. https://stackoverflow.com/questions/55167611/flutter-ios-app-submission-issue-warning-missing-push-notification-entitlement

ok, closing.


V42, Repair CI, Update Dependencies, Fix Ephemeral DIsk SIze Issue & Tests

/easycla

/easycla

/easycla


Easy way to add TTA?

Hi Folks,
not an issue, but perhaps a question or an enhancement request. What would be the easiest way to add Test Time augmentation to the model? For now, I see the only way is to override NeuralNet forward iter, copying the code and adding TTA inside. Prefered way would be to do add it to predict, but predict has already the numpy version of the array so the data would move between gpu and cpu

This is a good question. Regarding general image augmentation, we have examples here:
https://github.com/dnouri/skorch/blob/master/notebooks/Transfer_Learning.ipynb
https://github.com/dnouri/skorch/blob/master/examples/nuclei_image_segmentation/Nuclei_Image_Segmentation.ipynb
Regarding your use case, these examples may not fit exactly, though. We probably need a section in the docs that gives best practices to achieve different kinds of augmentation. Note, however, there is not always the one best way. E.g. it may depend on whether you want to do it all in the pytorch world (torchvision) or want to do it in the numpy world (opencv, skimage).

Here is a rough guide of how to do augmentation/preprocessing:

train & test same
train & test different

random
Dataset.transform
2 different datasets

static
Dataset.transform, sklearn Pipeline
2 different datasets

Some comments regarding the different methods:
sklearn Pipeline

falls flat if your data does not fit in memory
applies to whole data, so no difference between train and validation
allows you to work in numpy world
allows you to work on whole data at once (sometimes better performance)
is called once per fit: faster
is called once per fit: no random augmentation possible
re-use extensive sklearn toolset (FunctionTransformer, StandardScaler, ...)
makes it super easy to swap skorch NeuralNet with another sklearn estimator

Custom dataset that overrides Dataset.transform

cannot differentiate between train and validation
can use pytorch or numpy
because of how pytorch's Dataloader works, only processes one sample at a time (sometimes slow)
called once per epoch, therefore random augmentation is possible
called once per epoch, therefore may be slower

Having separate Datasets

different processing of train and validation possible
re-use pytorch toolset (e.g. torchvision ImageFolder)
because of how pytorch's Dataloader works, only processes one sample at a time (sometimes slow)
called once per epoch, therefore random augmentation is possible
called once per epoch, therefore may be slower
cannot make use of skorch's internal train/valid split

I believe what we should add is the possibility for skorch Dataset to differentiate between train and test, which probably means we need a dataset_train and a dataset_valid (analogous to iterator_train and iterator_valid).
Notably absent is a possibility to work on batches of data once per epoch and with awareness for whether we deal with train or validation data. But this is hard to achieve because of the way Dataloader works, so I don't know whether this will ever come.

Hi @BenjaminBossan, thanks! However, I am talking about Test-time augmentation, which is not exactly preprocessing.
For example, null-flip TTA would do the following:

Do predict
Flip image, do predict on flilled
Flip prediction back
Average the predicts

Here is my best attempt to do this so far, all I can say it works clunky, occassionally failing with dim mismatch messages  and I have not yet found a more elegant/working solution.
Overriding NeuralNet.evaluation_step:
    def evaluation_step(self, Xi, training=False):
        with torch.set_grad_enabled(training):
            self.module_.train(training)
            if self.tta_type == "nullflip" and not training:
                Xi_reflected =  torch.flip(Xi.data, (3,))
                y_reflected = torch.flip(self.infer(Xi_reflected), (3,))
                return (self.infer(Xi) + y_reflected) / 2.
            else:
                return self.infer(Xi)

In TTA, I would not do too much to mess with the internals of NeuralNet. I have been creating a pytorch Dataset, where I can control how my data is getting augmented. For each augmentation, I run the dataset through NeuralNet to get the prediction. While experimenting, I would save the prediction to disk to see what is going on. After I am satisfied with the number of predictions/augmentations, I load the predictions from disk and combine the predictions together.
You can also do the same thing without saving to disk, and just keeping predictions in memory.

My advise was more on a general level, though the distinction for different kinds of preprocessing/augmentation are sometimes blurry.
For your specific example, I could see the possibility of using a pipeline where you duplicate and flip each image in advance, so that every 2nd sample is the augmented one. Then during inference:
def infer(self, x, **fit_params):
    y_infer = super().infer(x, **fit_params)
    y_infer = 0.5 * (y_infer[::2] + y_infer[1::2])
    return y_infer

This would apply the transformation to training and inference, though I don't see why not.
But as stated, it is difficult to find the solution for all kinds of preprocessing/augmentation.

@BenjaminBossan one reason why one would wanna do it only on predict is because the training time would decrease dramatically. I am thinking of another way to override predict and to do it there.


Add markdown support in the forum

Markdown support can be added to cirucuitverse form by using https://github.com/vmg/redcarpet and overriding formated_content helper

I would like to work on this, If this issue is not taken.

Go ahead @VaibhavUpreti

@VaibhavUpreti are you working currently?

@tachyons
Can you take a look at this https://github.com/nhn/tui.editor
We can have this type of interface for markdown editor for the forum . I found this Js library popular and well maintained for markdown and WYSIWYG editor.


Failed to initialize OAuth2 support: Error 1071: Specified key was too long; max key length is 767 bytes

Gitea version (or commit ref): https://github.com/go-gitea/gitea/commit/7ea4bfc56172faf588142ee5637f35c0344f7534
Git version: 2.17.0
Operating system: Linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
After updating to today's master, I got the mentioned error related to commit https://github.com/go-gitea/gitea/commit/5a62eb30df3a04e76e465824f525b4ffd920b562. Database is MariaDB 10.1.32 with default character set utf8mb4. Reverting the commit fixes the error.
CC: @lafriks.

utf8mb4 is not supported currently

happens here too. have that db since gogs, migrated to gitea and was never a problem before.

Which of these should I change to support gitea?
localhost> show variables like '%character_set_%';
+--------------------------+----------------------------+
| Variable_name            | Value                      |
+--------------------------+----------------------------+
| character_set_client     | utf8                       |
| character_set_connection | utf8                       |
| character_set_database   | utf8mb4                    |
| character_set_filesystem | binary                     |
| character_set_results    | utf8                       |
| character_set_server     | utf8mb4                    |
| character_set_system     | utf8                       |
| character_sets_dir       | /usr/share/mysql/charsets/ |
+--------------------------+----------------------------+
```

Tracked the issue down to this line which defines a indexed VARCHAR(400). Is it an option to change that to VARCHAR(191)?

Found a workaround from https://github.com/go-gitea/gitea/pull/3516#issuecomment-366648794:
innodb_file_format = Barracuda
innodb_large_prefix = 1
innodb_default_row_format = dynamic

In summary, mariadb < 10.2 and mysql < 5.7 don't support longer indexed colums by default. above setting apply the settings that are default on newer versions. I realize that upgrading mariadb would be the proper solution, but the Arch package still seems to be stuck on 10.1 and I don't want to compile it myself.

Found another workaround, create the table manually with an adjusted primary index.
CREATE TABLE `oauth2_session` (
  `id` varchar(400) NOT NULL,
  `data` text,
  `created_unix` bigint(20) DEFAULT NULL,
  `updated_unix` bigint(20) DEFAULT NULL,
  `expires_unix` bigint(20) DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

ALTER TABLE `oauth2_session`
  ADD PRIMARY KEY (`id`(191));
COMMIT;

Yes, it looks like I didn't add the primary key in the end :man_facepalming: You are indeed right - the maximum you can go with the 767 bytes and utf8 limit is 255 characters.


Porting codegen for ARM64 (Apple M1)

Tried to run this on my M1 (arm64-apple-darwin21.6.0) today, after a few naive patches it runs, but failed at llvm bc lowering with the message:
Assertion failed: (Val.getType()->isSized() && "Don't know how to create an empty vreg"), function getOrCreateVRegs, file IRTranslator.cpp, line 208.
PLEASE submit a bug report to https://github.com/zero9178/Pylir
Stack dump:
0.	Program arguments: /Pylir/build/bin/pylir /Pylir/unittests/pylir/Runtime/empty.py -fpie -c -o unittests/pylir/Runtime/builtins.o -I /Pylir/src/python
1.	Running pass 'Function Pass Manager' on module 'LLVMDialectModule'.
2.	Running pass 'IRTranslator' on function '@__init__'
Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):
0  pylir                    0x0000000107a505c8 llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) + 56
1  pylir                    0x0000000107a4f4bc llvm::sys::RunSignalHandlers() + 112
2  pylir                    0x0000000107a50ca8 SignalHandler(int) + 344
3  libsystem_platform.dylib 0x0000000184c6f4a4 _sigtramp + 56
4  libsystem_pthread.dylib  0x0000000184c57ee0 pthread_kill + 288
5  libsystem_c.dylib        0x0000000184b92340 abort + 168
6  libsystem_c.dylib        0x0000000184b91754 err + 0
7  pylir                    0x00000001053a3aa4 llvm::IRTranslator::getOrCreateVRegs(llvm::Value const&) + 1924
8  pylir                    0x00000001053b1cb0 llvm::IRTranslator::translateCall(llvm::User const&, llvm::MachineIRBuilder&) + 572
9  pylir                    0x00000001053b7e54 llvm::IRTranslator::runOnMachineFunction(llvm::MachineFunction&) + 3548
10 pylir                    0x00000001069c0e6c llvm::MachineFunctionPass::runOnFunction(llvm::Function&) + 532
11 pylir                    0x000000010787542c llvm::FPPassManager::runOnFunction(llvm::Function&) + 680
12 pylir                    0x000000010787cd58 llvm::FPPassManager::runOnModule(llvm::Module&) + 60
13 pylir                    0x0000000107875d00 llvm::legacy::PassManagerImpl::run(llvm::Module&) + 1660
14 pylir                    0x000000010501b5bc pylir::CompilerInvocation::compilation(llvm::opt::Arg*, pylir::cli::CommandLine&, pylir::Toolchain const&, pylir::CompilerInvocation::Action, pylir::Diag::DiagnosticsManager&) + 11676
15 pylir                    0x00000001050180c0 pylir::CompilerInvocation::executeAction(llvm::opt::Arg*, pylir::cli::CommandLine&, pylir::Toolchain const&, pylir::CompilerInvocation::Action, pylir::Diag::DiagnosticsManager&) + 676
16 pylir                    0x00000001050b7b18 pylir::main(int, char**) + 3076
17 pylir                    0x0000000104fd7450 main + 96
18 dyld                     0x000000010bcd508c start + 520
make[2]: *** [unittests/pylir/Runtime/builtins.o] Abort trap: 6
make[1]: *** [unittests/pylir/Runtime/CMakeFiles/PylirTestRuntime.dir/all] Error 2
make: *** [all] Error 2

I think this is because I set the ABI wrong, but I have no idea how to proceed from this.
Any help is appreciated!

Definitely looks like it is LLVMs lowering to MIR that seems to be failing, although this could still be caused by invalid LLVM IR produced by the frontend.
The compiler itself is already built and in bin, the build step that is failing is just part of a unit test. You could inspect the LLVM IR generated via running it as pylir /Pylir/unittests/pylir/Runtime/empty.py -fpie -I /Pylir/src/python -S -emit-llvm -o out.ll.
pylir is also capable of compiling LLVM IR produced by it, so to reduce your edit-debug cycle you can then also trigger codegen via pylir out.ll -c.
If you could upload the out.ll I will try to also take a look. If you want to debug this yourself, your best bet is recompiling and linking pylir against an LLVM build that has assertions enabled and debug info. Just rebuilding LLVM with -DLLVM_ENABLE_ASSERTIONS=ON -DCMAKE_CXX_FLAGS="-g1", should give you a build that is not too large on disk space, nor too slow to be practible (although stepping through code or trying to view every variable might not be possible without a proper debug build or at max -O1 optimizations).

I believe I have found the issue.
The assert that I was immediately hitting when I ran your file is the following:
https://github.com/llvm/llvm-project/blob/9181ce623fd8189252659da7c48de1982597b79c/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp#L1266
Apparently sections on MachO (the object file format used by Apple platforms), requires a specific encoding, where the segment type is encoded in front of the section name, separated by ,.
I use custom sections for global variables for the sake of the GC. A plugin that runs after LLVMs optimization pipelines aggregates all variables in specific sections into arrays which serve as roots for the GC. The names of these have to changed in order to be MachO compatible. That simply means prefixing sections that are read-only with __TEXT, and __DATA, for read-write (according to [0]).
[0] https://developer.apple.com/library/archive/documentation/Performance/Conceptual/CodeFootprint/Articles/MachOOverview.html
As a quick test I did Find-Replace on the LLVM IR file you sent:

section "py_const" to section "__TEXT,py_const"
section "py_coll" to section "__DATA,py_coll"
section "py_handle" to section "__DATA,py_handle"

Afterwards it successfully created an object file.
I will try to push a commit shortly fixing it

On further thought, this does not look like the assertion/error you are hitting, is that correct? At least based on the Stackdump.
Was the above one of the changes you did locally before you then hit that assert?

I think the error is not the error I am currently hitting. I am compiling for arm64-apple-darwin, the error you hit is probably for x86_64-apple-darwin or Mach-O in general.

I actually compiled your LLVM file with the specified search and replace, so with the arm64 triple, which worked without issues for producing an object file.
The reason I used the x86 triple in the test is due to not having an ABI implementation yet, and because I did not think it mattered.

While trying to write some additional tests I actually found a small reproducer that produces the exact same crash and backtrace as your original file:
target datalayout = "e-m:o-i64:64-i128:128-n32:64-S128"
target triple = "arm64-apple-darwin21.6.0"

@"builtins.next$handle" = private global ptr addrspace(1) null, section "__TEXT,py_root"

define void @__init__() gc "pylir-gc" {
  %statepoint_token = call token (i64, i32, ptr, i32, i32, ...) @llvm.experimental.gc.statepoint.p0(i64<PHONE_NUMBER>, i32 0, ptr elementtype(void ()) @builtins.__init__, i32 0, i32 0, i32 0, i32 0) [ "deopt"() ]
  ret void
}

declare void @builtins.__init__()

declare token @llvm.experimental.gc.statepoint.p0(i64 immarg, i32 immarg, ptr, i32 immarg, i32 immarg, ...)

run via pylir test.ll -S -o -.
By changing gc "pylir-gc"  to gc "statepoint-example" it can then also be run via LLVMs codegenerator via llc -O0 test.ll which will cause the same crash.
As to why the previous one did not reproduce for me, I have no clue.

Yup, I tried it locally, it produces exactly the same error.

The issue seems to be a bug/missing support for statepoints in GlobalISel which is enabled by default in AArch64 on -O0.
As a workaround for now you can add the -fast-isel option at https://github.com/zero9178/Pylir/blob/cc2357ba517ab62026f96e3eb2d8fa6a0a8ee928/src/pylir/Main/Toolchain.cpp#L164
Will do so in a patch shortly


Roll Flutter Engine from a8432b0ddd04 to 64cc650ce585 (3 revisions)

https://github.com/flutter/engine/compare/a8432b0ddd04...64cc650ce585
2022-09-02<EMAIL_ADDRESS>Revert "Fixed the nullability annotation on codec writeValue: (#35814)" (flutter/engine#35888)
2022-09-02<EMAIL_ADDRESS>Support basic back navigation in Android 13/API 33 (flutter/engine#35678)
2022-09-02<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from RpfJRN34CLIB4LuZv... to dbH811zrzIrw0HmVz... (flutter/engine#35887)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from RpfJRN34CLIB to dbH811zrzIrw
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Fix build error when minimum target is macOS 11.0

macOS 11.0 introduced an overload of Publisher.flatMap(maxPublishers:_:)
for when the failure type is Never, documented here:
https://developer.apple.com/documentation/combine/publisher/flatmap(maxpublishers:_:)-qxf
This causes a compiler error in Effect.fireAndForget when the minimum
deployment target is 11.0 or later:
Sources/ComposableArchitecture/Effect.swift:364:7: error: ambiguous use of 'flatMap(maxPublishers:_:)'
      self
      ^
Combine.Publisher:3:17: note: found this candidate
    public func flatMap<T, P>(maxPublishers: Subscribers.Demand = .unlimited, _ transform: @escaping (Self.Output) -> P) -> Publishers.FlatMap<P, Self> where T == P.Output, P : Publisher, Self.Failure == P.Failure
                ^
Combine.Publisher:3:17: note: found this candidate
    public func flatMap<P>(maxPublishers: Subscribers.Demand = .unlimited, _ transform: @escaping (Self.Output) -> P) -> Publishers.FlatMap<Publishers.SetFailureType<P, Self.Failure>, Self> where P : Publisher, P.Failure == Never
                ^

We can fix this by adding explicit type parameters to the Empty
publisher used in the flatMap().

@aroben Hm, I just created a fresh macOS project targeting 11 and CombineSchedulers appears to build just fine as is. Do you have a project that reproduces the issue on 0.5.1? And can you let us know how you're building things? (Version of Xcode, OS, etc.)

I'm using Xcode 13 Beta 4, running on macOS 11.5.
I can reproduce like this:
$ git clone https://github.com/pointfreeco/swift-composable-architecture
$ cd swift-composable-architecture
$ xcodebuild -workspace ComposableArchitecture.xcworkspace -scheme ComposableArchitecture build MACOSX_DEPLOYMENT_TARGET=11.0

I know this project is targeting 10.15, but my app is targeting 11.0 and builds in an environment where all dependencies get targeted to 11.0 as well by overriding MACOSX_DEPLOYMENT_TARGET. So this fix isn't strictly needed for swift-composable-architecture itself, though is needed for my build environment. It seems like a bit of harmless future-proofing for when swift-composable-architecture bumps its deployment target to 11.0+.

In normal usage, if you have an Xcode project targeting 11.0 that uses swift-composable-architecture, Xcode will still build swift-composable-architecture itself with a deployment target of 10.15, so the build error doesn't occur. It's only any issue when you change swift-composable-architecture's deployment target like in that xcodebuild invocation.


saving as does not change the modeller's title?

solved by https://github.com/ReMobidyc/ReMobidyc/pull/167


[issue-827] fix mispelling of OPERATING-SYSTEM

A fix for issue #827
SPDX 2.3 reference use the term "OPERATING-SYSTEM" instead of "OPERATING_SYSTEM" which is was is output by this library.
Documentation: https://spdx.github.io/spdx-spec/v2.3/package-information/#724-primary-package-purpose-field

Hello, can someone do a review of it?

Hi @loulou123546, thanks for your contribution! This has always been ambiguous as the json schema did not match the specification you cite. However, by now, the json schema allows for both dashes and underscores, so we could indeed switch to the more spec-conform dashes.
There is also the referenceCategory which has the same problem (see here in our code). We should change that to use dashes there, too, so that we stay consistent, I think.
Do you want to take care of that, too?
I have to look into why the pipeline does not start for this PR...


[Feature Request]: Want to create the about us page for the website

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to create the about us page for the website
Use Case
It will tell more about the website
Benefits
No response
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@apu52
hey, I'd like to contribute in this issue.
I have read the Contributing Guidelines.
I'm a GSSOC'24 contributor.
I have starred the repository.

Please assign the level of difficulty according to GSSoC24.

@apu52 please assign this issue to me

@apu52 please assign this issue to me

@apu52 please assign this issue to me

Hi! I would like to work on this issue. Can you please assign it to me?

hey! please assign i'd like to contribute

Hey, I would like to work on this issue, could you assign it to me.


Clarify use of loadbalancer.ips when running behind a reverse proxy

When Zulip is run behind one or more reverse proxies, you must configure
loadbalancer.ips so that Zulip respects the client IP addresses found
in the X-Forwarded-For header. This is not immediately clear from the
documentation, so this commit makes it more clear and augments the
existing examples to showcase this need.
Fixes: #19073
Testing plan: documentation change

Thanks for this contribution, and for catching the missing docs!
I think, because the steps apply to all proxies and not just Apache, it probably makes sense to factor out these instructions and make them their own section, which the proxy-specific sections can all then reference.  For instance, nginx is currently missing a reference, though you added one for the rest.
The new section should probably have steps like (and I've not wordsmithed this, so take with grain of salt):

Before placing Zulip behind a reverse proxy, it needs to be configured to trust the client IP addresses that the proxy reports.  This is important to have accurate IP addresses in server logs, as well as in notification emails which are sent to end users.

Determine the IP address(es) of your reverse proxy, as seen from the Zulip host.  These may, of course, not be the same as the public IP addresses of the reverse proxy.
Add the following block to /etc/zulip/zulip.conf.[loadbalancer]
ips = <IP_ADDRESS> # Use the IP addresses you determined above

If you have more than one reverse proxy, separate the IP addresses with commas.
Reconfigure Zulip with these settings.  As root, run /home/zulip/deployments/current/scripts/zulip-puppet-apply.

...and then you can refer back to that section in the nginx/Apache2/HAProxy/other sections.  The key currently-missing step is that adjusting zulip.conf doesn't work unless zulip-puppet-apply it.

@alexmv thanks for the review, good catch on the missing zulip-puppet-apply, I had forgotten about that bit! I reorganized the text following your suggestions, let me know what you think!

Merged, with minor tweaks, as baea14ee5719f3fc52ceb10cc6688ad2cb4193d5.  Thanks for improving the docs, @elopez!


Implement sweep of equilibrium constants

First approach: if both constraints and eq. constants are sweeped, the arrays must have the same dimension. One entry of each is taken at a time, no outer product (meshgrid) is created!


Offline tests

How would this feature be useful?
Packaging for distros requires running the tests in a VM isolated from the internet to avoid any pollution.  Offline tests are needed for this.
However, the test suite can also be quicker if it uses a fake PyPI server that only contains the packages needed for testing.
Describe the solution you'd like
pipenv project tests uses its own, as a fixture in the repo.
There are some other earlier standalone attempts
https://github.com/uranusjr/pytest-pypi-gateway
https://github.com/uranusjr/pytest-fakepi
https://pypi.org/project/pytest-pypi/
https://github.com/manahl/pytest-plugins has a devpi pytest plugin for https://github.com/devpi/devpi
Describe alternatives you've considered
An old style PyPI HTML simple index page.

@jayvdb I've done some of this, the easiest is to just setup a devpi instance part of the pytest, as fixture; see https://github.com/tox-dev/integration-test/blob/master/tests/conftest.py#L73-L118

This would be a great addition. I've never liked how non-hermetic and slow pipx's tests are.
@gaborbernat your approach looks nice and straightforward. With pipx, the following packages are tested:
["cloudtoken", "awscli", "ansible", "shell-functools", "pycowsay", "black"]

The reason so many are tested is to exercise pipx/venv_metadata_inspector.py, which has been built as more of an art than a science. I make a guess at how to parse the files and check that things work, then add the difficult packages to CI to make sure there aren't regressions.
Ideally a minimal sdist could be created that exercises all the code paths in that file, but I don't have the tribal knowledge or time to do that, so I added more CI tests.
To make offline tests work, we'd have to add the source code for all those projects to pipx, then build+upload their respective wheels to the pydev server (as you did in tox's fixtures). That seems like a lot of bloat, so I would be interested to hear any other ideas. It would be easy to do for pycowsay for sure though.

Why add sources. We can always just add the wheels/sdist of those directly, not? Similar to how virtualenv bundles wheels to the project. The test projects wheels could be considered as test resource 👍

Also the wheels/sdist can be fetched from PyPI, and cached for re-runs.  Then they can be either added during the sdist process, and/or a doc generated describing which objects to fetch from PyPI and where to put them in order to pre-cache for offline test runs.
@dephell went as far as adding an offline job to ensure that the tests dont accidentally use network when they shouldnt.

I don't think this belongs to sdist at all, or offline. Best is to use a pytest fixture with appdirs user data storage place. One then can use pip download to acquire the wheels from the remote.

Offline is the title of this issue. ;)

I'm saying to provision the wheels at the first run for the machine if not present already. The other option is to bundle the wheels with the code.


STYLE: Remove C-style (const void *) casts

Following the C++ Core Guidelines, which says (August 3, 2020):

Don't cast between pointer types when the conversion could be implicit
...
Don't use C-style (T)expression or functional T(expression) casts

http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines.html#prosafety-type-safety-profile

Unfortunately, this style change doesn't compile on any platform 😭


Implement bridge and native backing for math functions Q# depends on

Implement the following math functions
function ArcCos (x : Double) : Double   arccos__body
function ArcSin (y : Double) : Double   arcsin__body
function ArcTan (d : Double) : Double   arctan__body
function ArcTan2 (y : Double, x : Double) :
                        Double          arctan2__body
function Cos (theta : Double) : Double  cos__body
function Cosh (d : Double) : Double     cosh__body
function Sin (d : Double) : Double      sin__body
function Sinh (d : Double) : Double     sinh__body
function Tan (d : Double) : Double      tan__body
function Tanh (d : Double) : Double     tanh__body
function Ceiling (value : Double) : Int ceiling__body
function Floor (value : Double) : Int   floor__body
function DivRemL(dividend : BigInt, divisor : BigInt) :
                    (BigInt, BigInt)    divreml__body
function IEEERemainder (x : Double, y : Double) : Double
                                        ieeeremainder_body
function Log (input : Double) : Double  log__body
function Sqrt (d : Double) : Double     sqrt__body

and other functions in the following requested order:
__quantum__qis__sqrt__body
__quantum__qis__ceiling__body
__quantum__qis__log__body
__quantum__qis__powd__body
__quantum__qis__arctan2__body
__quantum__qis__mini__body
__quantum__qis__absi__body
__quantum__qis__absd__body

__quantum__rt__int_power
__quantum__rt__pauli_to_string

__quantum__qis__drawrandomint__body
__quantum__qis__message__body

Refer the internal work item 25149 for more details.

Following https://github.com/microsoft/qsharp-runtime/pull/497, a few of those functions will no longer be intrinsic. In particular, Ceiling, Floor, MinI, AbsI, and AbsD should have pure Q# implementations such that providing explicit implementations for those should no longer be needed once #497 completes. The per-repo build completed, but that PR is currently awaiting an end-to-end build.

Implementing
function DivRemL(dividend : BigInt, divisor : BigInt) :
                    (BigInt, BigInt)    divreml__body

has been postponed until BigInt is imeplemented. See #525.


No information of Source Control type for Initialising repository

In SCM view when there is no repository, it is not clear in which SCM repository is initialised.

I don't understand... what do you mean? If you click on the Initialize Repository action, you will get to say where you want to initialize a repository.

@joaomoreno Updated the issue to be more clear. How do I know that initialise repository action is from git. What if there is other scm with initialise action?
Note we show git label when there is repository


multiple redundant dock items

Version: 1.39.2
Commit: 6ab598523be7a800d7f3eb4d92d7ab9a66069390
Date: 2019-10-15T15:33:00.827Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 19.0.0
Steps to Reproduce:
I haven't been able to isolate the steps to reproduce the problem. I did a fresh OS X install yesterday, downloaded vscode first and this is already happening.

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

@bpasero Do you know what it means when icons appear to the right of that separator?

/duplicate https://github.com/microsoft/vscode/issues/60579

YES! I've added the mac label to that issue, that's why I couldn't find it...


Continue unifying build scripts to build multi-platform

#62

Blocked by https://github.com/brefphp/aws-lambda-layers/issues/62#issuecomment-1467790855


RFC - Adds folder_watcher component

Description:
Request For Comments - folder_watcher emulates Watchdog, and fires an event on a file addition, modification or deletion. The state is the sensor is the last event. Note that you may need to add the folder to your whitelist_external_dirs.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: folder_watcher
    folder: /images

Checklist:

[ ] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Why not extend the folder sensor?

@kellerza yes I think that makes the most sense. Will close this for now.


3d layout component is not imported properly

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
hfss.modeler.insert_layout_component(
comp_file=str(fdir_model / model_3d["pcb"]),
coordinate_system="Guiding_Pin",
name="pcb",
layout_coordinate_systems=["Global", "C5", "CS_1", "H0"],
)
Steps To Reproduce
3dlayout comp is imported into HFSS, but not properly.

Which Operating System are you using?
Windows
Which Python version are you using?
3.7
Installed packages
all

missing info of "Info:="
https://github.com/ansys/pyaedt/blob/55a89f7dd04a594dc47b4f7d0abf0325ecd3c1ef/pyaedt/modeler/cad/Primitives3D.py#L1688
oComponentManager.Add(
    [
        "NAME:DCDC-Converter-App_main",
        "Info:=",
        [],
        "CircuitEnv:=", 0,
        "Refbase:=", "U",
        "NumParts:=", 1,
        "ModSinceLib:=", True,
        "CompExtID:=", 9,
        "ModelEDBFilePath:=",
        "D:\\_pycharm_project\\my_docs\\workflows\\1_WF_cad_assembly\\models\\DCDC-Converter-App_main.aedbcomp",
        "EDBCompPassword:=", ""
    ])

Below is an example of correct code recording from UI.
oComponentManager.Add(
    [
        "NAME:DCDC-Converter-App_main",
        "Info:=",
        [
            "Type:=", 29,
            "NumTerminals:=", 0,
            "DataSource:=", "",
            "ModifiedOn:=",<PHONE_NUMBER>,
            "Manufacturer:=", "",
            "Symbol:=", "DCDC-Converter-App_main",
            "ModelNames:=", "",
            "Footprint:=", "",
            "Description:=", "",
            "InfoTopic:=", "",
            "InfoHelpFile:=", "",
            "IconFile:=", "BlueDot.bmp",
            "Library:=", "",
            "OriginalLocation:=", "Project",
            "IEEE:=", "",
            "Author:=", "",
            "OriginalAuthor:=", "",
            "CreationDate:=",<PHONE_NUMBER>,
            "ExampleFile:=", "",
            "HiddenComponent:=", 0,
            "CircuitEnv:=", 0,
            "GroupID:=", 0
        ],
        "CircuitEnv:=", 0,
        "Refbase:=", "U",
        "NumParts:=", 1,
        "ModSinceLib:=", True,
        "CompExtID:=", 9,
        "ModelEDBFilePath:=",
        "D:\\_pycharm_project\\my_docs\\workflows\\1_WF_cad_assembly\\models\\DCDC-Converter-App_main.aedbcomp",
        "EDBCompPassword:=", ""
    ])


artik10: Enable host mode at boot up

This enables us to use an eth to usb adapter for example so we have
networking through the usb3 port on the artik10 devel board.
Signed-off-by: Florin Sarbu<EMAIL_ADDRESS>

@agherzan @lifeeth @petrosagg

LGTM

Please add changelog entry

SGTM

You want this in v1.1.1 release?


⚠ PrivateHD has degraded performance

In fe06cb6, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3922 ms

Resolved: PrivateHD performance has improved in 587b40c.


Cannot renew certificate from win-acme

My domain is: https://sctch.co.il
I ran this command: from win-acme
It produced this output:
[INFO] Force renewing certificate for [Manual] sctch.co.il
[WARN] First chance error calling into ACME server, retrying with new nonce...
[INFO] Authorize identifier: sctch.co.il
[INFO] Cached authorization result: valid
[INFO] Authorize identifier: www.sctch.co.il
[INFO] Authorizing www.sctch.co.il using http-01 validation (FileSystem)
[INFO] Answer should now be browsable at http://www.sctch.co.il/.well-known/acme-challenge/UG-ajdVVR6hrDeu2cCt0vQ95YOdeum_mxY5jlXOqX40
[WARN] Preliminary validation failed, found (null) instead of UG-ajdVVR6hrDeu2cCt0vQ95YOdeum_mxY5jlXOqX40.DBfAjJBOeiMcz-ocDTx2iwCr3rAEO64lpMPir1lhtmk
[EROR] {
"type": "urn:ietf:params:acme:error:unauthorized",
"detail": "The key authorization file from the server did not match this challenge "UG-ajdVVR6hrDeu2cCt0vQ95YOdeum_mxY5jlXOqX40.DBfAjJBOeiMcz-ocDTx2iwCr3rAEO64lpMPir1lhtmk" != "hello world"",
"status": 403
}
[EROR] Authorization result: invalid
[EROR] Renewal for [Manual] sctch.co.il failed, will retry on next run
My web server is (include version): apache
The operating system my web server runs on is (include version):Windows10 & XAMPP
My hosting provider, if applicable, is:
I can login to a root shell on my machine (yes or no, or I don't know):
I'm using a control panel to manage my site (no, or provide the name and version of the control panel):
The version of my client is (e.g. output of certbot --version or certbot-auto --version if you're using Certbot):
I am having trouble renewing the certificate win-acme cannot renew trough http://www.sctch.co.il
Is there any way of making win-acme renewal through http://sctch.co.il instead of www.sctch.co.il
The site cannot be reached through www .

There are a lot of tips and tricks here: https://www.win-acme.com/manual/validation-problems


Added missing sRgb getter and setter to RenderTexture.

Fixes #1092.
In cases where the user wants the sRGB conversion to take place in the RenderTexture itself as opposed to within a Texture they load somewhere else, they can now use the getter/setter to do so.

Since not many people seem to need complete sRGB support, I'd say wait until I can figure out the other half as well.

Any progress on this, @binary1248?

Nope. 😀

Bump

We should just put it on hold until more people care about this feature. 😁


Remove linkerd containers from acorn logs output

This modifies the acorn logs log to, by default, filter out logs from containers called linkerd-proxy or linkerd-init. These logs can optionally be included by passing the --include-proxy-containers flag.
Checklist

[x] The title of this PR would make a good line in Acorn's Release Note's Changelog
[ ] The title of this PR ends with a link to the main issue being address in paranthesis, like: This is a title (#1216). Here's an example
[ ] All relevant issues are referenced in the PR description. NOTE: don't use GitHub keyworkds that auto-close issues
[x] Commits follow contributing guidance
[ ] Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
[x] Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description

Still need to extend the tests to account for this new parameter. It's pretty late rn so I will work on that tomorrow.

The best way to test this is to run the logging integration test in a cluster that has the acorn-linkerd-plugin installed. Due to a weird timing issue, I had to make a modification to this integration test in order for it to actually work with linkerd sidecars in place, which is explained in a comment in the code.


Add possibility to add data of sold lunches

Backend works but frontend does not. Adding can be done in the database repository.


Translated the totality of desktop views and common views, a great bunch of mobile views

Wow, Thanks a lot @gutfuckllc!! 👍


Change operator precedence.

I think that the order of priority of operators in VB also applies to VB 6. (Although the description is VB.NET, it is linked from Operator Precedence of VB 6 document.)

https://msdn.microsoft.com/en-us/library/aa262417(v=vs.60).aspx
https://docs.microsoft.com/en-us/dotnet/visual-basic/language-reference/operators/operator-precedence

Coverage remained the same at 60.039% when pulling bc34e8145d7054c599b57f034787b60d02b9feb7 on fossamagna:fix-operator-precedence into 510ad05c2af8d5db154a4a0e62c452c20d194802 on uwol:master.


add \ae, \AE, \oe, \OE, \o, \O, \ss with unicode support

Technically, some lines in fontMetrics.js such as this one should be removed. You can also feel free to leave this to #992 if you'd like.

I can remove those.

@kevinbarabash What are your thoughts about supporting these commands in text mode?

@edemaine I assume you mean "math mode".  I didn't realize that they appear in both.  I'll put up another PR for that.

@kevinbarabash "LaTeX throws warnings (LaTeX Warning: Command \ae invalid in math mode on input line 7.) if we use these commands in math mode, but technically it does support them (generating textords)."

@edemaine I was testing using http://quicklatex.com.  I tried it again with pdflatex locally and am seeing the same issue you are.  In that case let's not bother.  The change was non trivial to b/c while we have the glyphs in KaTeX_Main-Italic they need to either be added to KaTeX_Math-Italic or we would've had to special case things in the code.

If they're specified as textords, I thought they'd be rendered in text mode using text fonts. I could be wrong though. (That is also what LaTeX does: it renders them in Roman font by default.)

For text mode they are textords and they do get rendered using KaTeX_Main-* which are the text fonts.


In the "Applied Visual Design" section of learning CSS, the definition of text-align:justify is not clear..We can set it to:"Stretches the lines so that each line has equal width (like in newspapers and magazines)"

If possible please assign this issue to me! as I am new to open source

When I'm trying to do the PR its showing this:

Please review the PR @moT01 @gikf

@nshah1729 please be patient, people will review when they can.

@nshah1729 please be patient, people will review when they can.

Alright!


Bug #176 fix systemd unit quoting

I also came across the following bug.
https://github.com/cloudalchemy/ansible-node-exporter/issues/176
Quoting the Value didn't solve it for me. See: https://github.com/cloudalchemy/ansible-node-exporter/pull/144
However, quoting all parameters did solve it for me.

This role has been deprecated in favor of a the prometheus-community/ansible collection.


Perception: isolate variables inside loop at hough_transfer

modules/perception/common/image_processing/hough_transfer.cc:150:47: error: inconsistent deduction for 'auto': 'int' and then 'std::_Rb_tree_const_iterator'
for (auto idx = 0, i = max_vote_lines.begin(); i != max_vote_lines.end();

Umm... not well tested and need to rework. thank you @muleisheng


Add functions to lookup op classes by id and vice-versa.

Those are helpful for e.g. building Transactions into memory from JSON and similar low-level utilities.

Codecov Report

Merging #93 into develop will increase coverage by 0.06%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##           develop     #93      +/-   ##
==========================================
+ Coverage    67.63%   67.7%   +0.06%
==========================================
  Files           51      51
  Lines         4082    4097      +15
==========================================
+ Hits          2761    2774      +13
- Misses        1321    1323       +2

Impacted Files
Coverage Δ

bitsharesbase/operations.py
75.19% <86.66%> (+0.7%)
: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 4b90d0d...25fef98. Read the comment docs.


Support adding screenshots to feedback

Problem Statement
The Electron SDk should support adding screenshots to feedback.

Solution Brainstorm
Why does this not work?

for some more context, when clicking the add screenshot button, i see a quick "flicker" of content where the bug reporting module takes up the full screen (presumbaly to show the screenshotting tool). then it quickly closes and i see the red error message "not supported"

The widget screenshot capture uses navigator.mediaDevices.getDisplayMedia to capture the window but Electron disables this by default.
You can enable a window capturing itself by calling this in the main process after the app ready event:
  session.defaultSession.setDisplayMediaRequestHandler((request, callback) => {
    callback({ video: request.frame })
  });

Adding this detail to the docs

https://github.com/getsentry/sentry-docs/pull/11694

Do re-open this if this doesn't fix your issue!


Detours via Nuget package ?

I need to use Detours in a project so the simplest answer is "use Detours' nuget package" - except that doesn't exist :-(
I tried adding Detours source into my project but for complicated reasons its fails and requires invasive changes to my project settings and/or structure
I could clone Detours source + makefiles and make a project whose sole purpose is to run make to build Detours' static lib for my project to consume but that's irksome (to live with as well as to setup - local builds, build pipelines, etc)
Is there a reason Detours offers no Nuget package?

You can use the vcpkg package. NuGet is generally a poor package manager for native dependencies.

Yea, the vcpkg that @sylveon contributed is a good option.
See: https://github.com/microsoft/vcpkg/tree/master/ports/detours

@sylveon Be that as it may Nuget has a certain currency in my process. An official nuget from the Detours team would be ideal
@bgianfo  I'm not familiar with vcpkg. How do I use that? Do I have to build something or can I point VS2019 at something to import it?

If you follow the instructions in the vcpkg readme for setting it up and then run vcpkg install detours:x64-windows (replace x64 by the preferred architecture), it should be automatically picked up by Visual Studio under <detours/detours.h>

Ping
Still using a forked copy of Detours source and still interested in prebuilt Nuget packages

Ping
Still using a forked copy of Detours source and still interested in prebuilt Nuget packages

I think I am using a nuget package and it works well enough. 4.0.1

Ping
Any news?

Ping. Any update?

No updates as of yet, sorry.
@DrusTheAxe Do you or the WindowsAppSDK team have cycles to help contribute the Nuget creation automation? That would certainly help move the needle here.

No updates as of yet, sorry.
@DrusTheAxe Do you or the WindowsAppSDK team have cycles to help contribute the Nuget creation automation? That would certainly help move the needle here.

I don't anytime soon. Will ask around but folks are generally pretty booked with higher pri work

FYI, there is this nuget package: https://www.nuget.org/packages/Detours
which to the nuget package manager looks like it's created by Microsoft:

but turns out it isn't:

kinda scary that it's this easy to spoof being microsoft

I tried using the vcpkg, looks like the commands have changed, now we have to use the add command like this vcpkg add port detours howerver after doing necessary setup on VS 2022, I get this error: detours.h(47,1): error C1189: #error:  Unknown architecture (x86, amd64, ia64, arm, arm64)
Can instructions to set it up on VS be published on the readme file so that we don't have to spend an hour to figure out how to use this library? Also, publishing to nuget is way better

NuGet kinda sucks for C++. AFAIK, it's still vcpkg install detours, from when I tested some weeks ago.
Your error most likely is cause you need to include windows.h first.

 AFAIK, it's still vcpkg install detours, from when I tested some weeks ago.
Your error most likely is cause you need to include windows.h first.

I got the vcpkg working and setup detours, but when I sync the repo on another machine, I again need to setup vcpkg and some of the initialization files need to be regenerated. I felt this doesn't scale. May be I don't know how to make it work on multiple machines just with cloning the repo.

You can include a vcpkg.json file in your project for it to automatically restore detours on build. Newer versions of VS come with vcpkg built-in.

You can include a vcpkg.json

I already have vcpkg.json file and following is the content of the file:
{   "dependencies": [     "detours"   ] }
I synced the repo on another machine, tried to build it on VS using Ctrl+Shift+B and I see this error. error C1083: Cannot open include file: 'detours/detours.h': No such file or directory
I don't see how the vckpg setup can be made portable. I don't want to manuvally configure vcpkg on every machine, I jsut want to clone the repo and continue developing. That's the way it works with nuget. The package download part is abstrabed away to the background.

Did you install vcpkg with VS installer? That should make it work without needing to manually configure it.

FWIW NuGet for C++ doesn't restore automatically on build until you install the targets via the installer manually too

yes, I have it installed and I initially follwed these instructions: https://learn.microsoft.com/en-us/vcpkg/get_started/get-started-msbuild?pivots=shell-cmd
On the new machine, I did  not clone the vcpkg repo, I guess I will ahve to clone the repo and perform the setup on every machine. I was looking for a self contained project setup.

On a brand new OS, I checked vcpkg in the VS installer (not a manual clone & integrate) and was able to build a project that uses vcpkg by just hitting F5

Here is my project, please let me know how to configure it: https://github.com/amithegde/Detour

ok, finally I was able to get it working. I had to remove the vcpkg-configuration.json and setup vcpkg as mentioend here: https://learn.microsoft.com/en-us/vcpkg/get_started/get-started-msbuild?pivots=shell-cmd
Additionally I removed files under vcpkg_installed and now building on VS downloads the required files and then builds the project


[homework]  Reading & experience report

Asking

[x] Ask ChatGPT for help completing this homework

Feeling, Writing, Thinking

[x] Post an experience report (a paragraph or several) as a comment on this issue. This experience report should convey what motivates you to understand and practice data thinking. An experience report may describe a collection of experiences (which may or may not relate to data), and some questions to consider may be helpful: What were you or those involved feeling? Doing? Thinking? Asking? Where were they? What systems were involved, if any? Who would you want to better understand the incentives at play?

if you are stuck, ask for help on Zulip, come to office hours, make a plan of tasks to do and ask for feedback on the plan
try using dictation on iPhone, Android, computer, and ask ChatGPT to summarize it
ask Lex for ideas or ChatGPT for ideas
edit using https://hemingwayapp.com/, Grammarly, or any other tool to help
read the below articles for ideas

Doing

[x] Review the first class: https://panopto.ut.ee/Panopto/Pages/Sessions/List.aspx?folderID=43bb180c-79a6-4324-b055-afa400ecd1a0 - ask on Zulip if you can't access this
[x] Review the Figma whiteboard from the first class: https://www.figma.com/file/SsoNLhaZwnSGkADd4eS0D8/230220-introduction?node-id=0%3A1&t=jR2hISrDchAPapYR-1
[x] Create a LinkedIn profile. (If you are not comfortable using your real name, feel free to use a fake name + email)
[x] Post your LinkedIn profile on Zulip -- Zulip is open source and enables us to analyze our chat messages with natural language processing tools such as ChatGPT, which we will do later in the course!
[x] Read https://matt.might.net/articles/how-to-email/ to understand principles of communication over email or Zulip
[x] Read parts of the Mailchimp Style Guide (https://styleguide.mailchimp.com/voice-and-tone/ or https://styleguide.mailchimp.com/writing-principles/) to understand how written communication can be guided by principles
[x] Read https://georgesaunders.substack.com/p/on-worry to understand a professional writer's perspective on writing.
[x] Read https://every.to/chain-of-thought/writing-essays-with-ai-a-guide to understand how to use voice recording, dictation on your smartphone or computer, and ChatGPT, and Lex to help you write
[x] Read https://simonwillison.net/2022/Dec/5/rust-chatgpt-copilot/ to understand how ChatGPT and CoPilot can help you learn data thinking, programming, and statistics skills
[x] Read http://web.archive.org/web/20170208134519/http://storyshots.tumblr.com/post/25032057278/22-storybasics-ive-picked-up-in-my-time-at-pixar -- especially the "Once upon a time" story telling framework Pixar used for its movies.
[x] Read https://www.newyorker.com/magazine/2023/02/13/the-dubious-rise-of-impostor-syndrome to help understand concepts that can increase our psychological safety and trust in class and how asking ChatGPT can help reduce our fear of asking for help when stuck.
[x] Read https://jaan.io/my-friend-radicalized-this-made-me-rethink-how-i-build-AI/ to learn about my journey in understanding the incentives that can affect our thoughts, feelings, and actions in society and the incentives that affect how AI such as ChatGPT is built.

Programming & Infrastructure

[x] Create a GitHub Codespace using the data thinking template here (from https://github.com/onefact/datathinking.org-codespace)
[x] Send a screenshot to Zulip of a plot you made in the GitHub codespace (of any dataset or data you generated as in the live coding demo from the first lecture; ask for help if this is unclear)
[x] Sign up for Lex - message Jaan on Zulip for an invite code
[x] Sign up for Figma as a student (https://www.figma.com/education/students/)
[x] Install copyq on Windows Powershell or Linux using Chocolatey, or install Maccy on Mac using Homebrew. This will save you a lot of time throughout the class as you copy and paste things from ChatGPT!

Experience report on why I want to practice data thinking and what motivates me
In my current job, I work with large amounts of data. That includes data from other people such as their decisions, thoughts, and problems, as well as raw machine data.  With machine data the goal is simple, analysed, improved and put it in the correct place. As I continue to work with data, I've come to realize that data is becoming increasingly important and is the way of the future.
I recently read a report stating that data is currently more expensive than oil, which is an interesting comparison. As time goes on, the biggest players in finance, technology, and business will be those who hold the keys to all the data. This data will be used to create products and services that improve the lives of everyday people, who will be the end customers for these products.
With this in mind, I am motivated to practice data thinking and improve my skills in working with data. I understand that data thinking involves not only technical skills, but also creativity and a willingness to question assumptions and explore alternative solutions. I am excited to learn new techniques and tools to better analyze, interpret, and communicate data. By practicing data thinking, I can contribute to the success of any organization I am a part of and help create a better future for everyone.

Experience report on why I want to practice data thinking and what motivates me
In my current job, I work with large amounts of data. That includes data from other people such as their decisions, thoughts, and problems, as well as raw machine data.  With machine data the goal is simple, analysed, improved and put it in the correct place. As I continue to work with data, I've come to realize that data is becoming increasingly important and is the way of the future.
I recently read a report stating that data is currently more expensive than oil, which is an interesting comparison. As time goes on, the biggest players in finance, technology, and business will be those who hold the keys to all the data. This data will be used to create products and services that improve the lives of everyday people, who will be the end customers for these products.
With this in mind, I am motivated to practice data thinking and improve my skills in working with data. I understand that data thinking involves not only technical skills, but also creativity and a willingness to question assumptions and explore alternative solutions. I am excited to learn new techniques and tools to better analyze, interpret, and communicate data. By practicing data thinking, I can contribute to the success of any organization I am a part of and help create a better future for everyone.


[zh] improve translation for node components

improve translation for node components, and add some anchors in overview components page.

Looks like we are including extra files into this PR, please check.
/hold

I've fixed this.
Thank for reminding.
/unhold

/lgtm
/approve


Capturing response body for http exceptions

How would I capture the response body for a URL that doesn't return a 2xx status, which is an HttpException?
Say I want to capture the HTML of a url that returns 404. HttpClient's HttpException's StatusCodeException does not seem to have any way of capturing the response body.
Thanks!

Ah got it!
let opts = defaults { redirects = 0, checkStatus = Just (\_ _ _ -> Nothing)) }
getWith opts "http://httpbin.org/status/418"

Thanks for figuring that out. I was struggling with the same issue.


invalid link

First of all i say thanks to this article author for the opening of  Windows 11  documents to public  visible.
Keeping Windows 11 up-to-date  link redirected to  Keeping Windows 10 up-to-date
so it must be replaced
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: db554047-b9ca-f5c9-d79f-4a70686a66db
Version Independent ID: e05bf4bc-ab29-9d32-bd53-5ca3e70aba1b
Content: Windows 11 requirements - What's new in Windows
Content Source: windows/whats-new/windows-11-requirements.md
Product: w10
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

i am adding correct link to this article now

i am closing this issue. because link is updated from author side.


Question :  embeddingModel.embed vs  embeddingModel.passage

Hi whats the difference and what's the relation with python version since the python version I believe only have 1 kind ?
from fastembed.embedding import FlagEmbedding as Embedding
from typing import List
import numpy as np

documents: List[str] = [
    "passage: Hello, World!",
    "query: Hello, World!", # these are two different embedding
    "passage: This is an example passage.",
    "fastembed is supported by and maintained by Qdrant." # You can leave out the prefix but it's recommended
]
embedding_model = Embedding(model_name="BAAI/bge-base-en", max_length=512)
embeddings: List[np.ndarray] = list(embedding_model.embed(documents)) # Note the list() call - this is a generator

Thanks

Cool, thanks


Add support for listing environment variables of a device.

Current coverage is 49.34%

Merging #12 into master will decrease coverage by -0.77% as of c489444

@@            master     #12   diff @@
======================================
  Files           44      45     +1
  Stmts         1281    1301    +20
  Branches       173     178     +5
  Methods          0       0
======================================
  Hit            642     642
  Partial         65      65
- Missed         574     594    +20

Review entire Coverage Diff as of c489444

Powered by Codecov. Updated on successful CI builds.

Hi @bartsaintgermain can you review & approve this pull request?


feat(container): add OOTB dashboard

What does this PR do?

Add new OOTB dasboard for the container integration.

Motivation
Showcase new container metrics.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

hi @maycmlee and @yzhan289
I have included your changes in the PR.
could you please review it again. 🙇

hi @maycmlee and @yzhan289
I need a last approval after that I merge a last typo fix.


Altering excluded files blocks Vite HMR

Great plugin thx! One issue made me move to vite-plugin-live-reload. When I set path to watch for only some extensons like 'src/**/*.ts' then altering other files, prevent Vite from hot reload that other files. I was testing this with .sccs files.

Interesting, probably related to reusing the same watcher than Vite is using.
Normally, you wouldn't need to force a reload for any file that is imported (such as .ts files), as Vite would already reload the page on changes.
Can you share your use case?

Normally Vite is hot reloading only one file I changed. In normal/properly written app I think this is expected behaviour but in my app this lead to many memory leaks on older version of the file, which Vite is not removing. Also old code was sometimes still running and conflicting with the new version of the file. So for me best option is to reload whole page to have fresh start.

Closing for now, as watching files that have HMR support is out of scope for this plugin.


Fix summary bug and modify article-5 for it

Background
Fixes #5 by changing markdown(inline=true) filter to safe, as is in post_macros.html:
https://github.com/schoenenberg/zola-paper/blob/643041d20be22e2f32a31ac8759e0431d93f28db/templates/post_macros.html#L5-L7
Also modified article_5.md to actually "trigger" this issue to prevent regression :)

Thanks, I like it!


Missing iris_*_world_with_joy.launch

Hi,
just tested PX4 ROS SITL Setup, following the guide from https://pixhawk.org/dev/ros/sitl#joystick_configuration. But failed when I tried
roslaunch px4 gazebo_iris_house_world.launch

or
roslaunch px4 gazebo_iris_outdoor_world.launch

Checking the launch file Firmware/launch/gazebo_iris_outdoor_world.launch and Firmware/launch/gazebo_iris_house_world.launch, it seems two other launch files, iris_outdoor_world_with_joy.launch and iris_outdoor_world_with_joy.launch, are missing.

Thanks for reporting. I'm currently reworking this and will have something more wrapped up tomorrow.

Any updates on this issue? The repository still seems to be missing the files.

@kPanesar you could try other launch file or build a new world for it.

@weiweikong True, I was just wondering if it there were any updates as it has been a few months. Thanks!

Outdated.


Use Xenial, default sudo, matrix on Travis

The failures look to be the same as master.

Nice, thanks!


fix: allow popup swipe for balance finder

Summary

Please summarize your changes, describing what they are and why they were made.

...
Changelog
- Please write a list of changes

Relevant Issues
Closes #4471
Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[ ] Linux
[ ] Windows

Mobile

[ ] iOS
[ ] Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick all of the following boxes that are relevant to your changes.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

instead of adding onBusyChanged, you could simply add to packages/shared/components/popups/BalanceFinder.svelte
import { popupState, closePopup } from '@lib/popup'
...
// prevent popup close if isBusy
$: if($mobile) {
    $popupState.hideClose = isBusy
}
...

this solution also works though, so accepting 😸 Please merge if you dont want to change the logic, perhaps is not worth spending too much time now on code optimization

Changed it, because it's much more elegant 👍


feat(settings): keyboard shortcut to open app

revive #1032
closes https://github.com/gitify-app/gitify/issues/995
Changed the shortcut key to be CommandOrControl+Shift+G since CmdOrCtrl+Alt+G is taken by Google Drive on macOS (this explains the previous flickering I reported on #1032).

What if we give the ability to set the shortcut they want?

What if we give the ability to set the shortcut they want?

Could that be a separate enhancement?


Switch to stestr for test runner

Summary
This commit switches the default test runner for qiskit-terra to use
stestr instead of the built-in runner from stdlib unittest. By default
this switches the default mode of execution to be parallel instead of
serial which can lead to significant speed ups in test execution time.
Details and comments
Closes #730

On hold, until we advance in the discussion at #730.

Just for comparison there is a noticeable speedup by running tests in parallel. Looking at the first python test run in travis it runs about ~1.9x faster:
From master (without this patch):
516.53 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/436905020#L1325
With this patch:
279.16 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/436928335#L1327

It's even more noticeable on appveyor (because the nodes are slower and smaller)
With this patch:
73.2527 seconds https://ci.appveyor.com/project/QISKit/qiskit-sdk-py/build/1.0.2699/job/v05h7a2y1k3e5pxk#L849
On master (without this patch):
307.82 seconds https://ci.appveyor.com/project/QISKit/qiskit-sdk-py/build/1.0.2696/job/tsgv4rgw3tqlx8a0#L790

Just for comparison there is a noticeable speedup by running tests in parallel. Looking at the first python test run in travis it runs about ~1.9x faster:

From master (without this patch):
516.53 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/436905020#L1325

With this patch:
279.16 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/436928335#L1327

While on it - can you revise the comparisons, as the number look a bit off? Note that master executes the online_tests, which the PRs are not - and they are very likely a good source of time spent (and one that might be difficult to shave off, unfortunately).

@diego-plan9 ah good call I forgot about that. I chose the results from here and from #1085 for comparison. For travis:
unittest runner:
291.7 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/440170798#L1445
stestr:
277.7 seconds https://travis-ci.org/Qiskit/qiskit-terra/jobs/439867792#L1316
which makes more sense to me than the earlier numbers, because the travis nodes are pretty fast so the gains by running in parallel with only fast executing tests should be less. It makes more of a difference on resource constrained environments like a laptop or appveyor. That being said for the online tests it should be a bigger difference because the slow online tests will eat a worker while the rest of the tests use the remaining 47 workers (we might need to tune the stestr scheduler to force that configuration for online tests though to get the best performance).
For appveyor:
unittest runner:
184.4 seconds https://ci.appveyor.com/project/QISKit/qiskit-sdk-py/builds/19434294/job/ehrwdm66e4414tti#L852
stestr:
101.3 seconds https://ci.appveyor.com/project/QISKit/qiskit-sdk-py/builds/19415964/job/jhruacgfufv0tpyj#L829

On hold until a discussion in #730 reaches consensus.

have we decided what we are going to do about this?

No longer on hold, let's move forward. @mtreinish do you mind to solve the conflicts with master?

The xunit xml output is blocked by a packaging bug on windows in the upstream project that converts the subunit v2 format output stream from stestr into the xunit xml format. I've pushed a PR here: https://github.com/testing-cabal/subunit/pull/33 to try and fix it. Once that merges and is in a release we can add that back. Alternatively we can carry a small (like 3-5 line) script somewhere in the repo to perform the conversion. If you think it's worth doing I can add a patch for the script and call that in appveyor.


About the use of push flow

hello,  init encoder, get frame from encoder and then send to streamQueue。。。。
have some examples of this code?

For example, I want to transfer the local video.

Not support stream file source yet.
I will update in the future.


Additional Fixture: Varytec Street Bar

https://www.thomann.de/gb/varytec_led_street_bar_16x3w_rgb_ip65.htm

Thanks but this fails validation so cannot be accepted.
You need to use the Fixture Editor of a QLC+ 4.12.0 test build.
Also, please use the new syntax.
Otherwise, post this in the forum and I'll converti it when I have some time.


youtube subtitle doesn't work

subtitles item in dict is always as none with python api
My code :
#!/usr/bin/env python
-- coding: utf-8 --
import youtube_dl
def getdata(url):
with youtube_dl.YoutubeDL({'outtmpl': '%(id)s%(ext)s'}) as ydl:
result = ydl.extract_info(url, download=False)
url = result['url']
title = result['title']
thumbnail = result.get('thumbnail')
#print('url: %r, title: %r, thumbnail: %r' % (url, title, thumbnail))
# writecvs(url, title, tags)
print result
print "-------------------------------------"
print result.get('subtitles')
getdata('https://www.youtube.com/watch?v=I4jqq4Tml1s')
I have try video with auto-subtitle and real subtitle

You'll have to add the writesubtitles parameter:
import youtube_dl
def getdata(url):
    with youtube_dl.YoutubeDL({'outtmpl': '%(id)s%(ext)s', 'writesubtitles': True}) as ydl:
        result = ydl.extract_info(url, download=False)
        url = result['url']
        title = result['title']
        thumbnail = result.get('thumbnail')
        #print('url: %r, title: %r, thumbnail: %r' % (url, title, thumbnail))
        # writecvs(url, title, tags)
        print result
        print "-------------------------------------"
        print result.get('subtitles')
getdata('https://www.youtube.com/watch?v=I4jqq4Tml1s')

Youtube subtitles info is only extracted with that option because it requires an additional request, so it will take some more time to complete the extraction.

thanks a lot, sorry