Question about OBSERVE state

What is purpose of number of OBSERVE steps > size of REPLAY_MEMORY?

Hello @mrgloom ,
I set OBSERVE steps so big just for demo purpose :smile:
If you are trying to reproduce the model,
I've added a section about that.

Hi @mrgloom ,
If above comments have answered your question, would you please close this issue?
Thanks!

I'm still not sure how number of OBSERVE timesteps estimated, it's just arbitary number BATCH < OBSERVE < REPLAY_MEMORY ?
Also what if I can't do all 3000000 at one time, how training can be continued? Just set OBSERVE to same value, load CNN weights, and set EXPLORE = (3000000 - steps_already_trained) ?

Hello @mrgloom

arbitary number BATCH < OBSERVE <= REPLAY_MEMORY

However, I set it according to the reference paper and empirical result.

Yes

Also is there something special about OBSERVE state, for example should bird pass through a pipe at least once during this state or it's not necessary?
Or OBSERVE state just used to init replay memory?
Also I run 2 training cases (about 150000 timesteps) one with recommended parameters and another with no EXPLORE state at all (I set FINAL_EPSILON and INITIAL_EPSILON to 0)
I found that without EXPLORE state it also learn to play fine, but my intuition about this that it will choose more long routes and this will lead to more risky playing, and with random actions at each timestep with small probability model learn to play more safely(so it's some kind of regularization?).
What my intuition can't understand is that how model learns to play game if during OBSERVE state bird do not pass any pipes.

OBSERVE is only used to fill in the replay memory.
Regarding why it still works without EXPLORE state, I think it's because this network is an overkill for this game.


File field required on edit page

Hi,
If I want to edit a video, the title for example, I am asked to select a file as the file input is marked as required.
Shouldn't it be optionnal on the edit page ? As we may only want to change the title or upload a thumbnail.
Thanks !

Didn't all

A fix for this issue has been published in version 0.3.1 on PyPI

Hi, sorry for the weird message above (and the closed action ?), really don't how it came up. Anyway, thanks a lot the this commit :)


How to solve the syntax conflict between JinJia2 and CSS in chapter 21.3 ?

I met such a question that JinJia2 used in CSS inline style will cause a conflict that the default block string will be misunderstood as a css property value in visual studio code . The detail is :
<span id='message_count' class="badge"
            style="visibility: {% if new_messages %}visible
            {% else %}hidden {% endif %};">

I happend to find a related plausible solution that is to set app.jinja_env.block_start_string = '(%', this solution asks me to alter every existed block start string , that would be boring. So is there anyway to solve such question besides this?

Are you just concerned about the way the code is highlighted in VS Code? In that case you may want to install a Jinja extension. I don't really have experience with this, but a Google search returned https://marketplace.visualstudio.com/items?itemName=samuelcolvin.jinjahtml. Hopefully that works.

Are you just concerned about the way the code is highlighted in VS Code? In that case you may want to install a Jinja extension. I don't really have experience with this, but a Google search returned https://marketplace.visualstudio.com/items?itemName=samuelcolvin.jinjahtml. Hopefully that works.

The above code  will be shown in VS Code like this:

As I explained, this is not a problem, it's just that VS Code does not understand the Jinja template syntax. Have you tried installing a Jinja plugin?

As I explained, this is not a problem, it's just that VS Code does not understand the Jinja template syntax. Have you tried installing a Jinja plugin?

Yes, i have installed Better JinJa,  it seems that this plugin changes nothing. But it does really not matter, what i really care about is that syntax conflict, and now i know i made a mistake. Thank you so much.

As I explained, this is not a problem, it's just that VS Code does not understand the Jinja template syntax. Have you tried installing a Jinja plugin?

Oh, i made a mistake, thank you for your reply. By the way, i have installed Better JinJa, but it seems that this plugin changes nothing, did i miss some key step?

I'm not sure if you made a mistake or not. As I said above, I'm not familiar with the Jinja plugins available for VS Code, so I don't know how they are used, or if they need some special configuration.
My opinion is that this can be ignored. As I said above, this isn't an error in your application, it's just that VS Code does not understand the Jinja template syntax.

I'm not sure if you made a mistake or not. As I said above, I'm not familiar with the Jinja plugins available for VS Code, so I don't know how they are used, or if they need some special configuration.
My opinion is that this can be ignored. As I said above, this isn't an error in your application, it's just that VS Code does not understand the Jinja template syntax.

ok, now that it is not a problem, i will leave aside it. thank you for your reply, sincerely.


HMR with Remix

Description
When I update a recipe or a token used in panda config everything is rebuild accordingly except the css file. In order to see effective changes, I need to restart remix dev server.
On the other hand, if I update a css property in any page or component, everything is rebuilt including the css file.
Link to Reproduction
https://codesandbox.io/p/github/efflam/remix-pandacss/main
Steps to reproduce
No response
JS Framework
React w/ Remix 1.18.1
Panda CSS Version
^0.6.0
Browser
No response
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
Bug reproduction repository : https://github.com/efflam/remix-pandacss

This has already been fixed in one of the recent PRs. We'll release an update shortly.
If the issue persists after the release, I'll re-open this issue.

Hi, I'm on Remix 2.5.0 and Panda 0.27.1 and this is still an issue.

After some testing, I can confirm that the bug reappeared on version 0.24.0.
Version 0.23.0 is the latest version without this bug.

@fcisio Does this happen in v0.27.2?

Yes, seems to still happen in v0.27.2

I just tested this with the latest remix and it works as expected.
Here's the app/root.tsx
import type { LinksFunction } from '@remix-run/node'
import {
  Links,
  LiveReload,
  Meta,
  Outlet,
  Scripts,
  ScrollRestoration,
} from '@remix-run/react'
import styles from './index.css'

export const links: LinksFunction = () => [{ rel: 'stylesheet', href: styles }]

Here's the  app/index.css
@layer reset, base, tokens, recipes, utilities;

* {
  color: orange;
  font-size: 40px;
}

https://github.com/chakra-ui/panda/assets/6916170/4f7320a6-2b0e-4ded-8203-5ed03dbdb55b


'viminfofile' requires double backslashes on Windows

Unlike other options expecting filenames as values (backupdir, undodir, verbosefile, helpfile etc.) viminfofile requires double, not single backslashes.
For example set backupdir=c:\test\backups and set verbosefile=c:\test\verbose.txt work as expected (and, I think, consistently with the description at :help option-backslash), and set viminfofile=c:\test\viminfo.txt sets viminfofile to "c:testviminfo.txt".
OS: Windows 7,
Gvim: 8.1.0126 x86

Unlike other options expecting filenames as values (backupdir,
undodir, verbosefile, helpfile etc.) viminfofile requires
double, not single backslashes.  For example set backupdir=c:\test\backups and set verbosefile=c:\test\verbose.txt
work as expected (and, I think, consistently with the description at
:help option-backslash), and set viminfofile=c:\test\viminfo.txt
results in setting viminfofile to "c:testviminfo.txt".

Thanks for reporting this.  The option is missing the P_EXPAND flag.
I'll fix that.
--
hundred-and-one symptoms of being an internet addict:
216. Your pet rock leaves home.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Upgrade the runtime images with rhods runtime,ref:80b11e7

Upgrade the runtime images with rhods runtime,ref:80b11e7
This would change the runtime image from reference of opendatahub to RHODS(modh) quay.
Related-to: https://github.com/opendatahub-io/notebooks/issues/183
Reference: https://quay.io/repository/modh/runtime-images?tab=tags
commit id: 80b11e7

You may find an open PR on the live-builder repo with the upgrade:
https://gitlab.cee.redhat.com/data-hub/rhods-live-builder/-/merge_requests/248

Hey, just noticed a detail regarding the recent update. The sha digest is updated but we need also to update the registry! it points to workbenches

Good catch, thanks for spotting it.
I have made the change.

You may find an open PR on the live-builder repo with the upgrade: https://gitlab.cee.redhat.com/data-hub/rhods-live-builder/-/merge_requests/248

Thanks for PR.
we can update runtime-images in live-builder separately from notebook images
as with notebook image update , it would causes mis-match on self-managed cluster.
as the old notebook-images would be pointing to old set of runtime-images
where as the live -builder will pull in new runtime-images.
causing image-pullback error.

/lgtm

Thanks for the review :100:
/approve

/retest-required

/retest-required

/retest-required


Uncaught ReferenceError: includeSource is not defined

[Enter steps to reproduce below:]

toggle node-debugger, autofill
click run, crashes immediately

Atom Version: 0.175.0
System: Mac OS X 10.9.5
Thrown From: Atom Core
Stack Trace
Uncaught ReferenceError: includeSource is not defined
At /Applications/Atom.app/Contents/Resources/app/node_modules/pathwatcher/node_modules/q/q.js:126

undefined

Commands
     -9:53.5 core:save (atom-text-editor.editor)
     -9:52.6 core:backspace (atom-text-editor.editor)
     -9:51.7 core:save (atom-text-editor.editor)
 11x -9:51.1 core:backspace (atom-text-editor.editor)
     -1:08.4 core:save (atom-text-editor.editor)
     -0:47.4 editor:newline (atom-text-editor.editor)
  2x -0:45.9 core:backspace (atom-text-editor.editor)
     -0:43.2 core:save (atom-text-editor.editor)
     -0:30.1 command-palette:toggle (atom-text-editor.editor)
     -0:27.2 core:confirm (atom-text-editor.editor.mini)
     -0:26.2 node-debugger:toggle (atom-text-editor.editor)
  5x -0:00.0 core:backspace (atom-text-editor.editor.mini)

Config
{
  "core": {
    "disabledPackages": [
      "coffeescript-be-gone",
      "jsformat",
      "merge-conflicts",
      "color-picker",
      "compare-files",
      "debugger",
      "autoflow-diffable"
    ]
  },
  "editor": {
    "showInvisibles": true,
    "showIndentGuide": true,
    "tabLength": 4,
    "invisibles": {}
  }
}

Installed Packages
# User
atom-beautify, v0.21.2
autoclose-html, v0.14.0
linter, v0.10.1
minimap, v3.5.6
node-debugger, v0.2.3
open-last-project, v0.1.1
terminal-panel, v1.9.2

# Dev
No dev packages

This is probably a bug in the node-debugger package.  As the package states in its README, it's still very buggy.  If you want though, you could open an issue there.


Training data with QID and PID

Thank you for creating such a great dataset for passage re-ranking!
I'm wondering if it is possible to release the top 1000 passages retrieved for each training/dev/test query with the corresponding QID and PID? The current training data is constructed with the raw text of queries and passages, which are too huge to use. Also, since the qrels files are actually constructed with QID and PID, it would make life much easier if the train/dev/test data are also constructed with QID and PID.

Yes, it’s a good idea.
Some people may not be interested to passage contents and just want to train based on current top retrieval results.


close #9

Remove the dependencies to fontawsome css and include the 4 icons directly in SVG format

Thank you, very appreciated!


We have an open PR for this component that's being worked on by @Maximilian-R maybe you two could sync your work?

Hi @R0tenur, can you check if this PR includes the same functionality you have added here? :) https://github.com/sebgroup/green/pull/735
If so, I can merge it now and finish the remaining angular issues in a new branch (the chlorophyll part is done)

I donat care about my code, that is copied from our application. Just want it to be in green so we don't Neeson to manage thw CSS there

Well, you can still check if the css matches? I am asking if there is anything from this PR that we should carry over to the other one

https://github.com/sebgroup/green/pull/735 was merged

Nice! Then we can refactor this code away from our application


Another License Question

Hi guys!
I want to make a product which includes twitter emojis and as you know they are an open source (https://twitter.github.io/twemoji/). They are already in the market with the exactly same product that what I am going to sell. I was wondering that I can use them in the product´s packaging too if I put the proper attribution (https://creativecommons.org/licenses/by/4.0/legalcode) in the online marketplace where I am going to sell the product.
Do you agree that with a good referencing in the website I should not have any legal issues?
Thank you very much.

Yes! There should not be any problems whatsoever if you are putting the license on your product and should just go on with the selling.

Great, thank you very much. Could I put the license on the website instead of the packaging if I am going to sell it online?

With two license on this project. Can we use twemoji for our web, if our web like twitter or facebook ?

When putting the attribution on your website, that is intended to cover the usage of Twemoji in its digital form on the website and would not cover attribution for when being used in physical form as graphics on packaging.
I assume that the packaging of your product will also include other copyright and trademark information, which would be an appropriate place to also include the Twemoji license.
In the case of the packaging only the license information for the graphics would need to be included as you're not using the script (obviously).

Copyright 2018 Twitter, Inc and other contributors
Graphics licensed under CC-BY 4.0: https://creativecommons.org/licenses/by/4.0/

Hi @bhaggs sorry to bother again on a further question on licensing (thought it'd be best not to open an entirely new issue one it).
In the scenario where we want to use an emoji on a physical (purchased) postcard created via a mobile app (imagine a postcard template within an app where the user can write text which could contain emoji) would the attribution be sufficient in the app under the postcard template itself or would you require it on the printed material as well? Asking mainly because the Postcard is full-paper/print and quite inflexible so trying to understand options.
Thank you very much for your input in advance.


Horizontal scrollbar click moves cursor

When clicking and dragging the horizontal scrollbar the cursor is moved to the line visible immediately above the scrollbar. This is particularlhy problematic when placing a cursor at the start of a selection then scrolling to shift-click the end of the selection.
Issue appears in v0.122.0 on Windows 7 and v0.121.0 on Windows 8.1

This appears to be an issue in v1.19.3 on Mac OS X Sierra v10.12.6.


Plushie Pajama Man option not working

Plushie Pajama Man unlock option does nothing. I have everything selected except for the suits, shower, and toilet.

Hey, thanks for reporting the issue - I copied wrong variable into my file (sorry!). I've just pushed v0.3.1 to Thunderstore, please check if it resolves your issue.

It seems to be working now


Binding tool releases should provide an  bt-arm64-apple-darwin flavor

It seems like the strategy used here to generate multiple binaries for multiple platforms is to rely on the OS default toolchain:
    strategy:
      matrix:
        include:
          - os: macos-latest
            target: x86_64-apple-darwin
            suffix: ''
          - os: ubuntu-latest
            target: x86_64-unknown-linux-gnu
            suffix: ''
          - os: windows-latest
            target: x86_64-pc-windows-msvc
            suffix: .exe

Unfortunately, this approach is limited by the number of OSes/ platform supported by the default provided runners by GH.
We could envision adding self hosted runner, but for a single project, that seems overkill.
I believe the long term solution is to rely on on rust toolchains https://rust-lang.github.io/rustup/cross-compilation.html

somebody did that there: https://gist.github.com/Tehnix/5209b8fefd368bfdb67ca8761d3b8009#file-ci-yml-L85

This is one now, see the latest release.


Rename plugin

According to Maven plugin guide, prefixing a plugin with "maven-" is discouraged and may be in violation of Apache trademarks.
Please rename the plugin.  Perhaps "pre-commit-maven-plugin"?

Oh I didn't know that.
In that case, let's rename it git-code-format-maven-plugin.


.NET Android export should validate that the TargetFramework is net7.0

Godot version
v4.2-dev5_mono_win64 (and 4.2-dev4_mono_win64)
System information
Godot v4.2.dev5.mono - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2070 with Max-Q Design (NVIDIA; <IP_ADDRESS>61) - Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 Threads)
Issue description
Android export fails because of a non-existent NuGet package that can't be restored:

Manually trying to add this package to my project reports that the closest version available is 7.0.0, and that the platform is incompatible with the generated csproj file.
dotnet add package Microsoft.NETCore.App.Runtime.linux-bionic-x64 --version 6.0.21
  Determining projects to restore...
  Writing C:\Users\Mikey\AppData\Local\Temp\tmp1057.tmp
info : X.509 certificate chain validation will use the default trust store selected by .NET for code signing.
info : X.509 certificate chain validation will use the default trust store selected by .NET for timestamping.
info : Adding PackageReference for package 'Microsoft.NETCore.App.Runtime.linux-bionic-x64' into project 'C:\Projects.temp\Project1\Project1.csproj'.
info : Restoring packages for C:\Projects.temp\Project1\Project1.csproj...
info :   CACHE https://api.nuget.org/v3-flatcontainer/microsoft.netcore.app.runtime.linux-bionic-x64/index.json
warn : NU1603: Project1 depends on Microsoft.NETCore.App.Runtime.linux-bionic-x64 (>= 6.0.21) but Microsoft.NETCore.App.Runtime.linux-bionic-x64 6.0.21 was not found. An approximate best match of Microsoft.NETCore.App.Runtime.linux-bionic-x64 7.0.0 was resolved.
error: NU1213: The package Microsoft.NETCore.App.Runtime.linux-bionic-x64 7.0.0 has a package type DotnetPlatform that is incompatible with this project.
error: Package 'Microsoft.NETCore.App.Runtime.linux-bionic-x64' is incompatible with 'all' frameworks in project 'C:\Projects.temp\Project1\Project1.csproj'.

running dotnet --list-sdks reports that I have both 6.0 and 7.0 installed:
dotnet --list-sdks
6.0.317 [C:\Program Files\dotnet\sdk]
7.0.400 [C:\Program Files\dotnet\sdk]

Steps to reproduce

On Windows 11 (yes I'm running 11 -- not 10 as the system info states) -- Download and run Godot 4.2-dev4 or 4.2-dev5, and create a new project.
Create a simple scene with some CSG boxes, and a camera, attach a working C# script to one of the nodes.
Generate the C# project files, and using visual studio 2022, add some code to the script.
Attempt to export the APK using the automatically downloaded template.
It fails with the above error about the missing nuget package and incompatible framework.

Minimal reproduction project
N/A

As described in the release blog post, you should target .NET 7.0 for Android: https://godotengine.org/article/dev-snapshot-godot-4-2-dev-5/#downloads

Well UX wise, I guess we should refuse to export if the TargetFramework is not net7.0.

@akien-mga:

Update: Even after manually switching the csproj to .NET 7 framework, it still fails with the same error:

I believe the new title is wrong and should be reverted.

Is that a choice that I have? -- Godot created the csproj and solution files. How do I chose what framework version to target? -- Just manually do it in Visual Studio, or is there an option in the Godot UX somewhere?

Yes, you can change the <TargetFramework> to net7.0 in the csproj manually or using Visual Studio.

Also Perhaps when Godot creates the project files, it should create ones that target a version of .NET that it supports?

Only the current experimental Android support requires targeting .NET 7.0. We considered changing the project creation to default to .NET 7.0, but it can break in some scenarios (see https://github.com/godotengine/godot/pull/72333).

Additionally: Even after manually switching the csproj to .NET 7 target framework in Visual Studio, it still fails with the same error

The current experimental Android support only supports architectures arm64 and x64. You seem to be trying to build for architecture x86.

Alright, yeah removing arm64-v8a and x86 and setting .net7.0 fixed it.
The deploy build button doesn't work, but that's a different issue. -- The .apk builds, and I can deploy it via command line;  so I think your original asssement is correct.
Though I would recommend two warnings in this case:

If the target framework is not net7.0.
If a non-supported architecture is selected (i.e. x86 or arm64-v8a -- or anything else other than the explicitly supported ones).


Support for Google Photo uploads

would be really nice.  saw the closed picasa support request (https://github.com/google/google-api-nodejs-client/issues/292), but thought that Google Photos would be supported long term.
can't find it in the developer APIs, but here's hoping.

There's no Google Photos API at this time.

There is now. Any plans for it?

+1 for adding Google Photos.


Not working in 52.5

Hello,
I have some issues to execute it in the new version of sketch.
Thanks

I experienced the same issue!


Handling of custom datasets

Can I use this code base to run the methods over multispectral images.
i have to take care of transformations differently.
Data will be .tif file rather .png
Backbones will be slightly different as now the first conv layer will have n in_channels compared to 3 in plain images

I'm closing this, but feel free to reopen if you have any new questions

Hi ! I have one issue while using transformation for custom data
Though, i am able to run my model but it is taking lot of time for 1 epoch. I am not sure whether it is due to dataloader or is it supposed to be that slow in general (20 mins for 1 epoch)
let me carry you through my dataset and transformation

i am reading a raster files using rasterio which returns  (4,64,64) array of type float32
Below, i am sharing my transformation pipeline

i have to do some transformations for rgb channels specifically like color jittering, grayscale
I see in your readme that it took  4m 55s for a resnet18 model to run 1 epoch of  imagenet data. I assume you use all arguments same what was mentioned in bash-scripts i.e batch_size 128 * (2 GPUs). It has 100,000 images (900,000 training and 10,000 validation).
I took the  batch_size 128*(4 GPUs) for a resnet 50 model. Total number of rasters i have is 27000( 21000 training and 6000 validation). i am running Barlow_twins model. i also changed num_workers to "1"
Do you think it is normal or am i having a performance bottle neck ?
Do you have any suggestion to boost  my training ?

Hey,
Next time, please reopen the issue, it's easier for us to see.
I'll try to go through all the points you made.
Regarding rasterio, I've never used it, so I can't know how fast/slow it is. I would suggest that you just create a dummy dataloader (just return random noise in the shape that you need) so you can see how long each epoch takes without relying on rasterio. Of course, if your bottleneck is this, unfortunately, there's not much we could do.
Your transformation pipeline seems fairly complicated, so I assume there's a large bottleneck there. Maybe you could also try to disable it temporarily as well.
Doing these 2 steps, you can roughly estimate how long it takes to do data loading and how long it takes to do your augmentations.
About the model, the 5 minutes is for a resnet18 on imagenet100, so it's 10 times smaller than the full imagenet. It is still larger than your dataset, though.
Regarding the number of workers, setting it to 1 is a huge bottleneck. Usually the more the better (up to some point), I would suggest 4-5 or more, depends on your gpu, CPU and SSD. Also note that this is per gpu, so if you have two gpus and 5 workers, it's actually 10 workers in total.
As you can see in our experiments, we have a 56% improvement for Barlow when using dali, which is much better optimized. So I would assume most of your bottleneck is on the data loading side, which you can confirm with the experiments I suggested.
Let me know if I can help in anything else!

Hey,
I'll try to go through all the points you made.
Regarding rasterio, I've never used it, so I can't know how fast/slow it is. I would suggest that you just create a dummy dataloader (just return random noise in the shape that you need) so you can see how long each epoch takes without relying on rasterio. Of course, if your bottleneck is this, unfortunately, there's not much we could do.
Your transformation pipeline seems fairly complicated, so I assume there's a large bottleneck there. Maybe you could also try to disable it temporarily as well.
Doing these 2 steps, you can roughly estimate how long it takes to do data loading and how long it takes to do your augmentations.
About the model, the 5 minutes is for a resnet18 on imagenet100, so it's 10 times smaller than the full imagenet. It is still larger than your dataset, though.
Regarding the number of workers, setting it to 1 is a huge bottleneck. Usually the more the better (up to some point), I would suggest 4-5 or more, depends on your gpu, CPU and SSD. Also note that this is per gpu, so if you have two gpus and 5 workers, it's actually 10 workers in total.
As you can see in our experiments, we have a 56% improvement for Barlow when using dali, which is much better optimized. So I would assume most of your bottleneck is on the data loading side, which you can confirm with the experiments I suggested.
Let me know if I can help in anything else!

Hey,
Thanks for the reply. I will implement your suggestions and get back to you.
One doubt regarding num_workers.  The more i use, of course it also becomes a bottleneck and it doesn't even start . As i use 4 GPUs , shall i go for 2 per gpu ?

It really depends on your hardware. I think it's more a matter of testing. Generally, after some number of works, you start to be bottlenecked by the GPU (so you don't see any speedups in training time) and then it will start to get slower and slower. Also, in a 4 GPU machine, I assume CPU/ram isn't going to be a problem, so you can probably keep the same number of workers.

So, i checked the time for data without transformation, it takes around 3 min / epoch with 4 GPUS  and ( 1 workers per GPU). It is such a huge difference

I see. Also, the part where you convert to numpy seems a bit strange to me, is it needed?

Sorry, i shared the snippet of old code. I had removed it.

i had also changed the raster_loader to get torch float32

ok! and if you don't do your transformations, you go from 20min per epoch to 3?

yes, exactly.

How are transformation functions on torch Tensor compared to PIL image ? Can there be performance bottleneck over there ?

ok! and if you don't do your transformations, you go from 20min per epoch to 3?

now i am using resnet 18 for testing, so it changed from 12 min per epoch to 3 but still a lot

I know that recently torchvision implemented some stuff to do transformations directly on tensors/gpu (I can try to find it later, but I think it was version 0.8 or 0.9). I tried this myself (not pushed too much) and I saw zero difference in performance.
I would suggest that you try adding one transformation at a time (there might be just one or a couple of transformations that are really heavy) for extra debugging. Apart from that, you could try kornia or other libraries, but I won't be of much help.
Nvidia's DALI will for sure boost your performance, but it's more complicated to implement the transformations, but you might wanna take a look at that if it can support the type of data that you need to load. The transformations themselves are more straightforward.

Thanks for the suggestion. I will try doing each transformations separately and see.
I have already tried kornia. It was of not that great help.
Regarding nvidia dali, i am not really sure do they have for custom data. I guess they mostly have for images, audios and videos not fr multi-spectral.

Yeah, I think it's not easily supported, but might be worth checking anyway.
I'm closing this issue, but if you wanna ask any questions/update if you find a good solution for you problem, let me know.
thanks!

Hi,
The bottleneck is Randomresizeandcropping. As soon as i removed the transformation, the speed is normal and fast. The bicubic interpolation is something which is taking lot of time. I didn't find any alternative solution to it but will try to search for something to tackle that.


Hello guys, here i am again to point out some more problems on our SIFRoleplaing sheet.

Hello to all again, first I would like to thank you @coldrage101 (not sure how to tag a person in roll20), for your amazing work, but, not everything is perfect yet, I believe that this system deserves a proper system since is a great one.
And I'm hoping that you can help all of us SIFRPG system users.
Now that the rolls are fixed, there are some minor bugs, me and a couple of friends tested and found some bugs.
The ability button "Knowledge" is trying to roll the "language" skill, which occurs on syntax error.
In "intrigue and combat" the "initiative" button rolls it correctly, but it shows no text on the chat, not even the results of the rolls. however, it adds to the initiative table successfully.
Still, on the "Intrigue and Combat" in the weapon section, all the weapons rolls are broken, the same as the first bug that I pointed out. It shows the number of dices, but it doesn't roll it.
And the last but not least, the "Movement" section in the "Intrigue and Combat" is bugged as well, not the whole thing, but there's a system problem regarding the rules from SIRPG. Movement is based on a pattern of "4 yards + RunSpecialty/2 - Bulk/2" This formula is working perfectly, except that in this system we can't have ".5's" never. When it comes to running speed the calculus is                                                                        " 4(4 Yards + Runspecialty/2 - Bulk/2 ) - Bulk. "
For example, Matt has a base movement of 4 yards, he has the run specialty on 2 and bulk of 1. Matt's final movement pool is a Move Speed of 5. and running speed of 5 * 4 - 1 = 19 Yards.
Another Example, Matt2 Has a base movement of 5 yards because of the quality "fast", he has the run specialty of 4 and bulk value of 2. Matt2 Has a Move Speed of 5 + 2 - 1 = 6 Yards. And a Running speed of 5 * 6 - 2 = 28 Yards.
Well, that will be all, thanks again, and I hope you can help us again. Thank you and have a nice day.

I'll have a look at it over the next few days.

I have a fix for the education rolling language, and another for language specialities dropping all of their dice.
The issues on with the initiative rolls and on the attacks exist in the original version of this sheet, but I shall look into them both.
I also wonder about the requirement for selecting a target token for attacks. It is very nice to see the requirement to hit along side the roll, but it does require the Narrator to have set up all enemies with there own character sheets, something uncommon to do with mooks, and also tells the players the strength of the opponent, something some GMs like to keep hidden. I would extend this to a lesser extent to the requirement for a token to be selected to roll initiative, though this is less of an issue as a token is needed for the turn tracker. This may however be a discussion for a seperate issue.
An additional bug I would like to mention is that when initiative is rolled for a token without a sheet, it throws an error: SyntaxError: Expected "(", ".", "[", "abs(", "ceil(", "d", "floor(", "round(", "t", "{", [ |\t], [+|\-] or [0-9] but "s" found.

I have now got fixes for the fighting and initiative rolls. I have removed the requirement for a token to be selected in both cases.
@shakasannin can you reiterate on the issues with movement? I lack understanding of the issue.

I'll do a pull request of the changes I have thusfar tonight.


Allow passing a checker as an executable instead of source

Just as it is with the regular source file / executable.

@A-dead-pixel Fixed in Toster 1.2.1

Thanks!


maybe add note to readme about finer fan control

I noticed in the readme that nothing about doing finer control of the fan curve is mentioned. The daemon seems to have no problem with doing this if you manually create fan speed and temp targets in the Json and works well for creating specific temp/fan targets. I havent tested beyond adding just one step fyi but fan control seems to work much better than radeon-profile and corectrl which both seem to cause the fan to get stuck in higher speed modes
I.E
default
"fan_control_enabled": true,
        "fan_curve": {
          "20": 0.0,
          "40": 0.0,
          "60": 50.0,
          "80": 75.0,
          "100": 100.0

edited to add 0rpm fan mode below 50c
"fan_control_enabled": true,
        "fan_curve": {
          "20": 0.0,
          "40": 0.0,
          "50": 0.0,
          "60": 50.0,
          "80": 75.0,
          "100": 100.0

While the daemon handles more curve points fine, the GUI is currently hardcoded to only 5 temperature levels and I'm not sure if this config gets displayed properly. I'll look into it.

No it doesnt get displayed in the GUI but the CLI shows it

The latest version now fully supports editing the curve from GUI.


[question] How to make one user object type a subclass of the other user object type?

I would like to make all superclass methods to be available to subclass objects. For example, if the type P is a parent of the type C, the object of the type C should have the method pmeth() which is defined in the type P.
The things I tried:

to assign prototypes: C.prototype = P.prototype. This failed with the message saying that C.prototype is read-only.
enumerate all keys in P and assign all functions to C:

for (var i = 0; i < P.prototype.keys(); i++) {
  var key = P.prototype.keys()[i]
  C.prototype[key] = P.prototype[key]
}

This also failed because there is no keys() function.
Objects have the flag extensible, and there's also the preventExtensions() function.
But what is the correct way to add the extension relationship between objects?

Assigning the prototype property of a function works as it should. If this doesn't work for you, a full example would be helpful in tracking down the issue.
Setting up subclassing with Javascript's prototype system can be a bit un-intuitive. Here's one way to do it:
function P() { /* parent constructor */ }
P.prototype.pmeth = function () { print("pmeth"); }

function C() { /* child constructor */ }
C.prototype = Object.create(P.prototype); /* create 'inherited' prototype object */
C.prototype.cmeth = function () { print("cmeth"); }

The keys function is a method of the Object object: var keys = Object.keys(obj). Note that it only returns the objects own properties, not those inherited from the prototype chain.


Add installation instructions for Postgres for scrape tutorial

Because I completely forgot about that... /sigh.

@StephanieKim if you're still working on this, feel free to continue - just cleaning up issues.


Failed at model compiling on mobile dGPU (nVidia)

Hi,
my mobile system has nVidia dGPU (NVIDIA GeForce RTX 4060 Laptop) and filed on the device at compile process.
It may better to use iGPU when dGPU is nVidia or have option to select GPU to use. Please consider some modification for improving on such system.
I changed script to use iGPU as below then it works
gimpopenvino/tools/stable-diffusion-ov-server.py line 132
    if device_name ==  "dGPU":
        device =  ["CPU","GPU.0","GPU.0"]
        device_int8 = ["CPU","GPU.0","GPU.0","GPU.0"]
        #device =  ["CPU","GPU.1","GPU.1"]
        #device_int8 = ["CPU","GPU.1","GPU.1","GPU.1"]

my environment:
・Windows11 Pro 10.0.22631 build 22631
・intel core ultra 9 185H
・System memory 32G
・iGPU: intel Arc Graphics
・dGPU: NVIDIA GeForce RTX 4060 Laptop

Error message:
Traceback (most recent call last):
  File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python39\lib\threading.py", line 980, in _bootstrap_inner
    self.run()
  File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python39\lib\threading.py", line 917, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\Public\GIMP\gimpenv3\Lib\site-packages\gimpopenvino\tools\stable-diffusion-ov-server.py", line 144, in run
    engine = StableDiffusionEngineAdvanced(
  File "C:\Users\Public\GIMP\gimpenv3\Lib\site-packages\gimpopenvino\tools\openvino_common\models_ov\stable_diffusion_engine.py", line 157, in __init__
    self.vae_decoder = self.core.compile_model(os.path.join(model, "vae_decoder.xml"), device[3])
  File "C:\Users\Public\openvino\python\openvino\runtime\ie_api.py", line 543, in compile_model
    super().compile_model(model, device_name, {} if config is None else config),
RuntimeError: Exception from src\inference\src\core.cpp:133:
[ GENERAL_ERROR ] Check 'false' failed at src\plugins\intel_gpu\src\plugin\program_builder.cpp:176:
[GPU] ProgramBuilder build failed!
Exception from src\plugins\intel_gpu\src\graph\include\primitive_type_base.h:58:
[GPU] Can't choose implementation for convert:Constant_19708 node (type=reorder)
[GPU] Original name: Constant_19708
[GPU] Original type: Convert
[GPU] Reason: Check '!kernels.empty()' failed at src\plugins\intel_gpu\src\kernel_selector\kernel_selector.cpp:70:
[GPU] Couldn't find a suitable kernel for convert:Constant_19708 params raw string: F16_BFYX_v1_p0_0_v1_p0_0_v512_p0_0_v512_p0_0;F32_BFYX_v1_p0_0_v1_p0_0_v512_p0_0_v512_p0_0

Hi there, did you use the latest commit ? As we are now handling this time of situation at complete_install level itself and you should not see this error as "GPU.1" won't even get populated in supported_device and you should not hit the "dGPU" scenario.
https://github.com/intel/openvino-ai-plugins-gimp/blob/main/gimpopenvino/tools/complete_install.py#L38

Thank you for sharing info, confirmed it works.


update all.json

Site
Urlscan
Screenshot

collablandsync.org
https://urlscan.io/result/0fae9eae-042c-484b-b3a1-49fb2f56cf67/
https://user-images.githubusercontent.com/14356826/180135229-54c43ad7-850b-4977-ae35-6fbae26c3913.png

dapp-supporteam.netlify.app
https://urlscan.io/result/ebbfcea0-87a8-48a8-8023-acdb03e460fd/
https://user-images.githubusercontent.com/14356826/180135255-275148ca-352f-40f5-8dcb-b74c272ba442.png

gkrypto.online
https://urlscan.io/result/8f446bc6-0882-4855-89ed-4d710919f767/
https://user-images.githubusercontent.com/14356826/180135245-e35dade0-108c-43de-8504-594fb86af5d3.png

interfacevalidation.on.fleek.co
NA
https://user-images.githubusercontent.com/14356826/180135253-df93f32e-3322-4cc6-b1ef-558f1151cf88.png

set-protocol.space
https://urlscan.io/result/14aba7c2-8c3f-44e5-961e-e5824f48ac1a/dom/
https://user-images.githubusercontent.com/14356826/180135242-3ab84335-47fa-4864-84a1-0b8ea1c3d695.png

synnex-network.xyz
https://urlscan.io/result/1abae5e2-2b98-46ac-b90f-c9357a38101a/dom/
https://user-images.githubusercontent.com/14356826/180135248-71cf9d38-82ad-4669-b54d-327e0bda4760.png

@jacogr trying to add interfacevalidation.on.fleek.co but the test said added urls › has no entries for allowed top-level domains
what does it mean?

Lemme check that one - the issue is that previously we had urls like storageapi.fleek.co which blocked the whole domain - and it has valid uses. So at this point anything *.fleek.co is white-listed.
For now drop that one, I'm guessing we need to make an exception (somehow) for *.on.fleek.co, i.e. those may be user URLs.

Putting in a work-around and will add that URL in https://github.com/polkadot-js/phishing/pull/1871

Okay, dropping interfacevalidation.on.fleek.co

thanks jaco!

Note: interfacevalidation.on.fleek.co was added in #1871 instead


Icon margins/padding in Edit Text too small

I have FontIconEditText and standard Edit Text above one another and the margins for the icon in the FontIcon edit text are non-existent. Text touches the icon so it looks different to the standard edit text control.
Maybe I missed something?

Oh, found this: http://stackoverflow.com/questions/3337371/androiddrawableleft-margin-and-or-padding
Thanks for an excellent library, btw!


Azure web app

can i easy convert it to a azure web app asp.net ?

this sample was designed as a desktop tool (winform), as a webapp I would recommend this other sample: https://github.com/Autodesk-Forge/model.derivative-WebAPI-sample


Section headers scroll off-screen

Unlike sectioned List views with section headers, the section headers in these sectioned ScrollViews scroll offscreen.
This is not quite the same as ScrollView. It is an issue when you have a long list with say 10 sections with 20 rows per section: no convenient way to see where you are in this tree without scrolling upwards.
The ScrollViewSectionKit behaviour is similar to say iOS Settings (e.g. Privacy and Security is split into Privacy/Security/Others) where the section headers do scroll.

Hey @vdhamer, at the moment this library doesn't support this feature, I could look into it, but once again given the fact that iOS 17+ offers practically unlimited customization options this library is obsolete and you can do pretty much anything with the List or ScrollView components.


DxfHatch Example

Hello,
I can not understand how to create the Hatch.
For example, I have a triangle with three points through which I want to start Hatch.
DxfPoint pt1 = new DxfPoint (0, 0, 0)
DxfPoint pt2 = new DxfPoint (1, 1, 1)
DxfPoint pt3 = new DxfPoint (2, 2, 2)
How to set its boundaries and is it possible to set them for a 3D plane?
             DxfLine ln1 = new DxfLine (pt1, pt2);
             DxfLine ln2 = new DxfLine (pt2, pt3);
             DxfLine ln3 = new DxfLine (pt3, pt1);

            DxfHatch hatch = new DxfHatch ();

HATCH objects are really complicated and this library can only do bare-bones experimental support for them.  That said, a hatch from a set of points could be constructed something like this:
var boundaryPath = new DxfHatch.NonPolylineBoundaryPath(DxfHatch.BoundaryPathType.Polyline);
boundaryPath.Edges.Add(new DxfHatch.LineBoundaryPathEdge() { StartPoint = pt1, EndPoint = pt2 });
boundaryPath.Edges.Add(new DxfHatch.LineBoundaryPathEdge() { StartPoint = pt2, EndPoint = pt3 });
boundaryPath.Edges.Add(new DxfHatch.LineBoundaryPathEdge() { StartPoint = pt3, EndPoint = pt1 });

var hatch = new DxfHatch();
hatch.PatternName = "SOLID";
hatch.BoundaryPaths.Add(boundaryPath);

As for a 3D plane, I have no idea if AutoCAD supports different z-values for a boundary; I've never tried it.  You can, however, have a hatch on something other than z=0 by setting the Z value of pt1, etc. to 0.0 and set hatch.Elevation to the desired z-value.

Often 3D functionality is created by a flat 2D drawing and a normal vector

confusing, yes. However, this worked for me with circles and polylines
and I have no idea about hatch.

Brett V. Forsgren<EMAIL_ADDRESS>schrieb am Do., 23. Juli 2020,
20:55:

Closed #136 https://github.com/ixmilia/dxf/issues/136.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ixmilia/dxf/issues/136#event-3581299646, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKQV6YWJ23FLYJMYUPXHW3R5CBSFANCNFSM4PDOA4FQ
.


Dreambooth support

This issue is to discuss Dreambooth (whether fully integrating it in this repo -training and inference-, or training via 3rd party, for example Colab, and doing inference in this repo).
Discussion and comparison with regular Textual Inversion is also encouraged.

+1
Will also note there have been discussions of making it easy to generate (or import) new concepts from the WebUI. Should support both textual inversion & dreambooth, and plans include having a "library" of these for ongoing use.
I think, given the purpose and intent of this repo, full integration should be the aim.

Interesting comparison:
https://www.reddit.com/r/StableDiffusion/comments/xjlv19/comparison_of_dreambooth_and_textual_inversion/

What I've noticed:
Textual inversion:
Excels at style transfer. "elephant in the style of Marsey"
May benefit from more images. My run with 74 images performed better than the one with 3
Best results (both in terms of style transfer and character preservation) at ~25,000 steps
DreamBooth (model download):
Far, far better for my use case. The character is more editable and the composition improves. It doesn't match the art style quite as well, though.
3 images worked better than 72
works extremely well with cross-attention prompt2prompt (the "img2img alternative test" script in automatic1111's UI)
1,000 steps (~30min on an A6000) is sufficient for good results
Worth mentioning - it's usable with deforum for animations
Combining the two doesn't seem to work, unfortunately. The next step might be either to directly finetune the network itself and apply one of these techniques afterwards, or possibly training the classifier.

My best success at Textual Inversion was with 3-5 images, so I'll try with many more.
Also, it seems Dreambooth may better preserve the character.
Results from post above:
Dreambooth

Textual Inversion

Even better, an actual implementation in 10G VRAM https://www.reddit.com/r/StableDiffusion/comments/xtc25y/dreambooth_stable_diffusion_training_in_10_gb/?context=3

If you're me (or like me), and you're wondering about the difference between the two, AFAUI (big caveat), TI can't do things like taking a fine-tuned (Dreambooth) model which is capable of creating these

and creating this

which is just utterly amazing

@tildebyte I tried this Colab (the one in the reddit post you share), but in the free tier, you have to pass --use_8bit_adam and not use full precision, which seemingly affects quality (I tried training with images of myself and it performed worse than TI).
It was a few days ago, so I may try again, just to see if they've introduced any other improvement.
Did you have any success with that Colab?

Colab doesn't have a free tier anymore (AFAIU); I haven't touched it since the change.

I think there is potential in TI, but I want to get Dreambooth to work, and compare them.
@tildebyte Here are some TI results, for comparison with what you refer to in https://github.com/invoke-ai/InvokeAI/issues/995#issuecomment-1272576376
Original images:  https://github.com/invoke-ai/InvokeAI/issues/517#issuecomment-1259486440. I used 5 of the 7 images to train.
After training TI with this repo (changing to num_vectors_per_token: 6 in v1-inference.yaml), these are some of the results I can achieve.
Workflow 1
Use txt2img playing with prompt weighting (N)
txt2img
"a painting of * :N on the beach in the style of van gogh" -s 50 -S 989419747 -W 512 -H 512 -C 7.5 -A k_lms for several N values.

Workflow 2
Obtain a concept with txt2img and swap it with * using img2img playing with different -f (strength) values
txt2img
"close-up of a man low poly" -s 50 -S<PHONE_NUMBER> -W 512 -H 512 -C 7.5 -A k_lms

img2img
"close-up of * low poly" -I outputs/preflight/000310.3319463269.png -S3319463269 for several -f values

txt2img
"Funky pop african man face figurine, product studio shot, on a white background, diffused lighting, centered" -s 50 -S<PHONE_NUMBER> -W 512 -H 512 -C 7.5 -A k_lms

inspired by https://publicprompts.art/funky-pop/
img2img
Funky pop * face figurine, product studio shot, on a white background, diffused lighting, centered -I outputs/preflight/000384.3231549968.png -S3231549968 for several -f values

(By the way, the training images in https://github.com/invoke-ai/InvokeAI/issues/517#issuecomment-1259486440 were personally created by the author of the comment, and the character  will appear on some project -which I find pretty cool- so you can use them to test TI but don't share them as training set)
I've also managed to train using a couple other concepts (hamburger, dog cartoon and myself), but results weren't that good. For people, I wonder if using some 'beautify' filter on the training set wold help, to make sure the face looks closer to a 3D character -very smooth, so it doesn't have to learn patterns inside the face, e.g. cheek colors, smiles, wrinkles, etc. and can focus on learning more generally the face shape).

All in all, I tend to prefer the "txt2img then img2img" workflow

@tildebyte But yeah, the word out there seems to be Dreambooth is better at styling. For example, this is a quick attempt (using txt2img && img2img) at your pictures above, and you can see how the style is a bit lost, and colors start to appear.

It may still be possible to do these things with Textual Inversion, but it may require a more complex workflow, while Dreambooth may be easier to use.
Still, if someone has successfully used free Colab for Dreambooth and has had success, you are encouraged to share it here! The more info we have (what Colab, obviously, but also number of training images, lighting, closeness...), the better.

Dreambooth is INCREDIBLE. No contest.

 It does run, and the result has some resemblance, but that's all.
Last attempt was with 50 training images and 150 reg images. --use_8bit_adam and --gradient_checkpointing as well for the free Colab.


Question
In the paper, the final curation step for the Pose Busters benchmark set brings the number of remaining complexes down from 428 to 308. It is described as: "Remove ligands which are within 5.0  A of any protein symmetry mate".

Is the code available for this final curation step? I am surprised by a couple of examples:

The 5S8I_2LY complex is not listed in the 308 remaining complexes (but is listed in the 428) but does not seem to violate the final condition.
The  7TE8_P0T, present in the list of 308, looks on the other hand like it could violate this condition (is the symmetry for the entire complex or only for the protein portion of the structure?).

Am I missing something? I may very well be misunderstanding the last condition. I downloaded the data from Zenodo and took the list of 308 ids from the paper directly.
RCSB examples

we might have to generate the crystal structure around the ligand using symmetry operations

Okay, thanks for the details, that's what I was missing.

7TE8_P0T is a ternary complex of two proteins whose binding is promoted by a small molecule. The biological unit includes the two proteins and the ligand between them. This structure passes the filter because both instances of P0T are far away from any protein symmetry mates.

Okay, wanted to make sure I knew what was going on.
Thanks for the quick reply, and once again for your work!


Fix #59 - Infinite scrolling

A change to HN caused this to no longer work correctly.
Changed from https://news.ycombinator.com/news# to https://news.ycombinator.com/news?p=#

I checked this, it still doesnt solve the issue.

can you take a look at it again

Thank you, it should work correctly in the next release!


Improve view method performance

Please merge #1542 first.

jenkins, test this please.

Jenkins CI test (for commit 5c874b934c3de777dff2b7a99141e802a9c5db5c, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins CI test (for commit 4e5b5e9bf2b6f64037d61f0f4ce64200f6e6accf, target branch master) succeeded!

LGTM.


sudo for linux

Well technically this is just leftover from the original repo and ./sshrd.sh ssh doesn't even work anymore with this fork IIRC.


Brightness percentage not reported correctly

There seems to be an issue with the reported brightness in Homebridge. Setting a brightness value from the Apple Home app does update the light bulb's brightness as expected, however, the reported brightness which is read back after by the Home app is always 100%. (The same occurs if the brightness is updated from the Homebridge dashboard.)
Plugin version: 2.5.2
Light bulb: TRADFRI bulb E27 WW clear 250lm
[20/09/2021, 17:25:45] [ZigBee] Updating state of device Bed with  {}
[20/09/2021, 17:25:45] [ZigBee] Updated state for device Bed is now  { state: 'ON', brightness_percent: 100, brightness: 38 }

Uhm, it seems a problem in the herdsman lib when reporting the percentage. I should problem use the non converted value and convert manually to %

This issue is still persisting in version 2.5.5.


Cannot run without .licensee.json file

~/random-project $ licensee
Cannot read ~/random-project/.licensee.json

Would be ideal to automatically create this file with licensee --init, or to specify in usage that it must be manually created first.

You can run without a configuration file using --license EXPRESSION and --whitelist LIST. See licensee --help.
I agree that at a very minimum, the error message should do more to help.

Thank you.


docs: use chunks link

I find the original link to be more appropriate

Although Dynamic Imports link articles are more comprehensive, the link content of Chunks is easier to understand named chunk.😀

I don't think so: the chunks one is an advanced section of the guide (under the hood) and most users don't need to know about chunks. Let's keep it this way, in the future, I might change the link to something more comprehensive.


github-controller crashloopbackoff

Expected Behavior
github controller should work on 0.7.1+
Actual Behavior
controlle crashes with error:
2019/07/08 06:00:31 Registering Components.
2019/07/08 06:00:31 Adding the GitHub Source controller.
2019/07/08 06:00:31 no kind is registered for the type v1beta1.Service in scheme "k8s.io/client-go/kubernetes/scheme/register.go:60

Steps to Reproduce the Problem
installed with following command:
kubectl apply --selector knative.dev/crd-install=true \
--filename https://github.com/knative/serving/releases/download/v0.7.1/serving-post-1.14.yaml \
--filename https://github.com/knative/serving/releases/download/v0.7.1/serving-beta-crds.yaml \
--filename https://github.com/knative/eventing/releases/download/v0.7.1/release.yaml \
--filename https://github.com/knative/eventing-contrib/releases/download/v0.7.1/github.yaml
kubectl apply --selector networking.knative.dev/certificate-provider!=cert-manager \
--filename https://github.com/knative/serving/releases/download/v0.7.1/serving-post-1.14.yaml \
--filename https://github.com/knative/serving/releases/download/v0.7.1/serving-beta-crds.yaml \
--filename https://github.com/knative/eventing/releases/download/v0.7.1/release.yaml \
--filename https://github.com/knative/eventing-contrib/releases/download/v0.7.1/github.yaml

Additional Info
running on minikube 1.2.0 wih kuberntes 1.15.0

+1
I miss this questions after I upgrade it to v0.7.1 from v0.7.0

+1
Would be good to have a release test for version issues like this :)

/assign @grantr
On Mon, Jul 15, 2019 at 11:38 AM Mark Vanderwiel<EMAIL_ADDRESS>wrote:

+1
Would be good to have a release test for version issues like this :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/knative/eventing-contrib/issues/481?email_source=notifications&email_token=AB4XEN6DAQFDCFMYZSXL2OTP7S7ZTA5CNFSM4H6YEPW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ6S5OY#issuecomment-511520443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4XEN6LJ3ETJYMFIBTBF73P7S7ZTANCNFSM4H6YEPWQ
.

--
Evan Anderson<EMAIL_ADDRESS>

@evankanderson @grantr I created https://github.com/knative/eventing-contrib/pull/489 to fix this issue. PTAL!


add examples of scaffolder action inputs to docs

Hey, I just made a Pull Request!
add examples of scaffolder action inputs to docs

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I am not sure if the e2e test failures are related to my changes.

Hoping a recent merge fixes it, lets see

I'm that fixed it. :)

Anything I can do here to help progress this?


Move an AWS course from books to courses section

What does this PR do?
Fix the mistake of #6752 PR
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Right addressed PR is #6752

@davorpa is there a way to make multiple PR ? if I do 2 PR in difference branch, the second one will hold the previous commit of the first PR (which is in difference branch)

Is discouraged to use incremental PRs sharing same branch (at least here, that we use squash as merge strategy).
Is more an idiomatic and organizative problem than ones of we can do. If you think that two commits should be in the same branch, maybe needs rename PR to grow it scope in the middle of both pushes

Is discouraged to use incremental PRs sharing same branch (at least here, that we use squash as merge strategy).
Is more an idiomatic and organizative problem than ones of we can do. If you think that two commits should be in the same branch, maybe needs rename PR to grow it scope in the middle of both pushes

I always made a new branch for new changes, but the new branch take the commits of the old branch (different branch names), i think i will figure out the solution
But can you tell me a advice to avoid that the new branch takes the commits of old branch ?

But can you tell me a advice to avoid that the new branch takes the commits of old branch ?

Easy! Is posible make a branch from any commit. Navigate from your repo timeline using any editor like VSCode or from web


refactor(Heading): update component to CSS Modules

Closes https://github.com/github/primer/issues/3716

Changelog

New

Changed

Refactor Heading to use CSS Modules

Removed

Rollout strategy

[x] Minor release

Looks like examples failure is the same thing you run into @jonrohan over in #4828 👀 Will wait for that to merge 👍


Parse user inputs to one of the trace event types supported by Invariant

The main goal of this PR is to change input given to the analyzer to be list of known trace events (Message, ToolCall and ToolOutput). This simplifies the analyzer code as now we know ahead of time what types can be expected (e.g. in selector), and if user input does not conform to the format it should not be parsed.
This PR still does not include two orthogonal changes that will be addressed separately:

Here we switch off function caching as it has problems with object methods like line.split(...) where caching would need to be aware of the current object value.
Monitor should expose function analyze(trace, new_actions) allowing it to become stateless and only raise errors that relate to new actions that have not yet been executed. This will remove the need for storing previous errors.

The code looks good and most tests seem to pass. I think there are two more things to fix:

The code in invariant.examples does not seem to be updated yet. These can be run as unit tests like so:
rye run python -m unittest discover -s invariant/examples -p "*_example.py"

We are changing tests/test_readme_examples.py, but the README does not seem to be changed. Lets make sure the code snippets there are in line with what we test in the readme tests. We may also want to make sure the Trace Format section is aligned with these new changes. I think the types there should be what users can expect to operate on in policy rules. Maybe this can exclude internal properties there.


Upgrade to latest version of pip (8.1.2)

I need to make the same change in st2-packages, etc. and make sure it works there end to end.

I'm also looking in upgrading pip and virtualenv used by the packages.
In the past we had issues with puppet based installer so we had to use an old version of virtualenv and pip.

Appears to be working fine on Ubuntu:

ubuntu@ip-172-31-47-162:~$ /opt/stackstorm/st2/bin/pip --version
pip 8.1.2 from /opt/stackstorm/st2/local/lib/python2.7/site-packages (python 2.7)
ubuntu@ip-172-31-47-162:~$ /opt/stackstorm/virtualenvs/st2/bin/pip --version
pip 8.1.2 from /opt/stackstorm/virtualenvs/st2/lib/python2.7/site-packages (python 2.7)


Add SonarCube to all repos

💡 Description
Per MGSS requirements, setup SonarCube Cloud and add to all repos.
⚔ Parent Epic / Related Tickets
No response

done


Upgrade jackson-jq from 0.0.11 to 1.0.0-preview.20191208

Upgrade JQ library to the latest version (used by JSON_JQ_TRANSFORM task).
Update the deprecated syntax of apply function.
Comparison between the 2 versions:
https://github.com/eiiches/jackson-jq/compare/0.0.11...1.0.0-preview.20191208

Pull Request Test Coverage Report for Build 4176

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
11 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.05%) to 69.386%

Files with Coverage Reduction
New Missed Lines
%

server/src/main/java/com/netflix/conductor/server/JerseyModule.java
1
95.45%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraEventHandlerDAO.java
2
68.42%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraMetadataDAO.java
8
56.48%

Totals

Change from base Build 4174:
-0.05%

Covered Lines:
12382

Relevant Lines:
17845

💛  - Coveralls

@elisherer The preview versions are advertised as stable and not ready for production use as per the documentation - https://github.com/eiiches/jackson-jq#branches-and-versioning.
We will wait for a stable release version before upgrading this library.


Have SiamuxAddr return an empty string if no host was specified in the settings

To avoid accidentally dialing localhost in case the port is set but the address isn't.

(nbd, but the commit prefix should be rhp here, not v2)


Submit a Cloud Definition

Do NOT copy/paste a definition from somewhere else. Read about the word you want to define and come up with your own definition. Copy/Paste submissions will be closed and not added.
Fill out the JSON with your submission:
    {
    "word": "Service Level Agreement",
    "content": "A contract between the cloud service provider and the client detailing expectations and performance from the selected services",
    "learn_more_URL":"https://www.techtarget.com/searchstorage/definition/cloud-storage-SLA",
    "tag":"service",
    "abbreviation": "SLA",
    "author_name":"codenamenuke",
    "author_link": "https://github.com/codenamenuke"
    }

Fill out the JSON below with the following.
Word (REQUIRED)
The word you are defining. Check this URL for all words we currently have.
Content (REQUIRED)
The definition. No more than 3 sentences.
learn more URL (REQUIRED)
Website where people can visit to learn more about the word.
tag (REQUIRED and select one)
Tech category the word fits in. Options:

compute
security
service
general
analytics
developer tool
web
networking
database
storage
devops
ai/ml
identity
iot
monitoring
cost management
diaster recovery

abbreviation (OPTIONAL)
If the word is commonly abbreviated, please provide it. For example, command line interface is often abbreviated as CLI.
author name (REQUIRED)
Your name.
author link (REQUIRED)
The URL you want your name to link to.

Good one, thank you for your contribution.


MVP: Add Student Test Cases

Hi @nihalramesh12629, can i contribute there?

@DanielVsh who r u

@DanielVsh who r u

I am an uni student and i need to contribute some github issue, if you dont mind i could take it

@DanielVsh nope


restructure spec and include metadata for docs

At the last community meeting, we talked about including the spec in our documentation. This can be done with a git submodule and copying over the files as a prebuild step in Docusaurus. In preparation for that, I've updated the structure of the spec and included additional metadata. No changes to the spec itself were made.
Here is are a few examples of what it looks like in the docs.

Signed-off-by: Michael Beemer<EMAIL_ADDRESS>

I think this is great.
As a side note, I like how the fact there's no delta in the specification.json proves no normative content has changed in PRs such as this.


Comments are only block style

Cmd-/ only uses the block comment style, even on single lines. This only occurs when using the Supercollider syntax and not other syntaxes. I've tried editing grammars/supercollider.cson for a fix but I'm really just shooting in the dark.
I'd much prefer that the commenting behavior be the same as in other syntaxes, where all lines that are commented out are done as single lines unless otherwise specified. I imagine this is an easy fix, I just don't know where to look.

I think it depends on which package you use for commenting.
https://atom.io/packages/search?q=comment
The grammar identifies what a comment looks like, and then the commenting package is what makes the editor do/undo it.
Different packages have their own language support:
https://github.com/javaguirre/comment/blob/master/lib/comment.json
https://github.com/whoisthecoon/atom-block-comment-plus/blob/master/lib/languages.js
Supercollider should identify as a javascript for purposes of comments. I don't really know how to make it do that:
https://github.com/crucialfelix/atom-supercollider/blob/master/grammars/supercollider.cson

It is definitely the comment package that determines this. Nothing I would want to do from inside atom-supercollider
This one promises to eventually support all languages:
https://atom.io/packages/multi-comment

Hello,
I know this issue is closed, but is there really no solution to this? It would be very helpful to be able to use line comments. Thanks!

Fixed in my fork.

I'm checking the solution from @woolgathering
I was under the impression that it has nothing to do with the language file.


python310Packages.aenum: switch to pytestCheckHook

Description of changes
Fix build (https://hydra.nixos.org/build/205560452)
Switching to pytestCheckHook is not the optimal solution but avoids to have to add decorators at multiple places.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

2 packages failed to build:

python311Packages.cassandra-driver
python311Packages.intellifire4py

23 packages built:

csvkit
home-assistant-component-tests.homematicip_cloud
home-assistant-component-tests.intellifire
home-assistant-component-tests.vulcan
home-assistant-component-tests.wallbox
onlykey-agent
onlykey-cli
python310Packages.aenum
python310Packages.agate-dbf
python310Packages.cassandra-driver
python310Packages.dbf
python310Packages.gremlinpython
python310Packages.homematicip
python310Packages.intellifire4py
python310Packages.vulcan-api
python310Packages.wallbox
python311Packages.aenum
python311Packages.agate-dbf
python311Packages.dbf
python311Packages.gremlinpython
python311Packages.homematicip
python311Packages.vulcan-api
python311Packages.wallbox


Installer for kbfs: INSTALL_ERROR stat /keybase.devel: no such file or directory

I'm trying to write a portfile for installing keybase through MacPorts but there seems to be some hardcoded paths around /usr/local. For instance, why can't we just use which to determine the path to the binary (or just search $PATH)?
00d Installer for cli: INSTALL_ERROR We need to be run with an absolute path to this executable to determine its full path.

After that error, the next (and more fatal) one occurs:
$ /opt/local/bin/keybase -d install
19:03:04.855647 ▶ [DEBU keybase ui.go:433] 001 Setting GPG_TTY to /dev/ttys001
19:03:04.855701 ▶ [DEBU keybase globals.go:193] 002 Keybase CLI 1.0.7-0
19:03:04.855719 ▶ [DEBU keybase globals.go:193] 003 - Built with go1.5.2
19:03:04.855729 ▶ [DEBU keybase globals.go:193] 004 - Visit https://keybase.io for more details
19:03:04.855746 ▶ [WARN keybase main.go:58] 005 Running in devel mode
19:03:04.855762 ▶ [DEBU keybase main.go:101] 006 + configureProcesses
19:03:04.855784 ▶ [DEBU keybase main.go:179] 007 + configureLogging
19:03:04.855797 ▶ [DEBU keybase main.go:188] 008 Disabling log forwarding
19:03:04.855807 ▶ [DEBU keybase main.go:181] 009 - configureLogging
19:03:04.855816 ▶ [DEBU keybase main.go:103] 00a - configureProcesses -> <nil>
19:03:04.855832 ▶ [DEBU keybase install_osx.go:333] 00b Installing components: [cli service kbfs]
19:03:04.855856 ▶ [DEBU keybase install_osx.go:336] 00c Checking command line
19:03:04.855871 ▶ [INFO keybase install_osx.go:375] 00d Checking /usr/local/bin/keybase (/opt/local/bin/keybase)
19:03:04.855930 ▶ [ERRO keybase standard.go:199] 00e Command line not installed properly (symlink /opt/local/bin/keybase /usr/local/bin/keybase: permission denied)
19:03:04.855958 ▶ [DEBU keybase install_osx.go:427] 00f Using binPath: /opt/local/bin/keybase
19:03:04.855968 ▶ [INFO keybase install_osx.go:428] 010 Checking service
19:03:04.861503 ▶ [INFO keybase install_osx.go:430] 011 Service: Installed (Action: None)
19:03:04.861541 ▶ [INFO keybase install_osx.go:449] 012 Checking KBFS
19:03:04.870690 ▶ [INFO keybase install_osx.go:451] 013 KBFS: Not Installed (Action: Install)
19:03:04.870770 ▶ [INFO keybase install_osx.go:453] 014 Installing KBFS
19:03:04.882182 ▶ [ERRO keybase standard.go:199] 015 Error installing KBFS: stat /keybase.devel: no such file or directory
19:03:04.882216 ▶ [INFO keybase cmd_install_osx.go:112] 016 Installer for cli: INSTALL_ERROR symlink /opt/local/bin/keybase /usr/local/bin/keybase: permission denied
19:03:04.882231 ▶ [INFO keybase cmd_install_osx.go:112] 017 Installer for service: OK
19:03:04.882238 ▶ [INFO keybase cmd_install_osx.go:112] 018 Installer for kbfs: INSTALL_ERROR stat /keybase.devel: no such file or directory
19:03:04.882248 ▶ [DEBU keybase globals.go:238] 019 Calling shutdown first time through
19:03:04.882261 ▶ [DEBU keybase login_state.go:887] 01a + Account "LoginState - Shutdown"
19:03:04.882311 ▶ [DEBU keybase login_state.go:889] 01b - Account "LoginState - Shutdown"
19:03:04.882344 ▶ [DEBU keybase globals.go:278] 01c exiting shutdown code=0; err=<nil>

When you do go build you should pass -tags production otherwise you will be building a devel version. (Thats why it says Running in devel mode.)
RE: using absolute path, in go, there isn't a great/safe cross platform way to get the full absolute path of the running executable, so currently we force you to use an absolute path when calling install. You can use which keybase install. In the future, we will support running without full absolute path but for now this is the safest way.
RE: KBFS error, you can safely ignore the KBFS install error. In custom or brew builds, KBFS isn't available and so will error. (KBFS is our encrypted filesystem and is not available for wide use quite yet.) I made a change a few weeks ago to not attempt KBFS install in these cases (where KBFS is unavailable), so the error message will go away in the next version 1.0.8.
Thanks for questions, etc, please lmk if you have any other questions, or issues.

Thanks for the info! Testing it out now then I'll push it up to MacPorts.


⚠ Fortnite has degraded performance

In 8b028f4, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1676 ms

Resolved: Fortnite performance has improved in 68214d0.


Run prettier fix

This should fix the CI errors.

i think it's still broken @Gusted do you know what causes it? https://github.com/pmndrs/racing-game/runs/2755044136?check_suite_focus=true

@Gusted do you know what causes it?

You forgot to add parentheses
-  const chassisBody = useStore(state => state.raycast.chassisBody)
-  const map = useStore(state => state.controls.map)
+  const chassisBody = useStore((state) => state.raycast.chassisBody)
+  const map = useStore((state) => state.controls.map)

i think it's still broken

It isn't. Maybe run yarn run prettier-fix before committing the code 😄. Or like I do it: Get notifications github actions failed. Knows right away to run the command and push another commit.

but thats the thing, the action crashes without giving a reason

but thats the thing, the action crashes without giving a reason

At the moment it list the files where there is a difference. I'm not sure if prettier has the ability to show the the exact diff.
But it should just error out as at this moment PR x could implement a change and didn't prettied and if I open another pr based off that commit it will prettied that file which could be totally unrelated to that PR.


support for optional params

Hi, thanks for writing this. I was surprised to find no standard sitemap support for SvelteKit and happy to find a possible solution so I don't have to roll my own.
However, I tried to use it and it seems that it doesn't support optional params properly. For example, take this hypothetical route:
/my-route/[[param1]]/[[param2]]
2 problems I encountered:

Putting the route in excludePatterns seems to get ignored and then it asks for paramValues to be set.
Error: Sitemap: paramValues not provided for: '/my-route/[[param1]]/[[param2]]'
Setting paramValues for the route incorrectly generates with trailing brackets after the supplied values:
https://somedomain/myroute/val1]/val2]

Thanks for the issue

Have you tried double escaping each of the square brackets?
e.g. .*\\[page=integer\\].* // e.g. routes containing [page=integer]–e.g. /blog/2

so in your case each would look like \\[\\[
It's weird looking but required for the JS regex. Lmk if you get this working or not.
I will add a test for this too to double check behavior.

Good catch. I'll need to update it to support optional params.

ok, yeah that's close but you gave me the clue I needed. the problem actually is that I have it in a group so I need to do this: ^/\\(group)\\/my-route.*

Thinking about behavior. Since I want to maintain a promise of "make it impossible to forget to add your routes", my thought is:

require paramValues to be provided, or to for the pattern to be excluded by the dev, even for optional params.

That way if a dev wants to, they exclude it. But won't forget it on accident. I'm trying to remove worries for devs.
Does that sound reasonable @dfoulger?
(I'll still need to update the path creation logic to create paths properly when the optional param syntax exists.)

I like the logging for missing route information, very helpful. Maybe some generated regex examples in the log for different ways of excluding or a link to examples could be even better. My first inclination was to just copy-paste the route in to exclude which of course failed cause I needed to escape the brackets and parens.
Also might be nice to be able to log those "errors" without failing based on a setting. I understand the automated test approach and will probably do that...but just thinking for production just log the error and render the sitemap w/o those paths.
If I had some logic to determine I was in a production setting could be something like this in the sitemap configuration:
failOnError: !production
If that was false, it would continue to log those messages but still render the rest of the sitemap.

Separate idea. Hmm. I'll have to think on this for a few reasons:

I prefer to leverage the framework (SvelteKit) where possible, rather than complicating the lib–i.e. devs can and should define SvelteKit's handleError hook to log any runtime errors, for their SvelteKit errors generally. It enables a dev to centralize their handling and logging of errors in a consistent way.
Devs currently have 2 ways to identify issues with their fetch of data for paramValues before they deploy: 1.) pre-rendering so the error occurs during build, or 2.) adding the example Playwright test if not using pre-rendering for the sitemap.
I can see the value you're suggesting (that 'Google at least sees the remaining subset of URLs'), but one could also argue that if the dev's database is not returning param values as expected, then this is a 500 error and should be caught and handled by handleError so that we're using SvelteKit handling for this and guaranteeing devs that how they've configured SvelteKit for their errors to be handled and logged is used--for consistency and predictability. Returning a partial sitemap to search engines, that is ostensibly valid (200 OK) but is actually not fully accurate, creates further unknowns in how search engines will choose to interpret this now smaller sitemap. In contrast, if crawlers encounter a 500 error, they know to check back later.

For these reasons, avoidance of the added complexity, and providing a clear path for devs rather than adding options that complicate best practices for the lib, my gut says it's best to not do this. But I'll give it more thought.

Locally, I have these working:

exclusionPattern working for one or more optional params,
the ability to specify paramValues for one optional param.

__
@dfoulger For your route mentioned in the OP that shows two optional parameters (i.e. /my-route/[[param1]]/[[param2]]), don't you encounter a build error due to a route conflict during npm run build?
When I set up similar like this:

SvelteKit gives an error during the build:
The "/optionals2/[[optionalA]]" and "/(marketing)/optionals2/[[optionalA]]/[[optionalB]]" routes conflict with each other from SvelteKit's prevent_conflicts method.

Yeah those routes would conflict. Svelte wouldn't be able to resolve which one to use since they both fall under the same /optionals/ root prefix. You would have to change one of them or add another component to the path to disambiguate.
I think I'm all set as far as regex goes for excludePatterns.
How will it work for the optional params?
Following my original /my-route/[[param1]]/[[param2]] example, I might do a paramValues config like this:
paramValues: {
	'/my-route/[[param1]]/[[param2]]': [
		[],
		['val1'],
		['val1', 'val2']
	]
}

and expect output like this:
<url>
  <loc>https://my-domain/my-route</loc>
</url>
<url>
  <loc>https://my-domain/my-route/val1</loc>
</url>
<url>
  <loc>https://my-domain/my-route/val1/val2</loc>
</url>

Yeah those routes would conflict. Svelte wouldn't be able to resolve which one to use since they both fall under the same /optionals/ root prefix. You would have to change one of them or add another component to the path to disambiguate.

Sorry, I'm still unclear how your example path is different from mine, to understand why you don't get a build error and I do. But maybe I'm not understanding your last sentence above.
(Given it's not a use a case I've had myself before.)
When I can get it to build, I can test out multiple optional params.
For now, I have only one optional param working locally, and can push that soon, but would like to sort this build error bc I think I have multi-optional params working otherwise; TBD.

To fix your build you just need to setup unique paths in group and out of group.
This is not allowed:
/path/[[opt1]]/[[opt1]]
/(group)/path/[[opt1]]/[[opt2]]

SvelteKit won't be able to (or at least currently doesn't want to) decide which of the /path/ routes to resolve since they are identical. I suppose they could say that a group takes priority over a non-group or vice-versa, but that would be a SvelteKit decision. I don't see mention of groups in the docs regarding route priority: https://kit.svelte.dev/docs/advanced-routing#sorting.
In my case the group/non-group paths don't collide. My example in the original comment doesn't use groups at all, but I'm glad that you are thinking about it for coverage.
You could do something like this for a test:
/path1/[[opt1]]/[[opt2]]
/(group)/path2/[[opt1]]/[[opt2]]

or this:
/path/1/[[opt1]]/[[opt2]]
/(group)/path/2/[[opt1]]/[[opt2]]

Thanks, I'm aware that'd conflict. But it's not that for me. (In my earlier errir message I cleaned up the group from one path aiming to focus on what I was trying to show, but accidentally left it in the other path, so I get why it looked that way.) Current error:
error during build:
Error: The "/(public)/optionals2/[[optionalA]]" and "/(public)/optionals2/[[optionalA]]/[[optionalB]]" routes conflict with each other

optionals2 doesn't have conflicts outside of it's directory.
To move forward, I'll remove routes intended to tests multi-optional params for now, ship single optional param support, and then come back to this.

Do you have +page.svelte in both the [[optionalA]] and [[optionalB]] directories? You only want one in [[optionalB]].

ah yes! like this

So it's one actual route and both optionalA and optionalB params must be specified simultaneously.
I'll make my test files like this to proceed:

Also, I just pushed v.0.14.9 which adds support and tests for specifying a single optional parameter.

well, the logic in the files would look for A or B to do whatever is needed but neither is required so have some default internal logic for when they are not supplied.

ahh ok I see that now playing with the new route.
Then I need to make more changes. That would imply that the stem path and all length of paths need to be handled--i.e.
/optionals
/optionals/[[optionalA]]
/optionals/[[optionalA]]/[[optionalB]]

I need to figure out a way to do that that isn't messy.
Currently handled:

exclusion of any number of optional params given it's a regex pattern.
specifying data for a single optional param (but the stem path of /optionals before the optional param appears isn't included by itself yet)

This won't be easy ngl b/c it means exclusions need to be handled at all possible levels as well.

this is throwing a wrench in...but for thought...
For the excludePatterns, regex provides a ton of flexibility...however I wonder if it isn't clearer and good enough to cover all cases to just exact exclude on the route params. Since you aren't excluding random incoming route params, just limiting what routes/params you want to generate out to the sitemap.

But that would prevent the ability to exclude certain groups like (authenticated) or stems like /dashboard though, which is useful.
I think I might be able to achieve this without too much work now: After collecting the file paths from disk, if a path contains an optional param, I'll process it to create paths for each versions of the route. Then the rest of the handling should remain the same. Could actually be easy

Multiple optional parameters is now supported, with tests, in 0.14.10 on NPM.
Instructions in the README
Let me know if you run into any issues and you can reopen this. Thanks for mentioning this one!


Add alpha blending, relative luminance, lab inits

Blending two colors with alpha compositing -
https://en.wikipedia.org/wiki/Alpha_compositing
Create contrasting color based on relative luminance
Convenient method for LAB color space

Great, thank you!


Loop upper/lower bound issue

Fix the underlying loop count identification bug that was exposed by a memop issue that hit on the web. The fix was suggested by Kount: given two possibly symbols that could be the bound for the loop count, if one symbol is itself a relative bound of the other, choose the other.

@LouisLaf @rajatd

Looks good to me.


feat: add health.lua and basic healthchecks

Adds some basic healthchecks that can be run with :checkhealth.
Here is an example of the output of :checkhealth blink.cmp:

Thanks!


Redirect bugfix

tiny but important fix:
diff --git a/bottle.py b/bottle.py
index 535ddd3..359ceaf 100644
--- a/bottle.py
+++ b/bottle.py
@@ -2418,7 +2418,7 @@ def redirect(url, code=None):
     res = response.copy(cls=HTTPResponse)
     res.status = code
     res.body = ""
-    res.set_header('Location', urljoin(request.url, url))
+    res.set_header('Location', url)
     raise res

Ideally meet the two use cases! Needs test writes test to pass by the two conditions.

I do not think that tests may be writed.
because problem displayed -- when we using complex Front-End (example as Mozilla Firefox.. but NOT curl) or complex system of back-end wrappers.
see next :)
easy example
see HTTP header Alt-Svc: ...
if back-end will use this HTTP header:
bottle.response.set_header('Alt-Svc', 'h2=":443"')

that Mozilla Firefox (v37++) will use 443-port (and https:// protocol) , instead of 80-port (and instead of http:// protocol)
BUT on front-end side (in User Interface of Mozilla Firefox, AND for Javascript-Files, and for CSS-files, and for GUI URL Field) -- we will see http:// protocol (not https:// protocol).
this easy example show that back-end and front-end -- may has different information about URL.
BUT when back-end will do redirect(...) -- it NOT must to force using backend information.

Uhm... http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.30
"The Location response-header field is used to redirect the recipient to a location other than the Request-URI for completion of the request or identification of a new resource. For 201 (Created) responses, the Location is that of the new resource which was created by the request. For 3xx responses, the location SHOULD indicate the server's preferred URI for automatic redirection to the resource. The field value consists of a single absolute URI."

@defnull , OK ! if relative-URL-path (in redirect) is conflicted with RFC.. I think will be fairly (rightly) to reject this pull-request #747 (and same issue:#748).
I sure that in real life (in modern time) -- relative-URL may be used (and it used! yes) in http-redirects. because relative-URL in http-redirects purposely not prone to problems .
but conflicting with RFC -- it strong reason too.
I will respect any your decision about accept-or-reject relative http-redirects in Bottle!

and what about adding new keyword-argument in redirect(...) ?
example:
def redirect(url, code=None, use_relative=None):
    if use_relative is None:
        use_relative = False

    # ...
    # ...
    #...

IMHO, that's will be great! :)

@defnull , very very sorry for my many comments :( .. but yet comment.
http://tools.ietf.org/html/rfc7231#section-7.1.2
*** begin of quote ***
The "Location" header field is used in some responses to refer to a
specific resource in relation to the response.  The type of
relationship is defined by the combination of request method and
status code semantics.
 Location = URI-reference

The field value consists of a single URI-reference.  When it has the
form of a relative reference ([RFC3986], Section 4.2), the final
value is computed by resolving it against the effective request URI
([RFC3986], Section 5).
*** end of quote ***

Oh, okay. RFC7231 obsoletes RFC2616 and is more recent. In that case the change should really be discussed (perhaps in a new issue). I'll open one, just a second.

Please merge this, it's still causing pain :cry:


[AKS] az aks create/az aks nodepool add: Remove restrictions on optional values for option --zones

Related command

az aks create
az aks nodepool add

Description

Remove restrictions on candidate values (previously only "1", "2", "3" are allowed) of option --zones in commands az aks create and az aks nodepool add.
Testing Guide

History Notes

[AKS] az aks create: Remove restrictions on optional values for option --zones
[AKS] az aks nodepool add: Remove restrictions on optional values for option --zones

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.

AKS

Please rerun related tests for this change.

There are 3 live test cases related to the option, queued live test pipeline to rerun these cases.

test_aks_availability_zones
test_aks_availability_zones_msi
test_aks_enable_utlra_ssd

Since the restriction will be removed from the core module in the coming new version of CLI, it is not necessary to make this change in the AKS module.


Unexpected behavior in realtime events in Flutter android app

Bug report

[+] I confirm this is a bug with Supabase, not with my own application.
[+] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Unexpected behavior in flutter android app. in some cases realtime events on update not fired. in some cases fired with wrong data
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

create table with rls policy

create table
  public.test (
    id bigint generated by default as identity,
    some_int bigint null,
    constraint test_pkey primary key (id)
  ) tablespace pg_default;

  BEGIN;
  ALTER POLICY "Enable read access for all users" ON "public"."test" USING ((some_int >= 1));
COMMIT;

put rows with some_int - [0, 1, 2]

run flutter app with your secrets

Future<void> main() async {
  final supabase = await Supabase.initialize(
    url: supabaseUrl,
    anonKey: supabaseAnonKey,
  );
  runApp(MyApp(supabase: supabase));
}

class MyApp extends StatelessWidget {

  final Supabase supabase;
  const MyApp( {super.key, required this.supabase});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: StreamBuilder<List<int>>(
        stream: getSomeInts(),
        initialData: const [],
        builder: (context, snapshot) {
          return const FlutterLogo();
        }
      )
    );
  }

  Stream<List<int>> getSomeInts() {
    return supabase.client.from('test').stream(primaryKey: ['id']).map((event) {
      log("event.toString():$event");
      return event.map((e) => e['some_int'] as int).toList();
    });
  }
}

you will see in log
event.toString():[{id: 2, some_int: 1}, {id: 3, some_int: 2}]

change in supbase some_int from 0 -> 3
you will see in log

event.toString():[{id: 2, some_int: 1}, {id: 3, some_int: 2}, {id: 1, some_int: 3}]

change in supbase some_int from 3 -> 0
log not fires

Expected behavior
event fires

change in supbase some_int from 1 -> 3
you will see in log

event.toString():[{id: 2, some_int: 3}, {id: 3, some_int: 2}, {id: 1, some_int: 3}]

its wrong data
Expected behavior
correct data [0,3,2]
Screenshots

System information

OS: Windows 10

Additional context
flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.13.6, on Microsoft Windows [Version 10.0.19045.2251]
Android Studio (version 2022.3)

version of supabse in pubspec of flutter

supabase_flutter: ^1.10.23

device

android emulator api33_x86_64

up

This is the expected behavior currently.
Because you have set the RLS so that some_int less than 1 is not readable, the update on step 5 is not readable from the client.
We are going to need to wait for the fix on the server side for events like this to be sent to the client.
Related https://github.com/supabase/walrus/issues/64


Unable to share vehicle - Request timed out

Hello everyone,
Just like everyone else I'm unique and want to have my very own specific setup to integrate my tesla into home assistant :smile:.
What I've got working for now is:

Running the container
Forwarding tesla.myowndevedsedomain.com to the tesla-http-proxy-docker container on port 8099 (Using NPM (Nginx Proxy Manager)) from my NPM host to a separate host running the container.
Completing steps 1 - 3:

However when I try to go through step 4, I run into the following issue on the mobile app (after clicking "Accept" when it asks me if I want to allow access to my Tesla)

On one hand I'd be interested in a direct solution for my problem. But I'd probably be able to figure this out if someone could help me understand where things are getting stuck.

Is this somewhere in a connection from Tesla to me?
Is this my public key not being correct?
Etc..

I also checked my NPM logs (which with tail -f * show any request being made to my domain) but this only shows the following:
==> proxy-host-28_access.log <==
[******* +0000] - - 206 - GET https tesla.*****.******** "/.well-known/appspecific/com.tesla.3p.public-key.pem" [Client ************] [Length ******] [Gzip -] [Sent-to ************] "okhttp/4.11.0" "-"

So I retried things this morning and now it worked :).
One thing that confused me is what the "proxy url" configuration setting was being used for. But I now figured out this is because homeassistant needs to connect to the tesla http proxy over an HTTPS connection for which we need a self signed certificate.
Hence setting the proxy url to for example: pi.deveinternal will generate an HTTPS certificate for that url. You should also then forward pi.deveinternal in your DNS server (e.g. your ubiquiti router or your adguard/pihole) to the host that's running the tesla http proxy.
That way home assistant can securely connect (over https) to the tesla http proxy docker container.
I think it would be useful to add some background information on this in the tutorial (in like a hidden dropdown that people can click open) to explain why they are configuring things they are.
Anyways thanks for everything, it works now :).

Ok I've also solved that. Apparently somewhere in the process either a refresh token expired or I copied the wrong public key. Anyway, I've deleted everything in the http proxy data directory and then reran the steps. Now it works :smile:

Excellent thanks for sharing!


Modularize cli.py

Goal is to modularize cli.py because it is getting huge...
propsed solution is to create separate sub-dir for each command and store command pre file.
envcloak/
├── __init__.py
├── cli.py                 # Entry point for Click commands
├── commands/
│   ├── __init__.py
│   ├── encrypt.py
│   ├── decrypt.py
│   ├── generate.py
│   ├── compare.py
│   └── rotate_keys.py
...

File names are only a suggestion, but they are a good reflection of the functionality they contain.
We should move out whole logic outside of cli.py and just import and add_command from import.
So it should look like this:
import click
from envcloak.commands.encrypt import encrypt
from envcloak.commands.decrypt import decrypt
from envcloak.commands.generate_key import generate_key, generate_key_from_password
from envcloak.commands.compare import compare
from envcloak.commands.rotate_keys import rotate_keys

@click.group()
@click.version_option(prog_name="EnvCloak")
def main():
    """EnvCloak: Securely manage encrypted environment variables."""

# Register commands
main.add_command(encrypt)
main.add_command(decrypt)
main.add_command(generate_key)
main.add_command(generate_key_from_password)
main.add_command(compare)
main.add_command(rotate_keys)

if __name__ == "__main__":
    main()

Ennsure that tests are passing 😆

TBD after #7 and #8.


Fix: Resolved conflicts and fixed typos in illustrations subcategory

Fixes Issue
Closes #588

Changes proposed
I resolved conflicts and fixed typos in illustrations subcategory

Screenshots

Note to reviewers

💃💃

lgtm

💃💃

@rupali-codes, we need a force push.


Inconsistent handling of file paths in 'gpt-engineer/main.py'

The code constructs file paths using different approaches, which can lead to inconsistency and potential errors. For example, the memory_path and workspace_path variables are created using Path objects, while the identity path is constructed using Path(os.path.curdir). It is advisable to use a consistent method, either Path or os.path, throughout the codebase to maintain clarity and prevent issues related to file path manipulation.

solved ✅