[Enhancement] Active hotbeds reworks

Feature title
Species infromation > Management > Active hotbeds
Description
The active hotbeds map should display a hot map of recent (last 2 weeks) observations independent of management status of nearby nests. This should occur at a high update frequency (daily) to allow for rapid response.
(Optional) Needed changes in backoffice
https://github.com/inbo/aspbo/issues/173
(Optional) Suggestion how to implement change
Might be naive but I think this can be achieved by doing a daily pull of the waarnemingen.be API.
We have the necessary credentials and 3 example scripts to get access.
the example scripts:
waarnemingenAPI.zip
Alternatively, with some minor changes to these scripts, we can create a GDPR proof version of the data in a googlesheet.
Which do you prefer?

Might be naive but I think this can be achieved by doing a daily pull of the waarnemingen.be API. We have the necessary credentials and 3 example scripts to get access.
the example scripts: waarnemingenAPI.zip
Alternatively, with some minor changes to these scripts, we can create a GDPR proof version of the data in a googlesheet. Which do you prefer?

@SanderDevisscher Yes, based on the scripts you shared, you can do a daily API pull. I see no need to write the data to a googlesheet, I would just add the data to the S3 bucket with a script from aspbo.

@mvarewyck best to put it on the PRD bucket directly then ?

@mvarewyck best to put it on the PRD bucket directly then ?

Yes, if you don't want to do a daily PR from UAT to PRD.
Then in the data processing script I would

add some checks on the data structure (e.g. no. of rows >0, column names) before the data is pushed to the bucket.
when the checks fail and there is no update of the data, make sure we get a notification (automatically with github actions?)

@mvarewyck 0 rows is a real possibility in the winter when they are not active. The app should display a disclaimer when this the case.
@TheJenne18 can you confirm the PRD - bucket exists and can be writen to by github actions from the aspbo repo ?

transfer to Aspbo => daily workflow directly to PRD bucket with some workflow tests


No date type in pandas

There is no date type in pandas, so what should we do about it?

We can simply skip the "date" type as this is a subtype of datetime. If people need a date it will just be stored as datetime with time=00:00:00
We can convert to a "python date" if all have time=00:00:00.  The dtype will be 'object', so we might need an extra attribute to identify date-columns
We can keep the datetime, and then simple add an attribute to the column identifying it as dates. (datetime64 is 8 bytes vs date 32 bytes)
Later, when the pandas dataframe is converted into a list of "ColumnMetaData", we can try to detect if the datetime is actually a date (all have time=00:00:00) and then store this information in the database
Not use pandas? Probably a bad solution, because we might need pandas for other stuff later and then it would be nice to have aligned the pandas types with the types in sprout)
?

Solved with #147 by using Polars.


Question about time data (timestamps)

Hi tRosenflanz,
Thanks for sharing the work, the well organized code really help a lot when I am trying to understand the model!
Just a small question, why not consider timestamps (ex: to_event, or time delta) as numeric data and provide it with attention weight? I've read through the Appendix A of the original RETAIN paper but still could not understand the meaning of

The entire process could be understood such that we use the temporal information not to embed each visit, but to calculate the attentions for the entire visit sequence. This is consistent with our modeling approach where we lose the sequential information in embedding the visit with MLP, then recover the sequential information by generating the attentions using the RNN.

Thanks!

It is essentially the same to treat them separately (and concatenate inside the model) or concatenate right away with a caveat that no dropout will be applied on the time data. Initially I coded this up in a way that disallows dropout on the "time" variables but I doubt it makes a huge practical difference.
The original text essentially means that timestep data is used to compute attention values (i.e. fed into the LSTM) but isn't embedded as part of the visit encoding (not put through the initial embedding/dense layer like codes are)

Thanks for your reply!


Fix spelling error and useless code.

Motivation
Fix spelling error and useless code.
Modifications
Verifying this change
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc

/pulsarbot run-failure-checks


Suggest to use lombok in the project

If possible, we could use lombok in this project so that the syntax of redundant getter and setter methods could be avoided.
https://github.com/flashvayne/mysql-mybatis-pagination/blob/edd39df0354611665e1754af65420731c71ebf47/src/main/java/com/github/flashvayne/dto/PageInfo.java#L15
Thank you.

Thank you very much for your suggestion.
I have simplified code using Lombok.
see commit


[xpack] Add new port

This is a header-only library for converting between c++ structures and json/xml

[x] Changes comply with the [maintainer guide]
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See docs/examples/adding-an-explicit-usage.md for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree [company="default"]
黄锡元
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "microsoft/vcpkg"                                                                                @.>;
发送时间: 2023年2月4日(星期六) 上午9:09
@.>;
<EMAIL_ADDRESS>主题: Re: [microsoft/vcpkg] [xpack] Add new port (PR #29439)
@xyz347 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:
(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"        Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is effective as of the latest signature
date below.
Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.
Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.
Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@microsoft-github-policy-service @.*** agree company="default"
黄锡元
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "microsoft/vcpkg"                                                                                @.>;
发送时间: 2023年2月4日(星期六) 上午9:13
@.>;
<EMAIL_ADDRESS>主题: Re: [microsoft/vcpkg] [xpack] Add new port (PR #29439)
@xyz347 the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree
and
@microsoft-github-policy-service agree company="your company"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@microsoft-github-policy-service agree company="default"
黄锡元
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "microsoft/vcpkg"                                                                                @.>;
发送时间: 2023年2月4日(星期六) 上午9:14
@.>;
<EMAIL_ADDRESS>主题: Re: [microsoft/vcpkg] [xpack] Add new port (PR #29439)
@xyz347 the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree
and
@microsoft-github-policy-service agree company="your company"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Fixes https://github.com/microsoft/vcpkg/issues/28703

Sorry for the wrong label. :(


Option to enable remote control?

Does the simulator have the ability to connect to the cloud and enable remote control? Even after I enable the toggle through the app, it still says that the device is "Local Only"

I doubt that would be possible, but if someone wants to write it...


JohnOlutubo: Workflows-bio, 2weeks

my fork of module repository

Learning Objectives
Learning Objectives

Priorities: 🥚, 🐣, 🐥, 🐔 (click to learn more)

There is a lot to learn in this repository. If you can't master all the material
at once, that's expected! Anything you don't master now will always be waiting
for you to review when you need it. These 4 emoji's will help you prioritize
your study time and to measure your progress:

🥚: Understanding this material is required, it covers the base skills you'll
need for this module and the next. You do not need to finish all of them but
should feel comfortable that you could with enough time.
🐣: You have started all of these exercises and feel you could complete them
all if you just had more time. It may not be easy for you but with effort you
can make it through.
🐥: You have studied the examples and started some exercises if you had time.
You should have a big-picture understanding of these concepts/skills, but may
not be confident completing the exercises.
🐔: These concepts or skills are not necessary but are related to this module.
If you are finished with 🥚, 🐣 and 🐥 you can use the 🐔 exercises to push
yourself without getting distracted from the module's main objectives.

🥚 0. Local Development Without Git
Practice the foundational workflows of software development by learning to write
Markdown locally on your own computer using Visual Studio Code (VSCode), the
Command Line Interface (CLI), and NPM scripts to automate your code's quality
(formatting, linting and spell checking).

[ ] 🥚 Folder Structures: You can explain how files and folders are stored
in your computer and can find, open or create files in your computer
without using the Command Line Interface.
🥚 Command Line Interface (CLI): In a Unix shell you can ...

[ ] open a new terminal window
[ ] navigate up and down directories using cd
[ ] list the contents of a directory using ls
[ ] view the contents of a file using cat
[ ] create new files using touch
[ ] create new folders using mkdir

[ ] 🥚 Markdown: You can write a document in Markdown with no syntax
mistakes that renders into a well-formatted document.
🥚 VSCode: You can complete these workflows in VScode, and can use
keyboard shortcuts to complete them all:

[ ] Opening a repository in a new window
[ ] Opening the VSCode terminal
[ ] Adding a new file
[ ] Adding a new folder
[ ] Deleting a file
[ ] Deleting a folder
[ ] Previewing a Markdown File
[ ] Formatting a Markdown document

[ ] 🥚 READMEs: You can write a README file that describes the project you
are working on, why it's helpful, and how someone can use it.
🥚 NPM: You can use NPM commands to verify your code's quality, this
includes ...

[ ] using npm install to install a project's dependencies
[ ] reading a package.json file to find which scripts are available for
the project
[ ] use npm run <script> to execute an npm script

🥚 Formatting Code: You can use Prettier to make sure all the code in your
project is well-formatted:

[ ] You can use the Prettier VSCode extension to format a document while you
are writing it
[ ] You can use npm run format to format all of the documents in your
project
[ ] You can use npm run format:check to make sure all files are
well-formatted

🥚 Linting Folder and File Names: You can ...

[ ] use npm run lint:ls to check all folder and file names in your project
[ ] you can fix all linting mistakes reported by npm run lint:ls

🥚 Linting Code: You can ...

[ ] use npm run lint:md to check all Markdown files in your folder for
linting mistakes
[ ] you can fix all linting mistakes reported by npm run lint:md

🥚 Spell Check: You can ...

[ ] use the Code Spell Checker VSCode extension to correct spelling mistakes
in your Markdown documents while you are writing
[ ] use npm run spell-check to check the spelling in all the files of your
project
[ ] update .cspell.json to add words that should be allowed in your
project

🥚 File and Folder Naming Conventions: You can ...

[ ] identify and follow the naming conventions for the project you are
working on.
[ ] use npm run lint:ls to check that all files and folders follow the
project's naming conventions.
[ ] you can correct any file and folder names to make them match the project
conventions.

[ ] 🐣 File Extensions: You can identify all of the languages covered at
HYF and give the correct file extension. You don't need to know the
languages, just recognize them.
[ ] 🐣 Touch Typing: You can write a Markdown file without looking at your
keyboard to find any letters, numbers or special characters. (slowly is
ok!)

🥚 1. Local Development With Git
Practice using Git to save and organize your development process. You will learn
how you can use Git to go back to previous versions of your project, and to work
on different changes in parallel.

🥚 Git: Using the CLI you can ...

[ ] Initialize a new git repository with git init
[ ] stage changes using git add <path>
[ ] check what is staged with git status
[ ] commit changes using git commit -m <message>
[ ] display your repository's git history using git log
[ ] create a new branch using git branch <branch-name>
[ ] check out a branch using git checkout <branch-name>
[ ] create a new branch and check it out using
git checkout -b <branch-name>
[ ] merge changes from one branch to another using git merge <branch-name>
[ ] return to a previous version of your project with git log and
git checkout <commit-hash>
[ ] stash and retrieve uncommitted changes with git stash and git pop

🥚 .gitignore: You can use a .gitignore file to describe which files
you don't want included in your git history.
🥚 VSCode: You can ...

[ ] use the Git Graph extension to to visualize your repo's commit history
[ ] use the Git Lens extension to investigate your repository's commit
history
[ ] explain how the file tree in VSCode can show you which files have
uncommitted changes

[ ] 🥚 Atomic Commits: You can save your development progress using small
commits with clear and helpful message.
[ ] 🐣 Feature Branches: You can organize your development process using
branches. You can create a new branch for each part of your project and
merge those changes to main when they are finished.

🥚 1. Local/Remote Development
Learn how you can connect your local Git repositories with a GitHub repository
to add more structure to your development process and to share your projects.

🥚 Licenses: You can ...

[ ] explain why it's important to include a license for your code on GitHub
[ ] choose a license for your projects that matches how you want others to
use your code

[ ] 🥚 GitHub SSH Key: You can connect your computer to your GitHub
account using an SSH key, clone using the SSH link, and push/pull using
your SSH connection.
🥚 GitHub Repositories: You can ...

[ ] create new repository on GitHub
[ ] write a description for your repository
[ ] turn on GitHub Pages
[ ] configure your repository to block pushing to main
[ ] configure your repository to block merging to main until Continuous
Integration (CI) checks have passed

🥚 Git Remote/Local Connection: You can ...

[ ] clone a remote repository to your computer
[ ] initialize a new repository locally and connect it to an empty remote
repository
[ ] push and pull changes between remote & local branches

[ ] 🥚 Pull Requests: You can create a pull request between two branches
in your repository and merge changes without causing any conflicts.
[ ] 🥚 PR templates: You can use a PR template to add a checklist to all
of your PRs so you are sure the code is great before merging to main.
[ ] 🥚 Continuous Integration: You can use GitHub Actions to check your
code's quality before merging a pull request to the main branch.
[ ] 🐣 Local/Remote Branching Workflow: You can use a branching workflow
that keeps mistakes away from the main branch and prevents conflicts
from happening in GitHub. For each contribution to the project you can ...

Check out a new local branch and write your code
Check out main on your local machine
Pull changes from remote main to local main
Merge changes locally from main to your new branch

Fix any conflicts on your new branch before pushing!

Format and lint your code
Push your new local branch to your remote repository
Open a Pull Request from the new branch to main

Go through the PR's checklist to make sure everything is correct
Make sure all CI checks pass!

Merge your new remote branch to main

You can delete the branch after it's merged if you want to

Week 1

[ ] I have pushed my progress to HYF Class19 repository

I Need Help With:
Exercises successfully completed.
What went well?
Creating my student-dio markdown.
What went less well?
Wrapping my head around the step-by-step approach to the workflow using CLI.
Lessons Learned
It is required to re-clone a respository into my desktop seprated from that of GitHub to be able to work on VS code.
Better understanding of the terminals, npm, git, git branches.
Sunday Prep Work

RETRO
What was not clear, where did you get stuck?
My vscode was unable to preview markdown. I had to uninstall and re-install.
What was clear, what did you master?
Markdown was clear.
Where can you still use some help?
I can still use dome help with git and the required commands and CLI
Where can you help others moving forwards?
I can help others to create a good markdown

hi @JohnOlutubo, We are creating different issues for check in and deliverable. this is your deliverable issue linked with your pull request. thats why it was closed automatically after I merged it.
You should  create new issue with check in template and copy paste your questions&answers.

oh you did mixed them, i think. okay, i will reopen and review this as your check in. Next time, please create different issues.

@JohnOlutubo if you want to overview of the steps, I wrote the steps that we did for student bio task, you can check it out #120

Wrapping my head around the step-by-step approach to the workflow using CLI.

Week 2
I have pushed my progress to workflow-group-3
I need Help With:
No need for help so far. Thanks to our coaches, and my group members.
What went well?
There was a pretty good teamwork and collaboration.
What went less well?

I struggled a little with synchronizing my local repo with the remote.
It was a tough one realizing I had to rename my md file into lowercase after creating a pull request.
My workflow was disrupted in the attempt to make corrections, and I had to start all over.

Lessons Learned

I now have a better grasp of the npm and Json packages, and dependencies.
I also learned that it is a good practice to use git status often to check the state of work and also to commit very often.
I am getting more familiar and comfortable with the CLI.

Sunday Prep Work
I started reading on Agile Development , and I will continue to practice HTML & CSS.


Handler ErrorTracker.Integrations.Oban has failed and has been detached

Hi! I'm seeing this in the logs and I'm not sure why that's happening.
I'm using these versions:
elixir 1.14.5
erlang <IP_ADDRESS>
phoenix 1.7.12
oban 2.18.2
oban_pro 1.4.11
oban_web 2.10.5
error_tracker 0.2.2

Development logs
[error] Handler ErrorTracker.Integrations.Oban has failed and has been detached. Class=:error
Reason={:badkey, :file, nil}
Stacktrace=[
  {ErrorTracker.Error, :new, 3,
   [file: 'lib/error_tracker/schemas/error.ex', line: 32]},
  {ErrorTracker, :report, 3, [file: 'lib/error_tracker.ex', line: 106]},
  {:telemetry, :"-execute/3-fun-0-", 4,
   [
     file: '/Users/jaime/code/myapp/deps/telemetry/src/telemetry.erl',
     line: 160
   ]},
  {:lists, :foreach_1, 2, [file: 'lists.erl', line: 1442]},
  {Oban.Queue.Executor, :emit_event, 1,
   [file: 'lib/oban/queue/executor.ex', line: 284]},
  {Oban.Queue.Executor, :"-call/1-fun-0-", 1,
   [file: 'lib/oban/queue/executor.ex', line: 84]},
  {Oban.Backoff, :with_retry, 3, [file: 'lib/oban/backoff.ex', line: 108]},
  {Task.Supervised, :invoke_mfa, 2, [file: 'lib/task/supervised.ex', line: 89]},
  {Task.Supervised, :reply, 4, [file: 'lib/task/supervised.ex', line: 34]},
  {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 240]}
]

Staging server logs
2024-08-23T07:46:01.833662+00:00 app[web.1]: 07:46:01.833 [error] Handler ErrorTracker.Integrations.Oban has failed and has been detached. Class=:error
2024-08-23T07:46:01.833679+00:00 app[web.1]: Reason={:badkey, :file, nil}
2024-08-23T07:46:01.833680+00:00 app[web.1]: Stacktrace=[
2024-08-23T07:46:01.833680+00:00 app[web.1]: {ErrorTracker.Error, :new, 3,
2024-08-23T07:46:01.833681+00:00 app[web.1]: [file: 'lib/error_tracker/schemas/error.ex', line: 32]},
2024-08-23T07:46:01.833682+00:00 app[web.1]: {ErrorTracker, :report, 3, [file: 'lib/error_tracker.ex', line: 106]},
2024-08-23T07:46:01.833683+00:00 app[web.1]: {:telemetry, :"-execute/3-fun-0-", 4,
2024-08-23T07:46:01.833683+00:00 app[web.1]: [file: '/tmp/build_477b1e97/deps/telemetry/src/telemetry.erl', line: 160]},
2024-08-23T07:46:01.833684+00:00 app[web.1]: {:lists, :foreach_1, 2, [file: 'lists.erl', line: 1442]},
2024-08-23T07:46:01.833684+00:00 app[web.1]: {Oban.Queue.Executor, :emit_event, 1,
2024-08-23T07:46:01.833684+00:00 app[web.1]: [file: 'lib/oban/queue/executor.ex', line: 284]},
2024-08-23T07:46:01.833685+00:00 app[web.1]: {Oban.Queue.Executor, :"-call/1-fun-0-", 1,
2024-08-23T07:46:01.833685+00:00 app[web.1]: [file: 'lib/oban/queue/executor.ex', line: 84]},
2024-08-23T07:46:01.833685+00:00 app[web.1]: {Oban.Backoff, :with_retry, 3, [file: 'lib/oban/backoff.ex', line: 108]},
2024-08-23T07:46:01.833686+00:00 app[web.1]: {Task.Supervised, :invoke_mfa, 2, [file: 'lib/task/supervised.ex', line: 89]}
2024-08-23T07:46:01.833686+00:00 app[web.1]: ]

According to the stack trace it looks like a problem with certain exceptions that do not report the file of your app in which happened.
We will take a look and push a fix for it. Will keep you posted.
Thanks for the early testing and reporting the issue :)

I'm not able to reproduce this. I tried with several exception, including some that are specially difficult (' 1/0` or calling an unexistent function) without being able to reproduce it.
According to the stacktrace you are getting from telemetry it looks like the exception has no stacktrace lines, which is weird.
I can prepare a fix without reproducing it, but it would be best to be able to see why that happens.
Do you know which kind of exception did happen at first place? 👀 @jaimeiniesta

Thanks for taking a look! I'm not entirely sure. I suspect it's an Oban worker that saves screenshots to disk and then uploads them to S3, but I'm not sure. I'll try to find it.

Hey, I've been able to reproduce this. This ScreenshotWorker, in case of timeouts, will end returning {:error, :timeout}, so that Oban catches the error and retries. This is when ErrorTracker crashes. Here's how to reproduce it. This worker will always return {:error, :timeout}:
defmodule Myapp.Workers.TestWorker do
  use Oban.Worker

  def test do
    %{}
    |> __MODULE__.new()
    |> Oban.insert!()
  end

  @impl Oban.Worker
  def perform(_) do
    {:error, :timeout}
  end
end

If you launch the worker this is what happens:
ex(2)>  Myapp.Workers.TestWorker.test()
%Oban.Job{
  __meta__: #Ecto.Schema.Metadata<:loaded, "public", "oban_jobs">,
  id: 297861786,
  state: "available",
  queue: "default",
  worker: "Myapp.Workers.TestWorker",
  args: %{},
  meta: %{},
  tags: [],
  errors: [],
  attempt: 0,
  attempted_by: nil,
  max_attempts: 20,
  priority: 0,
  attempted_at: nil,
  cancelled_at: nil,
  completed_at: nil,
  discarded_at: nil,
  inserted_at: ~U[2024-08-26 07:12:28.666320Z],
  scheduled_at: ~U[2024-08-26 07:12:28.666320Z],
  conf: nil,
  conflict?: false,
  replace: nil,
  unique: nil,
  unsaved_error: nil
}
iex(3)> [error] Handler ErrorTracker.Integrations.Oban has failed and has been detached. Class=:error
Reason={:badkey, :file, nil}
Stacktrace=[
  {ErrorTracker.Error, :new, 3,
   [file: 'lib/error_tracker/schemas/error.ex', line: 32]},
  {ErrorTracker, :report, 3, [file: 'lib/error_tracker.ex', line: 106]},
  {:telemetry, :"-execute/3-fun-0-", 4,
   [
     file: '/Users/jaime/code/myapp/deps/telemetry/src/telemetry.erl',
     line: 160
   ]},
  {:lists, :foreach_1, 2, [file: 'lists.erl', line: 1442]},
  {Oban.Queue.Executor, :emit_event, 1,
   [file: 'lib/oban/queue/executor.ex', line: 284]},
  {Oban.Queue.Executor, :"-call/1-fun-0-", 1,
   [file: 'lib/oban/queue/executor.ex', line: 84]},
  {Oban.Backoff, :with_retry, 3, [file: 'lib/oban/backoff.ex', line: 108]},
  {Task.Supervised, :invoke_mfa, 2, [file: 'lib/task/supervised.ex', line: 89]},
  {Task.Supervised, :reply, 4, [file: 'lib/task/supervised.ex', line: 34]},
  {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 240]}
]

Thanks for the detailed steps to reproduce it @jaimeiniesta ! The error makes sense because as the code of the worker is not generating an exception, no stack trace is returned from Oban.
I have prepared a fix in #70 which i hope will be released soon 🚂
Feel free to test it if you want, or even use that branch temporarily if it is breaking your envs.
And thanks again for so detailed issues. We are still polishing some parts and having reports from real users is really helpful 🤗

The fix has been released in v0.2.4 a few seconds ago 🚀

Hey, I'm trying the latest version, 0.2.4 and it works great, thanks!
I'm still thinking about what's the best strategy to deal with Oban errors, as I rely much on Oban's retry mechanism. For example, my ScreenshotWorker can deal with rate limits and timeouts in this way:
{:error, :rate_limited} ->
  {:snooze, 1}

{:error, :screenshot_timeout} ->
  if corrected_attempt(job) >= @max_attempts do
    {:cancel, :timeout}
  else
    {:error, :timeout}
  end

That is, for a rate limit we use Oban's snooze to automatically retry in 1 second. For a network timeout, we maye cancel the job after the max attempts has been reached, or return with {:error, :timeout} otherwise, so that Oban will automatically retry the job.
In this last case we don't really care about tracking the exception, because Oban is going to retry. We're only interested in tracking the error in the case of returning :cancel, as that's the last time Oban will deal with the job.
We have many other Oban workers that will fail often with network errors, timeouts etc., as our app is a web scraper.
What would be the best approach?

Hey @jaimeiniesta. I am currently working on a feature that will allow users to ignore errors that they don't want to be tracked.
I think that this would solve your issue. You could just ignore those errors and they wouldn't be tracked and wouldn't show in the dashboard either.

@crbelaus thanks, I've tried that branch and it works great, it's easy to define and ignore that matches on fields from the error or context, I would use that. 👍


Unify Code Examples' Style in README According to the Contribution Guide

Unify code examples' in README with the style specified in contribution guide:

Ending with semicolon
Spaces in JSON object

Thanks!


Table 3 - how to replicate experiments

Hi,
I really appreciated your work, the paper is clear and well-written.
I would need to replicate the experiments (especially on Split_Cifar100), could you then share the hyperparameters or a notebook?
Thank you very much

Hi @gdacciaro,
Thank you for your interest in our work and your positive feedback! I have another repo focused on the incremental learning method using the Avalanche library here: https://github.com/fagp/rebasin-IL.
I haven't closed your issue yet because this other repo only includes the RotatedMnist benchmark. However, I'm in the process of cleaning up and refactoring the code for the Split_Cifar100. Once completed, I'll add it to the repository, include a link to this new repo in the readme of sinkhorn-rabasin, and notify you since this seems to be your main interest.
Do not hesitate to reach out if you run into any additional problems.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 55f6faf.


[WIP] Let travis update our start page by committing to github

Please note that now travis can run jobs daily .. so we could have up to date facts

https://docs.travis-ci.com/user/cron-jobs/

@foxxx0 let's just close this


Use empty String if note does not have a value

The TestHuman now uses an empty string if a specific Note has no value.
This supports usage of optional fields in a Form.
This fixes #37

@rultor merge pls

@rultor merge pls

@aliceice OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge pls

@aliceice Done! FYI, the full log is here (took me 3min)


Support for adding custom headers to outbound requests

What ^ she said

Review ? @tushw @pagey

Hold off before merging, want to check something

Okay all clear. Review/Merge away.

Okay I refactored and rebased, let me know what you think @pagey.

@tushw addressed your concern on the unit tests by adding the loop.exec() call to catch the async runs. Good catch.

If thats everything, can this be merged in?

Yep, go for it.


Update Microsoft.Logging.Extensions dependency in bot builder package to 3.1.3

We’re attempting to consume the package in the Substrate Search repo since we’re integrating with Message Extensions, but we’re hitting version conflicts because our repo has moved to microsoft.logging.extensions 3.1.3 and the package depends on 2.1.0. Any chance we could update the dependency in the Microsoft.Bot.Builder package to resolve the conflict?

Hi @Nehaongithub, if your asp.net core app is targeting dotnet 3.1, you should get microsoft.logging.extensions 3.1 automatically, if not, you can try adding an explicit reference to 3.1.3 to your project and I think that should override the request for 2.1, let us know if that's not the case.
Gabo

Fixed by https://github.com/microsoft/botbuilder-dotnet/pull/6341


[FR] Add support for folder note

I use Folder Note a lot (ie all my folder have a folder note attached to them).
Problem, as MakeMD doesn't support them, I have multiple note and I lost the usefulness of them.

For me, the display for folder must be erased by the folder note.
Alternatively, adding the folder layout (cf. image)

to the folder note, like this :

This is being added in 0.6, stay tuned!
Get Outlook for iOShttps://aka.ms/o0ukef

From: Lisandra Simonetti @.>
Sent: Tuesday, November 22, 2022 6:32:39 PM
To: Make-md/makemd @.>
Cc: Subscribed @.***>
Subject: [Make-md/makemd] [FR] Add support for folder note (Issue #12)
I use Folder Notehttps://github.com/aidenlx/alx-folder-note a lot (ie all my folder have a folder note attached to them).
Problem, as MakeMD doesn't support them, I have multiple note and I lost the usefulness of them.
[image]https://user-images.githubusercontent.com/30244939/203441552-36f3e2f0-4fa9-4ffe-bc2e-a9de0f2330b3.png
For me, the display for folder must be erased by the folder note.
Alternatively, adding the folder layout (cf. image)
[image]https://user-images.githubusercontent.com/30244939/203441677-ede9b114-6e27-4eb2-bda9-f2c8f6894f7c.png
to the folder note, like this :
[image]https://user-images.githubusercontent.com/30244939/203441899-ef1666e9-9154-4a74-a621-b5ba4bdb7326.png
—
Reply to this email directly, view it on GitHubhttps://github.com/Make-md/makemd/issues/12, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAFHKNQ4MBM6YNKR5MHSQPLWJVJZPANCNFSM6AAAAAASILQC2I.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Added in 0.6


Support nested fields DESCRIPTION ingestion using Trino

Is your feature request related to a problem? Please describe.
Many of our tables have nested fields, for example of type struct<id:varchar,name:varchar,sort_order:bigint>.
We're using Trino database service with metadata ingestions, our stack includes S3 for storage and Apache Iceberg under the hood.
We saw that for non-nested fields or for the top level of a nested field there is a DESCRIPTION being ingested, however it's not the case for the nested fields themselves.
Describe the solution you'd like
We would like that DESCRIPTION for nested fields would be ingested exactly like it is for regular fields, and would be reflected in OMD table schemas, as it's important metadata for our users.
Describe alternatives you've considered
An alternative would be to manually update this metadata or to maintain a scheduled job but it will probably be very time consuming to develop and to maintain.
Additional context
We're using OMD version 1.1.1, Trino version 409.
You can see in the following pics that description exists in Iceberg metadata files also for the nested fields but actually in OMD it's not reflected.

@eladrus we are getting the information directly from trino like this:
row(name varchar, address row(city varchar, state varchar), age bigint) and it seems related to iceberg metadata file more compared to trino comments.
We already have a iceberg connector in the making which should resolve the issue you are facing, you can track the addition of iceberg from here #14825 and the issue #5448

Closing this issue as the nested descriptions can be fetched from the iceberg connector which can be found in #14825


Fix rendering issue

Fix rendering issue in keras.io by making change from "Call Args:" to "Call arguments:" which is allowed docstring to render.
Fixes: https://github.com/keras-team/keras/issues/20297

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.43%. Comparing base (7628a18) to head (9450aa3).
Report is 38 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (7628a18) and HEAD (9450aa3). Click for more details.
HEAD has 9 uploads less than BASE

Flag
BASE (7628a18)
HEAD (9450aa3)

keras.applications
4
0

keras.applications-numpy
1
0

keras.applications-torch
1
0

keras.applications-jax
1
0

keras.applications-tensorflow
1
0

keras-tensorflow
1
0

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #20301      +/-   ##
==========================================
- Coverage   66.34%   57.43%   -8.91%
==========================================
  Files         508      511       +3
  Lines       48264    48985     +721
  Branches     8882     9020     +138
==========================================
- Hits        32020    28135    -3885
- Misses      14425    19038    +4613
+ Partials     1819     1812       -7

Flag
Coverage Δ

keras
57.43% <ø> (-8.91%)
:arrow_down:

keras-numpy
57.43% <ø> (?)

keras-tensorflow
?

keras.applications
?

keras.applications-jax
?

keras.applications-numpy
?

keras.applications-tensorflow
?

keras.applications-torch
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Connection Issues with AWS OpenSearch on EC2 Deployment

Hi @pdphilip Firstly, thank you for this amazing package—it makes querying OS/ES very straightforward.
I am encountering a connection issue when deploying to an EC2 instance with AWS Hosted OpenSearch. While everything works perfectly on my local machine, the connection hangs and fails when using the package on the EC2 instance.
Details:

I am able to connect to the OpenSearch instance on the EC2 instance using CURL without any issues.
However, when using the package, the connection to the OpenSearch instance fails and hangs indefinitely.

Hey @aaronseanph, thanks for the kind words.
Have tried on AWS and got the same issue until I added the port 443 on the host, ex:
OS_HOSTS="https://xxxxxxxxxx:443"

I did see an error in the connector where some of the options were not applied (like SSLVerification - default true), so will update that, but that didn't affect my test.
If still no luck then see if you can connect with the OS PHP client directly: https://opensearch.org/docs/latest/clients/php/
If you can, then please share how you built the client (without sensitive details of course). Thanks

@pdphilip Thank you so much for your quick response and for providing a working solution! The issue is now resolved, and the package is working perfectly on the EC2 instance with AWS Hosted OpenSearch.
I really appreciate your help and the amazing work you’ve done with this package. It has been a huge asset for our project.
Thanks again!

Great @aaronseanph, fantastic that you brought this to my attention. I'll update the docs so others don't trip over this one.
Good luck with your project


Get rid of Figaro warning.

WARNING: Use strings for Figaro configuration. 587 was converted to "587".
WARNING: Use strings for Figaro configuration. true was converted to "true".

Coverage remained the same at 54.225% when pulling 74465024edc1892a7e2db0de2b739bb9f887500d on enderahmetyurt:patch-1 into a4508c4113197bf0ca0bb219d71faf2ddb427a2e on FarmBot:master.

Thanks!


Script and first run output to show when Open API changes

Signed-off-by: john court<EMAIL_ADDRESS>

This pull request provides a script to generate the openapi.json spec file from a running ACA-py.  The openapi.json can be checked in and used to compare in subsequent releases for unexpected changes.   Apart from explicit changes to the REST API the only changes should be the random/timestamp related "example" fields in the spec file.  This submit is in response to issue #742

Codecov Report

Merging #777 (fc019de) into master (a2ccc40) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #777   +/-   ##
=======================================
  Coverage   99.10%   99.10%
=======================================
  Files         256      256
  Lines       14010    14010
=======================================
  Hits        13885    13885
  Misses        125      125

How does this sort the output openapi.json file?
I tried running the script and the file I generated was in a completely different order than the checked-in file (so there were a lot of spurious differences between the files).

It doesn't sort the output at all, or shouldn't.  The intent is that the output coming from ACA-py is already sorted.  This was the intent of the script to detect changes.  I have run it many times but maybe there is something specific to my installation.  Can I ask what version of ACA-py you are running it against ?  as prior to V0.5.6 you may well see issues since this is why the script was created.

I have the latest master branch checked out, and I merged the code from this PR.

@ianco -- the idea for this came about because of @sklump 's change to make the openapi.json file come out sorted already. If that is not working, we need to look at where it is coming from.  Perhaps that feature was lost somewhere?

@swcurran  what's the PR?  I'll check to make sure it's included in my local ...

Looks like it is #738.

ok that's definitely included in my local

Can you post the output you are seeing ?   I may be able to tell what is happening based on what the differences are and having seen the wrong behaviour from ACA-py previously.   So the only other possibility is that the swagger codegen for some reason outputs different openapi.json randomly from the same raw swagger.json from ACA-py.  I haven't seen this to date and in particular there are options to swagger to artificially sort which we intentionally aren't using because that would cause other problems around detecting ACA-py changes.

So I just pulled up to the latest master and re-ran, it behaved as expected but one thing I realised is that if you are already running an ACA-py on your local machine, its possible the script will fail to start its own and instead end up pulling the swagger file from the pre-existing instance.  @ianco is this a possibility with your environment ?  I would still like to see your openapi.json but will make sure the script exits if the ACA-py start phase fails.  I will also change the admin port to be a non-standard one everyone seems to use so that there is less chance of connection to an existing instance by mistake.

I made sure everything was shut down and re-ran the script, the output was the same.  (It's possible another aca-py was running when I tested this earlier, I'm not sure.)
Here's my output:
openapi.json.zip

FYI the api list on the swagger page is now sorted (according to the PR that @swcurran referenced) however the generated opeapi json file is not.  It looks like the file you generate (checked into your PR) is different from mine, and neither are sorted the same as the swagger page.

Yep, it looks like there is an issue with swagger not maintaining the order of paths from the raw input.  The actual issues I had filed previously were around parameter order and object model order since they caused me wrapper code breakages.  I don't understand why it has consistently created the same output for me so I didn't see the issue (in fact I didn't even realise ACA-py was also sorting paths).  It could be as simple as different CPU speeds or timezone affecting internal ordering.    The sad thing is that it doesn't look like either OpenAPI or Swagger code generators have really addressed this issue so I might have to look at a different approach to formatting the "raw" interface file from ACA-py Into something more diff friendly. Will investigate.

@ianco I just need to challenge your assertion that the raw swagger output from ACA-py is sorting the paths ?  I took the raw output and feed it through an online JSON pretty printer and the result doesn't show paths sorted in any meaningful way ?
I went through the painful process of checking the first few hadn't changed order between raw and the pretty format.  Attaching pretty format file.
jsonformatter.txt

I have also just gone through #738.  That seems to be sorting swagger_dict["definitions"]  I don't see anywhere in the code where swagger_dict["paths"] gets sorted, I do see parameters within paths being sorted.

I haven't looked into the underlying files, so possibly you're right.
In the displayed swagger page (i.e. in the browser) everything is displayed sorted by message type and then path, I don't know where all this sorting is taking place.

I suspect it must be happening in the web server part of the code not in the core.

I have also just gone through #738. That seems to be sorting swagger_dict["definitions"] I don't see anywhere in the code where swagger_dict["paths"] gets sorted, I do see parameters within paths being sorted.

It sorts recursively, "asciibetically", so paths will get sorted within definitions etc.
https://github.com/hyperledger/aries-cloudagent-python/pull/738/files#diff-f3648d939ba43f8f8b8a42f8786f921dd598def57267dd9e3904a3a9ff391d6dR348

@ianco how do you come to that conclusion ? They are seperate dictionaries aren't they ?  I assume definitions are the model definitions :
   # order tags alphabetically, parameters deterministically and pythonically
    swagger_dict = self.app._state["swagger_dict"]
    swagger_dict.get("tags", []).sort(key=lambda t: t["name"])
    for path in swagger_dict["paths"].values():
        for method_spec in path.values():
            method_spec["parameters"].sort(
                key=lambda p: (p["in"], not p["required"], p["name"])
            )

    # order definitions alphabetically by dict key
    swagger_dict["definitions"] = sort_dict(swagger_dict["definitions"])

I can also find some time to look at this today

Do you need me to sort paths too? I see that server startup only sorts the values therein explicitly.

@sklump yes I think that is why @ianco is seeing different output when he runs it to me.  I really think this must be due to simple memory, cpu differences causing order differences on "paths" as I run it with different ACA-py versions and don't see any unexpected changes.
Thanks !

See https://github.com/hyperledger/aries-cloudagent-python/pull/785


Fix emitted code for variable statement with no bound variables

Fixes #3893.

Does this work for parameters?
function f({} = {}) { }
function g([] = []) { }

We have (tests/cases/conformance/es6/destructuring/emptyObjectBindingPatternParameter04.ts)[https://github.com/Microsoft/TypeScript/blob/8952ed66b2e3ba78d64e8399384dd55ecfccabd8/tests/cases/conformance/es6/destructuring/emptyObjectBindingPatternParameter01.ts] whose baselines seem fine.

Actually @JsonFreeman, since the issue was when an identifier appeared in the initializer, that's what we should be testing. I'll amend my tests.

Done.


Doc does not reflect new syntax

I am somehow unable to get drone-email working with the new syntax.
 \
  -v $(pwd):$(pwd) \
  -w $(pwd) \
  drillster/drone-email

I successul receive an email when I execute this locally.
In drone.yml I do have:
- name: notify
  image: drillster/drone-email
  host: $EMAIL_HOST
  port: 587
  PLUGIN_FROM<EMAIL_ADDRESS>  recipients_only: true
  environment:
    EMAIL_USERNAME:
      from_secret: EMAIL_USERNAME
    EMAIL_PASSWORD:
      from_secret: EMAIL_PASSWORD
    EMAIL_RECIPIENTS:
      from_secret: EMAIL_RECIPIENTS
    EMAIL_HOST:
      from_secret: EMAIL_HOST
  when:
    status:
      - failure

I also tested very other different settings, but I always get (on our drone server):

time="2019-05-13T11:10:50Z" level=error msg="Could not send email to \"********\": gomail: could not send email 1: gomail: invalid address \"\": mail: no address"
--
2 | gomail: could not send email 1: gomail: invalid address "": mail: no address

Locally I tested it with
- name: notify
  image: drillster/drone-email
  from<EMAIL_ADDRESS>  host: smtp.server.com
  when:
    status:
      - failure

then I do get
 drone exec                                                                                1 ↵
[test:0] + exit 1
[notify:0] time="2019-05-13T12:10:12Z" level=error msg="Error while dialing SMTP server: dial tcp :587: getsockopt: connection refused"
[notify:1] dial tcp :587: getsockopt: connection refused
2019/05/13 14:10:13 notify : exit code 1

I tested above docker command with setting "host=" and I got the same error message.
So locally on "drone exec" it seems that it does not even pick up the host value.
Therefore my fear is that my syntax is wrong.
Can you please have a look and document it in a proper way?

Stumbling over the same symptom:
level=error msg="Error while dialing SMTP server: dial tcp :587: getsockopt: connection refused"

@tobiasKaminsky Got it figured out: the configuration has to go under a settings header:
  - name: notify
    image: drillster/drone-email
    settings:
      recipients_only: true
      recipients:
        -<EMAIL_ADDRESS>      subject: News from drone!
      host: host.some.org
      port: 587
      from<EMAIL_ADDRESS>      username:
        from_secret: mail_user
      password:
        from_secret: mail_password

(using secrets stored in the repo settings on drone). That solved the issue for me.

I'm glad you figured this out on your own, I'm still running 0.8 so it'd be hard for me to find the problem. As soon as I update to 1.0 myself I'll make sure the docs are straightened out.


Portability Rework

arduino-cli libraries do not seem to work consistently across all machines.
My solution to this is, all libraries not native to the Arduino IDE, will be cloned from their source into the arduino/lib working directory.
The makefile build will still be exclusive to arduino-cli, but the libraries install command must only be git clones, for portability.

fixed the bug, no need for rework anymore


Programmatically accessing sections

Is it possible to extract section content programmatically using RazorEngine?
// template file
@section Header {
    <text>Hello from @Model.Name</text>
}
@section Body {
<EMAIL_ADDRESS>}

Do you want the section template content "<text>Hello from @Model.Name</text>" or the section output "Hello from myname"? And may I ask why?

I want to use Razor for email templates. It would be very convenient to store the email title alongside its body and use it like this (with hypothetical method names):
void SendEmail<T>(string recepient, string templatePath, T model)
{
    using(var reader = new StreamReader(templatePath))
    {
        var compiled = Engine.Razor.Compile(reader.ReadToEnd(), typeof(T), model);

        SendMail(
            recepient: recepient,
            title: compiled.GetSectionOutput("title"),
            body: compiled.GetSectionOutput("body")
        );
    }
}

Ah I saw people doing this before, please see #225 and #238. I think they may contain the solution you are looking for.

Thanks, I'll give it a go!


Use Github Pages for hosting.

Since this is currently a static site, we should be able to use Github Pages to host this. We can get the domain bmoreblacktechies.github.io for free, and it'll also be behind https, which is important.
I'm not sure if we need to be using a static site generator like Jekyll to leverage this.

I don't think so. On my repos I just go to setting and it generates the github pages and URL for me.

I think that this works for now! Im closing this issue unless others have objections.


grafana datasource not being out of the box

With devops-stack v0 we used to have by default a grafana_datasource config map declaring url: http://thanos-query.thanos:9090
This is not being created anymore

The datasource is actually added on the kube-prometheus-stack module. I think these lines is what you are referring to: https://github.com/camptocamp/devops-stack-module-kube-prometheus-stack/blob/c788c4e32ef1c30e4222aa1305ffb1b35b20bb8b/local.tf#L105-L121


Update esbuild dependency for vulnerability scanners

Description
Older esbuild versions use versions of Go which flag a number of CVEs on vulnerability scanners.
https://github.com/evanw/esbuild/issues/3802
https://github.com/evanw/esbuild/issues/3853
Vite currently has a dependency on esbuild ^0.21.3:
https://github.com/vitejs/vite/blob/0ca53cff9ff49108fcec75ff01d2445f9c2f2a4c/packages/vite/package.json#L88
Suggested solution
Bump dependency on esbuild to latest (^0.23.1) which uses later versions of Go and addresses the CVEs described in the GitHub issues above.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

We have received many reports of this but this is not a vulnerability in Vite. They only affect esbuild's server, which we don't use. Please update this in the scanner instead.


pull request for e2e test at 01/26/2017 04:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


polybar-msg cmd restart causes systray to disappear until second restart

When every I use the command polybar-msg cmd restart it restarts the polybar as expected, but it the systray on the bottom right seems to disappear (without any of the processes going away). It only seems to reappear after the running the commmand a second time.
Normal systray:

Disappeared systray:

What WM?

This issue is occurring on i3.

Can you post the entire polybar log when this happens. My guess is that the tray is positioned behind the bar by i3. Please also post the bar section of your config.

This is the output of polybar bottom & after doing polybar-msg cmd restart a couple of times.
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
warn: Termination signal received, shutting down...                                                                                1   879  12:13:01
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
error: tray: Failed to put tray above 0x0c00002 in the stack (XCB_MATCH (8))                                                                                1   879  12:13:05
warn: Termination signal received, shutting down...                                                                                1   879  12:13:05
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
error: tray: Failed to put tray above 0x0c00002 in the stack (XCB_MATCH (8))                                                                                1   879  12:13:07
warn: Termination signal received, shutting down...                                                                                1   879  12:13:07
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
error: tray: Failed to put tray above 0x0c00002 in the stack (XCB_MATCH (8))                                                                                1   879  12:13:07
warn: Termination signal received, shutting down...                                                                                1   879  12:13:10
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
error: tray: Failed to put tray above 0x0c00002 in the stack (XCB_MATCH (8))                                                                                1   879  12:13:13
warn: Termination signal received, shutting down...                                                                                1   879  12:13:13
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
error: tray: Failed to put tray above 0x0c00002 in the stack (XCB_MATCH (8)) ```
This is my config for my bottom bar (where my systray is located):
```;Bottom Bar
[bar/bottom]
monitor = ${env:MONITOR:HDMI1}
monitor-fallback = eDP1
bottom = true
width = 100%
height = 35

background = ${bar/top.background}
foreground = ${bar/top.foreground}

line-color = ${bar/top.background}
line-size = 2

spacing = 3
padding-right = 4
module-margin-left = 0
module-margin-right = 6

font-0 = NotoSans-Regular:size=10;2
font-1 = unifont:size=8;-1
font-2 = FontAwesome:size=10;3
font-3 = NotoSans-Regular:size=10;1
font-4 = MaterialIcons:size=12;1
font-5 = Termsynu:size=10:antialias=false;2

modules-left = i3
modules-right = cpu memory
tray-position = right
tray-maxsize = 24

enable-ipc = true

I think you are suffering from the same thing as reported in #425, just the other way around.
I think the solution here might be to set override-redirect = true. This will cause the bar to overlap with the other windows though, there is a section on our known issues page addressing this.

This should be fixed by #2609 if you are using the new tray module.


fix(docs): rehaul styling

:tada: This PR is included in version 0.98.0 :tada:
The release is available on:

GitHub release
v0.98.0

Your semantic-release bot :package::rocket:


change logo and link

I recommend these changes be withdrawn.
These updates are private use customizations.
Please see:
https://github.com/facebook/create-react-app/pull/13346/files


Loss of room-scale tracking if tracking is lost

Expected Behavior
Whenever the tracking is lost and resumed, the camera should readjust position and resume room-scale tracking.
Actual Behavior
The camera stays at the height where it lost the tracking and stays there, even though tracking was picked up again. From the looks of the BoundaryManager TrackingSpaceType.RoomScale is used only once in Awake for setup. I couldn't find anything related to detecting changes in tracking state.
P.S.: This is an issue detected by Microsoft themselves when they reviewed our app.
Unity Editor Version
MRTP4
Mixed Reality Toolkit Release Version
status from last Wednesday

This is also happening without the Mixed Reality Toolkit in a new Unity Project. Anyone knows a workaround?

So this is a unity bug?

Please add repro steps

If @Alexees and I are talking about the same problem, it is a Unity Issue imo

Enter Playmode
=> Positional Tracking is working
Make the Immersive Headset loose positional tracking by holding hands on the cameras
Release your hands again
=> Positional  Tracking won't work until you leave and reenter playmode (restart the app)

I believe this is also related to #1324

Probably.
I also tried the new patch version that was released today (2017.2.0p2) but no luck.
I guess there is nothing you can do about it, right?

Probably best to keep to the MRTP builds

@StephenHodgson if I'm not mistaken, the reason this doesn't work is because the UnityEngine.VR.WSA.WorldManager class is nowhere to be seen.
https://developer.microsoft.com/en-us/windows/mixed-reality/tracking_loss_in_unity

fixed in MRTP 5 which is available now

Confirmed fix in MRTP5
https://forum.unity.com/threads/custom-build-2017-2-rc-mrtp-windows-mixed-reality-technical-preview.498253/

Looks like GitHub didn't automatically close these issues when merged into master. I'll close these manually and look into why.


chore(l10n): [ja] update locale

partially related: Comments of https://github.com/vrc-get/vrc-get/pull/1561

ありがとうございます


Fix e2e tests for external-dns v0.10.0

Previously they were failing because the external-dns-values file was
missing (it was being gitignored so it was not obvious).
Signed-off-by: Christian Ang<EMAIL_ADDRESS>Co-authored-by: Edwin Xie<EMAIL_ADDRESS>What this PR does / why we need it

The external-dns e2e tests were missing the values file. Looks like we missed committing it because it was gitignored. We renamed the file to prevent missing the file again.
Details for the Release Notes (PLEASE PROVIDE)

NONE

Which issue(s) this PR fixes

Fixes: n/a
Describe testing done for PR

Ran the tests and ensured it was using the correct filename.
Special notes for your reviewer

/lgtm


Exercise yacht

Adding Exercise yacht

LGTM

Thanks @kotp, I just realised that global config.json entry was missing - added that and will now merge PR


Images not loading on localhost

I'm working on a nextjs app locally and noticed that the images in the UI aren't loading. I checked out the image sources it looks for it there seems to be an assumption that relative paths like /expand.svg should be served, which isn't always the case.

Looks like this has been addressed. Please file another issue if this I am mistaken.


Adds -DraftVersionVisibility option to Set-PnPList cmdlet

Type

[ ] Bug Fix
[x] New Feature
[ ] Sample

Related Issues?
Fixes #2282
What is in this Pull Request ?
New option to set the draft item security in a list. Content approval should be enabled on the list in order to see this value changing in the UI.
Possible values: Reader, Author, Approver
Example:
Set-PnPList -Identity "Testlist" -DraftVersionVisibility Approver

Thanks @milanholemans , have merged the PR !


Support for Meta's llama3.1

Thank you for your contribution, we really appreciate it. The following instructions will help improve your pull request and make it easier to receive feedback. If there are any items you don't understand, don't worry. Just submit the pull request and ask the maintainers for help.
Motivation
Please explain the motivation behind this PR and the goal you aim to achieve with it.
Modification
Briefly describe the changes made in this PR.
Checklist

Ensure pre-commit pre-commit run --all-files or other linting tools are used to fix potential lint issues.
Confirm that modifications are covered by complete unit tests. If not, please add more unit tests for correctness.
Modify documentation as needed, such as docstrings or example tutorials.

Thanks for your help. It has already been fixed by https://github.com/sgl-project/sglang/commit/444a02441a50f87d86c406901b3726b323a3fc0f


Move testing docs to Wiki

New docs here: https://github.com/finos/greenkey-discovery-sdk/wiki/Testing-Interpreters

:+1:
i like the table for built-in entities
https://github.com/finos/greenkey-discovery-sdk/wiki/Included-Entities


Grid of thumbnails?

Hello,
I love your work, just a feature request, do you plan to implement the grid of thumbnails   (just like in MWPhotoBrowser)?
Thanks

+1

Any update on this? :)


add KubeArmorController

Signed-off-by: achref ben saad<EMAIL_ADDRESS>

Please check if there are wrong names that you missed to change while merging three controllers together. Also, please check if you correctly bring the functions and intents for each controller to this unified controller. I can still see some incorrect points.

Can you please point me to the incorrect points, I ran the make manifests and the results are a carbone copy of the original.

Hey @nam-jaehyun , can you please revisit your comments and @achrefbensaad responses ... and provide a signal to go ahead/no-go with this?


PyGBe LSPR - Joss paper summary

@gforsyth @cdcooper84 @labarba
Here is what I would say a first draft for the lspr submission to joss. I'm not completely satisfied with it, I think it needs more work, but I need criticism to improve it. Would you mine taking a look at it and make changes, suggestions, comments, etc.
Thanks,
Naty

@labarba If you think the last version I pushed is ready to submit go ahead and merge the PR. To do the release I need to know which is going to be the tag we want to assign to the release when I merge it to master. After that I can generate the metadata that JOSS submission require and finally submit.


Code Quality: RemovableDrivesService, IStorageDeviceWatcher

Please try to break this.
Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

Screenshots (optional)
N/A

@d2dyno1 Can you do a quick pass-through of the first commit?

@lukeblevins I want to do some additional testing, do you have any ideas I can use to stress test the functionality?

@yaira2 I'm pretty satisfied with the reliability of this PR. Let's merge it before the changes get stale again.

/azp run


[DOC] Improve new user documentation

The framework lacks documentation on how to integrate sutsim into a firmware codebase.

Update README or create separate document outlining integration steps (fw arch assumptions, shared lib builds, cmake flags, conftest, etc.)
Improve documentation of example code

Maybe move docs like this to the Wiki


PowerSelectMultipleWithCreate triggers a form submit on enter

It seems like using PowerSelectMultipleWithCreate inside a form with a submit listener will trigger the submit action when using the Enter key.
This doesn't seem to happen when wrapping a regular PowerSelectWithCreate with the form.
e.g. in the dummy app:
<form {{on 'submit' (action onFormSubmit)}}>
  <PowerSelectMultipleWithCreate
    @options={{countries}}
    @searchField="name"
    @selected={{selectedCountries}}
    @onChange={{action (mut selectedCountries)}}
    @onCreate={{action "createCountry"}} as |country|
  >
    {{country.name}}
  </PowerSelectMultipleWithCreate>
</form>

triggers the onFormSubmit method to be called when pressing the enter key.
I'm currently trying to trigger the Enter in a failing test but it's somewhat complicated :thinking:
Seems like manually pressing Enter triggers a submit event on the form element.
If I do it via triggerKeyEvent() it doesn't work.

edit: It seems like the issue is that Multiple adds the input inline while the other one places the search input into the dropdown wormhole.
edit2: It seems like this is the expected behavior for any input inside a form element.
@cibernox do you think it would be a good idea to prevent the enter event inside the power-select trigger from bubbling up?

It seems like there's already Enter handling in power-select:
https://github.com/cibernox/ember-power-select/blob/master/addon/components/power-select-multiple.js#L49
If i add e.preventDefault(); near the e.stopPropagation(); it works as expected.


Improve unsaved changes message logic when a resource is about to be deleted

Reproduction steps

[ ] compose
[ ] admin


Fix error running unit tests

When running tests, the following error occurs:

VSTest: Could not locate executable.

https://github.com/cake-build/cake/issues/1522#issuecomment-341612194 indicates that there's an issue using VS 2019, and presents a workaround to find the correct path

Thanks a lot I did not know what was happening with this.


Footnotes don't get reset after Postprocess

Problem
Once markdown instance converted text containing footnotes, those footnotes' contents keep appearing at every single result converted by the markdown instance.
How to reproduce
import markdown

md = markdown.Markdown(extensions=["markdown.extensions.footnotes"])
md.convert("Foo[^1] Bar\n[^1]: Baz")
print md.convert("Foo")
print md.convert("Bar")

Desired Output

Foo

Bar

Current Output

Foo

Baz ↩

Bar

Baz ↩

As documented, this is a feature, not a bug. You need to call the convert method on the Markdown class between each call to convert. There may be valid reasons why a user may not want to reset between documents. For example, on the front/home page one could collect all the footnotes for multiple posts and display them al together at the bottom. As an added bonus, all of the links to the footnotes would then be unique with no duplicates.

I can't grasp your explanation. Can you give me a concrete example?

I can't grasp your explanation. Can you give me a concrete example? I know some kind of things have some advantages on keep their state but not with footnotes.

The one and only way to reset state should be to call Markdown.reset(). No other extension gets reset called without it explicitly being requested by the user, therefore we should not expect anything different for footnotes.
Regarding my stated use case, I had forgotten that when the UNIQUE_IDS config option is set to True it works across calls to reset. That said, the expectation is that there would be calls to reset - which is clearly documented.


Implement MapReduce updates processing job

In order to use data localization pending updates processing can be performed in Map function of simple MapReduce job with HBaseTableIntput (with map-per-region task).

Think over implementing map-only job for doing compaction. This may cause some spans of records to be compacted into multiple result records, which is usually (alwasy?) ok. Multiple updates processing results records should be supported already.

One concern is about writing into same table from a Mapper which may cause issues.


Array service not working

Hi!
Array service isn't working, because it hasn't tag 'swap.service'.

Hey @denlapaev. Thanks for your pull request.
Can you submit the pull request without the composer.json changes please ?

Sorry! It's automatically


Updated apt::source. Now should work with puppetlabs-apt versions 2.x

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

Hi,
I'm not planning to only support the 2.0 apt module since older PE versions are possibly pinned down to older apt module versions.
I'll be pinning the module to use apt < 2.0


Fix automator pause command ticking wrongly

The original code use the milliseconds method which means remain milliseconds besides full seconds. However, the automator obviously should tick by total milliseconds passed. So it would cause unexpected behavior with tick longer than 1s. (maybe happen when calculating offline progress)
The totalMilliseconds method means total milliseconds, as expected.

This looks good and works properly, nice catch.
Also, sorry for taking so long to get around to looking at this one.


fix: fix $__rate_interval parenthesis issue

Some prometheus queries are false since the parenthesis is misplaced on the $__rate_interval

Thanks!


[#296] Inconsistent zoom toggling

Closes #296.

Coverage decreased (-0.01%) to 88.28% when pulling 92d3f07fa5696bc4c788be0592bc480934d52c93 on feature/296-fix-toggle-zoom into f6e9bfd2a978557749e33209d963f307ad429804 on master.


placement of 	abnormal response to tactile stimuli

Currently abnormal response to tactile stimuli is a child of abnormal mechanical nociception; however, mechanical nociception is limited to painful stimuli while response to tactile stimuli is not. I think these terms should be siblings under abnormal touch/ nociception. Or we may want to consider separating these into abnormal response to touch and abnormal nociception. Mechanical nociception would still be a child of both but other touch responses could be placed only under the touch branch.
Reported by: sbello
Original Ticket: obo/mammalian-phenotype-requests/645

placement has been changed


[Bug]: Product price doesn't change on variant update

Operating system
macOS Big Sur
Node.js version
16.9.0
Shopify CLI version
2.0
Browsers
Chrome
Version
4 (Current)
Modifications
No modifications.
Details
Hi there, if variants have different pricing (red shirt: $10, blue shirt $20), the product price doesn't change when you swap between the two options.
Notice

[X] I read the notice

This issue should be fixed in the latest release


Open server socket

When the program starts it should open a serversocket and listen for incomming connections.
Each incomming connection should be handled in it's own thread.

Should I use an out/input stream to confirm connection to the server? Or how could I test that a connection is established and put into a new thread?

When accepting an incoming connection from the ServerSocket it will return a Socket object. From this object we can retrieve things like input and output streams and connected ip and other things. So we can transfer a reference for that object into the treads method.
For verification of this functionality you can open the output stream and send some simple Hello World text.
When it comes to writing tests for this we can't really do a unit test without making the code more testable. So if you want to verify that the code starts a server and calls accept waiting for connections it can be done using an interface and dependency injection:
https://stackoverflow.com/questions/38957460/how-to-test-method-that-internally-creates-and-uses-a-serversocket
Later in the project we can verify this behaviour by starting the whole server and using HttpClient for making calls against the server as a form of Integration tests.

https://docs.oracle.com/javase/tutorial/networking/sockets/clientServer.html

Issue resolved


adding checks for json end_document in http batching interceptors

a cherry-pick of https://github.com/apollographql/apollo-kotlin/pull/5892 on the main branch.

@duzinkie: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


feat: Allow to disable import insertion on single path glob imports

On by default as I feel like this is something the majority would prefer.
Closes #8490

bors r+


Update configuration files to match cpu/gpu quotas

Context
Our Terraform scripts appear to use an arbitrary limit on the number of nodes such as
min:0
max:100
Similar quotas may also exist within the JupyterHub configuration files.
This may cause a user to request a new server and even though JupyterHub truies to make the request, it can fail the underlying k8s infrastructure hits a quota for the maximum number of CPUs/GPUs in that particular cluster.
While 2i2c engineers can see that this limit has been reached and then try and rectifying the situation by request a higher quota this is not at all apparent to users.
This is especially confusing behaviour because JupyterHub appears to timeout after 600s with only fairly cryptic debug messages from KubeSpwaner.
Proposal
We could be maintaining an internal table of what quotas we actually have for different node type and for different quotas.
We could use make the quota limits by terraform or JupyterHub to make it clear when a new node should not be requested (or at least fail quickly) if we are in an over quota situation.
For nodes that have relatively small quotas, we should update the description of the machine type it make it clear that there are only a few such nodes available.
This particular issue raised its head for a recent m2lines tutorial where they wanted up ~30 GPUs for adhoc event but hit a limitation of only getting 2 GPU enabled nodes.
Updates and actions
The particular details of how to actually implement this UX improvement need to be clarified by @2i2c-org/engineering .  The proposal above is only an initial brainstorm on how it could be improved and is open for discussion.

From https://2i2c.slack.com/team/U01742BQ99N

Looking at Google Cloud quotas for use of CPU and GPU, it seems they are more relaxed overall than AWS.
Here are examples of GPUs and CPUs (from dedicated meom-ige cluster) that can be provided in a cloud region (note that the quota limitations are mostly in the region for GCP, not in a specific zone).
Have these quotas been increased by us, or did they start out like that?

Action:

[ ] Create a table of what our current quotas are for each project / region / zone


strictNullChecks with optional properties on class change type

TypeScript Version:  4.0.0-dev.20200504
Search Terms: optional class strictNullChecks generics type
Code
type RefType = number | string | Buffer;
type test<T, S extends RefType = number> = T | S;

class SomeClass {
  public prop?: test<string>;
  // type of "prop" when "strictNullChecks" is disabled "test<string>" (how it should be)
  // type of "prop" when "strictNullChecks" is enabled "string | number | undefined"
}

Expected behavior:
that prop is of type test<string> (or in strict mode test<string> | undefined)
Actual behavior:
prop is of type string | number | undefined
Playground Link:
not working as expected: Playground Link
working as expected: Playground Link
Related Issues:

#37163 might be related

the current behavior breaks functions that depend on destructuring types
function getRefProperty<T, S extends RefType>(ref: test<T, S>): S {
  // some code
}

test<string> is exactly equivalent to string | number, and test<string> | undefined is exactly equivalent to string | number | undefined, so there's nothing wrong there.
The way you've written getRefProperty you'll never get useful inferences for S.  Inference proceeds left to right starting at T, and the sole argument to the function is a T | ..., so T will be inferred to simply be the type of the argument in all cases.  But then S is entirely unconstrained by the parameter values, so there's nothing to do but set it to some sensible default and hope things work out.
Keep in mind that types are structural and type unioning is commutative and associative, so whenever you have T | S it doesn't matter to typescript what "part" of the type is in T and what "part" is in S.
getRefProperty is a type assertion in disguise.
// returns "chicken noodles", of course
declare var c: SomeClass;
getRefProperty<string | number | undefined, "chicken noodles">(c.prop);

@nmain take a look at this example:
class ObjectId {
  anObjectId!: boolean;
}

export declare type RefType = number | string | ObjectId | Buffer;

type Ref<
  R,
  T extends RefType = R extends { _id: RefType } ? R["_id"] : ObjectId
> = R | T;

declare function isRefType<T, S extends RefType>(
  doc: Ref<T, S>,
): doc is S;

class Model {
  _id?: string; // change to ?
  name!: string;
}

class Parent {
  child!: Ref<Model>;
}

const x = new Parent();

isRefType(x.child) ? x.child.anObjectId : x.child._id;

Here's a Playground Link
Depending on whether _id is defined as nullable or required, the type of Child changes from Ref<Model, ObjectId> to Ref<Model, string>, and a compilation error appears below.
Something similar, but separate, happens depending on whether child has an ? or a ! after it.

Thinking about it, it seems that if a property is defined as string | undefined, then the extends check is false. Is there any way to work around this?

@nmain then why does it work (as expected by me) when strictNullCheckes is false?
(so that the type is test<string, number>(test<string, number> | undefined) and not string | number | undefined)

The way you've written getRefProperty you'll never get useful inferences for S

when strictNullCheckes is false it is fully working as it is written, and it is very useful, that is why i made an issue on why it changes with this option
-> the whole intention about getRefProperty is to automatically infer the types needed

then why does it work (as expected by me) when strictNullCheckes is false?

If the argument's type is test<A, B> for some A and B and that hasn't been expanded or simplified or modified at all yet, then tsc sees that it's trying to fit test<T, S> and does the obvious thing and infers T as A and S as B.

@nmain yes, but shouldnt when strictNullCheckes (that adds | undefined) is active result in test<A, B> | undefined where it still could resolve the function?

I don't see anything indicative of a bug here. Please file a new issue with a more explicit explanation of what's wrong if you'd like us to come back to it - "This isn't what I expected" is not really indicative of a defect without more context.

@RyanCavanaugh here's an example where behavior is different with strictNullChecks on or off:
Playground Link
Just toggle strictNullChecks on or off.
The curious part is that simply changing the type on the first line to add undefined to it:
export type ObjectId = { _id: string } | undefined;
Resolves the issue, but it's unclear why. So I don't think there's a typescript bug here, just non-intuitive behavior.

Nevermind, I understand the problem. More context here if someone runs into this in the future and needs help: https://github.com/typegoose/typegoose/pull/254#discussion_r422502220 :)
Issue can probably be closed.

thanks @andreialecu for explaining why this is happening, but i would have expected an error from typescript of "incompatible types" or something like that


python3Packages.jaxlib build failure on x86_64-linux as of f804cc93

Issue description
Build of python3Packages.jaxlib failed on x86_64-linux as of f804cc939d80186b72506f5b23fe9f1e4764a1f4. This is currently breaking python39Packages.optax.
error: builder for '/nix/store/bggdp2r2srpb9clsvrynns7z6a671h31-bazel-build-jaxlib-0.4.4.drv' failed with exit code 1;
       last 10 log lines:
       > nvcc warning : incompatible redefinition for option 'compiler-bindir', the last value of this option was used
       > <built-in>: fatal error: when writing output to /build/tmpxft_000086a2_00000000-6_min_u64_reduce.cu.cpp4.ii: No such file or directory
       > compilation terminated.
       > Target //build:build_wheel failed to build
       > Use --verbose_failures to see the command lines of failed build steps.
       > INFO: Elapsed time: 10834.876s, Critical Path: 149.18s
       > INFO: 9878 processes: 4479 internal, 5399 local.
       > FAILED: Build did NOT complete successfully
       > FAILED: Build did NOT complete successfully
       >
       For full logs, run 'nix log /nix/store/bggdp2r2srpb9clsvrynns7z6a671h31-bazel-build-jaxlib-0.4.4.drv'.
       note: build failure may have been caused by lack of free disk space
error: 1 dependencies of derivation '/nix/store/arh9i170kj76slwpng7349gli0cclb8v-python3.9-jaxlib-0.4.4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/2iw59qdviasl0yfd85ypsww9cixcfiai-python3.9-optax-0.1.5.drv' failed to build

full build log
python39Packages.optax maintainers cc: @ndl
python3Packages.jaxlib maintainers cc: @ndl
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit f804cc939d80186b72506f5b23fe9f1e4764a1f4
Run nix-build -A python3Packages.jaxlib

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-1041-azure, Ubuntu, 22.04.2 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.16.1`
 - nixpkgs: `/nix/store/hh5wjf6d3cc0hy322sqkl7xdlq2dfqd2-source`

Contents of ~/.config/nixpkgs/config.nix:
{ allowUnfree = true; cudaSupport = true; cudaCapabilities = ["8.0"]; }

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags:<PHONE_NUMBER>582032442<PHONE_NUMBER>097490469

builds fine for me


[External Resource]Join Your Company's Workspace

Publish Date: May 1st 2024
This will be published under Getting Started for New Expensify
Tracking GH: https://github.com/Expensify/Expensify/issues/391470
Draft: https://docs.google.com/document/d/1Q5EjAJvzFX6O-H0cMgIeR_S0gEamr5llHtb0b1FIuLM/edit
cc @ren-jones

https://docs.google.com/document/d/1Q5EjAJvzFX6O-H0cMgIeR_S0gEamr5llHtb0b1FIuLM/edit

I'm ready to be assigned!

Draft pushed!


object has no attribute 'set_devices'. How to fix this?

AttributeError: 'OPTModel' object has no attribute 'set_devices'

Well I think just comment out this line and use model = model.to(device) instead. Works for llama-2.


AMPI failure with migration under Cray compiler due to tcmalloc bugs or incompatibility

Original author: Celso Mendes
Original issue: https://charm.cs.illinois.edu/redmine/issues/78

We are facing errors in AMPI with a version of a Jacobi code using the Cray compiler. This Jacobi code is such that  there are periodic calls to MPI_Migrate(), which activate migration when used with RotateLB.
To provide an isolation of the problem, we created a smaller code that reproduces a minimum of the Jacobi behavior and still presents the same execution error as Jacobi. The pseudo-code for this example is just the following:
MPI_Init()
allocate array (~1 KB memory)
loop
write to array
MPI_Barrier()
MPI_Migrate()
MPI_Barrier()
end-loop
When executed on two JYC nodes, the code above has a seg-fault after one or two migrations. There are two versions of the code, one in Fortran, one in C, and they both fail on JYC if compiled by a version of AMPI generated with the Cray compiler. Meanwhile, when the program is compiled by an AMPI generated with GNU, the code works fine.
The programs with the C and Fortran examples are on JYC, with proper Makefiles, pbs scripts and examples of the produced outputs for the good and bad executions. This is under ~cmendes/BUG-AMPI-BUG-2/ with a C/ and a FORTRAN/ subdir for the C and Fortran versions, respectively. To use the Makefile, it is necessary to have the correct PrgEnv module loaded, e.g. to produce the version with the Cray compiler it's necessary to have PrgEnv-cray loaded. The build can be done as "make frotate-cray", for the Fortran case and the Cray compiler.
The compilation flags in the Makefile are needed for building the real Jacobi example, which is intended to be expanded in the future with use of GPUs (hence dynamic linking is needed). The Charm++ version in use was extracted from the GIT repository on March-6/2013, and the Cray compiler in use was cce/8.1.5 (current default on JYC). The pbs script sets variable ATP_ENABLED such that a stack trace is displayed upon a seg-fault.

Original date: 2013-03-27 19:03:52

.Seem like isomalloc and tcmalloc dont like each other. A hack for now is to install malloc outside charm, force them to use dl* names, and link them using dlsym. Nothing else seems to be working. Steps-

Download ptmalloc and install a threadsafe version with dl* function naming enabled.
Checkout isomalloc-cray-hack branch of Charm.
Compile you code with additional LDOPTS="-L/u/staff/nikhilj/installs/ptmalloc3 -lptmalloc3 -lpthread -Wl,-rpath=/u/staff/nikhilj/installs/ptmalloc3" with os-isomalloc.

Original date: 2013-04-02 00:55:04

Per https://bluewaters.ncsa.illinois.edu/known-issues tcmalloc is known to be problematic on BW:

TCMALLOC: Codes may crash in tcmalloc. Cray is working on fixing these. In the mean time, use the flag "-hsystem_alloc" to avoid linking in tcmalloc. Using this flag may result in a new set of problems. So use it with caution. Continue to open bugs on Jira if you see problems with tcmalloc.

So, it's not just us or isomalloc's problem.

Original date: 2013-04-05 19:51:36

What's the resolution? Will ampicc make sure that -hsystem_alloc gets passed until this is fixed? Or are we just documenting the issue and instructing users to pass it themselves?
Either way, until Cray deals with it, let's keep it open on our tracker (status Upstream) so that we can clean up the workaround once it's unnecessary.

Original date: 2013-06-03 19:47:42

Update subject so I needn't read through description and notes every time.

Original date: 2017-04-11 20:46:46

Charm/AMPI do actually compile on the Cray Compiler now (CCE 8.5.4+), so I will try it out at some point before the release.

Original date: 2017-05-02 21:05:00

This is very low priority since it is A) AMPI-specific and B) CrayCC-specific.

Original date: 2017-09-28 19:28:30

I don't think there's going to be urgency for fixing this any time soon, so pulling it off any release target.


Dx Capacity Exceeds - Error handling

Fixes #2344
Checks if the dx produced "too many classes in main-classes.txt" error, If yes then recreates main-classes.txt by excluding the user-created classes. Executes the dx task again with the replaced main-classes.txt. The existing execute methods are used by other processes, so there was a need to create a separate stream handler method that can print to stdin and return the output buffer to check the type of error.
Excluding user source classes in main-classes.txt will add them in secondary dex files. Since the necessary Appcompat, Runtime, etc. classes are in primary dex and loaded initially by Dalvikvm, It's okay to include user-created screens in the secondary dex file if in case the error occurs. The projects with dx error and regular projects are tested and APK files worked fine on devices.
Project .aia that produces similar dex error and handled by the functionality:
CheckDex2.zip

Can one of the admins verify this patch?

Can one of the admins verify this patch?

It seems to me a simpler solution would be to only ever include Screen1 in the main dex and allow all other screens to be put in a secondary dex. If that fails, nothing else will work anyway as Screen1 must be present. That would eliminate the need to process and interpret the output of dx. On Android 5+ everything is converted at install time to oat from dex anyway so multidex performance will mostly just affect devices running earlier versions of Android.

It seems to me a simpler solution would be to only ever include Screen1 in the main dex and allow all other screens to be put in a secondary dex. If that fails, nothing else will work anyway as Screen1 must be present. That would eliminate the need to process and interpret the output of dx. On Android 5+ everything is converted at install time to oat from dex anyway so multidex performance will mostly just affect devices running earlier versions of Android.

@ewpatton you are right, I think Screen1 must be included in primary dex file, I will also ensure the working on earlier versions of Android devices. Thanks for correcting, Please don't merge it for now.

@ewpatton you are right, I think Screen1 must be included in primary dex file, I will also ensure the working on earlier versions of Android devices. Thanks for correcting, Please don't merge it for now.

@rupesh-kumar-lpu I've marked this as a draft for the time being until you are ready for it to be reviewed again.

@rupesh-kumar-lpu I've marked this as a draft for the time being until you are ready for it to be reviewed again.

We are no longer using dx as we have switched to d8 in the Java 11 update. This change is no longer needed.


fix(edge-identity/fs-override): fix enabled not being updated correctly

Depends on https://github.com/Flagsmith/flagsmith-engine/pull/119


Instrumented code for code coverage is not getting parsed correclty

#547 addresses a related issue.
(other related: https://github.com/aurelia/testing/issues/94)
Problem
What I found, is that the instrumented code (maybe special to this lib @jsdevtools/coverage-istanbul-loader) is using methods instead of simple properties

:warning: There is a bug when parsing code, that has coverage-helper-code.
More specifically, aurelia-validation does not parse the helper code correctly https://github.com/aurelia/testing/issues/94.
I found a fix, but would need to PR to the aurelia repo and stuff.
(Maybe we just fork ourselves, will create a ticket)

function (a) {
-    cov_2n0ovlwwxg.f[2]++;    // "normally"
+    cov_2n0ovlwwxg().f[2]++;   // maybe only @jsdevtools/coverage-istanbul-loader?
    return a.message;
}

Suggested fix
A fix I found is to update the parts of the regex by including () (open and close paren)
export function getAccessorExpression(fn: string): string {
  /* tslint:disable:max-line-length */
-  const classic = /^function\s*\([$_\w\d]+\)\s*\{(?:\s*"use strict";)?(?:[$_\s\w\d\/\*.['"\]+;]+)?\s*return\s+[$_\w\d]+\.([$_\w\d]+)\s*;?\s*\}$/;
+  const classic = /^function\s*\([$_\w\d]+\)\s*\{(?:\s*"use strict";)?(?:[$_\s\w\d\/\*.['"\]+();]+)?\s*return\s+[$_\w\d]+\.([$_\w\d]+)\s*;?\s*\}$/;
                                                                                ^ actual change here

Keywords:
@jsdevtools/coverage-istanbul-loader
code coverage instrumented code

@hiaux0 there @hiaux0 there are existing tests here https://github.com/aurelia/validation/blob/862da07647f653036e4e2b29d4318d7fa8fec698/test/property-accessor-parser.ts#L47-L54
It seems the fix is already figured out, can you help create a PR with some tests @hiaux0 ?

@bigopon

existing tests

Afai can see, it fixes just for normal properties, and not methods, ie.
- __gen$field.f[\'10\']
+ __gen$field().f[\'10\'] // I'm expecting `()`

create a PR

yeah, I planned to when I find some spare time