Connect Twitter account link with the icon

I tap on the Twitter button but, I didn't get specific about your account. I recommend you to put the specific link to your account

i didn't write properly


Fail state is not recognised

For some reason the Fail state is not recognised.
Step to reproduce
import { StateMachine, ExecutionError } from 'aws-local-stepfunctions';

const machineDefinition = JSON.parse(JSON.stringify({
  StartAt: 'ErrorState',
  States: {
    ErrorState: {
      Type: 'Fail',
      Error: '500',
      Cause: 'Some nasty error',
      End: true
    }
  },
}));

test('should throw an error if name is missing', async () => {
  // GIVEN
  const myInput = {};

  // WHEN
  const stateMachine = new StateMachine(machineDefinition, {
    // validationOptions: {
    //   noValidate: true,
    // },
  });

  try {
    const execution = stateMachine.run(myInput);
    await execution.result;
  } catch (e: any) {
    expect(e.message).toMatch('Execution has failed with the following error: Some nasty error');
  }
});

Desired outcome
The state machine should thrown an ExecutionError
Error
State machine definition is invalid, see error(s) below:
     SCHEMA_VALIDATION_FAILED: /States/ErrorState is invalid. must match exactly one schema in oneOf

Workaround
If I disable the state machine validation, the exception is thrown correctly.

This is actually expected, as per the spec:

Any state except for Choice, Succeed, and Fail MAY have a field named "End" whose value MUST be a boolean.

So removing the End field from the state should fix the issue


Allow pytest to run with sudo

To enable the execution of complex tests it is sometimes necessary to execute pytest with sudo.
A launch configuration supports the sudo flag. Is it possible to port this over for the execution of pytest / unittest tests started from the Testing tab?
It is quite annoying to have to execute some tests requiring elevated rights from the terminal and not be able to see the status of those tests in the test overview displayed in the Testing tab.

I have managed to find a workaround which makes me wonder even more why this feature isn't supported:
When creating a custom launch config to debug tests I am able to specify the "sudo" = true flag which works when executing my tests in debug mode. The test result is also properly collected after execution in debug mode.
Is there a way to create a custom launch config for regular tests?

After some more extensive research on outstanding feature request it seems like #21845 would solve this problem automatically if the "sudo" flag could be set to true for pytest/unittest run

Thank you for submitting your feature request and everyone who considered it! Unfortunately, this issue did not receive enough votes over the allotted time, and so we are closing the issue.


Chocolatey GUI not found in the PATH

Hi @gep13 , i wanted to give a try to chocolateyGUI package to make a visual demo. While playing with your software, i discovered that the exe was not in the PATH ...and sometimes it's pretty convenient.
Please find below a screenshot showing all this :

Hopefully you will find this issue interesting 😸

I'm curious how this would be useful for you? Gui can't really do anything that would make it relevant on the cmd besides, well, start. (That isn't already done by choco that is)

@RichiCoder1  : Sometmes, it's useful tu launch GUI from console.

Given the functionality that was recently added (https://chocolatey.github.io/ChocolateyGUI/docs/commands/feature) I believe that this is more important than it previously was.  As such, I think creation of a shim is a good idea to launch/run Chocolatey GUI from command line.

This has now been included, and will ship in the next release of Chocolatey GUI.


Transcoding is done...

Transcoding into Vala is done... but minor tweaks on the documentation must be done to address the changes from PHP into the 'language' being adopted here or elsewhere (programming languages wise). Any comments or guidelines on how the various versions will be presented?

No comments so I carried on and made a few tweaks on PHP references and converted them into Vala references.


Add an AtomicReference type

See https://github.com/sfackler/log4rs/commit/5ed2895bfe2ac92e30a4a5506bde98081bc85cce for a usage example.
I'm not sure AtomicReference is the right name - I stole it from Java, but we might want this to be called something else to indicate that it's tied up with Arc.
It's also a bit sketchy that we assume that Arc<T> is the same size as usize, but that seems preferable to defining a new smart pointer just for this.

I'm fine with the bit of sketchiness here. This lib is overall pretty experimental, anyway.
AtomicReference isn't great -- it doesn't strongly suggest what's going on here. But I'm struggling to do better. Some ideas:

ArcCell -- conveys the idea that this provides interior mutability over an Arc. Usually Cell indicates non-concurrent, though.
MutArc -- a changeable Arc?
... I guess that's it :)

I think I lean toward ArcCell. It's a shame we never did manage to get a more coherent naming scheme between Mutex, RwLock, Cell and RefCell.

ArcCell seems fine - updated.

Thanks @sfackler!


Predictable URLs

Improve Uffizzi environment URLs to be predictable. Potentially taking the form:
https://previews.uffizzi.com/github.com/waveywaves/backstage/pull/4
...or perhaps...
https://previews.uffizzi.com/#https://github.com/waveywaves/backstage/pull/4

@gadkins What's the use case of predictable URLs?

The use case would be:
-Rerouting https://previews.uffizzi.com/github.com/waveywaves/backstage/pull/4 to the latest uffizzi preview for the PR  i.e https://deployment-5033-saml-jackson-hje1.app.uffizzi.com.

The URL can act as a proxy for the ingress
The URL can show the status of the PR before it comes up. eg: If I create a PR and a Uffizzi Preview Environment is triggered, I can go to https://previews.uffizzi.com/github.com/waveywaves/backstage/pull/4 and see that the preview is in building, deploying, deployed, ready, not ready or failed mode.
Allows anyone to see the state of the preview.
Makes previews easily accessible. Currently you need to go to the UI or query for the preview via the CLI. In this case you only need to prefix a github URL with previews.uffizzi.com.

A few more details which might be helpful

Opening https://previews.uffizzi.com/github.com/waveywaves/backstage/pull/4 page creates a loading page for the preview
Doing a GET request on the URL returns the status of the preview

{
  health: "ready"
  preview_url: https://deployment-5033-backstage-hje1.app.uffizzi.com
}

Once the health is ready and preview_url is set, then the https://previews.uffizzi.com/github.com/waveywaves/backstage/pull/4 redirects to https://deployment-5033-backstage-hje1.app.uffizzi.com


404 handled incorrectly

Current behavior
If I access a random URL like https://demo.vuestorefront.io/test.html, I'm getting a timeout
Expected behavior
I should see a proper 404 Page Not Found.
Steps to reproduce the issue
Access a random URL like https://demo.vuestorefront.io/test.html and you will have a timeout:

Can you handle fixing this bug by yourself?

[ ] YES
[X] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[ ] This is a bug report for test version on https://test.storefrontcloud.io - In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] This is a bug report for current Release Candidate version on https://next.storefrontcloud.io - In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[X] This is a bug report for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version hotfix - In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Environment details

Browser: Chrome 79
OS: Mac OS
Node: 10
Code Version: Tag v1.10.5

Apparently this bug is related to 1.10 as it doesn’t occurs in 1.12: http://test.storefrontcloud.io/hdhd

This bug occurs on 1.11.0 also.


Use metrics listener for datadog distribution

Draft implementation for https://github.com/zio/zio/issues/7393
See also related ticket for zit-core: https://github.com/zio/zio/pull/7474
Note that this needs obviously needs more work (at least tests and benchmarks), at the current stage I'd just like to get some feedback before proceeding.

LGTM, I would like to see a README / example of a datadog setup like we currently have in the docs for prometheus and statsd.
The other thing that kind of bothers me from a usability point of view:
A user who is reporting other metric types like counters and gauges is required to use BOTH the statsd layer based on the state and also the listener based connector.
Perhaps we should offer a Layer combining both and have some proper documentation around the use cases. I think our initial discussion would be a good start for the README. Perhaps we should just dump the discussion into a file and beautify it later on

Thanks for the feedback @atooni!
The related zit-core ticket has been merged and released as part of 2.0.4: https://github.com/zio/zio/pull/7474
I'll started updating this PR and adding changes based on your feedback, will push it soon.

@petoalbert Thx for this - once the merge conflicts are resolved we can merge and create a new release of zio-metrics-
connectors


python310Packages.casbin: 1.17.2 -> 1.17.3

Description of changes
https://github.com/casbin/pycasbin/releases/tag/v1.17.3

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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 198907 run on x86_64-linux 1

2 packages built:

python310Packages.casbin
python39Packages.casbin


Allow to set Property.ClrType

Remove ClrType and IsShadowProperty from the Property constructor
Move CLR type validation to ModelValidator
Make IModel.GetReferencingForeignKeys extension methods and rename to FindReferencingForeignKeys

:shipit:


FLINK-13871: Consolidate volatile status fields in StreamTask

What is the purpose of the change
StreamTask has 3 volatile boolean fields (isRunning, canceled, failing) that are mutually exclusive and reflect different stages of stream task's lifecycle. We can represent them as a single enum state field.
Brief change log

Introduced StreamTask enum TaskState and got rid of existing isRunning, canceled, failing volatile boolean fields

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

Thanks everyone for the useful comments! PTAL on the latest changes, in short:

Checked all state transitions and documented them as part of https://github.com/apache/flink/pull/21923/files#diff-0c5fe245445b932fa83fdaf5c4802dbe671b73e8d76f39058c6eaaaffd9639faR268
Integrated Restoring state introduced as part of FLINK-29816
Added Failing as a separate State boolean field in TaskSate object (can always rework this as a followup)
Refrained from adding state transition validation as this is already happening as part of Task ExecutionState https://github.com/apache/flink/blob/8be94e6663d8ac6e3d74bf4cd5f540cc96c8289e/flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/Task.java#L907

Thoughts?

@pgaref @rkhachatryan @akalash, what's the next step for this effort?


Forc-deploy import wallet support if non-existent wallet

Description

Adds support for importing a wallet on forc-deploy
Resolves https://github.com/FuelLabs/forc-wallet/issues/139

The prompt about creating/importing a wallet should come before asking for the wallet password. Otherwise, once the tests are passing, LGTM 😃

The current behaviour performs the password check which is much more upstream in the code; changing that would imply a re-factor which may have knock-on effects.
I think this would be worthwhile addressing as a seperate issue.
If we have agreement; I can create an issue for that.

The prompt about creating/importing a wallet should come before asking for the wallet password. Otherwise, once the tests are passing, LGTM 😃

The current behaviour performs the password check which is much more upstream in the code; changing that would imply a re-factor which may have knock-on effects. I think this would be worthwhile addressing as a seperate issue. If we have agreement; I can create an issue for that.

I think we should discuss the necessary refactor to do this, because it feels a little confusing as it is. If i don't have an wallet it is highly likely that I don't have necessary context to understand what password I need to enter once that is prompted. So it might be worth doing the refactor before merging this.
If you can come up with some kind of a description of what you need to be changed exactly I am more than happy to help out! I want to clean forc deploy paths for some time now even have a tracking issue for it. #6259

The prompt about creating/importing a wallet should come before asking for the wallet password. Otherwise, once the tests are passing, LGTM 😃

The current behaviour performs the password check which is much more upstream in the code; changing that would imply a re-factor which may have knock-on effects. I think this would be worthwhile addressing as a seperate issue. If we have agreement; I can create an issue for that.

I think we should discuss the necessary refactor to do this, because it feels a little confusing as it is. If i don't have a wallet it is highly likely that I don't have necessary context to understand what password I need to enter once that is prompted. So it might be worth doing the refactor before merging this.
If you can come up with some kind of a description of what you need to be changed exactly I am more than happy to help out! I want to clean forc deploy paths for some time now even have a tracking issue for it. #6259

I believe this should be done in another PR as the scope could become much larger.
In this PR we just want to give option for user to import seed phrase; without changing/refactoring the upstream flow.

The prompt about creating/importing a wallet should come before asking for the wallet password. Otherwise, once the tests are passing, LGTM 😃

The current behaviour performs the password check which is much more upstream in the code; changing that would imply a re-factor which may have knock-on effects. I think this would be worthwhile addressing as a seperate issue. If we have agreement; I can create an issue for that.

I think we should discuss the necessary refactor to do this, because it feels a little confusing as it is. If i don't have a wallet it is highly likely that I don't have necessary context to understand what password I need to enter once that is prompted. So it might be worth doing the refactor before merging this.
If you can come up with some kind of a description of what you need to be changed exactly I am more than happy to help out! I want to clean forc deploy paths for some time now even have a tracking issue for it. #6259

I believe this should be done in another PR as the scope could become much larger. In this PR we just want to give option for user to import seed phrase; without changing/refactoring the upstream flow.

I don't think we should merge this as it is because as it is this will be confusing, my suggestion would be to keep this ready to go and first fix the upstream and after that using the already fixed upstream in this PR


Navbar changed

@Amankumar29 I think a non-animated navbar is nice. What you think @aashimgarg @arteevraina

lets try for it @anubhavsingh16 i will be removing the background of title and add some animation in that then it will look nice.

lets try for it @anubhavsingh16 i will be removing the background of title and add some animation in that then it will look nice.

for try big ok


Add functools32 for glance-store-check (1.2.x)

(cherry picked from commit 66421ec615fc219e85b8093b1c9a19178161bb78)

retest


Use ${PIXOO_REST_URL} in clockwise_swipe.sh

Use the variable defined in bash script example ${PIXOO_REST_URL} - so that the endpoint can be changed once in the variable and re-used in the script.

Nice catch!
Using the variable all over was my initial intent... seems I just forgot it along the way. :smile:
Thanks.


Commit for CI testing

Draft PR to test Linux CI
Signed-off-by: Kevin Leimkuhler<EMAIL_ADDRESS>

Closing as it looks like issue_1205 has just been flaky.


Fetching single item from cache when cache is filled with array

Sorry to create an issue in the main repository, but I couldn't find anything online and didn't receive an answer on the nuxt-apollo discord server.
Is it possible to fetch a singular item from the Apollo cache after I've fetched multiple items on another page?
I have already fetched stories on another page, but now I want to fetch a singular item story(id: $id) from the cache. So this is a fetchPolicy: 'cache-only' query. I have tried a few things, but it looks like Apollo does not know that stories is the plural of story (which makes sense). Is there anything I can add to let Apollo know that if I ask for a story it can also return the stories item with the same ID?
Would this for example work with something like the IntrospectionFragmentMatcher?
This is quite essential to a project I am working on. If anybody could point me in the right direction I would be eternally grateful!

I have the same issue.


[#678][JBEAP-26308]Revert to current tip should be a non operation

https://github.com/wildfly-extras/prospero/pull/684 backport to 1.1.x
Issue: https://issues.redhat.com/browse/JBEAP-26308

Thanks @yersan!


Merge license name with license icons

Fixes #345
It seems that we've been separating the LicenseName from the LicenseIcons and that caused an overhead in our architecture, and it caused the #345 as we are - unnecessary - working on some data (i.e. image.license)
So, I used the LicenseIcons Component to store the name of the license, instead of separating it.
Suggestion
What about renaming the LicenseIcons component to LicenseInfo?

The root of the problem here is in the API which has the image details. I filed a bug on the API repo https://github.com/creativecommons/cccatalog-api/issues/289. If that doesn't get fixed today, I'll merge this workaround. Otherwise I'll close this PR.
About renaming the component to LicenseInfo, it's better to have the separate components, so LicenseIcons is solely responsible for rendering the icons

Thanks for highlighting this to me @brenoferreira !
I thought it would be a good idea to keep both the text/icons together in one component as we use them together, and we will get rid of some unnecessary props as well.

@elharony the URL is now fixed, so we won't be needing this. But thank you for your contribution


(v1.3) Failing to get object from collection, double this.helper runs, etc.

Hi @Urigo,
Again, with my problems :)
I have a controller that looks like this:
angular.module('chatapp.admin').controller('AdminUserDetailsCtrl', function($scope, $meteor, $reactive, $stateParams, storage, $mdToast, $q, $interval){

	$reactive(this).attach($scope);
	console.log('PRINT 0: User Details controller instantiated');

	this.helpers({
		currentUser: () => {
			console.log('PRINT1: ', $stateParams.userId);
			let result = Meteor.users.findOne({_id: $stateParams.userId});
			console.log('PRINT2:', result);
			return result;
		}
	});

	this.subscribe('users', () => {
		console.log('PRINT3: subscribing');
	});

	console.log('PRINT4: currentUser helper', this.currentUser);

...
};

So, when I have the subscription block, the browser console looks like this:

As you can see, this.helpers is invoked twice. And only after the second call the model is ready.
On the other hand, when I comment out the block:
this.subscribe('users', () => {
	console.log('PRINT3: subscribing');
});

Then the browser console looks like this:

And you can see that this.helpers is called only once, but Meteor doesn't return user data from query. So model is empty.
Do you have any idea why is this so?

Furthermore, when I navigate to the users list (which is one page back), I see this.helpers from the user details called two more times, although the user detiails controller should have been destroyed.
Then, navigating between users list and user details multiple times, there seems to be a buffer of user id's of previously visited user details.
So, on the 3rd visit to random user's details page, this.helpers gets called several times with userId's of all previously visited users in reverse order.
Look at console now:

The first three calls to this.helpers are with the id of the current user I want to see, then this.helpers is called with id's of previously visited users. Yet, the model is populated with the data of the users I'm currently seeing.
This is totally bizarre :)

@MilosStanic what wrong with it? currentUser called first time when no users data loaded, and second time, when meteor loaded users data from server.

@Deadly0 Okay, I understand, seems logical. But I shouldn't need to subscribe to users at all. Should I?
Meteor.users.findOne() should return a non-empty object, even without subscription.
Also why is currentUser called repeatedly when I leave the page, as shown in my second post? And not only that, it calls it several times in reverse order of my previous visit to user details page.

Of course you should subscribe to users . If you not subscribed Meteor.users.findOne will return undefined every time. Remember, data about current logged in user loading automatically without subscription.
About second post. It's hard to say certainly, but when you navigate through pages it's changes some reactive sources, subscriptions, or destroy controller and currentUser method needs to be recalculated.

@Deadly0 this fact that you stated that data about current logged in user is loading automatically without subscription may be the resolution to my problem that I stated in #887
Please take a look at that issue (never mind the first few posts, start from the end). It appears that I have current logged in user data mixed into my users subscription no matter what I do. This reflects on the fact that the data about current user is always loaded into my paginated list of users (it used to be on every page, but after switching to 1.3 it's only on some).
This effectively means that it's impossible to do a paginated list of users correctly in Meteor.
Please correct me if I'm wrong. And if I'm right, do you have any idea how to make a paginated list of users that works properly?
Thank you very much. I think you've shed a lot of light on my knowledge about Meteor.

@MilosStanic is your problem current logged in user appears everywhere? I know the solution for the problem.

@Deadly0 Yes, that's my problem. Thanks.

@MilosStanic pagination in Meteor is non-trivial because results from multiple publications are combined. This is not only a problem for user pagination. More in depth article: https://www.discovermeteor.com/blog/pagination-problems-meteor/
One solution is to use this package: https://atmospherejs.com/percolate/find-from-publication

I think I'll close this as I got my answer from @Deadly0 in #887 about pagination, and thank you too, @sebakerckhof .
Regarding the original issue, @Deadly0 says this is okay, and I should subscribe to collection, even though I need only one object. Even though, I think I remember I didn't really need subscription for $meteor.object(). But I'm not sure.
@Urigo please reopen if you think this is an issue.

@MilosStanic you need to be subscribed to use $meteor.object(). The exception is current logged in user because accounts-base package already have this subscription.

@Deadly0 thanks again, it may very well be that I don't remember correctly, or have simply overlooked the fact.


add options to get/add session to/from the pool by id

Add option to add session to the pool with overridden sessionOptions (id and others).
Add option to get the session from the pool by id.
Add/update respective tests.

Just a notice/thought - removal or retired sessions - I do it with .filter(), but I also need to remove them from the map, I don't think there is a more efficient way than it's done now?
Also regarding overridden session options - I guess It should be enough to just check that there should be an object with optional sessionOptions - and the rest (check of the actual options) will be done while creating the new session during the next step?

Please resolve conversations when you fix the issue. Also please bump version to 1.1.0 and add changelog.

https://github.com/apify/apify-js/pull/953#discussion_r587716252 this is resolved @petrpatek @AndreyBykov ? I don't know what the problem was.


Adding a missing closing block quote

Current documentation ends up with a text paragraph stuck in a code block.

Thank you!


Issue with using a Custom Publish Function

Hi there,
To reproduce this, you can use the standard example https://github.com/AnnotatedJS/GreatAlbums
... and make the following modifications:

Define a publish function, say in PublishAlbums.js:

Meteor.publishComposite("albumsPub", function (tableName, ids, fields) {
  check(tableName, String);
  check(ids, Array);
  check(fields, Match.Optional(Object));

  this.unblock(); // requires meteorhacks:unblock package

  return {
    find: function () {
      this.unblock(); // requires meteorhacks:unblock package

      return Albums.find({}); // Return all albums
    },
  };
});

Define the publication is the table definition:

  pub: "albumsPub",

Add the missing packages:

reywood:publish-composite
meteorhacks:unblock

And all runs fine.
Now, just slightly change the DB request to something more restrictive, like:
return Albums.find({artist:{ $regex: /^A.*/i }}); // Now return only those starting with A

And...

Nothing is displayed
Playing with search gives strange results but nothing is displayed
By making queries from the browser debugger, the publication worked ok (also tried with plain Meteor publish instead of the composite one and had the same behavior).

Note that in the case of my real app, the query is more complex (some kind of security filter based on users associated with objects) and the above regexp is just a dumb example of something more restrictive than "all albums".
I tried to debug a few things in the meteor-tabular package but I'm stuck. Looks like the cursor is empty...
Any idea?

Same behavior using version 1.6.0 except that a "Processing..." box is displayed

... but it looks like I missed the server-side selector.


Example configuration for logs on Linux.

Adds a new example configuration file for the collector, this one specifically for collecting logs on Linux hosts.

I have read the CLA Document and I hereby sign the CLA

Did you have a plan for this being advertised somewhere (maybe https://github.com/signalfx/splunk-otel-collector#advanced-configuration)? Example configs in this directory aren't necessarily as useful as a dedicated readme section (another case of this was deferred in https://github.com/signalfx/splunk-otel-collector/pull/3800#issuecomment-1769508101).

@rmfitzpatrick I'll add a line or two to the README. This confg file is supposed to be useful even with minimum or no editing, so maybe we should drop "example" from its title.


RectClip doesn't split concave polygons

RectClip connects all clipped parts into single polygon. It adds edges along rect border to connect different parts even if these edges don't belong to initial polygon or its border at all.
In the image clipping via polygon-polygon intersection would provide three different polygons 1, 2, 3, while RectClip provides single polygon with contour, highlighted by red curve.
RectClip overload for multiple polygons also doesn't split any individual polygon in paths.

This isn't a bug, but a current limitation of RectClip.
And something I'm working on.


Configurable JS Extensions

In concert with #109 there's probably going to be a call for configuring what valid "js" extensions are. For example, esm and Node itself to some degree, are recommending using .mjs. If the parseJs option, or something like it, comes to fruition, we're going to be able to handle TypeScript, ESM, etc.
Would this addition be welcome on the v5 branch? I'd be happy to submit a PR, but would like to know it's something viable before I put the time in.

Yep, I guess we should add this option.

How about jsExtensions: Array<string>, defaulting to ['js']. We'll loop through the jsExtensions and look for a file corresponding to each one. If you need fancy parsing for some extensions, you'll need to provide a loadJs function (as discussed for v5 in #109) that looks at the file path and decides what to do from there.
@shellscape: Does that API sound like it will work?

@davidtheclark yeah that sounds good to me

In the v5 branch, this can now be accomplished by using a combination of searchPlaces and loaders.
@shellscape I think I recall that you want to use require hooks, right? If that's the case, you can use the regular JS loader for all the JS-like files you want, e.g.:
{
  searchPlaces: [
    'someting.config.js',
    'someting.config.mjs',
    'someting.config.ts',
  ],
  // Specify how to load the special extensions.
  // loadJs just uses require
  loaders: {
    mjs: cosmiconfig.loadJs,
    ts: cosmiconfig.loadJs,
  }
}


fix(lint): Extend --skip-schema-validation for lint command

closes #13413
What this PR does / why we need it:
Added full support of --skip-schema-validation for the lint command
If applicable:

[x] this PR contains unit tests

@mattfarina hi! Could you please check the PR?

Any updates here? We urgently need this fix

@banjoh hi! Could you please check it?

Hello @mattfarina, @banjoh , just a gentle reminder about this PR created months ago. If possible, could it be reviewed before the next Helm release? I’d appreciate any feedback to help get it ready in time. Thank you!

@scottrigby finishing touch for --skip-schema-validation


New version: tangshimin.MuJing version 2.2.3

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2023-12-12 15:07:15.591 [REPO] Setting error JSON 1.0 fields

@wingetbot waivers Add Validation-Executable-Error

As with previous versions, I struggle to get output from this application. I'm assuming that this is because I'm not using it correctly - possibly it requires an MKV as argument to have any output? I'm approving as running it generates no errors that I could locate.


[Feature Request] Additonal secret engines for Vault

I purpose that vela should support more secret engines besides key/value, like gcp, aws, and azure. This change could easily allow administrators to provide a set of credentials to be utilized by tools like terraform without needing to provide long lived credentials.
Some specific design considerations to think of:

Multiple values are returned by the secret engines mentioned above

for example, secret_key, secret_token, lease_duration, etc.

Should the secret be configured within vault directly or via vela?

configuring through vela can be challenging for specific secret engines/configurations due to the number of steps for setup

Vela specific metadata about these secrets cannot be stored within vault since it doesn't allow additonal metadata like the k/v secret engine does

for example, the aws secret engine only allows the following parameters when you add roles https://github.com/hashicorp/vault/blob/961f4468385aa08848baee1b27e64b93904c8bd7/website/source/api/secret/aws/index.html.md#parameters-3, so the metadata about the secret would need to be contained within vela and validated before injection

My initial thought on changes to the pipeline to accomplish this looks like:
version: "1"

steps:
- name: test
  image: alpine
  secrets: [ aws ]
  commands:
    - env | grep AWS_

secrets:
  - name: aws
    key:  aws/sts/foo
    engine: vault
    ttl: 60m

With the step of test returning the output of:
AWS_LEASE_ID=<value>
AWS_LEASE_DURATION=<value>
AWS_LEASE_RENEWABLE=<value>
AWS_ACCESS_KEY=<value>
AWS_SECRET_KEY=<value>
AWS_SECURITY_TOKEN=<value>

The above example showcases that vela will retrieve the credentials through the aws secrets engine and convert all of the returned values to be in the format of <secret name in vela.yml>_<key returned in vault>=<key value>.
My proposed workflow to accomplish the above looks like:

Vault administrator enables the new secret engine of AWS within Vault

i.e. vault secrets enable aws and vault write aws/config/root access_key=<access key> secret_key=<secret key> region=<region>

Vault administrator writes the role(s) they want

i.e. vault write aws/roles/<role name> credential_type=<credential type> ...

Vela administrator maps the Vault secret with a Vela secret

i.e. vela map secret <name of secret within vault> --name <name of secret within vela> --org <org> --repo <repo> ...

Vela stores the above metadata about the secret within the Vela database since Vault doesn't allow that type of metadata for non k/v store secret engines

some function like https://github.com/go-vela/server/blob/a41ef02c401ba05f8dc229d05de33236f6ceafbc/secret/vault/vault.go#L125 would need to be added to put the data into the vela database

User attempts to utilize the newly mapped secret with a pipeline

the existing function of https://github.com/go-vela/server/blob/a41ef02c401ba05f8dc229d05de33236f6ceafbc/secret/vault/vault.go#L37 would need to be modified to check the Vela database if the specified secret is not a k/v secret

Vela connects to Vault to receive the credentials if the validation to approve injection was successful
Vela injects the secrets into the pipeline

Looking for feedback on this before I attempt to write the code to accomplish it.

Hello 👋
The Vela admins have decided to condense all existing issues into a single repo found @ go-vela/community.
For a direct link to your counterpart issue in that repo, you can find it linked yesterday in the history of this issue.
If you have any questions, please feel free to direct them to the community repo.
Thanks for your input and feedback!


.NET Core Troubleshooting Logs nuget package name #25658

Originally posted by @bogan in https://github.com/MicrosoftDocs/azure-docs/issues/25658#issuecomment-621796732
Summarizing feedback:

This still appears to be a problem. The hyperlink in Point 1 still refers to .NET Framework version.
Also, the sentence beginning "Hence the version of Microsoft.AspNet.ApplicationInsights.HostingStartup..." also refers to the .NET Framework version.
Lastly, the sentence beginning "The latest version of Microsoft.ApplicationInsights.AspNetCore is 2.8.2..." now refers to an outdated version of Application Insights.
Another problem. The code snippet refers to the FileDiagnosticsTelemeteryModule class. It appears that this class has now been moved out of the HostingStartup package into the base ApplicationInsights package. See https://docs.microsoft.com/en-us/dotnet/api/microsoft.applicationinsights.extensibility.implementation.tracing.filediagnosticstelemetrymodule.initialize?view=azure-dotnet and microsoft/ApplicationInsights-dotnet#1059. The package Microsoft.ApplicationInsights v2.14.0 or later is therefore needed to be installed. In fact, I don't think you even need the HostingStartup package to be installed anymore, because everything that is needed is now in the base package. But not sure.

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

ID: 59b9aed0-b6a7-d3e9-d616-85e865966444
Version Independent ID: df2e545f-7f65-82b7-c793-2d04d91d7d9e
Content: Troubleshooting no data - Application Insights for .NET - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-troubleshoot-no-data.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

What is all this
Sent from my iPhone

On Jun 8, 2020, at 2:20 AM, BhargaviAnnadevara-MSFT<EMAIL_ADDRESS>wrote:

@bogan Thanks a lot for catching those errors and bringing it to our attention! We're investigating this and will get back to you shortly.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@mrbullwinkle The Troubleshooting Logs section might need some rework if #4 above is true. Want to know your thoughts before making any fixes.

#assign:lgayhardt

Thank you for helping us to improve the docs. We are tracking this internally. This item is currently on our backlog, but this thread will be updated when the work item is completed.
#please-close


timestamp cause checksum invalid Error

My project was using server rendering, and when I use reduxForm() with initialValues option, it threw an Error:
Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
 (client) estamp.1">1447671517920

1447671517228
<li s

Which is caused by the different timestamp value that created by Date.now().
Is there any way to prevent this error?

Oh, I forgot something.
This error only appeared when redux-devtools is opening. So timestamp is rendering in the devtools container. As a result, my project will not throw this error in the production environment.
I think this is not a problem at all. It's fine to close this issue now.
BTW, I read redux-form source, but cannot figure out what is the meaning of timestamp argument, can anybody tell me?

The timestamp argument is vestigial from a previous attempt at something. I'll remove it. Thanks.


Does not appear to update sub-tasks

Our dev team uses Jira "sub-tasks" to track the progress against larger backlog items (Jira "issues"). It'd be great if I could issue a Smart Commit against a sub-task rather than just an issue.

Hi @chrisgagne. I'm from a team at Atlassian that is currently in the process of migrating the ownership over to us from GitHub. As a part of this, we're currently going through the backlog of issues and prs to see what can still be addressed and what should be closed.
I've flagged this with the 'to triage' label so my team can discuss. We'll keep you posted on the status of this.

That's wonderful, thanks @rachellerathbone! I invite your team to think of this through the usual Scrum framework: the backlog item (issue in Jira) reflects the what while the task (sub-task in Jira) reflects the how. A developer is more likely to log work against a sub-task than a issue as that is where the "how" of the work is most likely to be described.

Hey @chrisgagne,
this integration should work with any type of Jira issue, including sub-tasks. You would just have to put the issue ID of the sub task in your commit message and the commit should show up in the sub task in Jira.
And smart commits should work with sub tasks, as well.
Or do you mean something else? What exactly would you like to achieve?

Hey @chrisgagne. Given it's been a few months since we've had any activity on this issue I'm going to go ahead and close it. If the above response didn't help and you need further assistance, please feel free to reopen.


As an invited member of a team, how to retrieve project files.

Hello,
Reading the tour of the new textile I can't understand how I can retrieve the files from a project of a team I was just invited to.
Do the files of the project need to be exported to a bucket first ? It's not clear what I need to do.
Maybe the documentation could be updated to better describe this use case.
Thanks for all the work,
Nicolas

hi @nlko sorry for the delay on response here. we've just released a new architecture for accounts. there are no more Projects. just devs, orgs, and buckets. https://docs.textile.io is up to date.


jdbc connection string not working inside docker container 18c

I am trying to connect database to java web application using jdbc connection string inside docker container 18c, tried "jdbc:oracle:thin:@<IP_ADDRESS>:1521:frsh" and "jdbc:oracle:thin:@localhost:1521:frsh"
but its giving error
[sql] java.sql.SQLException: Listener refused the connection with the following error:
ORA-12505, TNS:listener does not currently know of SID given in connect descriptor
My listener.ora looks like
LISTENER =
(DESCRIPTION_LIST =
(DESCRIPTION =
(ADDRESS = (PROTOCOL = IPC)(KEY = EXTPROC1))
(ADDRESS = (PROTOCOL = TCP)(HOST = <IP_ADDRESS>)(PORT = 1521))
)
)
DEDICATED_THROUGH_BROKER_LISTENER=ON
DIAG_ADR_ENABLED = off
and tnsnames.ora looks like
FRSH1=localhost:1521/FRSH1
FRSH=
(DESCRIPTION =
(ADDRESS = (PROTOCOL = TCP)(HOST = <IP_ADDRESS>)(PORT = 1521))
(CONNECT_DATA =
(SERVER = DEDICATED)
(SERVICE_NAME = FRSH)
)
)
and /etc/hosts looks like
<IP_ADDRESS>       localhost
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
<IP_ADDRESS>      4a125243bc08
My sid is frsh and i can sqlplus <schema_name>/<schema_password>@frsh  and run queries.

Hi @monojitsaba,
I think your issue is that you connect to the SID, not the service.
Please change the : to a / in your JDBC connect string: jdbc:oracle:thin:@localhost:1521/frsh

Amazing!!
Thx bro!!!
Still helping the needy!


Typescript should be ignore import only types when emit dist file

Bug Report

🔎 Search Terms

import only types emit include type file
🕗 Version & Regression Information

This changed between versions 4.0 and 4.3_
This is the behavior in every version I tried, and I reviewed the FAQ for entries about type___
I was unable to test this on prior versions because Typescript is too older_

⏯ Playground Link

n/a
💻 Code

// module.ts
export interface IModule { }
export class Module {  }

// app.ts
import { IModule } from './module';
class App {
    module: IModule;
}

Run tsc to emit Javascript files.
🙁 Actual behavior

// app.js
import './module';
var App = /** @class */ (function () {
    function App() {
    }
    return App;
}());
//# sourceMappingURL=index.js.map

The emitted app.js file should not include import './module'; becase I just import type.
🙂 Expected behavior

// app.js
var App = /** @class */ (function () {
    function App() {
    }
    return App;
}());
//# sourceMappingURL=index.js.map

More
import type { IModule } from './module'; could solve this issue, but it's fussy.

Try to use the importsNotUsedAsValues compiler options.
https://www.typescriptlang.org/tsconfig#importsNotUsedAsValues

Your expected behavior is exactly what I get with TypeScript 4.3 and the default init project settings.

aha, my tsconfig.json set importsNotUsedAsValues: "preserve". It works if I remove it. Thanks


Barcode Analysis: Resource not found

From https://travis-ci.org/AbdealiJK/file-metadata/jobs/136607124
319 . Analyzing File:Tour revolver.jpg
Exception in thread "main" java.io.IOException: Resource not found: file:///home/travis/build/AbdealiJK/file-metadata/tests/files/File_Tour_revolver.jpg
	at com.google.zxing.client.j2se.ImageReader.readImage(ImageReader.java:47)
	at com.google.zxing.client.j2se.DecodeWorker.decode(DecodeWorker.java:119)
	at com.google.zxing.client.j2se.DecodeWorker.call(DecodeWorker.java:75)
	at com.google.zxing.client.j2se.CommandLineRunner.main(CommandLineRunner.java:84)
Caused by: java.lang.IllegalArgumentException: Numbers of source Raster bands and source color space components do not match
	at java.awt.image.ColorConvertOp.filter(ColorConvertOp.java:482)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.acceptPixels(JPEGImageReader.java:1263)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.readImage(Native Method)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.readInternal(JPEGImageReader.java:1231)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.read(JPEGImageReader.java:1034)
	at javax.imageio.ImageIO.read(ImageIO.java:1448)
	at javax.imageio.ImageIO.read(ImageIO.java:1400)
	at com.google.zxing.client.j2se.ImageReader.readImage(ImageReader.java:45)
	... 3 more

114 . Analyzing File:Plateau universel.jpg
319 . Analyzing File:Tour revolver.jpg

Ok, so the actual error is Numbers of source Raster bands and source color space components do not match and it seems to make sense:
The exifdata for Tour revolver.jpg says:

ICC_Profile:ColorSpaceData: GRAY
File:ColorComponents: 3

Both of which are contradictory. The actual shape is (694, 801, 3).
I think it was because teh ICC Profile has references to an Apple computer and Epson corporation (printer?).
Not sure what needs to be done here. It seems like Java is getting confused because of invalid exifdata. Should I silence such errors ? Or let the error be shown ?
Note that Pillow/skimage doesnt care about it though ^^

It should be reported to the wiki, indicating that one tool cant read it, and let humans decide if it should be fixed.

Done :+1:
Wrote on the talk pages at

https://commons.wikimedia.org/w/index.php?title=File_talk:Tour_revolver.jpg&oldid=198779727
https://commons.wikimedia.org/w/index.php?title=File_talk:Plateau_universel.jpg&oldid=198780180


[ci skip] adding user @jvdd

Hi! This is the friendly automated conda-forge-webservice.
I've added user @jvdd as instructed in #24.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #24

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Fill out the package

This adds tests, documentation, and new functionality.

If anyone wants to review this, here are some pointers to things I wonder about:

to lazy-concatenate files that store arrays with different sizes, this provides a thin wrapper (load_chunked) around BlockArrays. It also uses @require to keep the packages independent of one another. Does this make sense, or is it just silly?
select_series is a bit "fragile" in that it requires that you specify everything except the integer count for the file series. I.e., myimage_*.png would match & order files myimage_1.png...myimage_1253.png and get the order right (it doesn't rely on alphabetization). However, will it be annoying to have to type out the myimage_ part of the filename?

I remember that you've made some demos to showcase how Julia is better than Dask array in both line codes and performance. Is that supported by this package? If so it would be great to share it here (because I failed to reproduce the performance difference last time when I tried this package). It would also be great if some performance tips/caveats can be added (e.g., how to iterate this array for optimized performance)

will it be annoying to have to type out the myimage_ part of the filename?

Looks okay to me. After all, users can explicit create the Vector{String} and pass it. (Might be a good tip to mention how https://github.com/vtjnash/Glob.jl can be used here)

By and large, this package doesn't assume mmap support and so doesn't replicate the performance advantage. It's vastly better with, e.g., TiffImages or NRRD directly. If you have a host of such files, opening each with mmap and then using StackedViews may be the better way to go. I will add this to the documentation.


chore: remove old/unused Render Action code

Removes old code left over from when we had the separate RENDER action's.
See: https://github.com/Microsoft/ConversationLearner-UI/pull/684
Originally we kept the api actions and added render actions but as I was implementing we realialized that we needed to support the passing of information between the two and they would often come in pairs. We since merged them into a single Callback the supports logic and render.
I didn't remove this at the time.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pay Patient validations.

Please check if the PR fulfills these requirements:

[x] After these changes, the app was run and still works as expected
[ ] Tests for these changes were added (if applicable)
[x] All existing unit tests were run and still pass

Please specify the type of change this PR introduces (Bug fix, feature addition, content update, chore, etc.):
Additional Notes:

Thanks. Yeah, that's probably a good idea. If I have to add another selection component, I'll create a standardized one in the common library.


DatePicker Not Working in inertiajs

Hello.. I tried to install tw-elements through npm and yet date picker isn't working.. im using inertiajs of laravel
const defaultTheme = require('tailwindcss/defaultTheme');
const colors = require('tailwindcss/colors');

module.exports = {
    content: [
        './vendor/laravel/framework/src/Illuminate/Pagination/resources/views/*.blade.php',
        './vendor/laravel/jetstream/**/*.blade.php',
        './storage/framework/views/*.php',
        './resources/views/**/*.blade.php',
        './resources/js/**/*.vue',
        './node_modules/tw-elements/dist/js/**/*.js'
    ],

    theme: {
        extend: {
            fontFamily: {
                sans: ['Nunito', ...defaultTheme.fontFamily.sans],
            },
            colors: {
                cyan: colors.cyan,
            }
        },
    },

    plugins: [
        require('@tailwindcss/forms'),
        require('@tailwindcss/typography'),
        require('@tailwindcss/line-clamp'),
        require('tw-elements/dist/plugin')
    ],
};

Include import from Vite in your welcome.blade.php file: @vite('resources/js/app.js'). FYI in next week we are going to publish some tutorials about integration Tailwind Elements with popularn front and backend frameworks.


Connection pool is not safe

Describe the bug
The connection pool is only useful when writing concurrent code, but it doesn't actually stop multiple goroutines from accessing the same connection simultaneously.
To Reproduce
The PR includes a test suite that reproduces the issue with the old Pool code, so long as it's edited to include the afterAcquire and beforeRelease test hooks.
Expected behavior

Pool.Exec(), Pool.Ping(), Pool.Query(), and Pool.Server() are safe for concurrent use. Two calls can only use the same connection serially.
When a query is submitted to the pool,

If there is an unused connection in the pool, it will be used to run the query immediately.
Otherwise, the pool will wait for a connection to become available and then run the query.

A connection is only returned to the pool when it is safe to do so.

Cursors perform their work via the connection, which means if the pool returns a cursor, the connection cannot be returned to the pool until the cursor is closed.

A mutex is not used in the hot path as per the expectation set in #488.

Screenshots
n/a
System info
n/a
Additional context
Problems with the prior pool implementation:

Pool.conn() uses the connections in round-robin order without checking if they are still in use. It appears queries and cursors can be multiplexed on one connection, so this would work fine in the single goroutine case. However, if multiple goroutines call the Pool there is no guarantee each Connection will only be used by one goroutine at a time.

Similarly there is a race condition in the lines:

pos := atomic.AddInt32(&p.pointer, 1)
if pos == int32(len(p.conns)) {
	atomic.StoreInt32(&p.pointer, 0)
}
pos = pos % int32(len(p.conns))

Suppose pointer == 9 and len(conns) == 10. Three goroutines could run in this sequence:
GR 1: pointer=10                pointer=0                uses conns[0]
GR 2:              pointer=11                            uses conns[1]
GR 3:                                       pointer=1    uses conns[1]
      ---------time------->

Because there is no "in use" check, two of the goroutines end up using the same Connection.

It sounds like a legitimate bug report.  I don't have a timeline for a fix.

I have just submitted a proposed fix in #498.

Not an issue; the Connection and Cursor docs say they do not support concurrent calls, but those comments are very old and aren't actually true.


[1.3.*] Types of parameters '__0' and 'value' are incompatible.

Bug Report
- [x] bug report
- [ ] feature request

Versions.
@typescript: 2.4.2
@rxjs: 5.4.3
@angular/cli: 1.3.1
node: 8.0.0
os: linux x64
@angular/animations: 4.3.4
@angular/cdk: 2.0.0-beta.8
@angular/common: 4.3.4
@angular/compiler: 4.3.4
@angular/core: 4.3.4
@angular/flex-layout: 2.0.0-beta.8
@angular/forms: 4.3.4
@angular/http: 4.3.4
@angular/material: 2.0.0-beta.8
@angular/platform-browser: 4.3.4
@angular/platform-browser-dynamic: 4.3.4
@angular/router: 4.3.4
@angular/cli: 1.3.1
@angular/compiler-cli: 4.3.4

Repro steps.
    let s: string = 'abc',
        b: boolean = true,
        n: number = 10;

    Observable.forkJoin([Observable.of(s), Observable.of(b), Observable.of(n)])
        .map(([s, b, n]: [string, boolean, number]) => s);

The log given by the failure.
ERROR in file.ts (6,18): Argument of type '([s, b, n]: [string, boolean, number]) => string' is not assignable to parameter of type '(value: boolean[], index: number) => string'.
  Types of parameters '__0' and 'value' are incompatible.
    Type 'boolean[]' is not assignable to type '[string, boolean, number]'.
      Property '0' is missing in type 'boolean[]'.

Desired functionality.
This code gets compiled on v1.2.7 but fails on v1.3.*

That's not an Angular-CLI issue.  TS 2.4 has more comprehensive type checking.  The forkJoin method's signature for the array overload doesn't work with a multi-type array.  The array is not actually needed anyway; there's an overload that handles the observables directly.  You can also manually specify the generic type argument for forkJoin.

@clydin Could you please provide some reference that forkJoin doesn't work with a multi-type array in TS 2.4? Is it by design? What would be a solution in this case?
In this example forkJoin accepts observables of multiple types.
I removed all the type casts in my example but I'm getting ERROR in file.ts (5,38): Argument of type 'string' is not assignable to parameter of type 'number'.:
let s: string = 'abc',
    n: number = 10;

Observable.forkJoin([Observable.of(s), Observable.of(n)])
    .map(responses => takesB(responses[1]));

function takesB(n: number) {
}

As I mentioned, you can remove the usage of the array.  The example that you linked doesn't use the array overload of forkJoin with observables of different types.  The array example (second one), uses an array of only numbers.  Also, in the code shown above, responses has an inferred type of string[] which is way it is failing.
Two other options are to either manually define the generic type parameter on forkJoin or cast the array parameter as any[].

Thanks @clydin. There are 3 working examples that get compiled under angular-cli 1.3.1 and TS 2.4:

cast the array parameter as any[]

Observable.forkJoin([Observable.of(s), Observable.of(n)])
    .map(([s, n]: any[]) => s);

surprisingly, I found out that if I pass observables into forkJoin not as an array but as a function arguments, the error disappears, so my original example works:

Observable.forkJoin(Observable.of(s), Observable.of(n))
    .map(([s, n]: [string, number]) => s);

manually define the generic type parameter on forkJoin.

Observable.forkJoin<string,number>(Observable.of(s), Observable.of(n))
    .map(([s, n]) => s);

Closing as there seems to be work-arounds and that this is a typing issue, not a CLI issue. Thanks @clydin


feat: Add a getter for unmanaged codes

Pending https://github.com/seamapi/seam-connect/pull/2774

I didn't see it do so much last time, but it updated the docs accordingly? neat!


Return anchor in get_messages when use_first_unread_anchor=True

If you call the get_messages API with the use_first_unread_anchor argument, the server ends up picking the anchor (center) of the batch of messages for you.  Here's the logic:
    if use_first_unread_anchor:
        condition = column("flags").op("&")(UserMessage.flags.read.mask) == 0

        # We exclude messages on muted topics when finding the first unread
        # message in this narrow
        muting_conditions = exclude_muting_conditions(user_profile, narrow)
        if muting_conditions:
            condition = and_(condition, *muting_conditions)

        # The mobile app uses narrow=[] and use_first_unread_anchor=True to
        # determine what messages to show when you first load the app.
        # Unfortunately, this means that if you have a years-old unread
        # message, the mobile app could get stuck in the past.
        #
        # To fix this, we enforce that the "first unread anchor" must be on or
        # after the user's current pointer location. Since the pointer
        # location refers to the latest the user has read in the home view,
        # we'll only apply this logic in the home view (ie, when narrow is
        # empty).
        if not narrow:
            pointer_condition = inner_msg_id_col >= user_profile.pointer
            condition = and_(condition, pointer_condition)

        first_unread_query = query.where(condition)
        first_unread_query = first_unread_query.order_by(inner_msg_id_col.asc()).limit(1)
        first_unread_result = list(sa_conn.execute(first_unread_query).fetchall())
        if len(first_unread_result) > 0:
            anchor = first_unread_result[0][0]
        else:
            anchor = LARGER_THAN_MAX_MESSAGE_ID

It'd be nice to actually have the server report what anchor it used (rather than needing to compute it on the client side), since that's going to be important for the client.
I think mostly this involves putting ret['anchor'] = anchor in an if statement near the end of get_messages_backend, and adding tests.

@zulipbot claim

@timabbott I was able to successfully return the anchor in the response.
What should the tests for this check for ?
Would they be something like this:
def test_use_first_unread_anchor_response(self) -> None:
   results = self.get_and_check_messages(dict(use_first_unread_anchor='true'))
   self.assertIn("anchor", result)

I believe the logic for the code that you've pasted in the issue description is already being checked in
test_use_first_unread_anchor_with_some_unread_messages in test_narrow.py L1483

That seems fine for that part.  I'd also update 1-2 existing tests that use get_messages with that option to add checks the actual value of the anchor.


Delete a file via web not giving errors, not allowing new branch if source branch protected

A few errors in v1.9-rc2, in that the Delete UI will not report an error if a bad filename, existing branch, etc. are given as then it throws an error in the template, since it uses the wrong template page for errors. It also won't let you delete and commit to a copy of the source branch if the source branch is protected, as it says that the new branch is protected, which it isn't.

Fixed by #7478


Submit questionare

Timestamp for each question and whole questionnaire,
general refactor, services timestampservice , preparedataservice
models - timestamp,
timestamp recorded when question is shown(starttime) and pressed next(endtime) : if this needs to be changed let me know
timestamp in (UTC milliseconds/ 1000)

@abhiglobalistic Please go ahead and resolve the conflicts offline and push to staging.


Multimer predictions degraded after updating from 2.2.3 to 2.2.4?

Hello, this is ranking_debug.json from a test multimer run that I had performed with version 2.2.3:
{     "iptm+ptm": {         "model_1_multimer_v2_pred_0": 0.8584136076107545,         "model_2_multimer_v2_pred_0": 0.8055674020453231,         "model_3_multimer_v2_pred_0": 0.6845710872128711,         "model_4_multimer_v2_pred_0": 0.8423123043127994,         "model_5_multimer_v2_pred_0": 0.8676285262204777     },     "order": [         "model_5_multimer_v2_pred_0",         "model_1_multimer_v2_pred_0",         "model_4_multimer_v2_pred_0",         "model_2_multimer_v2_pred_0",         "model_3_multimer_v2_pred_0"     ] }
and this is from an equivalent job, carried out using the same exact input, after updating to 2.2.4:
{     "iptm+ptm": {         "model_1_multimer_v2_pred_0": 0.40774006137839586,         "model_2_multimer_v2_pred_0": 0.44277279108467366,         "model_3_multimer_v2_pred_0": 0.442618376771567,         "model_4_multimer_v2_pred_0": 0.5631646773883024,         "model_5_multimer_v2_pred_0": 0.48998768122405234     },     "order": [         "model_4_multimer_v2_pred_0",         "model_5_multimer_v2_pred_0",         "model_2_multimer_v2_pred_0",         "model_3_multimer_v2_pred_0",         "model_1_multimer_v2_pred_0"     ] }
Is anyone else experiencing the same? I am not getting any obvious error upon running 2.2.4, so it's not clear to me why there should be such a significant difference...

Hi thanks for raising this. Can you please provide more details of the test
run so this can be reproduced?
On Thu, 22 Sept 2022 at 17:12, Luca Jovine @.***> wrote:

Hello, this is ranking_debug.json from a test multimer run that I had
performed with version 2.2.3:
{ "iptm+ptm": { "model_1_multimer_v2_pred_0": 0.8584136076107545,
"model_2_multimer_v2_pred_0": 0.8055674020453231,
"model_3_multimer_v2_pred_0": 0.6845710872128711,
"model_4_multimer_v2_pred_0": 0.8423123043127994,
"model_5_multimer_v2_pred_0": 0.8676285262204777 }, "order": [
"model_5_multimer_v2_pred_0", "model_1_multimer_v2_pred_0",
"model_4_multimer_v2_pred_0", "model_2_multimer_v2_pred_0",
"model_3_multimer_v2_pred_0" ] }
and this is from an equivalent job, carried out using the same exact
input, after updating to 2.2.4:
{ "iptm+ptm": { "model_1_multimer_v2_pred_0": 0.40774006137839586,
"model_2_multimer_v2_pred_0": 0.44277279108467366,
"model_3_multimer_v2_pred_0": 0.442618376771567,
"model_4_multimer_v2_pred_0": 0.5631646773883024,
"model_5_multimer_v2_pred_0": 0.48998768122405234 }, "order": [
"model_4_multimer_v2_pred_0", "model_5_multimer_v2_pred_0",
"model_2_multimer_v2_pred_0", "model_3_multimer_v2_pred_0",
"model_1_multimer_v2_pred_0" ] }
Is anyone else experiencing the same? I am not getting any obvious error
upon running 2.2.4, so it's not clear to me why there should be such a
significant difference...
—
Reply to this email directly, view it on GitHub
https://github.com/deepmind/alphafold/issues/597, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKS2BZQTT7WLUCNIVLWTRLV7SAOLANCNFSM6AAAAAAQTGX6SM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sure, of course. This was my input FASTA:

protein1
IDWDVYCSQDESIPAKFISRLVTSKDQALEKTEINCSNGLVPITQEFGINMMLIQYTRNELLDSPGMCVFWGPYSVPKNDTVVLYTVTARLKWSEGPPTNLSIQCYMPK
protein2
RSWHYVEPKFLNKAFEVALKVQIIAGFDRGLVKWLRVHGRTLSTVQKKALYFVNRRYMQTHWANYMLWINKKIDALGRTPVVGDYTRLGAEIGRRIDMAYFYDFLKDKNMIPKYLPYMEEINRMRPADVPVKYM

(which basically corresponds to our deposition 5IIB)

Also...
> python run_alphafold.py --helpshort
/usr/local/conda/miniconda3/envs/af2/lib/python3.7/site-packages/haiku/_src/data_structures.py:37: FutureWarning: jax.tree_structure is deprecated, and will be removed in a future release. Use jax.tree_util.tree_structure instead.
  PyTreeDef = type(jax.tree_structure(None))
Traceback (most recent call last):
  File "run_alphafold.py", line 39, in <module>
    from alphafold.relax import relax
  File "/media/3p5TBssd1/usr/local/alphafold/alphafold/alphafold/relax/relax.py", line 18, in <module>
    from alphafold.relax import amber_minimize
  File "/media/3p5TBssd1/usr/local/alphafold/alphafold/alphafold/relax/amber_minimize.py", line 25, in <module>
    from alphafold.relax import cleanup
  File "/media/3p5TBssd1/usr/local/alphafold/alphafold/alphafold/relax/cleanup.py", line 22, in <module>
    import pdbfixer
ModuleNotFoundError: No module named 'pdbfixer'

> python -m pip install pdbfixer
Looking in indexes: https://pypi.org/simple, https://pypi.ngc.nvidia.com
ERROR: Could not find a version that satisfies the requirement pdbfixer (from versions: none)
ERROR: No matching distribution found for pdbfixer

just my 2 cents, I don't have the issue with
    --use_gpu_relax=true \
    --model_preset=multimer \
    --max_template_date=3000-01-01 \
    --db_preset=full_dbs

pasted sorted values: (2.2.4 on the left and 2.2.3 on the right)
       "model_5_multimer_v2_pred_1": 0.8600176497731321,               "model_2_multimer_v2_pred_4": 0.8423162898031145,
       "model_4_multimer_v2_pred_1": 0.8355726960868132,               "model_2_multimer_v2_pred_2": 0.8345795690267899,
       "model_5_multimer_v2_pred_0": 0.8106497232061523,               "model_4_multimer_v2_pred_4": 0.8274770532116257,
       "model_5_multimer_v2_pred_4": 0.7618542529543955                "model_5_multimer_v2_pred_1": 0.8100580241139267,
       "model_1_multimer_v2_pred_2": 0.7462422015606857,               "model_1_multimer_v2_pred_1": 0.7556708441527138,
       "model_5_multimer_v2_pred_3": 0.7262914654950798,               "model_1_multimer_v2_pred_2": 0.7530240249541638,
       "model_4_multimer_v2_pred_2": 0.6850626261632934,               "model_4_multimer_v2_pred_0": 0.7453822555113954,
       "model_1_multimer_v2_pred_4": 0.6504379081135182,               "model_1_multimer_v2_pred_3": 0.7074747369853926,
       "model_4_multimer_v2_pred_3": 0.6282972043309893,               "model_4_multimer_v2_pred_2": 0.6217676770784255,
       "model_4_multimer_v2_pred_0": 0.5659582021061171,               "model_5_multimer_v2_pred_2": 0.5892705180579751,
       "model_1_multimer_v2_pred_3": 0.5530272925711416,               "model_1_multimer_v2_pred_4": 0.5886967664945151,
       "model_1_multimer_v2_pred_1": 0.550091036479859,                "model_4_multimer_v2_pred_1": 0.5816013102240326,
       "model_2_multimer_v2_pred_0": 0.537335888385569,                "model_2_multimer_v2_pred_0": 0.517386942967343,
       "model_2_multimer_v2_pred_2": 0.534451564803337,                "model_1_multimer_v2_pred_0": 0.5026680309406077,
       "model_4_multimer_v2_pred_4": 0.5258449296466584,               "model_4_multimer_v2_pred_3": 0.48747237933686044,
       "model_2_multimer_v2_pred_3": 0.515407234197668,                "model_5_multimer_v2_pred_3": 0.46593239818276644,
       "model_3_multimer_v2_pred_1": 0.499675159211949,                "model_3_multimer_v2_pred_2": 0.43583198796278766,
       "model_2_multimer_v2_pred_4": 0.4734043448923909,               "model_2_multimer_v2_pred_1": 0.42842107067582047,
       "model_3_multimer_v2_pred_0": 0.46022060251713326,              "model_5_multimer_v2_pred_0": 0.4262880199002601,
       "model_1_multimer_v2_pred_0": 0.4401072457868955,               "model_3_multimer_v2_pred_1": 0.42030398029562555,
       "model_5_multimer_v2_pred_2": 0.4233857428348928,               "model_3_multimer_v2_pred_4": 0.41386853973521476,
       "model_3_multimer_v2_pred_4": 0.4185971424788965,               "model_3_multimer_v2_pred_0": 0.4004228507119986,
       "model_3_multimer_v2_pred_3": 0.4112920469041982,               "model_2_multimer_v2_pred_3": 0.381738114315064,
       "model_3_multimer_v2_pred_2": 0.3990563873655274,               "model_3_multimer_v2_pred_3": 0.3696506974209459,
       "model_2_multimer_v2_pred_1": 0.3888554107330705,               "model_5_multimer_v2_pred_4": 0.35258544121857827

Thanks @lucajovine. Are you running this with the docker image? It should have all of the prerequisites installed. pdbfixer isn't available on pypi, so needs to be installed with conda.
Thanks @truatpasteurdotfr we also didn't see any issues in internal testing.

Hi, thank you - am re-installing 2.2.4 from scratch rather than pulling it into my existing installation (although this always worked fine in the past, maybe something odd happened this time). Will re-run the test after that and let you know what happened!
@Htomlinson14 yes this was from the docker image

Hello again, so I reinstalled from scratch 2.2.3 and 2.2.4 on two different machines and re-run 4 test jobs on each (two different  monomers and two different multimers, with multimer_1 being the same as above).
I am afraid that the problem persists, even though (1) it does not seem to happen with monomers and (2) it is more evident for one multimer than the other:
monomer_1
	2.2.3
		"model_1_pred_0": 92.51882397715578,
		"model_2_pred_0": 92.2056299645637,
		"model_3_pred_0": 92.03138974894895,
		"model_4_pred_0": 93.1682804560119,
		"model_5_pred_0": 93.65241384700947
	2.2.4
		"model_1_pred_0": 92.53758910826187,
		"model_2_pred_0": 92.94215515721487,
		"model_3_pred_0": 93.52598782948384,
		"model_4_pred_0": 93.42160017797602,
		"model_5_pred_0": 93.70445135155603

monomer_2
	2.2.3
        "model_1_pred_0": 85.85987502475369,
        "model_2_pred_0": 84.90289928449474,
        "model_3_pred_0": 86.95139195928073,
        "model_4_pred_0": 86.64831516433784,
        "model_5_pred_0": 86.92184064848513
	2.2.4
		"model_1_pred_0": 85.94278137586902,
		"model_2_pred_0": 84.20654974770405,
		"model_3_pred_0": 86.83553028330901,
		"model_4_pred_0": 87.14120498958022,
		"model_5_pred_0": 87.1268406084284

multimer_1
	2.2.3
		"model_1_multimer_v2_pred_0": 0.8501568830419893,
		"model_2_multimer_v2_pred_0": 0.8486699468672132,
		"model_3_multimer_v2_pred_0": 0.3229293496900477,
		"model_4_multimer_v2_pred_0": 0.8455477128109672,
		"model_5_multimer_v2_pred_0": 0.852226493932774
	2.2.4
		"model_1_multimer_v2_pred_0": 0.5117391870226631,
		"model_2_multimer_v2_pred_0": 0.4098071256975262,
		"model_3_multimer_v2_pred_0": 0.43044599005298734,
		"model_4_multimer_v2_pred_0": 0.5445095546638219,
		"model_5_multimer_v2_pred_0": 0.5859474538776686

multimer_2
	2.2.3
		"model_1_multimer_v2_pred_0": 0.8336408586490794,
		"model_2_multimer_v2_pred_0": 0.8327994809055634,
		"model_3_multimer_v2_pred_0": 0.8024798134822844,
		"model_4_multimer_v2_pred_0": 0.811666152078712,
		"model_5_multimer_v2_pred_0": 0.8072626282045929
	2.2.4
		"model_1_multimer_v2_pred_0": 0.7638937624852511,
		"model_2_multimer_v2_pred_0": 0.7730047759497934,
		"model_3_multimer_v2_pred_0": 0.8002452325904037,
		"model_4_multimer_v2_pred_0": 0.7938463939736111,
		"model_5_multimer_v2_pred_0": 0.7926498563280233

Hi thanks very much for this. I will investigate and get back to you.

Hi again, I'm not seeing the same performance issues on the fasta sequence you provided. Below are the results I have obtained:
2.2.3
        "model_1_multimer_v2_pred_0": 0.7026557510461339,
        "model_2_multimer_v2_pred_0": 0.4509017402688641,
        "model_3_multimer_v2_pred_0": 0.380578570853769,
        "model_4_multimer_v2_pred_0": 0.6910764110918439,
        "model_5_multimer_v2_pred_0": 0.7656100030107779

2.2.4
        "model_1_multimer_v2_pred_0": 0.6930976486128585,
        "model_2_multimer_v2_pred_0": 0.840286483893667,
        "model_3_multimer_v2_pred_0": 0.36765421415043464,
        "model_4_multimer_v2_pred_0": 0.6591293362123893,
        "model_5_multimer_v2_pred_0": 0.8106014168732179

What hardware are you using?

Hi, both jobs were run on PCs running Ubuntu 20.04, one with a Quadro RTX 5000 (2.2.3) and the other with a GeForce RTX 2070 (2.2.4).

Hi thanks for this. Would it be possible to reverse the experiment?
Also, what cuda versions are on these machines? (run nvidia-smi)

2.2.4 (2022-09-22) with data freshly re-downloaded on DGX A100 (driver 515.65.01) singularity image from docker docker://ghcr.io/truatpasteurdotfr/alphafold:main and ealier version with 2.2.3 (2022-09-13)
}tru@myrdal:~/alphafold$ head -n 8 alphafold-2022-09-13-1905-data-2.2.4-20220923-num_multimer_predictions_per_model_1-IALsG/issue-587/ranking_debug.json
{
    "iptm+ptm": {
        "model_1_multimer_v2_pred_0": 0.8568935778701908,
        "model_2_multimer_v2_pred_0": 0.8499162052690372,
        "model_3_multimer_v2_pred_0": 0.2961016070831388,
        "model_4_multimer_v2_pred_0": 0.8552748738237211,
        "model_5_multimer_v2_pred_0": 0.851201296704003
    },
tru@myrdal:~/alphafold$ head -n 8 alphafold-2022-09-22-2049-data-2.2.4-20220923-num_multimer_predictions_per_model_1-OCW7T/issue-587/ranking_debug.json
{
    "iptm+ptm": {
        "model_1_multimer_v2_pred_0": 0.8550884037892792,
        "model_2_multimer_v2_pred_0": 0.39117327804768387,
        "model_3_multimer_v2_pred_0": 0.6731524796823938,
        "model_4_multimer_v2_pred_0": 0.8525504678600079,
        "model_5_multimer_v2_pred_0": 0.8554723246526093
    },

On ColabFold, we've been using the latest version of jax. One user reported getting different results depending on which GPU they got. They report that A100 gives different result compared to V100 or T4.
I wonder if this is because of the latest version of jax used? Where old jax gave same result across GPUs, while new jax gives different results depending on GPU?

...good point @sokrypton : my first comment would've exactly been that I was not aware that the type of GPU that one used made a difference to the actual quality of an inference - other than of course by having enough VRAM to run the corresponding job to start with. But if it does, then this would be a quite crucial piece of information.
@Htomlinson14 : yes of course I can reverse the experiment. But since one of the machines is currently running other jobs using 2.2.3, is it fine if on that one I just create a new conda environment to build the 2.2.4 docker image and then running the reverse test, or could the slightly different setup introduce further issues?

Thanks for raising this. For now, if you are using an affected GPU we recommend pinning to previous versions of the repo. We will investigate the issue further.

OK sure, looking forward to hearing what you find...

Sorry I couldn't follow. Which GPUs are affected now?

I'm beginning to suspect the old jax library was more consistent across GPUs, while the new jax library is not.
Unfortunately, I don't have access to any high-end GPUs (like A100), so I can't do any tests on my side.
I think one informative test to do would be to do a run on 2.2.3 with A100 and non-A100, then repeat the same experiment with 2.2.4 on A100 and non-A100.

Same here, but if DeepMind/Alphabet wants to indefinitely lend us one to test out things I will not say no! ;-)

I think DeepMind/Alphabet only uses TPUs :P

It's OK, I'll take anything!!

I will be trying on a dual A40 server where I have enough free space to download data, not sure I can do that on my workstation with an RTX 2080...

A40 (48GB), Driver Version: 510.54
    "iptm+ptm": {
        "model_1_multimer_v2_pred_0": 0.8501128109383987,
        "model_2_multimer_v2_pred_0": 0.8540356829322757,
        "model_3_multimer_v2_pred_0": 0.6771353362331958,
        "model_4_multimer_v2_pred_0": 0.8452873344090073,
        "model_5_multimer_v2_pred_0": 0.8578954696460785
    },

RTX-2080ti (11GB), Driver Version: 515.65.01
    "iptm+ptm": {
        "model_1_multimer_v2_pred_0": 0.8535170484093717,
        "model_2_multimer_v2_pred_0": 0.8481280255454735,
        "model_3_multimer_v2_pred_0": 0.7481300076762746,
        "model_4_multimer_v2_pred_0": 0.8371862945478382,
        "model_5_multimer_v2_pred_0": 0.8328140753338644
    },

I will try to get access on a V100 and rtx1080 machine.

@truatpasteurdotfr can you repeat this but with 2.2.3 (to see if the outputs are more consistent between GPUs with the original pinned version of jax).

In case it is useful, here's results for the test posted in this thread from v2.2.4 + cray + A100 + nvidia kernel 510.85.02 cudatoolkit 11.7.0 + charliecloud in place of docker + databases downloaded on Sept 23, 2022 + removed the 4 offending entries from  pdb_seqres/pdb_seqres.txt that match ^CT05
{
"iptm+ptm": {
"model_1_multimer_v2_pred_0": 0.8440023053565382,
"model_2_multimer_v2_pred_0": 0.8485773361074703,
"model_3_multimer_v2_pred_0": 0.26827771102715653,
"model_4_multimer_v2_pred_0": 0.8483706916239296,
"model_5_multimer_v2_pred_0": 0.8648632996723908
},
"order": [
"model_5_multimer_v2_pred_0",
"model_2_multimer_v2_pred_0",
"model_4_multimer_v2_pred_0",
"model_1_multimer_v2_pred_0",
"model_3_multimer_v2_pred_0"
]
}

rtx 1080ti (Driver Version: 510.54 )out of memory, as it can not expand into the cpu RAM,,,
2022-10-01 07:07:58.693069: W external/org_tensorflow/tensorflow/compiler/xla/service/platform_util.cc:190] unable to create StreamExecutor for CUDA:0: failed initializing StreamExecutor for CUDA device ordinal 0: INTERNAL: failed call to cuDevicePrimaryCtxRetain: CUDA_ERROR_OUT_OF_MEMORY: out of memory; total memory reported:<PHONE_NUMBER>6

Hi all. We have run AF multimer on several different GPUs and are not seeing any significant differences between predictions. @lucajovine would you mind checking to see if there are any differences in the MSAs or templates, on the two machines you are using? Thanks!

@Htomlinson14 There should not have been, because the databases used by one of them were a clone of those of the other...

Thanks -- sounds like its not a data issue. Its been difficult to reproduce internally and we have tried on several different GPUs. We will continue to look into this internally but until then we would recommend using v2.2.3, which has no difference in features. It seems likely that jax numerics have changed between jax versions on your hardware. One way to check this would be to change the jax and jaxlib versions, rebuild the docker and rerun the model. One could do this at various jax versions to diagnose whether this is the issue.

OK thanks, will stick to 2.2.3 for now and - compatibly with other people's work on the same machine - keep in mind the check you suggested.

Hi is this still an issue with 2.3.1? We have made a number of changes

Hello @Htomlinson14, sorry for the late reply. I've now re-run this on 3 machines with different GPUs and I'd say that the issue is gone with 2.3.1 - see below (values are iptm+ptm). What do you think?
AF 2.3.1 multimer 1 test
machine 1 [NVIDIA GeForce GTX 1080 Ti]
[ 0,8077794390534917; 0,8215402620648127; 0,8433008957205687; 0,8105156284710106; 0,8079775024438258 ]
avg 0,8182227456
machine 2 [NVIDIA RTX A6000]
[ 0,7947834937227392; 0,8243430424260504; 0,8369237693307853; 0,7981287211285648; 0,8089823434610024 ]
avg 0,812632274
machine 3 [NVIDIA GeForce RTX 2070]
[ 0,8108233109051757; 0,8399800232202002; 0,8444787084706873; 0,8045022435563687; 0,793217238278859 ]
avg 0,8186003049
AF 2.3.1 multimer 2 test
machine 1 [NVIDIA GeForce GTX 1080 Ti]
[ 0,8516345799449417; 0,7958957889741578; 0,8747316035396051; 0,8482859347143514; 0,8390342197106722 ]
avg = 0,8419164254
machine 2 [NVIDIA RTX A6000]
[ 0,8473456341287774; 0,69242537109294; 0,8699402519080393; 0,8541400899908609; 0,8263367607321763 ]
avg = 0,8180376216
machine 3 [NVIDIA GeForce RTX 2070]
[ 0,7522403806380576; 0,7274785839292366; 0,8527659143674804; 0,8318865914332902; 0,836440904081855 ]
avg = 0,8001624749

Hi @lucajovine. Thanks very much for sharing this. These look very consistent to me so I would agree. I will close this issue but please reopen if the issue arises again. Best -- Hamish


Desktop Edge 2150 relnotes

Desktop Edge <IP_ADDRESS> release notes

Deploy preview for docsdocker ready!
Built with commit 6ea32819d44bc7260d727018564660353dcdd8d7
https://deploy-preview-9796--docsdocker.netlify.com


auto escape string in exec and other task

RegExp for escaping https://github.com/xxorax/node-shell-escape

@pstadler why? if i you this code, i get error.
remote.exec('mysql -uroot -proot -e "SELECT * FROM `table`"');

Error throw by reason of use "`"

I don't get it. Just escape your commands as if you would run them manually.

@pstadler may be add option to auto escaping command. I do not propose to use this module, just gave an example.


Running Upload script error

Hi,
I am trying run the upload test data script. But It seems that running it results in an error.
Error screenshot:

Is there any issue with test data?
Is it any version mismatch issue?

Hi @safdar-shahir,
are you trying to upload the data to the full-irs local deployment? In this case the most recent state of main is not working, due to a lot of changes to the EDC- and DigitalTwinRegistry-API recently.
Try checking out tag 2.6.1.
Let me know if this could resolve your issue

Hi @safdar-shahir,
since there weren't any updates on this issue, I assume your problems could be resolved and this issue can be closed.
Feel free to reopen it, if you still need help


Ensure that WindowsAppSDK-VersionInfo.cs is added to compile items before XamlPreCompile, to remove dupe

A bit of a hodgepodge PR:
Update to WASDK 1.4
Work around menu theme regression with 1.4
Eliminate WindowsAppSDK-VersionInfo.cs duplicate source warning
Remove bogus patch number
Support self-contained
Add publish profiles
Disable debug publish optimizations (inner loop)
Remove wapproj cruft

See also:
https://github.com/microsoft/WindowsAppSDK/pull/3819

Note: build failing because in Microsoft.WindowsAppSDK.DeploymentManager.CS.targets, the target GenerateDeploymentManagerCS renamed to GenerateBootstrapCS between 1.3 and 1.4.  When 1.4 is released, I'll update this PR to use it, which should resolve the build issue.

Note: build failing because in Microsoft.WindowsAppSDK.DeploymentManager.CS.targets, the target GenerateDeploymentManagerCS renamed to GenerateBootstrapCS between 1.3 and 1.4. When 1.4 is released, I'll update this PR to use it, which should resolve the build issue.

We should be sure to relnote this. We should also provide a less brittle way of doing this (or move this into our props/targets so that every app who wants the version number can get it easily).

good idea - updated the related WinAppSDK PR to simply use a property for including version info exactly once, before any compilation


Fixed fastlane spelling

fastlane^TM

 coverage/coveralls — Coverage decreased (-6.09%) to 66.045%

Oh well, sorry about that @neonichu, how can I fix that? :hear_no_evil:

lol

:scream:

No merge @neonichu? :disappointed:

@KrauseFx need to investigate what's going on with coveralls first :)

:cry:

:rocket:

Ya tu sabes


Remove Compass

Since Compass is essentially a dead project (Compass/compass#1999) and will not be supported in Sass 4.0 (sass/sass#1941)

[x] add deprecation/warning message to the log when using Compass
[x] remove/rewrite Compass tests
[x] remove (support for) Compass

Also we have support for Bourbon (#8) now to do most things Compass does (except the spriting)

Compass was removed in #139


bug: add missing ? operators in methods and unwraps in tests

Pull Request type

Please check the type of change your PR introduces:

[x] Bugfix

What is the current behavior?

Related: #141
What is the new behavior?

Linked PR missed to add this operator during refactoring, so fixed it.
Without this it won't propagate stackoverflow error and ignore it silently.
Inside tests call unwrap on methods that can fail

due to this unwraps needed to increase gas for few tests

Does this introduce a breaking change?

[x] No

Added changes to CHANGELOG.md

[x] No

Other information

(i don't think changelog is really necessary here, because its doesn't change behaviour of the program. But I can add it if you want)

this bug is kinda subtle to spot, fortunately I worked on the PR that this refactors so i was able to spot it.
Opened an issue in cairo: https://github.com/starkware-libs/cairo/issues/3960

(added ? to lot more places, and called unwrap on methods that return result in tests)


Mac/iPad: dashboard

@samuelclay wrote in a comment in #1247:

I've been using the macOS app a ton and it's been a great experience. One change I'd like to see is to match the up-to-date experience from the web. We have a configurable dashboard on the web that has real-time updated "widgets". Could we make a new "Dashboard" top level folder, same as on the web, with the story titles in groups matching the web? I publish those lists on /reader/feeds/:
"dashboard_rivers": [
    {
        "river_id": "river:",
        "river_side": "left",
        "river_order": 0
    },
    {
        "river_id": "river:infrequent",
        "river_side": "left",
        "river_order": 1
    },
    {
        "river_id": "river:global",
        "river_side": "left",
        "river_order": 2
    }
],

They should show 5 stories in the standard story titles format, and they should reload every 5 minutes (add a random 1-5 minutes per request so they don't all refresh together at once, some people can have dozens of rivers). I'd like to see this on both iPad and macOS, so I can leave the screen open and watch the news come in.
Bonus points for left vs right lists. And extra bonus points for single column, double column, and two horizontal columns (left "column" on top row, right "column" on bottom row).

I like this idea. Just the three river feeds, or are the What's New and NewsBlur Stats etc panes available too, to fully replicate the experience? It'd be a nice feature either way.

We don't have APIs for what's new and statistics, so let's start with the rivers. Check dashboard_rivers_view.js for details on which feeds to pull and what to hide.

When the macOS/iOS app calls /reader/feeds, there is no dashboard_rivers data, due to using the flat=true parameter. Changing that does include it, but breaks loading, so can you include it for when flat=true?
Also, it'd be nice to be able to customize the dashboard in the app; is there an API to change, add, or remove a feed in the dashboard? Being able to send back the JSON for the dashboard_rivers would be ideal, so I could implement moving feeds between sides, or reordering them.
By the way, I noticed some issues with the Dashboard on the web. Adding several feeds on the same row (in horizontal layout mode) can get unusably narrow; perhaps should limit that, though not unreasonable to let the user self-limit that. But changing the feed for a "widget" can result in the old feed appearing with the new title. I could add issues for those if they're not known issues.

Deployed dashboard_rivers on /reader/feeds?flat=true

To save dashboard rivers, POST to /reader/save_dashboard_rivers/ with the following form data:

river_id
river_side
river_order

You'll receive back a new dashboard_rivers that contains the entire set of rivers.

@sictiru You'll need these URLs and params, too. ☝

Thanks! I would have preferred a save endpoint that took the entire array of rivers, so I could implement moving feeds. With the single river save (and the web UI), you can only move a feed by changing an existing dashboard item's feed; if I had several feeds in the dashboard, and wanted to move one to the top, I'd have to do several changes. But a good start.

Work on this is underway, but I'm out of time for this month, so I'll finish it next week. I've committed the current progress on the dejal-alpha branch, but don't bother playing with it yet; there isn't much to see currently.

A NB Alpha TestFlight is available, but note that there's still a bunch of work to do on the Dashboard feature, so don't bother reporting any issues yet.


Improve editor property capitalization

Follow-up to #68455

Captialize stop words when they are the last word. e.g. List Script Names As.
Add stop words logic in extract.py.

Thanks!

Cherry-picked for 3.6.


fix: remove old faq topic per martin's request

This PR removes the old FAQ per @martinbuergi's request.

FYI, #73 populates that page, relocates to the troubleshooting section, and adds a redirect.

Got it. I'll make sure to note that for you. Thanks!


Add search to giftcard query

Issue: https://github.com/saleor/saleor/issues/13082
I want to merge this change, because I want to add full text search to giftCards query.

Impact

[x] New migrations
[x] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are either absent or guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migrations are either absent or optimized for zero downtime
[ ] The changes are covered by test cases

I've two gift cards:
first created in the dashboard and then used by customer in checkout
second user bought product gift card via checkout
{
  "data": {
    "giftCards": {
      "totalCount": 2,
      "edges": [
        {
          "node": {
            "id": "R2lmdENhcmQ6Mg==",
            "displayCode": "341A",
            "created": "2023-06-30T09:47:09.971152+00:00",
            "createdByEmail"<EMAIL_ADDRESS>            "createdBy": {
              "__typename": "User",
              "email"<EMAIL_ADDRESS>            },
            "usedByEmail"<EMAIL_ADDRESS>            "boughtInChannel": null,
            "lastUsedOn": "2023-06-30T10:12:35.269088+00:00"
          }
        },
        {
          "node": {
            "id": "R2lmdENhcmQ6Mw==",
            "displayCode": "1F52",
            "created": "2023-06-30T10:31:03.375165+00:00",
            "createdByEmail"<EMAIL_ADDRESS>            "createdBy": null,
            "usedByEmail": null,
            "boughtInChannel": "default-channel",
            "lastUsedOn": null
          }
        }
      ]
    }
  }
}

I tried to use search and find gift cards by code or email but got no results

Hmm maybe it's a matter of this test env- I think it doesn't have Cron that would reindex search vectors. Not sure how we tested that with other searches. But maybe it's something wrong with the code as well.

Hmm maybe it's a matter of this test env- I think it doesn't have Cron that would reindex search vectors. Not sure how we tested that with other searches. But maybe it's something wrong with the code as well.

ok, I will check it on local env


Map Styling for android

This pull request is to separate the styling into a separate pull request from #670.
I don't have access to a mac now so I'll create another pull request for ios by monday. But this code should compile normally on ios, but the map won't be styled.

ref #781


Bump libc version.

r? @kamalmarhubi

@homu r+

:pushpin: Commit ab1ffe1 has been approved by utkarshkukreti

:zap: Test exempted - status


add bundles support

I was made some experiments, and saw > 100 networt requests on my dist app loading.
Can you provide more info, how we can use bundles with jspm + aurelia, I think we need it

Yes, we need it ;) We are working on the bundling story right now. We will have a gulp task which will do it for you. That's not quite done yet. We can track that here.

Ok, I will also try to create that task (if I have time), thank you

:+1:

I've added some instructions to a blog post here - http://patrickwalters.net/my-best-practices-for-aurelia-bundling-and-minification/ - that should help implement bundling and minification (it is fairly straight-forward)
Until we get it pulled in to the CLI and added to the skeleton this may help.

@ahmedshuhel I've pushed the runtime support for this based on the bundle sample you sent me. I need a blog post from you that we an publish to show the community how to set up bundling for both js and html on their project.

@EisenbergEffect I am working with it.

@EisenbergEffect we may close this issue now.


-d  does not work with tab

using a custom delimiter I am getting this:
pgfutter --dbname tester --host mydb --port 32063 --username tester csv -d "\t"  ~/sc/data/jobs.csv
Please specify the correct delimiter with -d.
Header column contains a delimiter character: job_db_inx	account	cpus_req	exit_code	job_name	id_job	id_user	nodelist	priority	state	partition	timelimit	time_submit	time_eligible	time_start	time_end	time_suspended

I think this problem already explained in the documentation:

If you want to use tab as delimiter you need to pass $'\t' as delimiter to ensure your shell does not swallow the correct delimiter.
pgfutter csv -d $'\t' traffic_violations.csv

doesn't work even with the -d $'\t'.  on Windows 10 Pro.  Thanks.


warning: is expected to have version >=2.3.1, but has ;

After latest upgrade I'm getting this:
warning: resource plugin azure is expected to have version >=2.3.1, but has ; the wrong version may be on your path, or this may be a bug in the plugin

Here is what I have:
yarn list --pattern azure
yarn list v1.22.0
warning package.json: No license field
warning tms-pulumi: No license field
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>✨  Done in 0.22s.

I also upgraded azure-cli to 2.3.0 which seems to be the latest available on chocolatey

Also with version >=2.4.0-alpha.158559637

Hi both
I am sorry that you are getting this error - I am looking for a fix for this right now
Paul

Luckily it is only a warning ... things still work 😄

yeah I can see what the issue is :/ Since we moved to the new go modules structure the binary isn't picking up the version. We will have a fix published this afternoon :)

I'm also getting this error, even asked in the Pulumi slack but nobody could come up with a solution.

@stack72 how&when will the fix be available?
thanks!

Hi all
I’m currently working to upgrade the provider to the latest codegen so a fix is on its way
I got caught up preparing for the v3.0.0 beta of Pulumi-azure yesterday
Paul

Folks, I believe this is solved with https://github.com/pulumi/pulumi-azure/pull/510
I was able to build the binary and copy it to my local plugins directory and there were no warnings around the version
When this gets released, can someone test this for me?

sure, waiting for the release

Reopening until this is verified

@swoop-alex the release has been made - please can you check you are in a better situation? 🙏

I did
pulumi plugin install resource azure v2.4.0
then I did update in the package.json  "@pulumi/azure": "^2.4.0"
and then run pulumi apply
I still get "warning: resource plugin azure is expected to have version >=2.4.0, but has ; the wrong version may be on your path, or this may be a bug in the plugin"
after that I checked the stack json file and also there the new version is to be seen.
Am I missing something? How should I test this?

Ok, I actually did find the error!

https://github.com/pulumi/pulumi-azure/pull/511


add bootstrap target when running make/make all

For better DX.

Looks like a CircleCI. Requesting a re-run :).

We intentionally did not add make bootstrap to the default make target because it's a one-off target. It doesn't need to be called every time you rebuild the client.
In dev-v3 we added it there because we have smarter detection for when make bootstrap was called and dependencies an update. If we had the same thing in place here then I'd be open to consist this.

@bacongobbler I had the same thinking.
I thought about this a bit more and a there are a number of usability errors caused by not calling make bootstrap build. I then thought the user will not want it to be called all the time but for this they could call make build. I suppose doing the bootstrap each time for a make call is inconvenient but does it balance out in favour of  dependency issues?

I agree with @hickeyma on this. By instinct I would call make or make all when I run a project for the first time. Any subsequent rebuilds would be done by a make build. That's pretty much the convention if I am not mistaken.