Does passing 'training' flag to tf.keras.Sequential do anything?

URL(s) with the issue:
https://www.tensorflow.org/tutorials/customization/custom_training_walkthrough#define_the_loss_and_gradient_function
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/api_docs/python/tf/keras/Sequential
https://www.tensorflow.org/api_docs/python/tf/keras/Model
Description of issue (what needs changing):
It is unclear whether the Sequential class makes use a 'training' flag fed into it during training/inference, as the tutorial above implies.
Clear description
When building a custom model subclassing from tf.keras.Model, the standard signature for writing the call is as follows: def call(self, inputs, training=None, mask=None):
If my class includes submodels of the form Sequential, I am able to pass this flag forward but I'm unaware whether it's doing anything, as documentation from the class doesn't mention this flag.
Looking at the customization tutorial above, however, the flag is passed into a Sequential model that does not include layers whose behavior change during training/inference. So I don't know if that flag is doing anything.
Correct links
Is the link to the source code correct?
Parameters defined
Are all parameters defined and formatted correctly?
Returns defined
Are return values defined?
Raises listed and defined
Are the errors defined? For example,
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/feature_column/categorical_column_with_vocabulary_file#raises
Usage example
Is there a usage example?
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
Are there currently visuals? If not, will it clarify the content?
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style

If my class includes submodels of the form Sequential, I am able to pass this flag forward but I'm unaware whether it's doing anything

Any training boolean flag passed to a Sequential model will be passed to each and every of its layers, which will trigger training-only behaviours if any is implemented. Typically, one will have dropout applied conditionally on this flag, in which case passing it will indeed have an effect.
Note that using the default keras training loop (i.e. the fit method) will automatically pass this flag along (while it will be set to False in evaluate and predict).

Thank you for your answer. Can you please point me towards a source code file where I can see this behavior? Perhaps the docs should include this information as well since it's not very clear in my opinion.

You can see in the definition of tf.keras.Sequential.call here that it expects a training flag, and that it will indeed be passed to any sublayer whose call signature includes that keyword.
As for the use of training itself, you can notably read about it here in the Keras guide :)

@milongo, Does the @pandrey-fr's comment help in resolving your issue?

@gadagashwini , @pandrey-fr's comments do answer my questions, but I still feel the documentation for tf.keras.Sequential should include the training flag somewhere in the arguments i.e.
https://www.tensorflow.org/api_docs/python/tf/keras/Sequential#arguments_21 and make it explicit that the flag is passed to subsequent layers and made use of if those layers have it defined in their behavior, whether they're custom or not.

@milongo To be clear, the flag is passed by default as part of tf.keras.Model.fit, tf.keras.Model.evaluate and tf.keras.Model.predict. The fact that a model's call may accept a training argument is documented here :

If you subclass Model, you can optionally have a training argument (boolean) in call, which you can use to specify a different behavior in training and inference:

The fact that Layer instances may accept it (and will receive it if so) is part of their API, as indicated here.
That being said, it may be worth to augment the existing documentation of Sequential as you suggest it.

@pandrey-fr what is the behavior if the model is called in a custom training loop? Such as the following:
    for epoch in range(epochs):
        for step, (x_batch, y_batch) in enumerate(train_dataset):

            if epoch > mlp.warm_up:
                for layer in mlp.mean.layers:
                    layer.trainable = False
                for layer in mlp.variance.layers:
                    layer.trainable = True
            with tf.GradientTape() as tape:
                output = mlp(x_batch)
                loss = loss_fn(y_batch, output)
            grads = tape.gradient(loss, mlp.trainable_weights)
            optimizer.apply_gradients(zip(grads, mlp.trainable_weights))
            mean_preds, _ = output

You can see that I do not bother to pass the training flag anywhere.
Here, mlp is a custom class subclassed from tf.keras.Model, whose's call function is as follows:
    def call(self, inputs, training=None, mask=None):

        mean_predictions = []
        variance_predictions = []
        for idx in range(self.num_models):
            mean_predictions.append(self.mean[idx](inputs))
            variance_predictions.append(self.variance[idx](inputs))
        mean_stack = tf.stack(mean_predictions)
        variance_stack = tf.stack(variance_predictions)

        return mean_stack, variance_stack

You can see that I do not bother to pass the training flag anywhere.
(For clarity, mlp has two sets of ensembles, one in self.mean and another in self.variance, where each ensemble is defined as a list of tf.keras.Sequential modules):
self.mean = [
            tf.keras.Sequential([
                layers.Dense(512, activation='relu', input_shape=(self.input_dim,)),
                layers.Dropout(0.5),
                layers.Dense(256, activation='relu'),
                layers.Dropout(0.5),
                layers.Dense(self.output_dim)
            ])
            for _ in range(self.num_models)
        ]

Interestingly enough, my models train just fine, so I'm guessing the default internal value for the flag is training=True, even if the headers have it as training=None. I.e. somewhere, that flag's value is actually set to True.

As far as I know, if you do not pass training=True explicitly, your model should run without dropout. This may not be prejudicial to training, but if you meant to make use of it it would probably be worth it to pass the flag within your custom training loop.

You are welcome; I am glad this was helpful. Regarding documentation, perhaps @gadagashwini could assign the arbitration to a tensorflower; otherwise, maybe you could submit a PR to the tensorflow-docs repository?

You are welcome; I am glad this was helpful. Regarding documentation, perhaps @gadagashwini could assign the arbitration to a tensorflower; otherwise, maybe you could submit a PR to the tensorflow-docs repository?

Ahh I think you are correct. Passing the flag explicitly drastically affected training.
Thank you!
I could close this now, but what about the documentation issues I mentioned?

@milongo Are you planning to create any PR to update the docs? Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?


[RESOLVED] High ICMP ping response time

Problem started at 12:18:29 on 2023.07.28
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Operational data: Value: 748ms
Original problem ID: 34426
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23268&eventid=34426

Problem has been resolved in 7m 0s at 12:25:29 on 2023.07.28
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Original problem ID: 34426


Update Set-CASMailbox.md deprecating -IsOptimizedForAccessibility paramter

This parameter will not be supported by OWA team by Aug 19th, 2024. As OWA team will no longer support the light version outlook on the web.

@LynnetteZhong:

Syntax issues: you can't add extra text to parameters in the Syntax block or the section heading of the parameter description sections.
Never a good idea to mention specific dates. They have a nasty habit of sliding.
Cmdlet reference articles are single-sourced for Exchange Server (2010-2019) and ExO, so unless this thing is also going away from on-prem Exchange Server (doubtful), the parameter needs to remain in the article.
The parameter is already described as "Functional in on-prem Exchange only", which by definition means it doesn't work in ExO. So, whether it's available but doesn't work vs. it's not available shouldn't confuse customers (either way, it doesn't work for them).
If/when that parameter is actually unavailable to customers in ExO PowerShell, I'll make the following changes to the parameter description:

Remove the value "Exchange Online" from the "Applicable:" YAML property.
Change the first line of the description to, "This parameter is available only in on-premises Exchange."


Bigints should be 20

You have (implicitly) signed bigints, which cannot be more than 19 digits wide (the negative sign is not included).  Anwyay, that number is just a display width hint to pad out ASCII tables in the client, it doesn't actually change anything about how the numbers are stored or the range of values the column can hold. (I assume we don't use ZEROFILL anywhere).

Well vincent i only say they should be 20 because they are everywhere else in the database except in like 3 or 4 places where craig had recently made a table using this script.

@hibooboo2 Liquibase probably screws this all up with in generates the XML, I would guess.


support complex expressions like put x={a:1+3, b:len(foo)}

We should add support for complex expressions for records, arrays, sets, maps, etc.  We could start with records and arrays like javascript.
The syntax for records is
{ <field> : <expr>, <field> : <expr> , ... }

The syntax for arrays is
[ <expr>, <expr> , ... ]

etc
The runtme would have new expression ast nodes and new expr Evaluators that would evaluate each expression, build the zcode, and look up the zng.Type to form the zng.Value.
This is not particularly difficult at this point given all the progress on the software architecture.

Verified in zq commit 45ac6b4.
Cooking up an example like the one in the issue title:
$ echo '{"foo": [2,4,6]}' | zq -z 'put x={a:1+3, b:len(foo)}' -
{foo:[2,4,6],x:{a:4,b:3}}

The linked PR has many advanced examples in its tests, but here's some examples of simple use of literals for matching.
$ echo '{"foo": [2,4,6]}' | zq -z 'foo=[2,4,6]' -
{foo:[2,4,6]}

$ echo '{"foo": [2,4,6]}' | zq -z 'foo=[2,4]' -
[no output, since it was not an exact match]

$ echo '{"nested": {"foo": "bar", "num": 123 }}' | zq -Z 'nested={foo: "bar", num: 123}' -
{
    nested: {
        foo: "bar",
        num: 123
    }
}

$ echo '{"nested": {"foo": "bar", "num": 123 }}' | zq -Z 'nested={foo: "bar", num: 456}' -
[no output, since it was not an exact match]

Previously (such as at zq commit ec3250b) these were all just parse errors.
$ zq -version
Version: v0.29.0-160-gec3250b8

$ echo '{"foo": [2,4,6]}' | zq -z 'put x={a:1+3, b:len(foo)}' -
zq: parse error: error parsing Z at column 10:
put x={a:1+3, b:len(foo)}
     === ^ ===

$ echo '{"foo": [2,4,6]}' | zq -z 'foo=[2,4,6]' -
zq: parse error: error parsing Z at column 6:
foo=[2,4,6]
 === ^ ===

$ echo '{"nested": {"foo": "bar", "num": 123 }}' | zq -Z 'nested={foo: "bar", num: 123}' -
zq: parse error: error parsing Z at column 14:
nested={foo: "bar", num: 123}
         === ^ ===

Thanks @mccanne!


Update to golang 0.9.14 API

This PR bumps to the latest versions of the dependencies and updated the package to account for changed APIs.

Thanks @hkalodner!

Gj


use bash instead sh

force to use bash shell

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

why do we need for force to use bash shell?


properly validate and read all config variants

backport #392 to 0.30

Coverage decreased (-0.06%) to 89.176% when pulling aa8891f79951af96ec0e6150c2c789f3f964bcca on backport_fix_event_player into cc9caaaa4f9c9a4765e777ea5e2e34ef4dbb0662 on 0.30.


fix the issue #351

I fix the bug of num_cores parameter.

I read methods/bayesian_optimization.py and find the objective is defined as self.objective = SingleObjective(self.f, self.batch_size, self.funct_name). And I believe it should be changed into self.objective = SingleObjective(self.f, self.num_cores, self.funct_name). Otherwise, the num_cores parameter would be useless.

The original code in methods/bayesian_optimization.py
self.objective = SingleObjective(self.f, self.batch_size,self.objective_name)
I believe the code should be changed into:
self.objective = SingleObjective(self.f, self.num_cores,self.objective_name)

@apaleyes @lawrennd

Codecov Report

Merging #352 (f184bad) into master (0be0508) will decrease coverage by 0.36%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #352      +/-   ##
==========================================
- Coverage   77.44%   77.08%   -0.37%
==========================================
  Files          97       97
  Lines        4678     4678
==========================================
- Hits         3623     3606      -17
- Misses       1055     1072      +17

Impacted Files
Coverage Δ

GPyOpt/methods/bayesian_optimization.py
88.88% <100.00%> (ø)

GPyOpt/core/task/objective.py
57.69% <0.00%> (-28.85%)
:arrow_down:

GPyOpt/util/general.py
44.35% <0.00%> (-1.62%)
:arrow_down:

Continue to review full report at Codecov.

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


How to invest like a VC

Audience:
Anyone
Topic:
How to invest like a VC
Description:
Not that long ago, it took a company about 4 years to IPO. Today, the median time to IPO is more than 12 years. We are seeing a very similar shift in the crypto industry. We are observing a shift from projects getting listed on crypto exchanges on Day 1 to projects getting listed a year or two after the private rounds. As more and more institutional capital enters the industry, founders will choose to sell larger portions of their crypto assets to private investors and not list these assets for a longer period of time. Thus taking away the opportunity to get early into good projects, away from the retail investor.
The talk shall focus on the many emerging opportunities for the community to invest in early into projects, the strategy to adopt to have a low risk to return ratio and therefore understand the emerging economies of secondary markets in crypto presenting the same if not better opportunities to the crypto community comparable to VC funds and angel investors.
About:
Vaibhav Tyagi, VP Marketing & Strategy Capx
Links:
https://capx.fi
https://twitter.com/callmevaibt
https://www.linkedin.com/in/vaibt/

Hi, it would be a 10 minute talk
On Mon, 30 May 2022 at 5:45 PM, Aditya Gupta @.***>
wrote:

Hey! Can you please define the type of your talk?
—
Reply to this email directly, view it on GitHub
https://github.com/Web3-Conf-India/Proposal-2022/issues/54#issuecomment-1141084946,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWFLNQFAK6UHIE7WL7LLVHLVMSWN3ANCNFSM5XF5NIQQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add searching and filtering to platform browse projects view

Could be placed in a pop-up box to keep the UI clean
Search could be done in-browser using Lunr, similar to the dispatch simulations and simulators table.

An important architectural point for this would be to ensure that the project-card-list component does not contain any logic that handles filtering/searching. It should remain a purely functional component that is provided a list of projects as an input. Any filtering should happen at a higher level component that can then simply update the input to the card list component.

@jonrkarr I can start working on this if you have not yet

Please start with the processors so that we can get simulation results visualized in the platform. This is the key output of our our key advance.

list of files (manifest)
SED-ML content (needed for visualizations)
Vega

:tada: This issue has been resolved in version 7.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


@providesModule for public use?

Hello,
The Facebook F8 App makes quite heavy use of @providesModule with Flow. Now it's possible to use this ourselves if we include the module/files in the .flowconfig and have access to "global" modules we define with @providesModule without figuring out the relative paths.
This works great, however I'm struggling to find much about it in the documentation. Are we supposed to use this in our own environment or not? If not, why not?
It makes life so much simpler, but I can't work out whether I should be doing it.

@providesModule is a facebookism. I do not recommend that you use it, since all of your infra has to support it. But you can find out more about it at https://github.com/facebookarchive/node-haste and https://github.com/facebook/jest/tree/master/packages/jest-haste-map.

Agree with @zertosh. Haste is something Flow supports, but people outside of FB probably shouldn't be using it (unless you are @ide).

Thanks guys.
@samwgoldman is there anything similar which is less internal which can potentially produce this sort of functionality?

You're welcome to use haste, but it's not a Flow thing, which is why we don't have documentation for it. Do either of the links Andres shared help?

Yeah they do thanks, just trying to get my head around it.
The confusion really comes from the open source example of "your" F8 App which uses this all over the place.  There was an issue there (https://github.com/fbsamples/f8app/issues/72) about this, hence me bringing this up here as I thought it was a flow thing

@samwgoldman Just FYI we do use Haste in a couple of places with the understanding that it's on us to make it work for our use cases and that the code is the documentation.
Haste is pretty useful and I think the right way to do modules within packages (interpackage imports don't change at all) so I'd like to see it proliferate but no one's working on making that happen at the moment.

For the ones interested in using Haste, I've written a simple webpack resolver that can use it: https://github.com/flegall/haste-map-webpack-resolver

To have path aliasing, you can use webpack's resolve.alias. If you're using react-native and can't configure webpack, then babel's babel-plugin-module-resolver should work. If that, you'll also need to create an rn-cli.config.js file with this content:
'use strict'

var path = require('path')

var config = {
  getProjectRoots() {
    return getRoots()
  },

  getAssetRoots() {
    return getRoots()
  },
}

function getRoots() {
  return [
    __dirname
  ]
}

module.exports = config

@samwgoldman honesly I hope Facebook stops using it too, any time I (unintentionally wind up with multiple versions of a dep that uses @providesModule installed, I just get annoying useless errors: #6475


DB DATE_FORMAT in QueryBuilder

I have to create an example query using QueryBuilder, but I couldn't figure out how. Can this be done?
And if not, how can it be added to QueryBuilder?
SELECT * FROM contacts WHERE DATE_FORMAT(birthday,'%m%d') BETWEEN DATE_FORMAT('2016-11-01', '%m%d') AND DATE_FORMAT('2016-11-30', '%m%d')

I know that DATE_FORMAT supports mysql, oracle has a different function
http://www.sqlines.com/mysql-to-oracle/date_format

ever tried Expression? http://www.yiiframework.com/doc-2.0/yii-db-expression.html

Yes I checked with Expression does not work.

Why do you close when the problem is not solved?

How exactly it doesn't work? Which syntax have you tried?

$row = (new \yii\db\Query())
		->from('vtiger_contactsubdetails')
		->where(new \yii\db\Expression("DATE_FORMAT(birthday, '%m%d') BETWEEN DATE_FORMAT('2016-11-01', '%m%d') AND DATE_FORMAT('2016-11-30', '%m%d')"))->one();

I get an error:
Uncaught exception 'PDOException' with message 'SQLSTATE[42883]: Undefined functio

Can you post the rest of the error?

Updated the an earlier post

https://www.drupal.org/node/991898

What RDBMS are you using?

@dynasource
"SQLSTATE[42883]: Undefined function" This is the message from the database engine
@Alex-Code
PostgreSQL

does that function exists? https://www.postgresql.org/docs/7.4/static/functions-formatting.html

PostgreSQL doesn't have a DATE_FORMAT function.
You'll need to use https://www.postgresql.org/docs/8.2/static/functions-formatting.html
or create your own DATE_FORMAT function wrapper.

I know the function doesn't exist that's why I'm asking how to do it in QueryBuilder, so that it works with all available databases.
So YII doesn't support this function?

Nope, it does not. There's no way we can support everything.

@mariuszkrzaczkowski, try this:
$row = (new \yii\db\Query())     ->from('vtiger_contactsubdetails')     ->where(['between', 'birthday', '2016-11-01', '2016-11-30'])     ->one();

It would work, but I wanted the query to ignore the year

Depending on your project you could create an install script that creates function wrappers depending on the db system.
http://okbob.blogspot.co.uk/2009/08/mysql-functions-for-postgresql.html
Part way down that page is an example.

@mariuszkrzaczkowski, if so:
$row = (new \yii\db\Query())->from('vtiger_contactsubdetails')->where(['between', new Expression("to_char(birthday, 'MM-DD')"), '11-01', '11-30'])->one();

@Alex-Code
The problem with creating functions is that not all servers allow for that, and this is an app that can be installed on any server.
@alexraputa
This is not a solution to the problem.

I understand what you need but that's quite specific requirement so I'd suggest implementing it in your own subclass.


Clarify intention of the PACKAGE type

Make clear that the package type should be to a descriptive webpage and not a link to download artifact

Thanks!


cares: fix Windows platform detection and narrow scope of manual localhost resolution code

Detection of the Windows platform needs to be based on a constant defined in iomgr/port.h so that it gets disabled when libuv is enabled. The file grpc_ares_wrapper_windows_inner.cc is actually just rename of what was previously grpc_ares_wrapper_libuv_windows.cc, but that name was reused so git didn't pick it up as a rename.

The android failure is #19005. The Mac failures are #18968. The TSAN and UBSAN failures are #18511 (I think).

@apolcyn @AspirinSJL @markdroth Can one of you review this PR?

I'll let @apolcyn review this, since he's most familiar with c-ares portability issues.

The Interop Cloud-to-Prod failure is as previously mentioned. The Cloud-to-Cloud failure is #16579. The C/C++ MacOS failure is #18416.


Need notification for cancelled runs

I noticed that if I cancel a running action, I don't get any Slack notification about it.
I'm using if: always() on all of my notifications.
Am I missing something? or is this not supported?

Hey @gabiganam,
Could you please share your action configuration so that I can have a better look and suggest something?

Of course @ravgeetdhillon :)
This action for example, is a simple action that deletes a k8s namespace when a branch is deleted:
name: Delete namespace on branch deletion
on: delete

env:
   SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}

jobs:
  delete:
    if: github.event.ref_type == 'branch'
    env:
      REF: refs/heads/${{ github.event.ref }}

    runs-on: ubuntu-latest
    steps:
      - name: Slack notification
        if: always()
        uses: ravsamhq/notify-slack-action@v1
        with:
          token: ${{ secrets.GITHUB_TOKEN }}
          status: ${{ job.status }}
          notification_title: '1/2 - Deleting namespace...'
          message_format: ''
          footer: '<{run_url}|See Action Run> | <{repo_url}|Repo {repo}> | <{workflow_url}|{workflow} Workflow>'

     <several steps redacted here>

      - name: Slack notification
        if: always()
        uses: ravsamhq/notify-slack-action@v1
        with:
          token: ${{ secrets.GITHUB_TOKEN }}
          status: ${{ job.status }}
          notification_title: '2/2 - Namespace deleted!'
          message_format: ''
          footer: '<{run_url}|See Action Run> | <{repo_url}|Repo {repo}> | <{workflow_url}|{workflow} Workflow>'

BTW can you please take a look at this too? https://github.com/ravsamhq/notify-slack-action/issues/39

@gabiganam Notifications for cancelled runs are now available in v2.1.0. For more details, checkout the README.

Thank you for both fixes!


[Firebase] Thousands of warnings re: "url in link out suggestion must be owned by developer configured connected websites"

See: https://github.com/actions-on-google/actions-on-google-nodejs/issues/96

Solution: Remove link out suggestions, or use proxy.

We've removed the link out suggestions in the staging branch, still live in production however.

Not an issue in staging anymore.


Allow OMIS T&Cs to be changed via admin

This allows TermsAndConditions records to be added via the django admin.
After creation records cannot be changed or deleted as they might have been used in generated quotes.

Codecov Report

Merging #817 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #817      +/-   ##
===========================================
+ Coverage    96.65%   96.66%   +<.01%
===========================================
  Files          153      154       +1
  Lines         4457     4470      +13
  Branches       428      430       +2
===========================================
+ Hits          4308     4321      +13
  Misses          90       90
  Partials        59       59

Impacted Files
Coverage Δ

datahub/omis/quote/admin.py
100% <100%> (ø)

Continue to review full report at Codecov.

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

it might be nice to apply restrictions based on whether the terms and conditions have actually been used, but I'm not sure where that logic could be put.
@reupen

Yeah I thought about it but I think it's slightly less easy to understand for staff users and it's too much effort for what you get back.

Looks OK, but it doesn't seem ideal as there's still a lot of messaging etc. from Django about editing like 'You may edit it again below.'

What do you suggest? I could get rid of that message but I'm not sure if there are other occurrences.

@reupen
Improved it based on your feedback
Add
Without save and continue editing button.

Change
Without save buttons.

List
Without actions and with correct success message.

Yeah I thought about it but I think it's slightly less easy to understand for staff users and it's too much effort for what you get back.

I was more coming from the angle that if you create one, notice a mistake after saving it, then it's quite difficult to fix. Another option might be to emphasise that new terms and conditions should be tested in another environment first? Or another option might be to have new terms and conditions in a draft status initially (still editable) and then have to make them active to put it into use.
Also not sure if you've seen the 'Save as new' admin functionality, not sure how you feel about that https://docs.djangoproject.com/en/2.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.save_as
Just a few ideas, not critical (have already approved the PR 🙂).

Yes, you might be right but I can't spend too much time on this so let's see how they use it and I'll keep your ideas in mind in case it creates too many problems.
Thanks for the feedback!
BTW, I'm not a big fan on the new 'Save as new' button, it's not very intuitive :/


Improve DistributedCacheHelper. Closes #896

Remove duplicate DistributedCacheHelper and made new hadoop use the old impl.
Pulled code from deprecated hadoop classes and made small improvements.

I don't know enough about setting cache files in the job configuration this way. It seems reasonable, but has it been tested?

I am looking to see if any of the MR tests use it.  I think I am going to "un" fully qualify some of the references to help finding its use.

The TokenFileIT should use it.

The TokenFileIT should use it.

That was a good one, it uses 2 of the 3.  RangePartitioner looks to be the only class that uses getLocalCacheFiles but our test doesn't cover that code.  I can add at Test for it.

I don't know enough about the getCacheFiles() and getLocalCacheFiles(), but my expectation would be that we always want to try to get the local ones, because those are the ones in the distributed cache... whereas the other ones aren't actually cached... they're just references to the files outside the cache (in main DFS).

I don't know enough about the getCacheFiles() and getLocalCacheFiles(), but my expectation would be that we always want to try to get the local ones, because those are the ones in the distributed cache... whereas the other ones aren't actually cached... they're just references to the files outside the cache (in main DFS).

Its very confusing, I am trying to decipher.  Javadoc for addCacheFile and getCacheFiles.  The one for getLocalCacheFiles looks to be the most promising:

/**
   * Return the path array of the localized files.  Intended to be used
   * by user code.
   * @param conf Configuration that contains the localized files
   * @return A path array of localized files
   * @throws IOException
   * @deprecated Use {@link JobContext#getLocalCacheFiles()} instead
   * @see JobContext#getLocalCacheFiles()
   */

So it looks like getCacheFiles() was not intended to be for users, maybe that would explain deprecation.
Also, maybe we should consider not supporting the older mapred code since this is one case where you have to use the mapreduce methods if you don't want to use deprecated.

What is bizarre is...
addCacheFile -> uses "mapreduce.job.cache.files"
getCacheFiles -> uses "mapreduce.job.cache.files"
getLocalCacheFiles -> uses "mapreduce.job.cache.local.files"
Some internal mechanisms set the files in "mapreduce.job.cache.local.files"...

Also, maybe we should consider not supporting the older mapred code since this is one case where you have to use the mapreduce methods if you don't want to use deprecated.

This would make it hard to support the file-based token passing. We could just pull the file directly from HDFS, in that case. However, I'm not sure it's true that it requires the new API. I thought the old JobConf class does extend the internal JobContext which has the method for retrieving the local cache files.

Some internal mechanisms set the files in "mapreduce.job.cache.local.files"...

That's what I'd expect, but it's not clear to me the expected behavior that would explain how to use this properly.

Also, maybe we should consider not supporting the older mapred code since this is one case where you have to use the mapreduce methods if you don't want to use deprecated.

This would make it hard to support the file-based token passing. We could just pull the file directly from HDFS, in that case. However, I'm not sure it's true that it requires the new API. I thought the old JobConf class does extend the internal JobContext which has the method for retrieving the local cache files.

AFIAK there are two different ways to use the cache:

For mapred: DistributedCache.addCacheFile().  The older JobConf object extends Configuration.
For mapreduce: Job.getInstance().addCacheFile()  The newer Job class implements org.apache.hadoop.mapreduce.JobContext

And this is the best documentation I have found: https://stackoverflow.com/questions/21239722/hadoop-distributedcache-is-deprecated-what-is-the-preferred-api

OK I reverted the impl to what it was and just made the new Partitioner call the old impl code.  Removed the deprecation since its now used by the new package.
@ctubbsii what do you think of these changes?  This is all I really wanted to accomplish with this PR.  Other ideas you have we can do as follow on.


NullPointerException when calling matrixDiagV3

The following code works fine with 0.3.1, but now I get NPE in 0.4.0:
        EagerSession eagerSession = EagerSession.options().async(true).build();
        Ops tf = Ops.create(eagerSession);
        Operand<TFloat32> diagonal = tf.fill(tf.constant(new long[]{2}), tf.constant(1f));
        Operand<TFloat32> value = tf.fill(tf.constant(new long[]{1}), tf.constant(0f));

        Operand<TFloat32> output =
                tf.linalg.matrixDiagV3(
                        diagonal, tf.constant(0), tf.constant(2), tf.constant(2), value);
        Tensor tensor = output.asTensor(); // exception here

I got exception on both macOS and Linux.
java.lang.NullPointerException
	at org.tensorflow.EagerOperation.resolveTensorHandle(EagerOperation.java:193)
	at org.tensorflow.EagerOperation.resolveTensor(EagerOperation.java:166)
	at org.tensorflow.EagerOperation.tensor(EagerOperation.java:152)
	at org.tensorflow.Output.asTensor(Output.java:97)
	at org.tensorflow.Operand.asTensor(Operand.java:65)

@karllessard
You are absolutely correct, padding should be a scalar. I think earlier version didn't enforce that.
But the NPE is bug in the code. I will create a PR to address that.


[FEATURE-REQUEST] Uninstall script for TCP server

Describe the solution you'd like
I would like to have an uninstall script for cleanly removing the STRTCPSVR function.
Currently there is no controlled way to remove the TCP server and the  SCOMMANDER (or whatever the user specified during install) library. Like there is an install script I would like to have an uninstall script.
Describe alternatives you've considered
Manually removing the TCP server and the SC library. This requires looking into the code to see what commands were performed during installation, and perform the opposite.
Additional context

Closed by #49. Thanks!


Define $error page on a folder basis

Is your feature request related to a problem? Please describe.
Currently it is only possible to have one $error page (in the root of the project) but in various circumstances it can be nice to have a different error page depending on where you are in the application. (fe: in /products )
Describe the solution you'd like
Have the possibility to define a $error page per folder, in the same way we can define a $layout page per folder.
Describe alternatives you've considered
It is possible to use the page store to differentiate all this logic in the global $error page, just like one would do to differentiate between different status codes.
How important is this feature to you?
The current solution becomes very messy and unmanageable in large applications with custom error pages.

possible duplicate of https://github.com/sveltejs/kit/issues/519 (which somehow didn't turn up in my initial search)


chore(crates):  Replace actor references by component in crates

Feature or Problem
I wanted to rename actor references to component in the crates folder.
I started by renaming the crate wasmcloud-actor by wasmcloud-component and propagating modifications in other crates and tests.
I flag this PR to WIP because at the moment there is still work to do across the rust crates. Furthermore, there are still tests to fix (mainly from wash-cli crate).
Related Issues
This is part of  Rename actor to component: #1827
Release Information
Could be part of the next release as soon as the PR is validated I guess.
Consumer Impact
Rust crates which are owned and published by the wasmcloud project.

Sure, I edited the commit message.

This is an awesome amount of work to do this rename! Tons of internal doc updates, though this PR also includes changes to public APIs and CLI output, so let's make sure to mark this as a breaking change to go out with the next minor release of wash/other crates.
This is releasable in a patch version of the host as it's not affecting any host output/flags

I pushed one more little change since you approved the changes. I renamed the struct Actor by Component in the file crates/component/src/lib.rs.
As it is already a lot of changes, I think I will make another PR to replace the remaining references to actor from the crates.
@brooksmtownsend how do I mark this as a breaking change?

@brooksmtownsend Thank you for the explanations. I rebased the branch on main and marked it Ready for Review.

@nitame sorry about this, I didn't merge this in time and it looks like there's a couple more conflicts. Would you mind giving this a rebase, or do you mind if I just rebase it and merge after? This is all approved it just needs those conflicts resolved and I'll press the button 😄

@brooksmtownsend no worries 😄 I rebased and resolved conflicts.

Talked to @brooksmtownsend and merging for him

Thank you @nitame !


current issues with event segmentation code

I'm fixing these now but want to keep track of them somewhere:

plots under Plot the within and across correlation values as a function of k  and Plot the ratio of within/across ratio as a function of k start at x=0, but should start at x=2
normalizing video within/across correlation ratios to correct for non-monotonic behavior has typo.  Current: t=list(map(lambda x: x[0]/(x[1]-min(m[1])), m))[1:].  Should be: t=list(map(lambda x: x[0]/(x[1]-min([i[2] for i in m)), m))[1:]
wrong k-value is being subtracted for regularization parameter (should be (i+2)/250 instead of i/250 because of indexing of list t
maxk_video is returning wrong number of events because of the way t is indexed.  Should be maxk_video = np.argmax(t) + 2 instead of maxk_video = np.argmax(t)
possibly extend normalization used with video k-optimization to recalls as well?
the format of how embeddings is created and saved does not match how it's indexed when loading it into trajectory_analysis_and_fig.ipynb

after discussion, proceeding with the following edits:

fixing a bug in the correction in the video k-optimization to deal with negative across-event correlations
applying that same correction to the recall k-optimization for the following reasons:

want to establish the approach in a way that can account for recall trajectories with k-values that yield negative across-event correlations (like the video did)
applying the correction for negative correlations changes the optimal k-value for both the video and recalls.  Using it in one instance and not the other would improperly alter the proportion of video events recalled by participants

resampling the recall trajectories to the timeseries of the video before performing event segmentation (as is currently reported in the paper) for the following reasons:

It's important to allow for the possibility that participants recalled more events than were presented (e.g., as with PLI's/XLI's in list-learning literature).  Resampling helps fix some issues with large potential k-values
Resampling helps deal with participants who use relatively few sentences to recall relatively large numbers of events (a small number of recall windows limits the range of k-values when not resampled).
the negative correlation correction step sometimes increases the number of recalled events.  Resampling mitigates this for unrealistically high numbers of recall events
it increases the relationship between our results and independently created, manually generated event recall scores (i.e., "reality")
it causes the distribution of event recall performance across participants by our method to better reflect that of the manually generated event recall scores

Link to figure pdf: [link]

These have all either been addressed via bug fixes or are no longer relevant now that we've moved to using wasserstein distance for the K-optimization procedure.  Closing


Arch board feedback

Description
Address two pieces of arch board feedback:

Rename disableAuthorityValidation and getAuthorityValidation to include SafetyCheck
Reintroduce falling back to 2017 AppService endpoint if new Identity variables are not present.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API changes have been detected in com.azure:azure-identity. You can review API changes here
API changes
-         public T disableAuthorityValidation()
+         public T disableAuthorityValidationSafetyCheck()


Allow regex for requires

PR checklist

[x] Addresses an existing issue: #3392
[x] New feature, bugfix, or enhancement

[x] Includes tests

[ ] Documentation update

Overview of change:
Modifies noVarRequiresRule and noRequireImportsRule with an option to specify a regex for which to ignore the rule. Modeled after the ignore-module in noImportSideEffectRule.
Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

Closing due to age and inactivity. Feel free to re-open or create a new pull request if you decide to continue working on this.


Wildcard transitions do not work when using strict mode

Description
When using strict mode wildcard transitions don't work
Expected Result
Wildcard transitions work
Actual Result
Error message like Error: Machine 'main' does not accept event 'no'
Reproduction
https://codesandbox.io/s/new-smoke-sjwzt
Additional context
xstate version 4.7.0-rc7

@davidkpiano I'm curious what's the reason behind strict mode? I understand why it handled failed promises in a different way (although I believe that strict behavior in this regard should become the default and the only one in the future), but I don't quite understand why it also disallows "unaccepted" events. Isn't this exactly the true beauty of statecharts that we can send events to a machine and if it happens to be in a state which doesn't accept a particular event it then is ignored (like in the default setting)? If one truly wants to have this "strict" behavior wouldn't it be better to just recommend wrapping .send with a guarding code in user land?

@Andarist Interestingly enough, devs can create their own strict mode with a wildcard transition on the root state that can throw an error if an event is not handled in any other state.
I'll mark this as "invalid" since it doesn't make sense to use wildcard transitions and strict mode together.

@davidkpiano I see just now that it was even mentioned in the docs that on unknown events an error will be thrown


wip: use amount from monero-rs

Closes #46

This is working, gonna just wait for the PR on monero-rs to be merged, and then I'll fix the commits and Cargo.toml :)

@TheCharlatan I cannot assign you as a reviewer.


List all names MatchVariable accept for New-AzApplicationGatewayFirewallExclusionConfig

Please list all names MatchVariable parameter accepts for New-AzApplicationGatewayFirewallExclusionConfig. It's difficult to find what are the exact  names.

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

ID: c0c27cd7-15fd-7358-9740-734cfa6b641f
Version Independent ID: 9cbc5678-98e1-3c21-df0a-81d8b7e21241
Content: Web application firewall request size limits and exclusion lists in Azure Application Gateway - Azure portal
Content Source: articles/application-gateway/application-gateway-waf-configuration.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@TomaszOledzki , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@TomaszOledzki , You can get the list of all names that Match Variable accepts from the portal. The Match variable has a drop down box carrying all the names and it depends on the match type your are selecting from the portal.
Example:

@SubhashVasarapu-MSFT Thanks for the replay. You're pasting screen from WAF v2 I presume but docs on the site I'm referring is for WAF v1.
I think docs should be updated to either show WAF v2 or list the names for WAF v1 as suggesting in the first comment.
Kind regards,
Tomasz

@TomaszOledzki , Appreciate your patience. The above post i have mentioned earlier talks about the WAF for frontdoor. I am assigning this to the author for updating the document accordingly. @vhorne , Could you please provide your insights on this issue.

#reassign:winthrop28

I will be closing issues that are older than a year in order to research and cross reference them individually. I am recording each issue and compiling a list of articles to review. If you feel there is an urgent need to address something related to this GitHub issue, please open a new issue and I will reply in the new thread. Please also reference this GitHub issue number in the new issue. Thanks. #please-close


Fix Qt problems closing preview windows with the X button

WHen the X button is clicked to close a preview window, it wasn't running through the usual cleanup operations, resulting in a nasty hang when trying to quit Python. This adds some "close event" handling to perform the necessary steps.
We also need to tell the Picamera2 object that the preview disappeared spontaneously, and make sure the Qt App stays running even when the only window is closed in this way, otherwise we have no thread to create a new one.
Signed-off-by: David Plowman<EMAIL_ADDRESS>

@chrisruk You probably want to look over this PR too! I can't say it's not a bit hacky at the moment...
The problems are easy enough to demonstrate, you just need to run this:
from picamera2 import Picamera2, Preview
picam2 = Picamera2()
picam2.start_preview(Preview.QTGL)

Then click the X (close window) button in the corner of the preview window and finally try to quit Python. It hangs, and when you finally hit Ctrl-C it dies nastily.
For the second test, run the same thing and click the X button again. Now try and restart another preview by repeating picam2.start_preview(Preview.QTGL). (All this works fine if you use picam2.stop_preview() instead of clicking the X button.)
The first problem I fixed by adding a closeEvent method to do some cleaning up. Though I don't think I've done it in a very "Qt" way, which would probably want a base class that implements the method conventionally, and which we then derive from.
For the second problem, I found myself wanting the Qt "app" not to quit when you close its only window. Not sure if there's a better way, disabling the "close" button altogether might not be a bad idea either. Anyway, it's probably one to sleep on and think about next week!

Hi David, sure will take a look at this next week.  Have a good weekend!

Looking at this, this morning, sorry for the delay

No probs, actually I'm trying to do a "better" version of this. It seems more complicated, but I hope leaves things in a slightly tidier state. I'll post it in a bit.

@chrisruk Hi again - don't know if you spotted it, I just posted a very heavily revised version of this (link above as a "mention"). I think that one is probably better... although there's a lot more in it... maybe let me know what you think. Thanks!!


chore(dev-deps): update semantic-release monorepo

This PR contains the following updates:

Package
Type
Update
Change

@semantic-release/release-notes-generator
devDependencies
minor
12.0.0 -> 12.1.0

semantic-release
devDependencies
patch
22.0.6 -> 22.0.7

Release Notes

semantic-release/release-notes-generator (@semantic-release/release-notes-generator)
v12.1.0
Compare Source
Features

add support for ESM presets (#544) (53c18ce)

v12.0.1
Compare Source
Bug Fixes

deps: update dependency read-pkg-up to v11 (#557) (5756c18)

semantic-release/semantic-release (semantic-release)
v22.0.7
Compare Source
Bug Fixes

deps: update dependency read-pkg-up to v11 (#3047) (f09e55f)

Features

support extending ESM configuration files (#3036)(6900865)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


Update README.md

links not opening up in the browser

@wrussell1999 can you review this, please

@wrussell1999 Can you Assigned me?


Hi, this was a recent bug v2.28.0 and it's fixed from v2.28.1. Please re-install the storybook module if you're getting this issue. We're extremely sorry for the trouble.
rm -rf node_modules
npm install

thanks @mnmtanish !

I hope we could close this now.


Convert example to typescript

This converts the example from Javascript to Typescript.

@Jmgr this branch was merged into feature/upgrade-to-latest-version, was that intentional? The changes are not in main

@Jmgr this branch was merged into feature/upgrade-to-latest-version, was that intentional? The changes are not in main

Nope, that's definitely a mess-up.


WebConfigPropertyCollection: Escape curly braces2

Pull Request (PR) description
Fix issue #430 by doubling up curly braces enabling the use of URLRewrite variables.
This Pull Request (PR) fixes the following issues

Fixes #430

Task list

[X] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[X] New/changed code adheres to DSC Community Style Guidelines.

This change is

This is a rework of PR599 that had strange issues after the rebase was pushed.

@johlju is there anyway I can help move this along?

I agree that you could remove the extra check for the data type since its already a string.
It's been a couple of years since I submitted the original PR, so I don't remember the details but I believe curly braces are supposed to be variables, and my intent was to make sure that variables with url rewrite were fixed and not to escape braces in all scenarios.
I think it checks for single vs double braces to avoid re-escaping something that's been escaped already and to reduce the odds of unintended issues. There may been a specific reason for it too, but I don't remember and I no longer have any IIS machines to test against.


Fix postgres reference

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/324
Originally created by @bencodezen on 2019-01-24 19:11:01

Tutorial says both postregresql and sqlite

View 5 previous comments from the original issue on GitLab


Unit tests need to be updated

When running unit tests on OTS, there are many failures. Some of the mock() methods with Mockito don't work (e.g., because they mock a final method). Also, unit tests fail because the 'contract' of the methods has changed.

An example of the problem is that Gtu is now a class, with a final getSimulator() implementation, instead of an interface with a public getSimulator() definition. In order for Mockito to mock this method, add a file called mockito-extensions to /src/test/resources of the projects with this problem, with one line:
mock-maker-inline

and update the Mockito version to 5.x.

In OdAppliertest in ots-core, the test public void gtuFractionTest() does not work, and stops with a NullPointerException. I do not understand enough of the OdApplier to repair the test. @WJSchakel : can you take a look (it's not urgent).

@averbraeck the test was missing characteristics generators, which are no longer given by default in OdOptions as they rely on default GTU types etc. The test is fixed now.

And now the @Test is also uncommented (it got commented after I merged changes before committing).

Thanks! It works. All other issues also tackled (although there are still quite a few tests of which parts were already commented out). New version of OTS will be released this afternoon.


Error in 'Context' class

Hello,
although I'd love to use the framework, I can't because of an error in the 'Context' class:
Test Suite 'DBEventTest' started at 2015-06-19 17:38:48 +0000
Test Case '-[HappyFreelancerTests.DBEventTest testACD]' started.
fatal error: unexpectedly found nil while unwrapping an Optional value

When looking there, it is obvious why:
public class Context {

    private let stack: Stack!
    public let managedObjectContext: NSManagedObjectContext!    // the underlying managed object context

    private var ___background: Bool = false                     // background context machinery (you did not see it)

    internal var contextOptions: ContextOptions { return self.stack.contextOptions }

Because 'stack' is nil, the return statement fails when accessing the 'contextOptions' property.
Best regards
Hardy

ACD assumes the name of the .xcdatamodeld file should be the same as the project name. This stumped me as well but got past it. See #48.

Well, I created a new CoreData Model, and copied the entities from the old to the new one. Had Xcode create the NSManagedObject class files. Had ACDgen to create the DataContext file. Assigned all of them to both, App and Test Targets.
Still no change, the error situation continues.

Make sure the CoreData Model is the same name as the project. Xcode tends to replace spaces with underscores but that's not gonna work here. Also in your AppDelegate where Xcode puts all the CoreData logic, make sure the SQLite and CoreData models names match the project name as well. Maybe you can upload the project your working on and I can take a look, or take a look at the demo project to match with yours.

Anything to consider in the test target?

All other conditions are fulfilled, actually...

Add a new Model Version for . xcdatamodeld file can solve this problem.

Hello. I'll close this issue, but if you have further questions, please feel free to reopen it or open another one.


[Bug, Prod, GoodID] - ubipool claim does not show up on dashboard feed

Description
When having claimed from a ubi-pool, the transaction shows with name 'undefined' and amount '0'
Steps to reproduce

not sure, not reproducible on dev

Which network did you use?
Celo
Supporting data

@vldkhh

verified on prod via Nigerian users


crash on excessive video switching

I have an issue / a question (pick one) about Vlc.DotNet.
Generic information

Vlc.DotNet version : 3.0.0-develop294
Vlc.DotNet project used : WinForms
libvlc version : x86 & x64, by downloading from VLC FTP, testet with version 3.0.2, 3.0.6, 3.0.8
.net version : 4.7
Project language : C#
Project build architecture : AnyCPU
Operating system : Windows 10 x64

Summary
I'm having issues with fast video switching/loading and stopping. My application plays a video (.mp4) for a given amount of time and then stops and shows nothing for the same amount of time and then restarts this loop. For example with this time set to 10 seconds, it will play the video for 10 sec, then blank for 10 sec, then again the video for 10 sec. Until it is stopped by the user or in this case a crash. The crash happens (with the play time set to 10sec) after 1-2 hours of looping. If the time is set to 1.5 seconds, it will crash in about 15 minutes.
I reported this bug to vlc/videolan: https://trac.videolan.org/vlc/ticket/22427 . Turns out the problem is not directly in VLC but in usage of VLC. As tigros999 commented on the issue, it can be bypassed in VLC.dotNet

I was able to reproduce this bug while letting the app run from VS debug for a long time breaking on all exceptions.
The exception was: Managed Debugging Assistant 'LoaderLock?'
Message=Managed Debugging Assistant 'LoaderLock?' : 'Attempting managed execution inside OS Loader lock. Do not attempt to run managed code inside a DllMain? or image initialization function since doing so can cause the application to hang.'

It was on myVlcInstance = new VlcInstance?(this, myLibraryLoader.GetInteropDelegate?<CreateNewInstance?>().Invoke(utf8Args.Length, utf8Args)); of Vlc.DotNet?.Core.Interops\VlcManager?.cs. Continuing execution got the C++ crash and app closed.

So from that I found this: https://stackoverflow.com/a/31322244/990618

VlcManager?.cs becomes:

...
private VlcLibraryLoader myLibraryLoader;
private VlcInstance myVlcInstance;
...
	ThreadStart threadRef = new ThreadStart(() =>
	{
		this.myLibraryLoader = VlcLibraryLoader.GetOrCreateLoader(dynamicLinkLibrariesPath);
	});
	Thread myThread = new Thread(threadRef);
	myThread.IsBackground = true;
	myThread.Start();
	myThread.Join();
...
	threadRef = new ThreadStart(() =>
	{
		myVlcInstance = new VlcInstance(this, myLibraryLoader.GetInteropDelegate<CreateNewInstance>().Invoke(utf8Args.Length, utf8Args));
	});
	myThread = new Thread(threadRef);
	myThread.IsBackground = true;
	myThread.Start();
	myThread.Join();

[x ] I confirm that my issue doesn't happen in VLC itself. --> see VLC bug report: https://trac.videolan.org/vlc/ticket/22427

Please send me a minimal reproducing sample so I can check this out locally.
The code that was suggested doesn't feel like a proper fix for me:

Are you really destroying/recreating media player each time you need one? This method is only called when creating a new player. Calling Stop() and Play() again should be enough for your use case.
The fact that this happens randomly makes me think about some race condition/memory leak somewhere. Creating a thread can probably introduce a latency in the code, which would "save" you from the crash, but it's not a real fix.
If it was a deadlock issue, then you could do that kind of code around your "new VlcMediaPlayer" code, and see if that fixes anything.


FusedAdam Hangs during Initialization

I am testing the accelerate training code and I modify the optimizer of: https://github.com/huggingface/accelerate/blob/main/examples/nlp_example.py#L141
to the FusedAdam (from deepspeed.ops.adam import FusedAdam), then after "python ./nlp_example.py" I find that the progress hands on the line of "optimizer = FusedAdam(params=model.parameters(), lr=lr)".
What are the possible reasons?
https://github.com/huggingface/accelerate/tree/main/examples

Hi @lipiji I was unable to reproduce the bug. Could you provide information about the environment setup you are using?

Closing now. Feel free to reopen if needed.


register_dataclass does not handle __init__ methods

Description
import dataclasses
import jax

@jax.tree_util.register_dataclass
@dataclasses.dataclass
class X:
    x: int

    def __init__(self, *, y: bool):
        self.x = int(y)

leaves, treedef = jax.tree_util.tree_flatten(X(y=True))
jax.tree_util.tree_unflatten(treedef, leaves)
# TypeError: X.__init__() got an unexpected keyword argument 'x'

Looks like the classic unflatten-via-init error.
x-ref https://github.com/patrick-kidger/jaxtyping/issues/277.
System info (python version, jaxlib version, accelerator, etc.)
JAX 0.4.36.

I think this is working as intended: the docs are clear on this:

class attributes represent the whole of the object state, and can be passed as keywords to the class constructor to create a copy of the object.

Maybe we should more explicitly state that custom __init__ methods may break this contract.

Hmm, fair point that register_dataclass does document this limitation.
Tagging @kvablack here, probably the best thing to do is to use some other class-as-pytree implementation for your use-cases.

Tagging @kvablack here, probably the best thing to do is to use some other class-as-pytree implementation for your use-cases.

Now that register_dataclass is part of core JAX, I feel like it is quite desirable to use the built-in implementation. I will likely fork jaxtyping to try and handle https://github.com/patrick-kidger/jaxtyping/issues/277.


JVM libzip.so file lost

Thanks for this tool.
When I want to try docker-slim, I have a problem, I want to confirm if there is a problem with my usage, or is this a bug.
My Dockerfile is here
FROM maven:3.6.1-jdk-8-alpine AS builder

WORKDIR /app

COPY pom.xml /app/
RUN mvn dependency:go-offline
COPY src /app/src
RUN mvn -e -B package

FROM builder AS dev

RUN apk add --no-cache vim

FROM openjdk:8-jre-alpine

COPY --from=builder /app/target/gs-spring-boot-0.1.0.jar /

CMD [ "java", "-jar", "/gs-spring-boot-0.1.0.jar" ]

you can find it on https://github.com/tao12345666333/spring-boot-hello-world/blob/master/Dockerfile
(MoeLove) ➜  ~ sudo docker-slim  build  --http-probe local/spring-boot:1
[sudo] tao 的密码：
docker-slim[build]: state=started
docker-slim[build]: info=params target=local/spring-boot:1 continue.mode=enter
docker-slim[build]: state=inspecting.image
docker-slim[build]: info=image id=sha256:01e4898d1141763400d39111609425ba6232b8bf42f46a6033fdb2b7306dc75b size.bytes=103166686 size.human=103 MB
docker-slim[build]: info=image.layers index=0 name='local/spring-boot' tags='1'
docker-slim[build]: state=inspecting.container
docker-slim[build]: info=container status=created id=32b94537fe4fb4b68b0278384f7857d2a2a87b1bbf76dfb63dd3c98ac047ea58
docker-slim[build]: info=cmd.startmonitor status=sent
docker-slim[build]: info=event.startmonitor.done status=received
docker-slim[build]: info=container name=dockerslimk_17626_20190530031807 id=32b94537fe4fb4b68b0278384f7857d2a2a87b1bbf76dfb63dd3c98ac047ea58 target.port.list=[] target.port.info=[]
docker-slim[build]: info=prompt message='press <enter> when you are done using the container'

docker-slim[build]: state=processing
docker-slim[build]: state=building message='building minified image'
docker-slim[build]: state=completed
docker-slim[build]: info=results status='MINIFIED BY 3.48X [103166686 (103 MB) => 29652590 (30 MB)]'
docker-slim[build]: info=results  image.name=local/spring-boot.slim image.size='30 MB' data=true
docker-slim[build]: info=results  artifacts.location='/usr/local/bin/.docker-slim-state/images/01e4898d1141763400d39111609425ba6232b8bf42f46a6033fdb2b7306dc75b/artifacts'
docker-slim[build]: info=results  artifacts.report=creport.json
docker-slim[build]: info=results  artifacts.dockerfile.original=Dockerfile.fat
docker-slim[build]: info=results  artifacts.dockerfile.new=Dockerfile
docker-slim[build]: info=results  artifacts.seccomp=local-spring-boot-seccomp.json
docker-slim[build]: info=results  artifacts.apparmor=local-spring-boot-apparmor-profile
docker-slim[build]: state=done
(MoeLove) ➜  ~ docker run --rm local/spring-boot.slim
Error occurred during initialization of VM
Unable to load ZIP library: /usr/lib/jvm/java-1.8-openjdk/jre/lib/amd64/libzip.so
(MoeLove) ➜  ~ docker tag local/spring-boot:1 taobeier/spring-boot-hello-world:1
(MoeLove) ➜  ~ docker push taobeier/spring-boot-hello-world:1

the docker image I have beed pushed to DockerHub taobeier/spring-boot-hello-world:1

Thank you for providing the reference project! It'll be much easier to repro!
I noticed you don't have an EXPOSE instruction in your Dockerfile. I assume you use the -p parameter in your docker run command mapping 8080 to a host port...

I noticed you don't have an EXPOSE instruction in your Dockerfile. I assume you use the -p parameter in your docker run command mapping 8080 to a host port...

So the following error message
(MoeLove) ➜ ~ docker run --rm local/spring-boot.slim
Error occurred during initialization of VM
Unable to load ZIP library: /usr/lib/jvm/java-1.8-openjdk/jre/lib/amd64/libzip.so

Is it due to the absence of the expose port?

After I added expose, the problem still exists.
(MoeLove) ➜  spring-boot-hello-world git:(master) ✗ sudo docker-slim  build  --http-probe local/spring-boot:6
[sudo] tao 的密码：
docker-slim[build]: state=started
docker-slim[build]: info=params target=local/spring-boot:6 continue.mode=enter
docker-slim[build]: state=inspecting.image
docker-slim[build]: info=image id=sha256:8e5ccc8213f5fc3a3134dba8547b6dd543e7ae1ebad55084c334322efa919b15 size.bytes=103166686 size.human=103 MB
docker-slim[build]: info=image.layers index=0 name='local/spring-boot' tags='6'
docker-slim[build]: info=image.exposed_ports list='map[8080/tcp:{}]'
docker-slim[build]: state=inspecting.container
docker-slim[build]: info=container status=created id=8be77d9ce1a969255f7c4280e988235cae9f7dfe70a0c8e3d46f4e4c40f6afa8
docker-slim[build]: info=cmd.startmonitor status=sent
docker-slim[build]: info=event.startmonitor.done status=received
docker-slim[build]: info=container name=dockerslimk_5779_20190531054415 id=8be77d9ce1a969255f7c4280e988235cae9f7dfe70a0c8e3d46f4e4c40f6afa8 target.port.list=[32770] target.port.info=[8080/tcp => <IP_ADDRESS>:32770]
docker-slim[build]: info=prompt message='press <enter> when you are done using the container'

docker-slim[build]: state=http.probe.starting
docker-slim[build]: info=http.probe.call status=error method=GET target=http://<IP_ADDRESS>:32770/ attempt=1 error='Get http://<IP_ADDRESS>:32770/: dial tcp <IP_ADDRESS>:32770: connect: connection refused' time=2019-05-31T05:44:27Z
docker-slim[build]: state=processing
docker-slim[build]: state=building message='building minified image'
docker-slim[build]: info=http.probe.call status=error method=GET target=http://<IP_ADDRESS>:32770/ attempt=2 error='Get http://<IP_ADDRESS>:32770/: dial tcp <IP_ADDRESS>:32770: connect: connection refused' time=2019-05-31T05:44:35Z
docker-slim[build]: state=completed
docker-slim[build]: info=results status='MINIFIED BY 3.48X [103166686 (103 MB) => 29652590 (30 MB)]'
docker-slim[build]: info=results  image.name=local/spring-boot.slim image.size='30 MB' data=true
docker-slim[build]: info=results  artifacts.location='/usr/local/bin/.docker-slim-state/images/8e5ccc8213f5fc3a3134dba8547b6dd543e7ae1ebad55084c334322efa919b15/artifacts'
docker-slim[build]: info=results  artifacts.report=creport.json
docker-slim[build]: info=results  artifacts.dockerfile.original=Dockerfile.fat
docker-slim[build]: info=results  artifacts.dockerfile.new=Dockerfile
docker-slim[build]: info=results  artifacts.seccomp=local-spring-boot-seccomp.json
docker-slim[build]: info=results  artifacts.apparmor=local-spring-boot-apparmor-profile
docker-slim[build]: state=done
(MoeLove) ➜  spring-boot-hello-world git:(master) ✗
(MoeLove) ➜  spring-boot-hello-world git:(master) ✗ docker image ls local/spring-boot.slim
REPOSITORY               TAG                 IMAGE ID            CREATED             SIZE
local/spring-boot.slim   latest              92cc0239d51e        21 seconds ago      29.7MB
(MoeLove) ➜  spring-boot-hello-world git:(master) ✗ docker run -p 8080:8080 --rm local/spring-boot.slim
Error occurred during initialization of VM
Unable to load ZIP library: /usr/lib/jvm/java-1.8-openjdk/jre/lib/amd64/libzip.so

I think it is missing the file libzip.so when compressing the image.

Thank you for the additional info! I have managed to repro the condition. It looks like it's OpenJDK and Alpine related.
I almost managed to make it work by including a few extra files minifying the image (using the --include-path parameter). Here's what my docker-slim build command looks like: docker-slim build --http-probe --include-path=/usr/lib/jvm/java-1.8-openjdk/jre/lib/currency.data --include-path=/usr/lib/jvm/java-1.8-openjdk/jre/lib/tzdb.dat --include-path=/usr/lib/jvm/java-1.8-openjdk/jre/lib/rt.jar --include-path=/usr/lib/jvm/java-1.8-openjdk/jre/lib/amd64/libzip.so my/java-spring-boot-hello-world. You can try it yourself if you are curious... Just replace my/java-spring-boot-hello-world with your image name. This minified container prints the Spring/Spring Boot logo failing trying the load the net library: java.lang.UnsatisfiedLinkError: no net in java.library.path.


Node Decommissioning Doc Update

(Looking at https://cockroachlabs.com/docs//v20.1/training/node-decommissioning.html)
Flag --decommission has been deprecated, we should update our docs to use 'cockroach node decommission' then 'cockroach quit' instead.

Thanks, @irfansharif. This is in progress. @taroface will wrap it up once he's back from vacation. https://github.com/cockroachdb/docs/pull/7304


Record the assigned port in the virtualcluster

What would you like to be added:
when the api server port is allocated，the port should record in virtualcluster
Why is this needed:
configmap is used to store data, but it maintains the relationship between ports and virtual control plane. If someone modifies the configmap manually, the relationship between ports and virtual control plane will be confused

/assign


add holidays in 2022 year

add holidays in 2022 year

Thank for your help. Can you change main.py for suitable your new change in setting.ini?

Sorry,I am busy with other work.
Vào Th 3, 21 thg 12, 2021 lúc 09:17 Nguyễn Đình Tiến Nam <
@.***> đã viết:

Thank for your help. Can you change main.py for suitable your new change
in setting.ini?
—
Reply to this email directly, view it on GitHub
https://github.com/namndt/AutoKBYT/pull/1#issuecomment-998413297, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB4KWL7STTT422QBQEWCXEDUR7PTBANCNFSM5KPBZGEA
.
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.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Được gửi từ iPhone Đức Thọ


Local setup guide

Hey @dvargas92495 ,
Really excited by what you're building! Wanted to set the project up locally to test out a few ideas that I had! It would be great to document instructions for local setup and connecting with roam.
I don't know much about javascript/typescript so I have no idea where to start 😅

Managed to set it up :) will raise a PR.

Hey @makalaaneesh !
I was hoping to flesh out https://github.com/dvargas92495/roam-js-extensions/blob/master/CONTRIBUTING.md as soon as someone expressed interest, so thanks for reaching out!
There is so "local setup" per se. My flow is that each PR deploys to https://davidvargas.me/${PR_NUMBER}/${script.js}. This doesn't work as well for forks, but we could talk more on how to facilitate that.
Another option is to npm install and then npm run build locally, then copy and paste the minified output into Roam for testing. Let me know if there's anything else I could do to help!

Ah, okay! How do you test your code before creating a PR though? Do you run it directly in your browser console?
Here are the steps I had followed:

Install npm

brew install node

Install dependencies

npm install

Build Project

npm run build

Serve the JS

npm install -g http-server
http-server in the project dir

setup src in roam/js to http://<IP_ADDRESS>:8080/build/${name}.js

My dev cycle has been testing in the PR... haha. So there was no testing locally. Basically deployed to the branched path and test in my personal roam directly


 tag uses "title" and currently fails the title attribute scenario

Summary
<abbr> tags, as standard, use a "title" attribute to provide an expanded version of an abbreviation: https://www.w3.org/TR/html52/textlevel-semantics.html#the-abbr-element
Currently, unexpected failures are happening on our abbreviation tags because the current code does not include  in the whitelist for tags that are allowed to use a "title": https://github.com/bbc/bbc-a11y/blob/8ccd025ba26b3b11cc22aafdb81ed6b04a566c21/lib/standards/tests/titleAttributesOnlyOnInputs.js
Expected Behaviour
<abbr title="Sophie Beaumont">Sophie</abbr>
Should not trigger a failure of the scenario "Non-input element has title attribute", as <abbr> with a title is semantically correct, and screen readers will read out the title of the tag instead of the abbreviation.
Current Behaviour
Non-input element has title attribute: //div[@id='u765878345439809']/div/div[2]/div/div/div/div[2]/ul[1]/li[13]/span[2]/abbr
Above error occurs in any situation where <abbr> tag is used with a title attribute.
Possible Solution
Add <abbr> to the whitelist in the above mentioned test.
Context & Motivation
We use a list of players in our Lineups for our scorecard pages: https://www.test.bbc.co.uk/sport/rugby-union/match/EVP2632964
Each player's name uses an <abbr> tag which allows us to use a shortened version of the name for visual display purposes only. (No truncation or wrap needed on mobile breakpoint)
Thanks. :)

Hi @sbeaumontweb
Thanks for raising the issue.
Your possible solution looks like the best! Do you want to raise a PR to fix it?
Thanks!

" and screen readers will read out the title of the tag instead of the abbreviation."
I should rectify this incomplete statement - screen reader settings can be changed as such that they read out the title of the abbreviation instead of the content of the tag. Some are not set this way. However, responsible use of the title attribute by the standards (don't repeat information or use for critical information) shouldn't harm accessibility as the information inside the tag would still be valid.

Most AT won’t read the title which is probably a good thing as the information is intended to be suppmental not critical.
If it’s important information a label would be more appropriate. If it’s fluff that also isn’t present visually, I don’t understand what it’s there for.
The test cannot check for appropriate or contextual use so I am very hesident to consider it a complete test.
At the least I think this should warn the user to check or ask themselves the question on if the content in the title is essential to understanding the content / context.
This has been written about a fair bit in other places:
https://openinclusion.com/blog/presenting-abbreviations-acronyms-for-screen-reader-users/
This answer also links out to the spec and the various comments about relying on titles on abbr etc.
https://stackoverflow.com/a/32892825
Some additional views and comments from twitter here: https://twitter.com/jamieknight/status/1069581095128231936?s=21
BBC-a11y isn’t a validator of the specification. It’s a tool which tries to help teams deliver great experiences.
While an abbr and optional title fall within the spec, the risk of misuse and the relatively edge case nature may make it simpler to do without then.
I’d support moving it from an error to a warning, as an in between position.


catkin build davis_ros_driver error

The platform is ubuntu16.04 , ROS is Kinetic.
When I catkin build davis_ros_driver, it prompts this:
[ 71%] Built target davis_ros_driver_nodelet
/home/uavlab/dvs_ws/devel/lib/libcaer.solibusb_strerror
collect2: error: ld returned 1 exit status
CMakeFiles/davis_ros_driver.dir/build.make:260: recipe for target '/home/uavlab/dvs_ws/devel/lib/davis_ros_driver/davis_ros_driver' failed
make[2]: *** [/home/uavlab/dvs_ws/devel/lib/davis_ros_driver/davis_ros_driver] Error 1
CMakeFiles/Makefile2:113: recipe for target 'CMakeFiles/davis_ros_driver.dir/all' failed
make[1]: *** [CMakeFiles/davis_ros_driver.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
[davis_ros_driver] <== '/home/uavlab/dvs_ws/build/davis_ros_driver/build_env.sh /usr/bin/make --jobserver-fds=3,6 -j' failed with return code '2'
Failed   <== davis_ros_driver [ 0.6 seconds ]
[build] There were '1' errors:
Failed to build package 'davis_ros_driver' because the following command:
Command to reproduce:
cd /home/uavlab/dvs_ws/build/davis_ros_driver && /home/uavlab/dvs_ws/build/davis_ros_driver/build_env.sh /usr/bin/make --jobserver-fds=3,6 -j; cd -
Path to log:
cat /home/uavlab/dvs_ws/build/build_logs/davis_ros_driver.log
Exited with return code: 2
Please give me advise, thank you!!!

Looks like a linking error related to libusb_strerror and libcaer (see also #68).
Official libcaer build is now used, see README for further details.


[BUG] Bugs on the poll page

Question cards are not matched the design.

Typo

Words next to radiobuttons are lowercased.

Typo.

Validation error message is uppercased.

The list of all questions is shown just on the top of the page at 0-1024px width of the window.
But in design there are two different pages for questions list and for questions sliders.

Clicking on breadcrumbs reloads the page if the page is already opened.
https://github.com/fictadvisor/fictadvisor-web/assets/111194749/d7ee0798-8044-44f5-bc62-3c33c05d2f95


Update dependencies and workflows

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

Project coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #480   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          139       139
  Lines         4286      4286
  Branches       288       288
=========================================
  Hits          4286      4286

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


How do you set branches

I keep getting this error because i'm not using the master branch
      code: 'ERELEASEBRANCHES',
      details: 'A minimum of 1 and a maximum of 3 release branches are required in the [branches configuration](https://github.com/semantic-release/semantic-release/blob/master/docs/usage/configuration.md#branches).\n' +
        '\n' +
        'This may occur if your repository does not have a release branch, such as `master`.\n' +
        '\n' +
        'Your configuration for the problematic branches is `[]`.',

I don't have enough information to know what might be happening here. Are you able to link to GitHub Actions logs and your repository?

Hmm, could be because main isn't in the default list of release branches^1. This looks related: https://semantic-release.gitbook.io/semantic-release/usage/configuration#branches

Im pretty sure ive set main in the   .releaserc
https://github.com/RampedIndent/polylauncher
https://github.com/RampedIndent/polylauncher/actions/runs/4233076111/jobs/7353573764#step:16:592

The one time i managed to make semantic-release run successfully i had to set the branches in the workflow file
https://github.com/RampedIndent/polylauncher/actions/runs/4232522924/workflow

This workflow ignores your repository's semantic-release configuration^1. From that link:
    # Clear out any existing semantic-release configuration.
    #
    # This step is necessary in my projects that use a .releaserc.json file
    # with a non-stock semantic-release plugin, because semantic-release throws
    # an error when the plugin is referenced in the configuration but not
    # present in the node_modules directory.
    #
    # Therefore, a quick and drity workaround is to clear out the configuration
    # before invoking semantic-release.

This means semantic-release's default value for branches^2 is used. We could hack in support for main branches specifically, but it seems more desirable to support the host repository's arbitrary branches configuration.

This workflow ignores your repository's semantic-release configuration1. From that link:

is that step only necessary for .releaserc files that use non stock semantic release plugins?
if so maybe there could be a toggle to disable that step if it is not necessary

the command yq '.branches | {"branches": .}' .releaserc.yaml  > .releaserc.json will isolate the branches and convert them to json

Implimented a fix for .json , .yml, and .yaml files that should work

Should probably just delete the plugins section

This is the approach I was leaning towards too, but it turns out we can override the plugins section with a CLI option. Please give it a shot and let me know if the issue is resolved!

it runs successfully after i commented out some additional sections in my .releaserc

Hmm, were these the necessary changes? https://github.com/RampedIndent/polylauncher/commit/e9415015203be7b28086dcd158698d51c370ad4c

I got this error so i commented out all the stuff that i figured wasn't required.
https://github.com/RampedIndent/polylauncher/actions/runs/4273772915/jobs/7439904486

Thanks for the link, copying the error here for posterity
Run : calculate next semantic-release version
[6:42:41 AM] [semantic-release] › ℹ  Running semantic-release version 20.1.0
[6:42:41 AM] [semantic-release] › ✔  Loaded plugin "verifyConditions" from "semantic-release-export-data"
[6:42:41 AM] [semantic-release] › ✔  Loaded plugin "analyzeCommits" from "@semantic-release/commit-analyzer"
[6:42:41 AM] [semantic-release] › ✔  Loaded plugin "generateNotes" from "semantic-release-export-data"
[6:42:41 AM] [semantic-release] › ✘  An error occurred while running semantic-release: Error: Cannot find module '@semantic-release/git'
Require stack:
- /home/runner/work/polylauncher/polylauncher/noop.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1075:15)
    at resolveFileName (/home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/resolve-from/index.js:29:39)
    at resolveFrom (/home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/resolve-from/index.js:43:9)
    at module.exports (/home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/resolve-from/index.js:46:47)
    at loadPlugin (file:///home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/semantic-release/lib/plugins/utils.js:59:54)
    at default (file:///home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/semantic-release/lib/plugins/normalize.js:18:24)
    at file:///home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/semantic-release/lib/plugins/index.js:76:11
    at Array.map (<anonymous>)
    at file:///home/runner/work/_temp/.semantic-release-action_next-release-version/node_modules/semantic-release/lib/plugins/index.js:75:34
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/home/runner/work/polylauncher/polylauncher/noop.js' ]
}

I found a way to mitigate this issue, using the same pattern: 538bd6f
Thank you for the bug reports!


Add a security policy

Hey there!
I belong to an open source security research community, and a member (@ranjit-git) has found an issue, but doesn’t know the best way to disclose it.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

Hello @JamieSlome. If this security issue does not involve personal credentials or secret tokens you can very well disclose it here.

@Yomguithereal - you can find the report here:
https://huntr.dev/bounties/1529a17d-3d26-41e9-906a-8ebc3593a513/


Issues in Google Auth

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
Issue1

Enable google auth in Ember UI
Navigate to Google Auth page
In Site Access section, select Restrict access to only authorized users and groups.
add a user and click on Save
Navigate to another page say Global --> Roles and navigate back to Authentication.
Site Access section does not have content
Error in console:

component.js:103 Uncaught TypeError: Cannot read property 'forEach' of undefined
    at n.accessModeChanged (component.js:103)
    at g (index.js:223)
    at g (core_object.js:98)
    at Function.e.create (core_object.js:593)
    at e.t.create (index.js:566)
    at r.i.create (index.js:3958)
    at Object.evaluate (runtime.js:3058)
    at e.t.evaluate (runtime.js:1724)
    at e.t.evaluateSyscall (runtime.js:5229)
    at e.t.evaluateInner (runtime.js:5185)

Issue2

After steps in Issue 1, Logout.
Error seen on Rancher:

And user is unable to login after.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5-head commit id: 41f173a
Installation option (single install/HA): Single

On 2.5.8-rc11

Enable google auth in Ember UI
Navigate to Google Auth page in Dashboard/Cluster Explorer
In Configure who should be able to login and use Rancher section, select Restrict access to only authorized users and groups.
add a user and click on Save
Any further search in the Configure who should be able to login and use Rancher section does not give any error.
User is able to logout and log back in.


Profile v3: Show restricted transition notes in profile page notes when access

Follow on to https://github.com/studentinsights/studentinsights/pull/2004, part of https://github.com/studentinsights/studentinsights/issues/1998 and https://github.com/studentinsights/studentinsights/issues/1990.
Who is this PR for?
HS counselors and housemasters
What problem does this PR fix?
With the changes to the profile page to move restricted notes inline, and to move transition notes into the profile notes list, restricted transition notes hadn't been included yet.
What does this PR do?
This copies code from EventNote into TransitionNote to use the same guards for the content of restricted notes on the model.  It updates Authorizer to handle TransitionNote models (copying from the code handling EventNote models) and updates the authorizer method about reading transition notes to apply more narrowly to who can write transition notes.  Note that the UI path for writing transition notes will be removed when profile v2 is removed, and there's not yet a replacement.
Relatedly, refactors <RestrictedNotePresence /> to require a URL for fetching the restricted note content, and update the various call sites to parameterize this for EventNote or TransitionNote data.
Screenshot (if adding a client-side feature)
with access, profile v3 showing both transition notes

with access, profile v3 click to fetch restricted transition note text

with access, profile v3 click to fetch event note restricted text (unchanged)

with access, profile v2

Checklists

[x] Author checked latest in IE - Student Profile v2
[x] Author checked latest in IE - Student Profile v3
[x] Author improved specs for code in need of better test coverage
[x] Author included specs for new code

selfie


Allow custom upload function

This PR adds the possibility to use a custom upload function. This could be useful when you can't use an ajax call because you have to call a REST API with a JWT Token or you want to use a third-party service in the cloud.
Example of use:
var editor = new EasyMDE({
    uploadImage: true,
    imageUploadFunction: function(file, onSuccess, onError) {
        try {
            var imageUrl = uploadImage(file)
            onSuccess(imageUrl)
        } catch (error) {
            console.log(error)
            onError(error)
        }
    },
});

@Ionaru  I've created this PR based on https://github.com/Ionaru/easy-markdown-editor/pull/104 in order to resolve what I commented here: https://github.com/Ionaru/easy-markdown-editor/pull/104#issuecomment-514144909.
Please let me know if it makes sense to you. Thanks 😉

#104

Thanks @Ionaru  😉


esp32s3, restarting cdc software serial doesn't work (IDFGH-12813)

Answers checklist.

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

IDF version.
v5.1.4 and 5.2 release branch
Espressif SoC revision.
ESP32-S3 (QFN56) (revision v0.1)
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
Development Kit.
esp32s3 tdisplay s3
Power Supply used.
USB
What is the expected behavior?
I expect to be able to call pseudo code that looks like
tinyusb_driver_install
tusb_cdc_acm_init

// use cdc serial

tusb_cdc_acm_deinit
tinyusb_driver_uninstall

// sleep a bit or do something else usb mass storage
tinyusb_driver_install
tusb_cdc_acm_init

and I would expect the cdc software serial working again but it doesn't show up with lsusb after a cdc and driver deinit and uninstall
I get no errors, everything is ESP_OK but the device doesn't show up - an esp_restart is required for things to show up again.
using
espressif/tinyusb: "^0.15.0~9"
espressif/esp_tinyusb: "^1.4.4"
and tried with esp-idf tags v5.1.3, v5.1.4 and 5.2 release branch
What is the actual behavior?
I would expect to be able to cycle the usb software cdc on/off/on
Steps to reproduce.
Something like https://github.com/espressif/esp-idf/tree/master/examples/peripherals/usb/device/tusb_serial_device could be modified to also deinit cdc device software and uninstall tinyusb
Debug Logs.
logs of first init vs next init don't seem to differ.

More Information.
basically same issue (1) as https://www.reddit.com/r/esp32/comments/1c50bn1/esp32s3_tinyusb_issue/ which to me also doesn't seem resolved (same as OP)

Hi @greenaddress
Thanks for opening the issue, as you pointed out, I was already trying to solve it on Reddit.
I will try to reproduce it and let you know.

@peter-marcisovsky thanks for getting back
Any news on reproducing? would it be of help if we provided some minimal/hello world for both 5.2 and 5.1.4 to show the issue?

Hi @greenaddress ,
we have all necessary information and we are able to reproduce the behavior, thanks a lot.
Also, to re-start tinyusb driver we need to shutdown it properly (this feature is not implemented in upstream as well: https://github.com/hathach/tinyusb/blob/a1abbd70cbc00878a34f0b7db24805fc78b6b8d8/src/tusb.h#L142)
Based on the requirements, this could be done in several different ways, so we will return back with some solution asap.
Meanwhile, do you have specific requirements and do you plan to change the USB device class during the tinyusb driver uninstall-install routine?

@roma-jam I plan to use usb storage as a host (msc) when not using cdc as device.
msc works but i can't bring back cdc (even if i never used msc)

@roma-jam any news or suggestion/work around?

Hi @greenaddress ,
Sorry for such a long delay with a reply.
Currently, there is not so much to do, because to use tinyusb driver again (which means to restart the device but with another configuration) we need to:

Disable all enabled class drivers (if there is one)
Disable USB interrupt
Deinit underlying hardware layer and flush all internal variables in tinyusb driver to be able to start again.

So, when the according API are not available, the only one option is to fulfill a whole reset of the mcu unit.
To make it available without the whole reset, changes should be done in:

esp_tinyusb component (https://github.com/espressif/esp-usb/tree/master/device/esp_tinyusb) ;
tinyusb component (https://github.com/espressif/tinyusb);


Add aad obo integration test case

Add aad obo integration testcase azure-spring-boot-test-aad-obo module, public azure-spring-boot-test-aad resource.
`

/azp run java - spring - tests

/azp run java - spring - tests

/azp run java - spring - tests

I'm going to merge this PR because the error seems unrelated to the obo integration test.
@chenrujun please take a look at this test error: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=665037&view=logs&j=364de821-210b-596e-0928-4e0d2b619091&t=4057129a-b461-5555-fc9f-95b5eaf73fb2&l=257.

@saragluna .
Got it. I already let JackLu to fix it.


fix: DEV-2671: Change rotate anchor of bbox

Hi @juliosgarbi!
This PR was created in response to a PRs in upstream repo:

https://github.com/heartexlabs/label-studio-frontend/pull/750

/frontend build lsf master

/frontend build lsf master


All cards float when hovered

All cards 'pop' or float when you hover over them. Can this be made optional?

Adding "no-transition" to class fixed this.