Toggle splits stopped working after the last YNAB application update.

Defective ToolKit Feature Name:
Add a Toggle Splits Button
Expected Behavior
A split category should show or hide the split categories when the Toggle Splits button is clicked.
Actual Behavior
The Toggle Splits button is not displayed. There is no way to show the split categories IF I had toggled them to not show before I refreshed YNAB.
Steps to Reproduce the Behavior

Refresh YNAB to latest version (v.1.12887)

Export of Toolkit Settings (Bottom Left of Settings Page)
System Settings
ToolKit Version: 0.7.3
Operating System (Mac/Windows/Etc): Windows 7
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 2.0.2743.116 m
Other Extensions:

@dbaldon this should be fixed after our last release, lmk if it's not?

It is indeed fixed.


[SD] Add a json file for model names information.

This commit simplifies the code to identify the model name for a particular set of flags. This is achieved by introducing a json file that stores the model names information. The models are uploaded in gcloud with these names.
Signed-Off-by: Gaurav Shukla<EMAIL_ADDRESS>

This simplifies only the web part. Once https://github.com/nod-ai/SHARK/pull/685/ will land, I will update the CLI part as well.
@powderluv @PhaneeshB


zkasm_runner::ExecutionResult fileds redesign proposal.

Currently zkasm_runner::ExecutionResult looks the following way:
pub struct ExecutionResult {
    /// Path to the main zkAsm file that was executed.
    path: String,
    /// Status of the execution.
    pub status: ExecutionStatus,
    /// Error message in case the execution failed.
    pub error: Option<String>,
    /// Profiling information about this execution.
    /// Only populated for the successful executions.
    counters: Option<Counters>,
}

And I have a few questions about it:

is it true that if and only if status is ExecutionStatus::Success, than error contains None and counters contains Some?
is it true that if and only if status is ExecutionStatus::RuntimeError, than error contains Some and counters contains None?

If answer for both questions above is "yes" does we really need status field? It sounds like we already have info about status in error and counters fields. And, keeping one more such field will need to add extra attention to keep this fields consistent (be sure answer for both questions above is always "yes"). Maybe it will be better to remove this field and add method:
impl ExecutionResult {
    pub fn status(&self) -> ExecutionStatus {
        match self.error {
            Some(_) => ExecutionStatus::Success,
            None => ExecutionStatus::RuntimeError,
        }
    }
}

Or maybe even better will be to squash all this fields into just two:
pub struct ExecutionResult {
    /// Path to the main zkAsm file that was executed.
    path: String,
    /// Contains profiling information in case of successful execution and error message
    /// if execution failed
    counters: Result<Counters, String>
}


[🐛 BUG] Setting dimensions of the metric control

What went wrong? 🤔
The issue lies in how you're setting the width and height properties in the metric widget.
Taipy GUI uses a specific syntax for styling, and width and height should be defined within a style attribute.
Try modifying your page string as follows:
page = """
<|{value}|metric|style={"width": "300px", "height": "300px"}|>
"""

Or, for multiple styles:
page = """
<|{value}|metric|style={"width": "300px", "height": "300px", "font-size": "24px"}|>
"""

This should correctly apply the width and height styles to your metric widget.
Additionally, consider using units like px for pixels or % for percentage-based sizing.
For more information on Taipy GUI styling, refer to the official documentation:
Acceptance Criteria

[ ] A unit test reproducing the bug is added.
[ ] Any new code is covered by a unit tested.
[ ] Check code coverage is at least 90%.
[ ] The bug reporter validated the fix.
[ ] Related issue(s) in taipy-doc are created for documentation and Release Notes are updated.

Code of Conduct

[X] I have checked the existing issues.
[X] I am willing to work on this issue (optional)

What is this issue about?

@Yubarajhalder,
Can you provide a clearer explanation of the issue?
For now I don't see anything to work on.

Hi @Yubarajhalder
There are several problems in your control definition:

style is not a thing in Taipy 4.0.
To style elements, you can use a CSS file or the style parameter of the Page class.
You cannot create a Python dictionary in a Markdown or HTML visual element definition: Taipy uses the f-string facility that uses curly braces to indicate a Python expression to represent.
You must create a dictionary, then reference it.

So, if style existed, you would do:
style = {}
page = """

"""

And it still wouldn't work (in Taipy 4.0).
Now in Taipy 4.0, the way to do that would be:
page = Markdown(
    "<|{value}|metric|>",
    style={ "width": "300px", "height": "300px", "font-size": "24px" }
)

Please let us know if this solves this issue.


no multiple selection !

Is there a way to stop multiple selection from albums. Currently user can able to select multiple photos. Its not in requirement for me,  so is there a direct way with your library to achieve this? If you didn't implemented such thing then also let me know, in which class and method I will able to stop it? I'll surely look by my self.
Thanks a ton for such a good work !! +1

@dbotha please help in this regard.

I'm able to make it work like this,
Change this method in OLPhotoViewController.m file.
-(void) collectionView:(UICollectionView *)collectionView didSelectItemAtIndexPath:(NSIndexPath *)indexPath {
    //[self updateTitleWithSelectedIndexPaths:collectionView.indexPathsForSelectedItems];
    [self onButtonDoneClicked];
}

@khanamir agree it needs to be fixed in the library itself, sorry I've been incredibly busy as of late. I'll try get round to fixing it soon.

@dbotha, have you ever get time to update it?

This is now possible. Take a look in the last paragraph of the usage section: https://github.com/OceanLabs/FacebookImagePicker-iOS#usage


736 part 1of3. Add new status 'Started' to request.rb

Resolves #001
Description

Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update
Documentation update

How Has This Been Tested?

Screenshots

This feature appears to have been addressed on a different PR.


Are content scripts injecting multiple instances of React?

First of all, sorry if my analysis is wrong. Thanks a lot for this clear and effective template, it really makes things a lot easier!
As far as I understood, content and content-ui entries under pages are injected automatically to the website we're on. Also we have the ability to inject the content-runtime if we click to the "Click to inject Content Script" button.
I'm wondering if both bundles are injecting their own node_modules into the current document, even when they're shared across. OR do they share the same React instance (or perhaps other common modules) somehow? If they do, I think I have missed it, and I'd be curious to see where it's being done. 🙂
If marking shared node_modules as externals or similar isn't practical, I think an alternative way to go could be causing a dynamic import to run in the content/content-ui scripts via chrome.runtime.onMessage or a similar mechanism, instead of appending another IIFE.
I think this could be a good feature request, if it isn't already covered. Thank you.

Hi. Thanks for the nice comment. Each page context's bundle has its own files for the shared dependencies. I think we can try some things to fix this inefficiency, thanks for the great suggestion!


[Don't merge] Add DaoOperator for FeeHandler + KyberDAO, allow updating proxy in FeeHandler

Changes:

Add update network proxy contract in FeeHandler, not using proxy to update network address
Add DaoOperator to handle setting up data in FeeHandler and managing campaigns in KyberDao
Check same KNC token in both Staking + DAO contracts

reminder: merge with this commit - https://github.com/tranvictor/smart-contracts/commit/20e9af148a4eb847a4954670d28c6dbf1643d31e.
which inserts campaign creator and burn config setter into the contract.

Already created separate PRs for:

Add DaoOpearator
Update Proxy + Network in FeeHandler
Compare KNC address in Dao + Staking
Clean up revert messages and comments


Update benchmarks section in READEME.md to link to the bench repo

Per discussion https://github.com/quickwit-oss/tantivy/issues/1983
This is to make the information available for folks that want to learn about benchmark setup details.

thanks


[Request] Text-based QR codes

This would require a slight modification to the CSS of the 3dshacks subreddit...
.md pre>code {
    font-size: 1em;
    line-height: 1em;
    font-family: 'Courier New', monospace !important;
}

After that though, you could do text-based QR codes right in the comment
See ASCII QR Code Creator for example source code

At this point I'm only interested in images.


added personal photo

having these were part of original idea. 🏷
glad no changes to post structure required.

PR looks clean.  merging.


Added sidebar, icon package, and implemented sidebar functionality

[TASK] Add Navigation Sidebar #26 : Completion

Not all files were included, submitting new PR


add NoSleep.js in to force phone browser to stay awake

https://github.com/richtr/NoSleep.js
This is the one i found after 10 second search. maybe another one would be better.

the api is ultra simple which is great
first test if it works demo here
need to work in a-frame too

FYI in Safari 46 this will be fixed.
Prevented the display from going to sleep when playing WebRTC video tracks (r225822)

obsolete?

nope, it is needed or the phone will go onsleep after a timeout.
so during the ar experience goes to sleep..

not relevant anymore in my opinion.


[FR] Navigate through lists with enter/shift+enter when the find widget is open

#152481 is nice, but it lacks the ability to navigate through the search results with enter/shift+enter, I know "Pressing DownArrow will focus the first list element which was found." but it's less intuitive and less convenient than with enter/shift+enter.

@joaomoreno

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

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


Influxdb Cloud api documentation incorrect

https://docs.influxdata.com/influxdb/cloud/api/#section/Authentication/TokenAuthentication

Header should be Authorization, not Authentication.

This was closed by https://github.com/influxdata/openapi/pull/232


[top/dv] Fix spi device failure

When the interrupt keeps firing, PC can not go back to the main program,
Changed to disable the interrupt in interrupt routine instead of in the
main program
Thanks Sri for helping debug this issue.
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

ah Eunchan, honestly i dont think this is important to change now,
especially if there are other things to do.
I feel like this portion of the SPI functionality is also mostly verified?
(i could be wrong). But we could just file an issue for future improvement
if we think that's a good idea.
On Mon, Nov 8, 2021 at 8:27 AM Eunchan Kim @.***> wrote:

@.**** commented on this pull request.
In sw/device/tests/sim_dv/spi_tx_rx_test.c
https://github.com/lowRISC/opentitan/pull/9077#discussion_r744891465:

@@ -107,6 +107,11 @@ void handler_irq_external(void) {
case kTopEarlgreyPlicIrqIdSpiDeviceTxUnderflow:
spi_device_irq = kDifSpiDeviceIrqTxUnderflow;
CHECK(expected_irqs[spi_device_irq], "Unexpected TX underflow interrupt");

 // TxUnderflow will fire every cycle, so disable this interrupt once fired.

Thanks for the info. I agree if an event occurs and SW does not clear the
cause of the event, HW would be better to notify SW once.
In SPI_DEVICE, the tx underflow condition occurs at every byte when FIFO
is empty. The logic does not constantly fire the interrupts at every cycle
but every 8 SCK cycles. So, to mute the interrupt event, I think logic
needs to be change to check the ASYNC_FIFO read depth too and does not
assert twice until the read depth becomes non-zero.
Does it sound good to everyone? Then I can create a PR to change the
interrupt behavior (maybe RX overflow too?)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/9077#discussion_r744891465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSQD7BK6LWLBTOLZSKTUK727TANCNFSM5HOYJS5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

ah Eunchan, honestly i dont think this is important to change now, especially if there are other things to do. I feel like this portion of the SPI functionality is also mostly verified? (i could be wrong). But we could just file an issue for future improvement if we think that's a good idea.

Filed an issue at #9106
I think this underflow interrupt is mostly verified. If we want to change the design, it's great that we also consider other peripherals.


Incorrect CC0029 message

In the following code:
Public Sub Dispose() Implements IDisposable.Dispose
	' Do not change this code.  Put cleanup code in Dispose(disposing As Boolean) above.
	Dispose(True)
	GC.SuppressFinalize(Me)
End Sub

there is a message CC0029, saying that GC.SuppressFinalize should be called inside Dispose method. It is!..

@mc0re @code-cracker/owners This was already fixed on #452.

Ok, I'll wait for the new release then :-)


Upload docs to pythonhosted / pypi

Pypi now has a place to host docs for python packages. It would be good to upload our docs there.
You can now host documentation at http://pythonhosted.org/control. To upload documentation, prepare a .zip file that is unpacked into this URL. Only static pages are supported. The zip file must have a top-level "index.html".

Version 0.7.0 is now available at http://python-control.readthedocs.org


Jsonnet + VSCODE + autocomple

Hi to all.
Can somebody help with autocomplete in vscode? I install all plugins (and autocomplete works in kubernetes-prometheus folder) but I can't import new library with autocomplete

Ok...Little investigation. I think the autocomplete problem lies somebody near imports in the library. For example:
local d = (import 'doc-util/main.libsonnet'),

And error:
Can't find path 'doc-util/main.libsonnet'. If the file is not in the current directory, it may be necessary to add it to the 'jsonnet.libPaths'. If you are in vscode, you can press 'cmd/ctrl-,' and add the path this library is located at to the 'jsonnet.libPaths' array

But I don't have doc-util installed at all)

This is a support issue, not really part of this project.
Feel free to reach out to the Tanka community via GitHub Discussions or in #tanka on Grafana Slack.

This is a support issue, not really part of this project.
Feel free to reach out to the Tanka community via GitHub Discussions or in #tanka on Grafana Slack.


Adds a 'local' publisher

Adds a new publisher type that allows for publishing to the local disk of the compute node. This allows users to test out the publisher functionality without needing to set up a remote storage service making it easier to get started with Bacalhau.  Currently this implemention is does not include features such as content lifetimes, or secure access, and so is not recommended for production use yet. This publisher is useful for testing and development purposes.
The local publisher comprises two parts. The publisher is responsible for taking the output from a job, compressing it and moving it to a location where it can be access by the second part.  This part is the HTTP server which delivers the content back to the caller.
By default the HTTP server listens on port 6001, but this can be changed by setting the --local-publisher-port flag on the compute node.  The HTTP server will serve the content from the directory specified by the local-publisher-directory flag.  If this is not set, the HTTP server will serve the content from a subdirectory of the configured bacalhau storage directory. Finally the --local-publisher-address flag can be used to specify the address that the HTTP server listens on.  The default for this varies by environment, in test and development environments it uses localhost, in production environments it uses public to obtain a public address. Of course, you can set these values in config in preference to using the command line flags if the defaults are not suitable.

Minor comments. How are you going to make this the default publisher instead of IPFS? Are you going to use the newly added --default-pubisher flag in the requester side?

Yes.
I've reverted the firewall change here, and will set up the default for the public network in a follow on PR so it is easy to revert when we don't want it.

On second pass this is more complicated than I thought - why do we need a server for the compute node to write data to it's disk?

The server is necessary for clients to be able to obtain content from the compute nodes. Once the data is published, the user is given a URL to this server.

I see, understood. Then my only other comment I'd like more discussion on is the compression by default

What is your concern with compression? We need to archive the data to serve it in a single response. Compression sounds obvious to me.

As stated here: https://github.com/bacalhau-project/bacalhau/pull/3431#discussion_r1489906299

As a user publishing the result of a job, I'd be caught off guard by the published data being a tarball when that wasn't what I published. And if I have subsequent jobs reading this data (file inputs) I'll then need to handle the untar-ing in the job, which is unnecessary toil imo.

As a user, if I wanted to publish a tarball I'd go-ahead do that explicitly, but I would never want (or expect) the application to do that without me instructing it to.

We need to archive the data to serve it in a single response

Do we really need that? Or is it just convenient? Gut check says it's the latter.
If we want a single response surely we could dynamically packaged, transferred, and unpacked the results instead? Isn't this what we already half-do in the case of get for S3 - decompress the data on write?

bringing back the discussion we had with S3 publisher. How the publisher stores the data doesn't have to match what the user generated, specially that bacalhau get will handle the decompression. Users are not expected to access data directory, and each publisher should store data in its own efficient way for both storage and retrievals.
I hope we conclude this discussion here as a significant time was already spent while discussing S3 downloader

@frrist have made the relevant changes, and created an issue for a wider discussion about compressed results.  Are you okay for me to merge this now?

How the publisher stores the data doesn't have to match what the user generated

In some cases, absolutely. But surely the user deserves at least one option where they can store the data as they wrote it. Right now we lack this! Let's not make this sound like a binary decision - it's not.

Users are not expected to access data directory

This just isn't true - maybe for the users you're thinking of?
I think we need to shift our view of bacalhau. It's not just a tool for running a single task and producing data for users to grab. It's about chaining together multiple tasks, where each one takes the output of the previous and adds its own layer. It's a continuous process of tasks building on each other's outputs, not just a one-off data dump.


Add parameter cursor in functions

I saw that there is a function to get the cursor:
    def _get_cursor(self, obj: dict):
        if cur := find_obj(obj, lambda x: x.get("cursorType") == "Bottom"):
            return cur.get("value")
        return None

but I also noticed that even if you get the cursor - pass it to the function through a parameter - it's impossible, only if you do it with the help of the _gql_items function, but I would like to see an implementation method in this library - simpler
Is it possible to add a cursor to the parameter and return in the function (eg followers) - the last received cursor or just return the cursor to each user?

Hi. Need to think about how this can be implemented.
Each api call function in API class have a second method ended with _raw. Those functions returns plain httpx.Response objects, so you can get last token yourself from it.
Not raw function was designed to get feed of parsed objects (tweets / users) for further processing.


KtFmt infers with KotlinX Knit

When using KtFmt (Google style) together with KotlinX Knit tool it reports incorrectly formatted code for markers for Knit inside KDocs.
Execution failed for task ':spotlessKotlinCheck'.
> The following files had format violations:
      src/main/kotlin/com/github/nomisRev/kafka/AdminSettings.kt
          @@ -54,8 +54,7 @@
           ·*···}
           ·*·}
           ·*·```
          -·*·<!---·KNIT·example-admin-01.kt·-->
          -·*·<!---·TEST·lines.isNotEmpty()·-->
          +·*·<!---·KNIT·example-admin-01.kt·-->·<!---·TEST·lines.isNotEmpty()·-->
           ·*/
           public·fun·Admin(settings:·AdminSettings):·Admin·=·Admin.create(settings.properties())

  Run './gradlew :spotlessApply' to fix these violations.

Using KtFmt 0.30.0 through Spotless Gradle plugin.

Thanks for the report. Does Knit require these two <!--- --> to be in separate lines?

@cgrushko yes
Directives in markdown files must always start at the beginning of the line and have the following general format for single-line directives:
<!--- <directive> [<parameters>] -->

or the following format for multi-line directives:
<!--- <directive> [<parameters>]
<text>
-->

We're planning to integrate a new kdoc formatting engine: https://github.com/facebook/ktfmt/issues/339
IIRC we checked that it solves this issue specifically.

@cgrushko I think there was some regression here. When currently running KtFmt over my project it completely changes how my KDoc is formatted breaking Knit support completely.
I.e.
 * <!--- INCLUDE
 * import arrow.core.raise.ensure
 * import arrow.exact.Exact
 * import arrow.exact.ExactError
 * import arrow.exact.exact

becoming
 * <!--- INCLUDE
 * import arrow.core.raise.ensure import arrow.exact.Exact import arrow.exact.ExactEither import
 * arrow.exact.ExactError import arrow.exact.exact import arrow.exact.exactEither

Also @see seems to be moved inside of a ``kotlin code-fence. Sadly this prevents us from using KtFmt on all my OSS projects again 😞

v0.47 formats like this
/*
<!---·KNIT·example-admin-01.kt·-->
<!---·TEST·lines.isNotEmpty()·-->

 * <!--- INCLUDE
 * import arrow.core.raise.ensure
 * import arrow.exact.Exact
 * import arrow.exact.ExactError
 * import arrow.exact.exact
 * -->
*/

Looks fixed to me, but I don't know much about Knit so feel free to reopen if I'm missing something


dataset

Hello, author, through your paper, I found that you divided 1251 pieces of data into training, verification and testing for experiments, rather than taking part in the 21 challenge to obtain 251 verification sets for experiments. Is this a convincing division?

Hello, we evaluated all the methods using the same data split for a fair comparison. Yes, in our future works, we hope to evaluate results using Brats continuous evaluation platform.

Hello, I randomly divided the data set for testing, and found that the TC index is greater than WT during the evaluation. Is this reasonable.

We look forward to your reply

Hello, What are the values you got?

Hello, I am a test set randomly divided by myself. During the evaluation, the TC index is about 0.001-0.005 higher than the WT index. The same is true for tests on other models. Is this because of my randomly divided test set? I look forward to your reply.

For example, wt=0.9086, tc=0.9098

yes, that might be the case.

OK, thank you very much. I used the same test set to test on multiple models. The TC index is greater than the WT index. Then I can judge whether each model is good or bad by the size of this index

We look forward to your reply

OK, thank you very much. I used the same test set to test on multiple models. The TC index is greater than the WT index. Then I can judge whether each model is good or bad by the size of these coefficients?

yes, You can evaluate all three tumor classes (i.e., ET, TC, and WT) and get the average dice coefficient of the three classes and rank the models.

Hello, author, are the other comparative experimental results shown in your paper all obtained under the same epoch?

We look forward to your reply

Hello, not under the same epoch. Results were shown for the best model saved after evaluating model performance on the validation dataset.

Hello, I found that many models run hundreds of epochs and save the last model. At this time, how can we judge that this is the best model? In this case, it is difficult to judge how many epochs are the best performance in the comparative experiment.

In this case, it is difficult for us to obtain the optimal performance of each model in how many epochs. Can we compare the performance under the same epoch?

During training, we use a validation dataset to identify the best model that would be performed well for unseen test cases. It is the recommended method in the machine learning process. Once you save the best model (You can evaluate your model after every epoch during training and save the model), then you can evaluate your model on the test dataset. That's why we split our dataset into, training, validation, and test. If you set an epoch, say max_epoch=100, how can you say that your model is converged or not? Some models can be overfitted after the 70th epoch, so getting the model at the 100th epoch for all the baseline models will not be a fair comparison.

if you are following someone's paper, then its better to do experiments in their way.

Well, thank you very much.


The results about vanilla ViT with adversarial adaptation.

Hi, thanks for releasing the open-source project of transformer-based UDA!  May I ask you some questions?
I tried to reproduce the Baseline result, which denotes vanilla ViT with adversarial adaptation. Differently, I fixed the bs as 32, the input size as 224 * 224. Accordingly, I got the results on the setting of office31, W@A， D@A.

[1] Safe Self-Refinement for transformer-based Domain Adaptation (https://arxiv.org/pdf/2204.07683.pdf)
Obviously, [1] uses the Timm library and gets a very strong baseline. Have you thought about switching to Timm to improve performance? It's hard to compare because everyone implements baseline in different ways and the results vary greatly.:sweat_smile:
Additionally, I had a problem with the training baseline mentioned before.

Model performance rises and then deteriorates rapidly. The training parameters are as follows: --beta 0.1  --gamma 0. --theta 0.
Have you ever encountered such a problem? Can you give me some advice?

Hi, I am sorry for the late reply. I was extremely busy in the last two weeks (moving and onboarding).
Q1: Right, the current issue of ViT-based UDA is that existing work uses different baselines with quite different implementations. In terms of timm, it can be an option, but the main contribution of this work is to provide a baseline for ViT-based UDA. Therefore, we actually haven't spent a lot of time in improving the performance.
Q2: Yes, I did observe the same issue in some datasets (maybe one or two). Have you tried any training tricks in addressing this problem? Seems like an overfitting problem.

Thanks for your answer! I didn't delve into why vanilla ViT with adversarial adaptation would have the phenomenon depicted in fig. But according to my personal understanding and observations, the reason may be the problem of degeneration of the adversarial training process (i.e., the game is out of balance), so when some other constraints are added, this problem will be solved.

This might be a reason of that loss curve. Right, training with adversarial loss is a little bit tricky.

您好，我能请教您一个问题吗？您有vision transformer只添加对抗域适应的代码吗？可以分享吗？我最近在研究这个方向，可是可以借鉴的代码实在太少，我对代码的学习还远远不够，如果不能分享给我，您可以告诉我这篇论文的model部分区分源域和目标域数据了吗？

@fanbowen232218094636-spec
you can refer to https://github.com/tsun/SSRT/blob/master/main_ViT_baseline.office_home.py (CVPR2022) which provides the vanilla ViT with adversarial adaptation.
Once the code is run through and understood, you will soon be able to strip out the baseline from some projects yourself

@fanbowen232218094636-spec<EMAIL_ADDRESS>

哈哈哈哈哈好的

您好，我能请教您一个问题吗？您有vision transformer只添加对抗域适应的代码吗？可以分享吗？我最近在研究这个方向，可是可以借鉴的代码实在太少，我对代码的学习还远远不够，如果不能分享给我，您可以告诉我这篇论文的model部分区分源域和目标域数据了吗？

Yes, we do have this code. Can I send to your email address?
Yes, the global domain discriminator tries to distinguish the source domain from the target domain.


Needs ForEach Polyfill

Does not support IE 11, get the error message "Object doesn't support property or method 'forEach'".  Adding react-app-polyfill and other polyfill offerings to project do not help... the issues is inside the carousel libraries where the polyfill libraries are not being imported.
Looks like this is specific to SPfx (SharePoint Framework) webparts.  The carousel works when standing on it's own, but put it into a SharePoint Framework webpart and the forEach calls all blow up internally to the infinite-react-carousel node_module.

Hi @clayo,
What is your package version?

Whatever the latest is.  Did an npm install on it yesterday.
Sent from my iPhone
Clay Osborn
Software Development Team Lead
direct<PHONE_NUMBER> | office<PHONE_NUMBER>
email<EMAIL_ADDRESS>| web redlevelgroup.com
Modern Workplace Webinar
On Feb 18, 2020, at 7:20 PM, Hans<EMAIL_ADDRESS>wrote:

Hi @clayohttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclayo&data=02|01|cosborn%40redlevelgroup.com|3c7e6c0b0a784f019f4808d7b4d1678a|ac56cd7ed609407a9eb4c77b623198f3|0|0|637176684243823322&sdata=3VYuHL8UVzvYgC%2B78TKcpZzNcfyY%2BQIfbIPrMt8X3TQ%3D&reserved=0,
What is your package version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fg787543%2Finfinite-react-carousel%2Fissues%2F14%3Femail_source%3Dnotifications%26email_token%3DADGU43SSLNXNJ6FKFOL6KSTRDR3JRA5CNFSM4KXIS2KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMF3R6A%23issuecomment-587970808&data=02|01|cosborn%40redlevelgroup.com|3c7e6c0b0a784f019f4808d7b4d1678a|ac56cd7ed609407a9eb4c77b623198f3|0|0|637176684243823322&sdata=nan6E4aAiLB2c4uPKw66wHa8Q5P6EtdohQp3%2Fk%2BLEis%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADGU43SKGD66WMMRC4O77XLRDR3JRANCNFSM4KXIS2KA&data=02|01|cosborn%40redlevelgroup.com|3c7e6c0b0a784f019f4808d7b4d1678a|ac56cd7ed609407a9eb4c77b623198f3|0|0|637176684243833320&sdata=kphQJ3n3A4sZqm1cGuc0jiscV6gvGmlhK4Mh4K0Km0w%3D&reserved=0.

Ok, I remove all of forEach. I will release later

I already release the new version on npm, please try again, Thanks.

That did the trick!!  That was some awesome response time.  Thank you!
Clay Osborn
Software Development Team Lead
direct<PHONE_NUMBER> | office<PHONE_NUMBER>
email<EMAIL_ADDRESS>| web redlevelgroup.com
Modern Workplace Webinar
From: Hans<EMAIL_ADDRESS>Sent: Tuesday, February 18, 2020 8:50 PM
To: g787543/infinite-react-carousel<EMAIL_ADDRESS>Cc: Clay Osborn<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [g787543/infinite-react-carousel] Needs ForEach Polyfill (#14)
I already release the new version on npm, please try again, Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fg787543%2Finfinite-react-carousel%2Fissues%2F14%3Femail_source%3Dnotifications%26email_token%3DADGU43R667WK66KPBLDIE7TRDSF3LA5CNFSM4KXIS2KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMGBFHQ%23issuecomment-587993758&data=02|01|cosborn%40redlevelgroup.com|0b49a01e05a64694fb5208d7b4ddfd57|ac56cd7ed609407a9eb4c77b623198f3|0|0|637176737848760670&sdata=v4vmvez9ZDeR0kGfITjjdXPscsiUcmDntm%2BnBYBAIs0%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADGU43QOOFJY7S4U6TM2J73RDSF3LANCNFSM4KXIS2KA&data=02|01|cosborn%40redlevelgroup.com|0b49a01e05a64694fb5208d7b4ddfd57|ac56cd7ed609407a9eb4c77b623198f3|0|0|637176737848760670&sdata=6L54yP2IjBBTzgcSERFkL4a3wvYZUqOvO6nAJQahw6c%3D&reserved=0.


Add leave button to lobby and game screens.

If a player leaves during game -> abort the game for all players
if a player leaves the lobby -> remove from game, update playerlist for everyone else

closed with commits: f8ce2af 0653d69 3a173ee 26867a5


[Issue]: GroupChat keeps calling all the available agent and doesn't stop even after output from one agent is generated

Describe the issue
When I have multiple agents configured in a group chat. If the question is very specific to one agent [say text to sql], the group chat manager invokes other agents as well even though they are not required.

Steps to reproduce
groupchat = GroupChat(
agents=[user_proxy,coding_assistant,simulator_worker_agent,text_2_sql_agent],
messages=[],
max_round=24
)
manager = GroupChatManager(groupchat=groupchat, llm_config=llm_config)
Screenshots and logs
No response
Additional Information
AutoGen Version: 0.2.9
Python 3.11

Can you show your llm_config for the group chat manager? Which model are you using?
One thing we are working on is graph-based group chat, which allows user to add constraints to the speaker transition: #857

[{
"model" : "minerva-llm",
"api_key": "key",
"base_url": "https://endpoint.openai.azure.com/",
"api_type": "azure",
"api_version": "2023-07-01-preview"
}]
groupchat = GroupChat(
agents=[user_proxy,fetch_schema_agent,simulator_agent],
messages=[],
max_round=24
)
manager = GroupChatManager(groupchat=groupchat, llm_config=llm_config)
We are using azure gpt4 turbo model

How are the text2sql and rag agent defined? Is the "description" field used to describe their roles?

closing as stale - missing info from OP. please reopen I you want to update


Dont throw exception but display unknown mission

Prepares #1134

Please somebody review

@Geosearchef would u mind reviewing?


7645re/added conditional elif block

https://github.com/ForNeVeR/Cesium/issues/517
this is still a draft, there are problems with naming, but I'm still more concerned about finding more corner cases to fully test the code for performance and optimization

Not important, but maybe #elifdef and #elifndef can be added here as well?

I would suggest against extending the task definition. Let's focus on immediate issues and add other features later.
In particular, #61 is blocked on #elif specifically.

Okay, let's finish this task and then create an issue marked area:standard-support?


Added travis CI integration

@ryanleary do you mind enabling travis CI for this project? Seems I don't have the privileges to do so. Going to finally make this a pypi build so people can install it from pypi!


Managing optional dependencies

Springtime currently offers a nice, streamlined way to download data from several sources. It was designed to be extensible, in the sense that it should be easy to add more data sources. However, new data sources typically come with additional requirements (e.g. #54) and licenses (e.g. #63). This could eventually render the whole package cumbersome to work with.
It would be nice if we could separate the package into a core module with optional data source subsets. There are two main options:

Import the springtime core package into new datasource subset packages.
Import optional datasource subset packages into springtime. This would be my preferred option, but it requires a mechanism to register subset packages into the main tool.

Option 1 might lead to duplication and it moves away from the ideal of having a one-stop-shop for all your data workflows. This is why I would prefer option 2. To make that work, we need a mechanism to register subset packages into springtime.
Perhaps something like:
pip install springtime
springtime register springtime-phenology, springtime-stemmus-scope, springtime-rsdat

Where the last command should:

(pip) install the requirements of the subpackages
tell springtime about the additional datasets in the subpackage (perhaps through a config file?)
...

Then we can make springtime, on load, search for Dataset objects in the packages specified in the config file, and add those to the list of Datasets it can work with.

Flake8 has a very comprehensive documentation on writing plugins. We could use something similar here.
There are several plugin frameworks for Python, e.g. Pluggy, used for pytest. But we could also use entrypoints directly, as they're part of the standard library and we might not need much more functionality.


[BUG] Footer and card is overlapping

Description
The card is overlapping with fotter footer
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Yes, we need a responsive grid view of the container having cards for any number of cards.
Someone who has worked on a similar thing earlier can easily fix this.
Assigning you since you've shown interest to solve it.


Fixes KVM module isolated build

The build has been fixed by removing the relative path to the parent model. Now this plugin can be built in isolation, as long as the parent artefacts are present in the m2 local repo.

Ping @miguelaferreira @borisroman
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ cloud-plugin-hypervisor-kvm ---
[INFO] Installing /data/git/ekholabs/cosmic-plugin-hypervisor-kvm/target/cloud-plugin-hypervisor-kvm-5.0.0-SNAPSHOT.jar to /root/.m2/repository/cloud/cosmic/cloud-plugin-hypervisor-kvm/5.0.0-SNAPSHOT/cloud-plugin-hypervisor-kvm-5.0.0-SNAPSHOT.jar
[INFO] Installing /data/git/ekholabs/cosmic-plugin-hypervisor-kvm/pom.xml to /root/.m2/repository/cloud/cosmic/cloud-plugin-hypervisor-kvm/5.0.0-SNAPSHOT/cloud-plugin-hypervisor-kvm-5.0.0-SNAPSHOT.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 17.897s
[INFO] Finished at: Wed Feb 10 06:37:29 GMT 2016
[INFO] Final Memory: 33M/329M
[INFO] ------------------------------------------------------------------------
[root@cs1 cosmic-plugin-hypervisor-kvm]# pwd
/data/git/ekholabs/cosmic-plugin-hypervisor-kvm
[root@cs1 cosmic-plugin-hypervisor-kvm]#

go build


nspr: 4.34 -> 4.34.1

Description of changes
No announce on the list, no bugs targeted for this release. :shrug:

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Looking at the mercurial history, the only change that isn't just changing versions seems to be Bug 1769293 - Add API PR_GetPrefLoopbackAddrInfo. r=rrelyea.


Support time zone

Have a time zone ID in the config file to specify how to read the cron files correctly.

We now support timezone per team and globally! Once the rewrite is done you'll be able to see it


MGMT-11124: support IMAGE_SERVICE_BASE_URL env var

Description
As a preparation to remove the support for IMAGE_SERVICE_SCHEME:

Added support for IMAGE_SERVICE_BASE_URL when should be used by all deployments instead of IMAGE_SERVICE_SCHEME/HOST.
When IMAGE_SERVICE_BASE_URL is specified, it should be used instead of IMAGE_SERVICE_HOST.
The current IMAGE_SERVICE_HOST var is deprecated for now, until IMAGE_SERVICE_BASE_URL is exclusively used.

How was this code tested?
Updated tests and verified on local env.
Assignees
/cc @carbonin
Checklist

[x] Title and description added to both, commit and PR
[x] Relevant issues have been associated
[x] Reviewers have been listed
[ ] This change does not require a documentation update (docstring, docs, README, etc)
[x] Does this change include unit tests (note that code changes require unit tests)

/hold

Should this be WIP until https://github.com/openshift/assisted-image-service/pull/88 is merged since this includes both commits?
/hold

/unhold


Fix Double PathFinder

Motivation and Context
Due to legacy reasons, we create a pathfinder object in the python sim after the cpp sim has been initialized. This means that the navmesh loaded via the scene dataset get's unloaded!
How Has This Been Tested
Via tests. We have a lot of tests that require the navmesh to be loaded correctly and I do not see python saying it loaded the navmesh, so cpp must be doing it correctly!
Types of changes

Bug fix (non-breaking change which fixes an issue)

Close also resets the config to its default value so that'll be pretty edge-casey, but yeah, I will do the same thing as sceneManager_ to avoid a potential segfault.


Migrate existing tests to NUnit test framework

All tests that are already written should be moved into NUnit.
@Regis128 is responsible for .NET interfaces tests
I am responsible for all other already existing tests

 Migrate tests to NUnit

Under review as #39

Closed as 8e61747


swift和objective-c混编项目引入此库后打断点后直接闪退。

Btl was compiled with optimization - stepping may behave oddly; variables may not be available.
Message from debugger: The LLDB RPC server has crashed. The crash log is located in ~/Library/Logs/DiagnosticReports and has a prefix 'lldb-rpc-server'. Please file a bug and attach the most recent crash log.
查看carsh log显示信息如下:System Integrity Protection: enabled
Crashed Thread:        8  RPC packet thread for client tid 006737a7 (6764455)
Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY
Application Specific Information:
Stack dump:
0.	While loading conformances for 'BTLRemotePushHelper' in module 'Btl'

While reading inherited conformance for type 'BTLRemotePushHelper'
While cross-referencing conformance for 'NSObject' in module 'ObjectiveC'
While ... to 'JRPersistent' in module '__ObjC'
If you're seeing a crash here, check that your SDK and dependencies are at least as new as the versions used to build 'Btl'

abort() called
大概推测应该是jrdb问题，但是不知道如何解决，希望作者能个方案帮我解决下问题。best wishes!

你这个是在release模式下的崩溃，可能很多信息会被隐藏，能否在debug模式下看看会不会出问题


Read Error

Hello I found out an error while doing this:
$client = new Sabre\DAV\Client($apis["cloud1"]);
$response = $client->propfind($user["token"], array(
   '{DAV:}displayname',
   '{DAV:}getcontentlength'
));

Below is the issued error:
[Fri Jan 13 18:09:23 2017] [error] [client X.X.X.X] PHP Parse error: syntax error, unexpected 'finally' (T_STRING), expecting catch (T_CATCH) in /vendor/sabre/xml/lib/Reader.php on line 79

Looks like your php version is to old?

I have 5.4 but I can use the lib to write to DAV... I'll try using sabre 3.0 and post back.

Still having the same issue having switched back to 3.0.0.
Here is the composer entry:
"sabre/dav": "3.0.0"
sabre/event is at 2.0.2 and sabre/dav is at 3.0.0
I'm using the new keyword in the code, since i'm calling it from a class function. Not sure if that is the reason its having issues...

I hope you're using sabre/dav 3.0.9, and not 3.0.0. Are you getting the exact same error with that version?
By the way, you should really just be using supported PHP versions. 5.4 is out of support for a long while now. There's no excuse!

@evert yes I am.
You are absolutely right about using supported versions, but unfortunately for me, there is no officially supported version of PHP7 at Openshift. Although there is a community version for PHP7, it does not receive automatic security updates and does not have Apache with it.

How did you install sabre/dav?

My composer.json looks like this:
{
"require": {
"altorouter/altorouter": "1.1.0",
"sabre/dav": "3.0.9"
}
}
At the sabre/xml composer.json i see this:
"require" : {
"php" : ">=5.5.5",
"ext-xmlwriter" : "",
"ext-xmlreader" : "",
"ext-dom" : "*",
"lib-libxml" : ">=2.6.20",
"sabre/uri" : ">=1.0,<3.0.0"
},

In that case the wrong sabre/xml got installed. Try reinstalling it. This won't work correctly unless you are actually running this on PHP 5.4.

I see now that sabre/xml 1.5.0 is installed. Will try and post back results.

Still same error.

Please do some more investigation. You should be getting 1.4.x which does not use finally.

1.5.x won't even install on PHP 5.4.

I believe this is a composer issue. I will try to force sabre/xml to 1.4.x

Composer is behaving as expected. The problem is more likely that you are running composer on a newer version of PHP later than 5.5, but then you run the actual application runs on PHP 5.4.

I just got into that myself. but thanks for pointing me that :D


Add Tutorials and nbsphinx to documentation

Pull Request
What does this PR do?
Adds a Tutorials section to the documentation with four Jupyter Notebooks, and updates the documentation's generation configuration to support nbsphinx, a Jupyter Notebook html renderer. Closes #506.
Notes:

Replace (issue) above ↑↑↑ with the issue this PR closes to automatically link the two.
This must be done when the PR is created.
Add multiple Closes #(issue) as needed.
If this PR is work towards but does not close an issue, simply tag the issue without mentioning Closes.

Description
Describe the changes proposed by this PR below to give the reviewer context below ↓↓↓
(description)
Request Review
Be sure to request a review from one or more reviewers (unless the PR is to an unprotected branch).
Versioning
When opening a PR to make changes to PrimeQA (i.e. primeqa/) master, be sure to increment the version following
semantic versioning.  The VERSION is stored here
and is incremented using bump2version {patch,minor,major} as described in the (development guide documentation)[development.html].

[ ] Have you updated the VERSION?
[ ] Or does this PR not change the primeqa package or was not into master?

After pulling in changes from master to an existing PR, ensure the VERSION is updated appropriately.
This may require bumping the version again if it has been previously bumped.
If you're not quite ready yet to post a PR for review, feel free to open a draft PR.
Releases
After Merging
If merging into master and VERSION was updated, after this PR is merged:

[ ] Create a release from the master with version equal to VERSION
[ ] Not merging to master or VERSION not updated

Checklist
Review the following and mark as completed:

[ ] Tag an issue or issues this PR addresses.
[ ] Added description of changes proposed.
[ ] Review requested as appropriate.
[ ] Version bumped as appropriate.
[ ] New classes, methods, and functions documented.
[ ] Documentation for modified code is updated.
[ ] Built documentation to confirm it renders as expected (see here).
[ ] Code cleaned up and commented out code removed.
[ ] Tests added to ensure all functionalities tested at >= 60% unit test coverage (see here).
[ ] Code cleaned up and commented out code removed.
[ ] Release created as needed after merging.

I tried running the DPR notebook on collab, but the collab environment is throwing conflict errors for pip install. We need to get the PrimeQA setup compliant with collab.

@jdpsen as discussed you need to get the whl file either from the branch or let this merge and then try again and it should work.


Transcribing file with Brain class?

Hi,
I've been following the pretraining tutorial: https://colab.research.google.com/drive/1LN7R3U3xneDgDRK2gC5MzGkLysCWxuC3?usp=sharing .
In this notebook, you do transcription on the pretrained model stored in Huggingface.
from speechbrain.pretrained import EncoderDecoderASR
asr_model = EncoderDecoderASR.from_hparams(source="speechbrain/asr-crdnn-rnnlm-librispeech", savedir="./pretrained_ASR")
asr_model.transcribe_file("./LibriSpeech/dev-clean-2/1272/135031/1272-135031-0003.flac")

Later in the tutorial, you create a brain class to wrap this model for further finetuning.
I wonder if it's possible to transcription with the brain object instead of the asr_model? For example, I had to fine-tune the model with Brain class first and then I would like to do the transcription.
Is that possible? Or do I need to save the model and load it again with EncoderDecoderASR class?

Ok thanks!


Vertical Tree Problem

I'm trying to create a basic vertical tree, ie.
 A
 |
 v
 B
 |
 v
 C

Eventually there will be an actual tree (ie. C will go to D, E, etc.) but so far I can't even get the above to work.  Even when I use a TournamentTree or OrderedTree, the order I pass doesn't matter; I always get:
 A
 |
 v
 B     --->   C

Well, technically the order does affect what node goes in positions A/B/C, but the point is that I can only make an L-shaped graph, not a vertical one.
Is there any way to tell Dracula to use one vertical level per single-edge pair of nodes (although obviously when there's an actual branch I still want all children of that branch to be on the same vertical level)?

This would be just by writing a new layouter... have a look at the existing ones to see how it's done!


chore: remove import * statements from bqplot.ipynb

Description
A tiny PR that closes #203; removes the import * statements from the bqplot example notebook. They are discouraged since they mix-up the namespace with too many imports. We could consider adding https://github.com/asmeurer/removestar as a pre-commit hook, but I don't think it's needed too much because this was the only failing example we had.

This has been fixed in main, no longer needed


GET /sites/{site-id}/drives/{drive-id}/items/ returns 500 error code generalException

Describe the bug
I'm trying to enumerate through all files available on all SharePoint drives.
When qurying for all DriveItems in a SharePoint drive using GET /sites/{site-id}/drives/{drive-id}/items/, a 400 error is returned telling me that filter is required. I can't find any documentation that mentions that GET /sites/{site-id}/drives/{drive-id}/items/ requires the filter query option.
So I supply a filter that will return all drive items like so:
https://graph.microsoft.com/v1.0/sites/{site-id}/drives/{drive-id}/items?$filter=name+ne+'x'

Which returns a 400 error like so:
{
    "error": {
        "code": "invalidRequest",
        "message": "Expand cannot be null or empty.",
        "innerError": {
            "date": "2022-09-16T06:10:41",
            "request-id": "3f9fb196-7bfd-42df-8128-19f454e0566e",
            "client-request-id": "6ed0a895-f84e-88d2-5707-5e776177fb04"
        }
    }
}

So I add a random expand:
https://graph.microsoft.com/v1.0/sites/{site-id}/drives/{drive-id}/items?$filter=name+ne+'x'&$expand=createdByUser

and it gives me this 400 error:
{
    "error": {
        "code": "invalidRequest",
        "message": "Expand either analytics($expand=allTime) or analytics($expand=lastSevenDays) is required.",
        "innerError": {
            "date": "2022-09-16T06:13:29",
            "request-id": "000b2108-b8c3-45e2-9c37-99a26ffd465a",
            "client-request-id": "ba57ccbb-43f8-3389-fbe8-dbd15faef889"
        }
    }
}

So I change it to use the required expand:
https://graph.microsoft.com/v1.0/sites/{site-id}/drives/{drive-id}/items?$filter=name+ne+'x'&$expand=analytics($expand=lastSevenDays)

Which results in this 500 error:
{
    "error": {
        "code": "generalException",
        "message": "General exception while processing",
        "innerError": {
            "date": "2022-09-16T06:14:33",
            "request-id": "5cbdf340-ee85-4578-9437-6924de6312bb",
            "client-request-id": "07c72c89-32d8-0e66-7d1d-3e10ab20cb30"
        }
    }
}

To Reproduce
Steps to reproduce the behavior:

Go to https://developer.microsoft.com/en-us/graph/graph-explorer
Use API version v1
Follow the steps I did above^

Expected behavior
I want all driveItems to be returned to me when I call GET /sites/{site-id}/drives/{drive-id}/items/ with or without query options.

Hey @vruss,
With that query signature, I believe you would have to specify an item id... hence the Bad request error.
At least, that's what I see in the docs provided here.

Alternatively, you can get all items in a user's drive with the GET me/drive/root/children endpoint
More is explained here: https://learn.microsoft.com/en-us/graph/api/driveitem-list-children?view=graph-rest-1.0&tabs=http
I can't find anything else in the docs that point to obtaining all driveItems in any other way.

Sorry we can't be of more assistance as we handle only Graph Explorer related queries and not issues pertaining to the Graph API.
If you need further assistance, like specific to OneDrive or Sharepoint Sites, you could post a question here: https://learn.microsoft.com/en-us/answers/products/graph, and the relevant team will help you out.
If you have more questions related to Graph Explorer let me know.


License transition is not mentioned anywhere I could find

Projects which join the ASF must provide their source under the AL 2.0.
This should be made clear in the FAQ. I think it needs to be covered under each of the following sections:

What is special about The Apache Way?
What does “Direct to TLP” entail?
Is Apache Petri right for you?

I agree this could be better explained.

Please see the updates to petri.apache.org/faq and petri.apache.org/cookbook
PRs welcome

I think the AL needs to be mentioned as a key part of the Apache Way, and should be mentioned as a pre-requisite in "Is Apache Petri right for you"

I've made more changes. The addition of the AL to the "Is Apache Petri right for you" may get pushback as we planned to be open to simply educating an external community w/o them necessarily becoming a TLP.


cmd ref: add remote example in gc

dvc gc has two fundamental modes - local and remote. We should be extremely clear about what does each mode do and what it does not do.

See https://github.com/iterative/dvc/issues/1266#issuecomment-433332445

I'll write an example based on an SSH remote. I think that's more straightforward than other options.
Or should I use a local remote? We may need to reflect (maybe in a separate example) the use of --projects.

@iesahin better to use an artificial ("local") remote for simplicity, or S3 if we go being realistic but not-that-easy-to-reproduce

reflect (maybe in a separate example) the use of --projects

Good idea but unrelated here. Maybe contribute that one to #2064

Good idea but unrelated here. Maybe contribute that one to #2064

What I mean is something related to this option in gc.

You may have multiple projects sharing the same remote cache and if you don't mention all of them in -p option, you may delete the other projects' artifacts sharing the same space.

Woah I didn't even remember about that option. May be OK to leave that out. Let's 🙏 try to get a good basic example for gc -c/r and then maybe mention --projects. Not critical


Question: Security Of Refresh tokens In Resource Owner Flow

I have a public application that uses long standing Refresh Tokens to access an API (Both read and write).  I was wondering about the security of using access tokens in this scenario, specifically mitigating against the risk of brute forcing a token.
Is a brute force not a worry due to the number of combinations?
Is it ok to have the same ClientID and Secret for every install of the application (Where the ease to extract it presumably renders them useless). and if not....
Should I have a different ClientID and Secret for every installation of the application in the wild or is there a better way such as passing through a user ID as well and linking that to the refresh token?
Thanks in advance for any pointers.

Is a brute force not a worry due to the number of combinations?
An access token is a JWT, which means it is signed (assures the JWT has not been tampered with) and encoded. So I don't think brute forcing a token is a concern. Of course there are other concerns with the use of access tokens like XSRF (if you are using cookies), man in the middle attacks, poorly validated tokens, etc. But most of this have a known solutions.

Is it ok to have the same ClientId and Secret for every install of the application
Yes. Just remember that ClientId and Secret are the application's credentials, so they have to be safeguarded.

Should I have a different ClientId and Secret for every installation of the application in the wild
It depends on your business rules. But just consider that there is a trust relation between the authorization server and the client. The authorization server must know of the client. So in this scenario, for every installation of the application you would have to "inform" the authorization server that there is a new client(for example having the user "manually" request a ClientId and secret from the authorization server).

or is there a better way such as passing through a user ID as well and linking that to the refresh token?
You should use the SubjectId standard claim for this (sub). So yes, you could "link it" I guess along with any other claims you deem necessary. But if you are worried about the fact that access tokens and refresh tokens are self contained tokens (they have the information required to be granted authorization and don't need to be checked by the authorization server once granted), consider using reference tokens instead (which use the introspection end point) to have more control over their lifetime. For more on reference tokens and introspection: https://leastprivilege.com/2015/11/25/reference-tokens-and-introspection/

Hope this helps.

Thanks for the reply,
I may be wrong but I was under the impression that IdentityServer's refresh tokens are simply a random id with no data such as Sub and no signature.  A brute force would just fire random ID's at the Token Endpoint and hope for a match.
My worry was the more users you have the more active refresh id's there are to match against but I've been thinking about it and even if you have a refresh token for every human on the plant, the chance of matching one would still be almost 0. I think the combinations are so big I just can't visualize it.
Identity Server seems to use 128bit ID's for the Reference Token Handle which I've increased to 256bit to satisfy my own Anxiety.
I could lock it down even further by requiring the refresh token in the form UserID.RefreshTokenHandle when sent to the endpoint but I think this is probably overkill?

Yes my bad, reference tokens are opaque tokens.
I don't think is a good idea to resend the UserId outside the access token.
I believe it is overkill.

Thanks, I'll leave it as I currently have it then.


Deprecate unused utility classes

This deprecates Pair, Timer, and SeekableArrayStream since they are no longer used internally.

Coverage decreased (-0.4%) to 56.305% when pulling 861cd217b7c37c6dd6bc8d3658a39287678f5453 on tomwhite:deprecate-unused-classes into 05298aefa4f91998c335525fee00fc48a2559cc8 on HadoopGenomics:master.

Thanks for looking @heuermh. I've added the text you suggested.

Coverage decreased (-0.4%) to 59.384% when pulling 9101f16f3b122ab71ae7aa9e48d1cc6a2807eb49 on tomwhite:deprecate-unused-classes into 2298633ac40b082bcb70b86d40bb19d7221a40e6 on HadoopGenomics:master.

LGTM, thanks @tomwhite


[oneDNN] Optimize MIN_FIRST mode quantization with oneDNN

This PR is improves performance of MIN_FIRST mode quantization using oneDNN library.

@penpornk Any update on this PR?

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

Hi @penpornk Can you please review this PR ? Thank you!

@penpornk I have addressed the comments. Thanks!

The test was already failing on Windows (with just removing the ENABLE_MKL condition in mkl_quantize_op_test.cc.) Could you please help take a look? The error message:
I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: SSE SSE2 SSE3 SSE4.1 SSE4.2 AVX AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
[       OK ] MklQuantizeV2OpTest.small_uint8 (22 ms)
[ RUN      ] MklQuantizeV2OpTest.small_int8
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (-1 not equal to -2)
Expected: true
i = 1
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (1 not equal to 3)
Expected: true
i = 2
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (-2 not equal to -4)
Expected: true
i = 3
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (-25 not equal to -49)
Expected: true
i = 4
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (-81 not equal to -128)
Expected: true
i = 6
tensorflow/core/framework/tensor_testutil.cc(134): error: Value of: IsEqual(Tx[i], Ty[i], t)
  Actual: false (-127 not equal to 0)
Expected: true
i = 0
[  FAILED  ] MklQuantizeV2OpTest.small_int8 (1 ms)

Thanks for reporting this issue. Does it fail on windows without this PR if ENABLE_MKL is dropped?

@penpornk We tried reproducing the issue on BDW/SKX/CLX with oneDNN V3.x, but we could not reproduce.

@penpornk Thanks for reporting this issue. Does it fail on windows without this PR if ENABLE_MKL is dropped? Also could you please mention CPU kind (Broadwell/Skylake/Cascadelake etc.) and oneDNN version.

Yes. It failed without this PR if ENABLE_MKL is dropped. Based on the log, the machine is a Broadwell.
@cantonios A quick fix is replace the ENABLE_MKL test guard with defined(TF_PLATFORM_LINUX_X86_64). Would you be okay if we only run the test for Linux (and not Windows) for now?

@cantonios I have disabled the test on Windows. Let me know if it is okay.

I'd rather it be fixed on Windows.  This is an MKL-specific test.  If it fails on Windows and something is wrong, not only the test should be disabled, but also the original implementation.

@cantonios Sounds good. @mdfaijul Let's disable this specific op rewrite on Windows?

Thanks for the suggestions. We are evaluating the impact on Windows for disabling rewrite.

@cantonios Sounds good. @mdfaijul Let's disable this specific op rewrite on Windows?

Thanks for the suggestions. We are evaluating the impact on Windows for disabling rewrite.

@penpornk We have evaluated disabling QuantizeV2 op rewrite on Windows client CPU. We observed up to 2X degradation in performance.

@penpornk We have evaluated disabling QuantizeV2 op rewrite on Windows client CPU. We observed up to 2X degradation in performance.

Thanks for investigating.  But if we can't trust the rewrite on Windows, then I'm not sure how we can justify putting it in.  We should probably disable it until we figure out why the tests are failing.

@penpornk @cantonios I think I was able to find the issue. It looks like it is a compiler issue in Windows. A fix has been made (https://github.com/tensorflow/tensorflow/pull/56776/commits/77fc2fff3c8a2413d882fd766e7ff71fdf45a112). Please check.
I found that when T = qint8

Windows: std::is_signed<T>::value is false
Linux: std::is_signed<T>::value is true

Hi @penpornk  Can you please review this PR ? Thank you!

Closing stale PR


Change the backing base name type of DeclName to DeclBaseName

The new DeclBaseName type will later be able to hold either normal identifiers (as they exist now) or special names that don't have an identifier (like subscripts)
The TODO in this PR will be addressed in PRs for subsequent commits of #6848.

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 4f0ab96626882a7b9bcc0b5f481f3f95103d0455
Test requested by - @ahoppen

Let's try Linux again
@swift-ci please test Linux platform


TROJ_GEN.R002V01IT23

This is probably a false positive, but since the penultimate version this has been detected in Virustotal

Obviously it's a false positive. Don't know why now it's reporting the app as a malware but it's completely false. The project is open source and everyone can read the code and search malicious behaviors.


N/A version is not installed yet

Installing latest NVM on a Ubuntu 14.04 EC2 AMI using:
curl https://raw.githubusercontent.com/creationix/nvm/v0.7.0/install.sh | sh
Produces the message:
N/A version is not yet installed
whenever ~/.nvm/nvm,sh is invoked
It will, however, install and use Node.js versions.
Haven't seen this message before - just appears to have started today?

N/A version is not yet installed  The reason you see this info is because you have set a alias for a specific version of node, and you delete this version after, so the alias is now N/A, use nvm ls you'll get it!

Actually you can get stuck to this phase if you try to set to default a nonexistent version.
jkarttunen@juha~> nvm alias default v4.3.2
! WARNING: Version 'v4.3.2' does not exist.
default -> v4.3.2 (-> N/A)
jkarttunen@juha~> nvm alias default v4.2.3
N/A: version "v4.3.2" is not yet installed
jkarttunen@juha~> nvm list
N/A: version "v4.3.2" is not yet installed

@jkarttunen ooh, that's a great find, thanks!

@jkarttunen however, I can't reproduce that problem. what version of nvm do you have, and what does nvm debug print out?

Already deleted it. nvm 0.30.1

jkarttunen@juha~> rm -rf ~/.bower
jkarttunen@juha~> rm -rf ~/.npm
jkarttunen@juha~> curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.30.1/install.sh | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  7735  100  7735    0     0   8324      0 --:--:-- --:--:-- --:--:--  8317
=> Downloading nvm from git to '/Users/jkarttunen/.nvm'
=> Cloning into '/Users/jkarttunen/.nvm'...
remote: Counting objects: 4308, done.
remote: Compressing objects: 100% (27/27), done.
remote: Total 4308 (delta 14), reused 0 (delta 0), pack-reused 4281
Receiving objects: 100% (4308/4308), 1.06 MiB | 446.00 KiB/s, done.
Resolving deltas: 100% (2521/2521), done.
Checking connectivity... done.
* (HEAD detached at v0.30.1)
  master

=> Source string already in /Users/jkarttunen/.profile
=> Close and reopen your terminal to start using nvm

jkarttunen@juha~> nvm debug
nvm --version: v0.30.1
$SHELL: /usr/local/bin/fish
$HOME: /Users/jkarttunen
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
nvm current: none
which node:
which iojs:
which npm:
npm config get prefix: $NVM_DIR/nvm.sh: line 1562: npm: command not found
npm root -g: $NVM_DIR/nvm.sh: line 1562: npm: command not found
/Users/jkarttunen/.nvm/nvm.sh: line 1565: [: : integer expression expected
jkarttunen@juha~> nvm install 4.2.3
Downloading https://nodejs.org/dist/v4.2.3/node-v4.2.3-darwin-x64.tar.gz...
######################################################################## 100.0%
WARNING: checksums are currently disabled for node.js v4.0 and later
Now using node v4.2.3 (npm v2.14.7)
jkarttunen@juha~> nvm alias default v4.2.3
default -> v4.2.3
jkarttunen@juha~> nvm ls
->       v4.2.3
default -> v4.2.3
node -> stable (-> v4.2.3) (default)
stable -> 4.2 (-> v4.2.3) (default)
iojs -> N/A (default)
jkarttunen@juha~> nvm alias default v4.3.2
! WARNING: Version 'v4.3.2' does not exist.
default -> v4.3.2 (-> N/A)
jkarttunen@juha~> nvm ls
N/A: version "v4.3.2" is not yet installed
jkarttunen@juha~> nvm debug
N/A: version "v4.3.2" is not yet installed

ah - nvm isn't compatible with the fish shell. It only supports sh, dash, ksh, zsh, and bash.

Oh yeah, thanks. Seems to work on bash side.

For those having this issue with fish, drop down to sh by executing sh, then do this with a version of node you have installed (nvm ls):
default -> 0.10 (-> v0.10.40)
sh-3.2$ nvm use 0.10
Now using node v0.10.40 (npm v1.4.28)
sh-3.2$ nvm  alias default 0.10
default -> 0.10 (-> v0.10.40)```

On Mac Terminal, I used nvm alias default system and then nvm use system to get rid of this error.

Hi I tried on Mac Terminal but it returns the same error. Any idea what could be the reason?

zhoukai07 can you please comment with the error that Mac Terminal is giving you? Then I will look more into it.

@alexmiddeleer what does nvm debug print out?

@ljharb
Alexs-MacBook-Pro-3:desktop-fe alex$ nvm debug
nvm --version: v0.31.4
$SHELL: /bin/bash
$HOME: /Users/alex
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
nvm current: v6.9.1
which node: $NVM_DIR/versions/node/v6.9.1/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v6.9.1/bin/npm
npm config get prefix: $NVM_DIR/versions/node/v6.9.1
npm root -g: $NVM_DIR/versions/node/v6.9.1/lib/node_modules

@alexmiddeleer can you file a new issue with this info? That should work fine, as long as $HOME isn't a symlink, and there's no device mounting going on, and nvm wasn't installed by homebrew.

I have used nvm to install node version 6.9.5. But when I run the command nvm -ls , what is this lts/argon -> v4.7.3 (->N/A)?

@ankit-prgmr because the "argon" LTS is 4.7.3, and you don't have that installed. You only have "boron" installed.

@ljharb Thanks for responding. So it is only showing the LTS version for argon available, right? Doesn't mean that I have some parts of it in my system? (I mean the leftover files after uninstall)

You don't have anything in your system; every time nvm makes a network call it looks up what the LTS versions are and stores that as aliases. You can nvm unalias lts/argon if you want, but every network call nvm makes will restore it.
I do see that you're not using the latest nvm - a bug was fixed that makes lts/* point to argon instead of boron by mistake. Rerun the install script in the readme and that will be fixed.

@ljharb Cool. Thanks for clearing it to me :+1: :)

Similar problem. I am new to react native so I have no idea what is going on. I tried downloading everything per the instructions provided by the website but I keep getting this error.
N/A: version "N/A -> N/A" is not yet installed.
You need to run "nvm install N/A" to install it before using it.

when I typed in nvm ls
Tong:~ tongwu$ nvm ls
       v0.10.48
         v7.8.0
         v7.9.0
->       system
default -> nod (-> N/A)
node -> stable (-> v7.9.0) (default)
stable -> 7.9 (-> v7.9.0) (default)
iojs -> N/A (default)
lts/* -> lts/boron (-> N/A)
lts/argon -> v4.8.2 (-> N/A)
lts/boron -> v6.10.2 (-> N/A)

Also this:
nvm --version: v0.33.1
$SHELL: /bin/bash
$HOME: /Users/tongwu
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: 'https://nodejs.org/dist'
$NVM_IOJS_ORG_MIRROR: 'https://iojs.org/dist'
nvm current: system
which node: /usr/local/bin/node
which iojs:
which npm: /usr/local/bin/npm
npm config get prefix: /usr/local
npm root -g: /usr/local/lib/node_modules
Tong:~ tongwu$ bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin16)
Copyright (C) 2007 Free Software Foundation, Inc.
Tong:~ tongwu$ run
-bash: run: command not found
Tong:~ tongwu$ ~/.bashrc
-bash: /Users/tongwu/.bashrc: No such file or directory
Tong:~ tongwu$

@twu You mistyped nvm install node as nvm install nod before. Try nvm alias default node.

@ljharb I tried nvm alias default node and got this Tong:~ tongwu$ nvm alias default node default -> node (-> v7.9.0) which I assume is good. Then I tried to install npm install -g react-native-cli but it didn't work so I tried curl -0 -L https://npmjs.org/install.sh | sudo sh. which resulted in series of downloads and the words It worked. However, when I tried to create a new project react-native init NewProject it gave me this
Installing react-native...
Consider installing yarn to make this faster: https://yarnpkg.com module.js:472
    throw err;
    ^
Error: Cannot find module 'semver'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/unsupported.js:2:14)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
{ Error: Command failed: npm install --save --save-exact react-native
    at checkExecSyncError (child_process.js:481:13)
    at execSync (child_process.js:521:13)
    at run (/usr/local/lib/node_modules/react-native-cli/index.js:294:5)

which ends with
 stderr: null,
  stdout: null,
  pid: 24098,
  output: [ null, null, null ],
  signal: null,
  status: 1 }
Command `npm install --save --save-exact react-native` failed.

You have to nvm use first also, or restart your terminal.

I got this when I tried nvm use after restarting the terminal
No .nvmrc file found

Node Version Manager

Note: <version> refers to any version-like string nvm understands. This includes:
  - full or partial version numbers, starting with an optional "v" (0.10, v0.1.2, v1)
  - default (built-in) aliases: node, stable, unstable, iojs, system
  - custom aliases you define with `nvm alias foo`

hmm, after nvm alias default node, nvm use should use the latest installed node version.

which is why I am very confused. Thanks for trying though.

@twu781 if you do nvm use default what happens?

I got Tong:~ tongwu$ nvm use default Now using node v7.9.0 (npm v4.2.0)

when I try to make a new project it says npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed. at one point.
Now it allows me to initialize a new project but I get this when I try to generate the simulator  Command `run-ios` unrecognized.

Both of those warnings are part of the project you're using, or npm - not from nvm. Once npm and node work, nvm is not involved.

nvm alias default system work for me on macOS Sierra with Homebrew

I installed the latest version of node using nvm

nvm install node 8
Once done run

nvm current

then list all versions of nvm by running

nvm ls
This will give you a list of all installed version of node on ur machine. And one of them will be default for me it was version 6, but since I already had started using newer function I was constently getting errors. In the end I ran

nvm alias default node
And once done the default version was moved to 8.2.1

nvm alias default stable or  nvm alias default stable etc. worked as expected.
The "default" link was broken.
A nvm ls shows this too.

It worked for me, and I think you can default to any version you like. I used nvm alias default 10.

On jenkins I added nvm alias default node in script and it worked!


amtool config routes command not available

The amtool routes visualization feature does not seem to be available. The routes command is not even recognized.
amtool --version
amtool, version 0.15.3 (branch: HEAD, revision: d4a7697cc90f8bce62efe7c44b63b542578ec0a1)
build user:       root@4ecc17c53d26
build date:       20181109-15:40:48
go version:       go1.11.2
amtool config routes --help
amtool: error: unexpected routes

The command will be available in the next release of AlertManager. But I think you should be able to use amtool from master against a 0.15.3 AlertManager.


multi: retain original copyright on files copied/modified from btcsuite

Early in the lifetime of the project here were a few files we either
copied entirely, or used as the basis for code within lnd. Before this
PR, this was not recognized by retaining the original copyright. With
this commit, we remedy that by explicitly noting the copyright in the
relevant files.
Fixes #423.
cc: @davecgh @marcopeereboom

ACK


[App Service] Fix #25616: az webapp browse not supported in cloud shell

Related command

az webapp browse
Description

resolves https://github.com/Azure/azure-cli/issues/25616
Testing Guide

az webapp browse -g -n
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

App Service


Retry does not trigger some formatter methods

cucumber (3.1.2)
Summary
I would like to use `--retry`` while having html / or a custom formatter implemented. I have realized
following events does not trigger while retrying a failed test case:
before_examples
after_examples
before_table_row
after_table_row

Expected Behavior
I would like to use formatter as while using a test case with examples. Ex.
  Scenario Outline: Verify filters in property 'for rent' page
    Given I navigate to a certain page
    When I filter using  "<Filters>" on certain page
    Then I verify the "<Filters>" in certain page
    Examples:
      | Filters      |
      | price_min    |
      | price_max    |

Current Behavior
Table row filter's only trigger for the first run, and does not trigger before before the retrying test case.

Hi, what do you think about this, is there a simple solution ? If you point me in correct place, I might work on a PR as well

Thanks for the report @serhatbolsu! I didn't work on the --retry feature at the point where the newer formatter API was added, so I'm not sure what the rationale was for not firing these events on retried scenarios. As such, I'm hesitant to say whether we'd accept a PR for this. Do you know what the reasoning behind this was @luke-hill?

Not sure sorry.
I know there was a separate issue request someone submitted a while ago about the exit code potentially being incorrect. But other than that I'm not sure.
--retry was added in cucumber3, and I know it was a first pass approach, and there could be some tweaks made to it. The only caveat to this is I know tables have had a few teething problems going to v3 because not many people use them. Aslak is probably the best person to advise here though. I've never used any of the example hooks you've mentioned, so I can't comment

@aslakhellesoy  need your assistance please


MesherySettingsPerformanceComponent.js converted class-based component into functional componen

Converted MesherySettingsPerformanceComponent.js class-based component into a functional component
Notes for Reviewers
Please check the above changes if I did anything wrong please let me know.
This PR fixes #8813
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 91ccb25bed55fe5a1b0f57f55df7a107345a92f8 at: https://651ab424451fdb0f0347264d--meshery-docs-preview.netlify.app

Let's discuss this on the Meshery dev call tomorrow,
Please add this as your agenda item in Docs: https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit

🚀 Preview for commit 8571e8431d238a808b5d7734f2c3f86dfbf038fd at: https://651d529e2b96d64ffe9f3c82--meshery-docs-preview.netlify.app

🚀 Preview for commit 677eb1ddf7670346ff29eea273c559d614e9f2c8 at: https://651d5a884049695df1a5921a--meshery-docs-preview.netlify.app

Hi Sir, Could you please review my PR and let me know if there are any issues? I'm eager to earn the Meshery Hacktoberfest badge, and your guidance will be greatly appreciated.

🚀 Preview for commit a52defbc86158f506fa26220a350bf30a60b3f9b at: https://65390662813b09068d3afd20--meshery-docs-preview.netlify.app

🚀 Preview for commit 3e61e5678a90ec866df4fd6294e85ec12600e344 at: https://65390785e54a6c09085743eb--meshery-docs-preview.netlify.app

🚀 Preview for commit 1a70bcb4b72e2b8db5ffd9249bbbdbb264519a5e at: https://653be1311a881c2d3684a694--meshery-docs-preview.netlify.app

🚀 Preview for commit 27431bbb857921ba975d08cca31df6dd79ac6392 at: https://653e4d26476a61684364a3c5--meshery-docs-preview.netlify.app

@csengineer1990 please revert unnecessary changes in this PR, I think this will also solve merge conflicts
It's hard to review your changes with unnecessary ones, pings us when you done with that let's get this merge

Sir, I apologize for the unnecessary changes in the pull request. It wasn't intentional, and I have now removed those changes and resolved the conflict. I don't know how these unnecessary changes occurred in the repository.
I appreciate your patience during this process. If the same issues persist, please guide me on the next steps to ensure the repository is synchronized without affecting the current pull request.  what should I do? and how can I solve with affecting the pull request? I would appreciate your help.

🚀 Preview for commit 6bb98203da00b3eaade0c672366b53db6b0be2e4 at: https://65447a075636ae428effd0e9--meshery-docs-preview.netlify.app

Hi Sir, I often encounter issues with merging and updating branches, which can lead to unintentional, unnecessary changes in my main pull requests. This complicates the process of reviewing the changes. What do you think I should do to solve this, sir Please help me.


Change text for claim papers in profile

In the author settings, we have instructions that ask users to go to legacy to claim their papers. Update these instructions with the current process on labs

TECH NOTES:

Change the text here https://github.com/inspirehep/inspirehep/blob/master/ui/src/authors/components/OrcidPushSetting.jsx#L16-L21

Also the text under the toggle should be changed.
I found a screenshot here: https://its.cern.ch/jira/browse/INSPIR-3450 , please let me know of the whole text so that I can provide a replacement.

 Clean your INSPIRE profile
          Before you export your publications to ORCID, you must clean up
          and prepare your INSPIRE author profile. The goal is to have a
          unique INSPIRE profile associated to you, with all works marked as
          yours.

          The most expedient way to clean your profile is to obtain an arXiv
          ID and login to INSPIRE using this arXiv ID. If you don’t use an
          arXiv ID, you can make changes to your profile as a guest, but all
          actions will require approval from the INSPIRE team, which may
          take a few days depending on how many requests are before yours.
          If you don’t mind waiting, you can follow the instructions below
          without signing in with an arXiv ID and still prepare your author
          profile for an ORCID connection. Please be aware that this
          approval process is necessary to ensure the quality of the
          information we have in INSPIRE.

          <ExternalLink href="//inspirehep.net/info/HepNames/connect_orcid#hide1">
            Instructions on cleaning your profile
          </ExternalLink>

Claim your work

          Once your profile is clean, you need to confirm that the{' '}
          <ExternalLink href={`//inspirehep.net/author/claim/${authorBAI}`}>
            listed works
          </ExternalLink>{' '}
          are yours. Only claimed works will be pushed to ORCID.

          Note that cleaning and claiming have to be done periodically, in
          order to keep your profile up to date

Connect your INSPIRE author profile to ORCID

          Once you clean up your INSPIRE author profile and you claim your
          works as explained above, you are ready to export them to ORCID.

    Note that if you use your ORCID when publishing a paper, INSPIRE will
    automatically attribute it to you and send this information to the ORCID
    register. If you don’t, or if the publisher doesn’t send this
    information to INSPIRE, you need to claim it.

    If you have questions or need assistance during any steps of this
    process, don’t hesitate to send a message to{' '}

If toggle is enabled:
This profile is already connected to the following ORCID:

Your{' '}
<ExternalLink href={//inspirehep.net/author/claim/${authorBAI}}>
claimed
{' '}
works will be exported automatically.

If toggle is enabled, keep the same text, but remove the ExternalLink:

This profile is already connected to the following ORCID:
Your{' '}

claimed
{' '}
works will be exported automatically.

If toggle is disabled, again, keep the same text but remove the link:

Your INSPIRE{' '}
<ExternalLink href={//inspirehep.net/author/claim/${authorBAI}}>
claimed
{' '}
works will be exported to your ORCID account.

Change the text under the toggle to:

Your INSPIRE works are not exported to your ORCID{' '}
 yet. Please note that only the
publications that are verified as yours on INSPIRE will be exported to
ORCID.
A new interface that will allow you to claim your papers is coming up soon. In the meantime, if you wish to claim a paper as yours, you can send your request at <TBD-----inspirehep.net>.


Fix incorrect URLs for milestone APIs

API URLs starting with "/" does not play well with Github Enterprise,
whose API endpoint looks like "http(s)://hostname/api/v3/".
This fixes incorrect URLs in miletones APIs.

ah, good catch.  thanks!  rebased onto master and merged as 1ee11e5972575f291558c46d04daca566b2845cd


[All Hosts] (ia) move same-origin article to more appropriate ToC node

Our lead for Documentation Strategy already approved this move, so it doesn't need to be reviewed by her again.

Docs Build status updates of commit 4a5539d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Getting "VIDIOC_DQBUF error, Invalid Argument" on Linux kernel 3.0.36+

Hi,
I get this error on kernel 3.0.36+:

VIDIOC_DQBUF error 22, Invalid Argument

when I run the following example code (provided by @TIS-Edgar at #11) for capturing images:
https://gist.github.com/10f04501f49b6b3bf75e
After debugging, apparently at the line 109 of v4ldevice.cpp (inside read_frame function) this error occurs.
I checked capturing video of a webcam using guvcview and it didn't any this problem.

Issue closed, since there was no answer anymore.


Make all onData and similar callbacks log to debug

Self-explanatory.

This one has been done with a simple push to main. Finished.


duplicate lines cause different result in tangram-es and tangram play

TO REPRODUCE THE ISSUE, FOLLOW THESE STEPS:

List the steps you used
To produce the result here
Don't leave any out

use style with
                     stroke: { color: black, width: 1 }
                     stroke: { color: global.text_stroke, width: [[1,1px], [12,2px],[16,4px]] }

RESULT:
Describe the result here.
first line is used
As result, following map style change https://github.com/matkoniecz/StreetComplete/commit/d1bc84f77e8695530a18803aaa9b862b678e890c causes change to rendered map with tangram-es interpreter
tangram play in both cases is using second stroke definition.
EXPECTED RESULT:
Describe what you expected to see here.
either

second line is used (as redefinition)
error is raised
both tagram play and tangram-es are using first line and ignore redefinition

ENVIRONMENT:

What operating system and device did you produce this issue on?

If you used a released version, what is the version number?

Redmi 4 Xiaomi

Android 6.0.1 MMB29M

com.mapzen.tangram:tangram:0.7.1

OTHER:
Any additional notes, comments, tips, tricks, gifs, suggestions, poetry, recipes, jokes
If you are able to record the log output from tangram-es, please attach it as a text file or share it using a site like https://hastebin.com/
Reported here as ignoring first line and using second as redefinition seems saner interpretation of repeated rules.
Though giving error message would be preferable in both interpreters.

The scene file excerpt you've posted is invalid YAML: repeated keys in a mapping are not allowed (http://www.yaml.org/spec/1.2/spec.html#id2802432).
However as you've suggested, Tangram ES and Tangram Play could be more helpful in this situation. To your suggestions:

Use the second line as re-definition: There is no concept of ordering in a YAML mapping so we can't meaningfully call one of these lines the "second" one.
Error is raised: This would be a helpful behavior, if we can get Tangram ES and Tangram Play to error in the same ways.
Use the first line and ignore re-deifintion: Again, because mappings are inherently unordered, we can't call one of them "first".

Thanks for creating a matching issue for this in Tangram Play, I'll see if there's a feedback mechanism for this that will work for both engines.

I think the most helpful would be to reject the style definition entirely (2.) because it is obviously an error.

Tangram is able to report many YAML syntax errors, but this one in particular isn't detected by the YAML parsing library we use. There are good reasons why it doesn't, but it's kind of a long story: https://github.com/jbeder/yaml-cpp/issues/60
To diagnose issues like this in the future, a separate YAML syntax validator may be the easiest solution. Gonna close this issue.


Sample TL Report - Redash Visualizations

Title of ticket:
Description
Summarize issue : Create Redash Visualization for Sample TL Report. Since OC environment is not ready @VineetBala-AOT  is developing this locally first, then will migrate to DEV, TEST
Dependencies
Are there any dependencies? - Since Environment is not Ready on OC, might need to migrate Visualizations to DEV, TEST once its up
DOD

[ ] Create Visualization for the listed reports (Locally)
[ ]  Confirm filters with business
[ ]  Confirm Visualization with Business
[ ]
[ ]

There are a number of different reports from team leads at IAO that we have as examples, they can be found in this folder on Teams: https://bcgov.sharepoint.com/:f:/t/03521/EiwqRPAOk_FMt4ddTbsrhqYBWth9drh_bz1EeRzb58O55w?e=3A7Grv
The Sample TL Report is a consolidation of the common fields we saw across most of the examples. They are generated using the same request reporting screen in AXIS:


Question: Anyone implemented this with Bootstrap Treeview?

I am trying to implement hierarchy view with https://github.com/jonmiles/bootstrap-treeview
Anyone used this lib with closure tree gem?

it doable. you just have to provide the correct json format.

Don't use it, find one that doesn't re-render itself over and over.


Add the option to pin items to start menu

Option to pin programs and folders to the start menu

@XPoppyX this issue should be resolved with #4059.


Hiding remove context menu when there are no colors in the history

Summary of the Pull Request
Hiding remove context menu when there are no colors in the history, fixes #8322
PR Checklist

[x] Applies to #8322
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
Validation Steps Performed
Manual test

@martinchrzan Please remove the "fixes" keyword from the PR description. It will close #8322 before release


facebook invalid key hash

how can i solve this problem?

Hey. You didn't add key hash in Facebook App Settings. See README, Step 6 in Android setup section.


set globalMetadata for API doc publishing

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

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


[Croissant] The API supports a parameter to produce either an abridged JSON-LD or a full-length JSON-LD

Issue: Croissant JSON-LD are embedded in Hugging Face's HTML page. As JSON-LD can get lengthy, they are truncated (see the code).
Proposed solution: Add a parameter ?full=true/false (with a default to false) to toggle between the abridged version or the full-length version of the JSON-LD. Hugging Face would use the abridged version to embed in their HTML page, while still allowing users to retrieve the full version to produce a valid Croissant file. Ideally the description of the abridged version would state that it's (or may be) abridged, and provide the URL of the full version. This is a temporary solution while waiting for  tags to support URLs rather than in-lined JSON.
@lhoestq @severo What do you think? Also, what convention do you usually use for the URL parameters? Do you have examples of such routes? If I have your go, I could implement this. Thanks for your input!

Good idea. Feel free to open a PR!

what convention do you usually use for the URL parameters?

We have no other boolean parameter, I think. @lhoestq should we pass:

?full=true/false
or ?max_columns=1000, and if not present: return the full response?

If we opt for the integer parameter, you could take example on https://github.com/huggingface/datasets-server/blob/1759d65fa668f35067175cfb3210666c1b7c3f47/services/rows/src/rows/routes/rows.py#L72
else, we should do the same, but for a boolean value. If so, I think checking for exact value true is OK, and coherent with what we do on the Hub (https://huggingface.co/spaces/enzostvs/hub-api-playground)

Ideally the description of the abridged version would state that it's (or may be) abridged, and provide the URL of the full version

OK, good idea. As we return a croissant content that we "directly" embed in the HTML, it would be better if it's part of the croissant specification, or at least not incompatible with it


List of prebuilt binaries and alternate location

Hi,
Where can I see the list of available of prebuilt binaries? And is it possible to provide an alternate location (say via an Env variable) for the install to look at?
Thanks,

You can find all available prebuilt binaries within the release section.
You cannot change the resolved / used binary on the install step. better-sqlite3 relies on prebuild-install and node-gyp to either find a suitable prebuilt binary or peform a compilation on install for the used runtime environment at the time of installation.
If you do not want to rely on this default behaviour when adding better-sqlite3 to a module, you might do so by ingoring the install script of better-sqlite3 ( npm install better-sqlite3 --ignore-scripts ). It is then up to you to download the suitable binary for the used runtime environment and either store it in one of the default lookup folders of the used bindings module or pass the path to the binary using the nativeBinding option of the Database constructor.

You cannot change the resolved / used binary on the install step.

Well actually you can configure a mirror location with an environment variable:
npm_config_better_sqlite3_binary_host_mirror=https://you/custom/location

NPM will automatically pick this up.
In the URL, you can also use a generic {version} placeholder, which will be replaced by NPM with the version requested:
npm_config_better_sqlite3_binary_host_mirror=https://you/custom/location/{version}/

@chgeo that's perfect, thank you so much


iOS Support

Description
The repo for neofetch on cydia seems to be down. Is there another repo or has iOS support been dropped?

See #1307. We no longer maintain any iOS repo.


Raise parallelism in grpc._cython._channel_test

Fixes #4899.

lgtm; failure in core test build is unrelated