Unscheduled Runs shouldn't be included in full status

Changes
Fixes #5282
When removing conditions, I accidentally made GetRunsStatus not skip including custom task ResolvedPipelineTasks without Runs in the PipelineRun.Status.Runs map. This resulted in cancellation failing, because the cancellation logic tries to patch each Run or TaskRun in the PipelineRun status (either in the Runs and TaskRuns maps for full embedded status, or ChildReferences for minimal or both embedded status). Unscheduled PipelineTasks for Tasks were already properly excluded from the status map, and ChildReferences was fine either way, but due to the oversight I mentioned above, unscheduled PipelineTasks for custom tasks were included in the full embedded status map, and cancellation would error due to that.
This will need to be cherry-picked to v0.37.x and v0.38.x, since the issue was a regression.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Fix cancellation of PipelineRuns with unscheduled custom tasks.

/assign @afrittoli

/test pull-tekton-pipeline-alpha-integration-tests

/retest

cc @tektoncd/core-maintainers - we're going to be cherry-picking this back to 0.37 and 0.38, so an lgtm would be much appreciated. =)

thanks @abayer 👍
/lgtm


WASM support

I get this error when trying to build for wasm:
   Compiling async-broadcast v0.7.1
error[E0599]: no method named `wait` found for struct `SendInner` in the current scope
    --> /home/flupke/.cargo/registry/src/index.crates.io-6f17d22bba15001f/async-broadcast-0.7.1/src/lib.rs:1647:1
     |
1647 | / easy_wrapper! {
1648 | |     /// A future returned by [`Sender::broadcast()`].
1649 | |     #[derive(Debug)]
1650 | |     #[must_use = "futures do nothing unless .awaited"]
1651 | |     pub struct Send<'a, T: Clone>(SendInner<'a, T> => Result<Option<T>, SendError<T>>);
1652 | |     pub(crate) wait();
1653 | | }
     | |_^ method not found in `SendInner<'_, T>`
1654 |
1655 | / pin_project! {
1656 | |     #[derive(Debug)]
1657 | |     struct SendInner<'a, T> {
1658 | |         sender: &'a Sender<T>,
...    |
1665 | |     }
1666 | | }
     | |_- method `wait` not found for this struct
     |
     = help: items from traits can only be used if the trait is implemented and in scope
     = note: the following trait defines an item `wait`, perhaps you need to implement it:
             candidate #1: `Strategy`
     = note: this error originates in the macro `easy_wrapper` (in Nightly builds, run with -Z macro-backtrace for more info)

which is caused by this in the event-listener-strategy crate:
    /// Wait for the future to complete, blocking the current thread.
    ///
    /// This function uses the [`Blocking`] strategy to poll the future until it is ready.
    ///
    /// The future should only return `Pending` if `Strategy::poll` returns error. Otherwise,
    /// this function polls the future in a hot loop.
    #[cfg(all(feature = "std", not(target_family = "wasm")))]
    #[cfg_attr(docsrs, doc(all(feature = "std", not(target_family = "wasm"))))]
    fn wait(mut self) -> Self::Output
    where
        Self: Sized,
    {
        // SAFETY: `self`/`this` is not moved out after this.
        let mut this = unsafe { Pin::new_unchecked(&mut self) };

        loop {
            if let Poll::Ready(res) = this
                .as_mut()
                .poll_with_strategy(&mut Blocking::default(), &mut ())
            {
                return res;
            }
        }
    }

Do you think it would be possible to add WASM support behind a feature flag, or is this fundamentally needed?

This crate relies on RwLock, so I don't think WASM support would work.

This crate relies on RwLock, so I don't think WASM support would work.

Good point. Would a Mutex be possible? We can always have conditional code to use a RwLock only for WASM.


Log debug information to a file

🚀 Feature
Create a log file for each run with debug information.
Motivation
User can read the logs and understand in which order things got executed. For example, logs could contain this information:
prepare_data_called
setup called
started train loop
training step called
backward called
zero grad called
etc...
Pitch
Use Python logging and redirect all messages with level "debug" to a file in e.g. the logger folder.
Additional context
Suggested by @williamFalcon in a slack discussion.

@awaelchli can I work on this issue? If yes, could you or any maintainers assign me?

@vasudev-sharma Great, looking forward :rabbit:

This issue is a bit old and I'm not sure anymore if we have a strong need for it. But if you wish, please feel free to explore it and make a proof of concept. You may want to take a look at where we call the hooks in the Trainer.call_hook() method. This could be a place where we do a debug log :)


Fix deps

Fixes all dep conflicts
cc/ @TabDigital/public-api @rprieto

Current coverage is 98.95%

Merging #7 into master will not affect coverage as of 3e4cb65

@@            master      #7   diff @@
======================================
  Files            2       2
  Stmts           96      96
  Branches         0       0
  Methods          0       0
======================================
  Hit             95      95
  Partial          0       0
  Missed           1       1

Review entire Coverage Diff as of 3e4cb65

Powered by Codecov. Updated on successful CI builds.

Spring cleaning time 🕥 - PR has been inactive for a while, closing!


Flows blog doubt

In your blog post about Retrofit with Kotlin Flow, you used an API helper class instead of a repository. Was this approach only for the tutorial, or can we also follow it in an actual project? Thank you for your help." @amitshekhariitbhu

You can use it.


toolbar_height attribute spelling

 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.

Could you sign a CLA please?

I signed.

Indeed, I can see your username in the list of contributors. Thank you!


Problem authenticating with facebook

I'm trying to implement authentication with Facebook but got a weird error.
XMLHttpRequest cannot load https://www.facebook.com/dialog/oauth?response_type=code&redirect_uri=http%…facebook%2Fcallback&scope=email%2Cpublic_profile&client_id=***. Response for preflight is invalid (redirect)

My server is setted like:
router.post('/facebook', passport.authenticate('facebook', {
    scope: ['email', 'public_profile']
}));

router.get('/facebook/callback', passport.authenticate('facebook'), (req, res, next) => {
    console.log('hello')
    if (req.user) return res.json(req.user)
});

And on the client:
export class MyAuthConfig extends CustomConfig{
  defaultHeaders = {'Content-Type': 'application/json'};
  tokenName = 'auth_token';
  authToken = 'JWT';
  providers = {
    facebook: environment.facebook,
    google: environment.google
  };
}

On the environment I only have the clientId and redirectUri
Can't figure out what is the problem.

Why is the redirect uri Facebook? It should be your site and should be configured in your Facebook api dashboard

It is configured to be the localhost.

Try setting the redirect URI to the full URI: http://localhost:9001/auth/facebook/callback instead of just http://localhost:9001/.
Also how do you call the authenticate method? do you subscribe to it?

I changed the redirect URI still throws the same error. Yes I'm subscribing to the authenticate method:
  authenticate = (provider) => this.auth.authenticate(provider).subscribe({
    next: (r) => console.log(r),
    error: (err: any) => console.error('Error', err),
    complete: () => this.router.navigateByUrl('/')
  })

I guess it's something to do with my server settings but I can't figure out what is wrong.
Facebook is redirecting me to the correct URI in the error the URI is encoded that's why it looks like it's facebook.

Look at this fork, Its a bit old but it includes Facebook login, maybe it can help:
https://github.com/MosheBiton/ng2-ui-auth-example

Can you make an example using PassportJs? I'm trying for days to make this works but I'm receiving the Response preflight is invalid (redirect). I would very appreciate it if you can help me.

Since I'm not using PassportJs I can't help you...
If you've found out what your issue was, please post it so other people will have a solution for this.
I'm closing this issue for now.

I'm running in to the same issue

Does somebody find a good issue? I still got this error issue..


GH-262 - fix issue gui unresponsive when adding a todo

Fixes: https://github.com/mattermost-community/mattermost-plugin-todo/issues/262
Before:
https://github.com/user-attachments/assets/652f51f3-d255-4fb8-b7cf-8bb35993cacf
After:
https://github.com/user-attachments/assets/02138a5d-9ee4-467f-957c-9b6525e92ad1

@Kshitij-Katiyar Can you please take a look?

@Kshitij-Katiyar Can you please take a look?

@hanzei Do we need to review this PR or something else?

Yes, please review the PR

Yes, please review the PR

@hanzei Looks good to me

@hanzei @pvev We have created a PR to disable to playwright tests for now https://github.com/mattermost-community/mattermost-plugin-todo/pull/269


Notary flist

We need an flist that is able to run the notary gedis actor and the lapis REST layer on top.

@rkhamis please assign this to someone.

Notary flist : https://hub.grid.tf/bola_nasr_1/notary_autostart_development.flist
it's merged flist from :
startup toml flist : https://hub.grid.tf/bola_nasr_1/notary_startup.flist (Based on)
JumpscaleX flist : https://hub.grid.tf/tf-autobuilder/threefoldtech-jumpscaleX-autostart-development.flist

@BolaNasr where is the code that create these flist please ?

@zaibon   i created bash script in flist ( auto build)
https://github.com/threefoldtech/jumpscaleX/commit/6405c51589d369a6001d0d725f953cf8b3f42d07
and then my startup.toml run this bash file

Verification steps:

create a container using zos and this flist test notary flist https://hub.grid.tf/abdelrad/test_notary_autostart_development.flist  which merged between those two flists

startup toml flist : https://hub.grid.tf/bola_nasr_1/notary_startup.flist (Based on)
JumpscaleX flist : https://hub.grid.tf/tf-autobuilder/threefoldtech-jumpscaleX-autostart-development.flist
from inside container
js_shell
i run this command
 gedis_client = j.clients.gedis.get("notary", port=9900)
after that i go this error


Disable/Remove Right menu

I don't need to use Right menu, so how can I remove or disable it?

@lethianqt94  It's a bug. I update the code, check the new version 0.2.5. Now you can just set nil in DPSlideMenuManager.shared.setup function. You can disable left side, right side or both of them. have a nice day ;)

can we setup controller in programatically?not using storyboard.

@HongliYu  Thank You
On Mon, Dec 24, 2018 at 9:55 AM HongliYu<EMAIL_ADDRESS>wrote:

Closed #13 https://github.com/HongliYu/DPSlideMenuKit-Swift/issues/13.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HongliYu/DPSlideMenuKit-Swift/issues/13#event-2042729556,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgVFcZyE3PPYc8SGZf8U5Q63VgGDi05hks5u8FdBgaJpZM4RGnMt
.


Remove Sprockets Deprecation warning in test runs

Summary
When running the test suite, there are multiple entries of:
DEPRECATION WARNING: autoprefixer-rails was deprected. Migration guide:
https://github.com/ai/autoprefixer-rails/wiki/Deprecated (called from run at /home/travis/.rvm/gems/ruby-2.7.2/gems/autoprefixer-rails-<IP_ADDRESS>/lib/autoprefixer-rails/sprockets.rb:22)

This is a relatively recent thing.
Casa fixed it (https://github.com/rubyforgood/casa/issues/571) by switching to Webpack (https://github.com/rubyforgood/casa/issues/579), apparently. So that might work here?

This specific warning doesn't seem to be present in latest (20 Dec 2021) rspec checks
Closable?


Issue 104

Pull Request Review
Checklist

[x] Issue : https://github.com/IBM/kubernetes-driver/issues/104
[x] List of related PRs : N.A.
[x] Project builds without any errors : N.A.
[x] Unit Tests : N.A.
[x] Ran manual functional “smoke” test (does product as a whole still work?) : N.A.
[x] User Story meets the acceptance criteria and passes
[x]  Description of the changes:  Updating install instructions for resource drivers as they are outdated. Tested the updated helm version 3 commands for installing the Kubernetes driver helm package and it is working fine.

This install command is for Helm v3 CLI. I have tested it in OCP. Before running this install command I have checked the Helm version and then executed the Helm Install command. Below is the snapshot for the same.

@dvaccarosenna, Now I have added the helm version in the doc.


Set robolectric's url of maven repositories to https

Thanks for submitting a PR! Please read these instructions carefully:

[X] Explain the motivation for making this change.
[X] Provide a test plan demonstrating that the code is solid.
[X] Match the code formatting of the rest of the codebase.
[X] Target the master branch, NOT a "stable" branch.

Motivation (required)
Tests in my project were failing due with error:
Unable to resolve artifact: Unable to get dependency information: Unable to read the metadata file for artifact 'org.robolectric:android-all:jar': Cannot find parent: org.sonatype.oss:oss-parent for project: org.robolectric:android-all:jar:4.1.2_r1-robolectric-0 for project org.robolectric:android-all:jar:4.1.2_r1-robolectric-0
  org.robolectric:android-all:jar:4.1.2_r1-robolectric-0

from the specified remote repositories:
  central (http://repo1.maven.org/maven2),
  sonatype (https://oss.sonatype.org/content/groups/public/)

After investigation I found out The Central Repository requires https (source: https://support.sonatype.com/hc/en-us/articles/360041287334 ).
I implemented fix proposed on this isssue: https://github.com/robolectric/robolectric/issues/1536
Test Plan (required)
Unit tests are now working for me

I'm closing this PR because solution in #1296 is better 😄


json string with 0 on prefix of numbers are not considered a valid JSON by serde_json::from_string

For example see this output:

Here the nums array have 01 as 2nd element which makes the string invalid for JSON conversion, i am paring the string with help code i have written livejq which is using serde_json::from_string to convert the string to JSON.
if let Ok(parsed_json) = serde_json::from_str::<Value>(&input) {
    // Token is a valid JSON
    // recursively print the JSON values
    print_json(parsed_json, true, 0);
} else {
    // Token is not a valid JSON
    // So just print it
    println!("{}", input);
}

I think the desired result would be ignoring the 0 in the prefix of number.

I think this is correct behavior according to the specification of JSON.


If we continuously send "Hello" messages to the "conversational retrieval chain," the model produces weired/wrong answers.

System Info
langchain==0.0.170
openai==0.27.4
python
windows
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[X] LLMs/Chat Models
[X] Embedding Models
[X] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[X] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
I am using the "ConversationalRetrievalChain", Whenever I send a message to a model, it produces wrong results with history, whereas if I send the same message without history, it produces correct results. The issue here is that the model is forming the wrong standalone question, and the same wrong standalone question is passing to the OpenAI Model.
Steps to reproduce:

Use the Conversational Retrieval Chan
Pass the history on to the chain.
Send the "Hello" message continuously.
Here you can observe wrong/weired answers.

The above scenario is Just an example. But whenever we ask some question, immediately if we send the message "Hello", the "conversational retrieval chain" forms the wrong standalone question, hence the openAI model producing wrong answers.
This is the code I am using: -
_template = """
Use the following pieces of context to answer the question at the end.
{context}
If you still cant find the answer, just say that you don't know, don't try to make up an answer.
You can also look into chat history.
{chat_history}
Question: {question}
Answer:
"""
CONDENSE_QUESTION_PROMPT = PromptTemplate(
template=_template,
input_variables = ["context","question", "chat_history"],
)
chain = ConversationalRetrievalChain.from_llm(
llm= azure_chat_api_llm_objct,
retriever=vectors.as_retriever(),
verbose=True,
chain_type="stuff",
memory = memory,
get_chat_history=lambda h:h,
# condense_question_prompt=CONDENSE_QUESTION_PROMPT,
combine_docs_chain_kwargs={"prompt": CONDENSE_QUESTION_PROMPT},
return_generated_question=True,
)
chain.run("Hello")
Note:- I have tried multiple way like removing "CONDENCE_QUESTION", without prompt, with different types of prompts, etc..but still its producing wrong standalone question.
Ex:-
_template = """
Given the following conversation and a follow up question, rephrase the follow up question to be a standalone question.
Chat History:
{chat_history}
Follow Up Input: {question}
Standalone question:
"""
Thanks in advance, and please correct me if I made any mistakes in the code.

Expected behavior
It should form a proper standalone question when we pass history before passing to model.

In between the conversation, if we ask the question like 'what we have discussed so far', model is unable to answer this question.

I am facing a similar issue. Did you find a solution?

I am facing similar problems too

"ConversationalRetrievalChain" is not suitable here, especially when handling the history of messages. I have replaced it with the chain "RetrievalQA." RetrievalQA is producing better results when compared with "ConversationalRetrievalChain."

I am facing a similar issue. Did you find a solution?

"ConversationalRetrievalChain" is not suitable here, especially when handling the history of messages. I have replaced it with the chain "RetrievalQA." RetrievalQA is producing better results when compared with "ConversationalRetrievalChain."

I am facing a similar issue. Did you find a solution?

"ConversationalRetrievalChain" is not suitable here, especially when handling the history of messages. I have replaced it with the chain "RetrievalQA." RetrievalQA is producing better results when compared with "ConversationalRetrievalChain."

"ConversationalRetrievalChain" is not suitable here, especially when handling the history of messages. I have replaced it with the chain "RetrievalQA." RetrievalQA is producing better results when compared with "ConversationalRetrievalChain."


zkSync rich wallets no longer exist

When you run npx zksync-cli dev start it gives you a link to the list of private keys and addresses you can use with the local network:
https://docs.zksync.io/docs/tools/testing/era-test-node.html#use-pre-configured-rich-wallets
However, this link no longer exists.

This is also broken: https://github.com/zksync-sdk/zksync-ethers/issues/176

Main page looks broken too, looks like there might be 2 docs pages still routing?

Cache cleared fixed the issue on broken images!
On the rich wallets, I don't understand how that PR fixes the issue though? Where is the list of rich wallets hosted now?

hey @PatrickAlphaC The new version of the zkcli-in-memory-node package used in the CLI contains the fix in the rich wallets URL. With v1.7.2 the rich wallets URL is correct now.

Also, we deployed a fix to help fix the old docs cached version.

Confirm fixed. Thank you.


Package name is same for all the generated apks

Actual Behaviour
Package name is same for all the generated apks. In android user is not able to install more than one app with same package name.
So let user is interested in more than one conference and tries to install apks for them, then it will create problem. User will not able to install more than one app at time.
Expected Behaviour
The package name for different apk for particular conference should be different which can be decided from conference website
i.e. if website is abc.xyz.com then package name will be com.xyz.abc.
Would you like to work on the issue?
Yes, would need some help

I too would like to work on this @Shailesh351

@Shailesh351 The app generator replaces the package name itself. There is no need for this issue.
Please refer to this

@iamareebjamal are we checking if app_package_name is unique ?

No, but as we don't maintain all event's database, it will be difficult for us to handle that
Actually we do maintain the db, but IMO, it is out of scope of the project as we will have to traverse all events from REST API and solution will be intractable

I think we can add timestamp in package name

That will make each generated app different even for same event, which we don't want

oh yes, sorry !! very bad idea

@iamareebjamal Thanks. I haven't looked at generator.normalize methode where app_name is again changed as event_name.


State of Translations

I noticed that a lot of the translations do not have a lot of translated or approved strings -> should we maybe set a base level of translated strings before which we hide the language behind a 'not finished translations' selection/link?
@SchrodingersGat @eeintech

@matmair not following what you mean here. Are you talking about in the user language selection box?

@matmair not following what you mean here. Are you talking about in the user language selection box?

Yes, most languages are under 10% translated - that's not a really helpful choice.

Understood. Yeah I agree that this would be a good idae.


Move some types and values out of pkg/logs/config

What does this PR do?
This moves

LogSource and LogSources and related functions into pkg/logs/sources
LogStatus and Info-related types and functions into pkg/logs/internal/status

Motivation
Avoiding some package reference cycles down the road (specifically, allowing the auditor/registry to refer to a status-related type)
Additional Notes
I left pkg/logs/status where it is -- it's a different kind of status, is global, and has some detailed ties to host metadata that I didn't want to try to untie.
Describe how to test/QA your changes
This should not have any noticeable effect, as it only renames things.  Validate that the logs agent can still perform a basic logging operation.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

(rebased on top of the recent journald changes)


fix(#840): display the records with the same address

Description

discord username: afetharita#0001
closes #issue
Please describe your changes. Also describe your aim and content. Do not forget to list the dependencies required caused by those changes.
## Things to check before creating a PR

[ ] I have inspected my topic, checked.
[ ] If it is a core feature, executed detailed tests.

Creating PR rules

[ ] PR must be created for an issue with approved tag. Otherwise PR will be rejected.
[ ] Relevant issue number: The issue number related to PR must be attached to head of PR header, after prefix after prefix # must be attached in parenthesis. A header like this could be used "prefix(#issue_number): PR header" .
[ ] A descriptive and understandable title: The PR title should clearly describe the nature and purpose of the changes. The PR title should be the first thing displayed when the PR is opened. And it should follow the semantic commit rules. For example, a title like "docs(#issueId): Add README.md" can be used.
[ ] Related file selection: Only relevant files should be touched and no other files should be affected.
[ ] Format and Lint Suitability : The code should be made in accordance with a certain format standard and examined according to the lint rules.
[ ] Clean commit history: The commits where the changes are made should be clear and organized.
[ ] Opening PR when job is completed: PR should be opened when job is completed and sent for review by other team members.

Changes

[ ] A new feature (a change that adds a new feature, not a breaking change)
[ ] A new refactor (a change that is not a breaking change, that improves the readability or performance of the code)
[ ] A breaking change
[ ] Documentation change

How were these changes tested?
Please describe the tests you did to test the changes you made. Please also specify your test configuration.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

preview is failed, I can not test it. but maybe we can add a gap between badges for accessibility.

@oguzergul Ekledim 👍

same location part could be bold or italic.

@fdemir can you please approve the pr?

@el seems like something wrong with localisations

@dopplerDistortion That is an issue on the development branch, I fixed that too


Assorted cosmetic tweaks

See commit messages for more details.
Closes #107

Merged with https://github.com/Pear-Trading/FoodLoop-Web/pull/118


make create relationship macros work with and without ctes

resolves #
This PR resolves an issue where the adapter could not materialize tables and views with CTE and without CTE.
Description
The PR changes teradata__create_table_as and teradata__create_view_as macros to accommodate models with CTE's and without CTE's.
Checklist

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

@dbeatty10 ,
I believe this PR addresses the CTE vs. non-CTE syntax in tables and views. I tested it manually with 4 models (a view with cte, a view without cte, a table with cte, a table without cte).
I couldn't figure out if the adapter has any automated tests that I could extend?

@adamtwo ,
I just added integration tests here: https://github.com/dbeatty10/dbt-teradata/pull/8
See the README within the test directory for instructions on running them. Pay particular attention to the "Troubleshooting" section at the very bottom. Let me know if you run into any issues.
I ran your proposed changes against the integration tests, and most of them look good. There are a few tests not able to pass yet, but I think they are unrelated to your changes. I reported https://github.com/dbeatty10/dbt-teradata/issues/7 and will submit a fix for that.
Thank you for making this improvement for CTE vs. non-CTE syntax! I think this can be merged once you're ready.

Thanks for adding the DBT test suite. I'll need to learn how to extend it as Teradata Vantage has some corner cases that the DBT adapter test framework doesn't test, e.g. a view with vs. without CTE or a seed with a few thousand rows.
Ill explore a bit and try to include these tests in this PR.

OK, I've ended up a lot of ground with this PR:

I eliminated the need for dbt_drop_relation_if_exists stored procedure. I use the SQL hint syntax /*+ IF EXISTS */ to inform connections.py that it should ignore specific exceptions. Since Teradata doesn't support hints, it treats /*+ IF EXISTS */ as a regular comment and ignores it.
Added tests for the table/view with/without CTE cases.
Implemented create_schema and drop_schema macros.
All these steps above simplify the test setup. All that is required right now is to have a vantage instance available, point to it, and run the tests. It's a prelude to automating our tests in GH Actions.

Major wins in this PR!

eliminating the dbt_drop_relation_if_exists stored procedure.
custom tests for Teradata-specific behavior
create_schema and drop_schema macros

Really sets things up perfectly for automating tests.
Incredible work, @adamtwo !


Simulator tick events in share page

requires https://github.com/microsoft/pxt/pull/7644
closes https://github.com/microsoft/pxt-arcade/issues/2521
the share link I have in target config.json is a copy of the asphodel game from the hour-of-code page with the tick event extension added along with these blocks. It won't load on the live site right away as it requires https://github.com/microsoft/pxt-common-packages/pull/1199

(ran into the same issue with narrowing types we hit with the status bar / testsprite extension -- passing in a number for the tick data is fine, but then it narrows currentlevel incorrectly to a string instead of a number. Probably worth fixing that bug at some point :) )

I was matching the file to avoid a 100% diff :( happy to switch it if we want though~


about Pytorch-pretrained-BERT

Could the authors tell us about the version of Pytorch-pretrained-BERT used in the code? Something wrong with
from pytorch_pretrained_bert.optimization import BertAdam, WarmupLinearSchedule
the output is:
ImportError: cannot import name 'WarmupLinearSchedule'

Here is the version I used:
>>> import pytorch_pretrained_bert
>>> pytorch_pretrained_bert.__version__
'0.6.2'


[INFINITY-2977] Security docs for HDFS

@elezar @NimaVaziri I think this is done now. It corrects a bunch of things in Kafka too.


Pre-set links for certain groups of users

What feature do you want to see added?
I would love a possibility to somehow define sets of links for groups of users - ideally same Security/Auth groups as used by Jenkins overall and jobs/folders in particular. So if a user is a group member, they would see certain links instead of copying them around individual user profile, as is the current state of the art.
Maybe the auth groups could in fact be the mechanism: add some centrally defined lists only visible with group membership, then they are automatically merged into system-provided list of links (perhaps take care to dedup, if any). The current "Application Links" would simply be something visible to anonymous or just-authenticated users.
Upstream changes
No response
Are you interested in contributing this feature?
No response

hmm that is a niche use case I would say. Maybe it is possible to write an extension point that allows to inject links. Then you can write your own plugin that does this.


Use Executors to create a thread pool, there is a risk of resource exhaustion.

Use Executors to create a thread pool, there is a risk of resource exhaustion.

#3263

can not read your picture, and how to prove that is a risk.

This way of creating a thread pool will cause resource exhaustion.

can you repeat how the resource exhaustion?

newFixedThreadPool is a fixed number of threads, using an infinite queue.

@zhaohai1299002788 Could you submit a PR to fix it?


Update README.md

Fix the 404 page not found for the Keras functional API

@NourEldin-Osama ,
Thank you for taking the time to do this. Can you also update the other 2 links? The link right above and the the link right below the one you updated.
Thanks!

@NourEldin-Osama , Thank you for taking the time to do this. Can you also update the other 2 links? The link right above and the link right below the one you updated. Thanks!

I just updated the other 2 links


Why should I use this package and not jus sentry/sentry w/monolog?

Hi, one thing I think the README is missing is description of what this package gives you that the monolog integration does not. Would you mind giving some hints and I'll create a PR for the README.
Kind regards,
Tarjei

Hi, thanks for the suggestion!
Well, IMO the main advantages are:

easiness of configuration: add the bundle, set the DSN and you're done
container registration: you get the sentry.client public service available
automatic wiring to app: the events have the following things added automatically to the events:

user
Symfony environment
app path
hostname
excluded paths (cache and vendor)

Solved by #115

I think this bundle still misses some benefits monolog intergration has:

Exception logging inside your application (a.k.a "Recoverable exceptions")
Configurable log level
Buffering

For me the best solution for this bundle is moving to automatic registration of monolog handler, so we can keep easyness of configuration and DI alongside the flexibility of monolog handler usage
WDYT?

Registering this as a Monolog handler would mean relying on the MonologBundle itself, and I think it would be hard and fragile at the same time; also, that is already achievable with other packages.
As for recoverable exceptions, you can already do it, with a try...catch where you use the sentry.client service to capture the exception, and later recover from it.

using sentry client makes code dependent on sentry which is less preferrable than general LoggerInterface

That's why I was pointing you toward other packages, like https://github.com/facile-it/sentry-psr-log

No offence, but this one looks like barely supported clone of original raven client with some improvements
https://github.com/getsentry/sentry-php/blob/master/lib/Raven/Client.php#L622-L648
https://github.com/facile-it/sentry-common/blob/master/src/StackTrace/StackTrace.php#L69-L92
Having monolog bridges in official sentry would be more consistent, since exception are already logged according to PSR-3 in exception context field in native console and web kernel listeners

Yes, I know that; I'm just saying that doing that job is partly out of scope for this lib, and partly incomplete, since that approach does not cover the case of fatals, or errors under PHP less than 7.

I see the problem of using 3rd party packages alongside the bundle in that fact that unhandled exceptions will be logged twice - one within this bundle and one with generic PSR-3 logger in built-in handler. But this library works well as raven client bootstrapper, so I think we can just introduce an option to disable handlers for this case here?

That would be a significant amount of work; I'm willing to accept PRs in that direction though...


Stop requiring pyOpenSSL.

This is no longer needed.

Good catch! #1518 (combined with https://github.com/google/oauth2client/pull/395 and https://github.com/google/oauth2client/pull/396) made it so we don't need this any longer.

LGTM.
@tseaver PTAL?

Thanks, @rimey!


Correcting rule name

I added the incorrect rule name to my project following the docs here. Then looked at the source and looks like this is the right name

Thank you for the fix.


AWS Lambda SDK: Ensure not to capture logs as issued by the AWS runtime

Reported internally by @eahefnawy
On function crash AWS runtime logs the error to stdout via console.error call, which we instrument. In result we've observed that error is also captured as user caught error, which is not true.
This patch ensures we do not capture console.* calls as issued by the runtime.
Additionally improved tests to confirm on whole set of captured events

This looks good but would it make sense to still capture this error but as an uncaught error? I would image this would still contain some useful information

@Danwakeem it is captured as uncaught. Issue is that it was reported as both uncaught and caught


Chunking local changes into INSERT-and-DELETE-changes and DELETE-change to upload both separately

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2031
Description
Modifying squashing by chunking local changes into insert-and-update-changes AND delete-change (if present) to be able to upload both separately.
Alternative(s) considered
Providing following configuration for developers

SQUASH_EVERYTHING, i.e., Not upload local insert + delete operation
UPLOAD_EVERYTHING, i.e., Upload every local operation
For this we need more implementation hence this is a temporary fix for Issue #2031

Type
Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

closing in favour of #2133


Fork history of this resporistory

Victor, forgive my English.
Looking at the code this is a fork of nodemcu, rather than standard eLua.  I don''t have a problem with that. But you've removed all acknowledgement of copyright information for Lua, eLua and nodemcu; and this is a problem if anyone wants to use this code base legally outside of Russia -- and possibly inside Russia as well.
What is your reasons for this?
Even so, I will take a look at how this fork performs and feed back any comments to you. :-)

+1 thank you Terry, exactly my thoughts
In short this repository is violating the, very permissive, MIT license because it fails to keep the reference to NodeMCU as required. It seems to be using a non-standard license itself (not sure, it's in Russian).
If your issue doesn't yield a satisfactory response I'll report it to GitHub. I'm pretty sure their terms and conditions require us to respect OSS licenses.

It's mentioned here that it's based on NodeMCU: http://esp8266.ru/forum/threads/esplua-vmesto-nodemcu.449/ This should go together with the correct license into this repo though.

If your issue doesn't yield a satisfactory response I'll report it to GitHub.

I contacted GitHub and was told that the copyright holder should file a DMCA takedown according to https://help.github.com/articles/dmca-takedown-policy/. Hence, this would be something for @nodemcu, @vowstar et. al.

https://github.com/pvvx/EspLua/blob/master/app/cjson/LICENSE
https://github.com/pvvx/EspLua/blob/master/app/coap/LICENSE.txt
https://github.com/pvvx/EspLua/blob/master/app/lua/COPYRIGHT
https://github.com/pvvx/EspLua/blob/master/app/spiffs/LICENSE
https://github.com/pvvx/EspLua/blob/master/app/u8glib/COPYRIGHT
https://github.com/pvvx/EspLua/blob/master/LICENSE

@pvvx Victor, I see that you have included the standard Russian translation of the core terms of the MIT License in your LICENCE, but also read the MIT guidelines and the Lua Licencse.
Even though this is a liberal copy-left, the only restriction is that you should remove valid license statements in contributors code that you have used.  The "contract" is that you allowed to use build upon other people's work, but that you should acknowledge this.  You must not (implicitly or explicitly) claim their work as your own.

Their work built on my work and they should admit it. They can not (directly or indirectly) to say that their work is their own. :)

NodeMCU and EspLua is not finished developing. At the end of the development would be desirable to see that my decisions are excluded from NodeMCU. EspLua expects the complete exclusion of wrong decisions NodeMCU.

@pvvx Victor your first commit was in June.  NodeMCU first open-sourced their source base last November.  Can you provide any evidence that you were collaborating with and provided source to the nodeMCU team before November 2014?
Reworking code to remove unfavourable copyright content is an understandable aim in many circumstance.  However this is code released under an MIT licence.  Is it reasonable to remove fair acknowledgement in this case?

Some can be found on http://www.esp8266.com/

I do not see that the latest version NodeMCU was released in November 2014.

I see a violation of the license here - https://github.com/TerryE/nodemcu-firmware/tree/master/app/lwip

Victor, I see that working with open source and MIT Licenses is all about sharing and respecting each other's contributions.  Some of us don't add our copyright because we aren't looking for public acknowledgement. Some do and if they sign their code with a copyright then we should respect that and keep the copyright.
If you contribute a module to a repository, or add you name to an authors list after making non-trivial improvements, then that is OK.
Removing someone's copyright does not show respect. It is in breach of the MIT licence.
I am not sure what my fork of the nodemcu violates.  It is a fork with additions.  I have not removed anyone else's copyright statements.

@TerryE - Your code is built on LwIP. But it is not specified.
The "contract" is that you allowed to use build upon other people's work, but that you should acknowledge this. You must not (implicitly or explicitly) claim their work as your own.

Basics build my code indicated in README.md (provides links to NodeMCU)
https://github.com/pvvx/EspLua/blob/master/README.md

@TerryE  - You have appropriated the code?
https://github.com/TerryE/nodemcu-firmware/blob/master/app/lwip/app/dhcpserver.c
or NodeMCU ?

Because of the large number of violations of MIT NodeMCU I can not copy it violation of the license.

I have already done an analysis of the copyright statements in the entire nodemcu codebase.  68 of the 217 copyright statements relate to LWIP.  The NodeMCU repository removes none of these.
You are the only author that I know who have actively removed content copyright pages, but as to your general point about the copyrught statements in the nodemcu repository see nodemcu/nodemcu-firmware#656 where we're already proposing to fix this.

http://download.savannah.gnu.org/releases/lwip/
lwip-1.4.0.zip:lwip-1.4.0\COPYING

Я вижу, что вы стерли лицензию LwIP

I see that you have erased license LwIP

Lua Licencse: https://github.com/pvvx/EspLua/blob/master/app/lua/COPYRIGHT
"For details and rationale, see http://www.lua.org/license.html ."

@TerryE  - Why did you delete a file from the license https://github.com/TerryE/nodemcu-firmware/blob/master/app/modules/bit.с?
http://luaforge.net/projects/bitlib/

Do you understand how git works? My repository is a fork of the nodeMCU one. this file is the same as funshine's original commit on 22 Dec 2015: bit.c,  which is the same as the (eLua version)[https://github.com/elua/elua/blob/master/src/modules/bit.c) apart from a tweak to the includes. Yes this seems to based an lbitlib.c, but this retains the copyright statement as-is ans is under the same MIT licence.
The only one around here deleting copyright statements is you.  This is my last response.

@TerryE - Why do you complicate the search for guidance on the license by deleting the file "readmе"?


Added more friends.

r? @steveklabnik
Closes #1150
Closes #1154
Closes #1167

Whoops! I don't know how I missed this. It's got a conflict though, would you mind rebasing? Thank you!

In the process of a rush move to San Francisco. Will take care of this next week!


Add support for functional updates

Motivation
Here is a minimal example to motivate this:
#[flux_rs::refined_by(f1: int, f2: int)]
pub struct X {
    #[field(usize[f1])]
    field1: usize,
    #[field(usize[f2])]
    field2: usize,
}

impl X {

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: new_x.f1 == val })]
    pub fn modify_field1(&mut self, val: usize) {
        self.field1 = val;
    }

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: new_x.f2 == val })]
    pub fn modify_field2(&mut self, val: usize) {
        self.field2 = val;
    }
}

#[flux_rs::sig(
    fn (self: &strg X[@old_x])
        ensures self: X { new_x: new_x.f1 == 1 && new_x.f2 == 2 }
)]
fn modify_f1_and_f2(x: &mut X) {
    x.modify_field1(1);
    x.modify_field2(2);
}

The function modify_f1_and_f2 cannot be verified:
error[E0999]: refinement type error
  --> src/main.rs:27:32
   |
27 |   fn modify_f1_and_f2(x: &mut X) {
   |  ________________________________^
28 | |     x.modify_field1(1);
29 | |     x.modify_field2(2);
30 | | }
   | |_^ a postcondition cannot be proved
   |
note: this is the condition that cannot be proved
  --> src/main.rs:25:34
   |
25 |         ensures self: X { new_x: new_x.f1 == 1 && new_x.f2 == 2 }
   |                                  ^^^^^^^^^^^^^

warning: `flux-issue-sandbox` (bin "flux-issue-sandbox") generated 1 warning
error: could not compile `flux-issue-sandbox` (bin "flux-issue-sandbox") due to 1 previous error; 1 warning emitted

Specifically, we lose all information about f1 when calling modify_field2. In order to make this pass, we need to add some information to the post condition of modify_field1 and modify_field2. Mainly, old_x.f1 == new_x.f1 and old_x.f2 == new_x.f2 for modify_field2 and modify_field1 respectively.
In small cases like this, this isn't too bad, but in cases where there are many fields it could get quite tedious.
Proposal
I suggest we add some kind of support for functional updates on structs. There are a few ways I think this can be done, but unsure what is viable.
Special Function
One way that might be interesting is by adding some primitive definitions for a functional update. This could be an internal, similar to something like map_store which automatically adds the necessary "state preservation" expressions in postconditions.
At a high level, it could take in the old and new objects, along with the fields that should be updated and their corresponding values. For all the fields that should be updated, it would enforce that new.field == value and for all other fields it would enforce that new.field == old.field
This could be used like so:
impl X {

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: functional_update(old_x, new_x, { f1 : val }) })]
    pub fn modify_field1(&mut self, val: usize) {
        self.field1 = val;
    }

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: functional_update(old_x, new_x, { f2 : val }) })]
    pub fn modify_field2(&mut self, val: usize) {
        self.field2 = val;
    }
}

#[flux_rs::sig(
    fn (self: &strg X[@old_x])
        ensures self: X { new_x: new_x.f1 == 1 && new_x.f2 == 2 }
)]
fn modify_f1_and_f2(x: &mut X) {
    x.modify_field1(1);
    x.modify_field2(2);
}

Add an attribute
Alternatively, we could add an attribute to Flux that changes the Z3 encoding to one that supports these sort of funcitonal updates:
#[flux_rs::stateful] // stateful indicates that updates to this struct are functional
                                // (i.e. only change the precise field they are updating preserving the others
#[flux_rs::refined_by(f1: int, f2: int)]
pub struct X {
    #[field(usize[f1])]
    field1: usize,
    #[field(usize[f2])]
    field2: usize,
}

impl X {

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: new_x.f1 == val })]
    pub fn modify_field1(&mut self, val: usize) {
        self.field1 = val;
    }

    #[flux_rs::sig( fn (self: &strg X[@old_x], usize[@val]) ensures self: X { new_x: new_x.f2 == val })]
    pub fn modify_field2(&mut self, val: usize) {
        self.field2 = val;
    }
}

// this works now
#[flux_rs::sig(
    fn (self: &strg X[@old_x])
        ensures self: X { new_x: new_x.f1 == 1 && new_x.f2 == 2 }
)]
fn modify_f1_and_f2(x: &mut X) {
    x.modify_field1(1);
    x.modify_field2(2);
}

I'm not exactly sure how this would work, but I imagine we would have to change our encoding of structs to use Arrays?
Other thoughts
Out of these two, I'd imagine an attribute is probably better. Not sure if anyone has any better thoughts?

It just occurred to me that Rust already has syntax for functional updates (a.k.a. struct update syntax) so we can copy that. I'm imagining something like
    #[flux_rs::sig(
      fn(self: &strg X[@old_x], usize[@val])
      ensures self: X { new_x: new_x == X { f1: val, ..old_x } })]
    pub fn modify_field1(&mut self, val: usize) {
        self.field1 = val;
    }

or using the value directly as an index
    #[flux_rs::sig(
      fn(self: &strg X[@old_x], usize[@val])
      ensures self: X[X { f1: val, ..old_x }] )]
    pub fn modify_field1(&mut self, val: usize) { ... }

I suppose this would need to be accompanied by (regular) constructors that specify all the fields:
    #[flux_rs::sig(
      fn(self: &strg X[@old_x], usize[@val])
      ensures self: X[X { f1: val, f2: old_x.f2 }] )]
    pub fn modify_field1(&mut self, val: usize) { ... }

In a way, the real feature is adding constructors. Functional updates can be added as sugar on top of that.
 For example, we currently allow
fn(self: &strg X[@old_x], usize[@val])
ensures self: X[old_x.f1, old_x.f2]

which desugars to something that's morally equivalent to
fn(self: &strg X[@old_x], usize[@val])
ensures self: X[{f1: old_x.f1, f2: old_x.f2}]

Internally, we check {f1: old_x.f1, f2: old_x.f2} as a constructor for an unknown sort we need to infer.
The reason we support X[old_x.f1, old_x.f2] is historical. At the very beginning, types had multiple indices. At some point, we decided that all types should be indexed by a single value [^1]. The value may be an aggregate sort and can have multiple fields so we didn't lose expressivity [^2]. At the time I decided to keep the old multi-index syntax (implemented as sugar on top of constructors), but we could remove it if it makes everything more coherent.
[^1]: More specifically, we went from multiple indices to being indexed by a tuple and then to a different (nominal) sort per type.
[^2]: Having types indexed by a single value is fundamental to the way we do refinements on generic types, so it is not just a matter of taste. This is obvious in retrospect, but it wasn't back then.

@vrindisbacher I'd start by adding constructors to the syntax of expressions
https://github.com/flux-rs/flux/blob/689fed1cecc65de6878d246c5f6204a57072b887/crates/flux-syntax/src/surface.rs#L457
and then modifying the parser (let's hope there's no ambiguity)
https://github.com/flux-rs/flux/blob/689fed1cecc65de6878d246c5f6204a57072b887/crates/flux-syntax/src/grammar.lalrpop#L484
This will reveal the various places where we need to add an extra arm to handle the additional case. Most importantly, we will need to add a case when desugaring
https://github.com/flux-rs/flux/blob/689fed1cecc65de6878d246c5f6204a57072b887/crates/flux-desugar/src/desugar.rs#L1358
To implement that we will need to add constructors to fhir as well
https://github.com/flux-rs/flux/blob/689fed1cecc65de6878d246c5f6204a57072b887/crates/flux-middle/src/fhir.rs#L940
That will reveal more places where we need to make changes and so on...

hmm, on second thought, maybe it's better to start by preparing fhir for the changes.
Right now the syntax of refinements expressions in fhir is stratified into RefineArg and Expr. I think it would make everything easier if we remove the stratification and put everything in Expr.


Background and Color Multiplier should use consistent color units

Create an emitter
Click Color Multiplier
Observe color values are normalized (0-1)
Click background color
Observe color values re 0-255

Expected: Both should use 0-1 or 0-255 consistently.
Note: Vic suggests 0-255.

Color values are now standardized where RGB values are always 0-255, while alpha is 0.0 - 1.0.

The color change modifier has been removed, as it was confusing to determine how the color multiplier should change per second.  Instead this was replaced with an Ending Color Modifier which allows you to put in what color multiplier you want the particle to have on the last frame of it's lifetime, and the particle will linearly interpolate between its starting and ending color.


ESP-IDF v5.3 I2C Old driver working, new driver not. (IDFGH-13508)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Environment: PlatformIO with v5.3 and an ESP32-WROVER-E on a custom board and a Texas Instruments BQ27441 Fuel Gauge and external 4.7k SCL/SDA pull-up resistors.
I wrote some code that uses the deprecated I2C <i2c.h> driver, which works fine at 400kHz:
#define P_I2C_SDA 22
#define P_I2C_SCL 23
#define BQ27441_ADDRESS 0x55

i2c_config_t i2c_conf =
{
    .mode             = I2C_MODE_MASTER,
    .sda_io_num       = P_I2C_SDA,
    .sda_pullup_en    = GPIO_PULLUP_DISABLE,
    .scl_io_num       = P_I2C_SCL,
    .scl_pullup_en    = GPIO_PULLUP_DISABLE,
    .master.clk_speed = 400000,
};

i2c_param_config(I2C_NUM_0, &i2c_conf);
i2c_driver_install(I2C_NUM_0, i2c_conf.mode, 0, 0, 0);

i2c_set_timeout(I2C_NUM_0, 400000); // ~5ms

i2c_master_write_to_device(I2C_NUM_0, BQ27441_ADDRESS, (uint8_t []){ 0x00, 0x01 }, 2, portMAX_DELAY);
i2c_master_write_to_device(I2C_NUM_0, BQ27441_ADDRESS, (uint8_t []){ 0x01, 0x00 }, 2, portMAX_DELAY);

i2c_master_write_to_device(I2C_NUM_0, BQ27441_ADDRESS, (uint8_t []){ 0x00 }, 1, portMAX_DELAY);
uint8_t val[2];
i2c_master_read_from_device(I2C_NUM_0, BQ27441_ADDRESS, val, sizeof(val), portMAX_DELAY);

printf("Device Type: %04X\r\n", (val[1] << 8) | val[0]);

Then I decided to rewrite this to use the new I2C driver in <i2c_master.h> like so:
#define P_I2C_SDA 22
#define P_I2C_SCL 23
#define BQ27441_ADDRESS 0x55

i2c_master_bus_config_t master_cfg = {
    .clk_source = I2C_CLK_SRC_DEFAULT,
    .i2c_port = I2C_NUM_0,
    .scl_io_num = P_I2C_SCL,
    .sda_io_num = P_I2C_SDA,
    .glitch_ignore_cnt = 7,
    .flags.enable_internal_pullup = 0,
};

i2c_master_bus_handle_t bus_handle;

i2c_new_master_bus(&master_cfg, &bus_handle);

i2c_device_config_t bq27441_cfg = {
    .dev_addr_length = I2C_ADDR_BIT_LEN_7,
    .device_address = BQ27441_ADDRESS,
    .scl_speed_hz = 400000,
    .scl_wait_us = 5 * 1000, // BQ27441 can stretch up to 4ms, use 5ms to be safe.
};

i2c_master_dev_handle_t device_handle;
i2c_master_bus_add_device(bus_handle, &bq27441_cfg, &device_handle);

// This call returns ESP_ERR_INVALID_STATE.
i2c_master_transmit(device_handle, (uint8_t []){ 0x00, 0x01 }, 2, -1);
esp_rom_delay_us(100); // Per BQ27441 datasheet.

i2c_master_transmit(device_handle, (uint8_t []){ 0x01, 0x00 }, 2, -1);
esp_rom_delay_us(100); // Per BQ27441 datasheet.

uint8_t val[2];

i2c_master_transmit(device_handle, (uint8_t []){ 0x00 }, 1, -1);
esp_rom_delay_us(100); // Per BQ27441 datasheet.
i2c_master_receive(device_handle, val, 1, -1);
i2c_master_receive(device_handle, val + 1, 1, -1);

printf("Device Type: %04X\r\n", (val[1] << 8) | val[0]);

This code however works only at 100kHz, not at 400kHz. At 400kHz the line:
i2c_master_transmit(device_handle, (uint8_t []){ 0x00, 0x01 }, 2, -1);  fails with ESP_ERR_INVALID_STATE and the logs show:
E (510) i2c.master: I2C hardware timeout detected
E (520) i2c.master: s_i2c_synchronous_transaction(870): I2C transaction failed
E (550) i2c.master: i2c_master_transmit(1072): I2C transaction failed

Thoughts on why the new i2c driver doesn't work at 400kHz while the old one does?

May you check the scl/sda data with logic analyzer to see what exactly happens？

@western-hoolock Check if this helps: https://github.com/espressif/esp-idf/issues/14129#issuecomment-2295882059

May you check the scl/sda data with logic analyzer to see what exactly happens？And probably compared with old one. From the log and description, I guess there might happen with long stretch.

Unfortunately I don't have a logic analyzer or access to one.
The BQ27441 datasheet mentions that it can do clock stretching up to 4ms, in both the new and old driver examples I posted I set it to 5ms to be on the safe side:
Old driver:
 i2c_set_timeout(I2C_NUM_0, 400000); // ~5ms
New driver:
.scl_wait_us = 5 * 1000, // BQ27441 can stretch up to 4ms, use 5ms to be safe.
One difference I noticed though is that the old driver takes APB clock cycles as units and the new driver takes the number of microseconds.
But yeah, I wish I had access to a logic analyzer so I could actually confirm that the  new driver actually respects the allowed timeout for clock stretching.

So... i understand, this is a bug in the driver.

line 589 change the timeout value, and line 604 change it back. Please move line 589 behind line 604 and try again..

@mythbuster5
Would you backport this fix to release branches? I hope v5.2.3 can include this fix.


Display problem for PAINT annotations

The same annotation is shown as positive and negative (NOT/IBA) for one gene product (MGI:MGI:102966, in the PTHR11866 family):

I dont know how general that problem is but it really needs to be fixed soon.
It should by NOT by IRK.
No IBAs should have NOT.
@mugitty can you confirm that the original data is OK (that GO Central imports?) - although the fact that it looks OK on Pantree suggests that the data is OK: http://pantree.org/tree/family.jsp?accession=PTHR11866
Thanks, Pascale

AmiGO is pretty passive, so I suspect this is an upstream problem and there
actually are both annotations being fed in. Will check when I'm in the
office later, or perhaps Seth can before I get there.
On Tue, Jul 31, 2018 at 1:54 AM pgaudet<EMAIL_ADDRESS>wrote:

The same annotation is shown as positive and negative (NOT/IBA) for one
gene product:
[image: image]
https://user-images.githubusercontent.com/4782928/43449151-074ccc4c-94b0-11e8-8886-60d66ae457b8.png
I dont know how general that problem is but it really needs to be fixed
soon.
Thanks, Pascale
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/geneontology/amigo/issues/522, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcuEDLeEz6ezBrMp_Fmy7eiUIsCAjjfks5uMBs-gaJpZM4Vn2of
.

This is definitely upstream--holding here until we find the right home.

Corrected in ftp://ftp.pantherdb.org/downloads/paint/presubmission/gene_association.paint_mouse.gaf.gz on Aug 7, 2018
Keep open until the correction trickles through in AmiGO.

Additional tests from @pgaudet
This should exists: MGI:104328 + NOT GO:0004252 + IBA
This should NOT exist: MGI:104328 + GO:0004252 + IBA


Added tapping circles

By "doesn't seem to prevent additional clicks" do you mean it should prevent additional clicks? Also I've thought about triggering it on space-bar but the space-bar could also be keep pressed.
I may be able to take a look and trigger the "tap" animation in the function that sends the tap to the backend (this would probably require saving the last clicked point). Also this would show the tapping animation only when the move actually gets sent to the server. Is that what we want?

By "doesn't seem to prevent additional clicks" do you mean it should prevent additional clicks? Also I've thought about triggering it on space-bar but the space-bar could also be keep pressed.
I may be able to take a look and trigger the "tap" animation in the function that sends the tap to the backend (this would probably require saving the last clicked point). Also this would show the tapping animation only when the move actually gets sent to the server. Is that what we want?

For the first question, I just meant I'm happy with the behavior. I'm glad you can "click through" the animation. I'd prefer if none of this animation logic hits the backend, I don't think it is necessary. For the case of holding the spacebar, just trigger the animation every turn so it matches the fast move animation. Otherwise, trigger the animation at the correct spot (either in the bottom-center of the screen, on the charge move buttons, or on the switch buttons) on "keyboard up."

By "doesn't seem to prevent additional clicks" do you mean it should prevent additional clicks? Also I've thought about triggering it on space-bar but the space-bar could also be keep pressed.
I may be able to take a look and trigger the "tap" animation in the function that sends the tap to the backend (this would probably require saving the last clicked point). Also this would show the tapping animation only when the move actually gets sent to the server. Is that what we want?

For the first question, I just meant I'm happy with the behavior. I'm glad you can "click through" the animation. I'd prefer if none of this animation logic hits the backend, I don't think it is necessary. For the case of holding the spacebar, just trigger the animation every turn so it matches the fast move animation. Otherwise, trigger the animation at the correct spot (either in the bottom-center of the screen, on the charge move buttons, or on the switch buttons) on "keyboard up."

Yeah what I meant is that on the fast move turn I could trigger the animation. The issue could be that is gets triggered two times when you use the mouse. But I'm saying this without looking at the code. Tomorrow I'll take a look and modify the code appropriately.


Add missing changesets

As mentioned in https://github.com/graphql/graphiql/pull/2010#issuecomment-962743181

Added missing changeset for graphql-language-service-utils
Added patch changeset for graphql-language-service-utils dependants

I also found that graphql-language-service-types was not published with the only change it had, which is adding graphql 16 in the peer dependencies:

Added patch changeset for graphql-language-service-types

@acao

@dotansimha do you know why changesets isn't auto-incremementing these releases like lerna publish did? are we missing config?


fix: 🐛 Correct the migration sequencing

Description
Previously we had a version 9.4.03 which if was executed, prevented 9.4.1 and 9.4.2 from being executed. This will make sure both those migrations are successfully executed
Breaking Changes
NA
JIRA Link
NA
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 9.6.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


build: don't set -std flag manually

Fixes: https://github.com/iovisor/bpftrace/issues/884

@mmarchini  this seems to have broken the runtime tests in my vagrant box as it now builds with -std=gnu++14 instead of c++14.  The runtime tests on new PRs seem to suffer from it too.
not sure why tough

I started to look into it this morning but had to stop. I'll look again later today or tomorrow morning.

https://github.com/iovisor/bpftrace/pull/897 should fix it


[Bug] autogeninternals still referenced in container-flags

Page link
https://kyverno.io/docs/installation/#container-flags
Description
the autogeninternals CLI flag was removed in 1.9.0 but it is still mentioned as a valid config at https://kyverno.io/docs/installation/#container-flags
Expected behavior
This flag should be removed from the docs
Slack discussion
No response

Thanks, will fix. Also updated release notes with mention of it (was actually in RC.1 release notes).


Add chunking to CERTIFICATE response

Fixes #1018.
Still needs unit test coverage.
Signed-off-by: Steven Bellock<EMAIL_ADDRESS>

The https://github.com/DMTF/libspdm/blob/main/unit_test/test_spdm_requester/chunk_get.c#L552 test needs to be overhauled. We can discuss that in the meeting tomorrow.

question: Have you tested spdm-emu, with chunk enabled?

question: Have you tested spdm-emu, with chunk enabled?

I synced to head of spdm-emu and ran spdm_requester_emu and spdm_responder_emu and they worked.


Fix: Do not use stdClass to mock something different

This PR

[x] uses a concrete class when mocking instead of stdClass

💁♂ There are other places where stdClass is used to mock something different, but it appears to be a lot of work to fix these tests. Maybe someone else is interested in fixing them?
Running
$ git grep stdClass

yields
tests/Unit/Application/SpeakersTest.php:154:    private function getSpeakerWithNoTalks(): \stdClass
tests/Unit/Application/SpeakersTest.php:157:        $stub = m::mock(\stdClass::class);
tests/Unit/Application/SpeakersTest.php:164:    private function getSpeakerFromMisbehavingRelation(): \stdClass
tests/Unit/Application/SpeakersTest.php:167:        $stub     = m::mock(\stdClass::class);
tests/Unit/Application/SpeakersTest.php:171:        $talk = m::mock(\stdClass::class);
tests/Unit/Application/SpeakersTest.php:184:    private function getSpeakerWithOneTalk(): \stdClass
tests/Unit/Application/SpeakersTest.php:187:        $stub     = m::mock(\stdClass::class);
tests/Unit/Application/SpeakersTest.php:191:        $talk = m::mock(\stdClass::class);
tests/Unit/Application/SpeakersTest.php:204:    private function getSpeakerWithManyTalks(): \stdClass
tests/Unit/Application/SpeakersTest.php:207:        $stub     = m::mock(\stdClass::class);

@mdwheele
Conflicts have been resolved!

Thank you, @mdwheele!


Considering not to get snapshot for transaction commands in gRPC threads

Now if a transaction command is not queued in the latch, TiKV will get snapshot for it in the gRPC thread.
https://github.com/tikv/tikv/blob/6d0edd7e690d376e651aecf6f0bbefdeec22c390/src/storage/txn/scheduler.rs#L528
But it's possible that it takes much time to get the snapshot because of the mutex in rocksdb. It results in frequent gRPC disconnection.

775528:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775529:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775531:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775532:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775533:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775534:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775535:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775537:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775539:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775540:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775541:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775542:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775543:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775544:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775545:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775546:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]
775548:[2021/10/01 10:42:59.090 +00:00] [Error] [kv.rs:931] ["KvService response batch commands fail"]

I can reproduce this errors by sleeping when getting snapshot, but the frequency doesn't match.
1152:[2021/10/12 19:09:47.364 +08:00] [ERROR] [kv.rs:1034] ["KvService response batch commands fail"]
1685:[2021/10/12 19:16:33.050 +08:00] [ERROR] [kv.rs:1034] ["KvService response batch commands fail"]

[2021/10/12 19:16:31.040 +08:00] [INFO] [client_batch.go:583] ["batchRecvLoop fails when receiving, needs to reconnect"] [target=<IP_ADDRESS>:20160] [forwardedHost=] [error="rpc error: code = Unavailable desc = keepalive watchdog timeout"]

I think we can make SnapshotList lock-free and remove the lock totally. FYI https://github.com/facebook/rocksdb/issues/5083.


add

https://www.kyungnam.ac.kr/

@Nj-L Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

official website URL: https://www.kyungnam.ac.kr/main/Default.asp
IT related url :https://www.kyungnam.ac.kr/ce/main/
official email certification :
[image: email.PNG]
2020년 4월 15일 (수) 오후 10:21, Philip Torchinsky<EMAIL_ADDRESS>작성:

@Nj-L https://github.com/Nj-L Please provide us with:

the school official website URL, if different from the domain you
are submitting
an URL of a page on the official website where an IT-related
long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK)
showing that the school recognizes the domain which you are submitting as
an official email domain for the students.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/8111#issuecomment-614036627, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6KRXJ7YZXBP77D3TJTL6DRMWYFBANCNFSM4MGV6TIQ
.

@Nj-L Pull request merged. Thank you!


NoLocalizationBehavior: go forward after penalized

When the robot has been penalized in the game controller, it is put back to the edge of the field. The behavior should therefore go forward when no localization is present.

Done via JustUnpenalized decision


[Suggestion]: Support Bedrock's expanded set of formatting code colors, for the editor gui bits

Looks like Bedrock has 11 more colors to be had from formatting codes.
Why should they get all the fun?
(also, more colors accessible for those that haven't gone into writing html tags (and using color="" attributes)
https://minecraft.fandom.com/wiki/Formatting_codes

While I like the idea, implementing custom formatting codes is not really easy as we just let vanilla handle it and they hardcode it

This is old now, but I wouldn't suggest doing it with actual formatting codes... just have Herecles' parser insert the write  for Hermes to render it.


Fix GitHub release

Codecov Report

Merging #276 (48ebca2) into main (750f457) will decrease coverage by 0.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #276      +/-   ##
==========================================
- Coverage   95.32%   95.10%   -0.22%
==========================================
  Files          10       10
  Lines        1369     1369
==========================================
- Hits         1305     1302       -3
- Misses         64       67       +3

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


How to send more than 20 bytes fast to Android app?

I have a custom board with nrf52832 and softdevice s132. It says that theoretically it can reach speed up to 190 kBytes/s. But when I try to send a String splitted in 20 chars, in the NRF Connect I see it sends only the last 20 bytes of that string.
In Arduino serial it prints correct all the parts.
If I add a delay(1000); between every st.setValue(buf[i]) AND read every second in the NRFConnect, then it display every 20 bytes of that string.
I've tried with an Android developer to create a custom app and the behavior was the same. I can get that string only with a delay(100). If I decrease that delay or eliminated it, I loose chunks of string.
The problem is that I need to send up to 1 Mbytes of data to phone and in this way it seems a headache.
Do you have/know an example with long strings?

This is pretty much the same behaviour I'm seeing. Streaming data is consistently unreliable and causes the entire BLE mechanism to just stop transmitting. I'm going to investigate adding various length delays between transmissions but this shouldn't be needed.

@dots123 I don't know if you've found a solution to your issue however, if you find the function called setConnectionInterval which is part of the BLEDevice, you can change the "default" connection interval from 50ms down to 7.5ms in some instances. I have done this on my BLE device and connection to a Samsung Galaxy S tablet, the data transfer is now super-fast with no lost packets.

Great! I'll try it!
Thank you!

@dots123 Did this work for you? I've had some massive performance improvements since changing the connection interval but I think really, I need to target the latest nRF52 SDK and soft-device to get extra performance benefits.

I had some improvements. I've got up to 11k, which is acceptable for now. If you change also Advertisment interval (30 instead of 100), maybe you can go higher. But it's ok if you use it only with Android. Ios has different settings accepted and you need to use them. I haven t tried but I ve read many problems with that.
On Thu, Oct 25, 2018 at 13:04<EMAIL_ADDRESS>wrote:
@dots123 Did this work for you? I've had some massive performance improvements since changing the connection interval but I think really, I need to target the latest nRF52 SDK and soft-device to get extra performance benefits.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I believe you should look at SoftDevice! I use 132 v2. But other versions have improved speed. Take a look on Nordic SoftDevice' site.
I dont think that bootloader or others will solve this problem.
On Thu, Oct 25, 2018 at 16:24<EMAIL_ADDRESS>wrote:
Thanks for the hint about the advertisement interval, I'll give that a try.
I think I'm going to push forward an update which targets the latest SDK and improvements from Nordic and at the same time, update the bootloader for the Sparkfun nRF52832 breakout board. The SDK currently used is pretty out-dated and it doesn't look like there's much future progress planned here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok, I'll take a look.
The bootloader is aside from this and the issues, but I'd quite like to develop (well, its a necessity really) an open OTA solution for the Sparkfun board.

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes...

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes...

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes in one transaction...

You can't send more than 20 in one transaction.

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes in one transaction...

You can't send more than 20 in one transaction.

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes in one transaction...

You can't send more than 20 in one transaction.

It seems it is possible by changing MTU limit: https://stackoverflow.com/questions/63590616/what-is-the-maximum-att-mtu-allowed-by-bluetooth-le
Not sure though

@webzoid I've set bleSerial.setConnectionInterval(10, 15); still, I can't set more than 20 bytes in one transaction...

You can't send more than 20 in one transaction.

It seems it is possible by changing MTU limit: https://stackoverflow.com/questions/63590616/what-is-the-maximum-att-mtu-allowed-by-bluetooth-le
Not sure though


Newlines are not properly encoded in the API

Per:
https://discuss.ai.google.dev/t/function-call-string-property-is-double-escaped/37867
Underlying tool calling API in Gemini (including 2.0 flash) are not encoding newlines correctly in JSON output (sometimes double encoding... sometimes only partially encoding)
This means the API for tool calls is not workable on text:
Failing example:
from google import genai
from google.genai import types

echo = types.FunctionDeclaration(
    name="echo",
    description="Echoes the input text",
    parameters={
        "type": "OBJECT",
        "properties": {
            "string": {
                "type": "STRING",
                "description": "Text to Echo",
            },
        },
    },
)

echo_tool = types.Tool(
    function_declarations=[echo],
)

client = genai.Client(api_key="ABC")
instruction = "echo the text hello."

MODEL_ID = "models/gemini-2.0-flash-exp"

to_echo = "printf('\\n')\\;\nprintf('hello<>')";
response = client.models.generate_content(
    model=MODEL_ID,
    contents="echo the text: " + to_echo,
    config=types.GenerateContentConfig(
        temperature=0.0,
        tools=[echo_tool],
    )
)
print(to_echo)
print(response.candidates[0].content.parts[0].function_call.args["string"])

This will output:
printf('\\n')\\; printf('hello<>')
So

A newline was eaten
\n was double encoded

(This is an upstream bug, but I am hoping you can escalate internally, library is implemented correctly here, it is simply parsing json and it comes back incorrect from the gemini api)

Tagging in @MarkDaoust to help route - and thank you for sharing this feedback, RE: bugs in the JSON outputs!

+1 makes Gemini hard to use with tools, requiring a bit of hacking

Also want to add the same thing happens when using VertexAI


Added Strong Number Calculator

Fixes Issue🛠
#1556
Description👨💻
A Strong number is a number whose sum of the factorial of its digits equals the number itself. For instance, 145 is a Krishnamurthy number because
1!+4!+5!=145.
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[X] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[X] I am an Open Source contributor
[X] I have performed a self-review of my code
[X] My code follows the style guidelines of this project
[X] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷
https://github.com/user-attachments/assets/579f54f3-546e-4d63-a818-385bbd625142

@KajalBhatt123 Add empty field validations. Also you can add brief description of why it is or it is not strong number.

@Nihal4777  updated

@Rakesh9100 This PR is reviewed & can be merged.


Roll Engine from 3256f4f0c41b to 386467ead4a3 (2 revisions)

https://github.com/flutter/engine/compare/3256f4f0c41b...386467ead4a3
2022-03-18<EMAIL_ADDRESS>Roll Skia from 9de6144f7d90 to ac8c554dbec7 (1 revision) (flutter/engine#32107)
2022-03-18<EMAIL_ADDRESS>Fix iOS keyboard physical keyboard doesn't work in add to app (flutter/engine#32098)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Greening of Software Designs using AI

Predicting Environmental Impact in Software Design: Leveraging Large Language Models for Pre-Operational Emissions Estimation
Related issues or discussions:
Please list all related issues so that we can see the originating conversation, e.g. #10 or https://github.com/Green-Software-Foundation/discussions/discussions/5
Tagline: A very short description of the project MAX 8 WORDS
Abstract:
In today’s world, emissions are typically calculated after deployment. The Impact framework aids in assessing environmental effects across various system layers by retrieving observations from the different components of the software.
However, when software is conceptualized during requirements/design phases, system designer make design choices which directly impact the overall operational carbon emissions.
This approach suggests utilizing Large Language Models (LLMs) to proactively predict emissions from design choices, enhancing decision-making by including potential environmental impacts alongside financial and non-functional considerations.
By generating relevant observations and predicted values for these options, LLMs could inform the Impact framework, allowing for pre-operational emission estimation. This project aims to develop an observation schema and leverage AI to predict environmental outcomes, offering a novel way to integrate sustainability into software development.
Quote: Include made up quote from a business leader.
Audience: Who is the primary audience for this solution? Who would we need to market this product to for it to be a success at scale? Where do these people go for their information? Which conferences would people talk about this project in? MAX 100 WORDS
ToC: How will this project support one or more of our ToC pillars
e.g. It will improve tooling by “xxxxx”.
Governance: Which working group(s) do you think should govern this project?

[ ] Community
[X] Open Source
[ ] Policy
[ ] Standards

Operating Model: Will this project operate based on:

[ ] Consensus - Goal is everyone agree to every change so we are speaking with one voice when the deliverable is released.
[ ] Maintainers - The Project Leads listen to feedback and incorporate it back into the project if they see fit.

Problem: List 3-5 problems this audience has that this project will attempt to address. Describe Why is this project better at solving a problem compared to parallel proposals or implemented projects? MAX 200 WORDS
Solution: Try to make this as detailed as possible. The topics given below are just suggestions; address them only if they are relevant to your problem:

Rough design and scenarios on the probable effects, if any.
The use of diagrams is encouraged to elucidate concepts.
Address any possible objections.
Explain how the solution solves each of the problems listed in the problem section.

Closure: How do we know that the project succeeded? This has to be measurable if possible. Make references to successor projects, if any.
FAQ: Add anything here that doesn't fit into the above sections. It can be blank, to begin with; as questions are asked and points clarified use this section to document those clarifications.

FYI @navveenb


feat: enable tailwind JIT mode

Related: https://github.com/mitchazj/vrt-starter/issues/1

Thanks!


Move Buffer and friends to corlib, netstandard, system.runtime

Before the move, we also agreed to do the following cleanup:

Cleanup span/buffer equality APIs (as discussed in review) <= Done (https://github.com/dotnet/corefxlab/pull/1646)
Rename Buffer to Memory
Remove offset from OwnedBuffer.Pin
Remove OwnedBuffer.Add (will submit PR for this soon – using Unsafe.Add - https://github.com/dotnet/corefx/pull/20886)
Remote IsRetainable and change Release return type to bool (PR needs to be updated - https://github.com/dotnet/corefxlab/pull/1648)

Span and Buffer ctros should be the same. See dotnet/corefx#23471

Is this work item solely about removing Memory(T[], int) ctor or are there other constructors we want to add/remove?

@KrzysztofCwalina, can we consider this work/issue complete? The extension work can be tracked by the api review issue - https://github.com/dotnet/corefx/issues/23862

Yeah, go ahead and close it. Just make sure the extensions issue does not get lost.


How to execute executables in Debian

Hello There!
I installed Debian 10 Buster KDE in my Chromebook to run Eclipse IDE (as Crostini frequently crashed) but after installing, I cant execute the executable. At first I thought it got corrupted somehow but I tried reinstalling(both Eclipse and the chroot) but to no avail. Can anyone know how to fix it?
Error Code:
(buster)sanjay@localhost:~/Desktop/Debian/eclipse$ ./eclipse
-bash: ./eclipse: Permission denied
Sorry I don't know how to reproduce the error.
EDIT:I FORGOT TO SAY THAT I EVEN CHANGED THE PERMISSIONS TOO
(buster)sanjay@localhost:~/Desktop/Debian/eclipse$ chmod +rwx eclipse
(buster)sanjay@localhost:~/Desktop/Debian/eclipse$ ./eclipse
-bash: ./eclipse: Permission denied

@sanrobin,
Do a long listing of your eclipse file and see who owns it & who can execute it with:

ls -l ./eclipse

Also, is the file ./eclipse in your $HOME directory or somewhere else?
If it's in your $HOME directory run the following command and make sure there is no 'noexec' in the mount options:

mount | grep ~/

Hope this helps,
-DennisLfromGA

I found the culprit. It was ChromeOS itself ! I moved the executables to desktop and it worked. Sorry for late reply.