AudioDownloader

Project description
To write a python Script which uses google API to search videos for the keyword and download the audio of the respective video using pafy library
Relevant Technology
Python, Ideas about APIs
Complexity

[x]  Beginner - This project requires no or little prior knowledge of the technologies specified to contribute to the project
[ ]  Intermediate - The user should have some prior knowledge of the technologies to the point where they know how to use it,
[ ]  Advanced - The project requires the user to have a good understanding of all components of the project to contribute

Yeah sure, you can work on it. Comment on the issue.I will assign it to
you.
Regards,
Biswajeet Sahoo
On Sat, Sep 26, 2020 at 4:48 AM Pâmela G<EMAIL_ADDRESS>wrote:

Hey! I have interest in doing this and I'm going to open a PR during
Hacktoberfest!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/bislara/code-n-stitch/issues/1#issuecomment-699210275,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOAYSLNIGGAVNETVN6HDCTSHUQMPANCNFSM4RZZHWHQ
.

--
Biswajeet Sahoo,
Roll number - 117ec0323
*Undergraduate student in the ECE department *
National Institute of Technology, Rourkela, Odisha 769008
Phone : +91 97787 56623
LinkedIn https://www.linkedin.com/in/biswajeet-sahoo-172329171/  |
Facebook https://www.facebook.com/biswajeet.sahoo.lara?ref=bookmarks

Hey! I have interest in doing this and I'm going to open a PR during Hacktoberfest!
Thanks! :)

Sure. Go for it 👍

hey, i will start working on this soon :)

Hey, can I use Youtube API instead of Google API?

yes yes sure.

Any update? Feel free to reach out to me if you need any help.

Cool. Do let me know if you need any help

I will make a separate configuration file so that each user can generate their key when using the application, and there will also be a readme explaining how to do it

I'm almost done working on it, I'm just waiting for the answer to an issue I found from a package I'm using in the application to proceed.

Okay cool.

Finally open!


Some values in AndroidManifest.xml are not resolved

<uses-feature
name='android.software.leanback'
required='@res/0xFFFFFFFF'>
while required is 'true'

Fixed on:
https://github.com/googlesamples/android-classyshark/pull/23


Provide example how to initialize library for case when Application XAML is absent

Is your feature request related to a problem? Please describe
My assembly is XAML component in native application, so it does not contain Application XAML with global resources dictionary, so i do not how to correctly initialize WPF-UI library so it works with my XAML Window.
Describe the solution you'd like
In cases when XAML windows works aka embedded component in non-WPF applications (addins in native app, components in WinForm app and so on) it would be useful to see how to correctly initialize the library (controls and theme).
Describe alternatives you've considered
No response
Additional context
No response

for example. It could be something like:
            if (Application.Current == default)
            {
                new System.Windows.Application();
            }

            Debug.Assert(Application.Current != default);
            if (Application.Current != default)
            {
                var dictionaries = Application.Current.Resources.MergedDictionaries;

                if (!dictionaries.Any(d => d is ThemesDictionary))
                {
                    Application.Current.Resources.MergedDictionaries.Add(new ThemesDictionary() { Theme = Wpf.Ui.Appearance.ThemeType.Light });
                }

                if (!dictionaries.Any(d => d is ControlsDictionary))
                {
                    Application.Current.Resources.MergedDictionaries.Add(new ControlsDictionary());
                }
            }

Hi! I'm also having this problem. Have you found a solution yet?
Anybody able to help?

@laurabeseler do you have or could you make a sample project by chance? Just something that has the same setup as what you're looking to use this in.

Hi, I actually managed to get it to work.
I used a ResourceDictionary to add the themes and control dictionaries:
<ResourceDictionary.MergedDictionaries> <ui:ThemesDictionary Theme="Light" />         <ui:ControlsDictionary />  </ResourceDictionary.MergedDictionaries>
And then in the main class library project that uses my UI project I had to manually load the Wpf.Ui.dll assembly.


Refactor query param infix operators to avoid 2.13 error

What's the problem?
When using +? operator to add a query param to an Uri value in Scala 2.13.3 the following warning is raised:
[warn] ${BASE}/Test.scala:14:37: multiarg infix syntax looks like a tuple and will be deprecated
[warn]   val uri = uri"localhost" / "test" +? ("key", "value")
[warn]                                     ^

The warning only dissapears if no-infix syntax is used:
(uri"localhost" / "test").+?("key", "value")

...which completely breaks the goal of those operators.
Changes included in this PR

Separate usages of +? and +?? in tests to be able to see the error (if someone tries to compile the first commit of this PR it should fail with multiarg infix syntax looks like a tuple and will be deprecated.
Since +? has two two-param version methods, we need to rename +?(T, Seq) to ++? to avoid clashing between them once they're changed to their "tuple" version.
Change +?, ++? and +?? two-param operators to receive a tuple.

Referenced PR
This fixes #3719

Looks good. I'm not that certain about introducing the new ++? operator though, but it looks as a reasonable change to me.

Yeah, me neither, but I couldn't find other alternative given the clash after erasure 😞


Move ks_util into kubeflow/testing

kubeflow/testing#271 adds the ks_util module so we delete the copy in
kubeflow/tf-operator and use that copy.

kubeflow/testing#271 also updates the ks code to better support
different versions of ks by determining the ks version to use based
on the apps app.yaml file.

This change is

Coverage remained the same at 65.003% when pulling 0bb93ab74e408bbf754fc9ec1fb273af73e7db7f on jlewi:test_util into 41065e87d5b87ab4b4bbd30e4b18361bb87d9de8 on kubeflow:master.

/assign @richardsliu
/assign @johnugeorge

/lgtm
/approve


Debugging simple javascript file triggers lengthy download

Issue Type: Bug
Wanted to step through some simple JS functions and went to  kick off a simple node debug session only to have to wait a super long time for OmniSharp and .NET Core Debugger to load.
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:31:32.854Z)
OS version: Darwin x64 18.7.0

/needsMoreInfo
Could you provide exact steps that you used and configuration in launch.json?


Having problem running /notebooks/playground.ipynb

I got this error
InvalidArgumentError (see above for traceback): Must have updates.shape = indices.shape + params.shape[1:] or updates.shape = [], got updates.shape [155,372], indices.shape [154], params.shape [10000,372]
	 [[{{node ppo/observe-timestep/store/ScatterUpdate}} = ScatterUpdate[T=DT_FLOAT, Tindices=DT_INT32, use_locking=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ppo/initialize/latest/initialize/state-state, ppo/observe-timestep/store/mod/_255, ppo/strided_slice, ^ppo/observe-timestep/store/AssignSub/_261)]]

when running this cell block
# Instantiate and run the environment for 5 episodes.
wrapped_env = WrappedEnv(env)
runner = Runner(agent=agent, environment=wrapped_env)
runner.run(episodes=5, max_episode_timesteps=2000)
print("Stats: ", runner.episode_rewards, runner.episode_timesteps, runner.episode_times)
try:
    runner.close()
except AttributeError as e:
    pass

I didn't modify the notebook so I am wondering where the error comes from. Seems something goes wrong with data collection and causes dimension dismatch.

Hi, sorry for the delay. Is this still happening?


Wrong port used in config/puma

ENV['PORT'] in config/puma returns 5000 even when PORT: 3000 in .env

The .env file is read by the dotenv gem, not your system/RVM... it shouldn't get the port 3000 if you have this kind of setup.
The ENV['PORT'] on config/puma.rb is used by production environments, where you have the ENVs set via the system (like heroku config:set) and not .env files.

@TiuTalk so If I am in my development machine and run foreman start the port should be 5000? I find this confuse. Where the port 5000 comes from?

The default port on config/puma.rb is 3000:
https://github.com/Helabs/pah/blob/master/lib/pah/files/config/puma.rb
The default por of foreman is 5000:

Your web process loads on port 5000 because this is what Foreman provides as a default in the $PORT env var.

https://devcenter.heroku.com/articles/procfile#developing-locally-with-foreman

ping @marciliojrs


authenticating proxy user/password

Hi,
I just wanted to mention, that I do not think that having
proxy user and password as a environment variable is
safe to do. As safety is one of ocicl's primary concerns,
I think you should look into this.
Marc

This is standard practice.  If there's a better way to do this, please reopen the ticket.

but not safe!
I recently saw a user level proxy, that stored the password in an encrypted file.
I think, this is a good solution. Unfortunately I do not remember, waht it was...


Changing copy

👷 Deploy Preview for ar-stem-study processing.

Name
Link

🔨 Latest commit
7a488a46143967d5f1dcc164d7393982b998b200

🔍 Latest deploy log
https://app.netlify.com/sites/ar-stem-study/deploys/629990205aa3680008198427


Unable to delete cluster because of coredns PDB

After creating a cluster with eksdemo create cluster test, I am unable to delete it using eksdemo delete cluster test.  I get the following error: 1 pods are unevictable from node xxxx.  This is a result of the pod disruption budget set on coredns.  eksctl has implemented a way to get past this by using the disable-nodegroup-eviction flag, eksctl delete cluster --disable-nodegroup-eviction.  This has been documented in this GitHub issue

fixed in #139


Removing dynamically created pods on chart remove

Help request
Our chart can dynamically create pods via k8s API calls. Which we must add to metadata for removing these pods when we remove all chart?
I tried to add the same metadata as (they are) in pods, which were created by Helm chart. But it didn't work /help
metadata:
  labels:
    app.kubernetes.io/app: ...
    app.kubernetes.io/instance: ...
    app.kubernetes.io/managed-by: ...
    helm.sh/chart: ...
    name: ...

Does Helm 3 can handle that case?
Output of helm version:
v3

@djklim87 I'm a little unsure what you are doing here. Do you mind providing an example of the templates involved? What do you mean you dynamically creating pods?

I have deployment which calls Kubernetes API and can create pods like that:
POST /apis/apps/v1/statefulsets HTTP/1.1
Authorization: Bearer $TOKEN
Accept: application/json
Connection: close
Content-Type: application/json'

{
  "kind": "StatefulSet",
  "apiVersion": "apps/v1",
  ...
}

As a result, we get statefulsets, deployments, services, and others, which don't relate to our chart.
i.e. they work as chart part, but if I will uninstall chart, these pods are still leave in k8s

In other words, I need the way to allocating new pods to my chart

@djklim87 The k8s resources you are creating are independent of the Helm chart and as such unknown to the chart. The Helm release therefore doesn't manage these resources which includes removing the resources.
You would need to declare in your chart, what resources you want deployed.

@hickeyma Is there a way to do that? Maybe I can use some labels/metadata which Helm understands and receives? Or some callback, or another way? Or Helm has no ability to do that?

It might be worth taking a look at using kustomize in conjunction with Helm. There currently is a proposal open in Helm #5276 which might provide some background.

Ok, as I understand Helm stored data of installed charts inside, not in Kubernetes? Or he synchronizes this data with cluster, and after make some actions like removing or updating? If it synchronizes I can just add some labels to my pods and Helm will think that it's own?

You would need to update the release data store in the cluster.
Here are a some ways you could do that:

In Helm 3, it is stored as secrets in the namespace of the release. You can find them as follows:
kubectl get secrets -l "owner=helm" --all-namespaces
There will be a secret per release version/revision.
You can then the Helm release object with command similar to the following:
kubectl get secrets <secret_name> -n default -o jsonpath='{.data.release}' | base64 -d | base64 -d | gunzip
You could then update the Helm release object with the new Kubernetes objects and then replace the existing secret in the cluster.

Similar to suggestion 1, only that you do this through the Go SDK. The library is available from /pkg and pattern to use the library can be seen in /cli. Example of using v3 library is in 2to3 plugin

Check to see if any tools out there. helm-take-ownership was one such tool for Helm 2.

Thanks)


Improving the Ruby Feature.

This is a work-in-progress draft pull request demonstrating some improvements to the Ruby feature to address https://github.com/devcontainers/features/issues/603 and https://github.com/devcontainers/features/issues/757.
In the current state, the Ruby Feature now has parity with the Node Feature. The issue where multiple Ruby version managers have been installed has been solved. Currently, RVM is used. Although I prefer rbenv (and the poll seems to perhaps favor rbenv as well), I noticed the Node Feature uses NVM and doesn't provide an option for nodenv or any other Node version managers. Since Node is frequently used with Ruby for web development (Rails, Sinatra), it seemed to make since to make the first step resolving the key problem and making it mimic the Node Feature.

Copied from https://github.com/devcontainers/features/issues/757#issuecomment-1860734245, although I did make a few additional (mostly minor) edits to this comment:
@ThomasOwens Thank you so much for taking this on!  To summarize my current thoughts here:
As an incremental improvement, this is fine.  But if we are going to bump the version number for backward incompatibility, I personally vote for delaying until we can make rvm fully optional.  This is only a minor improvement over the current version: rbenv is unusable now, and (after this PR, in its current form) rbenv will still be unusable without more backward incompatible changes.
I personally consider the "minimum feature set" to be:

Don't set GEM_HOME or GEM_PATH at all.  They will screw up alternate version managers or other customization added by the user (either before or after this feature).  (Adding rvm or rbenv specific directories to PATH should be just fine.)
Support os-provided, at least for apt-based distributions.  Perhaps the code for this can be copied from other features.
Use the version of ruby that's already on the PATH, if the version number matches the requested version.  This allows the ruby feature to cooperate with the official docker hub ruby image or the user's own custom base image.  Perhaps the code for this can be copied from other features.
When the version of the available ruby does not match the requested version, install the requested version somehow that won't break (existing or future) standard installs of rvm or rbenv.  How it's installed is not part of my "minimum feature set", so long as my env vars constraint is met.  🙂

My recommendation on how to satisfy my "minimum feature set":

Do not install either rvm or rbenv (unless requested by the user by a config option).
Do install ruby-build to /usr/local, unless it's already installed on the PATH or (perhaps) disabled by a config option.  As far as I can tell, this is the only version management tool that is actively supported by the core teams of CRuby, JRuby, and TruffleRuby.  It works just fine without rbenv.  And, even if it is unused, it isn't large and installing it shouldn't hurt anything.
When necessary, install from source using ruby-build to either /opt/ruby or /usr/local.  Ensure the default PATH can find all of ruby's bins.

I'm not sure if it should be part of the "minimum feature set", but I think we should probably do one of the following, to aid with backward-incompatible upgrade pains:

Ensure that the user can use rvm, even with GEM_HOME and GEM_PATH unset.  Check the feature against at least a two base images: the default devcontainer base image, and the default docker hub ruby image.  I suspect rvm might work without those env vars on the default devcontainer images, but it might break if the user bases the feature on the docker hub ruby image (which also sets these env vars!!!).
Document that users may need to set these environment variables themselves to make everything play nicely with rvm.

None of the following ar.e in my personal minimum feature set... but they would all be nice to have:

If you make the install location user-configurable, document that variable as only applying to the ruby installed by ruby-build (i.e: not the os-provided version, nor to any version installed via rvm or rbenv).
If the OS provided version matches the requested version but isn't installed, install it (at least for apt distros).  The version detection code in other official features can probably be copied to accomplish this.
A config option to install a version manager: none or rvm or rbenv (or chruby).
Install via rvm when rvm is installed (by the feature, or already on PATH).
Install via rbenv install when rbenv is installed (by the feature, or already on PATH).
If we need (or want) any further customization to how our default installation of rubygems behaves, use their recommended 'operating_system.rb' and not environment variables.  Environment variables are great for overrides, but (IMO) they are unsuitable for defaults.

What do you think?  (My apologies: I'll probably be too busy with other end-of-year stuff both at work and in my personal time, and I probably can't really offer any more than my criticism right now.)


add staking contract for POS V2

Description
PoS V2 has new staking contract now.
Script need to consider this to get each address's accumulated rewards to verify network calculation.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] I have opened an issue and discussed the solution suggested in this PR
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Further comments
Andrey, NoamB, and I had discussion about this in slack and reviewed before PR.

add guardian's staked amount info with POS V2 staking contract


[WIP] Integration tests

This is currently a work in progress where I try to add the initial code for our integration tests. Expect things to change until this is ready.
To skip building of Cake before running tests
./tests/integration/build.ps1 -SkipBuildingCake
```

This require Cake to be present somewhere under `./build/`.

#### To run all tests

./tests/integration/build.ps1

#### To run a single test

./tests/integration/build.ps1 -Target "Cake.Common.IO.DirectoryAliases.CreateDirectory"

@patriksvensson this looks really nice! Very clean! I like it! :smile:


Maintenance: integrate utility with CI/CD merge process

Description of the feature request
Problem statement
Every time a PR is merged a number of checks and actions are run on the utilities to make sure that the changes introduced are compatible with the existing code (this is somewhat redundant as these same checks also run in the PR itself - but better be safe than sorry).
Additionally, the workflow that runs on merge also builds the documentation and API docs and publishes it.
Summary of the feature
This unit of work tracks the activities and changes needed for the new utility to be part of the merge workflow.
Code examples
N/A
Benefits for you and the wider AWS community
N/A
Describe alternatives you've considered
N/A
Additional context
N/A
Related issues, RFCs
#846

Linting & testing is already being run as part of the merge workflow. Likewise, now the docs have been merged the file is already picked up (although not present in the navbar). The API docs would already pick up the new utility, however due to this line, it's currently explicitly excluded.
Closing this issue as completed.


Async grpc upsert

[ch10321]

@benjaminran  can you review?

@mutayroei @benjaminran
Since all the info form the grpc error is condensed in the debug string, we can pass this to PineconeException and get a 'nicer sounding' error. However _InactiveRPCError would be visible in the trace. Would the ideal behavior be to raise PineconeError at an earlier step?


Disable Rails/Blank

My bad. Earlier I forgot to do one thing. Should be properly disabled now.

As I just pinged you, I am still getting Rails reports in my rubocop; do I need to do anything to reload the latest rubocop config?

I really need a concrete example. Please give me a link to repository in which this problem occurs (if in all of them, then just give me a link to any repository in which you have experienced that).
Also, are these incorrect reports from Hound CI or from Rubocop that you execute locally?

Rubocop I run locally

I bet it's due to incorrect .rubocop.yml. This is the example of correct file:
inherit_from:
  - https://raw.githubusercontent.com/riboseinc/oss-guides/master/ci/rubocop.yml

# local repo-specific modifications
# ...

(https://github.com/riboseinc/oss-guides/blob/0f6a733a403a63a3bb103e0b7c7f6b24207d1221/for_new_projects/ruby/.rubocop.yml)

I guess I need to think out something to make this setup easier to use and less error prone, but such improvement may require another changes in configuration in projects. Maybe Cimas will help to perform such mass update?

Yes it will. @CAMOBAP?
But I do have the correct file.
isodoc/.rubocop.yml:
# This project follows the Ribose OSS style guide.
# https://github.com/riboseinc/oss-guides
# All project-specific additions and overrides should be specified in this file.
inherit_from:
  - https://raw.githubusercontent.com/riboseinc/oss-guides/master/ci/rubocop.yml

# local repo-specific modifications

AllCops:
  DisplayCopNames: false
  StyleGuideCopsOnly: false
  TargetRubyVersion: 2.4
Rails:
  Enabled: true

And isodoc is still complaining about Rails errors like Rails/Blank.

Maybe Cimas will help with performing such mass update?

@skalee @opoudjis to be on the same page why we cannot add this to https://raw.githubusercontent.com/riboseinc/oss-guides/master/ci/rubocop.yml or create 'own' copy of rubocop.yml in https://github.com/metanorma/metanorma-build-scripts for example

@opoudjis sorry for late answer.
That file has
Rails:
  Enabled: true

This is why. Either change true to false or delete these two lines (Rails checks have been recently disabled in the style book, but that can be overridden like in shown example).

@CAMOBAP I'm not sure what do you mean, sorry.
Two files must be present in every repository: .hound.yml and .rubocop.yml. Regarding the latter, I would rather keep it as short as possible with the style book being inherited. Following content seems optimal for Metanorma:

https://github.com/lutaml/expressir/blob/0e4ee005800356f64127f693f8980bb7a542fece/.hound.yml
https://github.com/lutaml/expressir/blob/0e4ee005800356f64127f693f8980bb7a542fece/.rubocop.yml

Regarding making things simpler, I am seriously considering encapsulating the style book in a gem. This should be cleaner to use, simpler to install, and most importantly, we can easily set version constraints on Rubocop and its extensions.

@skalee for those ones it's possible, I will do this

@skalee @opoudjis @ronaldtse one more question in case if I will find some project-specific configuration, should I just drop it or move to some .local.rubocop.yml and include .rubocop.yml in this .local.rubocop.yml ?

Also related topics:
1
For now, we have two configuration repositories:
- https://github.com/metanorma/metanorma-build-scripts - metanomra org, and one repo from plurimath org
- https://github.com/relaton/support - relaton org
Should we create a separate repo for each organization?
As an alternative: we can use 'groups' feature of cimas config and list all repositories in a single place https://github.com/metanorma/metanorma-build-scripts/blob/master/cimas-config/cimas.yml#L850 I think this may be easier to maintain
2
It feels like cimas tool by itself still not friendly, so I would like to ask you if you have a chance to try it please share your feedback:

maybe documentation isn't clear outdated somewhere
general feedback on this tool also appreciated
in case of any problems, feel free to report them to https://github.com/metanorma/cimas/issues

@opoudjis sorry for late answer.
That file has
Rails:
  Enabled: true

This is why. Either change true to false or delete these two lines (Rails checks have been recently disabled in the style book, but that can be overridden like in shown example).

Thank you! Edited all my rubocops.

Rails/Blank thing has been resolved, so I'm going to close this one. Let's continue Cimas-related discussion in https://github.com/metanorma/metanorma-build-scripts/issues/64.


Bond Your token dialog just hangs with console errors

Describe the bug (required)
A clear and concise description of what the bug is.
I am trying to Bond my tokens.  I picked my transcoder, entered my amount, and hit 'submit'.  I now just hangs, with a Network changed error and MetaMask RPC Error.
Expected behavior (required)
A clear and concise description of what you expected to happen.
I expected the transaction to complete and my tokens to be bonded.
To Reproduce (required)
Steps to reproduce the behavior:
Stated above
Device (required):

Device: MacBook Pro
OS: Mac OS Sierra
Browser Chrome, Safari
Version 76.0.3809.100 (Official Build) (64-bit)

Screenshots (optional)
If applicable, add screenshots to help explain your problem.

Console logs (optional)
Paste related logs and/or error messages from the console here.
Network issues, such as the cURL output from failing requests, can also be pasted here.

Additional context (optional)
Add any other context about the problem here.

Hmm. Unfortunately, I'm not sure there's much we could do in this case — that's showing a failed network connection from your computer to the Ethereum RPC provider, Infura. Perhaps we should show an error instead of hanging in this case, but I don't think there's much we could do other than that.


Hotkeys for Mac (again :-)

Can you add hotkeys for mac ?
normally for mac user:
cut it's cmd+x
copy it's cmd+c
paste it's cmd+v
have you plan to implement undo , redo ?
undo: windows : ctrl+z / mac: cmd+z
redo: windows : ctrl+shift+z or ctrl+y / mac: cmd+shift+z or cmd+y

will fix ctrl <-> cmd button on mac
undo/redo already implemented but with only CTRL button

sure for ctrl+z and ctrl=y on mac version?
because it's doesn't  work for me.
I have a AZERTY keyboard

hmm, CTRL+z/y works on my virtual machine
in Code Editor and Sprite Editor
needs investigation...

I test the others Hotkeys, all I'ts ok except ctrl+z and ctrl+y.
I can't test ctrl+~  because ~ is alt+n for us. but I think its works for others keyboards versions

I added CMD button support. Could you pls test it?
https://1drv.ms/u/s!AlYcILUh3RFajMFesAOTcLJZl15q9Q
Thanks

Hi,
I test Cmd button it works with cmd+c, cmd+v ;-)
Undo Redo with cmd or ctrl doesn't work :-(

Made another fix for ctrl+c/v
Please check it
https://1drv.ms/u/s!AlYcILUh3RFajMFfgO4DM1qU2cwjmQ

no always not :/

And last one https://1drv.ms/u/s!AlYcILUh3RFajMFgP9a90DNYO7aHcA
Don't have ideas anymore :(

Unfortunately it doesn't work :-(
maybe your virtual machine distorts the results...

Could you just fo test put two buttons on the interface one for undo, one for redo to see if the cause is the hotkey or the system?

Added test undo/redo buttons to the toolbar https://1drv.ms/u/s!AlYcILUh3RFajMFhc_5YKkBPwSWAjQ

Hi,
It works very well with undo redo buttons
mmm...

maybe try test with a other combinaison hotkeys? I know isn't the best but just to try help you found the  bug

I created a test screen, just hold cmd+z/y buttons and tells me what codes do you see.
https://1drv.ms/u/s!AlYcILUh3RFajMFisTJBCCFcq4c0fw

I see ctrl: 224, z: 26 , y:28, cmd:227

When you press CMD+Z do you see both codes (227,26) at the same time or something else?

this is a screenshot,
when I press cmd+z

Please try this https://1drv.ms/u/s!AlYcILUh3RFajMFp85SbNl1N8X2rOg

So I test,
it doesn't work.
On the other hand, I have a bizarre behavior. if I use the undo button on the editor and make cmd+Y the redo function works but only the redo .

ok, what is going on if you press cmd+w instead cmd+z?

It works with CMD + W !!!!!!  :-D

Hum!  undo works with CMD + W :-)

gotcha :) will make a test soon

a test should work with AZERTY keyboard
https://1drv.ms/u/s!AlYcILUh3RFajMFqNczY9Z5mSn0N8w

Great !!!! It works!!
Now you need to handle the others cultures/localisation of keyboard.
But we see the end of tunnel.
You can close this issue  ;-)

fixed


can't work with bedrock models

Issue
i followed the manual , supplied all the aws credentials in various ways (exported them in the shell , added them to local .env file) but i keep getting 403 error:
litellm.exceptions.AuthenticationError: litellm.AuthenticationError: BedrockException Invalid Authentication - b'{"message":"The security token included in the request is invalid."}'
i am connecting via aws profile , supplied it also with the region in the parameters , but with no affect.
this did work for me :
aws bedrock-runtime converse --model-id anthropic.claude-3-5-sonnet-20240620-v1:0 --messages '[{         "role": "user",         "content": [{"text": "Create a list of 3 pop songs."}]     }]' --profile staging
Version and model info
aider 0.59.1

Thank you for filing this issue.
The error ->

The security token included in the request is invalid.

tells you that a token got received but it was invalid. Please double check your token and that you did not include additional characters like apostrophes or similar.

Then please double check if they do not include additional characters if you set those environment variables.
E.g. NOT THIS ->
export AWS_ACCESS_KEY_ID="super_secret"

but this ->
export AWS_ACCESS_KEY_ID=super_secret

checked and validated it is correct.

Then something else must be wrong.
AWS does not like the token it receives, see error.
This is known to work in other cases.
I am afraid I can't help you further.

You can try to debug this yourself:
Start aider with --verbose to see what it picks up from where.
Search/ask in the LiteLLM repository for help -> https://github.com/BerriAI/litellm

Have the same issue, I am sure everything is correct and I get the same error

I'm labeling this issue as stale because it has been open for 2 weeks with no activity. If there are no additional comments, it will be closed in 7 days.

I'm closing this issue because it has been stalled for 3 weeks with no activity. Feel free to add a comment here and we can re-open it. Or feel free to file a new issue at any time.


Switch to CMake

This PR switches the build process to CMake at it's the least bad build system I've seen so far. :wink:
@marqh - does this help with your build issues?

:+1: on cmake. Might make building on windows easier too...

Let's start with this, and refine if we find issues. :+1:


DSKaKhel: longestPalindrome.cpp

This algorithm is used to calculate the longest palindromic substring in a given string

Language used: C++
Algorithm: String
username: suraj0223
suggestions: Basic2Boss

Please send this link of PR here


Update PostgreSQL from 13.4 to 13.5

This resolves 2 CVEs and a number of bugs
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/3685

Builds currently broken due to the licensing issue. This should be good to go once that is green.

Passes adhoc with a rebase


No config file produced

Just downloaded rangedpumps-0.7.1 for 1.15 and no config file was produced when I ran the game.  I'm not playing with any RF so I can't get the pump to work and can't find a config to change it.  I'm looking for rangedpumps.cfg in the config folder

It's a toml file in the serverconfig folder.


Feature/atx 10 adhere to best practices

Coverage increased (+4.9%) to 61.111% when pulling e572e0e98dbc61b2283e1aba78950904698a44e6 on feature/atx-10-adhere-to-best-practices into 7851ba98cc3f539d9a0fd66b24063946e77d7a85 on development.

Coverage increased (+4.9%) to 61.111% when pulling a1515ca625353ac34a856c5888c8e1035851cbb1 on feature/atx-10-adhere-to-best-practices into 7851ba98cc3f539d9a0fd66b24063946e77d7a85 on development.


Support IO priority

https://github.com/opencontainers/runc/pull/3783
https://github.com/opencontainers/runtime-spec/pull/1191
Also we need to update oci-spec-rs.

@lengrongfu Sure. I'm looking forward to reviewing your PR.


⚠ Hartman Foundation has degraded performance

In 255d74e, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2449 ms

Resolved: Hartman Foundation performance has improved in fc73a16.


API: constrain measure names to be printable and 255 chars or less

The Java API constrains measure names to be printable and <= 255 chars. This isn't part of the spec, but perhaps it is supposed to be?

I noticed similar restrictions in the Java Resource implementation. I would say we should only add them if they are intended to be part of the spec.

Is this worthy of the question label? I added it because I think it probably is.


Horizontal scrolling

Would be nice to have either of the following:

horizontal scrolling with trackpad/mouse without having to click the left/right buttons
an option to make the boxes are stack vertically instead

The first request is a duplicate of #86
I'll try to find a way to handle the horizontal scroll.
Can you create another new issue for your second request with more details?

I'm ok with just the first option


when my app goes idle (put phone to sleep) and i turn it back on, the websocket doesn't reconnect

I just started using this library. I put the websocket connection in a provider. However, I put my phone to sleep and open it back up, the websocket doesn't attempt to reconnect.

Any solution?

I just started using this library. I put the websocket connection in a provider. However, I put my phone to sleep and open it back up, the websocket doesn't attempt to reconnect.

did you manage to find a solution to it?


UI | Table Detail pane on the Lineage tab doesn't show the column count

Affected module
Does it impact the UI, backend or Ingestion Framework?
UI
Describe the bug
A clear and concise description of what the bug is.
Table Detail pane on the Lineage tab doesn't show the column count
To Reproduce

Click on node in the Lineage tab
Check for column count on the Table Detail pane on the right

Screenshots or steps to reproduce

Expected behavior
A clear and concise description of what you expected to happen.
Correctly show the column count
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]: 0.11.5
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

FYI - I did not enable Profiler yet on this specific table, so I guess row count being not available is OK.

@sidhreddy both row count and column count are picked up by profiler

@harshach isn't the columns here mean the column count? Isn't that part of metadata extraction?

we can add it from column count, but the idea was to display the metrics from profiling. But yes we can use column count from the metadata

Got it. Would be able to make it consistent with the Table Home page? And also add the Schema name?


Adjusting local auditor

Draft

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (847a40d) 87.64% compared to head (89fd9f5) 87.65%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #380   +/-   ##
=======================================
  Coverage   87.64%   87.65%
=======================================
  Files          41       41
  Lines        8570     8583   +13
=======================================
+ Hits         7511     7523   +12
- Misses       1059     1060    +1

Impacted Files
Coverage Δ

akd/src/auditor.rs
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update Redis service plans

This is a cost neutral change (micro-ha costs the same as tiny)

Change cache Redis plan from tiny to small-ha (for more caching
space and redundancy in case of PaaS failures)
Change queue Redis plan from small-ha to micro-ha (as this is now
only used as a queue and will require significantly less memory) and
upgrade to Redis 5
Change eviction policy for queue instance to noeviction (as it
should now not randomly fill up, and if it does, it's because of a
serious error)

Before Deploying

[x] Clear database 1 on queue Redis instance (which is now no longer used and has stale cache data in it blocking a downgrade)

Review app deployed to https://teaching-vacancies-review-pr-2756.london.cloudapps.digital

Turns out this won't work 😞
New, faffier PR inbound shortly.


dbt seed truncate tables

resolves #112
Description

dbt seed command fixed with expected behavior from dbt global project to truncate table in order remove all rows from the existing seed tables and replace values. As explained in issue 112, the current seed command in dbt-spark appends to existing seeded tables instead overwriting.

Checklist

[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

hello @drewbanin, I signed the CLA with my username mv1742 and email but the check still fails. Any ideas?

Let's see if we can push that over the line.


Add the 'sprite' collision ref docs

[x] Collision api docs.

Hmm, obstacles have changed overnight...don't merge yet.

Close this one due to newer api's.


add Py2 as an internal API for optimization and dogfooding

This PR adds the Py2 struct and PyAnyMethods trait from #3361 as private APIs which we can use internally.
The idea is that:

we can still benefit from the performance advantages of not touching the pool where possible internally
this lets us start learning about this API so that we can completely satisfy ourselves it's the right API for users
it lets us start making incremental progress with migrating to the new API in reviewable steps

If there are no immediate concerns, I would be keen to merge this now I've pushed the coverage up. I think it'll be useful to keep anything else we build optimal.

If there are no immediate concerns, I would be keen to merge this now I've pushed the coverage up. I think it'll be useful to keep anything else we build optimal.

No concerns, but also did not find any time to actually look at this specific PR.

Would you like me to hold until you've read it? I can also amend post merge if you're happy with that route.

Is it really pressing, i.e. will this be part of 0.20? I suspect that this is basically what we discussed elsewhere quite in depth, so I am not particularly anxious. But I also do not really see a reason we would need to rush this?

No urgency other than that I'm enthusiastic to make stuff go faster 😂 There's no API changes here though, so this can wait until 0.20.1 if needed. For 0.20 I'm just looking at various recent bug reports to understand if any need breaking changes to fix, and otherwise I'm hoping we can release within maybe a week.

Revisiting this now that 0.20 is released, I benchmarked this branch against main (raw numbers plus + diff):

          identify_object_type: [195.49 ns 196.00 ns 196.52 ns]
                                [-0.3530% +0.2300% +0.7163%] (p = 0.43 > 0.05)

      collect_generic_iterator: [7.5128 ms 7.5424 ms 7.5733 ms]
                                [-3.0232% -2.3853% -1.7609%] (p = 0.00 < 0.05)

                        call_0: [20.362 µs 20.388 µs 20.422 µs]
                                [+0.2598% +0.4492% +0.6769%] (p = 0.00 < 0.05)

                 call_method_0: [70.852 µs 70.907 µs 70.971 µs]
                                [-2.5288% -2.3959% -2.2817%] (p = 0.00 < 0.05)

        ordered_dunder_methods: [36.186 ns 36.200 ns 36.213 ns]
                                [-22.930% -22.817% -22.713%] (p = 0.00 < 0.05)

               ordered_richcmp: [36.631 ns 36.649 ns 36.666 ns]
                                [-22.134% -22.049% -21.959%] (p = 0.00 < 0.05)

     err_new_restore_and_fetch: [111.64 ns 111.73 ns 111.82 ns]
                                [-0.3089% -0.0347% +0.2335%] (p = 0.80 > 0.05)

           err_new_without_gil: [16.658 ns 16.760 ns 16.856 ns]
                                [-4.6907% -4.0453% -3.3965%] (p = 0.00 < 0.05)

   extract_str_extract_success: [3.7456 ns 3.7469 ns 3.7482 ns]
                                [-0.5158% -0.3141% -0.1338%] (p = 0.00 < 0.05)

      extract_str_extract_fail: [25.260 ns 25.321 ns 25.381 ns]
                                [-1.2420% -0.9741% -0.6840%] (p = 0.00 < 0.05)

  extract_str_downcast_success: [3.1552 ns 3.1602 ns 3.1665 ns]
                                [-0.2737% -0.0839% +0.0852%] (p = 0.37 > 0.05)

     extract_str_downcast_fail: [2.2645 ns 2.2678 ns 2.2717 ns]
                                [+0.0363% +0.1339% +0.2700%] (p = 0.02 < 0.05)

   extract_int_extract_success: [4.4889 ns 4.4962 ns 4.5040 ns]
                                [-0.0016% +0.1069% +0.2132%] (p = 0.04 < 0.05)

      extract_int_extract_fail: [110.09 ns 110.16 ns 110.23 ns]
                                [-0.8899% -0.7757% -0.6548%] (p = 0.00 < 0.05)

  extract_int_downcast_success: [4.6468 ns 4.6485 ns 4.6502 ns]
                                [-0.2473% -0.1213% -0.0161%] (p = 0.04 < 0.05)

     extract_int_downcast_fail: [2.2598 ns 2.2608 ns 2.2617 ns]
                                [+1.1040% +1.1620% +1.2187%] (p = 0.00 < 0.05)

 extract_float_extract_success: [3.5183 ns 3.5197 ns 3.5210 ns]
                                [-0.0716% -0.0015% +0.0669%] (p = 0.97 > 0.05)

    extract_float_extract_fail: [94.550 ns 94.627 ns 94.709 ns]
                                [-0.4231% -0.2887% -0.1431%] (p = 0.00 < 0.05)

extract_float_downcast_success: [3.5179 ns 3.5194 ns 3.5210 ns]
                                [-0.0914% -0.0166% +0.0618%] (p = 0.68 > 0.05)

   extract_float_downcast_fail: [2.2483 ns 2.2502 ns 2.2523 ns]
                                [-0.0554% +0.0402% +0.1338%] (p = 0.43 > 0.05)

            enum_from_pyobject: [570.63 ns 570.99 ns 571.36 ns]
                                [-0.2673% -0.1184% +0.0376%] (p = 0.14 > 0.05)

             list_via_downcast: [373.75 ps 373.89 ps 374.03 ps]
                                [-0.1157% -0.0436% +0.0332%] (p = 0.26 > 0.05)

              list_via_extract: [1.2798 ns 1.2802 ns 1.2807 ns]
                                [-0.0320% +0.0330% +0.0964%] (p = 0.31 > 0.05)

       not_a_list_via_downcast: [373.79 ps 373.95 ps 374.12 ps]
                                [-0.0807% +0.0020% +0.0825%] (p = 0.96 > 0.05)

        not_a_list_via_extract: [22.195 ns 22.243 ns 22.291 ns]
                                [-0.8011% -0.6032% -0.4014%] (p = 0.00 < 0.05)

   not_a_list_via_extract_enum: [332.53 ns 332.72 ns 332.92 ns]
                                [-1.8772% -1.6851% -1.4651%] (p = 0.00 < 0.05)

             f64_from_pyobject: [3.5184 ns 3.5195 ns 3.5207 ns]
                                [-0.1180% -0.0424% +0.0303%] (p = 0.27 > 0.05)

             clean_gilpool_new: [13.690 ns 13.696 ns 13.702 ns]
                                [-0.0077% +0.0706% +0.1515%] (p = 0.08 > 0.05)

             clean_acquire_gil: [53.250 ns 53.270 ns 53.290 ns]
                                [-0.0817% -0.0104% +0.0584%] (p = 0.78 > 0.05)

             dirty_acquire_gil: [58.179 ns 58.256 ns 58.349 ns]
                                [-0.0564% +0.1029% +0.2864%] (p = 0.23 > 0.05)

                getattr_direct: [70.388 ns 70.437 ns 70.487 ns]
                                [-7.6346% -7.3034% -6.9781%] (p = 0.00 < 0.05)

                getattr_intern: [16.056 ns 16.072 ns 16.088 ns]
                                [-10.832% -10.727% -10.626%] (p = 0.00 < 0.05)

                     iter_list: [903.68 µs 906.28 µs 908.90 µs]
                                [-0.1725% +0.1213% +0.4078%] (p = 0.42 > 0.05)

                      list_new: [438.67 µs 439.40 µs 440.24 µs]
                                [-0.8039% -0.2354% +0.2559%] (p = 0.41 > 0.05)

                 list_get_item: [512.93 µs 513.48 µs 514.07 µs]
                                [-3.7039% -3.5536% -3.4087%] (p = 0.00 < 0.05)

       list_get_item_unchecked: [451.73 µs 452.50 µs 453.32 µs]
                                [-0.3969% -0.2051% -0.0033%] (p = 0.05 < 0.05)

            sequence_from_list: [1.2798 ns 1.2802 ns 1.2807 ns]
                                [-0.2589% -0.1713% -0.0898%] (p = 0.00 < 0.05)

               first_time_init: [1.6841 µs 1.6859 µs 1.6878 µs]
                                [-0.8102% -0.4418% -0.1040%] (p = 0.01 < 0.05)

             drop_many_objects: [4.7113 µs 4.7126 µs 4.7139 µs]
                                [-0.7234% -0.4719% -0.2567%] (p = 0.00 < 0.05)

                    iter_tuple: [495.81 µs 496.14 µs 496.49 µs]
                                [-0.9445% -0.6637% -0.4283%] (p = 0.00 < 0.05)

                     tuple_new: [434.86 µs 435.16 µs 435.45 µs]
                                [-1.0320% -0.3762% +0.2590%] (p = 0.27 > 0.05)

                tuple_get_item: [354.50 µs 354.81 µs 355.19 µs]
                                [-0.0440% +0.0855% +0.2054%] (p = 0.18 > 0.05)

      tuple_get_item_unchecked: [289.84 µs 290.88 µs 292.63 µs]
                                [+0.0291% +0.2019% +0.5000%] (p = 0.09 > 0.05)

           sequence_from_tuple: [1.2791 ns 1.2796 ns 1.2801 ns]
                                [-0.0385% +0.0173% +0.0750%] (p = 0.55 > 0.05)

                tuple_new_list: [166.36 µs 166.47 µs 166.57 µs]
                                [-0.0515% +0.0184% +0.0907%] (p = 0.62 > 0.05)

                 tuple_to_list: [120.33 µs 120.80 µs 121.10 µs]
                                [-0.9240% -0.5622% -0.2445%] (p = 0.00 < 0.05)

                 tuple_into_py: [41.072 ns 41.093 ns 41.117 ns]
                                [-0.7140% -0.6204% -0.5210%] (p = 0.00 < 0.05)

I took a look into a couple of the bigger wins, and here's my explanation for them:

ordered_dunder_methods / ordered_richcmp - this -22% change is because PyAny::gt no longer needs to put the intermediate value from PyAnyMethods::rich_compare onto the GILPool, plus we use Py2<'_, PyAny> instead of PyObject instead of the fn inner, which is a little faster to drop because it doesn't need to check if the GIL is acquired.
getattr_direct / getattr_intern - I think this is mostly accounted for by being able to use Py2<'_, PyString> instead of Py<PyString> as the container for the attribute name, which again is faster to drop for the same reason.

Overall, it looks like this PR is managing to already make a slight improvement to benchmarks. I'd really love to merge this and start submitting follow-up PRs to rework the rest of PyO3's types (internally) over the next patch release cycle. @adamreichold perhaps if you're still short on time at the moment, you'll forgive me if I merge this and get on with optimising? 🙏

(force-pushed to rebase)

An approval is an approval, thanks! I can always submit follow-up PRs if @adamreichold wants to see changes later. I expect there will be a fair bit we'll learn by making these internals changes.
🚀

Now that I've read all the discussion again I think it's best to call it GIL (or Gil) once we release it.

Now that I've read all the discussion again my preference is to call it GIL (or Gil) once we release it.

The only reservation I have about the GIL name is that it doesn't make sense in a nogil world, so we have to break users again with a big rename. I sorta hope that we can pick a name which is good for the future, though maybe I am getting ahead of myself.

An approval is an approval, thanks! I can always submit follow-up PRs if @adamreichold wants to see changes later. I expect there will be a fair bit we'll learn by making these internals changes.

As you probably noticed, I am currently very much swamped by $DAYJOB and while I try to keep reading and take care of that little NumPy thing, I am rather happy if things are not blocked by me.
And yes, a review is a review is a review.


Modify Swagger docs

Wishlists:
Create; List; Update; Read; Delete
Products:
Create; Update; Read; Delete; Action "Delete All"; Action "Move"
Model names:
Create_Wishlist_Model
Rename_Wishlist_Model
Read_Wishlist_Model
Create_Product_Model
Update_Product_Model
Read_Product_Model
List_Wishlist/Product_Model

Codecov Report

Merging #137 (247c8f4) into main (943f9ea) will increase coverage by 1.76%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   95.46%   97.22%   +1.76%
==========================================
  Files           7        7
  Lines         595      577      -18
==========================================
- Hits          568      561       -7
+ Misses         27       16      -11

Impacted Files
Coverage Δ

service/routes.py
97.76% <100.00%> (-0.44%)
:arrow_down:

service/error_handlers.py
92.85% <0.00%> (+31.09%)
: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 943f9ea...247c8f4. Read the comment docs.


Mermaid Diagrams: theme and text size refactoring

          Theme is the colors on the diagrams, the default is on gh, we have something much nicer rendering IMHO on our site:

I think it's great as is for a start! We can do a latter PR getting mermaid styled and sized really nice.
Originally posted by @nuke-web3 in https://github.com/risc0/risc0/issues/1959#issuecomment-2183768672

These docusaurus docs may shed some light on this: https://docusaurus.io/docs/next/markdown-features/diagrams#configuration

@pdg744 could you clarify what you want for the mermaid graphs? I'm not sure I understand the issue

@nahoc great to see the mermaid themes improved in #2190. do you have any thoughts on how to improve the readability of the diagrams on these pages? when rendered via github, mermaid diagrams have a nice zoom option, but that doesn't show up on the dev docs site.
https://dev.risczero.com/api/getting-started
https://dev.risczero.com/api/recursion

@nahoc I hear you tried getting something like GH has (example) for all it's mermaid diagrams setup but https://docusaurus.io/ is a blocker - can you elaborate a bit why? I would really like to use mermaid with a better rendering wrapper vs. refactoring it out of our docs.
I don't expect this to be a critical thing have, as we have limited use for them, but they are fantastic at flowcharts at least. We do need other diagram tools anyway to supplement where it's not a good tool. So if it's too much of a PITA, we can decide to refactor them out 😢


Enable display of 'ghost' buildings

Enable so called 'ghost' buildings (i.e. a transparent image of a building which appears on build order execution).

Done. Now placeholders can be retrieved using GetUnits call. However, OnUnitCreated events are not reliably issued.


Error setting final_id in fetch_segmented_data

final_id = (end_block_id if (end_block_id is not None) else 0x7fffffff)
end_block_id will never be None

Bug fixed


region worker may be the bottleneck if S3 latency is high

Should make it parallel like the way https://github.com/ngaut/unistore/pull/616 does.

Fixed by https://github.com/ngaut/unistore/pull/620


nordland download url is out of date

hello, i found that the url is out of date in download_nordland.py
and i can't find the cleanImageNames.txt, could you fix this problem? thanks!!
util1.download_heavy_file("https://cloudstor.aarnet.edu.au/plus/s/8L7loyTZjK0FsWT/download?path=%2F&files=summer.tar.gz",
join(raw_data_folder, "summer.tar.gz"))
util1.download_heavy_file("https://cloudstor.aarnet.edu.au/plus/s/8L7loyTZjK0FsWT/download?path=%2F&files=winter.tar.gz",
join(raw_data_folder, "winter.tar.gz"))
util1.download_heavy_file("https://cloudstor.aarnet.edu.au/plus/s/8L7loyTZjK0FsWT/download?path=%2F&files=cleanImageNames.txt&downloadStartSecret=crd03ou9qji",
join(raw_data_folder, "cleanImageNames.txt"))

Hi, you can download the Nordland dataset from here.
I haven't been able to automatize the download and format due to an issue of gdown when downloading large files, so you need to download summer and winter splits of Nordland from browser and then adapt the script. I will fix the script once the issue is fixed on gdown

but still I can't find where to get cleanImageNames.txt
:(

oh, may the files can be download from here

If you are looking for that CloudStor hosted version of Nordland, it's here now. For download commands, please see this.

Hi @oravus, thank you for your help. I tried to fix the download script using the links you provided, but the winter.tar.gz seems corrupted. I tried multiple times to download it and extract with multiple libraries, but I'm getting an error and only 8970 out of 35768 are downloaded. For example, when extracting with tar -xvzf  winter.tar.gz I get this error (after thousands of images are correctly extracted):
winter/images-34312.png
winter/images-34373.png
winter/images-34381.png
tar: Skipping to next header

gzip: stdin: invalid compressed data--format violated
tar: Child returned status 1
tar: Error is not recoverable: exiting now

The summer tar is fine. Could you have a look at the uploaded winter tar?

I think I have encountered this extraction issue before. Probably ideal to reupload the winter tar, will message here once that's done.

@gmberton, Winter (and Spring) tars are now updated, thanks to @Ahmedest61.
We have tested it at our end for download/extraction. It would be great if you can also test your script.
Also, to test if the downloaded file is correct, you can run
sha256sum winter.tar.gz
It should return c95a1cf99ef2cdcf9e1d9b47de1d8248a194a6c8deeb976edd099562ab20e1c2

Thank you @oravus , the links work well now :)


Add GNOME 40 support

Metadata has 40.0 but I think it's just 40.

SaucySauce, I'm not sure.
I've tried to use 40 and it doesn't work on my fedora.
I guess I have to create for all versions (40, 40.0, 40.0.0).
I'll do it soon.

@SaucySauce , I've updated extension. Check it out, please.

MR #12


M2 Mac Still doesn't start app automatically

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Looked through the wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the bug
Since I upgraded to M2, even with updated OSX and Monitor Control app, I cannot get the app to start automatically after a restart or power on. This is even though Monitor Control app is set to start on start up (see screenshots below). I also provided a shallow overview of my Mac Studio specs below. What can be done to resolve this? I really like the app but it is a pain to manually open it every time.

Steps to reproduce
I cannot get the app to start automatically after a restart or power on. This is even though Monitor Control app is set to start on start up
Expected behavior
That I am after startup, able to change the brightness settings with my keyboard f1 and f2 keys without manually starting the app
Anything else?
No response
Environment Information (please complete the following information)
- macOS version: Sonoma 14.2.1
- Mac model: M2 Mac Studio
- MonitorControl version:4.2.0
- Monitor(s): 2. BenQ GL2450H and Bell U2723QE
- Apple Silicon/M1 (yes or no): Yes

I have an almost identical issue on an M2 machine: difference for me is that a volume with MonitorControl is mounted on login instead of actually launching the application.
When I Right Click > Show In Finder it also mounts it instead of showing me the actual application. It's the only item that does that, all others show me the app within the application folder. Looks to be app specific.
@waydabber any thoughts?

Tagging @waydabber, @JoniVR, and @the0neyouseek as I see you as members of MonitorControl in the contributors section: thanks for all your contributions, this tool has been a great help in my day-to-day.
I see more recent issues that were opened without much engagement, plus a PR from 2023 that's still awaiting review. Is this project still being maintained?


Remove duplicate entries from EBINS during install-beam target

Hi,
as mentioned on #342 make install fails on my gentoo system because make install tries to install lfe_scan.beam twice.
During the whole compilation magic src/lfe_scan.xrl will get turned into src/lfe_scan.erl . As EBINS is essentially a list of al .erl, .xrl and .yrl files with the endings replaced by .beam this results in the problem that EBINS will contain lfe_scan.beam twice as this name is generated from both the .xrl and the .erl file. As this problem seems to affect the install target I tried not to mess with anything else (I don't know a lot about make).
This fix is kinda ugly but it resolves the issue for me. Not sure if this will work well on other systems but it might be worth a shot.

Is there a way we could use $^ rather than shelling out?

Due to the implicit rules it looks like no. It also seems like if we simply remove the xrl part from EBINS then all with be well, maybe. According to a quick search on my phone, we don't have any .yrl files either, so maybe we could remove or comment out the related parts of the Makefile for clarity.

Removing the .xrl part results in lfe_scan.beam not being generated on my system, which is why I tried this solution.

Ah ok.

I'm going to take a look at this, but I'll probably close this PR, cherry pick the commits, and work on the issue in a new branch (i.e., any commits made so far will get full credit 😉).

I've cherry-picked the original commit for this into a new PR on the LFE 2.0 branch).
Closing ....


WIP: Prepare Python 3.11

Blocked until HA-core/package_constraints.txt is bumped to aiohttp-3.8.3

Blocked until HA-core/package_constraints.txt is bumped to aiohttp-3.8.3

That won't happen aiohttp 3.8.3 has a serious memory leak, we have reverted that upgrade and are awaiting upstream. One thing is for sure, it will never ben 3.8.3 again :)

Noted, thanks for the explanation - waiting for upstream bump of aiohttp above 3.8.1 - adjusted the initial comment to say > 3.8.1 :)

@brefra (or @bouwew) could you look at plugwise/connections/socket.py I think pylint is rightfully stating the no-raise-else and it suggest to just de-indent (which I now did)
@bouwew why did we have HomeAssistantError in pyproject (otherwise we'll need to re-add it prepended by the module, just like with the built-ins)

For record keeping, following up with aiohttp 3.8.4 as now in core (though core is also still waiting for 3.9 release)

@CoMPaTech, HomeAssistantError, must be a left-over or a mistake.
Not used anywhere.

As discussed on the side, not defaulting to 3.11 yet


Deployment workflow enhancement

The deployment workflow currently uses two sets of branches:

For the project pages: master branch for the source code and gh-pages branch for the generated website.
For personal or organization webpages: source branch for the source code and gh-pages branch for the generated website.

Describe the solution you'd like
The above-mentioned behavior can be unified. For both use cases, master can be used for the source code and gh-pages can be used for deployment.
Describe alternatives you've considered
NA
Additional context
NA

I have raised a PR for this - https://github.com/alshedivat/al-folio/pull/343. I am still unsure about the manual deployments. Happy to hear suggestions.

I have raised a PR for this - https://github.com/alshedivat/al-folio/pull/344.

Hi! I was following the previous instruction and source branch, and after sync with upstream, github actions are no longer triggered. What should I do to fix it?

@ivanpuhachov Can you share your repo link?

Fixed it.

.github/workflows/deploy add source to the branches:

on:
  push:
    branches:
      - master
      - source

On Github - repo - Settings - Pages select gh-pages as a source branch – as it is written in current readme file. In old instructions it was master

Yes, this is one way. But the idea was to have source code in the master branch. So you could also do the following:

Delete the master branch.
Rename source to master.
Push all the future changes to master. The generated site will be pushed to gh-pages which should be selected as the source branch in the publishing source.

Got it, thank you!


Page load time with S3 is excessive

First of all: great plugin, really nice of you to dedicate your time on this!
I was wondering if I'm doing something wrong because my page load times are really high when using Media Manager with S3. I think it's because you store the thumbs in the media disk (also S3) and check for the thumbs of all the images on every request. If they're there, fine, if not it means another roundtrip to S3 for every image to do the resize.
If that's right, I get why you do it but would it be possible to build a sort of cache option for "advanced users"?
I was thinking, if you register (in local disk cache) which image have already been resized, you could serve the link to the image immediately if found; if not, do the resize thing, write that you've done so to local cache and serve the link.
If users change the image, there could be a button in the plugin to invalidate the local cache, which would resize every image again (on request) because it's not found in local cache. Cache warming would be another possibility but not essential.

Hi Frank, I will have to look a bit deeper into this. Yes, the plugin checks if the thumbnail exists, and creates it if it doesn't. If in your case it is slow EVERY time, that would mean that the checking itself is slow... did I understand that right?

Maybe not for one thumbnail but, for instance, I have a carrousel with 20 images. For each of these images it will do (at least) one roundtrip to S3 so even if it takes say 50ms, that will come to about 1 second in the best case (aka all images are already there).
If the script could check locally if a thumbnail exists (through a registry), it wouldn't need all those roundtrips to S3.

Unfortunately this is still a massive issue and makes the plugin unusable. Having 15 images on a page, already loaded once before so the thumbs are created still causes a very long page load. 30 seconds or more.


Issue with multiline object assignment as a static class property

I encountered an issue while trying to define a multiline object as a static method of a class. Here's an example:
class Example

    @STATIC_OBJECT = {
                       'a':1,
                       'b':2,
                       'c':3,
                       'd':4
                     }

    constructor: () ->
        alert('This is working!')

becomes:
(function() {
  var Example;
  Example = (function() {
    function Example() {}
    Example.STATIC_OBJECT = {
      'a': 1,
      'b': 2,
      'c': 3,
      'd': 4
    };
    return Example;
  })();
  ({
    constructor: function() {
      return alert('This is working!');
    }
  });
}).call(this);

Notice the premature return Example; and then the unassociated block that holds the constructor (and any other class members if they existed).
However, if I format the definition as follows:
class Example

    @STATIC_OBJECT = {
                       'a':1,
                       'b':2,
                       'c':3,
                       'd':4 }

    constructor: () ->
        alert('This is working!')

the resulting Javascript is correct:
(function() {
  var Example;
  Example = (function() {
    Example.STATIC_OBJECT = {
      'a': 1,
      'b': 2,
      'c': 3,
      'd': 4
    };
    function Example() {
      alert('This is working!');
    }
    return Example;
  })();
}).call(this);

The only next line indentation for the close bracket that seems to work is:
class Example

    @STATIC_OBJECT = {
                       'a':1,
                       'b':2,
                       'c':3,
                       'd':4
    }

    constructor: () ->
        alert('This is working!')

which results in the correct:
(function() {
  var Example;
  Example = (function() {
    Example.STATIC_OBJECT = {
      'a': 1,
      'b': 2,
      'c': 3,
      'd': 4
    };
    function Example() {
      alert('This is working!');
    }
    return Example;
  })();
}).call(this);

This could just be a strictness of the whitespace interpreter in Coffeescript. I thought it worth mentioning, though, in case it wasn't intentional given that any of the above would work in Python.

Non-technical question: Why do you want the members of your object indented a billion times? Don't you think that's just "asking for it" in a significant-whitespace language?
Back to work:
The problem you're having is that you're dedenting twice. You probably didn't know that or you would also have picked up on this working example:
class Example

    @STATIC_OBJECT = {
                       'a':1,
                       'b':2,
                       'c':3,
                       'd':4
                       }

    constructor: () ->
        alert('This is working!')

Did you see what the problem was? You were dedenting to the } and then dedenting once again to close the class. I don't mean to say this is definitely not a problem, but I'd like to see a more convincing example than this one. I'd call that syntax "discouraged". Where'd you learn to format like that? It's (subjectively, of course) disgusting. #1275 will disallow this style entirely, so I'm in favour of closing this.

Yeah, I see now the dedenting issue. I wasn't aware of the looser indenting in Coffeescript interpreter compared to Python and and #1275 would certainly help clear that up.
As for the subjective response, I wouldn't use this much whitespace in something short but this was a quick example. In the code that actually led me to this issue the entries were much longer and would not fit on a single line. As for the style guide it's an in accordance to and in extension of the PEP8 and Google's Python Style Guides for indentation of multi-member, multi-line definitions. It follows the spirit of the EMACS hanging indent and is the default formatting for some popular Python IDEs such as Wing.
On the more objective side, this whitespace behavior deviates from Python interpreters, which have "implicit line joining inside parentheses, brackets and braces." Python interpreters ignore whitespace within object and array definition and within parentheses allowing much more flexible style formatting.
That leaves me with the question, is this a case of the whitespace being too significant? Does it serve some valuable purpose that I'm unaware of (being relatively new to Coffeescript)?
If so, I'd be curious what it is and how it would be incompatible with the Python behavior.
If not, then I'm not really sure why the whitespace should be significant in such cases.
Unless the Coffeescript community is trying to use the language/interpreter to impose adherence to a strict style, which personally (back to the subjective) I'd find somewhat off-putting.

My understanding is that significant indentation allows the following:
object =
  foo: 42
  bar: ['yep', 'nope', 'maybe']
  baz: -> no

(Squiggly brackets optional.)

Yep, quite right David, and without braces it makes good sense for the indentation whitespace to be significant as the interpreter in that case has only indentation to go on. And given the possibility of additional definitions within the object, as you've done with baz, maybe there's an excess complication with ignoring whitespace if the braces (or brackets or parentheses) were present as part of the definition.
However, if the braces were applied the following seems possible:
object = { # Begin ignoring indentation whitespace
  foo: 42
  bar: ['yep', 'nope', 'maybe']
  baz: ->
    # Stop ignoring indentation within this definition
    no
  qux: 12 # Back to ignoring whitespace again until either another significant entry or until the closing braces is found.
} # Since whitespace doesn't matter this brace can appear at any indentation

It could be tricky to interpret without a qux, because it would go directly from significant whitespace to the close brace, but if the interpreter was expecting to encounter a close brace given the previous opening it could consume the brace at any indentation as the end of the definition block.

@sernst: that compiles just fine. I'm not sure what point you're trying to make.

The culprit is that how exactly CoffeeScript "significant whitespace" works is never detailed anywhere--not on the official site nor the books. You have to dig into the sources, issues, and/or Stack Overflow questions.

@satyr: I completely agree, that is a problem.

Well, I'd agree that the lack of documentation on exactly how whitespace works is a problem. I was relying on my understanding of whitespace from Python to inform my coding when I encountered the issue that started the ticket instead of working from an understanding of the behavior in Coffeescript.
However, @michaelficarra, my point is that a block definition with significant whitespace is over-constrained. Why should both braces/brackets and whitespace matter at the same time? Yes, the example above does compile, but only because I correctly used whitespace while inside the braces. There's nothing I can see that is syntactically significant in having whitespace matter within an explicitly-defined block. I was pointing out earlier that Python ignores whitespace within braces/brackets because the braces/brackets provide the necessary block definition themselves.
This is an analog of solving a linear system of equations with two unknowns using three equations. You can throw out one of the equations and still solve the problem. Unless there's a good reason for imposing whitespace within a defined code block then you should be able to throw that out as well and interpretation of the code still be possible and in doing so provide greater style and formatting flexibility to developers.
Personal style preferences aside, why shouldn't this work:
object = {foo:42, bar:['yep', 'nope', 'maybe'],
          baz:'always', qux:12}

or this work:
object = {
  grault: 'which is another used style convention'
     foo: 42,
     bar: ['yep', 'nope', 'maybe'],
    quux: 'delimiter alignment'
}

or even this work:
object = { foo:42,
           bar:['yep', 'nope', 'maybe'],
             baz:'always',
             qux:12,
           quux:24 }

In Python you can split definitions across multiple lines in explicit blocks (of all types) and indentation is insignificant because it's not necessary for the definition in the presence of a well-defined, explicit block.
It comes down to the question: what is a good reason for Coffeescript to enforce significant whitespace inside an explicit block definition?

@sernst: you make a good point.
@jashkenas:

what is a good reason for Coffeescript to enforce significant whitespace inside an explicit [object] definition?

Probably because you can have implicit objects inside of explicit ones. Like so.
obj = {
  a:
    b: c
  b: [
    {c: d}
    e: f
  ]
}

The original issue is fixed in CS2:
var Example;

Example = (function() {
  class Example {
    constructor() {
      alert('This is working!');
    }

  };

  Example.STATIC_OBJECT = {
    'a': 1,
    'b': 2,
    'c': 3,
    'd': 4
  };

  return Example;

})();


Calculate industrial production in Mt/a before calculating corresponding energy demand

This makes clearer what the actual demand for steel, cement, chemicals, aluminium etc. in megatonne per year is in the model.
The split between primary and secondary is handled in an intermediary step which takes today's production routes (in industrial_production_per_country.csv in Mt/a) and calculates the ideal 2050 routes (in industrial_production_per_country_tomorrow.csv in Mt/a).
Also now industry_sector_ratios.csv represents the genuine energy demand in MWh per tonne material - there are no tricks for primary/secondary split manipulation anymore (this is taken care of in build_industrial_production_per_country_tomorrow).
Hopefully this is all more transparent.
This PR addresses Issue https://github.com/PyPSA/pypsa-eur-sec/issues/62.
Still to do:

[x] Split ammonia demand out from "basic chemicals"
[x] Create a file with today's energy demand per sector per country (from JRC-IDEES) for future pathway optimization
[ ] Options to choose endogenously in model between BF+CCS and DRI+EAF in model

The competition between DRI+EAF and BF+BOF+CCS will have to wait.


Change permissions of all test_* files to non-executable so they are not missed by nose

If a file is set as executable, it isn't automatically run by nose test runner.
@jgrund pointed out that we have a number of test_* tests that are not being run because they have one of the executable permission bits set. These should be set as non-executable (as should all python files in the project as we should always run with python explicitly, e.g. with an assigned interpreter, this would reduce potential bugs with different file permissions)

relevant file permissions amended by:
find ./ -type f -executable -name 'test_*.py' | xargs chmod a-x

TODO: fix permissions on all python project files


support workbench.action.terminal.runSelectedLine for run current line and go to the next line

Currently, when you don't select anything, workbench.action.terminal.runSelectedText will run the line where your cursor locate. This is awesome.
But when you are write a script, you often want to run code line by line. It would be very convenient if there is a function like "run current line and go to the next line".

This is a good candidate for a simple extension, we would want to avoid putting something so specific into the core editor to prevent feature bloat.


[VL] Integrate upstream DirectBufferedInput with executor

Description
Upstream velox add DirectBufferedInput to replace BufferedInput, and use a optional executor do async load or prefetch, but its executor was passed from HiveConnector which means this executor is global instance, I'm not sure Gluten's thread mode or memory mode could fit it, will keep tracking.

DirectBufferedInput has less performance boost, my benchmark result in TPCH sf30 cloud storage is 160s(without executor) vs 152s(with executor).

@zhli1142015 Did you noted the perf regression?

yes, we noticed the regression also 10% totally for our TPCDS runs.

should we fallback to bufferInput?
directBufferedInput reads page by page, BufferInput reads row group by row group.
rowgroup read ahead looks not work with direct buffer input.

I thought @Yohahaha opened an issue for this: https://github.com/facebookincubator/velox/issues/7535.

@FelixYBW have you try compare the perf between DBI and BI?

Yes, The issue I faced now is row group read ahead doesn't work with DBI. without the read ahead perf is worse

Yes, The issue I faced now is row group read ahead doesn't work with DBI. without the read ahead perf is worse

thanks, I will reopen above issue to solve it.
we could use BI default in oap-velox for now. @zhouyuan

Just tested a IO bound query, DBI 4.6 min, BI 1.6 min


Server build throws @nguniversal/express-engine warning

I get these warning
WARNING in<EMAIL_ADDRESS>(Emitted value instead of an instance of Error) Cannot find source file '../../modules/ng-express-engine/index.ts': Error: Can't resolve '../../modules/ng-express-engine/index.ts' in '/Users/stevik/NodeJS/angular-webpack2-starter/node_modules/@nguniversal/express-engine'
 @ ./src/server.ts 9:0-62

WARNING in<EMAIL_ADDRESS>(Emitted value instead of an instance of Error) Cannot find source file '../../../modules/ng-express-engine/src/main.ts': Error: Can't resolve '../../../modules/ng-express-engine/src/main.ts' in '/Users/stevik/NodeJS/angular-webpack2-starter/node_modules/@nguniversal/express-engine/src'
<EMAIL_ADDRESS>3:13-34
 @ ./src/server.ts

WARNING in<EMAIL_ADDRESS>(Emitted value instead of an instance of Error) Cannot find source file '../../../modules/ng-express-engine/src/tokens.ts': Error: Can't resolve '../../../modules/ng-express-engine/src/tokens.ts' in '/Users/stevik/NodeJS/angular-webpack2-starter/node_modules/@nguniversal/express-engine/src'
<EMAIL_ADDRESS>8:15-34
<EMAIL_ADDRESS> @ ./src/server.ts

WARNING in<EMAIL_ADDRESS>(Emitted value instead of an instance of Error) Cannot find source file '../../../modules/ng-express-engine/src/file-loader.ts': Error: Can't resolve '../../../modules/ng-express-engine/src/file-loader.ts' in '/Users/stevik/NodeJS/angular-webpack2-starter/node_modules/@nguniversal/express-engine/src'
<EMAIL_ADDRESS>7:20-44
<EMAIL_ADDRESS> @ ./src/server.ts

Adding root('node_modules/@nguniversal') into EXCLUDE_SOURCE_MAPS array in constants.js helps.

I didn't notice that, but I merged your changes.


NO-JIRA: More snyk ignores

Can you please rebase on top of the new master?
/hold

/retest

/hold cancel

@mandre ?

/lgtm


Fix security vulnerability in Socket.write (#26)

The issue was caused by incorrect handling of data slices with non-zero start indices and could lead to arbitrary memory being leaked to the socket. See #26 for details.

Good catch, a precondition would definitely be a good idea.
It's also worth considering whether any of the Socket API should be public, because it's basically just a low-level wrapper around the platform-specific APIs and has the potential to have serious bugs if not used correctly. I also have a feeling that users of FlyingFox probably don't need Socket when AsyncSocket exists. But it's also quite a big breaking change, so it might be better to just document that it's unsafe to use (in the FlyingDocs 😉).

Socket hasn't been included in a tagged release yet, so I could remove it, but I do like the idea of users being able the configure the raw Socket, although this is possible anyway by just exposing the FileDescriptor there are some niceties in Socket like throws etc. I also really like SE-0299 and think the option wrapper in Socket is really nice so did want to share it;
try socket.setValue(true, for: .localAddressReuse)
try socket.setValue(2048, for: .receiveBufferSize)

Love the idea of FlyingDocs.

I agree, that is quite nice. I think it's probably sufficient to add a precondition to write and add documentation to each method of Socket saying to be careful and pointing to the alternative AsyncSocket methods. Adding a code snippet for each Socket method would probably be a good idea too, because then people are more likely to use them correctly. I can work on adding documentation for that if you want? (just as DocC documentation comments)


Documentation update for README.md in docs

Update readme doc to clarify the contact methods for non-customers, versus current customers.

being addressed elsewhere as part of larger changes.


Refactoring to reduce indirection

This will be the epic to avoid the need of merging a very big PR.

+1

+1 gj


{AppConfig} Fix bug in print preview of restore command

Description

There was a bug in converting KeyValue object to JSON during print preview of az appconfig kv restore command. Fixed the bug by accessing the correct property, which is called locked, instead of read_only.
Testing Guide

This is the unhandled error that will be fixed by this PR:
az appconfig kv restore -n test-appconfig-00  --datetime 2021-03-16T00:00:00Z

---------------- Preview (Beta) ----------------

Deleting:
The command failed with an unexpected error. Here is the traceback:
'KeyValue' object has no attribute 'read_only'
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-up6eyly2\knack\cli.py", line 233, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\commands\__init__.py", line 664, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\commands\__init__.py", line 727, in _run_jobs_serially
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\commands\__init__.py", line 720, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-c__gk16o\six.py", line 703, in reraise
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\commands\__init__.py", line 698, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\commands\__init__.py", line 331, in __call__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cg742dxv\azure\cli\core\__init__.py", line 816, in default_command_handler
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-4iz7mv8d\azure\cli\command_modules\appconfig\keyvalue.py", line 672, in restore_key
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-4iz7mv8d\azure\cli\command_modules\appconfig\_kv_helpers.py", line 664, in __print_restore_preview
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-4iz7mv8d\azure\cli\command_modules\appconfig\_kv_helpers.py", line 544, in __serialize_kv_list_to_comparable_json_list
AttributeError: 'KeyValue' object has no attribute 'read_only'
To open an issue, please run: 'az feedback'

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

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

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

cc @shenmuxiaosen

Hi @jiasli/ @fengzhou-msft , can you please review this PR and merge it?


tarsu.kz has changed to dulaty.kz

Name of the university has changed.
Taraz State University -> Taraz Regional University named after M.Kh.Dulaty

@tortkara I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host tarsu.kz not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Fix for having primay visibility in world options.

Brian, Since this is simple fix I am merging this directly. I hope you don't mind too much.
Ryan

Looks fine
On Saturday, February 27, 2016, MWY3510<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/bsavery/PRMan-for-Blender/pull/205
Commit Summary

Merged Cedric's changes to the UI
Revert "Merged Cedric's changes to the UI"
Fix for having primary visibility in world settings.

File Changes

M ui.py
https://github.com/bsavery/PRMan-for-Blender/pull/205/files#diff-0
(2)

Patch Links:

https://github.com/bsavery/PRMan-for-Blender/pull/205.patch
https://github.com/bsavery/PRMan-for-Blender/pull/205.diff

—
Reply to this email directly or view it on GitHub
https://github.com/bsavery/PRMan-for-Blender/pull/205.

--
<EMAIL_ADDRESS>508-274-8700

Ryan don't know what merging is.. But I'm sure it's ok..


Sub menu

Hi, is it possible to add subMenus and stuff like dividers please ?
There is nothing about it in the readme.md
However, I discovered Couscous and this theme, all simple and so effective, Big Thanks ;)

Hi, please see #5


DM-35057: Add support for hide code cells display param

Adds a checkbox to toggle the ?ts_hide_code query parameter to 1 or 0, related to https://github.com/lsst-sqre/times-square/pull/33
Generally adds support for display settings (introduced in https://github.com/lsst-sqre/times-square/pull/33), which are a different category of information that's also present in Times Square URL query strings.

Here's the checkbox for hiding code (checked by default) and resulting cleaned-up view of the notebook's outputs.


Defines and requires fs

Closes #707

Ha! The dangers of try/catch...
Since setting the PORT and ADDRESS environment variables works and is a much more common technique, we're not going to rush this one out, but it will go out in the next routine patchlevel.
Thanks!


compiler does not work

Type: Bug
after writing the code the code is not working or compiling or giving the error every time i try to run code in terminal
VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:06:49.809Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Hi @winnt3r2 ,
Thank you for contacting the VS Marketplace Team. To assist you effectively, could you kindly provide additional details about your concern with error message/ screenshot, this will help us investigate thoroughly.
Regards,
VS Marketplace Team

Closing the issue since there has been no updates on the information requested. If the issue still persists, please feel free to reopen or create a new issue.


Add CodeGen for Java (okhttp)

Tell us about the task you want to perform and are unable to do so because the feature is not available
Add CodeGen for Java
Library - okhttp

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

@ShiqiuC We do not assign issues. You are free to work on it and send across a PR.


Adds delete function at a class level - enables instances of models to be deleted without reading them first

This pull request adds a delete function to HashModel and JsonModel - so you can delete something you know the pk for without first having to read it from Redis.
So you can now do... (where Person is a HashModel but could be a JsonModel):
    Person.delete(pk)

before you had to do this:
    person = Person.get(pk)
    person.delete()

I've added a couple of tests too.
Closes #127

Codecov Report

Merging #132 (6d009a0) into main (2be0174) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   77.20%   77.31%   +0.10%
==========================================
  Files          12       12
  Lines        1154     1155       +1
==========================================
+ Hits          891      893       +2
+ Misses        263      262       -1

Flag
Coverage Δ

unit
77.31% <100.00%> (+0.10%)
:arrow_up:

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

Impacted Files
Coverage Δ

aredis_om/model/model.py
86.36% <100.00%> (+0.13%)
: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 2be0174...6d009a0. Read the comment docs.


Handle Falcor style observables from unhandled path datasource.

This seems fine to me (it's what we were patching locally in the some of the middleware we've been using). Feel free to merge. Bumping up the travis yml to go with Node 4, 6 should fix the build issue.

I think the local patching (using Rx4) was to make Observable instance methods available when using the datasource within a router. That worked since observables returned by route functions were converted to Rx5 internally. This change applies the same conversion to the observables returned by the unhandled data source methods.


move RadialGradientBrush from preview to public

Moving RadialGradient brush to public from preview now that the API review is complete. Spec document is here.

The spec still refers to GradientOriginOffset while we renamed it to GradientOrigin . Is there a PR for adjusting the spec to align with the current code? Or is that unecessary?

The spec still refers to GradientOriginOffset while we renamed it to GradientOrigin . Is there a PR for adjusting the spec to align with the current code? Or is that unecessary?

Yes. The spec will need to be updated.

I'm in the process of updating the spec right now.

/azp run

:shipit:

Does the Spec now align with the idl? @chingucoding @ranjeshj or @MikeHillberg ?

The idl is aligned with what was discussed. I think @mikehillberg is in the process of getting the spec up to date with the current API. So yes, IDL aligns with the spec that was discussed.

/azp run


Add command to create example cakefile from existing project.

It's not clear how I can specify which files and under which groups/folders should be added to the project at the moment of creation, only basic project settings. Please clarify.

For each target you specify the files to include and exclude, like so:
Project.new do |project|
  project.application_for :ios, 8.0 do |target|

    target.name = "Sup"
    target.include_files  = "DifferentAppFolder/*.*"
    target.include_files << "OtherFolder/*.*"
    target.exclude_files << "FolderToIgnore/*.*"
  end
end

The syntax used to match file and folder paths is Ruby's Glob syntax - http://ruby-doc.org/core-1.9.3/Dir.html#method-c-glob
I've just noticed I missed those properties off in the documentation so thanks for pointing this out :)

@maximkhatskevich  I've updated the documentation to explain the include_files and exclude_files properties.
https://github.com/jcampbell05/xcake/blob/master/docs/Cakefile.md#include-files
Any issues let me know :)

That’s great, thank you! What about brand new Xcode Assets catalog? it looks like file, but instead a folder. How do I specify that kind of file/folder in a Cakefile?
Also what are limitations of the Xcake? Is there something that I can not describe in Cakefile what is available in the *.xcodeproj file?
On January 8, 2016 at 01:12:30, James Campbell<EMAIL_ADDRESS>wrote:
@maximkhatskevich I've updated the documentation to explain the include_files and exclude_files properties.
https://github.com/jcampbell05/xcake/blob/master/docs/Cakefile.md#include-files
Any issues let me know :)
—
Reply to this email directly or view it on GitHub.

Xcake should detect bundles and special folders such as Asset Catalogs or Core Data Models correctly and will add them to the project correctly. In the case of assets you just need to make sure your file pattern matches folders with the extension .xcassets.
Here is a list of the current support of Xcake https://github.com/jcampbell05/xcake/blob/master/docs/Xcode Project Support.md
It should be enough to get a comprehensive project file up and running. But if there is anything you need or think should be changed let me know and I'll be happy to add :)

Sounds great! Thank you for quick answers.
One last question - is there a built-in tool/command to init Cakefile by reverse-engineering existing *.xcodeproj file so later I can continue controlling my project by just editing this file where it's needed? It would be also a very good starting point, an example to see how things should be expressed.

@maximkhatskevich  Not right now but it's something I've gave a lot of thought to. There are a lot of things to consider (i.e How do we generate the patterns to match the files and folders)
I may prototype something up to see how we could pull it off. I do want to lower the barrier of entry though :) so a cakefile init command I think would really help.  so if you're willing to try it when I build it, then great!

@jcampbell05 Sure, I would be glad to try it out when you have a beta! Feel free to let me know.

@jcampbell05 but I think the command should be xcake init.

Yup it should, good catch ;)

It is probably enough to give a worked replica, and then expect a developer to be able to take that and refactor it down with file/folder globbing patterns etc

@maximkhatskevich  Going to start working on this shortly, I'll create a branch you can use to try this out.

Wow, I look forward to it!

@maximkhatskevich @jcampbell05 Really interested in this feature as I need the ability to add UITest targets and schemes dynamically to an existing xcode-built (Ionic) project...I was wondering if you guys believed this to be possible with Xcake and/or had any example code or references to how it could be done.
I am an IC who is working with a client who needs to build many (100s) of branded apps for retail clients, using Ionic and Fastlane, and the final blocker is having to attach a UITest target to the project in order to run Fastlane screenshot during Fastfile processing.
Thanks for all the hard work you guys have put into this project BTW...it looks great.

@jcampbell05 my original question was a mix of 2 questions.

Sample Cakefile - this is done in #110 and #116.
Reverse-engineering Cakefile for a given existing Xcode project by calling something like "xcake init". First of all, this feature is actually not needed. It's better to take a look on sample Cakefile(s), write your own as well as re-organize project files and generate project file from that. Second, you currently implemented the "xcake init" and it just generates basic Cakefile which developer then can modify according to their needs.

So I think we can close this issue, it's resolved.

@menriquez if I understood your question correctly, you are asking about how to configure UI tests for a target, right? If so - then please take a look on our sample Cakefile here (under "/example/app/" folder - you can find it directly here while it's not merged into main branch https://github.com/jcampbell05/xcake/tree/feature/example/example/app).

@menriquez also please note, that currently UI tests are not working properly yet, we are working on it, here are the related issues #108 and #115.

We should probably attempt to do something like this since this appears to have driven the traction of Struct

@jcampbell05 agree, not having this functionality is the number 1 barrier for newcomers to start using our tool ASAP. It took me quite a while before I found some time to experiment with Xcake and migrate my project on Xcake manually.

@maximkhatskevich awesome, I'll study how Struct does it to see if we can have any pointers :)

Good idea!