Change sig of fn ScopeAttr::apply: replace two args with attr: Attribute

This allows better error messages when missing content at
end of input: https://github.com/dtolnay/syn/issues/1206
Also: remove fn parse_attr_group

Note: there is an unresolved question whether syn::Attribute::parse_args should support attributes with no args like #[my_attr]. If syn changes in this respect then the code here can be simplified slightly.


Multimap cache - not correct eviction/update.

Clearly the following is not correct :
CacheMultiMap is : '{
"vertx-consul" : {
"ids" : [ ],
"empty" : true
}
}'

addressed in https://github.com/romalev/vertx-consul-cluster-manager/pull/57.


TabletFriend with Cintiq

I love TabletFriend- thank you. I use it with artist software (Rebelle 6 right now) on a 3 monitor setup including an older Cintiq 22HD (with no touch screen). The Wacom 'on-screen shortcuts' software is awful so I went hunting and found TabletFriend. It's exactly what I needed. I also use a 'Tourbox' controller with my non-dominant hand.
Since I stab and poke the buttons on TabletFriend with my stylus it sometimes moves and jumps. It would be great if I could lock it down - as you can see in the pictures I keep it inside the workspace of the software, not docked outside.
That's really my only issue. Other ideas that would be helpful for my use case would be to have layouts linked to applications. (For example the layouts in my Tourbox controller software can seamlessly switch from Rebelle to Krita and back again.)
The ability to have multiple instances open at the same time would be useful.
Also allowing a second text field for tooltips if the button features text (no icon). Most of my small buttons have truncated and abbreviated labels which would benefit from a tooltip to remind me what they are. I'll be making layouts for several other software programs- Blender, Krita, etc- I can see this becoming an issue.
I have no coding experience, but I found modifying the layout enjoyable and as easy to 'get the hang of' as you described.
Screenshots of my Rebelle layouts, adapted from the default and default-mini you provided. Note how they blend in with the Rebelle tool set:

That's really my only issue. Other ideas that would be helpful for my use case would be to have layouts linked to applications. (For example the layouts in my Tourbox controller software can seamlessly switch from Rebelle to Krita and back again.)

I have already implemented this feature in develop branch,  will include it in an upcoming 2.0 release. As for the rest, would be cool, but not essential. Maybe in the future.


unknown flag error

🐛 Bug description

Followed hocus quick start guide, but got that error than trying to run on my linux machine
namai@parrot ~/hocus (main)
$ HOCUS_HOSTNAME="localhost" ops/bin/local-up.sh
Pulling docker images 📥 - ❌ in 1.23
unknown flag: --ignore-buildable

Above you will find the logs
Please report this problem here
🙏🙏🙏  https://github.com/hocus-dev/hocus/issues/new/choose  🙏🙏🙏

Environment

Hocus version (commit hash) latest ca0133326b445c1143ab622a46da451c4503737e
Parrot OS 5.3 (Electro Ara) Kernel 6.1.0-1parrot1-amd64
Any other environment information that may help

See this issue: https://github.com/hocus-dev/hocus/issues/11
Running sudo apt install --only-upgrade docker-compose-plugin may fix your issue.

It might be a good idea for the ops/bin/local-up.sh script to check the version of Docker compose, rather than just if it is installed.
https://github.com/hocus-dev/hocus/blob/main/ops/bin/local-up.sh#L76

It appears that --ignore-buildable was added with v2.15.0, released Jan. 5th: https://github.com/docker/compose/releases/tag/v2.15.0

A possible solution:
# Check minimum version of docker compose
# We require docker compose of at least version 2.15.0 which introduced --ignore-buildable
COMPOSE_VERSION="$(docker compose version | grep -o -E '[0-9]+(\.[0-9]+){2}$')"
COMPOSE_REQUIRED_VERSION="2.15.0"
if ! [ "$(printf '%s\n' "$COMPOSE_REQUIRED_VERSION" "$COMPOSE_VERSION" | sort -V | head -n1)" = "$COMPOSE_REQUIRED_VERSION" ]; then
  echo "Less than ${COMPOSE_REQUIRED_VERSION}"
fi

@regner Thank you for suggesting a fix! --ignore-buildable  is only needed when building the containers locally. After #120 got  merged that flag is not needed when pulling the agent/ui/setup images from quay. Will make a PR today which will remove the flag when using prebuild images, otherwise it will do a docker-compose version check.


rafi fix

please rafael.

"Berlin is the testicles of the West" - nikie kruschev

it is important they learn

dankeschon. please add germany to history

brie halloumi edam sarró


Attempted import error: 'AccountEntity' is not exported from '@azure/msal-common'.

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
3.24.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
2.1.0
Public or Confidential Client?
Public
Description
I have a React app (with create react app) with the following versions installed:
 "dependencies": {
    "@azure/msal-browser": "3.22.0",
    "@azure/msal-react": "2.0.22",
    "react": "17.0.2",
    "typescript": "4.4.3",
  },

It turns out that when importing the PublicClientApplication object the application tells me "Attempted import error: 'AccountEntity' is not exported from '@azure/msal-common'" i try to use differents versions but it's the same.
import "./App.scss";
import { Provider } from "react-redux";
import "bootstrap/dist/css/bootstrap.css";
import { STORE as store, PERSISTOR } from "./store/redux";
import { PersistGate } from "redux-persist/integration/react";
import MainComponent from "./MainComponent";
import "./constants/global.css";
import "./components/pages/styles/Mainpage.css";
import "./constants/responsive.css";
import "./constants/variables.css";
import { setInterceptors } from "./frameworks_and_drivers/Axios";
import { SnackbarProvider } from "notistack";
import { FilterContextProvider } from "./context";
import { Configuration, PublicClientApplication } from "@azure/msal-browser";

setInterceptors();

// MSAL configuration
const configuration: Configuration = {
  auth: {
    clientId: "clientID",
    authority: "authority",
    redirectUri: "redirectUri",
  },
};

const pca = new PublicClientApplication(configuration);

console.log(pca);
function App() {
  return (
    <Provider store={store}>
      <PersistGate persistor={PERSISTOR}>
        <SnackbarProvider maxSnack={3}>
          <FilterContextProvider>
            <MainComponent />
          </FilterContextProvider>
        </SnackbarProvider>
      </PersistGate>
    </Provider>
  );
}
export default App;

Error Message
<EMAIL_ADDRESS>Attempted import error: 'AccountEntity' is not exported from '@azure/msal-common'.
MSAL Logs
i can't get it because the const msalInstance = new PublicClientApplication(msalConfig);it self is not working
Network Trace (Preferrably Fiddler)

[ ] Sent
[X] Pending

MSAL Configuration
{
  auth: {
    clientId: "clientID",
    authority: "authority",
    redirectUri: "redirectUri",
  },
}

Relevant Code Snippets
All appears on app or main index files when import and use a new instance for PublicClientApplication

Reproduction Steps

Create react app
install @azure/msal-browser and @azure/msal-react
Make a new instance for PublicClientApplication
Then the error apprears

Expected Behavior
when i make a new instance for PublicClientApplication i can follow with the configuration of the MsalProvider on msal for react putting the msal config
Identity Provider
Entra ID (formerly Azure AD) / MSA
Browsers Affected (Select all that apply)
Chrome, None (Server)
Regression
No response
Source
External (Customer)

any update on this, or a version combination that can be used instead ?

I had same issue also and managed to fix it by installing "@azure/msal-common": "^14.16.0"

@DryJhin
Thank you for your support!
I solved this problem with your configuration.


[QUERY] How do we create managed disk with hibernation enabled

Query/Question
Creating Vm with hibernation capability returned exception saying "The referenced os disk should support hibernation for a VM with hibernation capability".
Azure team said hibernation should be enabled on OS disk, they gave this link.
In SDK, i couldn't find parameter to enable hibernation while creating disk. How do I create disk with hibernation enabled?
Disk osDisk = azureInstance.disks().define(diskName).withRegion(region)
.withExistingResourceGroup(rg).withWindowsFromVhd(image)
.withStorageAccountName(vhdStorageAccountName).withSizeInGB(size).withSku(diskSkuType)
.create();
Why is this not a Bug or a feature Request?
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: azure-resourcemanager:2.14.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@weidongxu-microsoft could you please follow up with @Prudvidhard on their question?

@XiaofeiCao Does disk have option for hibernate?
Code for vendor provider:
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/azure-resourcemanager-compute/src/test/java/com/azure/resourcemanager/compute/VirtualMachineOperationsTests.java#L1199-L1240

This worked for me. After disk created, I updated the disk with hibernation support. Creating VM with hibernation capability succeeded with this disk.
Disk osDisk = azureInstance.disks().define(diskName).withRegion(region)
.withExistingResourceGroup(rg).withWindowsFromVhd(image)
.withStorageAccountName(vhdStorageAccountName).withSizeInGB(size).withSku(diskSkuType)
.create();
Disk.Update osDiskUpdating = osDisk.update();
osDisk.innerModel().withSupportsHibernation(true);
osDisk = osDiskUpdating.apply();

Thanks @Prudvidhard
@XiaofeiCao Please support it at convenience API.

OK, will support it in our next release.

@Prudvidhard 2.22.0 has been released and disk hibernation support has been added.


Funkin.lua

Describe your bug here.
in sources and funkin lua when i compile it say
E:\JEUX\fnf-grafex-raw>lime test windows
source/FunkinLua.hx:2613: characters 7-36 : Bool should be Int
source/FunkinLua.hx:367: lines 367-371 : Int should be Bool
source/FunkinLua.hx:367: lines 367-371 : ... For function argument 'f'
source/FunkinLua.hx:473: lines 473-476 : Int should be Bool
source/FunkinLua.hx:473: lines 473-476 : ... For function argument 'f'
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

You need to update lua jit, look for info in psych repository

thanks its working but the arrow are like this and invisibl

e

Is this only on week7?

no all week i'll send you the bin ? and maybe all the files

?

All that added  and fixed.

THANKS ITS WORKING i love your engine its very cool

you even sorted it's much better


Dev Home tools API is missing (docs/interface.md)

Provide a description of requested docs changes
From https://github.com/microsoft/devhome/blob/main/docs/tools.md
"Create your XAML view and viewmodel. Your view class must inherit from ToolPage and implement requirements. Specifications for the Dev Home tools API`
"
the link to Dev Home tools API is missing (docs/interface.md)

I was following the docs as an intro to devhome and all make perfect sense until I got to the broken link above at which point, I couldn't follow the journey any further, if you are interested here is my notes on my initial onboarding experience.
https://github.com/colhountech/dev-home-notes


Cannot broadcast input array to size (1)

Hello -
I cannot seem to figure out why the code is throwing this exception:
ERROR: Caught exception in do_training(): could not broadcast input array from shape (37,37,37) into shape (1)
partial traceback:
2023-6-30 16:33:49.48: [TRA|SAMPLER|PID:2552997] :=:=:=:=:=:= Finished sampling for next [Training] =:=:=:=:=:=:
2023-6-30 16:33:49.52:

 ERROR: Caught exception in do_training(): could not broadcast input array from shape (37,37,37) into shape (1)

2023-6-30 16:33:49.53: Traceback (most recent call last):
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/software/DeepMedicPlus/DeepMedicPlus/deepmedic/routines/training.py", line 332, in do_training
    lbls_samples_per_path_tr) = sampling_job_tr.get()
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/conda_env/deepmedic/lib/python3.6/multiprocessing/pool.py", line 644, in get
    raise self._value
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/conda_env/deepmedic/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/software/DeepMedicPlus/DeepMedicPlus/deepmedic/dataManagement/sampling.py", line 198, in get_samples_for_subepoch
    channs_of_samples_for_path in channs_of_samples_per_path]
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/software/DeepMedicPlus/DeepMedicPlus/deepmedic/dataManagement/sampling.py", line 198, in <listcomp>
    channs_of_samples_for_path in channs_of_samples_per_path]
  File "/projects/site/pred/oncology_imaging/cluceruj/brain_mets/conda_env/deepmedic/lib/python3.6/site-packages/numpy/core/_asarray.py", line 83, in asarray
    return array(a, dtype, copy=False, order=order)
ValueError: could not broadcast input array from shape (37,37,37) into shape (1)

Can you please help?

Will this issue from DeepMedic repository help you?

I got it - it was that i was not using prior images that were registered to the current image, causing a shape issue.


Carousel not scrolling in Android when put inside a ScrollView but works in IOS.

Is this a bug report, a feature request, or a question?
bug report

Have you followed the required steps before opening a bug report?
(Check the step you've followed - put an x character between the square brackets ([]).)

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
yes

Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Tested on Android Version 9 pie
Is the bug reproductible in a production environment (not a debug one)?
yes

Environment

Environment:
React: 16.9.0
React native: 0.61.4
react-native-snap-carousel: ^3.8.4
Target Platform:
Android (9)
Expected Behavior

carousel should scroll when inside ScrollView
Actual Behavior

Carousel not scrolling in Android when put inside a ScrollView but works in IOS.
Reproducible Demo
here is expo snack to reproduce the issue, (Please run it in your android phone)
https://snack.expo.io/@sagar293/flatlist-inside-scrollview

Steps to Reproduce

(Write your steps so that anyone can reproduce the issue in the Snack demo you provided.)

Create a ScrollView in that render the card as shown in the expo snack
inside the card create the "Carousel" from "react-native-snap-carousel" to render the images
run it in your Android device

I am facing the exact same issue. Let me know if you have found the solution yet.

As a quick guess, this could be related to RN's Bug on Android concerning nested horizontal ScrollViews.
As a workaround, try to use the ScrollView Component of react-native-gesture-handler.
See https://github.com/facebook/react-native/issues/21436

same issue +1.

Confirmed workaround from @smm76 . I simply swapped the import and things started working on Android. Thanks so much!

+1 Confirmed workaround from @smm76

As a quick guess, this could be related to RN's Bug on Android concerning nested horizontal ScrollViews.
As a workaround, try to use the ScrollView Component of react-native-gesture-handler.
See https://github.com/facebook/react-native/issues/21436

I have the same issue  but in a bit more complex context:
A horizontal carousel nested inside a vertical ScrollView that is essentially a horizontal Flatlist item. Carousel doesn't work on Android device, works only on iOS.
Using react-native-gesture-handler's ScrollView didn't help.
I also tried the following approach as it's related, but didn't work neither:
How can I have nested horizontal ScrollViews in React Native?
smm76 @matrak-jason @sergey-king do you guys have any new information on this?

As a quick guess, this could be related to RN's Bug on Android concerning nested horizontal ScrollViews.
As a workaround, try to use the ScrollView Component of react-native-gesture-handler.
See https://github.com/facebook/react-native/issues/21436

Saved my life, I have snap carousel -> scrollview (vertical) -> scrollview(horizontal)
Problem was not working the horizontal scroll in android,
As you mentioned, I used the scrollview of react-native-gesture-handler
Working for me as well. thanks

As a quick guess, this could be related to RN's Bug on Android concerning nested horizontal ScrollViews.
As a workaround, try to use the ScrollView Component of react-native-gesture-handler.
See https://github.com/facebook/react-native/issues/21436

Thanks this worked for me

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2

This helped me
https://github.com/gorhom/react-native-bottom-sheet/issues/311#issuecomment-794418413

Thank you alexandermirzoyan! Using
<NativeViewGestureHandler disallowInterruption={true}>
   {...}
</NativeViewGestureHandler>

worked good for me in Android.
Here is the source again: https://github.com/gorhom/react-native-bottom-sheet/issues/311#issuecomment-794418413


read marks not displayed

I assume this was a faulty update? approx 3-7 days ago
None of my Livemarks have read checks anymore. Clicking on the links again does not make new ones appear.

They're back now :)


DBZ-7153 in memory cache to speed up infinispan

Already discussed with @Naros.
I tested it on our environment, including the cache reload from ISPN data during restart

There are two test failures attributed to this change:
[ERROR]   OracleConnectorIT.shouldNotEmitEventsOnConstraintViolations:2272
Expecting actual not to be null
[ERROR]   OracleConnectorIT.shouldUndoOnlyLastEventWithSavepoint:4668
Expecting actual not to be null

Applied to main (=2.5), thanks @sherpa003


Specific updates to Cheat Sheet documentation source file

Task management info
Relevant branches:

sam-edits-2, for Sam's edits
tj-doc-edits for TJ's edits

These two branches are starting off identical and represent the documentation deployed to GitHub Pages here.
Some notes on revising workflow

Please do not edit the doc-updates-fy23q1 branch directly. This will be where we merge together our individual edits once this round of revisions is done.
I've assigned myself to several of the to-do items below; feel free to assign me to other items you'd like me to address, and assign yourself to anything you'll be addressing.
When revising, you can either edit the source files directly and use this issue or your personal notes to keep track of to-do items, or create a pull request into doc-updates-fy23q1 for easier line-by-line commenting (you'll need to make a small commit first, so the branches aren't identical). If you'd prefer you can also put comments on the PDF documentation  (linked just below) for later implementation. Try to keep meta-notes out of the source files themselves.
Tyche.pdf
When you are done with this round of revisions, create a Pull Request merging your personal branch into doc-updates-fy23q1 and request at least one review.

Some notes on formatting

Do not delete any of the cross-references used to refer to Tables and Figures such as the example below. Using these cross-references instead of plaintext "Table X" lets us automatically keep table and figure numbering consistent across all sections of the documentation.

:numref:`tbl-designsvars`

Do not delete any of the table/figure headers either, such as the example below. These enable cross-referencing and become captions when the docs are compiled.

.. _tbl-designsdict:
.. table:: Data dictionary for the *designs* dataset which defines various technology versions resulting from R&D investments.

When working inside a doc source file, the only header marked with equals signs should be the one at the very top of the file. Headers marked with equals signs go into the main documentation table of contents.

[x] # To do items

[x] ## On line 205, re the mandatory Variables within the designs dataset:

[x] I  am unable to create the designs table. These are the questions that I am faced with when creating the designs table that are not being answered by the Cheat sheet

[x] -  I do not have any input output efficiency, lifetime, scale etc. Then should I put it as blank or None ??

[x] -  How to put in irrelevant information or not required information in the different columns

[x] ## On line 224, re the parameters dataset

[x] Questions and issues

[x] - I do not have any parameters What do I do with parameters file

[x] - I created an empty file and it failed.

Variable location, re the technology model (.py) file

[ ] Move this section further up, close to the section on the Jupyter technology analysis notebook
[ ] <Give that Numpy documentation is quite long (reference is 2000 pages; user manual is 500 pages), this is not very useful.  Need to briefly explain how these stacks are set up, how they are used, and why vectorization is so powerful here.>

Response: The link is specifically to numpy.stack rather than the entire Numpy documentation. The page explains what a stack is and provides several examples of how to use them; examples are also in the technology models already developed. A discussion on vectorization is better suited to the Approach section.

On line 329, re the indices dataset
ISSUE

[x] I  am unable to create the indices table. These are the questions that I am faced with when creating the indices table that are not being answered by the Cheat sheet
[ ]
[x] Questions -
[x] - When I put in a type of capital, whose capital do we put in the index
[x] - I do not have any input or metric. Then should I put it as blank or None
[x] - How to put in irrelevant information or not required information in the different columns
[x] - Elements of Capital have an associated Lifetime. Does this mean that the elements in the Lifetime variable of designs dataset should be put in the indices table

On line 393, re the results dataset

[x] ISSUE WITH MAKING RESULTS
[x] **What is mandatory what is not? I HAVE some ISSUEs while making the Results dataset.
[x] Are the cost,outputs, metrics mandatory**
[x] ** I am unable to understand how to fill up the index column for the different variable types. I have never described an index for cost, nor for output, neither for metric. Although I do know what metrics I defined in my technology model. **

Section to add: Modeling in Tyche

RJH: Before we implement this, let's review our Technology Model Example section to see if we can expand it to fit this need. Not all of the datasets are discussed there, but they could be.
Assign to: @rjhanes

Example case study 1- Consider a PV system. It consists of a set of PV modules that convert sunlight into direct current power, an inverter that transforms the dc of the PV module into ac for the power line, and all the supporting infrastructure and installation costs.  This system We will also use this example to explain this tutorial.  This is a typical example which showcases Tyche usage.
The technology model describes a process physically and economically (A classic but simple and generalized Techno-economic analysis)

Converts inputs to outputs based on user defined technology model and ad-hoc parameters.
The user defined model can integrate uncertainty in the input variables and parameters in the form of distribution. (For example, triangular distributions can be put as inputs for parameters or variables to incorporate uncertainty)
Requirement and data location for technology model is explained in Table 1.

DESCRIBE the PV or other system here in detail and how to modify it to fit the user’s needs.  This should be a large section to include the description of all the steps needed….
Section to add: Jupyter Technology Analysis Notebook
Assign to: @rjhanes
RJH: I think this will be difficult to understand unless we first walk through what the technology model is and what the input datasets cover. We can discuss at a high level what goes on in the analysis notebooks, but specifics will be difficult without the context.

Describe Jupyter model and what it does: The Jupyter notebooks are used once a technology model and input datasets have been developed. The notebook provides a convenient interface to the analysis methods built into the Tyche codebase, and can be used for ensemble evaluations, optimization studies, visualization, and exploratory analyses.
Describe Python model and what it does, bringing up to here the discussion from below: The technology model itself is implemented in Python and consists of a set of methods (functions) to calculate a technology's capital costs, output amount(s), and metrics of interest.
Then describe the supporting data sets below

Section to add: To begin the quick start guide, it needs to explain what Tyche is, what it does, and why this is of value to the user. Expand this to a full explanation

RJH: I strongly suggest we leave this discussion on the main page of the documentation, where it is currently. We cannot make this "cheat sheet" completely standalone and have it remain a "cheat sheet" instead of the entire documentation.

Wording change: "Module" to "Model" in functions dataset
Assign to: @rjhanes

See issue #88 and PR #108 . Anywhere there's a functions dataset example or discussion of the column names, update "Module" to "Model". That way, anytime we say "technology model", we're always referring to the .py file with the capital cost, etc methods.
Since this is a pretty minor change but could involve multiple source files, I'll do this after other revisions have been merged together but before the next version is deployed.

Add to installation instructions

Tyche isn't supported on Linux systems (nor will it be)

Rebecca:
Many thanks for all your work on this.  I look forward to talking on
Wednesday.
All the best,
sam
On Fri, Sep 30, 2022 at 7:22 PM Rebecca Hanes @.***>
wrote:

Task management info
Relevant branches:

sam-edits-2, for Sam's edits
tj-doc-edits for TJ's edits

These two branches are starting off identical and represent the
documentation deployed to GitHub Pages here
https://nrel.github.io/tyche-docs/index.html.
Some notes on revising workflow

Please do not edit the doc-updates-fy23q1 branch directly. This
will be where we merge together our individual edits once this round of
revisions is done.
I've assigned myself to several of the to-do items below; feel free
to assign me to other items you'd like me to address, and assign yourself
to anything you'll be addressing.
When revising, you can either edit the source files directly and use
this issue or your personal notes to keep track of to-do items, or create a
pull request into doc-updates-fy23q1 for easier line-by-line
commenting (you'll need to make a small commit first, so the branches
aren't identical). If you'd prefer you can also put comments on the PDF
documentation (linked just below) for later implementation. Try to keep
meta-notes out of the source files themselves.
Tyche.pdf https://github.com/NREL/tyche/files/9688101/Tyche.pdf
When you are done with this round of revisions, create a Pull
Request merging your personal branch into doc-updates-fy23q1 and
request at least one review.

Some notes on formatting

Do not delete any of the cross-references used to refer to Tables
and Figures such as the example below. Using these cross-references instead
of plaintext "Table X" lets us automatically keep table and figure
numbering consistent across all sections of the documentation.

:numref:tbl-designsvars

Do not delete any of the table/figure headers either, such as the
example below. These enable cross-referencing and become captions when the
docs are compiled.

.. _tbl-designsdict:
.. table:: Data dictionary for the designs dataset which defines various technology versions resulting from R&D investments.

When working inside a doc source file, the only header marked with
equals signs should be the one at the very top of the file. Headers marked
with equals signs go into the main documentation table of contents.

To do items On line 205, re the mandatory Variables within the designs
dataset:
I am unable to create the designs table. These are the questions that I
am faced with when creating the designs table that are not being answered
by the Cheat sheet

I do not have any input output efficiency, lifetime, scale etc.
Then should I put it as blank or None ??
How to put in irrelevant information or not required information in
the different columns

On line 224, re the parameters dataset
QUestions and issues

I do not have any parameters What do I do with parameters file
I created an empty file and it failed.

Variable location, re the technology model (.py) file

Move this section further up, close to the section on the Jupyter
technology analysis notebook
<Give that Numpy documentation is quite long (reference is 2000
pages; user manual is 500 pages), this is not very useful. Need to briefly
explain how these stacks are set up, how they are used, and why
vectorization is so powerful here.>

Response: The link is specifically to numpy.stack rather than the
entire Numpy documentation. The page explains what a stack is and provides
several examples of how to use them; examples are also in the technology
models already developed. A discussion on vectorization is better suited to
the Approach section.

On line 329, re the indices dataset
ISSUE
I am unable to create the indices table. These are the questions that I
am faced with when creating the indices table that are not being answered
by the Cheat sheet
Questions -

When I put in a type of capital, whose capital do we put in the
index
I do not have any input or metric. Then should I put it as blank or
None
How to put in irrelevant information or not required information in
the different columns
Elements of Capital have an associated Lifetime. Does this mean
that the elements in the Lifetime variable of designs dataset should be put
in the indices table

On line 393, re the results dataset
ISSUE WITH MAKING RESULTS
What is mandatory what is not? I HAVE some ISSUEs while making the
Results dataset. Are the cost,outputs, metrics mandatory
** I am unable to understand how to fill up the index column for the
different variable types. I have never described an index for cost, nor for
output, neither for metric. Although I do know what metrics I defined in my
technology model. **
Section to add: Modeling in Tyche

RJH: Before we implement this, let's review our Technology Model
Example https://nrel.github.io/tyche-docs/example-technology.html
section to see if we can expand it to fit this need. Not all of the
datasets are discussed there, but they could be.
Assign to: @rjhanes https://github.com/rjhanes

Example case study 1- Consider a PV system. It consists of a set of PV
modules that convert sunlight into direct current power, an inverter that
transforms the dc of the PV module into ac for the power line, and all the
supporting infrastructure and installation costs. This system We will also
use this example to explain this tutorial. This is a typical example which
showcases Tyche usage.
The technology model describes a process physically and economically (A
classic but simple and generalized Techno-economic analysis)

Converts inputs to outputs based on user defined technology model
and ad-hoc parameters.
The user defined model can integrate uncertainty in the input
variables and parameters in the form of distribution. (For example,
triangular distributions can be put as inputs for parameters or variables
to incorporate uncertainty)
Requirement and data location for technology model is explained in
Table 1.

DESCRIBE the PV or other system here in detail and how to modify it to fit
the user’s needs. This should be a large section to include the description
of all the steps needed….
Section to add: Jupyter Technology Analysis Notebook
Assign to: @rjhanes https://github.com/rjhanes
RJH: I think this will be difficult to understand unless we first walk
through what the technology model is and what the input datasets cover. We
can discuss at a high level what goes on in the analysis notebooks, but
specifics will be difficult without the context.

Describe Jupyter model and what it does: The Jupyter notebooks are
used once a technology model and input datasets have been developed. The
notebook provides a convenient interface to the analysis methods built into
the Tyche codebase, and can be used for ensemble evaluations, optimization
studies, visualization, and exploratory analyses.
Describe Python model and what it does, bringing up to here the
discussion from below: The technology model itself is implemented in
Python and consists of a set of methods (functions) to calculate a
technology's capital costs, output amount(s), and metrics of interest.
Then describe the supporting data sets below

Section to add: To begin the quick start guide, it needs to explain what
Tyche is, what it does, and why this is of value to the user. Expand this
to a full explanation

RJH: I strongly suggest we leave this discussion on the main page of
the documentation, where it is currently. We cannot make this "cheat sheet"
completely standalone and have it remain a "cheat sheet" instead of the
entire documentation.

—
Reply to this email directly, view it on GitHub
https://github.com/NREL/tyche/issues/105, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK2OV5ZETVGMFHSVUDTE2N3WA5Y5HANCNFSM6AAAAAAQ2FYNHM
.
You are receiving this because you were assigned.Message ID:
@.***>

Thank you for arranging all this nicely

@rjhanes  Hi Rebecca, I have completed my edits of the Cheat sheet. I was able to successfully address all the issues that may arise where building a model from scratch and I added some minor nuances of building the datasets. I have one remaining doubt. That is filling up the Investment column in the Investment dataset. I think the current description in the table is not super clear about what to put in the column. The tranches and categories are fine. But the investment column description seems a bit odd. I have put in a small paragraph after the investment dataset discussion with a bold heading Relationship between Designs, Parameters, Tranches and Investment.
In this paragraph,  I was thinking of writing this line - A scenario has a one -to- one relationship with a Tranche. I am wondering if this is true? Also linking Tranche to the Investment column, can we add a better description to fill in the Investment column info?
I think the category column description is fine although its not really clear that a single category can extend over multiplte tranches.

The Scenario/Tranche/Investment relationship is pretty flexible. I think the intended (but not required) use case is for a Scenario to consist of multiple Tranches that each define the investment level in a Category, and for Scenarios to be defined based on intended outcome (ie Fast Progress, Slow Progress). An Investment is also intended to be a grouping of Tranches but is defined based on the dollar amount investment (ie No R&D Investment, Moderate R&D Investment). Having these two ways of grouping Tranches lets us quickly do a couple different kinds of analyses without resorting to optimization. However, as you point out a Scenario or an Investment can also be defined to consist of just one Tranche (investment into just one Category), which I'm pretty sure will be much less common when we're applying Tyche to IRL applications. How Scenarios and Investments are defined will be determined as part of the decision context specification, with input from the decision maker; I think it's also likely that Scenario and Investment definitions will be heavily iterated upon in real applications.
I'm not sure what you mean by "a single Category can extend over multiple Tranches" - a Tranche is a unit of investment, and so one Category (category = research area) can have several Tranche options such as Low, Medium, and High Investment. The important thing is that Tranches for a single Category are mutually exclusive: it's not possible to have both Low and Medium investment levels into a Category. Tranches can be combined across multiple Categories to construct Scenarios and Investments.
Once your edits are wrapped up, create a PR back into doc-updates-fy23q1 and request my review. Once it's ready I'll also update Sam's doc branch with these revisions.

Closed for FY23 Q1 updates to the cheat sheet - see issue #107 for more general updates


Scrollbar doesn't react to window resizing

If you resize the window (and the container resizes as a result), the scrollbar's length remains what it was before until you drag it.
I know you can fix it by using Ps.update, but seems like this should be an out-of-the-box feature

Closed because I've realised there is a prior issue.


perform login on request

how can i implement login at some specific point, the current implementation forces the user to login as soon as you launch the app or open the home page, what if the home page isn't protected page is login a mandate there. How to avoid this.

If you are using react router, you can define which pages are public and which pages will be protected. Take a look https://reacttraining.com/react-router/web/example/auth-workflow

I have already done that as per the readme you have provided an hoc for the same. But the problem is in index.ts we create the authentication context so as soon as app loads it redirects to the login page which I do not want, it should preferably redirect only on protected pages rather than just making the user login on first load.

In the index.ts example you have a constant variable which is a Boolean if I try changing that value it never asks for login even protected pages are accessible without login than.

@mdmoin7  were you able to make this work ? If so can you please share it. I'm facing the same problem.


Create btrfs directory if it does not exist

We may want to create this as a subvolume if the parent is btrfs
or support mounting on load in the future.
Ping @cyphar, if you have any thoughts,  right now it is assuming the parent is btrfs but we can create a follow up to do something better here.

Codecov Report

Merging #1168 into master will decrease coverage by 0.29%.
The diff coverage is 12.5%.

@@            Coverage Diff            @@
##           master    #1168     +/-   ##
=========================================
- Coverage   31.85%   31.56%   -0.3%
=========================================
  Files          27       27
  Lines        2163     2183     +20
=========================================
  Hits          689      689
- Misses       1326     1345     +19
- Partials      148      149      +1

Impacted Files
Coverage Δ

snapshot/btrfs/btrfs.go
51.68% <12.5%> (-1.5%)
:arrow_down:

task.go
0% <0%> (ø)
:arrow_up:

process.go
0% <0%> (ø)
:arrow_up:

io.go
0% <0%> (ø)
:arrow_up:

container.go
0% <0%> (ø)
:arrow_up:

io_unix.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

LGTM
Not perfect, but it'll do. :pig:


Support multiple linters per file/language

LinterManager should allow more than one linter to be registered per language, and should loop through all linters registered for a language and report errors. I am ready to work on the UI part of this implementation (making the inline widgets and panel support multiple linters) - but I am not sure how to do the LinterManager part (I have not worked much with the plugins or workers).
This is the remainder of #33, so I thought we should move it into its own issue.
This is the main thing I'm waiting on before work on #52.

I'd like to bump this one, I'm trying to use JSCS and JSHint together. I'm very happy to do testing from a branch if it helps.
I assume the config in brackets.json should look like this once implemented.
"interactive-linter.javascript": ["jshint", "jscs"]


changelog: Add entry for S3 nginx proxying.

Fixes: https://github.com/zulip/zulip/pull/23713#discussion_r1063803234

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @alexmv!


Add IDA to the common interface

This PR implements the necessary algorithm and solve function to have IDA match the common interface for DAEs. The DAE solve function is similar to the ODE solve one, but there are some distinct differences in variable ordering in some of the IDA vs CVode functions which makes it too difficult to put them together (and there are some different checks that need to be done, etc.) so it's better to keep them separate. This will require the newest DiffEqBase tag.

Tests won't pass until the newest ecosystem tag goes through: https://github.com/JuliaLang/METADATA.jl/pull/7081

The post-tag Travis passed, but started to re-run only because of abs -> abs. so I think that's safe to pull it in.


[Breaking change]: BinaryFormatter disabled across almost all project types by default

Description
Beginning with .NET 8 Preview 4, the BinaryFormatter.Serialize and BinaryFormatter.Deserialize methods will throw NotSupportedException across nearly all project types, including console applications.
A compatibility switch exists for applications which need to reenable BinaryFormatter behavior. See below for details.
Version
.NET 8 Preview 4
Previous behavior
In .NET 7, the BinaryFormatter.Serialize and BinaryFormatter.Deserialize methods were marked obsolete as error. However, if an application suppressed the obsoletion, the methods could still be called and would still function properly in most project types (excluding aspnet, wasm, and MAUI). For example, the APIs would function properly within a console app.
New behavior
In .NET 8 Preview 4, these methods will throw NotSupportedException at runtime across all project types except WinForms and WPF. The APIs continue to remain obsolete-as-error across all project types, including WinForms and WPF.
Type of breaking change

[ ] Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[ ] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code may require source changes to compile successfully.
[X] Behavioral change: Existing binaries may behave differently at run time.

Reason for change
This is the next stage of the BinaryFormatter obsoletion plan, preparing for BinaryFormatter's eventual removal from .NET. See earlier breaking change notifications for additional context:

.NET 5 BinaryFormatter breaking changes
.NET 7 BinaryFormatter breaking changes

Recommended action
The best course of action is to migrate away from BinaryFormatter due to its security and reliability flaws.
However, should you need to continue using BinaryFormatter, please review the Recommended action section of the .NET 7 breaking change notification. That document provides a compatibility switch that can be set in your application's project file to reenable BinaryFormatter functionality. That compatibility switch continues to be honored in .NET 8.
Feature area
Serialization
Affected APIs

System.Runtime.Serialization.Formatters.Binary.BinaryFormatter.Serialize(Stream, object)
System.Runtime.Serialization.Formatters.Binary.BinaryFormatter.Deserialize(Stream)

Associated WorkItem - 91247

Not only is .NET Core still missing things present in .NET Framework, but now others are being removed. Lol.


Macro system needs some adaptations to work with LFC

A few modifications are needed to work with LFC.

We must split out DEFINE_TIMER_STRUCT and DEFINE_TIMER_CTOR since first goes in a header file and second goes in a source file.
We must expose (offset, period) and (min_delay, min_spacing) as ctor arguments, not hardcoded. This is to support setting those based on reactor parameters

see #82

Yes, all of them are affected. And I have done it as part of #71


[alchemy] Transition AlchemyLanguage to use WatsonGateway

WatsonGateway has been removed in the rest-kit branch. We are preferring to rely on basic authentication and the feature set of Alamofire.


warning when sampling grid is large

We should add a user warning when the sampling grid size of xy_from_grid reaches some large value.

I would like to work on this. Is there a certain dimension size value at which the sample grid should raise a warning or should the warning be used to keep the user from inputting large values?

Thanks for your interest @irvin2492!
This is somewhat arbitrary, but I'd say it would be good to warn the user if image, defined on line 54, will take up more than 1 Gb of memory. Assuming np.float64 pixels, this will happen when memory_in_Gb > 1, where memory_in_Gb is defined like this:
npix = 2 * sample_dim**2
byte_per_pixel = 8 # for np.float64
memory_in_Gb = byte_per_pixel * npix / 1e9

And the check should happen before xx and yy are created.

Issue closed by #3.


Pause autoscaling doesn't work from 0->1 or 1->0

Report
Trying the feature with RabbitMQ Scaler, I tried to pause the scaling setting the annotation autoscaling.keda.sh/paused-replicas: '0'.
The workload went to 0 properly but automatically the operator scaled the workload from 0 to 1 (because there were pending messages in the queue).
I have tried setting autoscaling.keda.sh/paused-replicas: '1' and the operator scaled to 1 and a few moments after, again to zero (because there wasn't any pending message).
It seems that the operator is not taking into account this annotation during the process to determine if the ScaledObject is active or not.
During the stage 1->N and N->1 works good
Expected Behavior
Pause the scaling in both stages, 0->1->0 and 1->N->1
Actual Behavior
The scaling is paused only in 1->N->1
Steps to Reproduce the Problem

Deploy a SO using RabbitMQ trigger scaling from 0 to X
Enqueue some messages in the queue
Add the annotation autoscaling.keda.sh/paused-replicas: '0'

Or

Deploy a SO using RabbitMQ trigger scaling from 0 to X
Ensure the queue is empty
Add the annotation autoscaling.keda.sh/paused-replicas: '1'

Logs from KEDA operator
1.6517537492817814e+09	INFO	scaleexecutor	Successfully scaled target to paused replicas count	{"scaledobject.Name": "rabbitmq-consumer", "scaledObject.Namespace": "default", "scaleTarget.Name": "rabbitmq-consumer", "paused replicas": 0}
1.6517537542743633e+09	INFO	scaleexecutor	Successfully updated ScaleTarget	{"scaledobject.Name": "rabbitmq-consumer", "scaledObject.Namespace": "default", "scaleTarget.Name": "rabbitmq-consumer", "Original Replicas Count": 0, "New Replicas Count": 1}

1.651753834290882e+09	INFO	scaleexecutor	Successfully scaled target to paused replicas count	{"scaledobject.Name": "rabbitmq-consumer", "scaledObject.Namespace": "default", "scaleTarget.Name": "rabbitmq-consumer", "paused replicas": 1}
1.651753859283409e+09	INFO	scaleexecutor	Successfully set ScaleTarget replicas count to ScaledObject minReplicaCount	{"scaledobject.Name": "rabbitmq-consumer", "scaledObject.Namespace": "default", "scaleTarget.Name": "rabbitmq-consumer", "Original Replicas Count": 1, "New Replicas Count": 0}

KEDA Version
2.7.0
Kubernetes Version
1.23
Platform
Microsoft Azure
Scaler Details
RabbitMQ
Anything else?
No response

CC @aryan9600


gulp-util is deprecated - should be migrated away from

The gulp team recommends upgrading migrating away from gulp-util using the guidelines in the article below:
https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5

There's a PR ( #53 ) about this issue, but author didn't make choice (and update the package in npm repository)


User name of jsbin is not in sync with github

Clicking the update account button in the profile page if jsbin is not updating the user name changes of github account. It holds the old github user name.

same here my username on jsbin is currently 3amsleep


Does it support adaptation?

Hello
Does the height of each group support self-adaptation?

If you mean a "masonry layout", the answer is "no".

Based on the non-existent browser support and potential accessibility issues, for now a masonry layout is not an option for this project.


Selected values that are disabled are shown in the button

I filter the possible options with disabled but if the user has selected anything and then those options are set to disabled, when I refresh the button text does not change.
Even if I click on a enabled option it just adds the selection but does not remove the disabled selection.

Fixed it by changing the source code to this:
  **//if** (!HTMLSelectElement.prototype.hasOwnProperty('selectedOptions')) {
    Object.defineProperty(HTMLSelectElement.prototype, 'selectedOptions', {
      get: function () {
        return this.querySelectorAll(':checked**:not(:disabled)**');
      }
    });
  **//}**

If you don't want to show disabled options that are selected in the button, set hideDisabled: true or data-hide-disabled="true". If you don't want to enable that, but still don't want to show disabled options in the button, a better alternative would be to actually update the select's value to not include the disabled options.


[SPARK-25656][SQL][DOC][EXAMPLE][BRANCH-2.4] Add a doc and examples about extra data source options

What changes were proposed in this pull request?
Our current doc does not explain how we are passing the data source specific options to the underlying data source. According to the review comment, this PR aims to add more detailed information and examples. This is a backport of #22801. orc.column.encoding.direct is removed since it's not supported in ORC 1.5.2.
How was this patch tested?
Manual.

Test build #98046 has started for PR 22839 at commit c9d89f4.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4501/
Test PASSed.

Test build #98046 has finished for PR 22839 at commit c9d89f4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98046/
Test PASSed.

Thanks @dongjoon-hyun This LGTM!

Thank you so much for review, @dbtsai .
Merged to branch-2.4.


Historical data requests start & time fix

Description

Fix calculation Time.GetStartTimeForTradeBars, should round down in data timezone because the data timezone rounded down, converted into exchange time zone will be the datas Time.

Related Issue

Closes #4035
Motivation and Context

History start and end time are wrong, as a result we
Requires Documentation Change

N/A
How Has This Been Tested?

manual testing
add regression tests
add unit tests for new implementation of Time.GetStartTimeForTradeBars

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Given the sensitivity and how easy it is to overlook issues in timezones, I suggest we flesh out the regression tests to include every asset class and timezone (even boring data-UTC-exchange-UTC ones).

OANDA - UTC-UTC
Futures - ET but complex partial open-close rules.
Equities - ET-ET
Crypto - Never close.


Getting Started "Clone Git Repository..." silently fails if you don't have git installed

Version: 1.55.0-insider (user setup)
Commit: 26a60dbecacc8bdacc06f7681ebe95398878a1cd
Date: 2021-03-29T10:27:14.607Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Steps to Reproduce:

Uninstall git if you have it
Open getting started/welcome page
Click "Clone Git Repository..."

Expected some friendly error message or something but the user sees nothing. Here's what is in dev tools:
  ERR command 'git.clone' not found: Error: command 'git.clone' not found
    at C._tryExecuteCommand (file:///C:/Users/Tyler/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1558:3364)
    at file:///C:/Users/Tyler/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1558:3245

Looks like maybe the git extension isn't activated until git is installed?

Does this issue occur when all extensions are disabled?: N/A

Yeah, we abort really early if Git is missing. We do set the git.missing context key, so maybe we should key off of that in the getting started?

I notice the option is still there for a split second then goes away, but it's not bad so marking as verified.


Blink wallet sending attachment

Description
Adds blink sending attachment. Solves https://github.com/stackernews/stacker.news/issues/1142 .
Screenshots

Additional Context
This attachment currently works only with the  blink's staging environment due to an issue in cors configuration, the pr should stay on hold until the fix is pushed to production.
Checklist
Are your changes backwards compatible? Please answer below:
Yes
Did you QA this? Could we deploy this straight to production? Please answer below:
Yes
For frontend changes: Tested on mobile? Please answer below:
Yes
Did you introduce any new environment variables? If so, call them out explicitly here:
No

This attachment currently works only with the blink's staging environment due to an issue in cors configuration, the pr should stay on hold until the fix is pushed to production.

Is there some discussion in https://github.com/GaloyMoney/blink-mobile or elsewhere that I can subscribe for updates regarding this? I searched for "CORS" but found nothing.

Is there some discussion in https://github.com/GaloyMoney/blink-mobile or elsewhere that I can subscribe to for updates regarding this? I searched for "CORS" but found nothing.

The conversation happened in the galoy dev chat, but there are these two related prs:
https://github.com/GaloyMoney/charts/pull/6885
https://github.com/GaloyMoney/charts/pull/6888
Also, it seems it went to production 1 hour ago,  so, later today i will rebase this pr and do a final test and then it should be ready to go :+1:

And do you have any feedback regarding the wallet documentation or ease of implementation? Was it straight-forward or did you have to read a lot of the existing code to get an idea how to add a wallet (additionally to the documentation)? You are the first contributor who added a wallet so we're very curious how easy it was and if there is anything lacking that we could improve.

It was pretty straight-forward actually, the only doubt i had was if i had to put these variable in index.js or client.js https://github.com/stackernews/stacker.news/blob/b7b4e4688d4fef9387f7f273e83a29325380b376/wallets/blink/index.js#L3-L5
the documentation seems to imply that index.js should export only the specified properties , but i ended up putting them there anyway since they are variables that server.js could use at some point, not sure if this respect the design

The current master has an issue that affect this pr, it seems something broke validation for optional fields in commit ae73b0c19fd277e1dfd2bac289a869b1dc980cfb .
The 'currency' field in the blink connector is defaulted to "BTC" by client.js, if unspecified, however since ae73b0c19fd277e1dfd2bac289a869b1dc980cfb if the field is left empty in the attach screen, the wallet does not attach and redirects back to settings/wallets without error.
I've tried to rebase this pr on the commit before ae73b0c19fd277e1dfd2bac289a869b1dc980cfb and it works as expected there.


Do you have any plan to support running SSIS packages using elastic job agent?

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

@Kalyanms1  Thanks for reaching out to us , but at this time it does not look like you are referring to a document but future path of a specific product . The repo azure-docs is only for docs which needs updates  . I could suggest you to please post your question at this location https://feedback.azure.com/forums/34192--general-feedback .
Going ahead and closing the issue as of now .


eval.py: Evaluation broken

Hey Guys, the last commits from today seems to break evaluation code. More precisely spectrogram2wav.
If you're not using log magnitudes, eval.py crashes during sampling because the buffer to generate the audio from is not finite. I checked eval.py against the previous revision. Assuming that you're not using log magnitudes (ie hp.use_log_magnitude = False), this here is the change:
audio = spectrogram2wav(np.power(np.e, s)**hp.power)
changes to
audio = spectrogram2wav(s**hp.power)
I'm not quite sure since I'm a bit confused by all the log vs non-log discussions going on, but I guess this is wrong. The paper says they're predicting mel-scale. Correct me, but isn't mel-scale already log?
So my assumption is that the first line is still valid for non-log magnitudes, right?

Technically speaking mel-scale is not exactly the same as log. See https://en.wikipedia.org/wiki/Mel_scale. The paper says they use melspectrogram and linear-scale log magnitude (spectrogram). So the spectrogram2wav converts the predicted magnitude to the waveform. It has nothing to do with melspectrogram.
The reason why people care about whether we appy logarithm to magnitude in training is two, in my opinion. First, we or at least I don't have a full understanding of why it is useful. Second, in practice it needs our attention in that there are three times of padding--reducing frames, dynamic padding, and convolution with same padding.
For now I don't know why there's a problem when you set use_log_magnitude to False. I'll check soon.

Oh now I know the reason. For the plain magnitude we must not allow negative numbers because of the power. I simply clipped the value to zero. https://github.com/Kyubyong/tacotron/blob/master/eval.py#L70

Well, you're right when you say that mel isn't exactly log. And to be honest, my explanation isn't much more then a first guess. I didn't check every part of the code and I agree with you: I'm not quite sure what the log stuff is all about.
But: Even after your latest commits, I can't generate non-silent audio from my model if use_log_magnitude is False


Auto defining fallbackLng in production build breaks v4.5

Describe the bug
Since this change of i18next v19.8.5 & the default config of next-i18next v4.5.0 in prod build it is not able to "stay" in the secondary language, SSR renders the secondary language but at client side, it changes to fallbackLng which is the defaultLanguage (which is the primary language)
Occurs in next-i18next version
v4.5.0
I'm not able to update to latest since my app uses custom server not just for translations
Steps to reproduce

clone https://github.com/felixmosh/next-i18next-bug-fallbackLng
yarn install
yarn build
yarn start
open http://localhost:3000
change to de (check that there is de subpath)
reload the page
the page shows in english

Expected behaviour
After reloading the page, it should remain on the Dutch lang
Additional context
It is not possible to turnoff this fallbackLng behavior since the fix is not available in v4.5.0

Hey @felixmosh – what exactly are you hoping for here? Not sure we can reasonably support previous major versions.

Is it possible to add the fix to v4?

Can you elaborate on why your custom server requirement prevents you from upgrading past v4?

If I remember correctly, starting v5 there is no lng middleware usage, which  do need for my custom server.
All of my app urls are coming from CMS, so I need to know the current lng.

And is next-i18next@beta not an option for you? I assume the Vercel team have taken custom servers into account?

They can custom servers into account since its handles are the first the get the response, therefore they need to handle lng.

Do you mean that the internationalisation in Next v10 doesn't support custom servers at all?

Think of that, custom server is the first thing that handles the request, and next.js handlers may come later in the pipe line.
For sure Next.js i18n doesn't work out of the box with custom servers...

I'd be curious to hear feedback on that from the Vercel team. In the meantime, if you'd like to open a PR against a v4 branch, I can publish the fix you need. Otherwise you'll need to maintain a fork.

Thanx, I will do...

I doesn't allows me to open a PR agains tag, can you open a branch from v4.5?

BTW, currently I am patching the lib with patch-package, it works as expected, if you don't want to maintain v4.5 anymore, it is ok to close this issue.
I just wanted to update that with latest i18next there is more chance to get that bug that I've described.

@felixmosh I've created a v4 branch. I won't be actively maintaining past versions myself, but if users want to do so, I'm fine with that!

The branch is based on v7 :]

Updated.

@felixmosh Do you want to keep this open?

Will you publish updates to v4? if so, I will make cleaner pr for it :]

Yep, as I said, I'll merge it if you want to do the work!

I'm closing this issue, since many tests are failing and i don't have much time to solve them.
Currently i'm patching the lib with patch-package.


Validators

You've detailed that validators are avaiable, what they do and how to write one but completely neglect to document how they are wired into the flow ...

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

ID: 72538432-fde5-22ab-7997-8a34b11aa24e
Version Independent ID: c469fa5b-2590-ee16-1491-51b90c069204
Content: Gather user input using a dialog prompt - Bot Service
Content Source: articles/v4sdk/bot-builder-prompts.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

@jamsoft Looking back at this, the information is hidden near the top of the article. I'll add a note and highlight corresponding code in the validation section itself.
Here's the relevant C# code from the sample:
            // Create the dialog set and add the prompts, including custom validation.
            _dialogSet = new DialogSet(_accessors.DialogStateAccessor);
            _dialogSet.Add(new NumberPrompt<int>(PartySizePrompt, PartySizeValidatorAsync));
            _dialogSet.Add(new ChoicePrompt(LocationPrompt));
            _dialogSet.Add(new DateTimePrompt(ReservationDatePrompt, DateValidatorAsync));

Added in the missing context to the Implement validation code section. The update should go live in a couple days.


Allow public sharing of real time dashboard

Describe the feature or enhancement
Currently it is possible to create a public shareable link through the settings, however, it does not allow for real time view. I'd suggestion to have an extra toggle on the share section "Allow Realtime", which would allow users to see not only the website's dashboard, but also the real time dashboard. Happy to contribute to this.

Honestly it probably doesn’t even need a toggle, IMO. Curious what the use-case is to not have them see realtime data. I would just assume there’s no downside and that they should allow realtime data by default to everyone with a share URL.


Remove redundant information from apidesc.md

And update a few bits that were out of date. This has been bugging me
for a while -- rest_api.md is the real API resource, but this has
existed and every time someone submits a patch that adds an API call,
if they remember to update apidesc (which is not always, and I never
do...) I always wonder why we have the file at all. Then I remember that
there's some overview stuff at the top.

Why don't we put the overview stuff at the top of rest_api.md? It's just 60 lines. We would only need the 2 sections Objects in the HaaS and API design philosophy.

@naved001, good call, done.

now that we'll have all the API stuff in one place, this should take care of #543


修复QA-Engineer在RunCode(执行用例)结束后，要求Engineer WriteCode(修改代码）, Engineer未修改，业务流直接结束的问题

User description
Features
修复QA-Engineer在RunCode(执行用例)结束后，要求Engineer WriteCode(修改代码）, Engineer未修改，业务流直接结束的问题
Influence
导致过早结束业务流程，Engineer代码有问题而直接结束流程；修复后会增加代码修改流程，增加成功率，但同时会降低完成率。请慎重评估是否合入。
Result
修改后，在QA测试发现原码有问题时，Engineer继续修改代码，进行重试。

PR Type
Bug fix, Enhancement

Description

Added and enhanced debug and warning logs across multiple roles to improve traceability and debugging.
Increased the test rounds allowed for QA engineers to ensure thorough testing and bug fixing.
Included RunCode in the actions that can trigger code writing, enhancing the engineer's role responsiveness.

Changes walkthrough 📝
Relevant filesEnhancement

base_llm.pyAdd Debug Logging for LLM Response

metagpt/provider/base_llm.py

Added debug logging for the response from the language model.

+1/-0

engineer.pyEnhance Logging and Action Handling in Engineer Role

metagpt/roles/engineer.py
Added import for RunCode.  Added warning logs for cases where no action is determined or nothing gets done.  Included RunCode in the set of actions that trigger writing code.

+4/-1

qa_engineer.pyIncrease QA Engineer Test Rounds Limit

metagpt/roles/qa_engineer.py

Increased the allowed test rounds from 5 to 20.

+1/-1

role.pyImprove Logging for Role Actions and Decisions

metagpt/roles/role.py
Added debug logging for actions handling and process flow.  Added warning log when no actionable todo is present.

+3/-1

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Codecov Report
Attention: Patch coverage is 77.77778% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 70.20%. Comparing base (f201b2f) to head (463d2a5).

Files
Patch %
Lines

metagpt/roles/engineer.py
50.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1267      +/-   ##
==========================================
- Coverage   70.22%   70.20%   -0.03%
==========================================
  Files         316      316
  Lines       18860    18867       +7
==========================================
  Hits        13245    13245
- Misses       5615     5622       +7

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

Engineer并未订阅RunCode消息，因此即使消息发给了Engineer，在_observe阶段也会被忽略掉。
建议参考Product Manager给Engineer转bug单的流程，将需要Engineer重写的代码作为bug单转给Engineer。参见metagpt/actions/write_prd.py的_handle_bugfix：
    async def _handle_bugfix(self, req: Document) -> Message:
        # ... bugfix logic ...
        await self.repo.docs.save(filename=BUGFIX_FILENAME, content=req.content)
        await self.repo.docs.save(filename=REQUIREMENT_FILENAME, content="")
        return AIMessage(
            content=f"A new issue is received: {BUGFIX_FILENAME}",
            cause_by=FixBug,
            send_to="Alex",  # the name of Engineer
        )

metagpt/roles/engineer.py中的_think:
        if self.config.inc and msg.cause_by in write_plan_and_change_filters:
            logger.debug(f"TODO WriteCodePlanAndChange:{msg.model_dump_json()}")
            await self._new_code_plan_and_change_action(cause_by=msg.cause_by)
            return self.rc.todo

Engineer并未订阅RunCode消息，因此即使消息发给了Engineer，在_observe阶段也会被忽略掉。 建议参考Product Manager给Engineer转bug单的流程，将需要Engineer重写的代码作为bug单转给Engineer。参见metagpt/actions/write_prd.py的_handle_bugfix：
    async def _handle_bugfix(self, req: Document) -> Message:
        # ... bugfix logic ...
        await self.repo.docs.save(filename=BUGFIX_FILENAME, content=req.content)
        await self.repo.docs.save(filename=REQUIREMENT_FILENAME, content="")
        return AIMessage(
            content=f"A new issue is received: {BUGFIX_FILENAME}",
            cause_by=FixBug,
            send_to="Alex",  # the name of Engineer
        )

metagpt/roles/engineer.py中的_think:
        if self.config.inc and msg.cause_by in write_plan_and_change_filters:
            logger.debug(f"TODO WriteCodePlanAndChange:{msg.model_dump_json()}")
            await self._new_code_plan_and_change_action(cause_by=msg.cause_by)
            return self.rc.todo

好的，感谢！


looses messages on network issues

I'm sending multiple single messages on the same topic (10 messages/sec). Everything works fine unless I start imitating network issues (running single kafka instance on VM and switching network adapter to "not attached" and back). During each disconnect 1-2 messages are constantly lost. They are not caught neither by producer error listener, nor by send callback. "requireAcks" parameter makes no difference.

The producer doesn't have a client side request timeout. Take a look at #289


KAFKA-12839: Let SlidingWindows aggregation support window size of 0

Create closed [start, end] time window: SessionWindow for slidingWindow aggregation. So that we can support time window size with value 0, and have correct aggregation results.
Make the TimeWindow as a time interval window container that holds the start and end time, so that we can support window store with window size of 0.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @ableegoldman , I found we already had a SessionWindow to represent the close time interval: [start,end]. So I directly use it for SlidingWindows aggregation window creation. I think this is what we want, right?
I have another thought, which is to rename the time interval related windows. Currently, we have 3 types of time interval window:
TimeWindow -> to have [start,end) time interval
SessionWindow -> to have [start,end] time interval
UnlimitedWindow -> to have [start, MAX_VALUE) time interval
I think the name SessionWindow is definitely not good here, especially we want to use it in SlidingWindows now, although it is only used for SessionWindows before. We should name them with time interval meaning, not the streaming window functions meaning. ex:
TimeWindow ->  LeftClosedRightOpenWindow
SessionWindow -> ClosedTimeWindow
UnlimitedWindow -> LeftClosedWindow
ref: the Classification of intervals section in https://en.wikipeadia.org/wiki/Interval_(mathematics)
Because these 3 window types are internal use only, it is safe to rename them. What do you think?

It seems super awkward and likely to lead to confusion/future mistakes to use something called a "SessionWindow" in the "SlidingWindowAggregate", although yeah, it's pretty much exactly the same otherwise and can be reused.
In other words, I totally agree with your proposal to just rename the existing Window implementations to describe the actual interval they represent, rather than some specific type of windowed operation that just happens to use them at the moment. (In fact I had written that first paragraph before I even saw your comment with the renaming proposal, great minds think alike huh 😜)
That said, those names are just super clunky.  Maybe it's my inner physicist, but sometimes mathematical precision just isn't appropriate for real-world usage (don't tell any mathematicians I said that!) Unfortunately I'm not crazy about any of the alternatives I can think up, maybe you can come up with some better ideas. Here's the best I could come up with:
TimeWindow --> InclusiveExclusiveWindow
SessionWindow / SlidingWindow --> InclusiveInclusiveWindow
UnlimitedWindow --> InclusiveUnboundedWindow
To me at least these feel more natural, ie it's clear what they mean without having to reference Wikipedia. I mean most people probably do know what open/closed mean, but inclusive/exclusive is more to the point. Also I think we can drop left/right and just imply it by the ordering. Thoughts?

@ableegoldman , really glad we have the same thought about it! (high-five :) )
I created a jira ticket KAFKA-13145 for this renaming improvement, and see if there are other opinions there.
For your suggestions: Yes, I agree most people probably do know what open/closed mean, but inclusive/exclusive is more to the point. +1
Thank you.

@mjsax @ableegoldman @lct45 , I've added tests. Please help review. Thank you.

@ableegoldman , please take a look if this is what you expected. Thank you.

@mjsax , as discussed, I've created a SlidingWindow for SlidingWindows aggregation. So we can support window size of 0 in SlidingWindows now.
But there's still a issue left:

We can't create a state store with window size of 0 because we use TimeWindow to represent the "windowed KTable" result.

The discussion thread is here: https://github.com/apache/kafka/pull/11124#discussion_r678908862
We can create another jira ticket to handle it, but not sure if you have any suggestion for this issue?
Thank you.

@mjsax What's the status of this PR? Do you still want to get it into 2.8.1? If so, would you have an ETA for it?

@dajac No, we're not aiming to get the fix into 2.8.1 -- no need to block on this

@showuon -- should we close this PR for now?
We have actually collected a few tickets that are not easy to fix without a DSL re-design. I would propose to add a new Jira label to tag all relevant tickets and we hold off working on them for now. When we do a DSL re-design we can cycle back?

Make sense to me. Close this PR and cycle it back after DSL redesign. Thanks.


Move API description of file watcher to doc proper

We don't want to be like HTTP and have no readable doc but big docstrings everywhere ;-)

Another thing is that it's not SimpleWatcher that people should implement but rather something <: FileWatcher, I've modified the code to reflect this, the doc should need to follow this. Basically it should be
mutable struct MyNewFileWatcher <: FileWatcher
# fields
end

FileWatcher is probably a bad choice since there's already the very similar stdlib FileWatching...?

Hmm you're right, though if someone wants to extend it they would have to explicitly write
mutable struct MyNewFileWatcher <: LiveServer.FileWatcher
# fields
end

because we don't export FileWatcher (so my previous comment was not complete). Of course the user could do using LiveServer: FileWatcher but I feel then the intent is pretty clear. Doing ... <: LiveServer.FileWatcher would make it sufficiently different from FileWatching I would think?

You're right.


Add more API tests (and actually run them)

I decided to finally start writing a few more API tests. So far I only added tests for color and ErrorBar, but I'll see to adding more.
While writing the tests I realized how confusing the empty proc that returns a bool is. Since it's not exported (the proc is, but color.nim is not), I simply renamed it to isEmpty without any deprecation.

Merging this now. I'll add more tests in another PR.


Make a query for a fully qualified tags, bypassing the lookup service

There's currently a limitation that for 'unknown' timeseries, they cannot be queried. Could we augment the querying API to allow the notion of a fully qualified query, where the user has specified all tags for a set of timeseries. This is very useful when referring to specific points in the past, referenced from documentation for incident scenarion's.

This is also something we've discussed as a means to harden systems against metadata unavailability. The client can periodically request a list of identifiers based on a filter that can be used for querying. It also would work as a way to freeze the state of a query in time, making the result more reproducable.


Windows workarounds

With these workarounds I got to the point where GraalVM complains about certain GCC tools not being around.
I haven't been able to fix this, since GraalVM assumes you execute your commands in a special Windows 7.1 SDK shell and clojure kind of assumes Powershell.
But except from that, I worked around some other issues I came across (see #6, #7 and #8)

I gave this a once-over and it looks good to me modulo comments :)

Please note that I don't need this PR myself anymore since I've reverted to an uberjar + simple .bat script for building clj-kondo on Windows. But for other users running into this, it might still be relevant.

Rebased/merged in 546a77a4492be2f1d9b48e43cb48fb0ebfa39cc2. Thanks!


Proposal: CJS or ESM output for esl-utils module

Can the consumer use from ESM Node project CJS modules
Trends in Node env
if we include support of ESM/CJS for esl-utils - > how do we need to extend our build process...

@ala-n
Upon the investigation based on the ESL project, I came to the following conclusion:
I made an attempt to modify our build process using newly added package.json export maps (more details on that https://www.sensedeep.com/blog/posts/2021/how-to-create-single-source-npm-module.html
https://habr.com/ru/articles/695482/)
In the end i got something like that:
  "module": "modules/mjs/modules/all.js",   "exports": {     "./*": {         "import": "./modules/mjs/modules/all.js",         "require": "./modules/cjs/modules/all.js",         "types": "./modules/types/modules/all.d.ts",         "default": "./modules/mjs/modules/all.js"     }   },
To crate separate export folders, it involved adding additional typescript config files for each module type. These config files are extensions of our existing tsconfig.buid.json file.
tsconfig-esm.json was added for esm and default module types:
{   "extends": "./tsconfig.build.json",   "compilerOptions": {     "module": "esnext",     "outDir": "./modules/mjs",     "target": "esnext"   } }
tsconfig-cjs.json was added for commonjs modules:
{   "extends": "./tsconfig.build.json",   "compilerOptions": {     "module": "commonjs",     "outDir": "./modules/cjs",     "target": "es2015"   } }
In order to prevent TypeScript from generating two type maps, it was separated into standalone process aswell:
`
{
"extends": "./tsconfig.build.json",
"compilerOptions": {
"outDir": "./modules/types",
"declaration": true,
"emitDeclarationOnly": true
}
}
`
The result package worked well in test repository, https://github.com/fshovchko/esl-monorepo-test throught webpack build process. Same goes for node environment that was using CommonJs module type.
But using ESM modules in Node resulted in a simillar error: https://github.com/Esri/calcite-design-system/issues/5077. In order to fix the issue, the file extensions should be specified in import/export statements. And esl's package.json should have "type": "module" set. But using the statement will result in error in non-module environment. One of the workaround to it can be using a separate package.json just for esm modules that will only consist of one field {"type": "module"}.
Another workaround that I found was proposed by a TypeScript itself (https://www.typescriptlang.org/docs/handbook/esm-node.html). Changing our module to "module": "NodeNext" seems to resolve all of the issues. It works well with CommonJs and ESM, doesn't require specified extensions, or basically any other configuration from our side. Looking at its compiled code, it looks like cjs based coade, with probably some extra fetures added to it. The code was tested in Node v14 and everything worked fine. Some extra explanation about "NodeNext" here: https://stackoverflow.com/questions/71463698/why-we-need-nodenext-typescript-compiler-option-when-we-have-esnext
About the Node version.

Starting Node 12+, it has support for CJS and ESM side-by-side.
Node.js 14 introduced the ESMLoader, which is responsible for loading and interpreting ES Modules. It enables features such as top-level await, dynamic import(), and import.meta.
Node.js 17 introduced import assertions. (https://nodejs.org/api/esm.html).
We don't use import assertions, so we should be good with any Node starting from v14.

In package json it can be specified as following:
  "engines": {     "node": ">=14.0.0"   },


[CNVS Upgrade] Use stylelint-config-dcos from npm package

ℹ Related to this PR: https://github.com/dcos/dcos-ui/pull/1092

This PR implements the use of the stylelint configuration from our npm module.

The style changes was reviewed here: https://github.com/dcos/dcos-ui/pull/1092


Bug fix for notebooks

Fixed custom output command not rendering correctly
Convert config and output paths to absolute paths

coverage: 81.67% (-0.09%) from 81.756%
when pulling 8f9aa1537692c7a4a793c728fbfc22403117e855 on fix_custom_output
into e60d7043d1a873172b0993f3f98e68d4ec6c1af9 on master.

Is this PR compatible with the current Jupyter env and can be merged immediately or this PR is only for the upcoming Jupyter env and have to wait for that one to go live before this PR can be merged?
If this is for the upcoming Jupyter env only, can this PR be merged to the branch fixnbs_pavics_betaimage (https://github.com/CSHS-CWRA/RavenPy/pull/356) from Travis so I can test all upcoming changes together?

It's a bugfix that I believe is compatible with the current env.


Login Failed - Google Account

I'm getting login error in my google account. The user and password are correct, I have no verification in 2 steps and I have disabled the captcha. What could be happening?

I made a "app password" for my google account, and used that. Then everything worked fine.

@Zerqz , how do you do that?

Like this

@Zerqz Thanks i had to edit my account to 2 step auth to be able to add a app password, it works

Why doesn't it use oauth2 instead of needing a username/password? I have 2FA on, but oauth2 would solve all these auth issues.


Fix Broken GitHub Actions Tests

Fixes #339

Coverage: 100.0%. Remained the same when pulling f194269bbded6f4b88998bf9fb0258de438ce8c5 on github_actions_fix into 1108c74f3a6c373edc1e896b6501f45c5fc37a57 on master.


refactor skipLink component

remove hardcoded links in skipLink component
introduce a skipLinkService that can be used by individual components to register a skipLink
improve UI for the skipLinks

@Xymmer  Since this component is not ready, I recommend to remove it entirely for 1.0 and add it as a new feature in a 1.x release.

We've discussed this feature and had a number of thoughts:

use a clear name, skipLink seems to be right
we should not hardcode any skipLinks as we did previously. We like a configurable solution, that customers can reuse to skip other portions of the page
we should not use anchor links, since these are driven by page IDs. We do not use page IDs in single page app, where components can be reused. We should use the scrollTo API.
this functionality is about skipping portions of the page. This is not so much related to components, but more related to page sections. We should connect the skipLinks to page slots, or page groups (header, footer or pageTemplates). This is all related to the PageLayoutService.
we like @developpeurweb to provide the skipLinks we need for Spartacus as well as a few ideas we see at existing commerce sites. Based on this, we can confirm whether the solution we come up with is valid.
would be great if @kacperknapik and @Zeyber can come up with a solution proposal. I'd like to be aligned with the solution.

@tobi-or-not-tobi please check #5048 (PR #5128) and decide if this ticket can remain closed or should be reopened

deprecated by https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/5128#issuecomment-554029794


Unable to use haxm while docker is installed

I had a problem similar with this guy #361 .
I solved it by running this command bcdedit /set hypervisorlaunchtype off and it solved my issue.
At this point in time, I restarted my PC and got an error from docker saying that it needs hyper-v enabled. I enabled it via bcdedit /set hypervisorlaunchtype auto. Now, my perminal gives me the following output for haxm:

Any help? I wanna use both applications on the same machine (windows 11 pro)

@Temepest74 , HAXM could NOT work with Hyper-V on. If you want to use HAXM  ,you should disable HyperV and run 'bcdedit /set hypervisorlaunchtype off'

@Temepest74 , HAXM could NOT work with Hyper-V on. If you want to use HAXM  ,you should disable HyperV and run 'bcdedit /set hypervisorlaunchtype off'

Could you help me with any workaround?
Is there anything like wsl but with windows instead of linux?
If not, will it work in a vm?


updates FakeXMLHttpRequest to fix empty event bug

See https://github.com/pretenderjs/FakeXMLHttpRequest/pull/19

Closing, duplicate of #132


Added changes to support columns list for GET queries.

Example:
public T get(K1 pk1, K2 pk2, Set<String> columns) {
    return mapper().get(pk1, pk2, columns);
}

final Entity entiry = dao.get(key1, key2, ImmutableSet.of("col1", "col2"));

Changed base branch since we plan to rename 3.0 to 3.x.

This can be easily achieved already with one of the following methods:

using an accessor

@Accessor
public interface EntityAccessor {

    @Query("SELECT col1, col2 FROM entity WHERE key1 = ? AND key1 = ?")
    Entity findEntity(int key1, int key2);

}

Using a custom resultset:

ResultSet rs = session.execute("SELECT col1, col2 FROM entity WHERE key1 = ? AND key2 = ?", key1, key2);
Entity entity = dao.map(rs).one();

Globally I'm not sure the use case is general enough to fit inside the mapper.

@adutra it is general enough, since the mapper itself uses it inside of it at line 107
PreparedStatement getPreparedQuery(QueryType type, Set<ColumnMapper<?>> columns, EnumMap<Option.Type, Option> options) {
It was there before my change.
Why not to provide the ability for the top level classes to pass the columns list?

Closing as we are not going to add any new features to the mapper in the driver 3.x series. A new mapper API will be published after the 4.0 release.
@vagifzeynalov sorry for closing so abruptly but you didn't include any tests so it's hard to evaluate how stable this PR is, and I still think that we could achieve the same effect using accessors. Thanks anyway for your time and effort.


[FLINK-23484][benchmark] Supports base benchmark for ChangelogStateBackend

What is the purpose of the change

Supports ChanglogStateBackend Benchmark. Will also prepare a related pr in flink-benchmarks repo.

Brief change log

Move benchmark utils from rocksdb repo to flink-test-util.
Add Changelog supports in StateBenchmarkBase.

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is already covered by existing tests, such as StateBackendBenchmarkUtilsTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

@flinkbot run azure

@Myasuka @fredia Could you help to take a review ?
This pr should be reviewed together: https://github.com/apache/flink-benchmarks/pull/75


DDG -DA案例里面的训练环境

❓ Questions and Help
DDG -DA案例里面的训练环境是什么，有没什么建议呢？用ryzen 3800x+64g内存跑24小时了，还是在Recorder 4a9884ee9f044ebe9d8e414aa89b7b20 starts running under Experiment 1 ... 状态，cpu占用一直是50%左右
We sincerely suggest you to carefully read the documentation of our library as well as the official paper. After that, if you still feel puzzled, please describe the question clearly under this issue.

Please press ctrl+c to interrupt the program and attach the detailed error traceback

Please press ctrl+c to interrupt the program and attach the detailed error traceback

thanks priviously it was running under pycharm's terminal, press ctrl+c types a ^c,but still goes on. now I've switched to ubuntu's bash terminal,run the programme,it's in experiment2 now.

Please press ctrl+c to interrupt the program and attach the detailed error traceback

Still, it wasn't working..I thought it was enviorment problem. So I did a pip install pyqlib agian.guess what, it upgraded lightbm to version 3.3.2,then it all works.


Fix Pytest

This PR resolves pytest.
This also includes the changes required in alembic/env.py as suggested by @0dm in #238.
@abrichr Ready to be Merged! :)

@abrichr Thank you for your prompt merge :) 🙏


Warn about last value being authorative

As a component developer, you might consider doing secure communications with a central node to check eligibility for the component. However, the ServicePointManager offers no protection against malicious code using the component and creating fraud HTTPS replies. A separate AppDomain or other protections such as payload encryption can be used to circumvent this.
Title
On the title describe
what you've fixed (or created) with this Pull Request (PR).
Summary
Insert a short (one or two sentence) summary here.
Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details
Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.
This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.
Suggested Reviewers
If you know who should review this, use '@' to request a review.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

I would hyphenate "last-executed", "HTTPS-based", and "security-critical".

@stevewhims thanks for correcting the text. It is indeed better in English.

@morganbr PTAL

The point about only respecting the last delegate is a great thing to document. However, I wouldn't suggest describing it in terms of malicious code in your process. As described here and elsewhere, AppDomains, CAS and Security Transparency should not be used to enforce a security boundary and we do not recommend loading untrusted code into your process.
For this documentation, I think it's fine to just talk about possible interactions between different components or threads adding to ServicePointManager and getting unexpected results.

For this documentation, I think it's fine to just talk about possible interactions between different components or threads adding to ServicePointManager and getting unexpected results.

@monty241 will you be updating this PR based on @morganbr comments?

This PR has become stale. @monty241 let us know if you'd like to address the feedback. Otherwise, we'll cover that for you. Thanks!

@morganbr PTAL again. Thx!
@rpetrusha can you do a quick UE review here? Thx!

Apologies, got dropped from top of mind. Happy it is now formulated OK.

Thanks for your comments @morganbr and @rpetrusha. I've addressed the feedback now.
@monty241 no worries. Thanks for creating the PR!

One nit that I should have noticed before, but I'll approve the PR.

This is what I get for removing the spaces from that sentence. LOL! 😄
Thanks so much @rpetrusha. All the comments are addressed.


Cannot use STATE2 to show trivial property on trivial function

I tried to use STATE2 to show a very trivial property on a very trivial program:
let bar () = ()

val equiv_test:
            unit
         -> STATE2.ST2 (unit * unit)
                (requires (fun h -> True))
                (ensures (fun h1 r h2 -> True))
let equiv_test () = compose2 bar bar () ()

This fails with
Unknown assertion failed
This is unexpected, because the following (more complicated) programm is successfully verified.
let bar a = a := 0

val equiv_test:
            a:ref int
         -> STATE2.ST2 (unit * unit)
                (requires (fun h -> True))
                (ensures (fun h1 r h2 -> True))
let equiv_test a = compose2 bar bar a a

This issue and #224 are the same and have to do with a known limitation in the way type inference currently works. I plan to improve it ... but it may take a couple of weeks before I get to it.
This program type-checks just fine:
val bar: x:unit -> STATE unit (fun 'p h -> 'p () h) (fun 'p h -> 'p () h)
let bar () = ()

val equiv_test:
            unit
         -> STATE2.ST2 (unit * unit)
                (requires (fun h -> True))
                (ensures (fun h1 r h2 -> True))
let equiv_test () = compose2 bar bar () ()

Without the annotation, when trying to pure an equality between the inferred type of bar (unit -> Tot unit) and the expected type in compose2 ... i.e.,  (x:a -> STATE (b x) (wp x) (wp x)) ... we get a failure, and the default to using subtyping instead. That leaves unresolved unification variables in the VC, which Z3 cannot reason about.
A temporary fix would be to not default to subtyping when the equality test fails and then we can report a better error message.
The long-term fix I have in mind is to allow the use of user-provided coercions when solving these equality constraints. But, that's more work.

This is solved, closing.

Thanks for checking and closing this. I implemented something a while back to address this and related issues but didn't have the chance to come back and check it out on the examples you have here.


Noisy result issue

Tried it for tagalog, unfortunately, the video IDs are too noisy and captures a lot of irrelevant languages.  I guess this is mostly effective for languages with exclusive scripts, like Japanese or Amharic.  For languages that mostly use Roman characters, a more sophisticated keyword choice is needed to really get valid IDs, otherwise there's a lot of irrelevant video IDs captured.  Or maybe get meta info using youtube-dl to confirm the language content.  Another way for videos with subtitles, compute perplexity against a tested language model to confirm language content.
Or am I missing something?

Hi. We are confirming the language code using youtube-dl but it is too noisy as you felt. In the Japanese case, only 10%~15% were available for ASR use. In my opinion, the possible solutions are:

Keyword cleansing: eliminating keywords duplicated with ones of other languages.
Language identification to downloaded subtitles.
Cleansing by ASR


bpf, perf: use more lightweight barriers

While perf tool itself uses full rmb(), mb() for walking the ring it
turns out they are not needed and haven't been updated in a long long
time, so smp_rmb() and smp_mb() can be used instead which are more
lightweight. Also fix fallback to be generic __sync_synchronize() for
everything non-x86.
Signed-off-by: Daniel Borkmann<EMAIL_ADDRESS>

This change is

test-me-please