WriteDescriptorSetBuilder's buffer_info is the wrong type

The builder for WriteDescriptorSet seems to be generating incorrectly. The following code fails in the SDK validation layer:
let buffer_info: vk::DescriptorBufferInfo = ...

let write = vk::WriteDescriptorSet::builder()
                ...
                .buffer_info(&[buffer_info[)
                .build();

...
device.update_descriptor_sets(&[writes], &vec![]);

It's hard to explain the validation error because I don't really understand what's going on, but it seems to have to do with the type of pBufferInfo, which is being translated into a slice argument and written as an array even though it's meant to be a single pointer. If I run the above code while running the api dump layer, I get something like:
            pBufferInfo:                    const VkDescriptorBufferInfo* = 0x7ffeec258ff0
                pBufferInfo[0]:                 const VkDescriptorBufferInfo = 0x7ffeec258ff0:
                    buffer:                         VkBuffer = 0x7fff903a6780
                    offset:                         VkDeviceSize =<PHONE_NUMBER>71088
                    range:                          VkDeviceSize =<PHONE_NUMBER>30592

Note that the values are garbage. If I instead set p_buffer_info directly:
let mut write = ...
write.p_buffer_info = &buffer_info

Then the code runs, and the api dump looks correct, too:
pBufferInfo:                    const VkDescriptorBufferInfo* = 0x7ffee3f4ef50

Thanks in advance! The bindings are buttery smooth otherwise.

I also just noticed that the builder is missing a descriptor_count method. That might be related to it thinking that buffer_info is an array.

buffer_info is in fact an array:

descriptorCount is the number of descriptors to update (the number of elements in pImageInfo, pBufferInfo, or pTexelBufferView

Here's how it's implemented:
    pub fn buffer_info(
        mut self,
        buffer_info: &'a [DescriptorBufferInfo],
    ) -> WriteDescriptorSetBuilder<'a> {
        self.inner.descriptor_count = buffer_info.len() as _;
        self.inner.p_buffer_info = buffer_info.as_ptr();
        self
    }

Are you calling build() by chance? The build method is not safe due to discarding the lifetimes. It's possible to create pointers to the stack without the unsafe  keyword here.

Ah, yeah, the issue is with build. Identical to #192. Thanks!

To clarify:

The build method is not safe due to discarding the lifetimes.

Calling build is always safe; that is why it is not marked unsafe. Inspecting the generated code will reveal it doesn't contain any unsafe blocks even transitively, so it would be pretty difficult for it to be unsafe.

It's possible to create pointers to the stack without the unsafe keyword here.

It's possible to do that in any safe rust code without any particular effort. That's what happens any time you use &foo to initialize a value of raw pointer type. Constructing raw pointers to anywhere is harmless; you can even do it from arbitrary usizes. It's dereferencing them, or passing them to an interface that may dereference them, that's unsafe.

I know that this has been discussed elsewhere, but what would be the (breaking) API change to prevent people running into this problem? For example, could the builder objects actually take ownership of the structs/vecs passed into them?

Unfortunately it's very difficult to do better than the status quo. For example, taking ownership doesn't work because you may not have ownership of the data in question to begin with, e.g. if it's persistent storage that gets reused in the next frame.

Do you have an example of that? I couldn't find one in my code. Correct me if I'm misunderstanding, but the handles are Copyable. So in the example above, the VkBuffer isn't being free'd, and neither is the buffer itself; just the array of handle objects. The builder could easily hold on to a vec of those.

Do you have an example of that?

Not off the top of my head. I mostly pass slices of array literals, which could in principle be replaced by passing owned arrays. Still, it's not in the spirit of ash to make it difficult to execute an efficient pattern that you could reasonably apply in C/C++.

The builder could easily hold on to a vec of those.

Holding a Vec internally means forcing heap allocations on users, which in an extremely performance-oriented API like Vulkan is a dubious proposition. A higher-level library that trades off some performance in favor of safety could more reasonably do such a thing; wgpu might be of interest if that sounds appealing.

Calling build is always safe; that is why it is not marked unsafe.

@Ralith We seem to discuss this every few months.  The reality is that this issue keeps coming up and people continue to foot-gun themselves with it.
Casting away the lifetime of a reference by converting it to a pointer is an inherently unsafe thing to do regardless of the keyword. The object in question is passed to native code which hides the fact that its de-referenced in an unsafe way. The existence of the lifetime on the builder structs gives a false sense of safety here.
Calling the native vulkan function requires unsafe anyway and marking build as unsafe would be a very clear indication the operation has an unsafe side effect.

The reality is that this issue keeps coming up and people continue to foot-gun themselves with it.

Whether something is particularly error-prone is almost completely orthogonal to whether it's safe or not. I certainly agree that this is an issue, but claiming incorrectly that build is unsafe only serves to confuse matters.

Casting away the lifetime of a reference by converting it to a pointer is an inherently unsafe thing to do regardless of the keyword.

When discussing the correctness of Rust code, "unsafe" is generally understood to have a specific concrete meaning. Disregarding that meaning makes it difficult to sensibly discuss correctness.

The object in question is passed to native code which hides the fact that its de-referenced in an unsafe way.

Dereferencing a raw pointer is always unsafe. I disagree that a reasonable reader would ever assume that a raw pointer passed to an unsafe function, in lieu of information to the contrary, would not be dereferenced--to say nothing of that this is probably the least subtle correctness issue affecting typical Vulkan calls.

marking build as unsafe would be a very clear indication the operation has an unsafe side effect.

build has no side effects, much less unsafe ones.

Whether something is particularly error-prone is almost completely orthogonal to whether it's safe or not.

Not when it creates memory safety issues such as pointers to the stack.

When discussing the correctness of Rust code, "unsafe" is generally understood to have a specific concrete meaning. Disregarding that meaning makes it difficult to sensibly discuss correctness.

Yes and allowing pointers to the stack to escape a function meets that criteria of "unsafe". You would never accept pure rust code to effectively transmute lifetimes away like this and not consider it unsafe.


Sorting event listener is not called when we use dnd/Sortable with antd table instead drag start and drag end is called when we try to sort the column

Sorting event listener is not called when we use dnd/Sortable with antd table instead drag start and drag end is called when we try to sort the column
@clauderic thank you in advance

This isn't an issue with @dnd-kit. Please re-open an issue in the ant-design repository or the react-component/table repository (which is what ant design uses under the hood to power its table component)


chore: brillig IR printer (WIP, kw/x86 base)

Description
Print brillig IR with labels. WIP
PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@kevaundray how do you feel about something like this getting in? Feels right to me - we usually want to trace at a higher level than raw brillig, and it could later be expanded to even print hints about the source code things come from etc

@kevaundray how do you feel about something like this getting in? Feels right to me - we usually want to trace at a higher level than raw brillig, and it could later be expanded to even print hints about the source code things come from etc

Yep this make sense -- I reverted initially because the call instruction PR had nothing to do with printing, ie we'd want this change to go straight into master -- it will be faster to merge than the call instructions PR.
I'm not sure about having debug_show being put in functions vs having a method which operates over the BrilligArtifact and prints it. The latter means there is only one place I need to worry about when I need to configure printing the IR. What you lose is granularity, ie in the former you can print an opcode and then print something related to the function in that same function and they will be interleaved. If you want to add hints or other information, I can see the former adding noise to functions depending on how much information you want to add. In the latter approach, you would need to store this information in the BrilligArtifact as a Map and then when you are printing the Brillig IR, you access the maps to pretty-print more information.
The reason why I'm not firmly in the "print the IR at once" boat, is because these methods are only in brillig_ir right now and if I were to add a new method which pushed a new instruction I think it would be easy to figure out that it would need a debug_show at the top.
What are your thoughts?

I guess one thing is that it would not be hard right now to make a debug print statement in the brillig SSA layer and get more info, whereas otherwise it would be a harder thing to trace. I'm open, I like this pound for pound though I think because this is scrappy enough to maintain easily and also general because you can emit prints in-between from the SSA layer, something harder to do one-shot

Also my reasoning for why the call branch is that this is when things start to get complicated - but open to backport, I guess slices wants it too. I also thought calls was close with Alvaro's recent fixing of arrays + wanted to peek at it

But I suppose the same goes for dynamic slices. TBH if I worked on both I'd just copy this as it's very easy little glue code

Downside of this approach is the scrappiness does show - STOP gets printed as RETURN before being patched, if you want to print something higher level and not print in the lower level utility functions its tricky, etc

Also my reasoning for why the call branch is that this is when things start to get complicated - but open to backport, I guess slices wants it too. I also thought calls was close with Alvaro's recent fixing of arrays + wanted to peek at it

I'd prefer to merge into master so that branches are as self contained for a feature as possible. It makes writing code harder, but it makes reviewing code easier imo which is what I like to optimize for!


[beta] adds React Native docs

published temporally there

Coverage remained the same at 79.584% when pulling 356df48beeb12b77fe3d9bb63baf61e5bf151570 on huafu:docs-react-native into 943d7828d0b5ca98d10619cdb3a4446291a2caa8 on kulshekhar:beta.


Apveyor: Update pacman DB before installing gtk

Ah perfect, thanks!

I think that updating Pacman DB is temporary solution, and better removed when apveyor image get cairo 1.15.12 by default.

Just note: while travis for #184 was good, merging have error https://travis-ci.org/gtk-rs/cairo/jobs/366794153#L707, bad if this random error.

All CI passed here

Thanks!


Check for broken hyperlinks

We should use a tool for validating that all hyperlinks are still valid.
A quick google search found this: https://github.com/tcort/markdown-link-check which actually also has Github actions integration: https://github.com/gaurav-nelson/github-action-markdown-link-check
There may be other, more suited tools out there - this was just the first one I found.

This works pretty good, added it and fixed a bunch of broken links.

Awesome. We can probably also add this to airnode later too

Figured I might as well do it now or it might get forgotten
Added here: https://github.com/api3dao/airnode/commit/406619522b7413dbe8245b896d82222419197393


“How IPFS Works” interactive explainer

Objective
Create an interactive explainer on how IPFS works within the context of a real-world use case (e.g. friends/family sharing vacation photos in an offline-friendly app on the plane trip home). Explainer must include the option to turn on/off “dev mode”, enabling the user to either go through the journey from a conceptual level or augment the concepts with more in-depth technical examples (ideally as functional as possible within the context of a website demo). Areas explored must include:

How files are hashed and turned into blocks, including de-duplication
How files are queried and shared between users (content addressing, DAG lookup)
How these p2p concepts are incorporated into an offline-first, privacy-preserving app context (featuring a reusable IPFS stack diagram)
Any potential intersection points with Filecoin, if appropriate
Relevant outbound docs/etc links throughout for those who wish to dig deeper

PRD
https://docs.google.com/document/d/1xujTTf0Y6viPp7P-JTzLGaAdJ6wwCxClXUjji9hq4xU/edit

Update: Per dev onboarding meeting 6 April, will re-frame this using the to-be-canonical example of NFT storage, and create a proposal for implementation in Q2.


[SPARK-50211][PYTHON][CONNECT][TESTS] Skip FeatureTests if torch is not installed

What changes were proposed in this pull request?
This PR proposes to skip FeatureTests if torch is not installed
Why are the changes needed?
If tourch isn't available, we should run the tests FeatureTests . That's an optional dependency.
To fix Python 3.13 build
https://github.com/apache/spark/actions/runs/11654255828/job/32447601706
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Manually tested.
Was this patch authored or co-authored using generative AI tooling?
No

Merged to master. Thank you, @HyukjinKwon .


Provide a function that calculates what needs to be changed in the environment for it to act like it was activated

I.e., update PATH and VIRTUAL_ENV. Return as a dict so that it can be | to os.environ; os.environ | activate().
Probably want something like:
def activate(env_dir=pathlib.PurePath(sys.executable).parent.parent, environment=os.environ) -> dict[str, str]: ...

This should be platform-agnostic (i.e., make it work on Windows).

The activation scripts unset PYTHONHOME, so that means either creating a function that takes in the environment and returns a copy with the appropriate changes, or change what the function does and is named, e.g. add_activation().

https://github.com/python/cpython/blob/main/Lib/venv/scripts/posix/activate.fish for what gets (un)set.


Constrain CupertinoContextMenu animation to safe area

Before
https://github.com/user-attachments/assets/84b86117-3820-4ac6-a204-81b123e5e543
After
https://github.com/user-attachments/assets/f3374e69-ea7a-4bbf-9c92-669d0520c922
Fixes https://github.com/flutter/flutter/issues/122951.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

unrelated but why can't we have https://pub.dev/packages/super_context_menu merged into the flutter framework itself? it shows the native context menu and it works flawlessly.

unrelated but why can't we have https://pub.dev/packages/super_context_menu merged into the flutter framework itself? it shows the native context menu and it works flawlessly.

My understanding is bringing something into the core framework means a guarantee of support for it from the flutter team, which I am not sure can be guaranteed.

unrelated but why can't we have https://pub.dev/packages/super_context_menu merged into the flutter framework itself? it shows the native context menu and it works flawlessly.

General answer for why we don't import X package that solves Y issue we have is that usually there's some complexity in it or implementation that we don't want to have in the core framework. Or it needs increased testing or design changes that requires a significant investment. For that package, it would add a Rust dependency which we wouldn't want to have.


add pre_exec job parameter to allow for setup commands

This allows to specify additional commands to be run before the actual "%(python)s -m distributed.cli.dask_worker"-call that starts the worker. This can be used to set up/source a virtual environment (or conda, mamba, ...), load a module, or whatever setup is needed.
Example usage for a venv:
pre_exec = ['cd ~/projects/dask-jobqueue/example', 'source venv/bin/activate']
cluster = HTCondorCluster(cores=1, memory="2GB", disk="4GB", log_directory = 'logs',
                          pre_exec=pre_exec, python='python3')

Hi @guillaumeeb ,
thanks for looking into this! I also first thought that env_extra might work, but unfortunately this is not sufficient: At least for HTCondor this only lets you specify environment variables (the function parsing that is defined here).
So something like
cluster = HTCondorCluster(cores=1, memory="2GB", disk="4GB", log_directory = 'logs',
                          env_extra=['TESTVAR=1', 'export TESTVAR2=2'],
                          )

Generates this in an HTCondor submission file:
Environment = "TESTVAR=1 TESTVAR2=2"

But things like this
env_extra=['cd ~/projects/dask-jobqueue/example', 'source venv/bin/activate'],

simply get swallowed and do not appear anywhere.

Addendum:
You are right that this works for e.g. SLURM. There all lines are simply added to the script (which is much simpler than the HTCondor submit files).
Do you consider this a "bug" in HTCondorCluster? I could also take all lines that are not selected by env_lines_to_dict() (see link above) and prepend those to the command instead of introducing a new parameter. This would be a bit intransparent to the user, though, because some commands are prepended and some are put into the HTCondor Environment parameter.

The way HTCondor submit files work I think it would be rather confusing to allow env_extra to include arbitrary commands (frankly I don't think that makes sense for any cluster). I think having some kind of helper script argument like the one proposed here would make sense if it's necessary, but I'm also confused about the purpose. As implemented sys.executable sets the Python interpreter to use which should be the same as the one in your virtual environment. Is there a particular reason that was not sufficient?

A common use case that I've had is to use env_extra to load environment modules to get the computing environment that I need for a particular set of tasks:
GROMACS_gpu_cluster = CustomSLURMCluster(
    name="GROMACS_gpu_cluster",
    nodes=2,
    mpi_mode=True,
    fork_mpi=True,
    queue_type="gpus",
    maximum_jobs=5,
    env_extra=[
        "module load Intel",
        "module load IntelMPI",
        "module load GROMACS",
        "module load dask",
        "module load jobqueue_features",
    ],
)

The code here is customised but the use case is hopefully clear.

The way HTCondor submit files work I think it would be rather confusing to allow env_extra to include arbitrary commands (frankly I don't think that makes sense for any cluster).

What I got from the discussion so far is that the parameter rather is a misnomer.

... but I'm also confused about the purpose. As implemented sys.executable sets the Python interpreter to use which should be the same as the one in your virtual environment.

Unfortunately that's not sufficient, because that's a symlink (venv/bin/python3 -> /usr/bin/python3) and sys.executable is set to the target. Furthermore we regularly need to load environment modules like @ocaisa described and also load software/packages from other sources (like cvmfs).

Either way would be fine for me. I guess it's up to the maintainers to decide here.

Thanks  @riedel for the feedback!

differ from slurm in that they are no scripts themselves but have their own syntax which makes things a bit different

Right, this is why I prepend commands in the Arguments = parameter to achieve what the scripts do for e.g. SLURM. Here the commands are separated by semicolon instead of newlines, but the logic is the same, isn't it?
Anyway, I would be happy to have a solution with a "proper" script (=with statements separated with newlines) to run in the job, because currently it is a quoting nightmare.
That being said, I don't fully get your proposal, yet:

Do you mean condor_submit in the script instead of condor_q?
Is EOF supposed to go before the fi?
What is submit command supposed to be? The actual python call starting the worker? (%(python)s -m distributed.cli.dask_worker ...)
I don't get where the "script that submits itself"-magic would happen.

Hi everyone,
We all agree this is important for any queuing system to be able to run some commands before starting the Dask worker.
Currently, this is supposed to be done using env_extra as stated in the doc:

Other commands to add to script before launching worker.

For HTCondor, the command only add environment variables, probably because of a misunderstanding due to the kwarg name, and also because HTCondor is a bit specific.
So I will probably reject this as is, but two things could be done:

propose and implement a better name for env_extra, with deprecation and everything.
Correct the behavior of HTCondorCluster. I'd say this would be by moving the current behavior of env_extra to another kwarg specific to HTCondor, and in the same time implementing env_extra as it should, with any of your proposal.

Any thoughts?

propose and implement a better name for env_extra, with deprecation and everything.

#393 proposes job_script_extra, see also #323

propose and implement a better name for env_extra, with deprecation and everything.

#323 proposes job_script_extra

Sounds good.

I'd say this would be by moving the current behavior of env_extra to another kwarg specific to HTCondor

I think that's not even necessary, because you can already use job_extra to inject any key-value pair to the submit file.

Wow, good catch for both issues @riedel, I did not remember those. So I'm closing this PR in favor of those two issues we need to fix.


Implement new profile page UI (https://hot.opensauced.pizza/{username})

What is needed?

Profile Image and Username
Contribution Activity feed
Contribution submission form

Design

The profile page will now be the default page for app.opensauced.pizza


Is it possible to set value for jsonpaths

just as on com.jayway.jsonpath.DocumentContext.set(), are there any implementations for settings a value for given jsonpath ?
   /**
     * Set the value a the given path
     *
     * @param path      path to set
     * @param newValue  new value
     * @return a document context
     */
    DocumentContext set(JsonPath path, Object newValue);

There is not, but I can add it.  Probably this weekend.

@KittyMac sounds great, looking forward.

@ilkerc v0.4.0 is up and now supports this.

This can be more easily implemented like this:
let oldJson = #"{"someValue": ["elem1", "elem2", "elem3"]}"#
let newJson: String? = oldJson.query(map: "$.someValue", {_ in
    return ["elem4", "elem5"]
} ) { root in
    return root.description
}
XCTAssertEqual(newJson, #"{"someValue":["elem4","elem5"]}"#)

The return value from your map callback is auto-converted to a JsonElement for you, so you can return any json-like-thing and it should handle it.  This is more costly performance-wise (since it needs to use as? to determine what the thing is at runtime), but should be fine unless discover it is too slow fo your use case.
Spanker's performance comes from making a hierarchy of JsonElements who reference the memory in the original storage (like a bunch of SubStrings referencing a single String, these lightweight objects are cheap to make so long as they don't need to make copies). This is the reason for the .parsed(), as the source data is guaranteed to be live inside of it.
If you wanted to do the above more performantly, then we want to move the creation of the JsonElement outside of the map.  Like this:
let oldJson = #"{"someValue": ["elem1", "elem2", "elem3"]}"#
let replacementElement = JsonElement(unknown: ["elem4", "elem5"])
let newJson: String? = oldJson.query(map: "$.someValue", {_ in
    return replacementElement
} ) { root in
    return root.description
}
XCTAssertEqual(newJson, #"{"someValue":["elem4","elem5"]}"#)

If you really want to create the element from a string, you can indeed use Spanker like so:
let oldJson = #"{"someValue": ["elem1", "elem2", "elem3"]}"#
let replacementJson = #"["elem4", "elem5"]"#
let newJson: String? = replacementJson.parsed { replacementElement in
    return oldJson.query(map: "$.someValue", {_ in
        return replacementElement
    } ) { root in
        return root.description
    }
}
XCTAssertEqual(newJson, #"{"someValue":["elem4","elem5"]}"#)

@KittyMac these are very elegant solutions, especially initializing JsonElement with anything is very useful. Thanks!


NetworkGame hooks are not working

I am desperately trying to get the NetworkGame hooks ("OnEnteredLobby", "OnGameJoined", ...) to work, but they are just not calling anything.
Not even when I put some debug prints into the "mods/base/lua/NetworkGame.lua" file - not even that get's logged to the console...
Did Overkill change some stuff up over here?
I could really get a hold of a method:

Method that get's called once a client loads into a lobby (or in-game) for the client-side

Then I could trigger some variables from the server to the client over the LuaNetworking methods.

Far as I know NetworkGame isn't used anymore, try search the codebase for where it might be made available? I'm sure I read somewhere on UC that Overkill has split NetworkGame into other classes, the methods you're wanting to change probably exist elsewhere, with similar code and method names.
I've been working with the network codebase quite a bit now, you're probably interested in either client/hostnetworksession and hoststateingame + hoststateinlobby :) Also networkmanager

Recent updates after the Microtransaction-Crisis have fixed it, it seems to work again...
I've found a different approach though, I haven't updated this issue in a while :D


Current release exports Project Settings, which overrides Project Settings in imported project

This may be fixable by changing usage of ExportPackageOptions.IncludeLibraryAssets, which I see we're not sure about anyway. I haven't tried different options yet.

Fixed in v11.


Cannot find module hash-generator

Steps to reproduce
Install
yarn add @cxing/vitejs-plugin-inject-css
Update Config
Build
vite build
λ vite build
failed to load config from C:\Projects\moneytransfer.web\ui\packages\euronet-ui\vite.config.js
error during build:
Error: Cannot find module '../node_modules/hash-generator/index.js'
Require stack:
-<EMAIL_ADDRESS>- C:\Projects\moneytransfer.web\ui\packages\euronet-ui\vite.config.js
- C:\Projects\moneytransfer.web\ui\node_modules\vite\dist\node\chunks\dep-9c153816.js
- C:\Projects\moneytransfer.web\ui\node_modules\vite\dist\node\cli.js
- C:\Projects\moneytransfer.web\ui\node_modules\vite\bin\vite.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)

you might have to install with --legacy-peer-deps flag? hash-generator is in the listed dependencies...

Same problem

I have the same problem - I suspect it's because i'm using yarn workspaces

Yeah same with yarn workspaces. Similar with the same file at the top of the stack so I took a quick look.
The source code here imports hash-generator lib normally. And the dependency is declared normally.
But when I look in my own library project<EMAIL_ADDRESS>it says,
// src/index.js
var src_exports = {};
__export(src_exports, {
  default: () => libInjectCss
});
var import_fs = __toESM(require("fs"));
var import_path = require("path");
var import_hash_generator = __toESM(require("../node_modules/hash-generator/index.js"));
var minify = require("../node_modules/@node-minify/core/lib/core.js");
var cleanCSS = require("../node_modules/@node-minify/clean-css/lib/clean-css.js");

I think if it weren't complaining about hash_generator, it might be @node-minify. If this isn't compatible with yarn workspaces for some reason, is it the build here or yarn to blame.
Anyhow thanks for posting this solution to the issues thread, I can copy and paste the code as well.


Creation and Profile Page: Have the ability to add notes to a scratch org

This is similar to some other suggestions.
There are times when I create an org using exactly the same project, plan and flow. So in the profile page the orgs are listed and look the same and I can't tell which org contains which testing data, which testing purpose, etc.
A use case:
I need to create two orgs with same project, plan and flow in Cumulus, however one org is strictly for multi currency testing, and the other org is for non multi currency testing.
It would be nice to have a Notes field on creation where I can specify what the purpose of the scratch org is for. Then, that notes field would be listed on the profile page for the list of my current scratch orgs.
Bonus feature: If I could edit the notes field on the profile page in case the org purpose changes.
See screenshot with notes below.

fyi @cdcarter and @jlantz

see also #233

@annejackson @melissabarber thanks to @Br4nd0R, you can now enter an Org Note when creating a scratch org in MetaCI.


Use Docker Compose in TCK Script

Presently, the tck script includes some wait statements to allow the various dockerized services time to become ready. We can use docker compose to avoid readiness race conditions and the need for the waits.

Isn't this a duplicate of #45 ?

duplicate of #45


TAU popup is not opened in Live Preview

Technical

open Wearable or Mobile project in DE
D&D Button
Open JS Assistant
Click Button
Click Insert Code
Event Listener > Click Popup Open
go Live Preview
Click button

Expected behaviour
Popup is oopened
Actual behaviour
No popup open

Did you added any pop-up to the project into Design Editor?

According to the doc
https://docs.stg.tizen.org/application/web/api/5.0/ui_fw_api/Base/popup.htm
tau.openPopup(to)
Opens a popup by calling the openPopup() method. The to argument is a pop-up container object or string.
so we need to clarify if string is enough or not.
But tried also with object without success.

@lslachciak
I test in latest TAU Design Editor, it looks working.
I think , we need to append POP-UP component first.
How about your think?

it is ok, but tizen documentation should be updated for tau.openPopup(to)
it is not true that
argument is a pop-up container object or string.
string text can not be an argument, only popup id
I'll update doc

PR merged, closing issue


Allow ramping down of users

Resolves: https://github.com/locustio/locust/issues/1488

Merge stop_user_instances() into stop_users()
Allow stop_rate to be passed to stop_users()
Negative or positive value for stop_rate does the same thing
Copy same ramping behaviour from spawn_users(), it seems robust
If no stop_rate passed to stop_users() then don't "ramp down", just stop all without sleeping
Add one extra debug log message when running test is edited with new user_count/hatch_rate values
Add test runner to ensure ramping down works

This is my first proper PR for this project so any and all feedback welcome 🙂
I have still yet to update any documentation...

If I understand correctly, if stop_timeout is set, we can't really guarantee any ramp down rate accurately, but could do best effort.

Well, in my worst-case example, we should still be stopping roughly 1 user per second, over time (except at the very last step where we last users will take around 1s extra)
For example, if we have 4 users and 2s timeout, it should take 5-6s to ramp down to zero if no users finish their iteration in time and ramp down is 1/s (and the iterations should be killed after 2 seconds and not sooner)

@cyberw OK I've added a second test under TestStopTimeout that basically repeats the other test but with stop_timeout set. Please have a look and see what you think.

@cyberw OK I've added a second test under TestStopTimeout that basically repeats the other test but with stop_timeout set. Please have a look and see what you think.

This test is too lenient:
self.assertTrue(user_count < 10, "User count has not decreased at all: %i" % user_count)
After 2.1 seconds of ramp down at 2/s and with one second delay I would expect to see 6 users running, any more than that is an error.
You can run higher ramp down rate to shorten the test.

This test is too lenient

OK fair enough.
I did some testing and did see some variation, like +/- 1 when using whole numbers for sleeping. So I increased the hatch rate to 4 and tightened the sleep times also. This makes the tests run faster and it also very tight with higher hatch rate. If I make it tighter than this I see some failures just from variation.

Looks great now

🎉

Only thing left is some documentation + an example.

Can you give me a hint here? I've looked through the docs and there's nothing really specifically mentioning hatch rate when decreasing users, decreasing users at all or really anything specific about hatch rate. And what kind of example would I show?

I think for the web gui it is kind of supposed to be self explanatory, so maybe a note there about the ability to set a lower number of users (tbh I'm not much of a web user myself). Actually just trying to reduce the number of Users using the web gui (running on master, without your changes) just kind of worked!? Maybe this feature already exists in some form!?
[2020-08-12 16:44:34,507] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.main: Starting web interface at http://<IP_ADDRESS>:8089
[2020-08-12 16:44:35,036] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.main: Starting Locust 1.1.1
[2020-08-12 16:44:44,361] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.runners: Hatching and swarming 2 users at the rate 1 users/s (0 users already running)...
[2020-08-12 16:44:46,377] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.runners: All users hatched: WebUser: 2 (0 already running)
[2020-08-12 16:44:52,926] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.runners: Stopping 1 users
^CKeyboardInterrupt
2020-08-12T14:46:07Z
[2020-08-12 16:46:07,024] lafp-mac-JG5J.int.svenskaspel.se/INFO/locust.main: Running teardowns...
...

I think for the web gui it is kind of supposed to be self explanatory, so maybe a note there about the hatch rate affecting both up and down ramping (tbh I'm not much of a web user myself).

OK true. I made some small changes to the wording around starting and editing a test.

Maybe there should be a an example similar to the unit test in /examples as well?

I don't think this is possible? The examples are all various types of locust files and you can't edit a running test in them. Or really do anything to show "ramping down" I don't think. I think this will be addressed in my other PR which we are also close to merging 😉

Nice!


Role Based Access Control Asign for create new user in backend UI

May you add the create new user in backend UI: Role Based Access Control Asign ?

Yes, and what is the problem?

Version 2.0.0 adds the web interface RBAC

Assign rights does not have the create action

The binding of the role to the user occurs here:
https://github.com/Dominus77/yii2-advanced-start/blob/3a9e5ac0b5d0b527aa3f2ecb7ea77f84419fd0cd/modules/rbac/controllers/backend/AssignController.php#L95

Download my latest version, there are bugs fixed and there are additions and improvements

New roles are created here:
https://github.com/Dominus77/yii2-advanced-start/blob/3a9e5ac0b5d0b527aa3f2ecb7ea77f84419fd0cd/modules/rbac/controllers/backend/RolesController.php#L88

May I using composer update Dominus77/yii2-advanced-start ?

Try it, but it's better to put it again.
As it is written in README.md
composer global require "fxp/composer-asset-plugin:^1.2.0"
composer create-project --prefer-dist --stability=dev dominus77/yii2-advanced-start advanced-project

Then move your files to a new project, the structure and the principle remained

May you plan save the Layout Options status in session or table?

what do you mean?

When I refresh backend page , The AdminLTE layout back to normal.

While I do not plan, but nothing prevents you from implementing this functionality)

Ok. thanks!!


3.1 cleanup resend email

Scaffolding side matching fixes to https://github.com/aspnet/AspNetCore/pull/14901

If this is for 3.1, then this should be cherry-picked to the 3.1 branch.
/cc @deepchoudhery


ImportError: cannot import name 'Singleton' from 'tsutils'

File "F:\my_AI\transcribe-main\app\transcribe\db\app_db.py", line 10, in
from tsutils import Singleton  # noqa: E402 pylint: disable=C0413
ImportError: cannot import name 'Singleton' from 'tsutils'

Please share complete screenshot of the command line output.
This should include the command to execute the script.

Seems like an environmental issue.
Were you able to resolve this issue, if so please share the steps that lead to resolution. Helps future users.


moar: 1.27.2 -> 1.30.0

Semi-automatic update generated by nixpkgs-update tools.
meta.description for moar is: Nice-to-use pager for humans
meta.homepage for moar is: https://github.com/walles/moar
Updates performed

Golang update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.30.0 with grep in /nix/store/zkssdiq9941z4kkvmh14a9qx801j1v9g-moar-1.30.0
found 1.30.0 in filename of file in /nix/store/zkssdiq9941z4kkvmh14a9qx801j1v9g-moar-1.30.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
moar

Instructions to test this update (click to expand)

Build yourself:
nix-build -A moar https://github.com/chmanie/nixpkgs/archive/5189f3a28d1e58243f5dab66a9b7ab5b5e44b2ca.tar.gz

Or:
nix build github:chmanie/nixpkgs/5189f3a28d1e58243f5dab66a9b7ab5b5e44b2ca#moar

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zkssdiq9941z4kkvmh14a9qx801j1v9g-moar-1.30.0
ls -la /nix/store/zkssdiq9941z4kkvmh14a9qx801j1v9g-moar-1.30.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @foo-dogsquared for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@ofborg eval


ModuleNotFoundError: No module named 'bardapi'

it doesn't work, i've downgraded the python version for whisper but it doesn't work with thi api. I don't know why?

If you're experiencing issues with the installation of the bardapi package in your current Python interpreter environment, you can try the following steps:
If you are using ipython and want to install bardapi, you can execute the following command in a separate shell or terminal window: pip install bardapi.
If you are using a virtual environment, make sure you activate the virtual environment and then install bardapi within it. You can use the command: pip install bardapi.
The issue you're facing is not directly related to Python version compatibility but rather an installation issue with the package. If you have any further questions or need additional assistance, feel free to ask.
Thank you!


Internal mint button in frame

Hello, I hav created some frames but I can't make it to have mint button that will use farcaster's internal wallet for minting instead if will open external url redirecting to the app's/frame's url.
The mint is ERC721 on Base network. Is there something specific todo in order to get the internal Mint button ? (for example NFT in Zora network only and also farcaster should include the frame/smart contract into a whitelist?)

That's a warpcast thing. They choose which contracts to mint in app and which to link out to.
AFAIK its Zora Contract, Base network, Free mint only that work as mint with Warps in Warpcast.


pod details show all hosts as parents/related

Each pod runs on a host. So it's very confusing that the 'parents' / 'related' links at the top of pod detail panels show links to multiple (all?) hosts:

In the report obtained from the UI I see the following kind of parent entries for a pod:
        "parents": {
          "host": [
            "ip-172-20-1-230;<host>",
            "ip-172-20-1-68;<host>",
            "ip-172-20-2-110;<host>",
            "ip-172-20-2-118;<host>",
            "ip-172-20-2-90;<host>",
            "ip-172-20-3-211;<host>",
            "ip-172-20-3-76;<host>"
          ],
          "replica_set": [
            "01f02c9e-03fb-11e7-9b48-0a97ed59c75e;<replica_set>"
          ],
          "service": [
            "14e2fb54-a01e-11e6-921b-0a97ed59c75e;<service>"
          ]

(this is from an old report, so ignore the fact that the hostnames differ)
The question is why that lists all the hosts.

I cannot work out where the host parents come from at all. [probe/kubernetes.Reporter.podTopology](https://github.com/weaveworks/scope/blob/v1.2.1/probe/kubernetes/reporter.go#L335 only appears to add replica set and service parents.

Where are you seeing this? It may be fixed by
https://github.com/weaveworks/scope/pull/2309
On Mar 11, 2017 12:26, "Matthias Radestock"<EMAIL_ADDRESS>wrote:

I cannot work out where the host parents come from at all.
[probe/kubernetes.Reporter.podTopology](https://github.
com/weaveworks/scope/blob/v1.2.1/probe/kubernetes/reporter.go#L335 only
appears to add replica set and service parents.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/scope/issues/2323#issuecomment-285860924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQOJMKE8SQO2FBlgczgsldi7K_h8nbqks5rkoTqgaJpZM4MaK9q
.

I've looked at some reports and what appears to be happening is that a single pod shows up in reports from multiple probes - in one with complete details, and in the others w/o any details.
I reckon one reason for this behaviour is the code fixed in #2309, which isn't deployed in our clusters yet. That causes the reporter to include all pods in a node's report.
However, with that fix in place, I suspect the broken behaviour can still occur when we hit the fallback case for not being able to talk to the local kubelet.
Perhaps the host parents info for a pod should be determined by the host of its containers.

Deploying #2309 to our dev cluster has indeed resolved the problem there.
So that just leaves the fallback case to worry about.

Perhaps the host parents info for a pod should be determined by the host of its containers.

Good point, although a pod can exist without its containers existing (e.g. pull image error).

a pod can exist without its containers existing

even w/o the 'pause' container?

Yes, probably the number of cases is narrower though.

TBH, I don't think this issue is worth worrying about until we hear complaints from users. One would hope that most installs have access to the local kubelet and hence don't hit the fallback case.


Validate common bud and from flags

Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rh-atomic-bot r+


Update translations

Hello translators! Spyder-Terminal version 0.4.0 is soon to be released and we'd like to ask for your help to update your respective translations.
We plan to do this release on Thursday, so it'd be great if you could help us with that by July 16th 10:00 pm UTC at most. It's ok if you cannot make it on time, we can release your updates on a future version.
French
@afvincent
@jnsebgosselin
Japanese
@sohnishi
Portuguese (Brazilian)
@sehnem
@manipuladordedados
Russian
@burrbull
@Qlogin
Spanish
@ccordoba12
@juanis2112
Simplified Chinese
@suokunlong
@kingmo888
@nnnewb
@gepcel
German
@alexschw
@be-we
@bnavigator
Best and thanks again!

@juanis2112 I already finished the translations for Spanish (:

Great! Thanks @steff456


When entering text, selecting all, and then using Command+Z to undo results in an error.

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
9.0+
Other Flutter Quill packages versions
No response
Steps to reproduce
When entering text, selecting all, and then using Command+Z.
Expected results
undo
Actual results
error
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Logs
Logs
Restarted application in 584ms.

════════ Exception caught by services library ══════════════════════════════════
The following assertion was thrown while processing the key message handler:
'dart:ui/text.dart': Failed assertion: line 2520 pos 16: '<optimized out>': is not true.

When the exception was thrown, this was the stack:
#2      new TextRange.collapsed (dart:ui/text.dart:2520:16)
#3      new TextSelection.collapsed (package:flutter/src/services/text_editing.dart:36:14)
#4      QuillController._handleHistoryChange (package:flutter_quill/src/widgets/quill/quill_controller.dart:254:25)
#5      QuillController.undo (package:flutter_quill/src/widgets/quill/quill_controller.dart:243:7)
#6      QuillEditorUndoKeyboardAction.invoke (package:flutter_quill/src/widgets/raw_editor/raw_editor_actions.dart:364:24)
#7      _OverridableContextAction.invokeDefaultAction (package:flutter/src/widgets/actions.dart:1811:28)
#8      _OverridableActionMixin.invoke (package:flutter/src/widgets/actions.dart:1676:9)
#9      Action._invoke (package:flutter/src/widgets/actions.dart:336:19)
#10     ActionDispatcher.invokeActionIfEnabled (package:flutter/src/widgets/actions.dart:666:28)
#11     ShortcutManager.handleKeypress (package:flutter/src/widgets/shortcuts.dart:851:83)
#12     _ShortcutsState._handleOnKey (package:flutter/src/widgets/shortcuts.dart:1042:20)
#13     _HighlightModeManager.handleKeyMessage (package:flutter/src/widgets/focus_manager.dart:1837:32)
#14     KeyEventManager._dispatchKeyMessage (package:flutter/src/services/hardware_keyboard.dart:996:34)
#15     KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:1061:17)
#16     BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:223:49)
#17     _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:567:35)
#18     _invoke2 (dart:ui/hooks.dart:344:13)
#19     _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:45:5)
#20     _Channel.push (dart:ui/channel_buffers.dart:135:31)
#21     ChannelBuffers.push (dart:ui/channel_buffers.dart:343:17)
#22     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:722:22)
#23     _dispatchPlatformMessage (dart:ui/hooks.dart:257:31)
(elided 2 frames from class _AssertionError)

KeyMessage: KeyMessage([KeyDownEvent#c441f(physicalKey: PhysicalKeyboardKey#c222c(usbHidUsage: "0x0007001d", debugName: "Key Z"), logicalKey: LogicalKeyboardKey#5989a(keyId: "0x0000007a", keyLabel: "Z", debugName: "Key Z"), character: "z", timeStamp: 19:19:03.247687)])
════════════════════════════════════════════════════════════════════════════════

Hi @lingyun000, can you switch to the new branch fix/text-selection-exception
and tell me if it fix the issue for you?

The current test is fine. Can you add a carriage return for cyclic searching without the need to click a button?
| |
王纪
|
|
@.***
|
---- Replied Message ----
| From | Ellet @.> |
| Date | 12/22/2023 20:42 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [singerdmx/flutter-quill] When entering text, selecting all, and then using Command+Z to undo results in an error. (Issue #1641) |
Hi @lingyun000, can you switch to the new branch fix/text-selection-exception
and tell me if it fix the issue for you?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

The current test is fine.

So does it solve the problem for you?

Yes, the answer solves the problem.
---- Replied Message ----
| From | Ellet @.> |
| Date | 12/30/2023 20:36 |
| To | singerdmx/flutter-quill @.> |
| Cc | lingyun000 @.>,
Mention @.> |
| Subject | Re: [singerdmx/flutter-quill] When entering text, selecting all, and then using Command+Z to undo results in an error. (Issue #1641) |
The current test is fine.
So does it solve the problem for you?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

When will the code be merged?

I'm not sure if it cause any other bugs
I need someone expert in this repo to review the code and the bug, and then we will decide if it should be merged or not

Can you expedite the review? This is a critical issue on PC.

Can you review this @adil192?
this feature was introduced by you and I can't really understand it, I did a workaround to fix the exception and that's all I can do

The current version does not have this problem.
If the problem still exists, please reopen and let us know how to reproduce the program.


Error running qinfo

Seems like the difference "stopped - started" is being treated as a DateTime, and something is looking for a time zone on it...
Python 3.6, Django 2.1.2, django-q 1.0.1, django-redis 4.10.0. Pytz 2018.7, not that I think it matters.
System check identified some issues:

WARNINGS:
?: (django_mysql.W003) The character set is not utf8mb4 for database connection 'default'
	HINT: The default 'utf8' character set does not include support for all Unicode characters. It's strongly recommended you move to use 'utf8mb4'. See: https://django-mysql.readthedocs.io/en/latest/checks.html#django-mysql-w003-utf8mb4
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.6/dist-packages/django/core/management/__init__.py", line 381, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.6/dist-packages/django/core/management/__init__.py", line 375, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.6/dist-packages/django/core/management/base.py", line 316, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.6/dist-packages/django/core/management/base.py", line 353, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.6/dist-packages/django_q/management/commands/qinfo.py", line 41, in handle
    info()
  File "/usr/local/lib/python3.6/dist-packages/django_q/monitor.py", line 127, in info
    exec_time = last_tasks.aggregate(time_taken=Sum(F('stopped') - F('started')))
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/query.py", line 370, in aggregate
    return query.get_aggregation(self.db, kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/sql/query.py", line 488, in get_aggregation
    result = next(compiler.apply_converters((result,), converters))
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/sql/compiler.py", line 1009, in apply_converters
    value = converter(value, expression, connection)
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/mysql/operations.py", line 252, in convert_datetimefield_value
    value = timezone.make_aware(value, self.connection.timezone)
  File "/usr/local/lib/python3.6/dist-packages/django/utils/timezone.py", line 264, in make_aware
    return timezone.localize(value, is_dst=is_dst)
  File "/usr/local/lib/python3.6/dist-packages/pytz/__init__.py", line 223, in localize
    if dt.tzinfo is not None:
AttributeError: 'decimal.Decimal' object has no attribute 'tzinfo'

Duplicate (but with more important issue info) of https://github.com/Koed00/django-q/issues/245 issue


image pull hangs with k8s (stuck in  'PodInitializing')

Description
In the context of k8s, when massive number of machines(>100 machines) try to pull the same image from either an internal registry/docker registry, all the pods are stuck in "PodInitializing" and in "Pulling the Image" state but that never happens. Once containerd on the system is restarted the pull succeeds.
Steps to reproduce the issue:

Setup k8s cluster with 100+ machines
Deploy a  daemonset that will pull an image from docker.io/internal registry

Describe the results you received:
Most of the pods in the ds are stuck in 'PodInitializing'
Describe the results you expected:
All the pods complete downloading the image and move to state 'Running'
Output of containerd --version:
containerd github.com/containerd/containerd 1.3.0

Any other relevant information:
We noticed this with both Ubuntu 18.04 and ClearLinux. We were able to reproduce this issue both at a CSP setup and our internal clusters.

Does k get pod -o yaml or journalctl -u containerd show an interesting message?

@AkihiroSuda Unfortunately do not have the cluster in the same state anymore but if memory serves me right, I did not see anything interesting in the logs. Just said that is trying to pull the container image and just stays there. I will try to see if i can get a cluster back in that state to gather more logs.

@ganeshmaharaj  can u give a try with version 1.3.4 ?

Unfortunately I do not have a cluster with the size to re-create the issue that i initially saw. Kindly go ahead and close this bug and i will test out version 1.3.4 on my next chance. If the issue persists, i will re-open the bug.


Global Scope for Unique validation rule.

Laravel Version: 5.5.17
PHP Version: 7.1
Database Driver & Version: Mysql 5.7

Description:
Global scopes are not being applied in built in Laravel rules.
Steps To Reproduce:

Apply a global scope to any model:

...

class Course extends Model
{
    ...

    protected static function boot()
    {
        parent::boot();

        static::addGlobalScope('center_id', function ($query) {
            $query->where('courses.center_id', 1);
        });
    }

    ...
}

...

Validate dummy data:

$this->validate(request(), [
    'class' => [
        'required',
        Rule::unique('courses', 'class')
    ]
]);

Check executed query:

[
    "query" => "select count(*) as aggregate from "courses" where "class" = ?"
    "bindings" => array:1 [
        0 => "dummy_data"
    ]
    "time" => 0.12
]

As you can see, the global scope is not being applied to the "unique" rule query. I didn't test it, but other rules could have the same problem.
Is this expected behaviour ?
Thanks !

Definitely the expected behavior.  Could easily be done with a custom rule though.

@devcircus But why ? I mean, why laravel doesn't inject global scopes into validation rules?
I know it can be done through custom rules, but I wanna know the reason which global scopes don't apply in validation rules.

The unique rule works with tables and columns, not models. Just like the DB facade, it does not apply scopes.

e.g. soft deletes utilizes global scope. You can't run a unique validation check while ignoring some rows.

@sisve @devcircus Okay, that makes sense. Now I know the reason behind this issue.
Thank you both.

Hi,
unique rule without global scopes is inconsistent with exists rule.
Exists rule is similar. It is also database rule, but is uses gloabl scopes.
This happens with lates laravel.
It should be at least some notes on documentation and some way to easily apply scopes by name.
What do you think?
WBR


Echo Occupancy/Vacancy

Hi,
I just got the 4th Gen Echo that has Occupancy/Vacancy. Is it possible to have a sensor for this in HA? Something like a motion sensor would be great.
Thank you in advance and for a great integration!

You can do this with fairly low effort.

Make a Template Light
Expose that light to Alexa via Home Assistant Cloud (or otherwise)
Go into the Alexa app, and create two routines: (1) to turn on your Template Light when the echo detects occupancy, and (2) to turn it off when not detected

Boom. You now have a signal for "occupied" vs "not" directly on home assistant, and it's actually quite responsive.

The suggested routine is probably the way to do this unless the App exposes the occupancy sensor so we can reach it via the API. I'll leave this open for a little here in case someone does provide info about the API. After that, I'll move this to a discussion and mark the routine as the answer.

You can do this with fairly low effort.

Make a Template Light
Expose that light to Alexa via Home Assistant Cloud (or otherwise)
Go into the Alexa app, and create two routines: (1) to turn on your Template Light when the echo detects occupancy, and (2) to turn it off when not detected

Boom. You now have a signal for "occupied" vs "not" directly on home assistant, and it's actually quite responsive.

I already have something like this and it hardly works well due to the cool down of the vacancy being so long - I think its over 30 minutes. I was hoping the API could do better and make it more like a motion sensor with a less of a delay

We only have what the App can expose. Do you see motion sensor type data coming from the app? If not, I think you're stuck with an occupancy sensor which is provided by Amazon.


"~/.zshrc file does not exist or is not readable"

I get the error in the title. Indeed my file does not exist since I use .zshenv to set ZDOTDIR.
The script should check if $ZDOTDIR exists and use $ZDOTDIR/.zshrc when it does.

Great find! We accept pull requests :)

This is already fixed, #51.


Updated channel and documentation to Rubocop v0.84.0

This PR proposes to update the Rubocop integration to support v0.84.0.

rubocop is at 0.85 as of a couple days ago.

The 0.85 release of Rubocop is breaking the build due to https://github.com/rubocop-hq/rubocop/issues/8077. A fix has been merged, but not released yet. I'll leave it up to Top Men whether to merge this in the meantime, or wait for the next release.

Closing this, as it's superseded by #238.

I think this should also be merged? @filipesperandio

CC: @noelia-lencina

Sorry @DanielWright @krzysiek1507 we have some work requiring us some more energy. We will get back to these upgrades soon.


NuGet package is out of sync with repository

I'm trying to use your API when I noticed that the Event model doesn't include a Dates property.  It looks like you made a change to this file back in April but never updated the NuGet package.

Updated NuGet, sorry for the delay. In case of any issues or suggestions for improvements do not hesitate to contact me directly in facebook twitter or any other social media.


[minor] added --hide-ping for hiding 'Received ping' and pong messages

intended to fix https://github.com/websockets/wscat/issues/107 and provide a workaround for https://github.com/websockets/wscat/issues/84
functionality
When using a -P or --hide-ping, the blue < Received ping (and pong) messages stop appearing in the console, which allows for a clean uninterrupted copy-pasteable transcript of the ws conversation.
This is opt-in to keep the change minor. I would suggest to make this default in the next major version

I agree that it's better to make the "print ping/pong" feature opt-in with a cli argument. I don't mind if the change is semver-major.

version with ping being opt-in: #113

#113 was merged instead


Follow up Cloud infrastructure Inuits

Improvement report
Follow up Cloud infrastructure Inuits. Placing:

[ ] UIs
[ ] Vault
[ ] Logging
[ ] Monitoring

@netsensei I think this is one that we've created together, but I cannot seem to recall the specifics. It feels like this is a double story by now – is that true?

This one was created after one of the weekly calls with our vendor. These were high-level bullets of things that still needed / need to be put in place. It's an epic, rather then a direct actionable issue. Needs to be hashed out in actionable bullet points, so we can track progress and close it.

@netsensei do you already have enough information to split off these stories? We've been tracking this in a milestone (that is past due date), but since this is an epic without actionable points we cannot really follow up on it.

Let's close this one. We'll need a separate board / tracker on our other GitHub to track infrastructure changes.


label spring.config.additional-location as optional

I need this merged ASAP please tested and working.

Workaround for Kubernetes:
Add an args parameter containing something unused (empty args: [] does not work as it gets omitted).
    spec:
      containers:
        - name: cool-pdf-converter
          image: eugenmayer/jodconverter:rest
          args: ['noop']


Transfer Again?  Needs more detail on how to do this successfully

I've had one server that I've been able to successfully inventory and transfer.  We have been working for about a week to cut it over, and data on the source has changed again, and there's about a 10% difference in the files on the source from what currently exists on the destination.  How do we "transfer again" and only those files that have changed?  I went through the transfer process again, and all it did was copy over all the same files to the destination server.  It would be helpful to have more information on this "transfer again" scenario as I think it could be quite common especially if you have a large volume of data that you want to transfer with a sizable change rate, and then a gap between when files are finally copied up and the server is actually cutover.

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

ID: 6614c040-d1c5-efb2-e29e-7cd93b3c9e55
Version Independent ID: f268e6ad-a69a-174d-0630-ad41aae0a9f1
Content: Migrate a file server by using Storage Migration Service
Content Source: WindowsServerDocs/storage/storage-migration-service/migrate-data.md
Product: windows-server-threshold
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

It seems disappointing to see that this was asked 3 months ago, and nothing.  I found this today, when I was looking for the answer to the same question.

Choosing to Start Transfer should do the incremental sync. You may get a warning that this will start the transfer all over again, but this is a bug. From my testing, it does actually do an incremental transfer, after the first time it runs.

Hello,
When i tried to transfert again, it's always all folders and files....
Did you do it in a particular way ?

Closing issue abandoned storage migration service.


Is 'Entities per second (partition)' limit example correct for a 'Range Query'

The document states 'A query that scans 1000 entities and returns 5 would count as 1000 entities.'.
It is unclear as to whether the eample is for a 'Partition Scan' query or also a 'Range Query' which filters on a range of indexed RK values?  I don't believe the example is correct for a 'Range Query', only a 'Partition Scan'.
Scenario:
A table contains 100,000 entities, with a PK of '1' and Row Key of either 'A','B','C',....'Z' and a 'deleted' flag attribute plus other fields (each entity 2kb size in total).  There are 500 entites in PK '1' with a RK of 'S'.

If a Range Query was issued (PK eq '1' and RowKey ge 'S' and RowKey lt 'T') which in the end returned 500 entities (all columns), I believe this would count/consume 1,000 'Entities per second'? Not 200,000 'Entities per second' as suggested - which indicates the query scanned every record in the Partition one by one as in a 'Partition Scan'.

If the same Range Query was issued with an additional filter (PK eq '1' and RowKey ge 'S' and RowKey lt 'T' and deleted eq 'true') which returned only 5 entities of the 500 entities in the RK range, I believe this would still consume the same 1,000 'Entities per second' – but it would only scan the entities in the RK range one by one, not the entire Partition one by one as in a 'Partition Scan'.

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

ID: effdf6ae-eecf-b92b-1045-353fd20e0813
Version Independent ID: ddba84f7-8db9-6a52-16c1-fcd9b54f011b
Content: Performance and scalability checklist for Table storage - Azure Storage
Content Source: articles/storage/tables/storage-performance-checklist.md
Service: storage
Sub-service: tables
GitHub Login: @tamram
Microsoft Alias: tamram

@bladerunner41 You may open a support ticket with our technical team for much detailed analysis of the issue: http://azure.microsoft.com/en-us/support/options/
If you have a support plan, I request you file a support ticket, else please do let us know, we will try and help you get a one-time free technical support. In this case, could you send an email toAzCommunity[at]Microsoft[dot]com referencing this thread. Please mention "ATTN subm" in the subject field. Thank you for your cooperation on this matter and look forward to your reply.

@bladerunner41 We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Corrected example

Added the hyphen before the i in the example

Thanks for fixing this! You'll see your change live this morning (Friday, 5/18)


Added a method to enabled grouping columns by array

Currently, if we want to do grouping by multiple columns, we have to write groupBy for each column. For example:
FooBarQuery::create()
   ->joinBaz()
   ->select(['Foo', 'Bar'])
   ->withColumn('COUNT(Baz.Id)', 'nbBazs')
   ->groupBy('Foo')
   ->groupBy('Bar')

with this PR, we just simply write:
FooBarQuery::create()
   ->joinBaz()
   ->select(['Foo', 'Bar'])
   ->withColumn('COUNT(Baz.Id)', 'nbBazs')
   ->groupByArray(['Foo', 'Bar'])

Well, thanks, but Propel1 is feature freeze. You can try it at Propel2

Ok, no problem :), i'm going to open PR on the Propel2 repo


Ecommerce data: change content-type

Hi there!
Thanks for this great library! I was wondering if I can change the content_type for the e-commerce data in this tag. I need to set it to 'product_group' and not 'product'. I see that I can change this in the code, however it is maybe better to create some extra setting for this. With an update, this code would be removed.
Kind regards,
Bas

Hi,
Thanks for the feedback. You can already do this.

thank you!


(maint) Update windows ruby builds

Ruby for windows was rebuilt with openssl 1.0.2f to address
CVE-2016-0701. This updates to use the latest ruby build.

I also like this :+1:

:+1:  pending travis


facilate npm publish

either np or git release + pre/post

https://zellwk.com/blog/publish-to-npm/


mpv use yt-dlp now

Closes #XXXX

Relates to #XXXX

[x] I have read the Contributing Guide.

Source for this info?

BTW can we shift to https://github.com/zhongfly/mpv-winbuild/releases now? It provides latest binaries built using github actions
check - https://github.com/zhongfly/mpv-winbuild/discussions/13

BTW can we shift to https://github.com/zhongfly/mpv-winbuild/releases now? It provides latest binaries built using github actions check - zhongfly/mpv-winbuild#13

We can do that when the official website (mpv.io/installation) mentions it.


[Suggestion] Remove the "Restart" option from the context menu of the stopped virtual machine

OS: All
Build Version: 20210325.1
Repro Steps:

Right click a virtual machine --> Select "Stop".
Right click the stopped virtual machine --> Select "Restart".
Check whether pop up an error or not.

Actual:
Pop up an error after restarting a stopped virtual machine.

Suggestion:
Remove the "Restart" option from the context menu of the stopped virtual machine.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Share package

It would be nice to share this package, so people can discover it exists.
It is already mentioned in linter repository, next to pedantic package, and it will be shown in linter lints page.
I would like to share it to sources like awesome dart, flutter etc.
Tips where we can share it are welcomed!

I am going to close this issue and reopen it when the package gets more stars.


Use balenalib base images

The newly released balenalib base images are smaller and target multistage builds.

Fixed, meged to master.


Stuck at 95% emitting with source-map

version<EMAIL_ADDRESS>OS: windows 10
After I added postcss  and enable devtool: source-map, building process stuck at 95% emitting .
If I remove devtool: source-map or make devtool: inline-source-map, all goes well.
Also I have external style file for component by styleUrls[].
My bits of webpack config for css:
  {
    // css - The pattern matches application-wide styles, not Angular ones
    test: /\.css$/,
    exclude: path.join(__dirname, `src`, `app`),
    use: ExtractTextPlugin.extract({fallback: `style-loader`, use: [`css-loader?{"sourceMap":false,"importLoaders":1}`, `postcss-loader`]})
  },
  {
    // the second handles component-scoped styles (the ones specified in a component`s styleUrls metadata property)
    test: /\.css$/,
    include: path.join(__dirname, `src`, `app`),
    use: [`exports-loader?module.exports.toString()`, `css-loader?{"sourceMap":false,"importLoaders":1}`, `postcss-loader`]
  }

Full config is here https://github.com/yurlovm/asp.net_4.6_angular_webpack/blob/master/WebApiAngular/webpack.aot.config.js

Experiencing the same thing, however my only loader for css is raw-loader which I had to add specifically for one module (ngx-datatable).
This only happens on windows and seems to be to webpack trying to emit a file with a colon in it, e.g. C:/path/to/source.css.map.
When running on jenkins it outputs a file to this path:
<EMAIL_ADDRESS>
Somewhat confirming that it is trying to emit with the drive letter in the path on windows.
I am honestly not even sure why a sourcemap is being created for this file.
Our styling based webpack rules:
{
    exclude: paths.globalStyles,
    test: /\.scss$/,
    use: [
        'raw-loader',
        {
            loader: 'sass-loader',
            options: sassOpts,
        }
    ]
},
{
    test: /\.css$/,
    use: ['raw-loader'],
},
{
    include: paths.globalStyles,
    test: /\.scss$/,
    use: ['style-loader', 'css-loader', 'sass-loader']
},

Hey, I've just cloned the repo and ran npm build:prod, and it seems to be working fine:

Is there something I need to change in the webpack.aot.config.js file?

Nevermind, I've just realised you're using Windows.. Will try and sort something out so I can test it on there!

Hello, look at compose-message.componentcss.map path, I think it should be relative path.

Also the result path for this map is worng. It should be ./client/css/*.css.map

Seems related to https://github.com/webpack/webpack/issues/2908


Confusing message about dependency conflicts

If I explicitly downgrade a package, the messages are very confusing.  It says that the package will be "downgraded due to dependency conflicts", which may be true in some narrow sense, but to me it's being downgraded because I explicitly told it to!  Any chance the message could avoid claiming a reason it's being downgraded, if it's too complicated to convey the actual reason?
$ conda install dask=0.11.0
Fetching package metadata .......
Solving package specifications: ..........
Package plan for installation in environment .../anaconda:
The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    dask-0.11.0                |           py27_0         591 KB

The following packages will be DOWNGRADED due to dependency conflicts:

    dask: 0.12.0-py27_0 --> 0.11.0-py27_0

Proceed ([y]/n)? y

Oops!  I meant to put this on conda, not dask!


Support nested JSON responses for REST generator

Accept valid json path, not just string.

#651 any use?

Completely solved. Thanks Leon!


IGD 2.0 support

Ever since the add_any_port method was added this library has had some support for IGD 2.0 although it by no means supports all the IGD 2.0 methods (or all the IGD 1.0 methods for that matter).

A first step would be to search for IGDv2 devices too (and then use the v1 methods).
This would be useful if there are IGDv2 devices that respond to a IGDv2 search and not a IGDv1 one.


HOTT-955 Remove a footer link

What?
Remove the footer link "Using the Trade Tariff tool to find a commodity code".
Jira link
https://transformuk.atlassian.net/browse/HOTT-955
Why?
I am doing this because:
The link removed pointed to exactly the same link as the one above.

There's a moratorium until circle ci is back up and running


ORM Adapter For NextAuth

NextAuth is a popular, almost de facto authentication framework for Next.js. It supports many ORM backends via adapters. If EdgeDB can provide a first party backend adapter, it would help accelerate adoption of EdgeDB. The model for NextAuth is very simple  and integration only takes a few lines of code.

Good idea! I've used next-auth several times in various side projects, should be quick to do. I'll look into it.


Fix: Message not showing

Both title and body should not be in cap letters, since if it's in cap letters, it doesn't works

Thanks for the contribution 🎉


The script always uses only the third argumant for wrting into file

./exercise1.sh apple 15 fruit one two three operand
cat  ~/mylogs/apple.log
1) fruit: normal
2) fruit: warning
3) fruit: normal
4) fruit: warning
5) fruit: warning
6) fruit: notice
7) fruit: error
8) fruit: normal
9) fruit: error
10) fruit: alert
11) fruit: request
12) fruit: warning
13) fruit: error
14) fruit: warning
15) fruit: request```

Logically. And we want to use not the third one?

Yes, the requirements specifically say: "iterates over the list of additional arguments".
So there should be log lines for all additional arguments.

Please look at this now.

@rbalashevich this looks better but still doesn't do what is required. It should "iterate over the list of additional arguments" , not choose random items from that list.

For times [x] iterates over the list of additional arguments - means  the resulting number of log lines should be (number_of_additional_arguments * x)

It's almost fixed. It now ignores the third argument... ;)
./exercise1.sh apple 3 one two three four five six
Number of arguments: 8
Name: apple
Count: 3
Additional arguments: two three four five six
Old file /Users/antweiss/mylogs/apple.log removed.
🔬 Dumping argument, it is "two" now. Checking is it a numeric or not
🔬 Dumping argument, it is "three" now. Checking is it a numeric or not
🔬 Dumping argument, it is "four" now. Checking is it a numeric or not
🔬 Dumping argument, it is "five" now. Checking is it a numeric or not
🔬 Dumping argument, it is "six" now. Checking is it a numeric or not

Notice how "one" is ignored....

What we have there: one is not an additional argument. Additional ones start from two. If we're checking additional arguments only, then the logic is correct.

@rbalashevich instead of arguing with me - please recheck the requirements.

apple is the file name
3 is the number of times to iterate
All the rest are additional arguments.

$ ./exercise1.sh Apple 10 fruit one two three operand daemon flow set
Number of arguments: 10
Filename: Apple
Count: 10
Additional arguments: fruit one two three operand daemon flow set
Old file /home/roman/mylogs/Apple.log removed.
🔬 Dumping argument, it is "fruit" now. Checking is it a numeric or not
🔬 Dumping argument, it is "one" now. Checking is it a numeric or not
🔬 Dumping argument, it is "two" now. Checking is it a numeric or not
🔬 Dumping argument, it is "three" now. Checking is it a numeric or not
🔬 Dumping argument, it is "operand" now. Checking is it a numeric or not
🔬 Dumping argument, it is "daemon" now. Checking is it a numeric or not
🔬 Dumping argument, it is "flow" now. Checking is it a numeric or not
🔬 Dumping argument, it is "set" now. Checking is it a numeric or not


enpolat/fix-devcontainer

DevContainer is using an image from an ACR , and the ACR is un-reachable.
This fix is for changing DevContainer to use local Dockerfile and install all the required tools locally.

LGTM


added cache update feature and brought into line with npm

Added the missing cache update feature and slight refactor to keep Travis, npm and github in similar styles.
Also updated to handle any form or giturl provided, crashes were happening occasionally due to the giturl coming in many different formats from npm.

@AdrianRossouw @mcdonnelldean Last one for today, tested and working of course!


Prefer transition names which match exactly

Some transitions are substrings of other transitions, and the current loop sometimes doesn't choose the best match. For example, it would never choose "QA" if there were also a transition "Deploy to QA". Ditto for "Open" and "Reopen".

Looks good, thanks!
-Cory

Released:
https://github.com/Netflix-Skunkworks/go-jira/releases/tag/v0.1.7


Update registry specification to include publishing endpoint

Updates the Swift package registry specification with the proposed publishing endpoint.
Motivation:
This PR serves as the implementation for the corresponding Swift Evolution proposal.
Modifications:

Adds a new PUT /{scope}/{name}/{version} endpoint

Result:
This PR updates documentation, and has no functional effect on Swift Package Manager itself.

pending review for https://github.com/apple/swift-evolution/pull/1424

@mattt with SE-0321 accepted, should we move forward to merge this, or do we need to update it first?

@tomerd I have no other changes on my end. I'll kick off a smoke test, and you can feel free to merge as soon as that's ✅ .

@swift-ci Please smoke test


No MediaQueryMatchers?

I noticed that MediaQueryMatchers is not exported in v9.0.0 so it broke my code:
import { MediaQueryAllQueryable, MediaQueryMatchers, useMediaQuery } from 'react-responsive';

interface ResponsiveReturnValues {
  isMobile: boolean;
  isTablet: boolean;
  isDesktop: boolean;
  isBigScreen: boolean;
  mobileOnly: boolean;
  tabletOnly: boolean;
  desktopOnly: boolean;
  useMediaQuery: (
    settings: Partial<MediaQueryAllQueryable & { query?: string | undefined }>,
    device?: MediaQueryMatchers,
    callback?: (matches: boolean) => void,
  ) => boolean;
}

How do I fix this? I can't seem to find anything on this.
For now my workaround is to go into types.d.ts and hardcopy the MediaQueryMatchers interface:
import { MediaQueryAllQueryable, MediaQueryType, useMediaQuery } from 'react-responsive';

export interface MediaQueryMatchers {
  aspectRatio?: string;
  deviceAspectRatio?: string;
  height?: number | string;
  deviceHeight?: number | string;
  width?: number | string;
  deviceWidth?: number | string;
  color?: boolean;
  colorIndex?: boolean;
  monochrome?: boolean;
  resolution?: number | string;
  orientation?: 'portrait' | 'landscape';
  scan?: 'progressive' | 'interlace';
  type?: MediaQueryType;
}

interface ResponsiveReturnValues {
  isMobile: boolean;
  isTablet: boolean;
  isDesktop: boolean;
  isBigScreen: boolean;
  mobileOnly: boolean;
  tabletOnly: boolean;
  desktopOnly: boolean;
  useMediaQuery: (
    settings: Partial<MediaQueryAllQueryable & { query?: string | undefined }>,
    device?: MediaQueryMatchers,
    callback?: (matches: boolean) => void,
  ) => boolean;
}

Changes are in 9.0.2

Will update now, thanks


Fixed crash on non ascii content of the tables

Ascii-Table3 throws RangeError: Invalid count value, when some unusual characters are used. Example:
const EMPTY = [null, null, null, null, null];
const info = {
  "map": "de_mirage",
  "home": { "score": 10, "players": ["NoHope⭕⃤ (絶望)", "𝓢𝓵𝓲𝔃𝓮𝓷𝓸𝓴"] },
  "away": { "score": 16, "players": [] },
};
const { home, away, map } = info;
const table = new AsciiTable3(map)
  .setHeading(`HOME ${home.score}`, `AWAY ${away.score}`)
  .setAlign(1, AlignmentEnum.LEFT)
  .setAlign(2, AlignmentEnum.RIGHT)
  .addRowMatrix(
    EMPTY.map((_, i) => [home.players[i] || "-", away.players[i] || "-"]),
  );

table.setStyle("unicode-mix");
const content = table.toString();
console.log(content);

It seems that an issue is strlen(str) >= maxLength check on line 105 in  src/ascii-table3.js, and then doing fillStr.repeat(maxLength - str.length) - str.length in this case has different value then strlen(str), so repeat method is being feed a negative value in this case, throwing an error.

You're right, it seems I have forgotten this change when moving from the non-unicode to the unicode support for strlen(). 😄


Add features to allow Terratest to work against the testing-test account

Just looking to see if current code works for terratest

Add force_destroy option & default lifecycle rule
In order for Terratest to work we need to be able to destroy the bucket.
To do this we have removed the lifecycle rule preventing destroy and
have added a new variable to set the force destroy option.  This is
needed as some other modules such as the bastion which use this module
create objects in the bucket.  This will also be useful for users who
wish to destroy buckets.
Added a default lifecyle rule for object management, this was listed in
the inputs as not required but was actually.  Using the rule in the
usage example as the sensible default.


feat: volume snapshot class

lvmo controller should create volume snapshot class for each deviceclass
Testing:

Creation of volumesnapshot class by the lvmo controller:

oc get volumesnapshotclass
NAME          DRIVER               DELETIONPOLICY   AGE
odf-lvm-vg1   topolvm.cybozu.com   Delete           5s

Volume snapshot class:

oc get volumesnapshotclass odf-lvm-vg1 -oyaml
apiVersion: snapshot.storage.k8s.io/v1
deletionPolicy: Delete
driver: topolvm.cybozu.com
kind: VolumeSnapshotClass
metadata:
  creationTimestamp: "2022-05-05T05:01:47Z"
  generation: 1
  name: odf-lvm-vg1
  resourceVersion: "19572561"
  uid: 26c2437c-26ec-4727-8a77-954cd1874c3a

Deleting the lvm cluster cleans up the volumesnapshot class.
Signed-off-by: Santosh Pillai<EMAIL_ADDRESS>

We should not be installing the VolumeSnapshotClass CRD. That will be part of the k8s cluster.
The suite_test would need this CRD to work. Else the tests will faile.

1.6517312682804594e+09	INFO	lvmcluster-controller	topolvm storage class	{"Request.Name": "test-lvmcluster", "Request.Namespace": "openshift-storage", "operation": "created", "name": "odf-lvm-test"}
1.6517312683031378e+09	ERROR	lvmcluster-controller	topolvm volume snapshot class reconcile failure	{"Request.Name": "test-lvmcluster", "Request.Namespace": "openshift-storage", "name": "odf-lvm-test", "error": "no matches for kind \"VolumeSnapshotClass\" in version \"snapshot.storage.k8s.io/v1\""}

Any suggestions to fix this?

Please add the rbacs to https://github.com/red-hat-storage/lvm-operator/blob/e611a42df60f183701fe8540b965da7d53dbe93a/controllers/lvmcluster_controller.go#L80

I've added the RBAC annotations above the ensureCreated method of topolvm_snapshotClass.go file.  Why do we need to add this in lvmcluster_controller.go?  (asking because its not very clear to me).

We should not be installing the VolumeSnapshotClass CRD. That will be part of the k8s cluster.
The suite_test would need this CRD to work. Else the tests will faile.

1.6517312682804594e+09	INFO	lvmcluster-controller	topolvm storage class	{"Request.Name": "test-lvmcluster", "Request.Namespace": "openshift-storage", "operation": "created", "name": "odf-lvm-test"}
1.6517312683031378e+09	ERROR	lvmcluster-controller	topolvm volume snapshot class reconcile failure	{"Request.Name": "test-lvmcluster", "Request.Namespace": "openshift-storage", "name": "odf-lvm-test", "error": "no matches for kind \"VolumeSnapshotClass\" in version \"snapshot.storage.k8s.io/v1\""}

Any suggestions to fix this?

Let me think about this. Off the top of my head , set a variable to disable snapshots and set that to true in the make test.

Please add the rbacs to https://github.com/red-hat-storage/lvm-operator/blob/e611a42df60f183701fe8540b965da7d53dbe93a/controllers/lvmcluster_controller.go#L80

I've added the RBAC annotations above the ensureCreated method of topolvm_snapshotClass.go file. Why do we need to add this in lvmcluster_controller.go? (asking because its not very clear to me).

Missed that. Having them in the topolvm_snapshotClass.go is better.

\retest

/cherry-pick release-4.11


Code search in organization

Feature Description
Currently it's possible to search only globally or in specific repository. There is many use cases when it's needed to search code in single organization.
Screenshots
No response

Wow, we need to search all allowed repositories' IDs in this organization


memcached sessions do not redirect

I set up memcache on two amazon ec2 instances. Along with a ElastiCache runing memcache. If I log into either of my instances separately I have no problem loging in then redirecting. But if I go through my AWS loadblancer it seems to just redirect me back to the login.xhtml page. But if I tail the logs in both instances I can see soap request and a 200 response code.
What I have.
It's a spring/tomcat 8 app
in my tomcat lib I have three jars.
-memcached-session-manager-2.1.1.jar
-memcached-session-manager-tc8-2.1.1.jar
-spymemcached-2.11.1.jar
in my context.xml I have
  <Manager className="de.javakaffee.web.msm.MemcachedBackupSessionManager"
        memcachedNodes="n1:memcache-one-node.drkx12.cfg.usw2.cache.amazonaws.com:11211"
             memcachedProtocol="binary"
             sticky="true"
             sessionBackupAsync="true"
             requestUriIgnorePattern=".*\.(ico|png|gif|jpg|css|js)$"
             transcoderFactoryClass="de.javakaffee.web.msm.JavaSerializationTranscoderFactory"
          />

I'm at a loss as to why it won't redirect on the load balancer. Like I said when I tail the logs I see the request working well and it looks like it is using the memcahce. Since i'm using the built in java serialization I figured I don't need to add anything to WEB-INF/lib/. Am I missing something obvious? Thanks

@magro Seen anything like this before?

@davedennis Sorry for the late response here. Is this still relevant or can we close this issue?


Decorative legacy codes working when disabled

Describe the bug
Players are able to use bold/underline in their names when using https://rgb.birdflop.com/ – but doing &lTest does not work.
I have hexnicks.decoration.* negated (set to false in luckperms) as well as:
disabled-decorations:
- italic
- bold
- strikethrough
- obfuscated
- underlined

set in my config. This appears to only be affecting legacy colors (as seen with the birdflop generator) when mixed with hex.
To Reproduce
Steps to reproduce the behavior:

Disable all decorations
Negate the hexnicks.decoration.* permission
Generate a name with bold/underline with birdflop
Set nickname, it will work.

Expected behavior
For bold/underline/other decorations to be filtered.
Screenshots

Software (please complete the following information):

Server Implementation: Paper
Minecraft Version: 1.19
Java Version: 17
Plugin Version: Latest

Bug reported stated it's fixed as of v3.0.2.


Vacations

Each group member should take 2 weeks (not necessarily in sequence) of vacation.
If you want to take a break, please make sure to announce about your break (in this issue and in the Telegram group) at least a week in advance.
Known vacations:
Tom - 23.5 - 4.6
Inbal - 23.5 - 4.6 💕
Or - 3-9.5 & 4-10.6

Member
Week 1
Week 2
Week 3
Week 4
Week 5
Week 6
Week 7
Week 8
Week 9
Week 10
Week 11
Week 12
Week 13

Tom

🍹
🍹

Inbal

🍹
🍹

Assaf

Dani

🍹
🍹

Or

🍹

Inbal -  23.5 - 4.6 :two_hearts:

Dani 21.5-4.6

As a result of change in the group members and according to the meeting with Tanya today (24.5) it was decided that i will join the group vication that ends in the 4.6 and resume work afterwards with them


Add a makefile target for deploy

Add the ability to deploy the compliance service using kustomize and an
existing Kubernetes cluster.
Additionally patches will add more functionality to kustomize to support
deploying the actual compliance service. This patch will be referenced
in the openshift/release configuration to ensure kustomize is used to
deploy the application in CI.
Fixes #43

/retest

/test e2e-aws

/test e2e-aws


Add instruction for encrypted root drives

Just a quick readme edit to handle a specific use case.
I'm not a native English speaker so I have made a few mistake.

REASON: Redundant
Sorry for taking so long to get to this PR! Basically in the README, this is already covered with Q: "Grub isn't detecting the theme" which tells the user to replace /usr/share/ with /boot/ so that applies to encrypted use cases. Therefore, I will not be merging.


Proposal: include all removed/renamed/deprecated attribute keys in yaml

Some (most?) SIGs including JS and Python consume the semconv yaml and generate a semantic convention helper package. When an attribute is renamed, this means the code generator no longer outputs it, resulting in a breaking change for the code generators. Especially now that the spec requires us to double-publish old and new semantic conventions when there is a stabilization effort which breaks semconv, it would be much easier if the generated packages contained deprecated/removed/renamed keys.
My proposal is simply to create a deprecated.yaml file which contains all removed attributes from all versions of the semconv for use by code generators. If a key is renamed, removed, or otherwise excluded from code generation it should be added to deprecated.yaml.

What do you think of adding a deprecated flag for semconv entries? That would make it easier to keep generating them in the same place (code or markdown tables) but add some note they are deprecated.

This flag already exists and can be used.
(I think it would be worth considering adding some more meta-information to be able to generate all schemas from the latest YAML model)

What do you think of adding a deprecated flag for semconv entries? That would make it easier to keep generating them in the same place (code or markdown tables) but add some note they are deprecated.

There is already a deprecated flag in the yaml schema which would be used for this purpose.

We discussed this in the SiG. While there are many concerns, I think there's enough interest to more formally flesh out this proposal.

Can you document the concerns here so I can respond to them?

If we needed to take down an attribute for some extreme reason (e.g. legal?) would this be allowed, and how would it happen.

I think this is vanishingly unlikely to happen and the same question can be made of the existing semconv. I think if it ever did happen we would simply delete the attribute and follow the appropriate semver process.

Would we have a "cleanup" process eventually, say every N years?  Would this be of benefit?

I don't really see a benefit of cleanup. Even if we deprecated hundreds of attributes per sprint it would take a very long time for the file to grow beyond a few megabytes. Having the full history also has the benefit of ensuring we don't accidentally re-create an attribute which shares a name with a previously deprecated attribute which could potentially confuse consumers.

For reference, the deprecated semantic attributes live here: https://github.com/open-telemetry/semantic-conventions/tree/main/model/deprecated

This was closed by mistake by the stale bot. Re-opening


[Snackbar] material design correct?

Currently the bottom-center snackbar on small displays is placed bottom = 0 and takes the full display width (like Material Design suggests). But I think its better to place it with a bottom margin and not over the full width (like in Google Drive) so it does not need special handling for iPhoneX.

But I think its better to place it with a bottom margin and not over the full width (like in Google Drive) so it does not need special handling for iPhoneX.

@daniel-rabe You are free to customize the output of our components.


Add haem to example projects

Add haem to example projects

I see the changelog check is failing, but I don't think it's necessary to mention this change in release notes. Poke me if you disagree.

Indeed not necessary. Thanks for sharing this!


Reduce stack use for certain arithmetic operations.

It seems like there'll be a performance cliff in both cases if the variadic template ends up being hit. But I don't think there's anything to be done about it - there's no warning or info equivalent of static_assert().

For context, the largest calls to both AllKnown() and MaximumOperation::Do() tend to be generated inside of IntrinsicSizeInBytes(), where the number of arguments is approximately the number of fields in the struct.  You'd have to have more than 128 fields before you get out of the O(lg(N)) domain, and you probably wouldn't notice much until at least 192 or 256 fields.
Or if your C++ compiler applies tail call optimization, it should collapse to O(N) anyway.
On a related note, I wasn't able to figure out how to write a proper regression test for this PR: I don't know of an even somewhat portable way to measure the high water mark on the stack.


Initial drop of java-openliberty stack at v0.2.0, matching appsody/st…

See the contents of: https://github.com/appsody/stacks/pull/650

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Why validation loss<training loss

excuse me, during the training, why the validation loss is less than training loss

because of overfitting? I think you should ask a more detailed question on StackOverflow.

if loss always equals 0     the acc result is very bad

If you refer to the accuracy displayed during training, then forget about that because it has nothing to do with the performance of the model.
Is this the problem or you mean that after training a model that achieves 0 loss, the performance as a classifier is very poor?


Fix Redux DevTools 2 jumps in the line

@hadeeb I added demo page to test DevTools (and render caches), Redux DevTools works great except one thing:

cd storeon (don’t forget to take the latest version from master)
yarn start
Increase counter a few times.
Open Redux DevTools.
Jump to some state.
Then jump to another state.
Jump from step 6 will be no applied. Seems like you can do only one step before any other event.

Maybe fixing https://github.com/ai/storeon/issues/6 could be helpful here


Dom element not creating without parent in config

Version

Phaser Version: 3.24.1

Operating system: windows 10

Browser: chrome

Description

If you do not specify the parent element explicitly, then the dom-elements are not displayed without errors.
Solution: paste 'parent: document' in config (or 'parent: window').

I'm happy to leave this as-is because the config docs specifically state:
Phaser.Core.Config#domCreateContainer - Should the game create a div element to act as a DOM Container? Only enable if you're using DOM Element objects. You must provide a parent object if you use this feature.

thank you so much i couldnt find anything telling me why it didnt display


keyStream disappears on sublevel

the prototype keyStream seems to be lost on the sublevel which may cause incompatibilities for code that expects it to be present, for example level-delete-range

I found https://github.com/rvagg/node-levelup/issues/53 and see that maybe keyStream is just going away. I'll fork level-delete-range maybe and just have it switch over to createReadStream for good.

Here's my test case:
var tap = require('tap');
var test = tap.test;

var db = require('level')('foo');
var sub = require('level-sublevel')(db);

test('db and sub db test', function (t) {

  test('test without sublevel', function (t) {
    t.type(db.createReadStream, 'function');
    t.type(db.keyStream, 'function');
    t.end();
  });

  test('test with sublevel', function (t) {
    t.type(sub.createReadStream, 'function');
    t.type(sub.keyStream, 'function');
    t.end();
  });

  t.end();
});

Result
TAP version 13
# db and sub db test
# test without sublevel
ok 1 type is function
ok 2 type is function
# test with sublevel
ok 3 type is function
not ok 4 type is function
  ---
    file:   /Users/user/git/dweinstein/test/test.js
    line:   18
    column: 7
    stack:
      - |
        getCaller (/Users/user/node_modules/tap/lib/tap-assert.js:418:17)
      - |
        assert (/Users/user/node_modules/tap/lib/tap-assert.js:21:16)
      - |
        Function.equal (/Users/user/node_modules/tap/lib/tap-assert.js:162:10)
      - |
        Function.type (/Users/user/node_modules/tap/lib/tap-assert.js:320:17)
      - |
        Test._testAssert [as type] (/Users/user/node_modules/tap/lib/tap-test.js:87:16)
      - |
        Test.<anonymous> (/Users/user/git/dweinstein/test/test.js:18:7)
      - |
        Test.emit (events.js:117:20)
      - |
        Test.emit (/Users/user/node_modules/tap/lib/tap-test.js:104:8)
      - |
        GlobalHarness.Harness.process (/Users/user/node_modules/tap/lib/tap-harness.js:87:13)
      - |
        process._tickCallback (node.js:442:13)
    found:  undefined
    wanted: function
    diff:   |
      FOUND:  undefined
      WANTED: function
              ^ (at position = 0)
  ...

1..4
# tests 4
# pass  3
# fail  1

can you make a pull request for this? Testing that a function exists means nearly nothing - check that it's a function that returns a stream that does the same thing as passing in {keys: true}. That, or make a pull request to level-delete-range that just uses createReadStream.
Probably we should consider keyStream a legacy api, but it probably won't be going anywhere... note that issue is from 2013!

the size of the test felt a little silly because the methods are just aliases and the fix is so trivial, but hopefully it'll prevent some other strange issue in the future