Generalize switch statements to have expressions with other types

At the August 2020 meeting with advanced topics / new proposals for P4_16, I presented a much more incremental addition to P4_16: one way to add tagged unions to the language.
Part of that was generalizing the switch statement to allow other types of expressions, which seems useful in its own right, independent of tagged unions.  This PR includes only changes for generalizing switch statements, and nothing about tagged unions.
While the diffs show lots of text changes in the section on switch statements, I have tried carefully not to change the behavior or restrictions of the switch statement that were there before when the switch expression is t.apply().action_run.  If you ignore the new possible types of the switch expression, the meaning should be exactly what it was before these changes.

Note that as far as implementation in p4c goes, I believe that several targets will be able to implement switch expressions more efficiently than the equivalent 'daisy chain' of if-then-else if statements, because of the guarantee that all switch cases are mutually exclusive, and have labels that are compile-time known values.  Thus it would be best if p4c preserved such new switch statements in p4c's IR, and not replace them with if-then-else if statements.
Mihai did a quick look at the IR during the meeting where I presented this, and he commented at that time that it appears that the current p4c IR for switch statements allows switch expressions to have other types.  If so, I hope that helps make it easier to implement p4c changes for this extension to the language, while preserving switch statements in the IR.

Anything, that allows the programmer to express their actual intent more clearly and unambiguously is a good thing. As @jafingerhut said, one thing it will certainly do is to allow compiler to better deduce mutual exclusivity of certain code paths (if-then-elseif-elseif-else chains are OK either way, but sequences of if-then statements are more difficult to prove to be mutually exclusive).

As some background details for questions that might come up in P4 language design work group meetings about the current behavior of the switch statement, I have made a list of test programs in p4lang/p4c that exercise various kinds of switch statements: https://github.com/jafingerhut/p4-guide/tree/master/switch-statement

Note that this sentence in this PR (after 4 commits it was still there, at least), is new, and perhaps controversial: "The last switch label must be followed by a block statement."
The current p4c implementation as of 2020-Sep-04 seems to treat a sequence of one or more labels at the end of a switch statement, with no following blocks statement, as if you had written { } as the blocks statement after them.  I would recommend that the LDWG decide on either this as the explicitly defined meaning of such syntax, or declare it a compile-time error: https://github.com/p4lang/p4c/issues/2527

A suggestion for how the proposed generalized switch statement might be implemented, by transforming the new switch statements into the existing switch statements, is described here: https://github.com/jafingerhut/p4-guide/blob/master/switch-statement/implementing-generalized-switch-statements.md
It does not seem to me that this belongs as part of the P4_16 language specification itself, but if it seems useful to include it as a companion document in this repository, perhaps linked from the spec, I could arrange that.

Would you like to try your hand at implementing this as a midend pass?

@mbudiu-vmw I am not sure, but I suspect it may be faster if I work at persuading someone at Intel to implement it, versus trying to implement it myself :-)

You would have no problem persuading me to do it, but I thought this could be a very good exercise for learning more about the compiler internals. I can provide assistance. First, of course, we would have to add support for generalized switch statements, and then this elimination scheme.

At the 2020-Sep-14 language design work group meeting, several possible additional generalizations to switch statements were discussed.  I will create a separate issue for those, and a separate PR.  At least Mihai suggested that this PR might move forward as a first step towards an even more general switch statement, with the further generalizations done as a separate PR after this one was approved and merged.
I have made one additional commit after that meeting, which is to explicitly say that if the last switch label has no body after it, the behavior is the same as if there were an empty body { } after it.  That is the behavior that p4c implements today.  There did not seem to be any strong opinions on whether this case should be legal or not, so this latest commit is the conservative approach of explicitly defining the current p4c behavior, rather than changing it to be illegal.

Implemented here: https://github.com/p4lang/p4c/pull/2575

I reworked the implementation to not require 'case'; the modified grammar is in the compiler pull request. If this PR is accepted we should amend the grammar as shown in the code changes.

@mbudiu-vmw At the last P4 language design work group meeting, you asked about adding an entry to the list of changes section near the end.  Just a few minutes ago I pushed commit #8 on this PR that adds such a section.  There was not yet such a section for the next version, so I semi-arbitrarily named the section with version 1.2.2 -- that can be edited later if a different next version number is desired.

@jafingerhut I think we agreed to merge this change in the spec after you apply some minor fixes. I would also suggest using version 1.3.0, since this is what I used for the if in parsers commit I just merged, so they end up in the same section, which we can rename later.

@jafingerhut I think we agreed to merge this change in the spec after you apply some minor fixes. I would also suggest using version 1.3.0, since this is what I used for the if in parsers commit I just merged, so they end up in the same section, which we can rename later.

@mbudiu-vmw I am unaware of any fixes proposed to this PR except the most recent one, which was to add a change log, which has been done, and I just pushed a change to make the version number 1.3.0 as you suggested.

@mbudiu-vmw I am unaware of any fixes proposed to this PR except the most recent one, which was to add a change log, which has been done, and I just pushed a change to make the version number 1.3.0 as you suggested.

I wasn't sure whether the note on the implementation stays external to the spec or not.

I wasn't sure whether the note on the implementation stays external to the spec or not.

If you are happy you can in principle merge this.

If you are happy you can in principle merge this.

@mbudiu-vmw I will merge it in as it is now (with next spec version being 1.3.0 as you recommend).
The implementation note in a separate file is linked by URL from the spec, which seems like the best approach to me, given that it is only a description of one possible way to implement this language feature, and isn't a requirement of any kind.

@mbudiu-vmw I will merge it in as it is now (with next spec version being 1.3.0 as you recommend).
The implementation note in a separate file is linked by URL from the spec, which seems like the best approach to me, given that it is only a description of one possible way to implement this language feature, and isn't a requirement of any kind.


Updating sinon to 15.0.4

Updating to sinon 15.0.4

Closing, since this is addressed by https://github.com/iTwin/appui/pull/396


Rename namespace from GDAXClient to GDAXSharp [#71]

Have performed this namespace rename, and updated the solution / project and folder names to match.

I don't use git much.... so bit rusty on it... not sure why it's showing commits from earlier PR ?
I created this branch after merging back from dougdellolio/master
The actual changes are the last 2 in the above list....

thanks for this. not quite sure why it shows older but you will have to rebase and fix the merge conflicts. then I can merge this

I don't know what I've done but my repository keeps showing the old commits as being ahead of this repository, even after a rebase....
Going to trash my repository and start again.... so I'll open an new pull request once the new repo / branch has the correct git info in it.....


Exporting name property instead of module instance

Could you remove the yarn file?

Thanks!!

Published as<EMAIL_ADDRESS>👍


Analytics Data Shows in Realtime but Not in Report

Hi, thanks for the great library, when I tried to send to google analytics, it shows up in realtime but doesn't shows up in "Events".
Is there any configuration that I missed?
I tried to print the POST body in this library, and here's one of request body:
{
    "client_id": "dc8e0e44-fa4d-47ce-9c80-59aa8e256fe5",
    "events": [
        {
            "name": "status_other",
            "params": {
                "status": "ERROR_CONNECTION",
                "session_id":<PHONE_NUMBER>,
                "engagement_time_msec": 0
            }
        }
    ],
    "user_id": "109"
}

I'm having the same issue.


Error: MONGO_URL must be set in environment

meteor app and mongodb on different GCE VMs in same project:
gce-deployment.sh:
................
export MONGO_URL='mongodb://root:****@<IP_ADDRESS>:27017'
(CHECKED: from mongo-client connecting OK)
[PM2] Starting /opt/bundle/main.js in fork_mode (1 instance)
[PM2] Done.
┌────────────┬────┬──────┬──────┬────────┬─────────┬────────┬─────┬───────────┬──────────┐
│ App name   │ id │ mode │ pid  │ status │ restart │ uptime │ cpu │ mem       │ watching │
├────────────┼────┼──────┼──────┼────────┼─────────┼────────┼─────┼───────────┼──────────┤
│ my_app │ 0  │ fork │ 8676 │ online │ 0       │ 0s     │ 4%  │ 20.1 MB   │ disabled │
└────────────┴────┴──────┴──────┴────────┴─────────┴────────┴─────┴───────────┴──────────┘
.pm2/logs/my_app-error-0.log:
Error: MONGO_URL must be set in environment
at Object. (packages/mongo/remote_collection_driver.js:36:11)
at Object.defaultRemoteCollectionDriver (packages/underscore/underscore.js:750:1)
at new Mongo.Collection (packages/mongo/collection.js:103:40)
at AccountsServer.AccountsCommon (packages/accounts-base/accounts_common.js:23:18)
at new AccountsServer (packages/accounts-base/accounts_server.js:18:5)
at meteorInstall.node_modules.meteor.accounts-base.server_main.js (packages/accounts-base/server_main.js:9:12)
at fileEvaluate (packages/modules-runtime/.npm/package/node_modules/install/install.js:153:1)
at require (packages/modules-runtime/.npm/package/node_modules/install/install.js:82:1)
at /opt/bundle/programs/server/packages/accounts-base.js:1838:15
at /opt/bundle/programs/server/packages/accounts-base.js:1849:3
Error: MONGO_URL must be set in environment

Did you also change your MONGO_URL here?
https://github.com/Q42/meteor-gce/blob/master/startup.sh#L71
Did you upload your changed startup.sh to your bucket?
To debug you could add an echo $MONGO_URL somewhere in your script to see if it picks up the right environment variable.

Thanks for the quick response)
of course!
export MONGO_URL='mongodb://' in place,
startup.sh is fresh and in gs://bucket.
But where i can insert debug row
echo $MONGO_URL,
if 'export' rows is last in startup.sh?

Hmm, it looks like more people have problems with pm2 not picking up environment variables in some cases:
https://github.com/Unitech/pm2/issues/83
Some people mention that setting the environment variables in a process.json and starting pm2 with that works:
http://pm2.keymetrics.io/docs/usage/application-declaration/
Could you try that?

I try
MONGO_URL='mongodb://root:cLx9g8L3@<IP_ADDRESS>:27017' pm2 start main.js --name 'my-app'
still:
Error: MONGO_URL must be set in environment

I try
MONGO_URL='mongodb://root:*****@<IP_ADDRESS>:27017' ROOT_URL='http://localhost' pm2 start main.js --name 'my-app'
.....................................................................................................
Note: you are using a pure-JavaScript implementation of bcrypt.
While this implementation will work correctly, it is known to be
approximately three times slower than the native implementation.
In order to use the native implementation instead, run
meteor npm install --save bcrypt
in the root directory of your application
so, need change startup.sh
cd bundle
(cd programs/server && npm install)
to:
cd bundle
(cd programs/server && meteor npm install --save bcrypt)
?

something I have no luck with your script  )
fresh experiment: prepare process.json:
env: {
      "HOME":"/home/",
      "MONGO_URL":"mongodb://root:***@<IP_ADDRESS>:27017",
      "ROOT_URL":"http://localhost",
      "MAIL_URL":"smtp://*****:*****@smtp.googlemail.com:465",
      "PORT":"3000"
    }

pm2 start process.json:
Error: Cannot find module 'fibers'
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Function._load (/usr/lib/node_modules/pm2/node_modules/pmx/lib/transaction.js:62:21)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (/opt/bundle/programs/server/boot.js:1:75)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)

The bcrypt warning is probably unrelated to this problem.
I think you're on the right track with the process.json.
Usually when it can't find fibers something went wrong with npm install. Check if that ran correctly and if the node_modules is in the right location.

Local app on local machine work ok,...
may be, incorrectly meteor build process?
how check it?

Hmm, I just noticed this script still installs Node 0.10: https://github.com/Q42/meteor-gce/blob/master/startup.sh#L17
Meteor now uses Node 4 as far as I know, so try replacing line 17 in the script with this:
curl -sL https://deb.nodesource.com/setup_4.x | bash -

I have done it from the very beginning:
my node row is:
curl --fail -ssL -o setup-nodejs https://deb.nodesource.com/setup_4.x &&
  bash setup-nodejs &&
  apt-get install -y nodejs build-essential nodejs nginx

make by hand:
curl -sL https://deb.nodesource.com/setup_4.x | bash -
apt-get install nodejs

Didn't work (
Error: Cannot find module 'fibers'

add to startup.sh row
npm install fibers source-map-support underscore semver
and now app launched with hand typing
pm2 start ..
only,
not automatically )

Glad you (almost) got it working.
I haven't used this script anymore in a long time, so for maybe a better up to date version you could also try Meteor Up: https://github.com/kadirahq/meteor-up

I liked your docker free method, it more improvable)


Vips can (now?) be compiled without webp support - Our class should detect that

I updated the vips version on travis from 8.6.3 to 8.10.6 and now gets the following error:
1) WebPConvert\Tests\Convert\Converters\VipsTest::testWebpsave
WebPConvert\Convert\Exceptions\ConversionFailedException: VipsOperation: class "webpsave" not found
/home/travis/build/rosell-dk/webp-convert/src/Convert/Converters/Vips.php:206
/home/travis/build/rosell-dk/webp-convert/tests/BaseExposer.php:43
/home/travis/build/rosell-dk/webp-convert/tests/BaseExposer.php:30
/home/travis/build/rosell-dk/webp-convert/tests/BaseExposer.php:46
/home/travis/build/rosell-dk/webp-convert/tests/Convert/Exposers/VipsExposer.php:37
/home/travis/build/rosell-dk/webp-convert/tests/Convert/Converters/VipsTest.php:190

Fixed in a4e23e6..8a66dd2


Fix MVFCheckOverflowCalculation_test. Set mvf-bu-retarget.py to 90 days.

Some rework to tidy up a few bumps in the testing:

reduced fork retarget period to 3 months (this makes testing faster later revert to 6 months if necessary)
Added "-forkheight=999999" option to the  p2p-versionbits-warning.py test.
MVFCheckOverflowCalculation_test - replaced a hard code

reduced fork retarget period to 3 months

If you mean to do this for the whole client instead of just the Python test, then I think you need to adapt src/mvf-bu.h, this line:
HARDFORK_RETARGET_BLOCKS = 25920,    // 180*144 blocks
Please also remove test-framework/arith.py unless you're using some of it in the tests (I don't see any commits which do, which makes me presume you accidentally committed it).

If you plan on making use of the code which you've check in as arith.py, we need to go over that file in detail and establish which parts are licensed under what licenses.
Also, it needs cleanup, a file header, etc.
We should discuss this on Slack in more detail.


MUIC-601: [Android] Vertical CTA button position

https://glia.atlassian.net/browse/MUIC-601
Note:
Rough implementation of Vertical Alignment support
Can test with enabling toggle in settings screen
REVIEW AFTER MUIC-610 gets merged

rebased onto develop


DestinyGuardianRankDefinition missing from DefinitionsEnumExtensions

The DestinyGuardianRankDefinition is missing from the extensions to be able to call ToStringFast.

Sorry for late response, will be adding those soon

2.10.1 should have that issue resolved


x64 designer preview problem

Hi there
I am using you library in my project and it is great how things get easier with it!
But now my problem: Our solution is configured as x64 platform. I noticed that the designer is not working correct and it shows all items in Column 0 and Row 0. I created a small Test app and noticed if I use AnyCpu or x86 everything is as expected.
Is this a known problem and if so, how can I solve it?
Thanks for you help and regards!

That’s an interesting issue. Perhaps AutoGrid needs to be compiled as x64 as well to work.
Try grabbing the source code for AutoGrid and adding it to your sample project and remove the NuGet package references to AutoGrid.

Tried you suggestion but the problem remains the same... I am afraid that this could be a VS Designer issue. I found similar problems regarding XAML designer and x64 only applications. Maybe you have an other idea how to solve?

Here’s another thing you can try. I believe there’s another AutoGrid style control on NuGet. See if you can find that and if it has the same problem then we can probably chalk it up to a designer issue.
There’s also different versions of Visual Studio to try. Maybe VS 2019 Preview fixes this.
I’ll be honest, I don’t have a Windows VM setup easily accessible at the moment to test any of this out. Otherwise I could give some of these things a try myself. Sorry.

Thanks for your help! Yesterday I tried other AutoGrid controls as well and the result was the same so I guess its really a designer problem :-( And I had the same idea and tried VS2019 as well but the designer has the same behavior.
So I think the only way is to switch to x86 or AnyCPU while using the designer, but that is not really possible in our project...
Anyway thanks again for you help!

There’s more things you can try.
I think the designer calls in to the UserControl constructor. Try putting a call to the AutoGrid’s PerformLayout() function. You’ll have to name your AutoGrid something and use that reference. Also make sure to put it after the InitializeComponent() call.
Also, have you tried using the StackPanel control in this library instead? I found in my WPF time I hardly ever used an AutoGrid. I almost exclusively used StackPanels. If you are laying out forms, then you might want to consider creating a FormControl type that controls the width of the label. This helps keep all the fields in alignment.

You might also be able to use Blend instead of VS for the designer functionality. It may or may not share the same designer under the covers.


Refactor selector

To make it more extensible

Need to take a look at this with a sober mind.

Maybe this will be simpler when service locator will be gone?

Maybe this will be simpler when service locator will be gone?

Def, so can't wait to drop SL

Maybe this will be simpler when service locator will be gone?

Def, so can't wait to drop SL

Does this make sense while SL is still there?

Does this make sense while SL is still there?

IMHO yes


React buttons default to type submit and will submit html form

Our current implementation uses a form whereby users can submit patients details and save them by submitting a form.
I have noticed that when clicking any of the buttons on react component it will attempt to submit our form. I believe this is because in React if you do not specify a type, then the buttons default type is submit.
If possible can the buttons specify type button so we do not get this issue?

Thanks for reporting @ryanlewis94 - I think this should be a straightforward change. Let me look into it

Sorry for the delay on this one @ryanlewis94, we've now released a fix in v7.0.5:

https://github.com/rcpch/digital-growth-charts-react-component-library/releases/tag/v7.0.5
<EMAIL_ADDRESS>

@mbarton it looks like gradient-labels-button does not specify a type and therefore is still causing the issue

Hmm yeah sorry about that - I thought I had covered off all the buttons. I'll have a look now

@ryanlewis94 I've released v7.0.6 which fixes the gradient labels button too

<EMAIL_ADDRESS>https://github.com/rcpch/digital-growth-charts-react-component-library/releases/tag/v7.0.6


simplify some csproj settings

We close this PR - since we have implemented all the suggestions in another major update #144
The reason for not merging this PR was due to the changes to the Demo project -> Not Needed.
Thx for the input and contribution.


Gracefully fail when looking for data in empty columns expecting nested objects

If a table is empty, and we try to get data from that table, and we are trying to traverse an object, the object's members will not be found and athena will throw an error.
We need to catch conditions where these tables are empty and prevent them from loading other data.

This has been resolved by the table builder pattern.


Rename TODO to TODO.md

(more standard, plays better with binary exclusions, etc)

Thank you for the PR. :+1:


Is it possible to update the theme dynamically via StyleProvider?

I'm using the latest 2.012 version. I ran node node_modules/native-base/ejectTheme.js and cloned two themes from the platform theme, day and dark. The only difference between the day and dark themes is the text color.
dark.js
    // Text
  textColor: '#ed2f2f',
  inverseTextColor: '#222',
  noteFontSize: 14,

day.js
   // Text
  textColor: '#000',
  inverseTextColor: '#fff',
  noteFontSize: 14,

The first time the component is mounted, the theme works correctly. However, when I click on a button to dynamically update the theme from day to dark, the <Text/> component doesn't update the text color correspondingly.
The debug tool shows everything seems to be fine. The componentWillReceiveProps in StyleProvider manages to set the new theme state, but the Text component just doesn't update its style on theme change.
import * as React from "react";
import {Container, StyleProvider,Text} from "native-base";
import getTheme from './native-base-theme/components';
import dark from './native-base-theme/variables/dark';
import day from 'native-base-theme/variables/day';

@observer
export class Screen extends React.Component {
   render() {
       let theme = getTheme(mobxStore.getViewMode == 'dark'?dark:day)
       return (
           <StyleProvider style={theme}>
               <Container>
                   <Text>Some Text</Text>
               </Container>
           </StyleProvider>
       )
   }
}

Any help please? Is my approach the right way of doing it?

@stonecold123 Check out https://github.com/GeekyAnts/NativeBase-KitchenSink/tree/v2.0-alpha1
This might help

This line to be specific:
https://github.com/GeekyAnts/NativeBase-KitchenSink/blob/v2.0-alpha1/js/AppNavigator.js#L321
On Wed, Apr 26, 2017 at 3:04 PM, shivrajkumar<EMAIL_ADDRESS>wrote:

@stonecold123 https://github.com/stonecold123 Check out
https://github.com/GeekyAnts/NativeBase-KitchenSink/tree/v2.0-alpha1
This might help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GeekyAnts/NativeBase/issues/772#issuecomment-297319147,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABpzOVXtp3LnbTkFx4VBmP1DnfGTFdKVks5rzw-dgaJpZM4NFuLg
.

@shivrajkumar @sanketsahusoft
I only have an Android. Even though the theme variable has changed from 'day' to 'dark', the color still fails to update across the apps. I notice that in the kitchen sink demo, the theme switching function is only available on IOS. Does Android support it?

@stonecold123 There is no point in supporting Platform and Material for Android as the Platform design of Android is actually the Material design.

Closing this issue due to inactivity
Please reopen if the issue continues

@SupriyaKalghatgi I'm having this issue as well could it get reopened? The theme does not update on any currently mounted components, only on ones that mount after the theme switches.

@Talor-A Can you post in here your code snippet along with RN and NativeBase versions?

@SupriyaKalghatgi is this related to #910 ?

So I've realized that this issue only happens when using themed components inside a react-navigation screen. is it something to do with react context?

Having the same issue. May provide a gist if you'd like.

I'd like to express interest in this too.

Resurrecting this fossil... But I'm interested in being able to switch the theme variable and have all components use this new theme. However, even if the root component (above StyleProvider) re-renders with a new theme, child components won't use the new theme

Experiencing this as well, portions of my components are updating, but not the entire thing. Some of my view tags re render, but not all within the same component? It makes no sense


not able to create a new scheduler

when ever I try to create a new scheduler (in lightmin embedded mode), I get the following message presented in the UI:

checking the UI does not help at all, there is no error or log statement in the logs.  But the error message in the browser console might be of help:
Request URL: http://localhost:9090/job-scheduler?init-add-scheduler=init-add-scheduler&scheduler-type=CRON&application-instance-id=ba3d5d66
Request Method: POST
Status Code: 403

For some reason it seems I'm not allowed to create a scheduler, but why? Does the user need to have a specific role?
Maybe I'm doing something else wrong, but there is not much help on that screen, e.g.:

why can I choose the Status? what is this for?
what is the expected syntax of the cron expression?
what does it mean to use the synchronous executor?
how do I add my own job incrementer?

This might also be related, I get the following message during startup in the logs and I'm not sure whether I need to do something or not:
org.quartz.core.QuartzScheduler          : Scheduler meta-data: Quartz Scheduler (v2.3.1) 'quartzScheduler' with instanceId 'NON_CLUSTERED'
  Scheduler class: 'org.quartz.core.QuartzScheduler' - running locally.
  NOT STARTED.
  Currently in standby mode.
  Number of jobs executed: 0
  Using thread pool 'org.quartz.simpl.SimpleThreadPool' - with 10 threads.
  Using job-store 'org.quartz.simpl.RAMJobStore' - which does not support persistence. and is not clustered.

@tuxdevelop is there a way how I can create/persist schedulers programmatically?

@imod with the up comming Version of spring batch lightmin we added a New CRON Validation. This should hopefully fix your Problem.
The expected CRON Statement should be a spring cron expression.

@adaptivecodes adding a scheduler via ui does works now, but still: there is no documentation about:

why can I choose the Status? what is this for?
what is the expected syntax of the cron expression?
what does it mean to use the synchronous executor?
how do I add my own job incrementer?

I think these things should be documented in the UI directly (not in some other online documentation). This is a problem in general, many icons are used, but none of it has some kind of a tool tip or any other information indicator.
and my last question was about if it is possible to add such a scheduler programmatically (e.g. at startup of the app in embedded mode)

why can I choose the Status? what is this for?
This is an option to start a Scheduler in a desired state. For example you can immediately RUN your CRON if you choose Status RUNNING.

what is the expected syntax of the cron expression?
We are using QuartzSyntax with the requirement of 6 Arguments. If you want to know the possible syntax you can have a look here:  http://www.quartz-scheduler.org/documentation/quartz-2.3.0/tutorials/crontrigger.html

what does it mean to use the synchronous executor?
In Spring you have the possibility to execute Tasks synchron or asynchron.
If you choose to execute your task Synchronous your Task will be executed via SyncTaskExecutor. Your task will be executed on the calling Thread meaning that you are participating in the same thread context with all its benefits (same Class Loader etc.).
Choosing the asynchronous option will result in generating a new Thread each time your task will be executed!

how do I add my own job incrementer?
At the moment there is no possibility in adding an own JobIncrementer. If this is a desired feature you can add a feature request.

Thanks for the suggestion with the tool tips. I will open an UI improvement ticket and add some "in application documentation".
Sure you can add as many scheduler programmatically as you want. You can use the EmbeddedAdminServerService for this.

All questions answered. This issue is closed due to inactivity.


BUG: No proper margins in text and image in hero section of events page

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?

please assign me this issue
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-ext contributor
[X] I have starred the repository
[X] I want to work on this issue
[ ] I check the issue doesn't exist

duplicate #277


Is Windows 2000 supported?

I got this error on Windows 2000 SP4 (32-bit):

Hi,
Unfortunately, no. As of Go 1.3, Windows 2000 is no longer supported by Go.


More control over max_max_new_tokens and memory behavior from generate args

============================================================================================================ short test summary info ============================================================================================================
FAILED tests/test_manual_test.py::test_chat_context - NotImplementedError: MANUAL TEST FOR NOW
FAILED tests/test_manual_test.py::test_upload_one_file - NotImplementedError: MANUAL TEST FOR NOW -- do and ask query of file
FAILED tests/test_manual_test.py::test_upload_multiple_file - NotImplementedError: MANUAL TEST FOR NOW -- do and ask query of files
FAILED tests/test_manual_test.py::test_upload_url - NotImplementedError: MANUAL TEST FOR NOW -- put in URL box https://github.com/h2oai/h2ogpt/ (and ask what is h2ogpt?). Ensure can go to source links
FAILED tests/test_manual_test.py::test_upload_arxiv - NotImplementedError: MANUAL TEST FOR NOW -- paste in arxiv:1706.03762 and ask who wrote attention paper. Ensure can go to source links
FAILED tests/test_manual_test.py::test_upload_pasted_text - NotImplementedError: MANUAL TEST FOR NOW -- do and see test code for what to try
FAILED tests/test_manual_test.py::test_no_db_dirs - NotImplementedError: MANUAL TEST FOR NOW -- Remove db_dirs, ensure can still start up and use in MyData Mode.
FAILED tests/test_manual_test.py::test_upload_unsupported_file - NotImplementedError: MANUAL TEST FOR NOW -- e.g. json, ensure error correct and reasonable, no cascades
FAILED tests/test_manual_test.py::test_upload_to_UserData_and_MyData - NotImplementedError: MANUAL TEST FOR NOW Upload to each when enabled, ensure no failures
FAILED tests/test_manual_test.py::test_chat_control - NotImplementedError: MANUAL TEST FOR NOW save chat, select chats, clear chat, export, import, etc.
FAILED tests/test_manual_test.py::test_subset_only - NotImplementedError: MANUAL TEST FOR NOW UserData, Select Only for subset, then put in whisper.  Ensure get back only chunks of data with url links to data sources.
FAILED tests/test_manual_test.py::test_add_new_doc - NotImplementedError: MANUAL TEST FOR NOW UserData, add new pdf or file to user_path and see if pushing refresh sources updates and shows new file in list, then ask question about that new doc
=========================================================================== 12 failed, 88 passed, 27 skipped, 1 xfailed, 1 xpassed, 9 warnings in 1636.51s (0:27:16) ============================================================================
(h2ollm) jon@pseudotensor:~/h2ogpt$

/home/jon/miniconda3/envs/h2ollm/bin/python3.10 generate.py --base_model=h2oai/h2ogpt-oasst1-512-12b --load_8bit=True --user_path=user_path --langchain_mode=UserData--debug=True --h2ocolors=False
Console output is saving to: /home/jon/h2ogpt/pycharm.log
bin /home/jon/miniconda3/envs/h2ollm/lib/python3.10/site-packages/bitsandbytes/libbitsandbytes_cuda121.so
Using Model h2oai/h2ogpt-oasst1-512-12b
Prep: persist_directory=db_dir_UserData exists, user_path=user_path passed, adding any changed or new documents
Loading existing db
100%|██████████| 204/204 [00:03<00:00, 64.19it/s]
0it [00:00, ?it/s]
Loaded 203 new files as sources to add to UserData
Loaded 9078 sources for potentially adding to UserData
Existing db, potentially adding 9078 sources from user_path=user_path
Removing 201 duplicate files from db because ingesting those as new documents
No embedding_function provided, using default embedding function: DefaultEmbeddingFunction https://huggingface.co/sentence-transformers/all-MiniLM-L6-v2
Existing db, added 9078 new sources from user_path=user_path
Prep: persist_directory=db_dir_github h2oGPT exists, user_path=user_path passed, adding any changed or new documents
Loading existing db
Prep: persist_directory=db_dir_DriverlessAI docs exists, user_path=user_path passed, adding any changed or new documents
Loading existing db
device_map: {'': 0}
Loading checkpoint shards: 100%|██████████| 5/5 [00:13<00:00,  2.69s/it]
Running on local URL:  http://<IP_ADDRESS>:7860
Running on public URL: https://8a8bcf87f324d57031.gradio.live

This share link expires in 72 hours. For free permanent hosting and GPU upgrades (NEW!), check out Spaces: https://huggingface.co/spaces


Cell annotations to factors

This is basically a duplicate of https://github.com/hubmapconsortium/portal-containers/pull/32 except here the source data file is hard-coded and comes from the vitessce-data S3.

Actually going to close this and the corresponding PR in portal-containers since they are pretty much duplicates and could both be improved. Will replace with a new PR shortly...


Prevent Vue.use to be used without options parameter unless it's specified as optional (with ?)

What problem does this feature solve?
Reproduction link
Please see: https://tinyurl.com/y5mlgqqh
Steps to reproduce

write a plugin
overload Vue.use ( see the given link above for detail )

What is expected?
During compiling it fails, because the types are not matching.
What is actually happening?
It compiles without errors
What does the proposed API look like?
A possible solution is to change the type in vue/types/vue VueConstructor to:
 use(plugin: PluginObject<unknown> | PluginFunction<unknown>, ...options: unknown[]): VueConstructor<V>;
See: https://tinyurl.com/y6anfs8b

Vue.use is already typesafe. You shouldn't override it, Provide the MessagePluginOptions to the generic of PluginFunction like this: https://github.com/vuejs/vuefire/blob/master/packages/vuefire/src/firestore.ts#L104-L107

Thanks for the advise but it still not satisfy all constrains which are intended in the change. Please see: https://tinyurl.com/yykyteu6

Okay, sorry, it took me some time to understand what you meant. Next time, make sure to correctly point what isn't working instead of saying it compiles without errors. I'm updating the title to reflect the intention. This is an easy PR and you already found the solution, can you send a PR with the change and the test? If you cannot that's fine, I will do it

Thanks for your politeness! I try to make a more clearer description next time! Is there any chanel, if I have question, to poke you?

if you need help with the PR message me on Discord but you are pretty much done with  https://tinyurl.com/y6anfs8b 🙂


[stable/postgresql] Ensure intContainer creates the data directory.

Signed-off-by: juan131<EMAIL_ADDRESS>What this PR does / why we need it:
This PR ensures the initContainer used to prepare the right permissions in the volume used to persist data are correct. To do so, it ensure the "data" directory exists so the PostgreSQL container doesn't face the error below:
 04:12:56.72 INFO  ==> pg_hba.conf file not detected. Generating it...
mkdir: cannot create directory ‘/bitnami/postgresql/data’: Permission denied
 04:12:56.73 INFO  ==> Stopping PostgreSQL...

Which issue this PR fixes

fixes https://github.com/helm/charts/issues/14390

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/retest

/hold until https://github.com/bitnami/bitnami-docker-postgresql/pull/149 is merged

/hold
Until bitnami/bitnami-docker-postgresql#149 is merged

/hold cancel

/retest

/hold
Wait for tag 11.3.0-debian-9-r27 to be released

/hold cancel
The tag 11.3.0-debian-9-r27 was released!
/retest

/lgtm


Bypass query params to GET requests for collections

Adds support for pagination and other query parameters for all collection endpoints.
Example:
client = RabbitMQ::HTTP::Client.new("http://localhost:15672/")
client.list_queues('my_vhost', page: 2, page_size: 2)

# for default vhost
client.list_queues(nil, page: 2, page_size: 2)

Thank you!


(1.0.0-1.18.2) Storage Drawers compatibility

Describe the bug
Interacting with Storage Drawers (extracting) triggers the "wrongtool" event
To Reproduce
Steps to reproduce the behavior:

Put item to Drawers
Extract item from Drawers

Expected behavior
Custom blacklist, or native integration with this mod
Which other mods caused the bug .
StorageDrawers-1.18.2-10.2.1

I am aware of the issue, the issue is worked on, I will provide a fix or integration with next update.


Pbmacintyre recipe additions

added a recipe for SMS on new voicemail

Found out how to "Close" the pull request... so maybe you don't have to cancel it on your side...


MM-12890 - Fix markdown preview error

Summary
Fix to not require post attribute, because preview markdown doen't give post attribute.
Ticket Link
[MM-12890] [master] Clicking on "preview" below message box results in blank screen and JS error - Mattermost
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0168918cf7e80e4ff.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0168918cf7e80e4ff


Add Delete button to the generated controller's index page.

#32

Although it's a typical way to implement the batch deletion, it should not be implemented in the controller. Instead, the List Behavior should be extended to support the mass deletion.
Also, the default controller template should not contain code specific to behaviors, because behaviors are optional. If any controller methods are required for a behavior, the controller generator should allow behaviors to inject methods to the controller template. But in case of this feature, implementing it in the list behavior is the best option.
Thanks!

Added to List Behavior. link


Error 500 - Database (storage/database.sqlite) does not exist

Hi,
I've been following these instructions to spin up a docker container running Benotes.
The instructions ask to wget this .env.sqlite.example which contains the following line:
DB_DATABASE=storage/database.sqlite

After following the instructions I was able to get to the login page, but upon typing in the email and password I setup nothing happened.
The docker logs showed the following at every attempt to log in:
[php-fpm:access] <IP_ADDRESS> -  18/Apr/2023:23:58:37 +0000 "POST /index.php" 500 /var/www/public/index.php 34.020 2048 88.18%

I was then able to access the laravel.log file and that's where I found out the following error was being output:
[previous exception] [object] (InvalidArgumentException(code: 0): Database (storage/database.sqlite) does not exist. at /var/www/vendor/laravel/framework/src/Illuminate/Database/Connectors/SQLiteConnector.php:34)

This led me to change the patch on the DB_DATABASE entry in the .env file to:
DB_DATABASE=/var/www/storage/database.sqlite

When doing so everything started working upon rerunning the docker run command.
I just wanted to mention this issue to see if perhaps for the time being the .env example could be updated to avoid anyone else encountering the problem.

Hi,
I'm not sure what caused your error but in my case the default configuration works as expected.
Have you tried to go back using a relative path after you've made it work ?


feat: [CET-1881]: Updating to use the combined et-service chart and 0…

….10.0 CET Release.

As part of the DevSpace work we have combined the et-service and et-receiver charts into a single one "et-service"
This can now be used as part of the release/smp/0.10.0


Feature Request:  Further improvement of Bubble sliders at very low brightness percentages

v3.1.1 was a big step in the right direction with the icon area of the slider now lighting up slightly when the light is on at low percentages. But I don't think it is enough of an indicator yet.  Below is both a Bubble slider and a Mushroom card slider for comparison:
Cards when light is off:

This comparison really shows how great the bubble card is for small screens!   It takes up only 2/3 of the vertical size compared to the Mushroom in its small horizontal mode.  And the Bubble provides full width sliding easily allowing two bubbles be placed side-by-side on a small screen with the slider fully usable!  The Mushroom's short slider just is not practical in side-by-side configuration.  BUT the mushroom card has the icon gray when the state is off. This makes it very easy to determine it is off.
Cards with light at 6% brightness in two different colors:
'

On small screens like a smartphone the Mushroom card icon area is very clear when a light is turned on even at a low percentage. The bubble's icon doesn't change at all and even the icon background area is only lit by a very small percentage.
My Bubble feature request is to "borrow" a couple of the Mushroom's features.   Have the icon gray when the light is off; and lit (either white or in the light's color) when the light is on. Have the icon's background indicate the color of the light, but scale it brighter at small brightness percentages.

Thanks for the comparison! I've never used Mushroom Cards and this is indeed a good example of how I can improve my icons more. I will do that in the next release!

This is a new feature in v1.4.0 and up, thanks again!


Strange performance behaviour

I'm testing a 1m35s audio on the cpu with int8, model large-v2
model = WhisperModel(model_size, device="cpu", compute_type="int8")
The input file is 16khz 16bit mono. With the string
segments, _ = model.transcribe("test.wav", beam_size=1, best_of=1, vad_filter=True)
the transcription time is 1m40s. However, if i pass the language argument which should skip the language detection, like this
segments, _ = model.transcribe("test.wav", language='it', beam_size=1, best_of=1, vad_filter=True)
the transcription time goes up to 1m59s instead of going down.

Can you enable the DEBUG logs and report the output for both runs? See https://github.com/guillaumekln/faster-whisper#logging

INFO:faster_whisper:Processing audio with duration 01:35.000
INFO:faster_whisper:VAD filter removed 00:00.752 of audio
DEBUG:faster_whisper:VAD filter kept the following audio segments: [00:00.752 -> 01:35.000]
INFO:faster_whisper:Detected language 'it' with probability 0.99
DEBUG:faster_whisper:Processing segment at 00:00.000
(text)
DEBUG:faster_whisper:Processing segment at 00:17.000
(text)
DEBUG:faster_whisper:Processing segment at 00:47.000
(text)
DEBUG:faster_whisper:Processing segment at 01:17.000
(text)
runtime: 01:37.214710
INFO:faster_whisper:Processing audio with duration 01:35.000
INFO:faster_whisper:VAD filter removed 00:00.752 of audio
DEBUG:faster_whisper:VAD filter kept the following audio segments: [00:00.752 -> 01:35.000]
DEBUG:faster_whisper:Processing segment at 00:00.000
(text)
DEBUG:faster_whisper:Processing segment at 00:17.000
(text)
DEBUG:faster_whisper:Processing segment at 00:47.000
(text)
DEBUG:faster_whisper:Processing segment at 01:17.000
(text)
runtime: 01:55.967254

Good point, but if i time everything like this:
from faster_whisper import WhisperModel
import datetime
model_size = "large-v2"
print(datetime.datetime.now())
model = WhisperModel(model_size, device="cpu", compute_type="int8")#5
print(datetime.datetime.now())
segments, _ = model.transcribe("test.wav", beam_size=1, best_of=1, vad_filter=True)
print(datetime.datetime.now())
for segment in segments:
print("%s" % (segment.text))
print(datetime.datetime.now())
segments, _ = model.transcribe("test.wav", language='it', beam_size=1, best_of=1, vad_filter=True)
print(datetime.datetime.now())
for segment in segments:
print("%s" % (segment.text))
print(datetime.datetime.now())
The first "segments, _ = model.transcribe(..)" takes 22s, the second one takes less than 1s, the first for cycle takes 1m38s and the second one takes 1m57s

This is expected. The langue setting determines when the first 30 seconds of audio are encoded by the Whisper model.
When the language is not set, the encoding happens in the transcribe method before returning the generator.
When the language is set, the encoding happens in the segment generator.

Your current usage seems fine so I don't have other recommendations. Feel free to open another issue or discussion if you have more questions.


Ruff (and flake8) doesn't report a situation where a variable may be  unbound

Consider the following snippet:
from random import random

def coin_toss():
    return random.randint(0, 2)

def maybe_runtime_error():
    if coin_toss():
        x = []

    # runtime error if coin_toss returned False, but not detected by the linter
    x = list(x)
    print(x)

def runtime_error():
    # always a runtime error, detected by linter
    y = list(y)
    print(y)

There are two problems with this snippet.

In maybe_runtime_error, the name x may not be bound to a variable when we come to evaluate list(x). This means we might get a runtime UnboundLocalError.
In runtime_error, the name y is never bound to a variable when we come to evaluate list(y). We'll always get a runtime UnboundLocalError when calling this function.

Problem (2) is spotted by ruff, but problem (1) is not.
$ ruff --isolated temp.py
temp.py:19:14: F821 Undefined name `y`
Found 1 error.

I was a little surprised by this! But flake8 has the same output:
$ flake8 --version && flake8 temp.py
6.0.0 (mccabe: 0.7.0, pycodestyle: 2.10.0, pyflakes: 3.0.1) CPython 3.10.9 on Linux
temp.py:19:14: F821 undefined name 'y'

I'm not sure if it's fair game for ruff to spot the problem with x---perhaps that's more the remit of a typechecker like mypy. (Though mypy didn't warn about this either when I tried it!) It'd be great if it was feasible to spot this at lint-time.

Configuration
[tool.ruff]
line-length = 88

# See https://github.com/charliermarsh/ruff/#pycodestyle
# for error codes. The ones we ignore are:
#  E731: do not assign a lambda expression, use a def
#  E501: Line too long (black enforces this for us)
#
# flake8-bugbear compatible checks. Its error codes are described at
# https://github.com/charliermarsh/ruff/#flake8-bugbear
#  B019: Use of functools.lru_cache or functools.cache on methods can lead to memory leaks
#  B023: Functions defined inside a loop must not use variables redefined in the loop
#  B024: Abstract base class with no abstract method.
ignore = [
    "B019",
    "B023",
    "B024",
    "E501",
    "E731",
]
select = [
    # pycodestyle checks.
    "E",
    "W",
    # pyflakes checks.
    "F",
    # flake8-bugbear checks.
    "B0",
    # flake8-comprehensions checks.
    "C4",
]

And as for the version:
$ ruff --version
ruff 0.0.240

Appreciate the clear issue! Right now, (1) is outside of our capabilities -- although I'm surprised that Mypy doesn't flag this... Pyright does:
❯ pyright foo.py
/Users/crmarsh/workspace/ruff/foo.py
  /Users/crmarsh/workspace/ruff/foo.py:13:14 - error: "x" is possibly unbound (reportUnboundVariable)
1 error, 0 warnings, 0 informations
Completed in 0.467sec

We could potentially include some kind of "possibly unbound" code for those cases. I'd have to think on it 🤔

I'm surprised that Mypy doesn't flag this

And me! For cross-referencing, it seems like the relevant issues are

https://github.com/python/mypy/issues/686
https://github.com/python/mypy/issues/2400
https://github.com/python/mypy/issues/4019

just wanted to drop in and say that I would find this extremely useful
it's really easy in long Python functions and code blocks when refactoring and moving variables inside nested conditionals or inline functions to nest a variable used somewhere else and only discover at runtime via a Sentry notification days later

Is this still unsupported?


What's the principle of operation?

Hi there, I'm trying to better understand the purpose of this command.
From the README:

It aims to provide a faster alternative to "go list all" to list available packages for import.

Are there any benchmarks comparing the performance?
Assuming it's faster, how does it achieve that, and what's the reason such an optimization could be not sent upstream into cmd/go so that go list all is equally fast?
Thanks!

I tried it on my workspace with many packages, and it's indeed much faster:
$ time go list all > /dev/null

real	0m7.265s
user	0m6.202s
sys	0m2.686s

$ time gopkgs > /dev/null

real	0m0.027s
user	0m0.009s
sys	0m0.015s

However, it does not appear to produce correct results. Many packages are missing.
$ go list all | wc -l
    4702

$ gopkgs | wc -l
    1689

The difference of go list all and gopkgs is that go list all looks for go packages in your $GOPATH/src while gopkgs looks in your $GOPATH/pkg.
As a result of this, only importable packages that have been installed (either by go get or go install) are listed by gopkgs.
Why go list all is much slower is because it has to read the source files while gopkgs only depends on the filepath.

That makes sense, thanks for explaining (and I think it's worth putting this in doc.go/README)!


Fix missing changeset

Hey, I just made a Pull Request!
Add missing changeset.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Could you link to the PR that this is for as well? Just so that we got that connection in the history. Just in a comment here on GitHub

Sure, and sorry for the delay. Here's the PR: https://github.com/backstage/backstage/pull/16090


Setup tests

Seems like codecov sometimes doesn't pick up the repo, idk, debug in main if that helps them find the repo


Dependencies Update Part2

Please note that the dependencies need updating again:

TinyMCE
Bootstrap
Bootstrap Bundle

Thanks!

Hi, I noticed the dependencies are needed to update again, but I can't do all the time, feel free to create a pull request with the changes, this kind of issues are not relevant.


Rewire slotproxy

Adds a function in slotProxy class to transfer rendering capability to
another particle
Builds on prior change of adding disable handle support

I'll approve once you land the handle patch and rebase on top, just to see more clearly what this patch is introducing :)

I think you can view the precise diff by fiddling the drop-down menu top left from the changes tab, and diffing vs. the appropriate other PR commit, which gives url as:
https://github.com/PolymerLabs/arcs/pull/3365/commits/20f6ba16e0133c5bccbe62bb9e4f768db07b14ba

In a way it is good that the tutorial particle with Parent + Child shows issues when we reload the parent. It is a super simple case that we should be able to debug.
As discussed f2f - I'm ok with you not fixing this yet, and instead switching to do file watching in ALDS + control buttons in Arcs Explorer. However if you would prefer to track down the issue with reloading the parent particle, then feel free to do that. We probably should have some chat about slot composing, so that you understand the abstractions to track down where it breaks.
Another option, we can track down this slot composing problem together when I'm in SF doing pair programming. In the meantime you could work on file watching + control UI, as discussed.
Up to you really, but I think pair programming when I'm in SF makes some sense, because slot composing code can be quite challenging to navigate through.

In a way it is good that the tutorial particle with Parent + Child shows issues when we reload the parent. It is a super simple case that we should be able to debug.
As discussed f2f - I'm ok with you not fixing this yet, and instead switching to do file watching in ALDS + control buttons in Arcs Explorer. However if you would prefer to track down the issue with reloading the parent particle, then feel free to do that. We probably should have some chat about slot composing, so that you understand the abstractions to track down where it breaks.
Another option, we can track down this slot composing problem together when I'm in SF doing pair programming. In the meantime you could work on file watching + control UI, as discussed.
Up to you really, but I think pair programming when I'm in SF makes some sense, because slot composing code can be quite challenging to navigate through.

The problem was without re-rendering all the providedSlots, the child particle disappear when the parent is reloaded. But I think for now it might be best if I revert the changes to the previous one. Doing pair programming on this sounds good, and in the meantime I'll start working on the file watching. :D


Parameters with no Position Attribute not defaulting to 'Named'

Steps to reproduce
function foo-bar {
    [CmdletBinding()]
    [OutputType()]

    param (
        [Parameter()]
        [String[]]
        $MyParam
    )
}

Generate markdown help with platyPS, and the parameter MyParam will have a position attribute value of 0 rather than named.  This is not a platyPS issue, as that just generates help based on what its given, so PowerShell is handing this value to platyPS, this is just a and easy way to see the data.  you can see a real example of this happening in my WinSCP PowerShell repo: https://github.com/dotps1/WinSCP/issues/92
Expected behavior
Position: Named
Actual behavior
Position: 0
Environment data

PSVersion                      6.1.3
PSEdition                      Core
GitCommitId                    6.1.3
OS                             Darwin 18.2.0 Darwin Kernel Version 18.2.0: T...
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

This can also be seen by defining a test function as an advanced function and pulling Get-Help -Parameter *:
PS> function Test-Function {
>> [CmdletBinding()]
>> param(
>> $One,$Two,$Three
>> )
>> }
PS> Get-Help Test-Function -Parameter *

Output:

-One <Object>

    Required?                    false
    Position?                    0
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

-Three <Object>

    Required?                    false
    Position?                    2
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

-Two <Object>

    Required?                    false
    Position?                    1
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

By default function parameters are positional
From the help file about_Functions_Advanced_Parameters
By default, all function parameters are positional. PowerShell assigns
position numbers to parameters in the order in which the parameters are
declared in the function. To disable this feature, set the value of the
PositionalBinding argument of the CmdletBinding attribute to $False.The
Position argument takes precedence over the value of the PositionalBinding
argument for the parameters on which it is declared. For more information,
see PositionalBinding in about_Functions_CmdletBindingAttribute.

I tried this already.  Same behavior.
Sent from my iPhone
On Mar 16, 2019, at 12:15 PM, RichardSiddaway <notifications<EMAIL_ADDRESS>wrote:
By default function parameters are positional
From the help file about_Functions_Advanced_Parameters
By default, all function parameters are positional. PowerShell assigns
position numbers to parameters in the order in which the parameters are
declared in the function. To disable this feature, set the value of the
PositionalBinding argument of the CmdletBinding attribute to $False.The
Position argument takes precedence over the value of the PositionalBinding
argument for the parameters on which it is declared. For more information,
see PositionalBinding in about_Functions_CmdletBindingAttribute.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/9151#issuecomment-473558694, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA-EpMca3E_FRK9Nv3IF4bRk4EDQ8soVks5vXRiWgaJpZM4b327X.

🤔 OK, just re-tested my quick-test in PS w/o PlatyPS. That does seem to work for PS's builtin help.
I guess PlatyPS must not be checking that value.
PS C:\Users\Joel> function Test-Function {
>> [CmdletBinding(PositionalBinding=$false)]
>> param(
>> $One,$Two,$Three
>> )
>> }
PS C:\Users\Joel> test-function
PS C:\Users\Joel> get-help test-function -parameter *

-One <Object>

    Required?                    false
    Position?                    Named
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

-Three <Object>

    Required?                    false
    Position?                    Named
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

-Two <Object>

    Required?                    false
    Position?                    Named
    Accept pipeline input?       false
    Parameter set name           (All)
    Aliases                      None
    Dynamic?                     false

Interesting.  I only tested with platyPS, I’ll move this issue over there then.  Thanks!!
Sent from my iPhone
On Mar 16, 2019, at 12:44 PM, vexx32 <notifications<EMAIL_ADDRESS>wrote:
🤔 OK, just re-tested my quick-test in PS w/o PlatyPS. That does seem to work for PS's builtin help.
I guess PlatyPS must not be checking that value.
PS C:\Users\Joel> function Test-Function {

[CmdletBinding(PositionalBinding=$false)]
param(
$One,$Two,$Three
)
}
PS C:\Users\Joel> test-function
PS C:\Users\Joel> get-help test-function -parameter *

-One
Required?                    false
Position?                    Named
Accept pipeline input?       false
Parameter set name           (All)
Aliases                      None
Dynamic?                     false

-Three
Required?                    false
Position?                    Named
Accept pipeline input?       false
Parameter set name           (All)
Aliases                      None
Dynamic?                     false

-Two
Required?                    false
Position?                    Named
Accept pipeline input?       false
Parameter set name           (All)
Aliases                      None
Dynamic?                     false

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/9151#issuecomment-473563703, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA-EpL0zmp_RNrCOL9di4O4dMBFp-hG2ks5vXR9qgaJpZM4b327X.


x265: 3.4 → 3.5

Motivation for this change

Lib update

Refactor to eliminate code duplication and crust

Don't use mkDerivation to build 10bits and 12bits static libs as we don't want
to install them in the nix store forever (only needed during the build)

Enable check phase when possible

Split headers into a "dev" output

Use SPDX 3.0 license identifier

Close #149817
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Outputs sizes comparison
# V3.5 shared lib (8+10+12 bits) and CLI and shared HDR10+ lib
$ nix path-info -sSh /nix/store/xvcw5hw5h77r7ycjwmy7nivk484s2f5z-x265-3.5
/nix/store/xvcw5hw5h77r7ycjwmy7nivk484s2f5z-x265-3.5      20.7M   58.3M
# V3.5 headers and pkgconfig
$ nix path-info -sSh /nix/store/l989a0vjwwywgs6ryxxmbw7h7rxaya8p-x265-3.5-dev
/nix/store/l989a0vjwwywgs6ryxxmbw7h7rxaya8p-x265-3.5-dev         109.4K   58.4M

$ nix path-info -sSh /nix/store/54klpxlyhq41d64ylfpmywhxrc13z6k7-x265-3.4
/nix/store/54klpxlyhq41d64ylfpmywhxrc13z6k7-x265-3.4      20.6M   58.2M
# plus
$ nix path-info -sSh /nix/store/y7vipkdwgmj0c3r2fgw87kvf3118p9df-libx265-10-3.4/
/nix/store/y7vipkdwgmj0c3r2fgw87kvf3118p9df-libx265-10-3.4         9.1M    9.1M
# plus
$ nix path-info -sSh /nix/store/7nc5jhi6jp33bxg52v48hz31z6dx7d2c-libx265-12-3.4/
/nix/store/7nc5jhi6jp33bxg52v48hz31z6dx7d2c-libx265-12-3.4         9.0M    9.0M

@GrahamcOfBorg test handbrake

Rechecking Darwin platforms
@ofborg build x265

Rechecking Darwin platforms @ofborg build x265

darwin is currently very broken on staging.

This broke aarch64-darwin:
[100%] Linking CXX executable x265
[100%] Built target cli
running tests
/nix/store/0qlq4s175mzlx9g0aqcdk95j92g1z7mf-stdenv-darwin/setup: line 1363: ./test/TestBench: No such file or directory
builder for '/nix/store/sgf9mmgk6c8b063182b6833k6g30is7x-x265-3.5.drv' failed with exit code 127

Changing unittestsSupport fixes it, but I'm not sure if that's the right solution:
diff --git a/pkgs/development/libraries/x265/default.nix b/pkgs/development/libraries/x265/default.nix
index 8ddcedf198b..409783b48c2 100644
--- a/pkgs/development/libraries/x265/default.nix
+++ b/pkgs/development/libraries/x265/default.nix
@@ -17,7 +17,7 @@
 , custatsSupport ? false # Internal profiling of encoder work
 , debugSupport ? false # Run-time sanity checks (debugging)
 , ppaSupport ? false # PPA profiling instrumentation
-, unittestsSupport ? (!stdenv.is32bit) # Unit tests - only testing x64 assembly
+, unittestsSupport ? (stdenv.isx86_64) # Unit tests - only testing x64 assembly
 , vtuneSupport ? false # Vtune profiling instrumentation
 , werrorSupport ? false # Warnings as errors
 }:

I suspect that the shebang needs to be fixed of ./test/TestBench.

./test/TestBench is a binary that isn't being built.
The conditions to build it are a bit screwy:
if(ENABLE_ASSEMBLY AND NOT XCODE)
    option(ENABLE_TESTS "Enable Unit Tests" OFF)
    if(ENABLE_TESTS)
        add_subdirectory(test)
    endif()
endif()

ENABLE_ASSEMBLY is enabled when:
if(ARM OR CROSS_COMPILE_ARM)
    option(ENABLE_ASSEMBLY "Enable use of assembly coded primitives" ON)

But that is never set because arm64 isn't in ARM_ALIASES. Once added:
set(ARM_ALIASES armv6l armv6l armv7l armv7a aarch64 arm64)

(On M1 uname -m returns arm64)
This causes a whole bunch of build errors like so:
/Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/aarch64/asm-primitives.cpp:104:33: error: use of undeclared identifier 'x265_12bit_pixel_satd_4x4_neon'; did you mean 'x265_pixel_satd_4x4_neon'?
        p.pu[LUMA_4x4].satd   = PFX(pixel_satd_4x4_neon);
                                ^~~~~~~~~~~~~~~~~~~~~~~~
                                x265_pixel_satd_4x4_neon
/Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/cpu.h:32:28: note: expanded from macro 'PFX'
#define PFX(name)          PFX2(X265_NS, name)
                           ^
/Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/cpu.h:31:28: note: expanded from macro 'PFX2'
#define PFX2(prefix, name) PFX3(prefix, name)
                           ^
/Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/cpu.h:30:28: note: expanded from macro 'PFX3'
#define PFX3(prefix, name) prefix ## _ ## name
                           ^
<scratch space>:63:1: note: expanded from here
x265_12bit_pixel_satd_4x4_neon
^
/Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/aarch64/pixel-util.h:28:5: note: 'x265_pixel_satd_4x4_neon' declared here
int x265_pixel_satd_4x4_neon(const pixel* pix1, intptr_t stride_pix1, const pixel* pix2, intptr_t stride_pix2);
    ^
In file included from /Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/encoder/analysis.cpp:33:
In file included from /Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/encoder/analysis.h:30:
In file included from /Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/predict.h:30:
In file included from /Users/zane.vaniperen/Documents/Coding/nixpkgs/x/x265_3.5/source/common/quant.h:29:

Looking at Homebrew formula, they don't use nasm on aarch64 Darwin:
https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/x265.rb#L21
Which means no ASM on this platform, which means there is nothing to test.
So I believe simply disabling the unit tests on this platform as done in https://github.com/vs49688/nixpkgs/commit/9b93014f5260ded20d77308808c69719b6d16ce2 in indeed the valid approach.


Make the library safe again once the compiler bugfix travelled through the release train

I think this can happen now that 1.4 has been released (which appears to be the required version if I interpret build.rs correctly).

Is this fixed? Is the library safe to use, does it leak memory? Just asking.

Oh, seems I never closed this - the library has safe API for quite some time again, and should also not leak memory.


Adding support for data_types_other

@yuanzhou I've made a change that should be deployed to the production version of entity-api. I hope I have updated the version appropriately

@maxsibilla we can't merge dev-integrate back to main. The work flow is:
To work on features in the development environment before ready for testing and releasing

Make new feature branches off the main branch
Make PRs to dev-integrate
Request Zhou as a reviewer to each PR. When all other reviewers have approved, he will approve as well, merge to dev-integrate, and redeploy on the DEV instance.
When a feature branch is ready for testing and release, make a new PR to main from the same source branch.

To release via TEST infrastructure

Make new feature or bug fix branches from main branch (the default branch)
Make PRs to main
Request Zhou as a reviewer as a reviewer to each PR. When all other reviewers have approved, he will approve as well, merge to TEST infrastructure, and redeploy on the TEST instance.
Developer or someone on the team who is familiar with the change will test/qa the change
When any current changes in the main have been approved after test/qa on TEST, Zhou will release to PROD using the same docker image that has been tested on TEST infrastructure.

Any bug fix/new feature branch should be off main when being created, and we PR to dev-integrate for the initial integration test, once ready, create another PR from the same source branch to main. That's where we deploy on TEST and when it comes to PROD release, we'll use the same docker image that has been tested on TEST from the same main branch code.
Merging dev-integrate into main will introduce unexpected code changes. For example, two developers work on different features, and both are merged into dev-integrate for the initial integration testing. But one developer has to make additional changes or fixes. At the same time, the other developer's changes are ready for release. If we merge dev-integrate into main we'll bring in the in-progress changes.
Can you create a new branch off main and re-apply the same changes made directly into dev-integrate then PR back to main? This will allow us to have an agreed workflow will also benefit long term to have a clean git commit history.


Update download extension icons in readme

Issue: https://github.com/FastForwardTeam/FastForward/issues/1160
As stated in the Issue, I have both the square and rounded versions of the Chrome icon, here in this PR I opted to use the squared version (to be as similar as possible to the current icon), but I personally think the round icon looks the best, so maybe we could switch to that.
Another thing to note in this PR is that the colors of the old and new Firefox download icon are slightly different (the new one has a darker blue), but that is easy to modify since it is a high-quality PNG picture, it is a matter of copying the color of the old icon and applying it on the new one using Paint/Photoshop.
And last but not least, I wasn't sure what the convention for icons is for FastForward. I wasn't sure if the icons should be somewhere in the project's folder (like in the docs folder), or if it should be hosted externally. Since the current icons being used are from user-images.githubusercontent.com , I assumed the project wants to keep its icons externalized, so I uploaded them to imgur.com (but no clue if this was the preferred way).
Here's the upload link with all three icons: https://imgur.com/a/2luHAqC
Fix(es):
There was no fix, this PR updates the Firefox and Chrome download icons in the project readme.

[X] I made sure there are no unnecessary changes in the code;
[ ] Tested on Chromium (Includes Opera, Brave, Vivaldi, Edge, etc);
[ ] Tested on Firefox.

using the https://github.com/FastForwardTeam/assets repo would be preferred instead of imgur but we'll have to edit it's readme

The respective authors of all files in this repository have, to the extent possible under law, waived all copyright and related or neighboring rights to the files.

to say that the files in the logo and logo_new are under CC0 and the Chrome, Edge and Firefox logo in the ext directory are the property of their respective owners.

https://github.com/FastForwardTeam/Assets/pull/2


autohotkey and autohotkey (install)

Checklist

[X] I have verified that this is the correct repository, and the package is maintained by the chocolatey-community user.
[X] I have verified that this is happening in the latest available version of the package.

Chocolatey Version
2.3.0
Chocolatey License
None
Package Version
2.0.18
Current Behaviour
Waiting for the update to 2.0.18 from <IP_ADDRESS>.  The package for autohotkey and autohotkey (install) are waiting for maintainer actions to release them for general use.
Expected Behaviour
autohotkey is update to 2.0.18 when I run choco upgrade
Steps To Reproduce

run choco upgrade all -y in PowerShell (administrator mode)
Does not update autohotkey or autohotkey (install) to V 2.0.18

Environment
- Operating System: Windows 10
- PowerShell Version: 5.1.19041
- Shell: NA

Chocolatey Log
I do not think a log is needed for the case that the packages are waiting for some maintainer step.

Anything else?
No response

There is already an issue for autohotkey being duplicated, as such I will be closing this issue in favor of the existing issue.

Duplicate of #2472


Add Time() and Nanos() methods

These methods can be used to extract time values
from V1 UUIDs

Coverage decreased (-1.6%) to 94.946% when pulling 552d1c4bee0be7a0fd51dcfdd99c335dac28899f on rkuris:master into 5bf94b69c6b68ee1b541973bb8e1144db23a194b on satori:master.

Coverage increased (+0.2%) to 96.751% when pulling a129dcc9d8afc93467742ee23260b60af6178af2 on rkuris:master into 5bf94b69c6b68ee1b541973bb8e1144db23a194b on satori:master.

@satori will you please consider this pull request, or let me know what needs to change to accept it?

This is a great addition -- We are trying to move away from gocql and this is the best alternative minus this one missing function.

@jbsturgeon as this has been oustanding since may, some of us have switched to using a public fork which includes the rollups and the time V1 functions. Please feel free to use the fork at https://github.com/rkuris/go.uuid which also contains a released version 1.2.0 that includes this fix: https://github.com/rkuris/go.uuid/releases/tag/v1.2.0

I've rebased this before, and would be happy to rebase it again if that means it will get accepted.

Would love to see this get added :)
Thanks for doing this!

A bug in the code for this PR has been fixed in https://github.com/gofrs/uuid/pull/31
Strongly recommend moving there, as this repo is not actively maintained.


Update how-to-write-quick-edits.md

Contributor guide pull request
Thanks for contributing to the Microsoft Learn contributor guide. Please read these instructions for processing your pull request (PR). Once you've read this text, delete it and write a brief description of the changes you're proposing.
Quality control

[ ] 1. Successful build with no warnings: Review the build status to make sure all checks are green (Succeeded).

[ ] 2. #Sign-off: Once the PR is finalized and ready to be merged, indicate so by typing #sign-off in a new comment in the PR. Signing off means the document is ready for review and can be published at any time.

Merge and publish

All PRs to this repository are manually reviewed and merged.
Once all feedback on the PR is addressed, we'll merge the PR into the main branch.

Need help?

If you have an open PR, tag the PR reviewers in comment with your question: @carlyrevier, @jehchow
Open an issue in this repo and tag the PR reviewers.

@AnUsHkAIT - You must accept the Contributor License Agreement before this PR can be processed. See the comment above this one for more info. Thanks!


Add support for direct instantiation of UIView from nib

See issue https://github.com/AliSoftware/Reusable/issues/4

Cool thx for the PR will look at it during the weekend (once I'm back from dotSwift) :wink:


WIP: remove superfluous dependencies from Proxy Attestation Server

Closes #362.

There are also calls to the diesel CLI that should removed from Makefile (which in turn should result in some cleanups in the docker images)

This is now ready for review.

+1 reached, merging.


revamped the project maintainer's section

Revamped the UI of project maintainers section on hovering.

Firstly the animation doesn't suit the cards
Secondly, the social media icons looks wierd

* Firstly the animation doesn't suit the cards

* Secondly, the social media icons looks wierd

can you suggest me some animation examples which will suit?
shall I keep the social media icons as present earlier?


Image quality on desktop not perfect

Images are high-res, but some Qt features don't appear to display them as high-res. Icons appear to be lower quality in some cases.

Nothing can be done with that. It is QT issue for all UI.


Merge Qa 0.9 back to main

What this PR does / why we need it:
Merge qa fixes back to main
Special notes for reviewer:
Checklist:

[ ] Added relevant notes to WIP-CHANGELOG.md
[ ] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Closed in favor of #210

Closed in favor of #210


Add support for Sony A7C (and probably others)

Add support for Sony A7C (and probably other cameras). Also add a stabilization time setting, change default step size (10 was out of range for Sony which supports [-7,7]).
For this to work, the Sony camera needs to be set to MTP, and set control by PC with save to SD card + PC.
egrun:
python bracketing_runner.py -o output_directory --stabilization_time 1 -s 3 -n 50
I couldn't get the download part to work, but I can't get it working even in darktable so long as the PC control option is set.

@trougnouf thanks for your valuable contribution!


Update Binary Markets Docs

Does the market creator get their M$100 back after the market resolves like liquidity providers do, or does it go to the market winner?

@ahalekelly Market creators get their liquidity back, unless it was a free daily market, in which case the money goes to Manifold (i.e. it is burned). This is for binary markets. For free response, the M$100 is part of the pool and gets distributed among the winning bets.

@jahooma @wasabipesto thanks. I can open another PR with this info but there's a lot more info missing from this doc that I don't know the answer to. Would probably be better to have someone who knows this better write it if you're not too busy.


FileTarget::$rotateByCopy: if not specified, automatically set to false when OS is not windows

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues

FileTarget::$rotateByCopy: if not specified, automatically set to false when OS is not windows

Why?

In our application on Yii 2, maxFileSize is set to be 1 GB (with 10 log files, the default value), and by default $rotateByCopy = true, which means, when log files need to be rotated, there may be as much as 9 GB of logs to be read into memory, and write to disk, which is meaningless on Linux servers and causes large pressure on disk IO, and thus make the server not responding to requests. Automatic OS detection in this pull request helps improve situation on non-windows servers.

you should choose a smaller file size then.

I can change it to 100 MB, but 900 MB of disk io will still take my server down for over 1 minute, and as is described in the comments, this flag is set to true because rotation fails ONLY on Windows, so why don't we just set it to false on non-windows servers to avoid such condition? And I have to say, that you guys didn't warn developers to use smaller maxFileSize in advance either, which I spent days to find out the reason of downtime. This is not friendly to yii users.

Okay now I understand the problem. Thanks for explaining. It makes sense now.

Applied to 3.0.


home-assistant: 2021.1.1 -> 2021.1.2

Motivation for this change
https://github.com/home-assistant/core/releases/tag/2021.1.2
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test home-assistant

Test case is working on x86_64-linux, deployed to aarch64-linux. WFM.

Test case is working on x86_64-linux, deployed to aarch64-linux. WFM.


transparents spatial spots for zero counts

I would like to have transparent spots for spots that have zero counts (or near zero) on a spatial scatter plot. I was trying the function squidpy.pl.spatial_scatter. An example shown below for a 10x Visium dataset. Can this be done?

yes, just pass a colormap that can do that for you, you can find how to build such colormap here: https://matplotlib.org/stable/tutorials/colors/colormaps.html

Ok. Can you show a simple example?


Errors of minimal examples

System description

OS: Ubuntu 22.04 LTS, 64bit
CPU: Intel i5

Conda environment description

pip: 23.0
python: different in different conda env. See detail below.
cmake: 3.22.1
gcc/g++: 11.4

Other common road package installed

commonroad.io

Error description
I successfully ran pip install commonroad-all, but encountered "ModuleNotFound" errors in several minimal examples given by the tutorial.

Minimal example of sumocr: ModuleNotFoundError (python=3.10.13): No module named 'simulation'. After running pip install simulation, I ended up with another error:

Collecting simulation
Using cached simulation-0.1.1.dev80-py3-none-any.whl.metadata (2.2 kB)
Collecting measurements (from simulation)
Using cached measurements-0.1.1.dev67-py3-none-any.whl.metadata (2.2 kB)
Requirement already satisfied: numpy in /home/ron/miniconda3/envs/cr/lib/python3.10/site-packages (from simulation) (1.26.4)
Collecting utillib[cache,cholmod,interpolate,options] (from simulation)
Using cached utillib-0.2-py3-none-any.whl.metadata (3.2 kB)
Requirement already satisfied: scipy in /home/ron/miniconda3/envs/cr/lib/python3.10/site-packages (from measurements->simulation) (1.11.4)
Collecting h5py (from utillib[cache,cholmod,interpolate,options]->simulation)
Using cached h5py-3.10.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (2.5 kB)
Collecting cachetools (from utillib[cache,cholmod,interpolate,options]->simulation)
Using cached cachetools-5.3.3-py3-none-any.whl.metadata (5.3 kB)
Collecting scikit-sparse>=0.4.1 (from utillib[cache,cholmod,interpolate,options]->simulation)
Using cached scikit-sparse-0.4.12.tar.gz (214 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Using cached simulation-0.1.1.dev80-py3-none-any.whl (68 kB)
Using cached measurements-0.1.1.dev67-py3-none-any.whl (66 kB)
Using cached cachetools-5.3.3-py3-none-any.whl (9.3 kB)
Using cached h5py-3.10.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (4.8 MB)
Using cached utillib-0.2-py3-none-any.whl (97 kB)
Building wheels for collected packages: scikit-sparse
Building wheel for scikit-sparse (pyproject.toml) ... error
error: subprocess-exited-with-error

× Building wheel for scikit-sparse (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [35 lines of output]
running bdist_wheel
running build
running build_py
creating build
creating build/lib.linux-x86_64-cpython-310
creating build/lib.linux-x86_64-cpython-310/sksparse
copying sksparse/test_cholmod.py -> build/lib.linux-x86_64-cpython-310/sksparse
copying sksparse/init.py -> build/lib.linux-x86_64-cpython-310/sksparse
creating build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/well1850_rhs1.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/illc1033_rhs1.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/illc1850_rhs1.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/well1033_rhs1.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/illc1033.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/illc1850.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/well1850.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
copying sksparse/test_data/well1033.mtx.gz -> build/lib.linux-x86_64-cpython-310/sksparse/test_data
running build_ext
building 'sksparse.cholmod' extension
creating build/temp.linux-x86_64-cpython-310
creating build/temp.linux-x86_64-cpython-310/sksparse
gcc -pthread -B /home/ron/miniconda3/envs/cr/compiler_compat -Wno-unused-result -Wsign-compare -DNDEBUG -fwrapv -O2 -Wall -fPIC -O2 -isystem /home/ron/miniconda3/envs/cr/include -fPIC -O2 -isystem /home/ron/miniconda3/envs/cr/include -fPIC -Isksparse -I/tmp/pip-build-env-9wi_glic/overlay/lib/python3.10/site-packages/numpy/core/include -I/home/ron/miniconda3/envs/cr/include -I/usr/include/suitesparse -I/home/ron/miniconda3/envs/cr/include/python3.10 -c sksparse/cholmod.c -o build/temp.linux-x86_64-cpython-310/sksparse/cholmod.o
In file included from /tmp/pip-build-env-9wi_glic/overlay/lib/python3.10/site-packages/numpy/core/include/numpy/ndarraytypes.h:1948,
from /tmp/pip-build-env-9wi_glic/overlay/lib/python3.10/site-packages/numpy/core/include/numpy/ndarrayobject.h:12,
from /tmp/pip-build-env-9wi_glic/overlay/lib/python3.10/site-packages/numpy/core/include/numpy/arrayobject.h:5,
from sksparse/cholmod.c:1226:
/tmp/pip-build-env-9wi_glic/overlay/lib/python3.10/site-packages/numpy/core/include/numpy/npy_1_7_deprecated_api.h:17:2: warning: #warning "Using deprecated NumPy API, disable it with " "#define NPY_NO_DEPRECATED_API NPY_1_7_API_VERSION" [-Wcpp]
17 | #warning "Using deprecated NumPy API, disable it with "
|  ^~~~~~~
In file included from sksparse/cholmod.c:1231:
sksparse/cholmod_backward_compatible.h:1:10: fatal error: cholmod.h: No such file or directory
1 | #include "cholmod.h"
|          ^~~~~~~~~~~
compilation terminated.
error: command '/usr/bin/gcc' failed with exit code 1
[end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for scikit-sparse
Failed to build scikit-sparse
ERROR: Could not build wheels for scikit-sparse, which is required to install pyproject.toml-based projects

Miniexample of commonroad-search (python=3.9): ModuleNotFoundError: No module named 'maneuver_automaton' and No module named 'motion_planner'. In addition, No module named 'SMP' occurred when running the first example of the tutorial: Uninformed Search Algorithms. I guess commonroad-search is not in commonroad-all (?), so I git clone the repo and run pip install -r requirements.txt, but it went wrong again. Is there a recommended way of installing the search tool? Hard to navigate among various places on the Internet.

Minimal example of commonroad-reach: FileNotFoundError: [Errno 2] No such file or directory: 'scenarios/ARG_Carcarana-1_1_T-1.xml'. Do we need to download the scenarios manually and where?

Minimal-example for Commonroad-RL (python=3.7.16, tool installed via pip install commonroad-rl)

Traceback (most recent call last):
File "/home/ron/Projects/Commonroad/gymtest.py", line 20, in
"reward_friction_violation": 0.}})
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/site-packages/gym/envs/registration.py", line 235, in make
return registry.make(id, **kwargs)
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/site-packages/gym/envs/registration.py", line 129, in make
env = spec.make(kwargs)
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/site-packages/gym/envs/registration.py", line 90, in make
env = cls(_kwargs)
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/site-packages/commonroad_rl/gym_commonroad/commonroad_env.py", line 135, in init
with meta_scenario_reset_dict_path.open("rb") as f:
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/pathlib.py", line 1208, in open
opener=self._opener)
File "/home/ron/miniconda3/envs/crrl/lib/python3.7/pathlib.py", line 1063, in _opener
return self._accessor.open(self, flags, mode)
FileNotFoundError: [Errno 2] No such file or directory: '/home/ron/miniconda3/envs/crrl/lib/python3.7/site-packages/pickles/meta_scenario/meta_scenario_reset_dict.pickle'

1. Installation issues
The commonroad-all installation supports currently only the following tools/version:

commonroad-io = "2023.3"
commonroad-vehicle-models = "3.0.2"
commonroad-drivability-checker = "2023.1"
commonroad-dataset-converter = "2023.2"
sumocr = "2023.2"
commonroad-scenario-designer = "0.8.0"
commonroad-route-planner = "2022.3"
commonroad-crime = "0.3.0"
commonroad-openscenario-converter = "0.1.0"
commonroad-reach = "2023.1.1"

We will update the tools/versions in the next couple of weeks and ensure that the corresponding tools work together properly.
Tools not part of this list will probably cause installation issues when installing them together with commonroad-all.
2. CommonRoad-Search
The installation of CommonRoad search is also detailed in the corresponding repository: https://gitlab.lrz.de/tum-cps/commonroad-search
3. CommonRoad-Reach
You can find the scenario in the repository here or on our website.

CommonRoad-RL
You can find the missing file here.

In general, the minimal examples will not work when only installing the PyPI package since the package, as the package does not contain the required example files. If you want to try out the minimal examples, I recommend cloning the corresponding repository. The cloned repository should contain scripts or notebooks which can be executed directly.
I will also ask my colleagues who are responsible for the corresponding tools to take a look at your problems.


ciao-controller: Add boot flag to attachment

Add a flag to indicate whether the attachment is for a boot volume.
If a user requests a detach of an attachment that is a boot device,
send a 403 to indicate this is a forbidden operation.
Fixes: #870
Signed-off-by: Kristen Carlson Accardi<EMAIL_ADDRESS>

Coverage increased (+0.02%) to 64.766% when pulling 2c563fa595f82ffb1083e4ed302e47d0ac54f732 on kaccardi:topic/issue-870 into 14a0a250477afc65964f9cc32b2eebe8c462d475 on 01org:master.

It does indeed fix #870.  Thanks.


35 | Search Insert Position | Python

Search Insert Position -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] Python

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign this issue to me @Yashica7 @SSKale1


security-context: Tweak seccomp description

It's not just about files; seccomp allows filtering any system call.

/approve


Fixed #528 Audio section

Description 📜
06.0 Introduction to Views in Android - Added Audio file .
Fixed #528
Type of change 📝

[x] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Android Dev #(Kotlin)

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Hey @rahulsain , Checkout this PR and if everything is fine , Kindly merge it ......................................


add signer & signature check at minting time

issue #26

Adds a signer / validator in order to validate off-chain that two same token ids will not be submitted
adds "onlyCreator" modifier

I took the liberty to reuse onlyCreator & its custom error on the protected function setRoyalties()
Long texts for errors are usually not recommended, as they add gas at deployment & when reverting. Prefer concise & clear custom errors instead

Prefer concise & clear custom errors instead

Still too verbose to be a real engineer :sweat_smile:
Thank you so much, dear brother, your work is a delight and is greatly, deeply appreciated here :pray:


Access to laion5B-index temporarily disabled

Hi,
Great repo! When trying to download the index on huggingface, it says access to this dataset has been temporarily disabled on the author's request: https://huggingface.co/datasets/laion/laion5B-index
Could you give some insights into when the index will be available again, or why it is currently unavailable? Thank you!

Please see https://laion.ai/notes/laion-maintanence/ and https://github.com/rom1504/clip-retrieval/issues/324
That was absurd...

Thank you!


How to generate a face with closed eye

Since we know that there is  no close eye expression in BFM, so how to generate a face with closed eye?
Thank you for sharing code!

I am unaware of code doing that.
The BFM does not model the eyeball independently, but it is connected to the eyelids. This can be changed in any 3d modeler software. Further, you would then need to come up with the deformations that close the eyelid such that it stays on top of the eyeball.

Yes, that means need a addtional 3d modeler software or other deformations code.
So in this project the  close eye face can't be generated if only shuffling those expression parametric randomly.
It will be in the works that support to generate close eye face in the furture?

We are an open source project and welcome contributions.
To the best of my knowledge I don't know of any current development for
this project on this topic.
Am 6/3/2020 um 10:25 AM schrieb Tao Zhong:

Yes, that means need a addtional 3d modeler software or other
deformations code.
So in this project the close eye face can't be generated if only
shuffling those expression parametric randomly.
It will be in the works that support to generate close eye face in the
furture?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/unibas-gravis/parametric-face-image-generator/issues/34#issuecomment-638043378,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABF76ZWOVCF7TAAYN5AZDS3RUYCF7ANCNFSM4NRM7UDQ.

Ok, thanks. I'll contribute my work if I made some progress on this topic.


Elastic Runtime Backup fails

Hi
We used cfops for backing up Elastic Runtime 1.5, but since we upgraded to 1.6 it fails. The ops manager backup still works fine.
Example:
./cfops backup --omu ubuntu --omp xxx -d backup --tl 'opsmanager,er' --omh <IP_ADDRESS> --du admin --dp xxx
invalid director credentials
Only the tile 'opsmanager' works. Above command fails obviously after ops manager was backed up completely.
We use the Elastic Runtime version 1.6.4 with Ops Manager 1.6.1 (vSphere), and cfops version v1.1.5.
Best regards, Christoph

It looks like the installation.json format changed. So the bosh director's IP is just empty, because it could not be retrieved from json.

No, not the format has changed, just bosh's "product name". In elasticruntime.go, 			Product: "microbosh"
needs to be replaced by
Product: "p-bosh"
Then the IP can be read again. I will check whether this resolves all issues and how easy a switch can be implemented dependent on the version. Maybe it's better to add a configuration for this switch, as I do not know exactly when this changed. Maybe it changed in a minor upgrade?

If you want my fix, you can merge https://github.com/huberchrigu/cfops/tree/feature/cf-1.6. I could not change the log level, so I used fmt.Printf to warn when a product's metadata cannot be parsed to ensure that the failure can be seen quicker in the future.

Please try this pre-release build with the env var ER_VERSION=1.6 set. https://github.com/pivotalservices/cfops/releases/tag/v1.1.18
I will remove the pre-release flag once I pull can pull in some more 1.6 data from live systems to provide a more complete test suite.

This works fine. Thanks!


Adjust our UA string to fix to make it compatible with fonts.g.c parsing on IE11.

Counterpart of:
https://github.com/pagespeed/mod_pagespeed/commit/f3639e84c0196a5f5151ff5e54ad57285db09b37

Note that I am having trouble running tests with native fetcher....

Change LGTM
I can reproduce that testing with the native fetcher does not work at all, created https://github.com/pagespeed/ngx_pagespeed/issues/1104 for that

FWIW, I worked on adding just enough ipv6 support to the native fetcher so we can run the tests with it again. State dumped over here:
https://github.com/pagespeed/ngx_pagespeed/compare/trunk-tracking...oschaaf-trunk-tracking-native-fetcher-ipv6?expand=1
Tests are passing again with it, barring the "Can inline Google Font API loader CSS" test, which probably needs this change to pass?