bug: dependency between helm and doctl for DO deployments

Describe the bug
When using digital ocean to deploy MARA, you need to first have installed and authorized the doctl program. If not, the
following errors will be encountered:
    Error: Unable to initialize DigitalOcean API client: access token is required. (hint: run 'doctl auth init')
    Error: Unable to initialize DigitalOcean API client: access token is required. (hint: run 'doctl auth init')

To Reproduce
Steps to reproduce the behavior:

Deploy MARA on DO w/o doing a doctl auth init.
As soon as you hit the helm sections.....
Boom....

Expected behavior
Should work; this is likely going to need to be resolved by just verifying that the doctl utility has been properly auth'd earlier in the process.
Your environment
n/a
Additional context
None.

One potentially quick fix is to hijack the logic where we test auth via doctl account get and change it to doctl auth init - if you are already auth'd it doesn't matter, and if you aren't it will auth you.


fix: lint errors

Fix a number of lint errors as identified by golangci-lint.

Will need rebasing once other PR's are merged.


fix create/update tags errors and update to function runtime

update to function runtime ~4
fix create/update tags errors
change caller format to upn
extend logging

2 things - can you update the requirements.psd1 file to reflect below - the modules in the original are really old now - I've tested with versions below
@{
    'Az.Accounts'  = '2.11.1'
    'Az.Resources' = '6.5.1'
}

Second thing - in the original I have called the tag "Creator" - not "CreatedBy" - however I like CreatedBy a bit better - what do you think about keeping it as CreatedBy and changing the original?

I changed it because I found it better. You are welcome to take it.


update getDEM to use the new dem-stitcher API

Description

The dem-stitcher API changed at several points; this is catching us up.
Type of change

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

Checklist:

[ ] I have added an explanation of what your changes do and why you'd like us to include them.
[ ] I have written new tests for your core changes, as applicable.
[x] I have successfully ran tests with your changes locally.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Fixes #335
Replaces #339

I am going to merge a major change hopefully in the coming weeks and it changes the API to how I originally had it. I am sorry this got lost by the waste-side.
Given this needs to be global and you are using glo_30 there are known issues over Armenia and Azerbaijan (see this issue ticket).
One thing that is worth noting at this point is whether you want the DEM to have geo-referencing tied to the center of the pixel ('Point' tag) or the upper left corner ('Area' tag). See the readme of dem-stitcher for some discussion about this and references.

To ensure that it works with the API you are using - please fix the version -and we can go back and update it later.

@cmarshak yes corner- versus center-referencing has been a big issue for us. Everything we do is corner-referenced (so Area tag). @dbekaert can you confirm on this?

@cmarshak I'll get the environment files updated tomorrow to fix the version of dem-stitcher.


Issue with conformer.ConformerEnsemble.generate_mol

Hello,
I found that when using OpenBabel to generate conformers via:
conformer.ConformerEnsemble.from_ob_ga(..., generate_rdkit_mol=True)
the function conformer.ConformerEnsemble.generate_mol errors out on the update_multiplicity part which assumes self.mol is already assigned, but at runtime it's None.  self.mol = mol is assigned further in the function. For the error to disappear it needs to be assigned before the update_multiplicity, but I don't know if that switch would not alter other functionality.
Best,
Andrzej

Thanks for bringing this to my attention.  Please give it a try and then I will update the PyPi and conda releases in the next few days. You should be able to install the updated version directly from GitHub with this command if you don't want to clone the repo: pip install git+https://github.com/kjelljorner/morfeus.git#egg=morfeus-ml

Thank you!


idk for me it showing error

D:\Kernel>python AndroidDriveSignity.py kernel kallsyms new-kernel
Error: check_modinfo symbol not found.
Symbol check_version Offset 0x1F64E00 has PACIASP disabled.
Error: module_sig_check symbol not found.
Kernel file patched and saved to new-kernel
this is the error i am facing can you help me out for solving this

I'm not sure what happened.
Could you upload kernel and kallsyms?

ok sure

Hey where I can contact you like discord or tg?


How can I train the pre-train model on chinese corpus?

Now I want to train a pre-train model on chinese corpus, but the details are not clear. such as, how to make the minimal changes necessary to support Longformer’s attention mechanism, how to take the attention pattern to plug into a pretrained transformer model.

We are working on some script that should make this easy.

@ibeltagy OK，when will you finish this work and publish the pre-train model based Chinese corpus?

We are not going to release a pretrained Chinese model. This is just a script to make it easy to train your own model on your own corpus.
It is a pretty easy procedure. Please check the other issues that asked the same question where we tried to explain the procedure in more detail.

We are not going to release a pretrained Chinese model. This is just a script to make it easy to train your own model on your own corpus.
It is a pretty easy procedure. Please check the other issues that asked the same question where we tried to explain the procedure in more detail.

Thank you very much for releasing the script. I will try to train a pretrained model based Chinese copus after that.
Now I'm trying to use LongformerSelfAttention to replace the encoder in transformer, then build a new model without pretrained model.

This notebook replicates our procedure for pretrianing Longformer https://github.com/allenai/longformer/blob/master/scripts/convert_model_to_long.ipynb. It can be applied to other pretrained models to get a version that supports long inputs.

@ibeltagy Thank you very much. I will try it now。

Closing. Please feel free to reopen or create a new one if you have more questions.

@liangxg787 Hi, have you trained a Chinese model? Could you please release your model?

@thesby now , huggingface have release chinese pretrain model
https://huggingface.co/models?search=longformer


Support Auto-Discovery

This will make the package work with auto-discovery in Laravel 5.5

Thanks!


Update cats-effect, cats-effect-laws to 3.4.9 in series/4.x

About this PR
📦 Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

from 2.5.5 to 3.4.9 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

💡 Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.0.0 (created no change)

github:typelevel/cats-effect/v3_0_0?sha=v3.0.0

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel" }
}]

labels: library-update, early-semver-major, semver-spec-major, version-scheme:early-semver, scalafix-migrations, commit-count:1

Superseded by #1718.


Keyless Entry

Fixes #1203.

Rendered

All possible changes have tradeoffs. For me, backwards incompatibility is a non-starter.
Perhaps there is a way of extending the current Entry API with genericity and conversion traits.
enum Entry<'a, K, V, Q = K> {
    Occupied(OccupiedEntry<'a, K, V>),
    Vacant(VacantEntry<'a, K, V, Q>),
}
// method on HashMap
fn entry<Q>(&mut self, key: Q) -> Entry<K, V, Q>
    where Q: Borrow<K>
// method on VacantEntry
fn or_insert<W>(self, value: W) -> &'a mut V
    where (K, V): From<(Q, W)>

Today's Entry is a special case of this interface with (K, V) == (Q, W). For keyless entry, you'd have a conversion from (Placeholder, (K, V)) to (K, V), and uglier invocations of entry.or_insert((k, v)) as the only disadvantage. If trait coherency is a problem, I suppose there could be a new type for key-value pairs. The biggest issue with genericity is always documentation and much more confusing error messages.

Let's reconsider what needs to be done.
The current Entry API is not enough in two scenarios:

I own a key, and want to retain it if the entry is occupied.
I have a reference to a key, and I'm not sure if I will need to clone a key to insert it into a vacant entry.

The problem in the first scenario is simple, because it only concerns OccupiedEntry.
The problem in the second scenario concerns VacantEntry and fn entry, so it's basically orthogonal to the first problem.
OccupiedEntry doesn't own a key simply because there's no internal need for a retained key. However, the user may want to retain a key, so we modify OccupiedEntry and add a method for key recovery. This fixes the first scenario, which I have in my refactoring PR.
struct NoKey;

struct OccupiedEntry<'a, K, V> {
    key: Option<K>,
    elem: FullBucket<...>
}

impl<'a, K, V> OccupiedEntry<'a, K, V> {
    pub fn take_key(&mut self) -> Option<K> {
        self.key.take()
    }
}

The original purpose of the entry API was to prevent double lookups. This can't fix that while entry_or_clone can.

All possible changes have tradeoffs. For me, breaking backwards compatibility is a non-starter.

I completely agree - I included the backwards incompatible alternative just for completeness. The main proposal is not backwards incompatible in any way.

Perhaps there is a way of extending the current Entry API with genericity and conversion traits like so:

This sort of API would probably be ideal from a usability standpoint, but I don't see any way that it can work. You take Q by value to ensure compatibility with passing a by-value key, but this doesn't work with implementations of Borrow, as, e.g., str: Borrow<String>, not &str, so you would have to pass a str by value, which is impossible. I don't think that any other trait would fare any better, though I can't completely rule that out.
There may also be compatibility issues with broken type inference, but I don't know how much of a problem that would be.

The current Entry API is insufficient in two scenarios:

I own a key, and want to retain that key if the entry is occupied.
I have a reference to a key, and I'm not sure if I will need to clone that key to insert it into a vacant entry.

Exactly. This RFC aims to solve only the second problem.

The original purpose of the entry API was to prevent double lookups. This can't fix that while entry_or_clone can.

I'm confused as to why you think that - not only can this API be used to prevent double lookups, but entry_or_clone can be implemented in terms of this API:
fn entry_or_clone(&mut self, key: &K) -> Entry<K, V> where K: Clone {
    match self.keyless_entry(key) {
        KeylessEntry::Occupied(entry) => Entry::Occupied(entry)
        KeylessEntry::Vacant(entry) => Entry::Vacant(entry.with_key(key.clone()))
    }
}

I'm confused as to why you think that - not only can this API be used to prevent double lookups, but entry_or_clone can be implemented in terms of this API

Sorry, I didn't read the drawbacks. IMO, not being able to guarantee that the key is the same is way too much of a foot gun to be acceptable.

@Stebalien
could you clarify what you mean by

not being able to guarantee that the key is the same

@jmesmon A user could write:
self.keyless_entry(&key1).or_insert_with(||(key2, 0)) += 1;

and end up with key2 in key1's slot.

Alternate alternative: combine this proposal and the entry_or_clone one.  Use an alternate Entry type (or just extend the existing one to support the new functionality), but have its insert method call to_owned() on the key rather than allowing it to be specified explicitly, reducing footguniness at the cost of some flexibility.  (to_owned() should be preferred over clone() because of things like searching a HashMap<String, X> with &str.)

I haven't given up on this issue. It can be decomposed into two essential steps.

I have a borrowed key, and want a keyless entry.
I have a vacant keyless entry, and want to populate it by computing a key with one of these actions:

I compute and give my owned key to the entry.
I want the entry to use a clone of the key it borrows.
The entry already owns an key, and nothing needs to be done.

Let's focus on the first step.
An API with ideal usability has to take a key that is either borrowed or owned. Cow is a natural fit for this purpose and is passed into entry2 in hashmap2. However, we would like to keep backwards compatibility without creating a separate component. Adding Into to the mix allows that:
pub fn entry<'a, Q, B>(&mut self, key: Q) -> Entry<K, V, B>
    where K: Into<Cow<'a, B>>,
          Q: Into<Cow<'a, B>>,
          B: Eq + Hash + ToOwned + 'a,
{
    self.reserve(1);
    let key_cow: Cow<'a, B> = key.into();
    self.search_mut(&key_cow).into_entry(key_cow).expect("unreachable")
}

The compiler will complain about K: Clone not being satisfied in various places. Yes, we broke backward compatibility. I gave up on solving that. Instead, we can write simpler bounds with Borrow in a similar manner. This will require a method with default type parameters that drive inference.
pub fn entry<'a, Q = K, B = K>(&mut self, key: Q) -> Entry<K, V, Q>
    where Q: Borrow<B>,
          K: Borrow<B>,
          B: Eq + Hash,
{
    self.reserve(1);
    self.search_mut(Borrow::<B>::borrow(&key)).into_entry(key).expect("unreachable")
}

These default type parameters are necessary, because some types have two different implementations of Borrow<…> and Borrow<Self>. We ask the compiler's inference to choose Borrow<Self>. This way, backward compatibility is be maintained for the following code:
fn foo<K>(map: HashMap<String, …>, key: String) {
	map.entry(key);
	// Types can be inferred in two ways:
	//   first
	map.entry::<String, String>(key);
	//   second, due to `String: Borrow<str>`
	map.entry::<String, str>(key);
}

As for the second step, the Entry API should be able to provide a function transforming an Entry<K, V, Q> into Entry<K, V, K>.

So I've got a suggestion (playground) which may not require an additional method. The idea is to introduce a new IntoOwned trait which unlike ToOwned takes self by-value:
pub trait IntoOwned<T>{
    fn into_owned(self) -> T;
}

impl<T> IntoOwned<T> for T {
    fn into_owned(self) -> T { self }
}

impl<'a, T> IntoOwned<T> for &'a T where T: Clone {
    fn into_owned(self) -> T {
        (*self).clone()
    }
}

impl<'a> IntoOwned<String> for &'a str {
    fn into_owned(self) -> String {
        self.to_owned()
    }
}

/* ... */

The entry method's signature can then look like this:
    // Entry would have to gain Q: IntoOwned<K> as a type param with default Q=K.
    fn entry<'a, Q>(&'a mut self, key: Q) -> Entry<'a, K, V, Q>
        where K: Hash + Eq,
              Q: Hash + Eq + IntoOwned<K>
    {
        // Build an `Entry` which stores `key` and calls `into_owned`
        // on vacant insertion only.
    }

The trouble with this is that we'd need to duplicate all the existing ToOwned impl-s (with some &'a-s in there). This is because
  impl<'a, T> IntoOwned<T::Owned> for &'a T where T: ToOwned

conflicts with impl<T> IntoOwned<T> for T due to the T::Owned == T possibility. Had we gone this route to begin with (and not have ToOwned) this version might have been more appealing.
As it stands, we could keep IntoOwned unstable and provide impl-s for T: Clone and the std smart pointers. It'd still greatly improve the situation, but the lack of extensibility for any Borrow/ToOwned pairs is disappointing.

Just to bring in some conversation from IRC. The only coherence trouble the impl runs into is of the form impl<T, U> X<T> for U where T != U. The typical arguments against negative reasoning don't apply here (at first glance anyway), so assuming that where clause was expressible the following ought to work:
impl<T> IntoOwned<T> for T {
    fn into_owned(self) -> T { self }
}

impl<'a, T> IntoOwned<T::Owned> for &'a T
    where T: ToOwned,
          T::Owned != &'a T
{
    fn into_owned(self) -> T::Owned {
        (*self).to_owned()
    }
}

I worked around the coherence issues an wrote up an RFC #1769

ping @gereeter @aturon
Status? Should this be closed, given the existence of #1769?

I've just proposed a move to FCP on #1769. I'm going to propose to close this RFC as part of the same decision.
@rfcbot fcp close

I approve of this being closed - I think #1769 is a better solution.

Ok, all checkboxes checked, closing in favor of #1769


Bug 1952576: csv_succeeded metric not present

csv_succeeded metric is lost between pod restarts.
This is because this metric is only emitted when CSV.Status is changed.

Description of the change:
Emit csv_succeeded/csv_abnormal metric during every CSV sync loop.
Motivation for the change:
csv_succeeded metric is lost between pod restarts.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive

/ok-to-test

How can I fix bugzilla/invalid-bug label on the PR?

@josefkarasek The bug bot is complaining about the current state of the BZ: https://github.com/operator-framework/operator-lifecycle-manager/pull/2213#issuecomment-866827324:
expected the bug to target the "4.9.0" release, but it targets "---" instead

In order to fix this, update the BZ's "Target Release" dropdown button to the 4.9.0, instead of the default value (empty release, "---"), save the BZ, and then comment /bugzilla refresh here.

/bugzilla refresh

Also, @timflannagan we don't need the bug number in the PR title right? We'll only need it when we downstream the PR?

Closing in favor of #2216


feat: add count API to collection

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
add count() API method to collection
Related Tickets & Documents

Related Issue #
Closes #321

Added/updated tests?

[ ] Yes
[ ] No, and this is why: please replace this line with details on why tests
have not been included
[x] I need help with writing tests

Is this change backwards compatible?

[x] Yes
[ ] No, and this is why: please replace this line with details on why?

Does it require updates to Tigris docs?

[x] Yes, and here is the link: tigrisdata/tigris-docs#380
[ ] No

Checklist

[ ] npm run build - builds successfully
[ ] npm run test - tests passing
[ ] npm run lint - no lint errors

[optional] Are there any post deployment tasks we need to perform?
@adilansari kindly take a look.
/claim #321

This looks good. Since you updated api submodule locally, it needs to be checked in as well. So just do git add api/proto & commit it.
Now, we only need to add unit tests similar to:
https://github.com/tigrisdata/tigris-client-ts/blob/854fa769c2a73b3cda566974fc5102a7f5274e9f/src/__tests__/tigris.rpc.spec.ts#L480-L494

Ah! Great!
I saw that you said, api/proto has been checked already. I'll go ahead and include the test now.

So @adilansari, I've been able to fix this; I figured that I needed to do npm run clean first, and everything worked fine.
All the tests are passing except the one I just added;
	it("count", async () => {
		const tigris = new Tigris({ ...testConfig, projectName: "db3" });
		const db = tigris.getDatabase();
		const countResponse = await db.getCollection<IBook>("books").count({
			filter: {
				op: SelectorFilterOperator.EQ,
				feilds: {
					author: "Marcel Proust",
				},
			},
		});
		expect(countResponse).toEqual(1);
	});

This is the error, I got: "12 UNIMPLEMENTED: The server does not implement the method count". So, this means that the server will have to expose a count endpoint before this can work, yeah?

You'll have to implement the API on test service like following:
https://github.com/tigrisdata/tigris-client-ts/commit/93d099072219f60879b87e3779c8eb6d82ec92a9#diff-ac9d8c670fceeff6594f39413c9f8183a60948451ae93d349150e6ad9421dcbaR321-R338

Okay. I'll have a look.
Thank you!

@kaf-lamed-beyt let me know if you need more help with test cases?

Ah yes! I'll definetly let you know.
I ran the build and clean scripts, coupled with the prettify script too — this morning, and they were successful. I haven't done the test command.
Hopefully, when I'm with my PC again... I'll do that and let you know. Apologies for not communicating well enough.

Hi @adilansari,
Kindly take a look at what I added to the test service file. My test is still failing though. I even tried updating the code, such that the server and client are in sync — with npm run build still nothing.

I'll accept this for now and fix the failure for you in a separate commit.

@kaf-lamed-beyt I have fixed the unit tests in this commit

I am happy to accept this for now and fix the failure for you in a separate commit. Thank you and please continue contributing @tigrisdata this would be a great learning experience :)

I'd definitely love to keep contributing @adilansari. I'm thinking of taking the docs issues next. What do you think.
I'd also love it if you can help recommend some that you think I might be able to tackle.

@kaf-lamed-beyt I have fixed the unit tests in this commit

Nice!! I skimmed through the files changed just now. I'll have aproper look with my PC when its back on, so I can learn from it.

I am happy to accept this for now and fix the failure for you in a separate commit. Thank you and please continue contributing @tigrisdata this would be a great learning experience :)

I'd definitely love to keep contributing @adilansari. I'm thinking of taking the docs issues next. What do you think.
I'd also love it if you can help recommend some that you think I might be able to tackle.

@kaf-lamed-beyt there are a few doc issues that you can take up: https://github.com/tigrisdata/tigris-docs/issues

I am happy to accept this for now and fix the failure for you in a separate commit. Thank you and please continue contributing @tigrisdata this would be a great learning experience :)

I'd definitely love to keep contributing @adilansari. I'm thinking of taking the docs issues next. What do you think.
I'd also love it if you can help recommend some that you think I might be able to tackle.

@kaf-lamed-beyt there are a few doc issues that you can take up: https://github.com/tigrisdata/tigris-docs/issues

Great! I'm having a look now.


Add Swagger annotations in 'UPDATE'

As a user
I need a UPDATE Swagger Docs
So that  we can use the API
Assumptions:

We would use Swagger
update update_promotions under the PromotionResource #118  in the routes.py
add expect with promotion_model
add marshal_with
add 404 and 400 descriptions in the response

Acceptance Criteria:
Given the /apidocs/
When we try to access PUT /promotions/{promotion_id} in the doc
Then we can see Parameters( promotion_model) and Responses code(404 and 400), and Responses 200 with response json values

When we send UPDATE request(Try it out) with all requirement parameters by following the doc
Then I can get code 200 and the response as the doc said
And I can see the updated promotion (e.g., by using LIST)

When we send UPDATE request(Try it out) with existing ID but NOT all requirement parameters by following the doc
(e.g., request body does not have "start_date")
Then I can get code 400
And I see the promotion is not updated (e.g., by using LIST)

When we send UPDATE request(Try it out) with NOT existing ID
Then I can get code 404

Given update_promotions function in PromotionResource Class
Then swagger annotations exist

We can see all response codes we need, and it worked from /apidocs Try it out !


New template "r"

So far I have chosen a format that does not have a Dockerfile to match many of the devcontainers templates and formats.
https://github.com/devcontainers/templates/tree/main/src
I don't know if this really works as there is no way to use this template, but the format is probably correct as it passes the test copied from https://github.com/devcontainers/templates.

this is cool.  It would be really nice if there was a good way to test the template.
I was testing out the setup in https://github.com/espm-157/devcontainers-rstudio/, and definitely starting to enjoy working directly in the vscode environment.  However working with the free-tier 2 GB ram VMs is really constraining without having packages pre-installed (discovered I could not install the dev version of terra package from source without crashing due to OOM).  For dev-containers to be effective on the free tier having a convenient way to start from pre-built images with most necessary packages installed might be important.

(<cough, cough>  r2u.  No compiling or building, all of CRAN. I still haven't set down for the vscode demo/setup but doing with gitpod.io was a breeze.)

(<cough, cough> r2u. No compiling or building, all of CRAN. I still haven't set down for the vscode demo/setup but doing with gitpod.io was a breeze.)

On amd64, I believe the images used in this template also does binary installs.
I don't think there is a way to do a binary install of the development version packages (Windows and macOS can install from R-universe......)
As for the pain of installing from CRAN, bspm is definitely easier, but installing packages using pak bypasses bspm, so I had to write a separate script to install the R package, and I have been putting off preparing that for now.
rocker-org/devcontainer-images#5

Sorry but that misses the point: add-ons like pak will never get system dependencies automated.  Never.  They can't.
r2u does that by building on the breadth offered by RSPM and by then adding the missing depth of using the in-distro tools to add system dependencies.  This goes both ways:  on 'user library addition', system libraries get added (the stringi -> libicu* or *gsl problems).  Equally importantly, apt is now connected to the use by the R packages and will never uninstall a (versioned) shared library used by an R package because "now it knows".
No pak or rig or ... tool can do that until it integrates with the OS.  Which RSPM likely never will as it focusses on breadth. But you build depth here with the devcontainers so you could (if you chose to) take advantage of it.

r2u does that by building on the breadth offered by RSPM and by then adding the missing depth of using the in-distro tools to add system dependencies. This goes both ways: on 'user library addition', system libraries get added (the stringi -> libicu* or *gsl problems). Equally importantly, apt is now connected to the use by the R packages and will never uninstall a (versioned) shared library used by an R package because "now it knows".
No pak or rig or ... tool can do that until it integrates with the OS. Which RSPM likely never will as it focusses on breadth. But you build depth here with the devcontainers so you could (if you chose to) take advantage of it.

As I have stated many times I fully understand the benefits of it. I just didn't tackle it first.
Of course I will work on this later.

@eddelbuettel I have created an issue to track this. rocker-org/devcontainer-features#66
I have seen Japanese users trying to use bspm on Google Colab and really understand the importance of this.

It seems to make more sense to use r-ver as the template name, given the need to separate the documentation when adding a template based on the R installation via apt later.

Note that, the current VSCode Dev Containers does not reflect upstream's vscode customization settings, so there seems to be no VSCode settings for R on the container launched using this template.
However, looking at https://github.com/devcontainers/templates, the other templates have the same structure, so I would like to leave it as it is first.

Thanks gents, didn't mean to kick off another round of discussion of binary install mechanisms in my comment above, but just for clarity I was specifically referring to the use case of when pre-built binaries are not available upstream.  For example, at this moment I'd very much like the 54 students in my classroom to be able to deploy the current github version of terra (v1.6.31) but none of our cloud environments (the berkeley data hub, the GitHub Codespaces images, RStudio cloud free instances etc) have enough RAM to build from source.  My thought was to just prebuild a docker image and push it to docker hub to use that as the base image in codespaces...  better ideas welcome

I believe Codespaces has the ability to build an image without pushing it to the repository, but still couldn't build it?
https://docs.github.com/en/codespaces/prebuilding-your-codespaces/about-github-codespaces-prebuilds

My thought was to just prebuild a docker image

Similarly/simpler (to me) I have sometimes created ad-hoc .deb packages (from open source parts) on Launchpad. Then it is back to the binary...

@cboettig R-universe now upload binary packages for amd64 jammy, so you can install dev version of terra with this command install.packages("terra", repos = "https://rspatial.r-universe.dev/bin/linux/jammy/4.2").

Thanks @eitsupi, that's really handy!
I also discovered that you were correct about the devcontainer build having more free memory, so I can just drop a one-line Dockerfile into the repo and codespaces can successfully run the install before the instance deploys: https://github.com/espm-157/devcontainers-rstudio/tree/main/.devcontainer.  Obviously this slows the initial deploy down but seems a reasonable compromise here.


⚠ ClamAV has degraded performance

In a3ac0d9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 102 ms

Resolved: ClamAV performance has improved in 13174e1.


A peft model requires .train() to be called in order to store gradients during backprop

A peft model requires .train() to be called in order to store gradients during backprop.
cc @younesbelkada
peft version 0.2.0.dev0
See the example here:
import torch
from transformers import AutoModelForCausalLM
from peft import LoraConfig, get_peft_model, prepare_model_for_int8_training

model = AutoModelForCausalLM.from_pretrained("edbeeching/gpt-neo-125M-imdb", load_in_8bit=True, device_map="auto")

lora_config = LoraConfig(
    r=16,
    lora_alpha=32,
    target_modules=["k_proj", "v_proj", "q_proj", "out_proj"],  # Are these the correct layers to target with LoRA ?
    lora_dropout=0.05,
    bias="none",
    task_type="CAUSAL_LM",
)
prepare_model_for_int8_training(model)
model = get_peft_model(model, lora_config)

with torch.cuda.amp.autocast():
    gen = model.generate(input_ids=torch.LongTensor([0,1,2,3]).unsqueeze(0))
    # model.train() # <-- UNCOMMENT THIS LINE
    output = model(torch.LongTensor([0,1,2,3]).to("cuda"))
loss = output[0].mean()
loss.backward()
for n,p in model.named_parameters():
    print(n, p.grad)

@pacman100 wdyt of adding model.train() in get_peft_model, happy to have a look and open a PR!

Hello @edbeeching, aren't all models by default in eval mode and we need to set model.train() at the start of the training loop? Or am I missing something?

I think you are right, maybe it should be the user's responsibility to add .train() when doing training
What about adding it inside prepare_int8_model_for_training ?

I completely agree that we should call model.train() at the start of the training loop. However in this example if you remove these lines:
prepare_model_for_int8_training(model)
model = get_peft_model(model, lora_config)

Gradients are still calculated on a standard transformers model, even when in eval model.

Hello, the issue was with torch.cuda.amp.autocast(): . Below code works without model.train() to get the gradients:
import torch
from transformers import AutoModelForCausalLM
from peft import LoraConfig, get_peft_model, prepare_model_for_int8_training

model = AutoModelForCausalLM.from_pretrained("edbeeching/gpt-neo-125M-imdb", load_in_8bit=True, device_map="auto")

lora_config = LoraConfig(
    r=16,
    lora_alpha=32,
    target_modules=["k_proj", "v_proj", "q_proj", "out_proj"],  # Are these the correct layers to target with LoRA ?
    lora_dropout=0.05,
    bias="none",
    task_type="CAUSAL_LM",
)
prepare_model_for_int8_training(model)
model = get_peft_model(model, lora_config)

- with torch.cuda.amp.autocast():
gen = model.generate(input_ids=torch.LongTensor([0,1,2,3]).unsqueeze(0))
# model.train() # <-- UNCOMMENT THIS LINE
output = model(torch.LongTensor([0,1,2,3]).to("cuda"))
loss = output[0].mean()
loss.backward()
for n,p in model.named_parameters():
    print(n, p.grad)

A good to know, thanks!


[DO NOT MERGE] Vendor in changes made to containers/image

containers/image returns a more detailed error message for oci and
oci-archive transports when the syntax given by the user is incorrect
Signed-off-by: Urvashi Mohnani<EMAIL_ADDRESS>

@mtrmac PTAL

ACK, please revendor on top of master again.

vendored master. Ready to merge once tests pass.


Sound stutter since commit "9a1df53"

Hi, I tried the new build (2.0.0_Version2) but having the same problem.

Ok can you upload a video of this with the cmd open

Ok here it is:
Record.zip
Looking at my recorded clip, I played the track "Angel" for around 1 minute. When the playback bar reached "01:14", I switch to the track "Kiss Them For Me" and sound beginning to stutter heavily, SpotifyXP hanging that is why I could not even click the red "Errors" dialog ..
Btw, the error message is "java.util.ConcurrentModificationException". Hope that helps.

I think i found the problem. These requests shouldnt be made because all songs are already added to the list. Thats a bug i will try to fix it when I have time

Hi! I grabbed the new build 1b7090e and the problem is still there.

Can you give me a link to the playlist

Sure, here is the link: spotify:playlist:7rTil6uINEa4kPcNn8Pax3

thanks

I just tried the commit 0d6df97 and 6a90f63, still no luck. I wonder if this happens to you as well?

It doesnt happen on my side

I have an idea. Install SpotifyXP on your main machine and check if it still happens

Just an explanation of the code that fetches all tracks. It does a while loop and it gets the total tracks in the playlist and everytime a track is added it will check if the counter equals the total tracks and if it does the while loop ends. So i dont known whats wrong

Can you install Windows XP on a vm and replicate this. Because then I can connect to it and debug the problem

Click on the website on "See it in action" and then run code if it works you have access to the api

The latest commit works fine. Thanks! Maybe one day we will found out the true root cause of this.


Contour and contour3 issues

This PR fix issues related to both contour and contour3 matlab functions.
For contour3 now is necessary to set TreatAs Plotly Option parameter to 'contour3'. For example:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1, 'TreatAs', 'contour3');

Then results looks as follow

For contour TreatAs is no needed. For example:
Z = peaks;
[M,c] = contour(Z);
c.LineWidth = 3;

mf = gcf;
f = fig2plotly(mf, 'offline', 1);

[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1, 'TreatAs', 'contour3');

The camera angle starts at the top, making it look 2d until you move camera manually.
What is the reason for 'TreatAs', 'contour3'?
If the user wishes to plot a 2d contour, they will use contour:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour(Z);

mf = gcf;
f = fig2plotly(mf);

However, we now get the same 2d design if we do not use 'TreatAs', 'contour3' in `contour3:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1);

Moreover if we add 'TreatAs', 'contour3' to contour, it will make it 3d.
If the user wants a 2d contour, they will use contour.
If the user wants a 3d contour, they will use contour3.
Having to manually specify an argument to make contour3 (a 3d contour plot), to be 3d is not a good design, especially since there is a separate contour plot for the 2d contour plots, there is no need to join them together.

[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1, 'TreatAs', 'contour3');

The camera angle starts at the top, making it look 2d until you move camera manually.
What is the reason for 'TreatAs', 'contour3'?
If the user wishes to plot a 2d contour, they will use contour:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour(Z);

mf = gcf;
f = fig2plotly(mf);

However, we now get the same 2d design if we do not use 'TreatAs', 'contour3' in `contour3:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1);

Moreover if we add 'TreatAs', 'contour3' to contour, it will make it 3d.
If the user wants a 2d contour, they will use contour.
If the user wants a 3d contour, they will use contour3.
Having to manually specify an argument to make contour3 (a 3d contour plot), to be 3d is not a good design, especially since there is a separate contour plot for the 2d contour plots, there is no need to join them together.

[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1, 'TreatAs', 'contour3');

The camera angle starts at the top, making it look 2d until you move camera manually.
What is the reason for 'TreatAs', 'contour3'?
If the user wishes to plot a 2d contour, they will use contour:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour(Z);

mf = gcf;
f = fig2plotly(mf);

However, we now get the same 2d design if we do not use 'TreatAs', 'contour3' in `contour3:
[X,Y] = meshgrid(-5:0.25:5);
Z = X.^2 + Y.^2;
contour3(Z);

mf = gcf;
f = fig2plotly(mf, 'offline', 1);

Moreover if we add 'TreatAs', 'contour3' to contour, it will make it 3d.
If the user wants a 2d contour, they will use contour.
If the user wants a 3d contour, they will use contour3.
Having to manually specify an argument to make contour3 (a 3d contour plot), to be 3d is not a good design, especially since there is a separate contour plot for the 2d contour plots, there is no need to join them together.

For some reason the code you tested did not have the updateContour3.m file fully updated. This made the problem with the default camera angle still continue. I just updated it and made a new commit. With this new commit the camera angle should work perfectly. Please update the new commit and retest
Regarding the TreatAs parameter you only need to follow the instructions to run contour and contour3 functions
I honestly think it's not difficult at all. simply if you are going to use the contour3 function you need to set TreatAs as contour3. If you are going to use contour you don't need to set anything. Is it very difficult to follow these instructions?
This must be the case, because the structure that matlab gives us does not allow us to distinguish internally in the code if it is a contour or contour3. We cannot internally discriminate that in the code. Then an additional parameter is necessary.
Please review the fig2plotly code in detail so that you see that this is so


uncaught exception: only asciicast v1 and v2 formats can be opened

Hi everyone,
I try to use asciinema-player but I have some trouble. I have installed the latest version of asciinema-player and implemented as it's installation README.md file. I am developing in Angular2+ this project. The asciinema-player code like below.

    <asciinema-player style="width: 100%; height: 100%;" src="https://asciinema.org/a/234381"</asciinema-player>

When I click to play button in asciinema-player, I gave an error like below.
I have tried https://asciinema.org/a/234381.cast, https://asciinema.org/a/234381.json 's file extensions and there are no reason. I encountered uncaught exception: only asciicast v1 and v2 formats can be opened error.
Has any idea about this problem?

Regards.
Erkan

Go to https://asciinema.org/docs/embedding
Should be a

Hi @mathieu-aubin,
Thanks for your reply. I have worked on this issue. I got the solution like below.
I am using Angular8 in my project. If I write    in the index.html, it works well. But if I want to use it in the component, I got problems. Because it seems like script injections.
I tried the script like below.
typescript
  scriptHtml = '';

  constructor(private sanitizer: DomSanitizer) { }
  //
  ngOnInit() {
    this.safeScript = this.sanitizer.bypassSecurityTrustHtml(this.scriptHtml);
  }

html

In this scenario, there are no value to show. Nothing to do.
I have tried another way and it works well.
typescript
  safeHtml: any;
  videoNumber: string;

  constructor(private sanitizer: DomSanitizer) {
  }
  ngOnInit() {
      this.videoNumber = '210847';
      this.safeHtml = this.sanitizer.bypassSecurityTrustHtml('');

  }

html

At this time, it worked well. I tried many thing to show asciinema in , but it did not worked.

Regards
Erkan

I wrote a index.html and put the downloaded .js, .css, asciicast.json in the same directory.

Results
I tried to open index.html in my linux firefox,

in a windows explorer,

in a windows chrome,

in a windows firefox, it works !

Only in the windows firefox it works. (not the linux firefox :cry:)

Hi @alexlai97
I think that you should change your source from .json to .cast file extension. I have tried like below, It worked in windows, Opera Browser, Firefox and Chrome.  I haven't tried in Linux, yet.

  ...
  <link rel="stylesheet" type="text/css" href="./asciinema-player.css" />
  ...

  ...
  <asciinema-player src="./ascii.cast"></asciinema-player>
  ...

Regards
Erkan

Hi, @coderkan
The .json file works (only) in windows firefox for me.
I tried the .json, .cast files, and it doesn't work for me on linux for firefox, vimb, surf, same error.
Should I enable CORS ? How do I do that ?

I am seeing the same error for my v2 (JSONL) asciicast file in Chrome, Firefox, and Safari on OS X.
Strangely, when I open the same file in development on localhost:3449/element.html, it plays fine.
The issue only occurs on the production build of asciinema-player.

Aha! Seems like my issue is more related to #72 - I was using the <asciinema-player> tag inside some React-rendered JSX.
I moved my tag to the static-HTML part of my page, and (still using a production build) the cast began playing, no problem.
So something is conflicting with React. I can work around this on my page for now, but down the road I may need the compatibility improved. Fingers crossed, I guess.

Hello everyone:
I alse meet this error too
I use asciinema-player in vue page
src set localhost .cast file
it always report:  Uncaught only asciicast v1 and v2 formats can be opened
Uncaught only asciicast v1 and v2 formats can be opened
But I set same .cast file into single html file,  it work good
the html file is same to vue page
why? why? why?

I also had this problem, have you solved it?@liminghua999

The player has been rewritten from scratch since all of the above was said so I'm going to close it. Feel free to try v3.0.

i  used vue3.0  and typescript  ，how to use this, i also had this problem : Uncaught only asciicast v1 and v2 formats can be opened


added modulo operation and appropriate compiler in README.md file

i added modulo operation feature and updated the README file with the right compiler command.

Hi @rodgersxy you're working on am old instance of the repo. You need to sync your fork. There is no src directory it was a miskae made before now and has been corrected. Sync your fork to be updated. And also work on a independent version of your calculator. Where you get to implement everything your self. You can seee  previous contributions and follow suite

ok, i have noticed that guys are building their own calculators ground up
in C programming within the same repo. So basically it is a folder with
your work and a README file?
On Mon, Jul 10, 2023 at 7:27 PM Destiny Saturday @.***>
wrote:

Closed #19 https://github.com/ALX-SE-Algorithmia/Demo-Project/pull/19.
—
Reply to this email directly, view it on GitHub
https://github.com/ALX-SE-Algorithmia/Demo-Project/pull/19#event-9777063387,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALJJAJJ2CNCTYDRMEG5TP4LXPQUOBANCNFSM6AAAAAA2EXHR5M
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Yes Boss you are right, the demo project isn't to build together on one calculator and make it better, it is to make people want to get a feeling of what it means to contribute before starting complete fledge projects.


protein serine/threonine kinase activity definition

Gillian, (a FB curator) pointed out that some of the
protein kinase definitions need a little tweaking:
term: protein serine/threonine kinase activity
goid: GO:0004674
definition: Catalysis of the reaction: ATP + a protein
= ADP + a phosphoprotein.
definition_reference: EC:<IP_ADDRESS>
>
Catalysis of the reaction: ATP + a protein
serine/threonine = ADP + protein serine/threonine
phosphate.
[to be consistent with protein-tyrosine kinase activity
; GO:0004713]
term: protein serine/threonine phosphatase activity
goid: GO:0004722
definition: Catalysis of the reaction: a phosphoprotein
+ H2O = a protein + phosphate.
definition_reference: EC:<IP_ADDRESS>
>
Catalysis of the reaction: protein serine/threonine
phosphate + H2O = protein serine/threonine + phosphate
[to be consistent with protein-tyrosine phosphatase
activity ; GO:0004725]
term: protein serine/threonine phosphatase complex
goid: GO:0008287
definition: An enzyme that catalyzes the removal of a
phosphate group from a protein.
definition_reference: EC:<IP_ADDRESS>
>
A complex, often consisting of a catalytic and a
regulatory subunit, which catalyzes the removal of a
phosphate group from a serine or threonine residue of a
protein.
Becky
Reported by: beckyfoulger
Original Ticket: "geneontology/ontology-requests/1566":https://sourceforge.net/p/geneontology/ontology-requests/1566

thanks www.movied.org


'SnapshotsOperations' object has no attribute 'resource_groups'

This is autogenerated. Please review and update as needed.
Describe the bug
Hi
I have a python error when trying to create a snapshot on a netapp volume
Command Name
az netappfiles snapshot create
Errors:
The command failed with an unexpected error. Here is the traceback:
'SnapshotsOperations' object has no attribute 'resource_groups'
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 718, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/netappfiles/_exception_handler.py", line 18, in netappfiles_exception_handler
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/netappfiles/custom.py", line 425, in create_snapshot
    location = client.resource_groups.get(resource_group_name).location
AttributeError: 'SnapshotsOperations' object has no attribute 'resource_groups'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az netappfiles snapshot create --account-name {} --name {} --pool-name {} --resource-group {} --volume-name {}

Expected Behavior
Environment Summary
Linux-5.4.0-110-generic-x86_64-with-glibc2.31, Ubuntu 20.04.4 LTS
Python 3.10.4
Installer: DEB

azure-cli 2.37.0

Extensions:
front-door 1.0.16
azure-firewall 0.12.0
ip-group 0.1.2

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

Additional Context

@bd-clara Apologies for the late reply. Thanks for reaching out to us and sharing this feedback. I am looking into this issue. I will reach out to you, if I need more information.

@bd-clara I was able to create the netappfiles snapshot successfully. I am sharing you the commands which I ran inorder:

az login
az account set --subscription "XXXXXXXX"

This command creates the Resource group

az group create --name MyRgName --location eastus

This command creates the netappFiles account

az netappfiles account create -g MyRgName --name MyAccountname -l eastus --tags Reason=Repro

This command creates the pool

az netappfiles pool create -g MyRgName --account-name MyAccountname --name Mypoolname -l eastus --size 4 --service-level standard

This command creates volume

az netappfiles volume create -g MyRgName  --account-name MyAccountname --pool-name Mypoolname --name myvolname -l eastus --service-level premium --usage-threshold 100 --file-path "unique-file-path" --vnet myvnet --subnet mysubnet --protocol-types NFSv3 NFSv4.1

This command creates snapshot

az netappfiles snapshot create -g MyRgName --account-name MyAccountname --pool-name Mypoolname --volume-name myvolname --name Mysnapname -l eastus

@bd-clara I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please let us know if you need any further assistance on this. Awaiting your reply.

@bd-clara The action is currently pending on you to follow the above suggestion. We will now proceed with closure of this GitHub issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.

It looks like OK, probably I forget to do an az account set -s.
It should be probably better catched
Best regards,


BLD: Use latest version of toolz.

Coverage decreased (-0.05%) to 87.169% when pulling d704851489a3573e43e45620d5d903420624e4cc on update-toolz into 790f307485bdb75fe6142392bee31d482e553c77 on master.

Coverage decreased (-0.05%) to 87.169% when pulling d704851489a3573e43e45620d5d903420624e4cc on update-toolz into 790f307485bdb75fe6142392bee31d482e553c77 on master.


Add contributing guidelines

Description of the idea
This is very important especially for new contributors, so we should add contributing guidelines.
Add (multiple) screenshot links (if applicable)
No response
Add any other context about the idea here
Any suggestions will do. Feel free to comment below
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Should I just include to core contributing guidelines?

Hi @krishdevdb, if you wanna work this issue, feel free to do it :)


Variable Availability during compile/parse-time vs run time

From above:

Another common use of expressions is in defining variables. Expressions can be evaluated at compile time or at run time. Compile-time expressions can be used anywhere; runtime expressions are more limited.
The difference between these syntaxes is primarily what context is available. In a parse time expression, you have access to parameters and statically-defined variables. In a runtime expression, you have access to more variables but no parameters.

I'm looking to understand exactly which variables are available during each of these phases, because it seems there are some subtle differences, specifically around queue-time variables.
To be very specific: I would like to control conditional insertion at the root pipeline based on a manually-entered queue-time variable.
For instance:

At parse-time, if only statically-typed variables are available, how am I able to perform conditional insertion with supposed queue-time variables like build.sourcebranch?
  - template: template.yml
    parameters:
      ${{ if eq(variables['build.sourcebranch'], 'refs/head/master') }}:
        param_1: value a
      ${{ if ne(variables['build.sourcebranch'], 'refs/head/master') }}:
        param_1: value b

However 'normal' queue-time variables are unavailable. That is, a variable not defined in the YAML, but is added in the UI when triggering a new build in AzureDevOps:

This works fine (statically-typed):
  variables:
    releaseBranchName: 'refs/head/master'

  - template: template.yml
    parameters:
      ${{ if eq(variables['build.sourcebranch'], variables['releaseBranchName']) }}:
        param_1: value a
      ${{ if ne(variables['build.sourcebranch'], variables['releaseBranchName']) }}:
        param_1: value b

But I've been unable to support manually overriding conditional insertion based on a 'normal' queue-time variable supplied from the UI  (ie: releaseBranchNameOverride):
  variables:
    # expected to work
    releaseBranchName: ${{ coalesce(variables['releaseBranchNameOverride'], 'refs/head/master') }}
    # not expected to work (runtime expressions are processed after compile-time, so conditional insertion can't be based on a runtime expression)
    # releaseBranchName: $[coalesce(variables['releaseBranchNameOverride'], 'refs/head/master')]

  - template: template.yml
    parameters:
      ${{ if eq(variables['build.sourcebranch'], variables['releaseBranchName']) }}:
        param_1: value a
      ${{ if ne(variables['build.sourcebranch'], variables['releaseBranchName']) }}:
        param_1: value b

In the commented-out runtime expression, accessing releaseBranchName as part of a Task, or even in a condition, would result in the expected value, which is confusing at first. This is one reason folks may want Template expansion to support Conditions, though I understand that's a runtime concern (perhaps solved w/ Condition projection on compilation?).

The latter results in the following InitializePipeline.txt when run against the branch refs/pull/1/merge and with releaseBranchNameOverride of refs/pull/1/merge:
Begin evaluating template 'template.yml'
Evaluating: coalesce(variables['releaseBranchNameOverride'], 'refs/head/master')
Expanded: coalesce(Null, 'refs/head/master')
Result: 'refs/head/master'
Evaluating: coalesce(variables['build.sourcebranch'], 'source-branch')
Expanded: coalesce('refs/pull/1/merge', 'source-branch')
Result: 'refs/pull/1/merge'
Begin transform: variables
Evaluating: value
Result: Object
Evaluating: pair['key']
Result: 'releaseBranchName'
Evaluating: pair['value']
Result: 'refs/head/master'
Evaluating: pair['key']
Result: 'sourceBranchName'
Evaluating: pair['value']
Result: 'refs/pull/1/merge'
End transform: variables

The end result: Despite being a 'static' (not expression) + known 'queue-time' value (which is seemingly known prior to running the pipeline), Template Parameter param_1 will not resolve to value a when a matching override value for releaseBranchNameOverride is supplied at queue-time.

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

ID: 77c58a78-a567-e99a-9eb7-62dddd1b90b6
Version Independent ID: 680a79bc-11de-39fc-43e3-e07dc762db18
Content: Expressions - Azure Pipelines
Content Source: docs/pipelines/process/expressions.md
Product: devops
Technology: devops-cicd
GitHub Login: @juliakm
Microsoft Alias: jukullam

Same issue here. Trying to run a template based on a condition. The conditional variable is set through the UI during queue time.
steps:
  - ${{ if ne(variables['runSetup'], 'false') or ne(variables['runSetup'], 'False') }}:
    - template: templates/080-muchos-setup.yml

The template is executed irrespective of the value set in runSetup.

Same issue here. Trying to run a template based on a condition. The conditional variable is set through the UI during queue time.
steps:
  - ${{ if in(variables['runSetup'], 'true', 'True') }}:
    - template: setup.yml

The template is never executed irrespective of the value set in runSetup.

Using $[ instead of ${{ results in the following error.
steps:
  - $[ if in(variables['runLaunch'], 'true', 'True') ]:
    - template: templates/050-muchos-launch.yml

/pipelines/bringup-accumulo-cluster-on-demand.yml (Line: 59, Col: 5): Unexpected value '$[ if in(variables['runLaunch'], 'true', 'True') ]',/pipelines/bringup-accumulo-cluster-on-demand.yml (Line: 66, Col: 5): Unexpected value '$[ if in(variables['runWipe'], 'true', 'True') ]',/pipelines/bringup-accumulo-cluster-on-demand.yml (Line: 70, Col: 5): Unexpected value '$[ if in(variables['runSetup'], 'true', 'True') ]'

can you do something like utcnow() or utctoday() in expressions?

@juliakm @WilliamAntonRohm Is there any update on this? This looks to me more like a bug in the product and not just a document bug. Can you please get the right people to look into this? Thanks!

@JoeBrockhaus, @ata18, @mwc360 -- if you've encountered an issue with the product itself, rather than an issue with the documentation, please submit your bug here.
You may find additional information on variable availability here:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot


[BUG] Build now fails due to sites/default being unwritable

Noticed a build failure when I pushed a commit to an open ISLE-DC PR this evening. Log is here.

Notice: ] Performed install task: install_select_profile
Notice: ] Performed install task: install_load_profile

In install.core.inc line 2329:

  Settings file:The Settings file is not writabl
  e.

The Drupal installer requires write permissions to ./sites/default/settings.php during the installation pro
  cess. The webhosting is
  sues documentation section offers help on this and other topics.

make[1]: *** [Makefile:580: starter-finalize] Error 1
make[1]: Leaving directory '/home/runner/work/isle-dc/isle-dc'
make: *** [Makefile:159: starter] Error 2
Error: Process completed with exit code 2.
0s
0s
0s
0s
0s
0s
0s
1s

```shell

I know we've dealt with similar issues in the past, is it a matter of doing another chmod / chown?

This might be the same issue here: https://github.com/Islandora-Devops/isle-dc/issues/306

Something such as Drupal's $settings['skip_permissions_hardening'] setting might be useful; though, would only expect it to be used in dev environments.


Adding environment variables for sub projects

The docker/docker documentation Makefile is used to make and push live docs.docker.com. This includes the documentation from the sub-projects such as Swarm, Compose, etc.  The Dockerfile was pulling uniformly from master for these projects.  This caused the early release of some Compose material when commits were cherry-picked say, for the birthday, and pushed live.
This is also fixing broken make docs:  fixes #12186
I've added environment variables to this file so we can pick from specific branches or tags for the particular branches.
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

ping @jfrazelle @tiborvass @fredlf   If this looks good, I'd like to merge this change, cherry pick it into docs and fix the existing problem with Compose today.

You forgot to include fixes ... in the description (sorry, can't edit on mobile)

LGTM in principle, but need confirmation from @jfrazelle and @tiborvass that this actually does what we want it to.

LGTM

Thanks everyone!


[USB Serial/JTAG] [VFS] add flag if driver has ever received bytes (IDFGH-8651)

This PR was spun out from https://github.com/espressif/esp-idf/pull/9983
introduces: usb_serial_jtag_has_ever_received_bytes_since_install();
I need a way to detect if a USB terminal is attached, so that I can decide which port to open my terminal on. If the user types a character over USB during boot, I open USB Console, otherwise I open UART Console.
(I also send a 'terminal probe request' during boot, so USB Console detection is automatic for most terminals 😊)
I like that this solution is less complex (i.e. more reliable) than calling read() or initializing linenoise or esp_console.

 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.

Looking around for alternatives, maybe I could instead use this function?
static inline int usb_serial_jtag_ll_rxfifo_data_available(void)
I'd just need to make sure nothing else reads the data, but I think thats already the case unless I initialize the usb_serial_jtag_driver.

I'll test usb_serial_jtag_ll_rxfifo_data_available, and close this PR if I find it to be sufficient.

Closing this PR.
This functionality can now be better implemented with bool usb_serial_jtag_is_connected(void) which was added to master on Februrary 26, 2023. https://github.com/espressif/esp-idf/commit/1a66459b4454f0b45b1910b2bf260c77aef119f4


What is FD 18 testing?

Hi.
Thanks for your awesome test files... I've been using them to implement a Z80 emulator.
My emulator passes everything I've implemented, but I can't get it to pass the test "FD 18".
Do you know what this is testing? I can't find that instruction combination documented anywhere...
Also, I seem to have a few unimplemented instructions that you test for. Are you privvy to some secret documentation? :D
Thanks,
Steve.

Hey,
Thanks for using the tests! Remember if you find a mistake, it may be my fault; do look up the docs. I made these because my original Z80 core was trash, and I had really benefited from similar tests by someone else.
There's definitely some documentation for how X,Y and WZ work that isn't easy to find. The Undocumented Z80 Documented is a good start.
The source for the tests is available in this repository under misc/code_generation/z80_test_generator.js, and my Z80 core is available in the component/cpu/z80 sub-directory.
As to what FD 18 is testing...I guess this may or may not be a bug. Regular opcode 0x18 is defined as "JR_c_e", which is an unconditional relative jump. If you read Z80 Undocumented, you'll see (in reference to DD and FD):
In general, after a DD prefix the instruction is executed as if the DD
weren't there. There are some exceptions:

* Any access to HL is treated as an access to IX (except EX DE,HL and EXX
  and the ED prefixed instructions that use HL).
* Any access to (HL) is changed to (IX+d), where d is a signed displacement
  byte placed after the main opcode (except JP (HL), which isn't indirect
  anyways.
* Any access to H is treated as an access to IXh (the high byte of IX)
  Except if (IX+d) is accessed.
* Any access to L is treated as an access to IXl (the low byte of IX)
  Except if (IX+d) is accessed.
* A DD prefix before a CB selects a completely different instruction
  set (see 1.5).

Anyway, none of those is fulfilled, so it's just treated as a regular 0x18. I know it doesn't show up in opcode tables like that, but it's what I've always read was the behavior for those opcodes.
I'll ask on the ZX Spectrum Emulator Discord, there's a lot of people more knowledgeable about the Z80 than I am there.
Thanks again for using the tests! I'll post back here when I get a response


Remove authorization header on /webhook/callback routes

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming features.

Feature Request
Currently JDA sends the Bot Authorization token on the Interactions route /interactions/:ID/callback which isn't needed but if provided still counts against the Bots Global Ratelimit, it would be desirable to not fill up the global ratelimit with requests which do not need to count against the global ratelimit if the header would be explicitly removed for this route 👍
Example Use-Case
should be clear but the use case would be to have less issues hitting the Global Ratelimit on bots who heavily use interactions (i had to learn this the hard way).

I think this could be handled by adding a Route#needsAuth which can be checked in the requester to ensure those endpoints don't send an authorization header.
Add a constructor parameter for needsAuth and a getter in route:
https://github.com/DV8FromTheWorld/JDA/blob/a35789442ba022c81ce9b5f62d5b6b9968e3895d/src/main/java/net/dv8tion/jda/internal/requests/Route.java#L328
Add a check on apiRequest.getRoute().getBaseRoute().needsAuth() here:
https://github.com/DV8FromTheWorld/JDA/blob/a35789442ba022c81ce9b5f62d5b6b9968e3895d/src/main/java/net/dv8tion/jda/internal/requests/Requester.java#L173-L174

For reference i also made an issue on discord api docs since its weird behavior which should be fixed on there side https://github.com/discord/discord-api-docs/issues/4142 but unsure if they consider fixing this.

This is not a problem with JDA and we will not be changing this as it has been stated that the "counts against the Bots Global Ratelimit" is an internal discord bug.

Fix is merged and should be rolling out soon(tm)


Setup GPIO Pin

Setup script should prompt for GPIO pin and set the config accordingly. Note that this should involve avoiding prompting for pins that CANNOT EVER be operated in GPIO mode.

See #25


C++ version with timer: Cannot set offset to 0x97: Bad file descriptor

Hi, I get the following error when I try to run the c++ version with a timer:
Cannot set offset to 0x97: Bad file descriptor

It work fine with no timer.
This is on Arch Linux on a Dell D5 SE 5505.
Thank you.

Actually nevermind, I had to do the steps to enable ec_sys with write_support.


librdkafka dependence when used at Windows

DLL load failed while importing cimpl: The specified module could not be found when building utils.

Hi, can you provide more information about exactly which part are you running?
Kafka is known for not running smoothly on windows, I'd suggest if possible, run in a lunix-based environment.

Hi - I guess the problem is not with Kafka. It's with "confluent_kafka" package as when you try to build with consumer code it indicates required libary "librdkafka " for windows is not available and hence code is not building required producer or consumer. I agree, it might work well in Linux.

Hi, This may help, but the author of librdkafka stands that it doesn't officially support Windows.
I think this would be a better issue for the library itself.

I'm closing this issue, let me know if I can help with something else.


Orange hyperlinks

Fixes #539.
Before:

After:

The brand guide is on point. Any good, simple web design has a single, strong accent color. It sticks to that one color religiously, and it uses it a lot.


Fix lookup for enum entries.

Example: ?yf SURVIVAL yields no result. It should be resolving the GameMode.SURIVIVAL at minimum

I've set up a test instance using the kord-0.7 branch, f00413c, and that seems to work:

I don't know of any other real examples that would break. Stuff like GameRule.CREATIVE would be pending #28 and is out of scope of this issue.


test: add unit test case for func GetSuperCID

Signed-off-by: Hu Shuai<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1271 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1271      +/-   ##
==========================================
+ Coverage   48.51%   48.53%   +0.02%
==========================================
  Files         121      121
  Lines        7788     7788
==========================================
+ Hits         3778     3780       +2
+ Misses       3698     3696       -2
  Partials      312      312

Impacted Files
Coverage Δ

supernode/config/config.go
77.08% <0.00%> (+4.16%)
:arrow_up:

Continue to review full report at Codecov.

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


TAble ID could be a non-integer UUID

Larastan Version: 0.6.13
--level used: 4

Description
Cannot cast Ramsey\Uuid\UuidInterface to int.
Laravel code where the issue was found
https://github.com/conedevelopment/bazar/blob/2fa9b9c820294e328ee879dbb9c3a5f04e3eb253/src/Models/Item.php#L95-L99
The migration:
https://github.com/conedevelopment/bazar/blob/2fa9b9c820294e328ee879dbb9c3a5f04e3eb253/database/migrations/2020_01_01_000300_create_bazar_items_table.php#L14-L17

Hi,
On top of the model file you can add /** @property Uuid $id */ to overwrite it.

I know - I know :)
How could one solve it in Larastan?

Somehow resolved in v0.7.0


Removing set doesn't add back recipes removed by said set

Version: craftingharmonics-1.9.4-<IP_ADDRESS>.jar
On my server I added one of my custom sets which removes all TiC tool forges and only adds back only the Knightslime version. When I /ch removeSet, it removed the recipe I added but did not restore the recipes I had originally removed in the set.

Huh.  That's... odd.
I'll take a look at it; I'm planning on releasing 1.5.0 today, if this is a quick fix, or I can easily determine the root cause, I will go ahead and push the fix with 1.5.0, and, if not, I'll try and get something in for 1.5.1.
Is this happening in a current version of your pack on Curse so I can download and test with the modlist and config, or is it in a separate / testing instance?
Thanks.

Be my guest! :smile:
http://minecraft.curseforge.com/projects/rebel-uprising

Sorry, I just got around to testing this, and can't reproduce it (in SSP) with your config with CH1.5.0.  Are you still having this problem in recent versions?
Also, have you noticed if this is just on server, or in single-player as well?
Thanks.

I honestly don't do much testing in SSP since my primary concern is server play.  I haven't gotten a chance to update my pack yet so I'll let you know if I still have the problem once I do.


[BUG]: Update multiple rows is not working. PostgresError: column excluded.name does not exist

What version of drizzle-orm are you using?
^0.30.0
What version of drizzle-kit are you using?
^0.20.14
Describe the Bug
I'm trying to insert multiple rows and update onconflict for those rows that have conflict.
export type OrderPhases = {
    restaurant_id: number;
    uuid?: string;
    name: string;
    from: string;
    to: string;
    delivery_time: string;
};

export const updateRestaurantSettings = async (
  orderPhaseValues: OrderPhases[],
  deliveryLocationValues: { restaurant_id: number; location: string }[]
): Promise<void> => {
  try {
    await db.transaction(async (tx) => {
      await db
        .insert(order_phases)
        .values(orderPhaseValues)
        .onConflictDoUpdate({
          target: order_phases.uuid,
          set: {
            name: sql.raw(`excluded.${order_phases.name.name}`),
            from: sql.raw(`excluded.${order_phases.from.name}`),
            to: sql.raw(`excluded.${order_phases.to.name}`),
            delivery_time: sql.raw(`excluded.${order_phases.delivery_time.name}`),
          },
        });

      await db.insert(delivery_location).values(deliveryLocationValues);
    });
  } catch (error) {
    throw new Error("Error in updateRestaurant: " + error);
  }
};

this is my schema for order_phases
export const order_phases = pgTable(
  "Order_Phases",
  {
    id: serial("ID").notNull().primaryKey(),
    restaurant_id: integer("Restaurant_ID")
      .notNull()
      .references(() => restaurants.id, { onDelete: "cascade" }),
    uuid: uuid("UUID").defaultRandom().unique(),
    name: text("Name").notNull(),
    from: text("From").notNull(),
    to: text("To").notNull(),
    delivery_time: text("Delivery_Time").notNull(),
  },
  (t) => ({
    unq: unique().on(t.restaurant_id, t.name),
  })
);

I wonder if I'm using excluded wrongly?
I have a requirement whereby i will be inserting/updating a list of values at once and each item will be a row in the table, I wonder how am I suppose to delete multiple rows for those values that are no longer in the list when send to updateRestaurantSettings function  ? Is it by checking the orderPhaseValues in updateRestaurantSettings and the order_phases table, then if the value exist in the table but doesnt exist in orderPhaseValues, that value needs to be deleted? I wonder if there's a more efficient way to perform this deletion.
Expected behavior
No response
Environment & setup
No response

@haocloo what was the problem in the end? I don't see that there was any specific PR for this issue, so I imagine there was some mistake in your code?
I'm asking because I'm probably making that same mistake.

I have this same error.

re-added this bug report here:
https://github.com/drizzle-team/drizzle-orm/issues/3024

really does seem to be a casing issue. Just renamed my column teorAlcoolico to teoralcoolico and it is now accusing another column:


[ceskatelevize] Fix playlist playlist url

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Extractor for ceskatelevize.cz was broken because the playlist URL changed. This fix contains only one char addition.

Duplicate #30154

Ok. Should I close it?

Sounds good.


While using the auto-hide sidebar feature, my chat goes o...

Description:
While using the auto-hide sidebar feature, my chat goes outside the edge of the window by a large margin, and I am unable to read responses. When hovering over the left side the sidebar appears as intended, but the other person's messages remain outside the window.
Steps to reproduce:
Turn on auto-hide sidebar feature under the "View" menu.
App version:
2.01 Beta
OS:
Windows 10 (latest)
Works in WhatsApp Web:
Extra:
Great UI for this report site, clean and generally aesthetically pleasing. You have a great design team.

I'm able to reproduce this bug.


Removing validation check from AzurePowerShellV4

This task has stopped working after this update. It was working fine for us until today - now we are receiving the error that this PR was meant to eradicate.

@andydkelly-ig
These changes has not been yet deployed so this should not be ideally cause of the failure. Can you please share debug logs.

ah - in that case we are now seeing the error this should resolve :) I worked around by changing Powershell V4 Task from "latest version" to 1.0.0

ah - in that case we are now seeing the error this should resolve :) I worked around by changing Powershell V4 Task from "latest version" to 1.0.0

Fantastic @andydkelly-ig  - that has solved the problem for me!


Raw Packet Support

TCP
UDP

Possible implementation: libpcap/xdp?

Superseded by https://github.com/projectdiscovery/interactsh/issues/506


Charm++ NQueens example

Original issue: https://charm.cs.illinois.edu/redmine/issues/564

NQueen and queens examples -- I don't see big differences between these, can be combined.

Original author: Yanhua Sun
Original date: 2014-10-14 20:48:49

removed nqueens and keep NQueen


Galactic Documentation error on createInstance

the docs and comments refer to a function called createInstance, but the function is actually named createSharedInstance
How can I make a pull request to fix that?

Since Galactic is EOL, and we removed the offending comments anyway, I'm going to go ahead and close this.


building docker compose in VM Ubuntu

this the content of my docker file    FROM nginx:1-alpine
RUN apk add --no-cache openssl inotify-tools
WORKDIR /etc/nginx/
ADD nginx.conf.envsubst nginx.https.available.conf.envsubst ./
ADD geonode.conf.envsubst ./sites-enabled/
ADD docker-autoreload.sh docker-entrypoint.sh /
ENTRYPOINT ["/docker-entrypoint.sh"]
RUN chmod +x /docker-autoreload.sh
RUN chmod +x /docker-entrypoint.sh
CMD ["nginx", "-g", "daemon off;"]                when i run the command build docker compose   i face this error:   ERROR [2/8] RUN apk add --no-cache openssl inotify-tools                                                     196.0s

[2/8] RUN apk add --no-cache openssl inotify-tools:
187.6 fetch https://dl-cdn.alpinelinux.org/alpine/v3.18/main/x86_64/APKINDEX.tar.gz
192.6 WARNING: fetching https://dl-cdn.alpinelinux.org/alpine/v3.18/main: temporary error (try again later)
192.6 fetch https://dl-cdn.alpinelinux.org/alpine/v3.18/community/x86_64/APKINDEX.tar.gz
195.7 ERROR: unable to select packages:
195.7   inotify-tools (no such package):
195.7     required by: world[inotify-tools]
195.7   openssl (no such package):
195.7     required by: world[openssl]

Dockerfile:3
1 |     FROM nginx:1-alpine
2 |
3 | >>> RUN apk add --no-cache openssl inotify-tools
4 |
5 |     WORKDIR /etc/nginx/
ERROR: failed to solve: process "/bin/sh -c apk add --no-cache openssl inotify-tools" did not complete successfully: exit code: 2
ERROR: Service 'geonode' failed to build : Build failed      I am not professional and it is a project of PhD which i have to done so i need a little bit info on the steps to solve the problem.

Hi @niloufar07, I'm seeing the same error. Were you able to resolve the issue?


Prepare for scpca-docs release: Portal wide metadata

Steps for a new release of scpca-docs

This release should include changes related to adding portal wide metadata.
The changelog will need to have a date added before this can happen.
Preparing for the release

[x] Does this release require an update to the CHANGELOG because there will be changes to downloadable files? If so, please create an issue tracking the CHANGELOG update and mark it as blocking this issue.
[ ] Are all other issues planned for this release resolved? If any issues are unresolved, mark this issue as blocked by those on ZenHub.
[ ] Optional: If not all changes in development are ready to be released, create a feature branch off main and cherry pick commits from development to that feature branch.
[ ] File a PR from the development branch (or the new feature branch) to the main branch. This should include all of the changes that will be associated with the next release.

[ ] If a CHANGELOG entry was required, add the date to the entry's header as part of this PR.

Creating a release

[ ] On the releases page, choose Draft a new release.
[ ] In Choose a tag, use the current date as the release tag (YYYY.MM.DD), then click Create a new tag: YYYY.MM.DD on publish.
[ ] Write a description of the major changes in this release. You may want to start with the auto-generated release notes to save time.
[ ] Optional: If not all issues have been addressed, save a draft to return to later.
[ ] Publish the release!

Just wondering if we want to address #316 quickly before this release?

Just wondering if we want to address #316 quickly before this release?

Seems fine to me. I added that issue to next sprint and set it as blocking this release.

Done!


Support Request: will this work?

Only want to execute a certain task the 3rd Monday of March every year
$schedule->addCommand('app:my-command)
            ->description('My Command')
            ->skip(
                'all other mondays of March',
                function () {
                    return ((new Date('today')) != (new Date('third monday of march')));
                }
            )
            ->yearly()
            ->months(3)
            ->weeklyOn(1)
            ->at('09:00');

@kbond also something weird happened when trying this.
when I replace ->weeklyOn(1) with ->mondays() the next run was in 2027?

Hey! This should do it:
$schedule->addCommand('app:my-command)
            ->description('My Command')
            ->skip(
                'all other mondays of March',
                function () {
                    return ((new Date('today')) != (new Date('third monday of march')));
                }
            )
-           ->yearly()
            ->months(3)
-           ->weeklyOn(1)
+           ->mondays()
            ->at('09:00');

->yearly() sets the full cron expression to January 1st, so you were changing the month but the day (1) was staying the same. When you changed to ->mondays() it was finding the next time a Monday in March is also the 1st - 2027 I guess :)

Admittedly, the API is a tad confusing as you've found. I copied from Laravel and didn't give it much thought. Feels like there's room for improvement.

Excellent, thanks for explaining.


Update Composer dependencies (2024-02-17-00-16)

Visual regression test passed!
View the visual regression test report


Reaching coordinate values

I can't reach the coordinate values
So I'm trying to create a function to delete all the coordinates of previously selected places. Basically the main purpose for that function is to erase the polygons I've created.
console.log(this.props.coordinates);
I'm trying to print out the coordinates with this line but it's returning me an "undefined" message. Am I doing something wrong?
Thanks in advance.

Hi @senakvkc
When created places and you have a marker, if you don't save marker here to ref ( ref={marker => this.markerRef = marker}). You will can't delete marker or show coordinate of marker.
So. What you need to do now is save places you created


Fix issue used port (#5189)

Pull request opened by the merge tool on behalf of #5189

Merged into branches iso5 in 8ab3eedb40a1cc28ddad5ba54d803a683badc47b


Bump IOGX v2.0

Notable changes:

Removed stylish-haskell,hindent and tidyrc in favour of fourmolu and editorconfig.
Added support for adding extra flags to cabal-fmt (namely --no-tabular)
Upgraded IOGX to new interface https://github.com/input-output-hk/iogx/pull/2

In order to start using fourmolu and editorconfig, run pre-commit run --all after checking out this PR and entering nix develop.
Modify the .editorconfig and fourmolu.yaml files  accordingly.
hlint is also enabled and will return tens of warnings -> edit .hlint.yaml accordingly.
It's possible that fourmolu and editorconfig will fight one another -> you can disable editorconfig in ./nix/pre-commit-check.nix

I removed networks and nomadTasks from CI because they are not derivations and so they can't be built.
However oci-images and operables are now part of CI.
Note that nix/marlowe-cardano/networks.nix has problems:
  testnet-pioneers = mkNetwork environments.marlowe-pioneers // {
    magic = 1567;
  };
  testnet-dev = mkNetwork environments.marlowe-dev // {
    magic = 1566;
  };

environments.marlowe-pioneers and environments.marlowe-dev are invalid: marlowe-pioneers and marlowe-dev are missing from environments from the more recent version of iohk-nix.
See here for the list of available environments:
https://github.com/input-output-hk/iohk-nix/blob/02f42375ee5c2bab1640f14c6389b7e91bbfec8b/cardano-lib/default.nix#L85
Some oci-images derivations also fail to build.
Test this locally:  nix build .#hydraJobs.oci-images.all.copyToDockerDaemon.x86_64-linux


[x/act V2]: Add MsgVote handling

https://github.com/orgs/warden-protocol/projects/12/views/1?pane=issue&itemId=78574759

Summary by CodeRabbit

New Features

Introduced enhanced voting logic for actions, allowing for better handling of approved and rejected votes.
Added functionality to create and manage action votes, including checks for action status and duplicate votes.
Implemented error handling for existing votes to improve robustness.

Bug Fixes

Resolved issues related to action timeout handling in the voting process.

I believe that AddApprover calling in AddAction (v1beta1/action.go 243) should be replaced with AddVote (where we add initial approver for action)

I do not think that the initial approver must be set during action creation. AddVote is added separately msg_server_vote_for_action.go => VoteForAction.
@Pitasi what do you think?

i think that 99% of the times if I'm creating an action I also want to approve it, so it makes sense to save some time and gas and do it


Link inside page doesn't go to the same location if accessed directly

Operating system: Windows 8.1
Browser: Chrome 57, Firefox 52, IE 11
Test case:

Open the following URL: http://bootstrap4.guide/
Click on the menu item "Breakpoints"
In a different browser window/tab, open the following URL: http://bootstrap4.guide/#three

The page at step 2. shows all the contents of #three (correctly taking into account the menu height).
The page at step 3. sees the top of the contents of #three hidden behind the menu.
In another example (https://pilierpublic.com/fr/help#install), the first question pointed to by this URL is hidden behind the menu.
As a result, one cannot give (e.g. by email) a URL with anchor inside a page unless this points to an element with enough padding so that it not be hidden behind the menu.

Those examples/templates are not official Bootstrap ones, so nothing to do with us. Suggest finding out who maintains them and contacting them.


[Bug] Omnisharp shows false errors on winforms/WPF designer code

Whats the issue?
The issue is that omnisharp shows false errors in code that references winforms / WPF components which would be implemented in the designer code. i.e. any references to controls placed on UI windows are flagged up as errors

@06needhamt: Sorry for the delay here. I realize that this issue is very old. However, if you're still interested in getting this working, I'd like to help.
If you're willing to help, I have a few questions:

What OS are you experiencing this with?
What version of VS Code are you running?
What version of the C# Extension for VS Code are you running?
Do you have a project that you could share that demonstrates the problem?
Could you provide your OmniSharp log from VS Code? To get this, open your project and then select View->Output and choose "OmniSharp Log" on the right side of the Output pane.

Thanks!

I'm on vacation right now will take a look when i am home
On Sat, 27 Aug 2016 at 05:37, Dustin Campbell<EMAIL_ADDRESS>wrote:

@06needhamt https://github.com/06needhamt: Sorry for the delay here. I
realize that this issue is very old. However, if you're still
interested in getting this working, I'd like to help.
If you're willing to help, I have a few questions:

What OS are you experiencing this with?
What version of VS Code are you running?
What version of the C# Extension for VS Code are you running?
Do you have a project that you could share that demonstrates the
problem?
Could you provide your OmniSharp log from VS Code? To get this, open
your project and then select View->Output and choose "OmniSharp Log" on the
right side of the Output pane.

Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-vscode/issues/63#issuecomment-242909766,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD-Cs4ir9TUp94fsjSDpfX1ib8nxRxDbks5qkBNggaJpZM4Hp-2s
.

@DustinCampbell Sorry For the delay just got back from vacation today
I am using Windows 10 Build 14393.51
With VS Code Version 1.4.0
And C# Extension Version 1.4.1
See Attached Project and OmniSharp logs
omnisharp-log.txt
Sample Project
Hope this helps you solve this issue please let me know if you need any more information

@06needhamt This issue is now extremely old. I'm closing it, but feel free to file a new one if you're still having issues.


Swap works from Solana, but not the other way?

Im sorry if I am being thick, but this code works perfect to buy tokenB, however I am unable to reverse the swap , it just keeps buying tokenB. If I change the tokenA <-> tokenB it fails to find the pair. Sorry for the silly question Im just totally confused. Thanks again for some good code tho!

Hey Chris, you can swap tokens for SOL buy changing the direction to out.

Yep I spent time testing this and swapping the direction it keeps going in one direction - have you tested this to make sure the directions isn’t hardcoded? I’ll try again when I get home thanks

I had to make a few changes but I got it to work thanks for the cool code!


FixFix redundant job scheduling by preventing same state transitions (e.g., RUNNING → RUNNING)

What problem does this PR solve?
In the current job scheduling logic, invalid state transitions (e.g., RUNNING to RUNNING) are not filtered, which causes redundant scheduling during resume operations. This PR adds a check to ensure that jobs cannot transition to the same state, preventing duplicate scheduling triggers and improving state consistency.
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?


Scala 3 support

This issue is probably a bit obvious, but some one had to create it :D
In terms of API it means that macro annotations would have to be replaced (with a macro function call ?).

But this is already dealt with in build.sbt and the master branch should be fully compatible with Scala 2.13.x. Are you referring to the latest published version by @jodersky? Until @jodersky finds the time to publish an updated version, I'd recommend you either try building from the source on master or use the version published on sonatype from my fork:
addSbtPlugin("xyz.kamyar" % "sbt-jni" % "1.4.3")


Issue related to laravel grid while mapping two tables

Hello,
We are trying to join 2 tables from sql. We have some views and in one view we have list, which is to be displayed and which will have 2 tables joining. So when we create one model how can we call other table columns in list as grid command line creates list of columns from one model.

Same problem with me, how we can add two tables by using join and get the result as well as search.

@Manupreeta94  have you find any solution?

No, finally I have changed, doing it custom a lot of work, but at least it's working fine.


Adding Preinstall packages on R Studio workbench

Fixes for #342
Description

Added the missing packages using the code recommended in the issue above
How Has This Been Tested?

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

This work will close

https://issues.redhat.com/browse/RHOAIENG-366 Preinstall needed packages on R Studio workbench
https://issues.redhat.com/browse/RHOAIENG-368 Issue while torch installation on R Studio workbench
and possibly https://issues.redhat.com/browse/RHOAIENG-2070 Recreate the issue on Installation of Torch on R-Studio


Support Service account token for Ceph CSI driver

Describe the feature you'd like to have
Currently, the CSI drivers acquire service account tokens is to directly read the token in the file system. However, this approach has drawbacks like : It will not work for csi drivers which run as a different non-root user than the pods.   CSI driver will have access to the secrets of pods that do not use it because the CSI driver should have a hostPath volume for the pods subdirectory to read the token.  The audience of the token is defaulted to kube apiserver, The token is not guaranteed to be available..
CSI service account token address these issues.
This issue tracks this new feature support in Ceph CSI.

Could you elaborate a little more on these points?

It will not work for csi drivers which run as a different non-root user than the pods.

Ceph-CSI only has a single executable running in the container, and it should have access to the ServiceAccount/token which get automatically mapped under the default /var/run/secrets/kubernetes.io/serviceaccount/token path. Do you recommend to use the Kubernetes API to fetch the ServiceAccount/token?

CSI driver will have access to the secrets of pods that do not use it because the CSI driver should have a hostPath volume for the pods subdirectory to read the token.

This is a phrase I do not understand, and confuses me with the previous quote. Should the ServiceAccount/token get mapped into the filesystem, or not? Why a hostPath?

CSI service account token address these issues.

Each Ceph-CSI container deployment/daemonset already has its own ServiceAccount? Maybe I just do not understand what you mean with a "CSI service account"?

@nixpanic sorry, I missed this notification.  let me add a detailed info about this soon. Thanks


Status of implementation?

Hey,
There's a discussion going on over at Hugo (https://github.com/spf13/hugo) about adding Asciidoc support.
Having a quick look at your code, I see that there are missing pieces that must be put in place for that to happen.
Is it possible to give a general indicator of how much is missing and if you have any plan of finish this? And if you need any help, with what?

Sorry, I missed your comment there.
This was a quick exercise for me to get into Go at the time, I don't intend to complete it anytime soon.
There are a lot of very large ruby files that would need to be translated into Go, and I would estimate the current implementation to be only 10-15% of the all asciidoctor project.
That project would be better off if cloned and re-created (instead of forked).


Devs want V2 classes with typed attributes

Going through the exercise of cleaning up mypy warnings in Ambassador, it seems clear that the V2 classes should be actual classes with attributes, rather than subclassing dict. It's basically impossible to type the members of a dict, and having the typing info helps (I found something like five actual bugs going through the mypy cleanup).

Take a look at typing.NamedTuple it saves a lot of time.


Custom MongoTemplate support

We use a custom MappingMongoConverter in our application. To do this with mongobee, we would need it to support custom MongoTemplates. The template that is used is currently newed up in a private method, restricting our ability to override it.

Any timeline on this, or want some help with it? I wouldn't mind making the enhancement.

hi. I had no time recently :( If you are eager, please feel free to do that. Just make the fork and then create a pull request. I will merge it after code review :) Thanks :)

That looks good for completing this issue/enhancement. I'd love to be able to override the executeChangeSetMethod as well, but that may be a different enhancement. Thanks!

released in ver. 0.11


fix some typos in the Docker configuration for the dapr-for-dotnet-developers getting started example

Summary
Fixes "connection refused" error that occurs in Index.cshtml.cs of DaprFrontEnd when trying to invoke "weatherforecast" method of "daprbackend"

@Youssef1313 - is this a false negative on the links verifier?

@IEvangelist The action run here is using the old version before the fix from https://github.com/dotnet/docs/pull/24372.
Ignoring the failure should be okay, subsequent PRs should succeed properly.
In case you don't want to merge with a failing check, merging the current main into the PR branch should do the work.