Revert "Remove redundant build tags (#833)"

This reverts commit 53c922ce9bc7a92a2d90c9bec8e1ed07a2fd23a6.
See https://github.com/cs3org/reva/pull/833#issuecomment-647361362
Fixes compilation issues on Windows in ocis-reva
@labkode @ishank011

I'm not sure if this is sufficient, https://github.com/cs3org/reva/pull/829 might also contain changes that need to be looked at for a better separation of platforms

@PVince81 #829 just adds one more layer of abstraction at the fs level. For platform-wise functionality, no changes have been made there. It's just weird in this case that the platform-specific files were not identified.

Fantastic, it builds again on Windows: https://cloud.drone.io/owncloud/ocis-reva/493/8/3
Yeah, weird that it doesn't work. Will need further research to be able to remove those tags.


Add different weather icons for night time

The weather icons currently display sun even after the sun sets. The sun state needs to be added and the respective night-time weather icons implemented.

Done.


URL issue with Advanced WP Columns plugin

Submit a feature request or bug report

[x] I've read the guidelines for Contributing to Roots Projects
[ ] This is a feature request
[x] This is a bug report
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and followed them (if applicable)

Replace any X with your information.

What is the current behavior?
Advanced WP Columns iframe-popup in text edtior opens /wp-content/plugins/advanced-wp-columns
which result in HTTP 404:
NOT FOUND

Sorry, but the page you were trying to view does not exist.

What is the expected or desired behavior?
Advanced WP Columns irame-popup should load correctly the columns editor interface.

Bug report
Please provide steps to reproduce, including full log output:

Install Advanced WP Columns plugin.
Activate Advanced WP Columns plugin.
Open a page in editor.
Select Advanced WP Columns button.

Please describe your local environment:
Bedrock version: Bedrock 1.7.3
WordPress version: WordPress 4.7
PHP version: PHP 7.1
OS: Debian Jessie amd64
Where did the bug happen? Development or remote servers?
Development (docker/docker-compose).

Other relevant information:
In Bedrock, plugins are installed to app/plugins (where also the folder 'advanced-wp-columns' resides).
The Advanced WP Columns plugin seems to have the default WordPress path hardcoded instead as it opens the iframe popup pointing to it using this default path instead the new one used in Bedrock installations.
The correct url would be /app/plugins/advanced-wp-columns/assets/js/plugins/views/columns.html instead of /wp-content/plugins/advanced-wp-columns/assets/js/plugins/views/columns.html.

The Advanced WP Columns plugin seems to have the default WordPress path hardcoded instead as it opens the iframe popup pointing to it using this default path instead the new one used in Bedrock installations.

There's nothing we can do about this on our side. This is an issue with that plugin itself.

@austinpray: But this would be fixable in plugin by looking up the plugins path
instead having it hardcoded as it is now?

Yes it's fixable by the plugin, not by us. They are doing things the wrong way.


HTTP Public Key Pinning (HPKP) support

Add checks for the HPKP.
More info:
https://en.wikipedia.org/wiki/HTTP_Public_Key_Pinning

Will do when I get a chance, but feel free to send a PR if you want to do it yourself.
Cheers,


Remove dyn from allocator

'static requirement could be probably lifted from allocator but after fighting for some time with borrow checker I gave up :disappointed:

Added more content, now dispatch to allocator is done outside of frame_buffer. Also included some changes in radio.rs regarding testability.

Split phy rework to separate PR


[Fix] Osc manager portgroup registration

New port group was not registered in the dictionary.
This was preventing multiple OSCInput instances to use the same port for different instances.
Fixes : #87

Seems it messed up with my fork again, gonna re fork and do it again

hi vux.
any progress with this? There was quite a lot of refactoring on the dev branch last week. I hope this doesn't cause too many merge issues... :(

That was fixed by other PR, that's why I closed this one


Should be able to remove server from cluster

If a node goes down permanently you should be able to remove the node from the cluster. Should remove it from cluster config and remove it from any shards.

closing in preference for #1471 (such a good idea Paul keeps having it!)


feat: port Data.Finset.Pimage

Currently blocked on missing port of pfun

There were some fixes with decidable necessary, perhaps not exactly how it was done now

Currently the formulation of the statement pimage_eq_image_filter is still broken, any hints welcome!


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d2bmi4pok4x5k7
Region: us-east-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

Ay,
Does your custom domain is already in use in a CloudFront distribution ? I got the same problem because one of my CloudFront has CNAME in it and when I try to add custom domain on Amplify I got your error.

I was using CloudFront but I deleted them to use Amplify but still getting this error

Ay,
Does your domain is registered in AWS Cert Manager ? I already got an error with a domain already used on Cert Manager too.

https://docs.aws.amazon.com/amplify/latest/userguide/custom-domains.html#cnamealreadyexistsexception-error
https://aws.amazon.com/premiumsupport/knowledge-center/resolve-cnamealreadyexists-error/
These guides are helpful for getting started with custom domains, for existing domains there are number of steps that will need to be completed (as outlined in the docs above) depending on your current hosting and DNS providers.
A CNAME alias (mydomain.com, sub.mydomain.com) can only be associated with a single CloudFront distribution at a time, if you receive this error then the domain is already associated with another CF distribution (either within the same AWS account, or potentially in a different account) and must be disassociated from the previous distribution before the distribution created for you by Amplify Console will work, the docs help outline how to do this (and you may need to check more than one account if you or your organization owns multiple accounts)
Additionally, if you manage your domain through Route53, make sure to clean up any hosted zone CNAME or ALIAS records that point to the old distribution. After completing the above, I would remove the domain from Amplify Console and start the flow over.

Closing this issue, please feel free to re-open if you are still having issues.


Road to deploy app failure

Hola!
thank you for integrate alfresco-js-api with IONIC, it's awesome! But I can't do it work :( .
I would like to share with you my road to try start the project, and you could add documentation in your readme.md.

First of all, when I try to install packages and launch "ionic serve" get this error in gulpfile:
There is an error in your gulpfile:  SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode  at exports.runInThisContext (vm.js:53:16)  at Module._compile (module.js:413:25)  at Object.Module._extensions..js (module.js:452:10)  at Module.load (module.js:355:32)  at Function.Module._load (module.js:310:12)  at Module.require (module.js:365:17)  at require (module.js:384:17)  at Object.<anonymous> (/Users/alberto/CordovaDev/ionic-alfresco-app/gulpfile.js:8:25)  at Module._compile (module.js:434:26)  at Object.Module._extensions..js (module.js:452:10)
module "@DenisVuyka/ionic-gulp-build-typescript" file index.js should prepend "use strict";

alfresco-js-api module works on alfresco 5.2, so you should warn in your readme.md in order to get it working. Latest EarlyAccess versions "[https://wiki.alfresco.com/wiki/Community_file_list_201606-EA]

Using postman I can do login, but using the app from Chrome Browser or Hybrid App, I get 403 error:
XMLHttpRequest cannot load http://localhost:8088/alfresco/api/-default-/public/authentication/versions/1/tickets. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8100' is therefore not allowed access. The response had HTTP status code 403.
I test it with other computer (same network) alfresco instance, without luck.
Could you please provide the way to avoid CORS in the app using alfresco local instance?
Kind Regards,
Alberto

Hello Alberto,
There are several prerequisites I had to mention in the readme, thanks for pointing out.

You need ionic 2 (beta)
As per Angular 2 dependency there's a node.js constraint

Verify that you are running at least node v4.x.x and npm 3.x.x by running node -v and npm -v in a > terminal/console window. Older versions produce errors.

I am using node v6.5.0 but may setup Travis later on to check earlier versions as well.

alfresco-js-api indeed requires alfresco 5.2
You can get more details on CORS within our Angular 2 components docs, [prerequisites page] (https://github.com/Alfresco/alfresco-ng2-components/blob/master/PREREQUISITES.md)

This project is still at very early stages so more content and details to be added,
Regards,
Denys

Thanks! Works like a charm!


Change exception thrown when realm not found for a given name and include error code in exception thrown.

Purpose
This PR changes the exception thrown at the AbstractUserStoreManager when a user store is not found for a given tenant domain name, to use ERROR_CODE_INVALID_DOMAIN_NAME constant and use an error code. This helps identify the thrown exception at a higher level.
Related Issue

https://github.com/wso2/product-is/issues/19181

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7803764660

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7803764660
Status: success

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7884298895

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7884298895
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/8277965419

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8277965419
Status: success


Enable PR notifications when Travis fails

Fixes https://github.com/prestosql/presto/issues/1900

This looks cool. Is there an example PR for a Java / Maven project?
Or can you add a dummy commit here with some test failures?

The build failed but @TravisBuddy didn't comment here.
I suspect Travis might be ignoring notification settings change in PR.
Example where it works

failure https://github.com/luna-rs/luna/pull/163#issuecomment-543735945
success https://github.com/luna-rs/luna/pull/166#issuecomment-545578574

i suggest that we merge this and see


RF-gate doesn't work without RF source in digi

Per Hes, an I-gate only config doesn't add heard stations to the historydb, so implicit IS-to-RF gated packets get dropped as the stations haven't been "heard" on the RF interface.
http://pastebin.com/Mypz0ndR

The pastebin has an expiration. Here it is in non-expiring form:
netax25rx packet len=42 from rx_socket;  family=17 protocol=200 ifindex=4 hatype=3 pkttype=0 halen=0
Received frame of 42 bytes from ax0: OH2RDK-2
erlang_add(OH2RDK-2, RX, 52, 1)
ax25_format_to_tnc() len=41 82 a0 96 60 60 66 60 9e|90 6e 98 b4 84 f0 ae 92|88 8a 64 40 63 03 f0 3a|55 54 46 2d 38 20 20 20|20 3a 74 65 73 74 7b 35|0d  = APK0030O|H7LZBxWI|DE2 c..:|UTF-8   | :test{5|.
13437298        OH2RDK-2  R     OH7LZB-8>APK003,WIDE2-1::UTF-8    :test{5
interface_receive_ax25() no receivers for source OH2RDK-2
telemetry_postpoll()  telemetrytime=1141s  labeltime=61s
telemetry_prepoll()
beacon_postpoll()
.. igate from aprsis
interface_receive_3rdparty() aif=0x8070300, aif->digicount=1
pbuf_alloc(31,26) -> 0xb7740000
.. parse_aprs() rc=OK  type=0x8008 srcif=APRSIS tnc2addr='UTF-8>APRS::OH7LZB-8 :ack5'  info_start=':OH7LZB-8 :ack5'
pbuf_free(0xb7740000)
historydb_lookup('OH7LZB-8 ') -> i=40 .. no match
.. parse_aprs() rc=OK  type=0x8008 srcif=APRSIS tnc2addr='UTF-8>APRS::OH7LZB-8 :ack5'  info_start=':OH7LZB-8 :ack5'
## produce 3rd-party AX.25 frames for transmit, and original source one for filtering:
ax25hdr 82 a0 a4 a6 40 40 60 9e|90 64 a4 88 96 65  = APRS  0O|H2RDKe
Formatted AX.25: OH2RDK-2>APRS:}UTF-8>APRS,TCPIP,OH2RDK-2*::OH7LZB-8 :ack5
 filter tnc2addr = UTF-8>APRS,TCPIP*,qAC,OH7LZB-UT
pbuf_alloc(59,75) -> 0xb7740000
historydb_lookup('OH7LZB-8 ') -> i=40 .. no match
.. parse_aprs() rc=OK  type=0x08 srcif=APRSIS tnc2addr='UTF-8>APRS,TCPIP*,qAC,OH7LZB-UT:}UTF-8>APRS,TCPIP,OH2RDK-2*::OH7LZB-8 :ack5'  info_start='}UTF-8>APRS,TCPIP,OH2RDK-2*::OH7LZB-8 :ack5'
## process source filter
 key='UTF-8' hash=15 .. inserting new history entry.
historydb_insert_heard(APRSIS) v=0xb7585000

Fixed in commit f38cd0652afc11a08662568068f4965f01ce25cd


Folder icon in front of environment name in drop-down disappears when selected

Version - Master with reskin UI
Steps:

Create an additional environment
Click on Environments drop-down on upper-right (notice the folder icon besides newly created environment)
Click on Environment to select it.

Results:
The folder icon disappears.
Expected:
Since the little home icon stays when you select that particular environment, seems like the folder icon should as well.

Wrong icon name when showing the not-default environment.

Verified fixed - Master with newest reskin


OWLS-69370: Introspect: Domain-in-Image: Add data home override support

Provide ability to centrally override default file store, and custom file stores for both domain-home-in-image and domain-on-PV installations.
Justification:

Aligns data storage with log storage
Makes it easy to centralize data storage

Implementation:

Add dataHome to Domain CR

If not set, default to overiding as /shared/data/[domain-uid]

If setting is "", means 'don't override'.

If setting is anything else, use this specific value.

Pass through as a well known env var to introspector and to 'custom sit-config'

Have introspector override location via sit-config

Overrides both default and custom file store 'Directory' attribute

Why is there a 'docs/charts/weblogic-operator-2.3.0.tgz' in the diff? Is this expected now?

It isn't strictly needed, but I think it'd be helpful that if the the dataHome is set, then the introspector and wl-start bash scripts should try and create this directory via a 'mkdir -p', and the script should fail if the command fails.  This is user-friendly as it will cause the job/pod to 'fail fast' before WebLogic itself attempts to create the directory and fails at runtime.

I'm not familiar with web-apps, but it looks like they can persist session state to files. I'm not sure, but it looks like this is configured on weblogic.management.configuration.WebAppComponentMBean /  SessionPersistentStoreDir and on WebServiceComponentBean which inherits from WebAppComponentMBean

Thanks. Let me take a  look at the WebAppComponentMBean and WebServiceComponentBean and get an understanding if we need to address this for them too.

Why is there a 'docs/charts/weblogic-operator-2.3.0.tgz' in the diff? Is this expected now?

When looking at the develop and master branches, these files looked to be committed.

Why is there a 'docs/charts/weblogic-operator-2.3.0.tgz' in the diff? Is this expected now?

When looking at the develop and master branches, these files looked to be committed.

Still looks strange to me - I don't remember seeing that before.

2nd commit incorporates changes based on the the code review comments:

Added tests for defining a dataHome with an empty ("") path.
When dataHome is not set or is "" means means 'don't override'.
bash scripts will try and create the dataHome directory (if override path provided) via a 'mkdir -p', and the script will fail (exit) if the command fails.

Still investigating the WebAppComponentMBean and WebServiceComponentBean to determine if we need to override the persistent store directory path.

Update If not set, default to overiding as /shared/data/[domain-uid] If setting is "", means 'don't override'. in the description so that it reflects the current behavior.

I should have noticed this earlier (sorry!), but it'd be good to update the documentation so that it covers dataHome.   Look for any mention of 'logHome' and twin up a similar discussion for dataHome - for example:
$ find ./docs-source -name "*.md" | xargs grep -l logHome
./docs-source/content/userguide/managing-domains/domain-lifecycle/startup.md
./docs-source/content/userguide/managing-domains/configoverrides/_index.md
./docs-source/content/userguide/managing-domains/_index.md
./docs-source/content/samples/simple/domains/domain-home-in-image/_index.md
./docs-source/content/samples/simple/domains/fmw-domain/_index.md
./docs-source/content/samples/simple/domains/domain-home-on-pv/_index.md

Apart from the missing doc, the change LGTM.

LGTM so far apart from missing WebAppComponentMBean/WebServiceComponentBean file based session state - which I assume should be included but still needs research. If this is added, the doc for dataHome throughout will need updated too...

Queried Steven Liu for verification of File based session state persistent store configuration and there isn't any way to override the persistent store directory through any mbean configuration.  Here is Steven's reply:
Hi Lenny,
There’s no configuration on MBean to specify the persistent store directory for File persistent session data.
There are different cases for how web container determins the directory:
If there’s no <persistent-store-dir> configured in session-descriptor or the <persistent-store-dir> is configured to be relative directory, the store directory will be created under ${domain_home}/servers/${server_name}/data/ directory.
If the <persistent-store-dir> is configured to be absolute directory, the configured directory will be used for persistent store.

Hope this helps!
-Steven

On Jun 20, 2019, at 9:03 AM, Lenny Phan<EMAIL_ADDRESS>wrote:
Hi Steven,
I have a couple of questions but let me give some background.  We're currently making a modification for the WebLogic K8s Operator where we are centralizing
persistent store directory location for both the default and custom file stores (OWLS-69370).  Since we're allowing the K8s Domain configuration to specify
this persistent store directory location for file stores, I'm also investigating whether this location can be used if a deployed web application specifies a
persistent store type of 'file'.  Is it possible to access the configuration of the persistent store directory through a configuration mbean and modify/update
the directory location?  I see WebAppComponentMBean/WebServiceComponentBean but don't see a way to access it when looking through the config mbean tree.
If it's not possible and configuration can only be done through the weblogic session descriptor than that's ok, I'm just verifying whether or not we can
override the persistent store directory location.
Thanks!
--Lenny

From the last review from Tom Barnes, added the ability to:

Create a symbolic link from the server's domain home 'data' directory to the centralized 'data home' directory specified by the 'dataHome' attribute in the CRD.  This allows each server's data directory to be centralized.

We provide a warning description/instructions and will fail server startup if there are files in both the  server's domain home 'data' directory and the centralized data directory (specified by 'dataHome' attribute) as we have no idea which files are important to the user's or are in conflict between the two directory locations.  As per the usual WebLogic approach to these kind of issues, we ask the user's to resolve the differences before restarting the server, if they intend to centralize their server's data storage.

Provided a container/server environment variable  ('KEEP_DEFAULT_DATA_HOME') that will  bypass the server startup failure due to data files being located in both the  server's domain home 'data' directory and the centralized data directory (specified by 'dataHome' attribute).  This allows the server to continue to startup and operate and keep its domain home 'data' directory in the default location while still allowing the override of the default file and custom file stores.

NOTE: This change does not override the WLDF diagnostics store as they are not uniquely named and so cannot be centralized, whereas the default and custom file stores are uniquely named.

Overall comments:

Unfortunately, the symbolic link support still seems fragile. Maybe turn it off via an experimental flag which can be enabled via a new env var, or, if you're ambitious, via a new experimental domain resource attribute?

Overall comments:
* dataHome should be documented in docs-source - I've posted potential locations in a previous comment

* if a line of a script can fail, this needs to cause the script to log an error and fail - I've just posted comments about two such locations

* otherwise all LGTM assuming 'ad hoc' testing is all good.

* unfortunately, the symbolic link support still seems fragile. Maybe turn it off pending future hardening JIRA (via an experimental flag which can be enabled via a new env var, or, if you're ambitious, via a new experimental domain resource attribute)?  maybe it should be enabled via a separate setting?

I agree that the symbolic link support is still a little fragile.  It's mainly contained in the startServer.sh script. We could just punt on it and go back to the original intent of just supporting file stores.  That is, separate the symbolic link feature as you previously suggested.

Looks good! I added a couple minor suggestions in case you'd like to address them.

Implemented your latest suggestions. Thank you.

Well, that was long strange trip. My apologies that the soft linking idea turned into a bit of a garden path. Here's my approval good sir.

No apologies needed.  Always learn something new, especially scripting as I hardly write scripts.  Thank you for your great reviews.


Nukie medics wont spawn properly

Description
Their outfit is referencing an invalid prototype, ClothingMaskBreathMedicalSecurity. Which was part of brigmedics I believe.
Reproduction
Try to set outfit the nukie medic onto anything, or spawn as one

StartingGearPrototype was not subject to the linter, resolved in https://github.com/space-wizards/space-station-14/pull/19307


Add adaptive syscalls lab to blog.

What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area blog
What this PR does / why we need it:
This PR updates the adaptive syscall selection blog with the lab built to try the feature out.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pmusa
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cc: @incertum

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, pmusa
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 4728730b5455f244661db28b25c1d1d556a356ef


fileArray and dataURLArray are not in the same order and it's not possible to match them up

If you drag 5 documents onto a control, the dataURLArray is generated in a non-specific order, often leading to it being in a different order to the fileArray, this is bad since the data URL contains no identifying data about the file apart from type.
As an example, if I attach 4 .docx files and a .msg to a control:
The types of the files in fileArray in the order the array is in:
[
	"application/vnd.openxmlformats-officedocument.wordprocessingml.document",
	"application/vnd.openxmlformats-officedocument.wordprocessingml.document",
	"",
	"application/vnd.openxmlformats-officedocument.wordprocessingml.document",
	"application/vnd.openxmlformats-officedocument.wordprocessingml.document"
]

The start of the dataURLArray objects:
[
	"data:application/vnd.openxmlformats-officedocument.wordprocessingml.document;base64,UEsDBBQABgAIAAAAIQCshlBXjgEAA...",
	"data:;base64,0M8R4KGxGuEAAAAAAAAAAAAAAAAAAAAAPgADAP7/CQAGAAAAAAAAAAAAAAABAAAAAgAAAAAAAAAAEAAAEgAAAAIAAAD+////AAA...",
	"data:application/vnd.openxmlformats-officedocument.wordprocessingml.document;base64,UEsDBBQABgAIAAAAIQCshlBXjgEA...",
	"data:application/vnd.openxmlformats-officedocument.wordprocessingml.document;base64,UEsDBBQABgAIAAAAIQCshlBXjgEA...",
	"data:application/vnd.openxmlformats-officedocument.wordprocessingml.document;base64,UEsDBBQABgAIAAAAIQBhd9IpggEA..."
]

As you can clearly see they're in a different order, and there is a high chance the .docx files are in the wrong order.
This is an issue for me as I am using the base64 component of the data URL to upload these files to the server with the data from the fileArray, which cannot be matched up if uploading more than one file.

In knockout-file-bindings.js, in fillDataToProperty, I made the following change to be able to match the fileArrary with the base64String Array:
                    function fillDataToProperty(result, prop) {
                        if (index == 0 && fileData[prop] && ko.isObservable(fileData[prop])) {
                            fileData[prop](result);
                        }
                        if (fileData[prop + 'Array'] && ko.isObservable(fileData[prop + 'Array'])) {
                            if (!file.id) {
                                file.id = result;
                            }
                            fileData[prop + 'Array'].push(result);
                        }
                    }

Reasonable, I ended up writing my own alternative ko binding extension for file upload and drag&drop

@safrazik , could you implement the changes into the library, and check the pull request that resolves another issue?

@DrJam , I still use this component and this still an issue for us. Is your alternative version available? Does it differ far from the original? Or do you know what to change in the knockout-file-binding.js to get this solved? I can see that the fix by michaeldeongreen above is implemented but it does not seem to be a complete solution.

@Dizzitune I’m the author of the component. It's interesting to know that the component is still used today. I already looked into this issue few years back and gave up in fear of breaking anything else that is working already, and of course, lack of time. I think I could give another shot now as I have more experience in this particular area, especially after my Vue File Agent component. I’m working on FileAgent v2 With support for React, Vue and vanilla JS.
https://github.com/safrazik/vue-file-agent
I assume using  Promise.all would be ideal to fix this issue. Are you using any Promise poly fill already?

@safrazik - I tried debugging the code myself but it was a bit difficult to find the cause of the issue. If you have time to fix it or if you could point me in the right direction it would be very appreciated :) Promise.all would not be a problem.

@Dizzitune Version 1.2.0 fixes this issue. Please test and confirm with your test cases.

@safrazik Yes! Works like a charm, thank you very much :)
It would be nice to do Vue, Blazor or something newer than 2014 sometime but some of us just have to stick to the dirty work ;)  KO Rules!

@Dizzitune I am in the process of moving from KO to Vue. I like Vue a lot better. Vue 3's reactivity is very similar to KO. KO was way ahead of its time on the reactivity aspect but way behind the times wrt components and virtual dom.
Once Vue 3 is out and vuetify ports over to Vue 3, then I would wholeheartedly suggest that you move over to Vue 3 and Vuetify.
I am finding that with KO I ended up copy and pasting stuff all over the place because of the lack of good component support. But with Vue, I am writing reusable components that I use many times. In the end it is much faster with Vue. I am converting as I go so that I no longer build KO UIs and my app is a mixture of the two.

@safrazik Yes indeed. I've worked with Vue some time and its great and of course a lot more powerful than KO. I think KO was just outstanding when it arrived and therefore we have a lot of old applications that are based on KO and in need of maintenance. Soo thats why this was relly helpful.
With KO I’ve added some features to the KO Component base and developed support for distributed system. So we have distributed KO Components that can be used like GUI-composition in other applications. At this point (a lot of coding) its quite easy to use.
To be honest I’m quite tired of Javascript today and hoping for some development including Webassembly and Blazor. We will see :)
Anyway, thanks again for your help and support.

 I only use typescript. The difference might seem small but it is really huge. I don't know how people can ship a bug free product in javascript, but with typescripts type system, and the refactoring tools that a type system provides, I feel much more confident with the quality of my front end.

Typescript is what we are using too but its still a lot of issues. Webassembly allow us to use c# native on client and server and that would be soo nice :)


Canonical/non-canonical toggle is not very visible

Solution: add a border around it

Relates to #655


Extension view height not correct when scrolling.

I just added an extension view with height is 100px, or higher, and mark at the top with setStickyExtensionView:YES ,orange view in this image: http://prntscr.com/8mo49z
but when i scroll the collection view, the height is changed, like http://prntscr.com/8mo5fa
How can i prevent this problem?
Thanks

Thanks for reporting! Can you please try the latest release? I think it's fixed:

Please note, I will update the library to 0.10.0 soon on cocoapods.


[1.x] Fix error when calling the orchestrate:provider command after generating service via the orchestrate:service command.

What does this PR do? :robot:
It re-cache's the config silently after orchestrating a new service, for the orchestrate:provider command to function properly.

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

Comparison is base (7ff6bde) 92.25% compared to head (fb1c36d) 92.27%.

Additional details and impacted files
@@             Coverage Diff              @@
##                1.x      #47      +/-   ##
============================================
+ Coverage     92.25%   92.27%   +0.01%
  Complexity      181      181
============================================
  Files            21       21
  Lines           633      634       +1
============================================
+ Hits            584      585       +1
  Misses           49       49

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


Text overlap in barchart

For some states the bar chart overlaps with the content behind I.e. confirmed, active, date and numbers.

We can fix this by increasing the height of the parent container and aligning the graph to the bottom of the parent.
Allow me to /claim this if you agree to this solution.

/Available

We need a good design solution for this

@shuklaayush We can move the name of the graph below it and have the date and number of cases in a single line on top left for eg. Apr 28 (27 -54).

@shuklaayush We can also have centre-aligned text with some decent margin-bottom below the stats div something like below

Note: In the above image the margin for stats div is missing

@kousiksatish Difficult to judge just by looking at an image. Create a PR?

@shuklaayush Please have a look at the PR above.

@shuklaayush is this issue still open? I'd like to work on it /claim

If this is still open I would like to /claim it


Inline comments (e.g., gcw) in Python

Hi,
I just discovered the plugin and getting up to speed. I've seen that with some languages, the plugin is smart enough to comment out an argument in a function call:
-- from this
vim.keymap.set('n', '<space>e', vim.diagnostic.open_float, opts)
-- to this
vim.keymap.set('n',--[[  '<space>e' ]], vim.diagnostic.open_float, opts)

In Python, however, the plugin doesn't return a valid syntax if we comment out an argument.
# from this
def my_function(a, b):
    return b
# to this
def my_function( #a, b):
    return b

I would have expected it to return something like this...
def my_function(b):  #a):
    return b

I'm assuming this is because Python doesn't have inline comments, but I was wondering if there was a way to achieve something similar to what I described. It would be particularly useful when refactoring code.

Yes, this is expected because python doesn't have block commentstring. And I am not sure, how is this achievable? Maybe using treesitter? But that being said, this is certainly out of scope for the plugin to support this OOTB.


Full support OKATO

The current version doesn't support OKATO code

https://github.com/sshaplygin/docs-code/pull/22


Correct spain INE CCAA on communities

See Issue #309 for explainations
Signed-off-by: Jimmy ESCRICH<EMAIL_ADDRESS>

@Ciloe, thanks so much.


Strange black space

Hi,
if you have an UIRefreshControl for your tableView which acts as the scrollView, the most top column is always getting black while scrolling.

Hi Marius,
Are you able to setup a sample project to show this? I've just tried this myself and I couldn't reproduce it.
Joe

At first I thought the UIRefreshControl might be the problem, it turned out it was something different.
After some time of examining and experimenting with your code, it turned out that my scrollView had some weird insets at the top which are caused by the interface builder. This resulted in a black cell at the top. I wasn't able to reproduce it with your Demo project as well, but it might have been my UITabBarController (If I set the bottomBar to the UITabBarController's tabBar, there was also black space when it was hidden.
My setup:
UITabBarController -> UINavigationController -> UITableViewController with JDFPeekaboo
                   -> UINavigationController -> ...

I had to select the UITableViewController's "Extend edges > Under Opaque Bars" setting to fix this - but I don't really know why. Maybe there is a better way to fix this or you should add that in your README.
If you still can't reproduce it with my setup, let me know and I'll upload my current project.
Cheers

Ah, I think I see what you mean. It sounds like your table view isn't extending underneath the navigation bar/tab bar, so, when JDFPeekaboo moves them out of the way, there is nothing underneath them to reveal. You're right, I should probably add this to the README. Thanks, @gbhrdt.

Updated README - thanks @gbhrdt. I'm going to close this.


[scss] remove plex unpkg @font-face declarations

In #1003 it was identified that plex condensed wasn't being loaded because it's not currently included in @carbon/type.
In #998 additional @font-face declarations were put in for these missing files. A url pointing to unpkg was used to avoid an increase in bundle size.
This should eventually be fixed upstream either by Carbon adding the required declarations to @carbon/type or @carbon/charts could expose/include this typeface through their package. Here is a link to internal slack discussion
Either way, once this is fixed upstream we should remove the @font-face declarations that were added in #998

Update - open PR on Carbon: https://github.com/carbon-design-system/carbon/pull/5680

Should be available in Carbon v11

:tada: This issue has been resolved in version 2.100.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SqueezeNet Implementation

Any feedback regarding model implementation, code formatting  anything else would be great.

Thanks for submitting this. I had a couple of smaller points on style and flexibility, but the architecture itself looks correct, and I'd be glad to pull this in if you tweaked those couple of little things.
I'm working on an ImageNet-sized dataset that should let us try this out on actual data, so I'll hook up a test using that when I've added that in. That should verify correct shape calculations throughout.

Any more improvements @rxwei @BradLarson ?

Sorry about the delay, had some unrelated test issues to fix.


fix: no directory issue

There is possibly a race condition between Next.JS creating a .next/vendor-chunks folder that throws an ENOENT error:
 ⨯ [Error: ENOENT: no such file or directory, copyfile '/Somewhere/monorepo/packages/db-project/node_modules/.prisma/client/libquery_engine-darwin-arm64.dylib.node' -> '/Somewhere/monorepo/apps/project/.next/server/vendor-chunks/libquery_engine-darwin-arm64.dylib.node'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'copyfile',
  path: '/Somewhere/monorepo/packages/db-project/node_modules/.prisma/client/libquery_engine-darwin-arm64.dylib.node',
  dest: '/Somewhere/monorepo/apps/project/.next/server/vendor-chunks/libquery_engine-darwin-arm64.dylib.node'
}

This commit makes sure the directory where this plugin copies files to exists.
Possible duplicate of https://github.com/prisma/prisma/pull/21370

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Bump to version 0.2.1

New changes from #7 affect the CLI, but not the library so let's just bump the patch version.

Done in #9.


feat: exporting onFail event

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update FaultInformationBtn to fix bugs

DM25 should print out the SPNs of the DM25 not the packet data value.
DM1 should not be printed twice when the button is fixed.

Created in the wrong place.


document which calls and field are not yet implemented in the analysis server protocol

Related to https://github.com/dart-lang/sdk/issues/23694, https://github.com/dart-lang/sdk/issues/23861, and https://github.com/dart-lang/sdk/issues/23640, it would be nice to set expectations about which calls or fields are populated / implemented. Having an @unimplemented annotation would save clients a lot of time debugging issues.

obsolete


Release ruby-style 1.25.1

Add rubocop 1.9 cops and disable a few others
Fix crash on extension checking, and eliminate new cop warning

Commits since previous releasecommit d79c6117fd1dbe253500fcca473071aa4feb545b
Author: Daniel Azuma
Date:   Mon Feb 8 16:00:30 2021 -0800
fix: Add rubocop 1.9 cops and disable a few others

commit 5fd0d420c460b9218154ddc956f63ff88b126ed4
Author: Daniel Azuma<EMAIL_ADDRESS>Date:   Fri Jan 29 13:33:21 2021 -0800
fix: Fix crash on extension checking, and eliminate new cop warning

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/ruby-style/releases/tag/ruby-style/v1.25.1


Huge update to get on par with MediaPipeUnityPlugin

This is a follow-up to https://github.com/vignetteapp/MediaPipe.NET.Runtime/pull/13 and should address the problems highlighted in issue #38, simply due to the updated packages.
@lubagov Let me know if this works for you!

Is this good to go?


Return OfferID on each confirmation

that will be good to accept only indicated offer

Very usefull! Could you tell me what other info is available on each confirmation if you know?

you can add a console.log($confirmation); to see all the info. Anyways i think nothing is usefull :)


Images not returned for Spotify links

Hi there, for Spotify links (that previously worked with last version of SWP), we now get only one image returned, blank: https://pixel.spotify.com/v1/sync

Have opened another issue with more information as I havent heard back


Unable to compile resinator on windows (using latest zig version and 0.11.0)

Download the latest zig prebuilt binaries (for windows)
At the time of writing, im using 0.12.0-dev.3043+00ff123b1
Then follow these steps
git clone https://github.com/squeek502/resinator.git
cd resinator
zig build

compiler errors :(

install
+- install resinator
   +- zig build-exe resinator Debug native 2 errors
src\auto_includes.zig:38:40: error: root struct of file 'compress.zstandard' has no member named 'decompressStream'
    var zstd_stream = std.compress.zstd.decompressStream(allocator, in_stream.reader());
                      ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
C:\Program Files\zig-0.12.0\lib\std\compress\zstandard.zig:1:1: note: struct declared here
const std = @import("std");
^~~~~
referenced by:
    getIncludePaths: src\main.zig:298:55
    main: src\main.zig:77:35
    remaining reference traces hidden; use '-freference-trace' to see all reference traces
src\auto_includes.zig:57:17: error: expected type 'error{OutOfMemory,InvalidWtf8,EnvironmentVariableNotFound}', found 'error{InvalidUtf8}'
                error.InvalidUtf8 => unreachable,
                ^~~~~~~~~~~~~~~~~
src\auto_includes.zig:57:17: note: 'error.InvalidUtf8' not a member of destination error set
error: the following command failed with 2 compilation errors:
C:\Program Files\zig-0.12.0\zig.exe build-exe -ODebug --dep aro --dep compressed_mingw_includes -Mroot=C:\OpenSource\resinator\src\main.zig --dep system_defaults --dep build_options --dep backend --dep Builtins/Builtin.def --dep Attribute/names.def --dep Diagnostics/messages.def -Maro=C:\Users\User\AppData\Local\zig\p\1220b51520b698aafd05caa6ee0b557148051d0fbb544dade82415e0af6a65b9f9bf\src\aro.zig -Mcompressed_mingw_includes=C:\Users\User\AppData\Local\zig\p\122024725c7d4a036e21dfc058790f8922b5bed7a4f4170f563410bad805f20aa761\compressed_mingw_includes.zig -Msystem_defaults=C:\OpenSource\resinator\zig-cache\c\f675b355841ee91ac5cdfba4ffba2f12\options.zig -Mbuild_options=C:\OpenSource\resinator\zig-cache\c\3fb0b6dc5967fa6a99fd63a05f8ec6e9\options.zig --dep zig --dep build_options -Mbackend=C:\Users\User\AppData\Local\zig\p\1220b51520b698aafd05caa6ee0b557148051d0fbb544dade82415e0af6a65b9f9bf\src\backend.zig -MBuiltins/Builtin.def=C:\OpenSource\resinator\zig-cache\o\92bf049ea4863d5e3d9c6509655fe87a\Builtin.zig -MAttribute/names.def=C:\OpenSource\resinator\zig-cache\o\0d4310355928579fe874237b86d7f04a\names.zig -MDiagnostics/messages.def=C:\OpenSource\resinator\zig-cache\o\d9f57286233b7bdcd0c8d10b42b905c6\messages.zig -Mzig=C:\Users\User\AppData\Local\zig\p\1220b51520b698aafd05caa6ee0b557148051d0fbb544dade82415e0af6a65b9f9bf\deps\zig\lib.zig --cache-dir C:\OpenSource\resinator\zig-cache --global-cache-dir C:\Users\User\AppData\Local\zig --name resinator --listen=-
Build Summary: 5/8 steps succeeded; 1 failed (disable with --summary none)
install transitive failure
+- install resinator transitive failure
   +- zig build-exe resinator Debug native 2 errors
error: the following build command failed with exit code 1:
C:\OpenSource\resinator\zig-cache\o\928854069b2f61e107dc722a736b4e0e\build.exe C:\Program Files\zig-0.12.0\zig.exe C:\OpenSource\resinator C:\OpenSource\resinator\zig-cache C:\Users\User\AppData\Local\zig --seed 0xfa3f3e57 -Ze42f6f9b9776c723

I also tried with zig version 0.11.0, but i still get compiler errors
Video demonstrating the compiler errors:
https://github.com/squeek502/resinator/assets/19608222/c9f1ed8d-e298-4a91-a517-9d2a5c774e98
I don't know how to write in Zig, so i could probably fix these errors myself if i knew the language well. The resinator tool looks promising for my needs, it's just that i dont understand zig to know what is wrong 😓

Sorry about that, got it compiling with latest Zig master version again in 92ee69c030d355107cb9e98148ce169da649d87a.
Note also that the Zig compiler has an interface to resinator built-in via the zig rc subcommand, so you can just use your Zig verison 0.12.0-dev.3043+00ff123b1 and do
zig rc your_rc_file.rc
and it'll work just the same (you can do zig rc /? to see the help menu).
See https://www.ryanliptak.com/blog/zig-is-a-windows-resource-compiler/ for more info about how resinator is integrated with/can be used with the Zig compiler.

ah sweet, awesome, i'll definitely use zig rc instead, thanks for the quick fix!


cmd/go: test with -run /something prints "no tests to run" inconsistently

When running a test with -run /subtest_name where no subtests match, it will print okay, even though no tests were run. Same is true if you use -run Test/subtest_name.
If instead, you do -run TestNameInPackage/subtest_name (where TestNameInPackage is the name of a test in that package) it will print "[no tests to run]"
Example:
➜  go test archive/tar -run /nomatchinghere
ok      archive/tar     0.145s
➜  go test archive/tar -run Test/nomatchinghere
ok      archive/tar     0.164s
➜  go test archive/tar -run TestWrite/nomatchinghere
ok      archive/tar     0.138s [no tests to run]

It has a similar issue when -v is set
/cc @bcmills @jayconrod @matloob

My inclination is to change the message to [no matching subtests] if we ran at least one top-level test and some component of the -run pattern ended up not matching anything.

Seems reasonable enough to me 👍

@bcmills This is in the 1.18 milestone; time to move to 1.19?  Thanks.

$ go1.19 test -v archive/tar -run=TestWrite/nomatchinghere
=== RUN   TestWriter
--- PASS: TestWriter (0.00s)
=== RUN   TestWriterErrors
--- PASS: TestWriterErrors (0.00s)
testing: warning: no tests to run
PASS
ok      archive/tar     0.466s [no tests to run]

Looks like this is fixed in go1.20 already:
$ go1.20 test -v archive/tar -run=TestWrite/nomatchinghere
=== RUN   TestWriter
--- PASS: TestWriter (0.00s)
=== RUN   TestWriterErrors
--- PASS: TestWriterErrors (0.00s)
=== RUN   TestWriteLongHeader
--- PASS: TestWriteLongHeader (0.08s)
PASS
ok      archive/tar     0.336s


Document Utility Classes w/ Demos!

Jira
http://vjira2:8080/browse/BDS-462
Summary
Adds documentation and demo within thebPattern Lab "utilities" section for all bolt utility classes (does not include @breakpoint rules).
Details
This PR includes all of the work needed to create new pages within the Pattern Lab "utilities" section for each of the bolt utility classes. Demos and demo scss was written to enhance and make possible the demos. This PR does not include any of the "@breakpoint" examples (or details).
How to test

Pull down this branch, run all the usual commands after pulling a new branch
Navigate to Pattern Lab "utilities" section and review each new page for accuracy in what it describes for implementation

@rockymountainhigh1943 @remydenton @mikemai2awesome @margoromo there’s still a bunch of things I’m seeing in the
https://github.com/bolt-design-system/bolt/blob/0e1dd08d1102c62bd267c59074bbb919607b5a26/apps/pattern-lab/src/styles/_docs.scss file I really think we should clean up before I’m good to merge this in — what I mentioned in my original review comments for this.
In a nutshell, “demo-specific” CSS we’re writing should be sticking to the same general coding methodologies and standards we’re shooting for across the board.
I’ll fill in the details once I get in but I think it ultimately comes down to two things:

Are our out of the box styles we have for layout and typography not good enough to provide the styling polish for our own docs pages in PL without having to write a ton of new CSS in the process?

If they aren’t, where are we lacking so those actual components / patterns / layouts we shipping can get tightened up?

Assuming there’s some truly unique “special snowflake” styles that aren’t satisfied by existing components, layouts and utility classes in the design system and — those special snowflake styles can’t broken down into reusable styles we ship for some specific reason (which we should call out) — nonetheless we should still be componentizing whatever else is left in our own codebase.

BEM classes shouldn’t be content or purpose specific — they should be as agnostic as possible to allow us the most reuse possible

Assuming the layout / spacing of things can’t be satisfied by utility classes, make sure we’re using the actual margin and padding utility classes provided by the design system, using the spacing scale provided by the system (ie small, medium, large, etc sizes). If there’s a reason why something we’re shipping isn’t good enough, this is when we’d flag it!

Why didn’t you break out the demo-specific styles written into heir own standalone Sass partial(s)?

What was the thought process / reasoning behind how certain demo sections get a particular amount of spacing?

There’s already been a ton of work we’ve done building out and improving our Bands, Wrappers, Headlines / Text, Grid, Utility Classes, etc etc so better understanding how you make the call when to reuse vs build would be super helpful to make sure we make the design system itself / usage of the design system as slick and easy as possible.


Use datadog's pip in the documentation

I had problems installing ga agent because datadog agent uses it's own python, not the system's one. I found that datadog uses python from /opt/datadog-agent/embedded/ directory.

Hi Alek
thank you very much for that!
I remember struggling with this also :D

You are welcome! Hope this will help someone in the future. And thank you for the agent, I love it.


Testing: ability to create test runs with no request

I'd like to implement 'Run last failing test' feature (by the way, it might make sense upstream). For me to do that in my extension, I need to know what currently enabled test profiles are.
// @connor4312

Just discovered 'rerun last run'!


masonry: Disable render tests by default

Set both ENABLE_RENDER_TESTS=1 and ENABLE_RENDER_SNAPSHOTS=1 in your environment to run the tests.

My concerns with this PR:

There's no documentation for how to run the tests yet, so nowhere to mention how to re-enable them.
I'm not sure we need 2 separate environment variables for now.

One other difference is that one part of this was reading the env variable at runtime via std::env::var while the other part was reading it at compile time via option_env!.

So, my expected updates for this PR would be.

Keep the change to read at runtime for both
Re-invert the SKIP_RENDER_TESTS flag, as that is only useful for Windows CI

Alternatively, we could close this pending the fixed snapshot tests (i.e. #233), but that is at least another couple of weeks away.
I can make these changes if needed.

I think we can close this now that render tests work?


update test-read_camtrap_dp.R back to updated version of datapackage.json

context
As per #191 test-read_camtrap_dp.R now refers to a specific commit for an example of camtrap-dp's datapackage.json:
test_that("file can be an URL", {
  # camtraptor is trailing camtrap-dp, refer to specific commit to keep using old version
  # dp_path <- "https://raw.githubusercontent.com/tdwg/camtrap-dp/main/example/datapackage.json"
  dp_path <-
    "https://raw.githubusercontent.com/tdwg/camtrap-dp/bb046c85a55bef2ced709357c0047f0136df8326/example/datapackage.json"
  dp <- suppressMessages(read_camtrap_dp(
    file = dp_path,
    media = FALSE
  ))
  expect_true(is.list(dp))
  expect_true(all(c("tbl_df", "tbl", "data.frame") %in%
    class(dp$data$deployments)))
  expect_true(all(c("tbl_df", "tbl", "data.frame") %in%
    class(dp$data$observations)))
})

refer to tests/testthat/test-read_camtrap_dp.R  and camtrap-dp/main/example/datapackage.json

We did this because at the time of #191 the test was failing because camtraptor was trailing an older version of camtrap-dp.

blockers
This issue is blocked by the neccesairy changes to read_camtrap_dp.R and camtraptor to use the new schema in general.
fix
Once camtraptor is ready: read_camtrap_dp() is updated to use/support the new schema) the test should be switched back to using the most up to date datapackage.json from camtrap-dp

This is no longer the case in camtrapdp, it refers to a version: https://github.com/inbo/camtrapdp/blob/14cc3cc2b8bf7399feef78f808ab5a62543ed9bb/tests/testthat/test-read_camtrapdp.R#L4


Facing Common endpoint error

The instructions here  don't specifically mention which supported account type to choose.
I selected "Personal Microsoft account users" since the article is related to configuring Microsoft Logins

But this results in an error.

I believe this is because of MSAL now using a common endpoint?
Creating a new App Registration with supported account type as "All Microsoft account users" works with any errors

We should update the documentation accordingly.

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

ID: a4c264c0-0c5c-298e-bb45-9b4891c8d162
Version Independent ID: ce69b990-0b4c-abda-cd2d-68f85cd8031e
Content: Microsoft Account external login setup with ASP.NET Core
Content Source: aspnetcore/security/authentication/social/microsoft-logins.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

#assign @Rick-Anderson @01binary

Looks simple, I can do it this evening. I don't have a project to test with anymore, but I will change exactly the way it was suggested.

@KalyanChanumolu-MSFT

But this results in an error.
Next time  paste in text rather than an image.

I tested this a month ago with personal and it worked.

@Rick-Anderson Here is the text
Exception: invalid_request;Description=The request is not valid for the application's 'userAudience' configuration. In order to use /common/ endpoint, the application must not be configured with 'Consumer' as the user audience. The userAudience should be configured with 'All' to use /common/ endpoint.
@01binary I have the sample. I will send it to you.

It sounds like the Microsoft Account OAuth provider is using a "common" endpoint, which makes sense. It's optimized for the "common case" of someone trying to login with Xbox, Outlook, or similar account.
The instructions that send you to Azure Active Directory to create a compatible App Registration should mention that you need to select a specific type that would be supported by the OAuth provider:

Haven't tried the code yet, but these options are reflected in App Registration manifest as follows:

Audience: AzureADMyOrg, URL = https://login.microsoftonline.com/{tenantid}/oauth2/v2.0/authorize
Audience: AzureADMultipleOrgs, URL = https://login.microsoftonline.com/organizations/oauth2/v2.0/authorize
Audience: AzureADandPersonalMicrosoftAccount, URL = https://login.microsoftonline.com/common/oauth2/v2.0/authorize
Audience: PersonalMicrosoftAccount, URL = https://login.microsoftonline.com/consumers/oauth2/v2.0/authorize

The OAuth provider for MS Account uses:
https://login.microsoftonline.com/common/oauth2/v2.0/authorize
See https://github.com/dotnet/aspnetcore/blob/master/src/Security/Authentication/MicrosoftAccount/src/MicrosoftAccountDefaults.cs#L13
I conclude therefore that the MS Account provider is hardwired to use "AzureADAndPersonalMicrosoftAccount" App Registration Audience, which is called "Accounts in any organizational directory (Any Azure AD directory - Multitenant) and personal Microsoft accounts (e.g. Skype, Xbox)" in Azure portal when creating a new App Registration. It seems like people should now that when they are going to use the Microsoft Account provider, then the App Registration that can work with it has to be created a certain way by picking the above option.

Trial results with various types of App Registrations, using MS Account provider and Blazor app I got from OP.

"Accounts in This Directory Only (Single Tenant)" selected when creating App Registration

unauthorized_client: The client does not exist or is not enabled for consumers. If you are the application developer, configure a new application through the App Registrations in the Azure Portal at https://go.microsoft.com/fwlink/?linkid=2083908.

I assume it's because Single Tenant must use Tenant ID in the URL, and since the MS Account provider is hardwired to use the common URL which does not contain a Tenant ID, it fails to find the App Registration specified by Client ID. This is expected.

"Accounts in Any Directory (Multitenant)" selected when creating App Registration

The OAuth dialog is shown with a UI error that says You can't sign in here with a personal account. Use your work or school account instead. I selected my work account and it accepted the choice. This is expected.

"Accounts in Any Directory and Personal Accounts" selected when creating App Registration

The OAuth dialog is shown, no error.

"Personal Only" selected when creating App Registration

Exception: invalid_request;Description=The request is not valid for the application's 'userAudience' configuration. In order to use /common/ endpoint, the application must not be configured with 'Consumer' as the user audience. The userAudience should be configured with 'All' to use /common/ endpoint.

This error is the reason why we should explicitly call out which App Registration type will work with MS Account provider.

I recommend the following change under the bullet point that says "Pick a supported account type":

The Microsoft.AspNetCore.Authentication.MicrosoftAccount package supports only App Registrations created using "Accounts in any organizational directory" or "Accounts in any organizational directory and Microsoft accounts" options.

I have to sync my fork and create a pull request, etc.

@01binary This is great.
@Rick-Anderson What would be the recommendation for customers that need to target "Microsoft Accounts only"?

What would be the recommendation for customers that need to target "Microsoft Accounts only"?

I think you'd need to scaffold Identity and write you own code to handle that.  @Tratcher  who's the right person to answer this question?

It should just be a matter of the developer setting
AuthorizationEndpoint to the right value, but I don't know what that is at the moment.

I suspected that was configurable (just the constant was hard-coded) but haven't looked there again. The OAuth URL for "Personal Microsoft Accounts Only" is https://login.microsoftonline.com/consumers/oauth2/v2.0/authorize. You can verify by creating an App Registration with that type, going to its Overview page, and clicking "Endpoints" button. Make sure the App Registration has "signInAudience": "PersonalMicrosoftAccount" in its Manifest, which means it's been created with that Personal type.
Here's what I got:
OAuth 2.0 authorization endpoint (v2): https://login.microsoftonline.com/consumers/oauth2/v2.0/authorize
OAuth 2.0 token endpoint (v2): https://login.microsoftonline.com/consumers/oauth2/v2.0/token
OpenID Connect metadata document: https://login.microsoftonline.com/consumers/v2.0/.well-known/openid-configuration

Ah, great. Yes, update the MicrosoftAccountOptions AuthorizationEndpoint and TokenEndpoint and re-validate.

I failed to sync my fork, so I re-created and re-cloned everything - sorry for the delay.

Thank you @01binary @Rick-Anderson

I suspected that was configurable (just the constant was hard-coded) but haven't looked there again. The OAuth URL for "Personal Microsoft Accounts Only" is https://login.microsoftonline.com/consumers/oauth2/v2.0/authorize. You can verify by creating an App Registration with that type, going to its Overview page, and clicking "Endpoints" button. Make sure the App Registration has "signInAudience": "PersonalMicrosoftAccount" in its Manifest, which means it's been created with that Personal type.
Here's what I got:
OAuth 2.0 authorization endpoint (v2): https://login.microsoftonline.com/consumers/oauth2/v2.0/authorize OAuth 2.0 token endpoint (v2): https://login.microsoftonline.com/consumers/oauth2/v2.0/token OpenID Connect metadata document: https://login.microsoftonline.com/consumers/v2.0/.well-known/openid-configuration

Good it's work for me, thamk's


Will not trigger when using the Photon feature included in JetPack

Just a quick bug report. It looks like the the lightbox will not trigger if there are parameters on the end of the image file. For example, this image link will open just fine in a lightbox:
http://www.dwellus.com/wp-content/uploads/2014/12/creighton-1.jpg
However, this will not work:
http://i2.wp.com/www.dwellus.com/wp-content/uploads/2014/12/creighton-1.jpg?resize=1200%2C800
I believe the resize parameter is causing the issue since I was able to get an image in a lightbox to open with this url:
'http://i2.wp.com/www.dwellus.com/wp-content/uploads/2014/12/creighton-1.jpg'
Thanks for the great plugin!!

I believe the problem is similar to the one reported in #64. As you mentioned, the lightbox doesn't support images including query strings.


I tried the-way view 14 but my cursor gets stuck. When I press Ctrl+d it kills the whole terminal.

Some of these, specifically those with serde crates, are easy to add as both import/export formats.

I primarily would like to have markdown and plaintext export.
sexps would be awesome too!

plaintext / markdown would be difficult to support as an import format, but easier for export. What would be the difference between plaintext and running the-way view?

It requires an index so I would need to write a for loop to get all the snippets out. I'd prefer to be able to just dump all the snippets to stdout or to dump snippets to stdout filtered by tags, etc... from the CLI directly.
I tried the-way view 14 but my cursor gets stuck. When I press Ctrl+d it kills the whole terminal:

I'm running the-way 0.15.0.
Originally posted by @jgarte in https://github.com/out-of-cheese-error/the-way/issues/127#issuecomment-1032193663

Some of these, specifically those with serde crates, are easy to add as both import/export formats.

I primarily would like to have markdown and plaintext export.
sexps would be awesome too!

plaintext / markdown would be difficult to support as an import format, but easier for export. What would be the difference between plaintext and running the-way view?

It requires an index so I would need to write a for loop to get all the snippets out. I'd prefer to be able to just dump all the snippets to stdout or to dump snippets to stdout filtered by tags, etc... from the CLI directly.
I tried the-way view 14 but my cursor gets stuck. When I press Ctrl+d it kills the whole terminal:

I'm running the-way 0.15.0.
Originally posted by @jgarte in #127 (comment)

Please can you provide the snippet which raise that issue (as code block or gist) ? This can be useful to reproduce the problem :)

 λ the-way view 14

■ #14. guix package template | scheme :guix:package:

(define-public XXX
  (package
    (name "XXX")
    (version "XXX")
    (source
      (origin XXX))
    (build-system XXX)
    (home-page "XXX")
    (synopsis "XXX")
    (description "XXX")
    (license XXX)))

I tried your snippet and I can't reproduce your problem. Can you provide us more information about your environment as:

shell (bash, zsh, fish etc)
terminal (gnome-terminal, xterm, alacritty etc)
Do you use a terminal multiplexer (as tmux) ?
Any relevant info

shell is bash
terminal is st
no terminal multiplexer when I tested that
I was using this Guix package.
Maybe there is a version mismatch in the Guix package that is causing the bug?

Does this happen with all of your snippets or just this one? One possibility is that something's going wrong with the ANSI codes. I've made a branch that clear ANSI styling at the end of each highlighted line - could you test this out?

Hi, sorry I'm late on this. Do you still have that branch I can test with?
Could you also provide the steps for testing?
Thanks

This has been merged to main now, is the issue still happening with the latest release?


A black bar (dark mode) on onClickOutside demo page

https://vitepress--vueuse.netlify.app/core/onClickOutside/
There is a black bar (dark mode) on top of line return { target }.

Fixed, thanks!

Fixed, thanks!


Do we need the Contributor Over Time chart?

Hi, we have generated a chart of contributors over time, which may be helpful to you.
You can take a look about:
https://contributor-graph-api.apiseven.com/contributors-svg?chart=contributorOverTime&repo=ossu/computer-science

It's interesting.

Interesting indeed! Thank you for sharing.
As this issue has not attracted any further comment in more than a month, I am closing it.


Error ux

Fixes #359
Making changes for error-ux for go and python SDKs

successful models build
https://github.com/open-traffic-generator/models/actions/runs/4456208500


installation

Hi, pls, how to install bonesi on linux, the link for the file download is not working

Hey, I'm having some problems with the installation, after issuing the command make, I get this error:
configure.ac:13: error: required file './compile' not found
configure.ac:13:   'automake --add-missing' can install 'compile'
Is this file needed because I can't find it, is there any workaround? I'm using Debian 8.
Thanks for your help.

Hello,
Yes, thanks for pointing this out deltasecurity!
In case the "compile" script is missing, you should perform a
$ automake --add-missing
In case there are different automake versions (not in this case but maybe on newer systems), you should regenerate the Make file using:
$ aclocal
$ autoconf
$ automake
Thanks!

You even need to install libnet1-dev as it was giving a dependency error of "libnet" while using command make.
So, add this :
sudo apt-get install libnet1-dev

In Ubuntu 18.04 LTS, I found this work for me.
First install some compiling tools (using apt install)
build-essential
libpcap-dev
libnet-dev
autoconf
automake
gcc
git
make
Then run the command like below:
git clone https://github.com/Markus-Go/bonesi.git
autoreconf -f -i
./configure
make
make install

Hey, I'm having some problems with the installation, after issuing the command ./configure, I get this error:configure: error: header missing
I'm using centos 7.5

Hello,
simply install the header files (-dev) for the library the configure script complains about. Typical candidates are libpcap-dev and libnet1-dev.

i am getting like this (anyone plz sort it out) im using ubuntu 16.04
/bonesi$ make
make  all-recursive
make[1]: Entering directory '/home/arvin/bonesi'
Making all in src
make[2]: Entering directory '/home/arvin/bonesi/src'
gcc -DHAVE_CONFIG_H -I. -I..     -g -O2 -MT bonesi.o -MD -MP -MF .deps/bonesi.Tpo -c -o bonesi.o bonesi.c
bonesi.c:121:17: warning: initialization makes integer from pointer without a cast [-Wint-conversion]
int rp_filter = NULL;
^
mv -f .deps/bonesi.Tpo .deps/bonesi.Po
gcc  -g -O2   -o bonesi bonesi.o  -lnet -lpthread -lpcap
make[2]: Leaving directory '/home/arvin/bonesi/src'
Making all in man
make[2]: Entering directory '/home/arvin/bonesi/man'
make[2]: Nothing to be done for 'all'.
make[2]: Leaving directory '/home/arvin/bonesi/man'
make[2]: Entering directory '/home/arvin/bonesi'
make[2]: Leaving directory '/home/arvin/bonesi'
make[1]: Leaving directory '/home/arvin/bonesi'
arvin@ubuntu:~/bonesi$ make install
Making install in src
make[1]: Entering directory '/home/arvin/bonesi/src'
make[2]: Entering directory '/home/arvin/bonesi/src'
/bin/mkdir -p '/usr/local/bin'
/usr/bin/install -c bonesi '/usr/local/bin'
/usr/bin/install: cannot remove '/usr/local/bin/bonesi': Permission denied
Makefile:285: recipe for target 'install-binPROGRAMS' failed
make[2]: *** [install-binPROGRAMS] Error 1
make[2]: Leaving directory '/home/arvin/bonesi/src'
Makefile:448: recipe for target 'install-am' failed
make[1]: *** [install-am] Error 2
make[1]: Leaving directory '/home/arvin/bonesi/src'
Makefile:354: recipe for target 'install-recursive' failed
make: *** [install-recursive] Error 1

Hello,
you need root privileges to install the compiled tool systemwide.

I think I had this issue, and I fixed it by running automake with some arguments.


Fix make install on OSX

Fixes #383
Signed-off-by: Ed Robinson<EMAIL_ADDRESS>

Thanks! LGTM pending tests.

@mtrmac CI fixed, PTAL


Add SAML Authentication Option to Connect

Currently pymapd cannot work with OmniSciDB without having legacy authentication enabled. This PR enables submission of the IdP form and connecting with the SAML response.

RBC for UDF support requires 'numba == 0.48'

This was already added @mikehinchey, and broke the build for some reason :-(


Set-MsolDomainAuthentication -DomainName  -Authentication Managed

This doesn't work
**Set-MsolDomainAuthentication -DomainName volunteer.devdaw.com -Authentication Managed
Set-MsolDomainAuthentication : Unable to change the authentication type of this domain because it is a subdomain. To change the authentication type of this subdomain, use the Get-MsolDomain cmdlet to find the root domain and  then change the root domain
authentication type.
At line:1 char:1

Set-MsolDomainAuthentication -DomainName volunteer.devdaw.com -Authen ...

  + CategoryInfo          : OperationStopped: (:) [Set-MsolDomainAuthentication], MicrosoftOnlineException
  + FullyQualifiedErrorId : Microsoft.Online.Administration.Automation.DomainNotRootException,Microsoft.Online.Administration.Automation.SetDomainAuthentication**

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

ID: 02c07bd2-01c5-77f4-11e0-6d942c5c0885
Version Independent ID: f61490c2-0f18-4609-29f1-99ccb4215ad2
Content: Change subdomain authentication type using PowerShell and Graph - Azure Active Directory
Content Source: articles/active-directory/enterprise-users/domains-verify-custom-subdomain.md
Service: active-directory
GitHub Login: @curtand
Microsoft Alias: curtand

@dawsonpaul
Thanks for your feedback! We will investigate and update as appropriate.

Hi @dawsonpaul, please refer to this document: https://docs.microsoft.com/en-us/troubleshoot/azure/active-directory/new-msoldomain-cmdle-add-subdomain
If the solution is not found, please let me know and I will open a support ticket.
Best,
James


DPL-455 - As Developers we need to add tests to the sample script so that we can refactor it and get it ready for adding to the support actions

User story
As Developers we need to add tests to the sample script so that we can refactor it and get it ready for adding to the support actions
Who are the primary contacts for this story
Neil
Who is the nominated tester for UAT
e.g. John S (don't include surnames in public repos)
Acceptance criteria
To be considered successful the solution must allow:

[ ] An RSpec test file for the sample script
[ ] A defined set of inputs and outputs
[ ] Sufficient test coverage to ensure that we are confident to refactor it

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Link to the script https://github.com/sanger/sequencescape/blob/develop/script/support/sample_move.rb

Removed from the board and deprioritised.


pycache

Hi,
very nice tool, thanks.
The distribution package has pycache dir in it. that breaks"python setup.py bdist_egg that in turn is used by buildout. Please fix the package!
sandro

After some time, the new version appears to fix this issue. Closing.


Bind dynamic form field  to a table of database

How do I  bind a dynamic form field  to a table of database?

You can't


no any data recived

hey adam,
first of all thanks for this awesome project I learned a lot from it
I am using Chevrolet Optra 2010 which is known by Lacetti
I connected to OBD-II to pin 6 and 14
changed the oscillator crystal clock to #define MCP2515_DEFAULT_CLOCK_FREQUENCY 8e6 in MCP2515.h
I tried the example codes (RPM and read) plus your code
all of them don't give me any reading (not even dump unreadable frames or data)
any suggestions or solutions?

answered in the other thread (issue #9)


Does this extension support VS Code Notebook?

Does this extenison work not only in Lua code files but also in VS Code notebook cells? It seems that syntax highlighting works but nothing else. One time suddendly all other features started working, but it looked more as a bug and I couldn't repeat it after restarting the editor. Every time I try to format a cell, VS Code asks me to install a Lua formatter from the marketplace. Can this extension be enabled or configured to work with cells?

I never used notebook, I will do some research.

Error checking doesn't work properly in notebook. The server does actually list all the errors in "PROBLEMS" panel, but it does not highlight them in cells. I don't know if it is intended behaviour though. The path to the ".ipynb" file shown in "PROBLEMS" panel is also magled. It looks like "notebook.ipynb#X16sZmlsZQ".


[dev] change update-changelog.sh to not leave data.tar.xz lying around

There's more to cleanup, but well, one piece at a time.
Signed-off-by: Stephan Renatus<EMAIL_ADDRESS>

Makes sense to me. 👍


Work edit view should show currently uploaded files

I bet we've already written this issue up elsewhere, but for now... I'm dropping this issue here because I'm closing a stale, orphaned PR in Sufia and need a place to put this:
From 87df6ee4c6e5f05c05ab9d7fef224f4be0a347b4 Mon Sep 17 00:00:00 2001
From: Paul Rosen<EMAIL_ADDRESS>Date: Fri, 13 May 2016 16:56:57 -0400
Subject: [PATCH 1/2] Add list of previously uploaded files to the edit files
 tab.

---
 app/forms/sufia/forms/work_form.rb                 |  1 +
 .../curation_concerns/base/_form_files.html.erb    | 23 ++++++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/app/forms/sufia/forms/work_form.rb b/app/forms/sufia/forms/work_form.rb
index 21966d5..266e295 100644
--- a/app/forms/sufia/forms/work_form.rb
+++ b/app/forms/sufia/forms/work_form.rb
@@ -4,6 +4,7 @@ class WorkForm < CurationConcerns::Forms::WorkForm
     include HydraEditor::Form::Permissions

     attr_reader :agreement_accepted
+    attr_accessor :file_sets

     self.terms += [:collection_ids]
     self.required_fields = [:title, :creator, :tag, :rights]
diff --git a/app/views/curation_concerns/base/_form_files.html.erb b/app/views/curation_concerns/base/_form_files.html.erb
index 54bff77..c05aa14 100644
--- a/app/views/curation_concerns/base/_form_files.html.erb
+++ b/app/views/curation_concerns/base/_form_files.html.erb
@@ -1,4 +1,27 @@

+         <% if f.object.file_sets.present? > 0 %>
+             <fieldset id="uploaded-files">
+                 <legend>Previously Uploaded Files</legend>
+

+

+

+                         Filename
+                         Display Label
+

+                     <% f.object.file_sets.each { |file| %>
+

+
+                                 <%= file.solr_document.label %>
+
+
+                                 <%= file.title %>
+
+

+                     <% } %>
+

+

+             </fieldset>
+         <% end %>

         <input type="hidden" name="redirect" value="<%= main_app.root_path %>">

From 2960253b054d263615a03cffd6c24fa3adbd5836 Mon Sep 17 00:00:00 2001
From: Paul Rosen<EMAIL_ADDRESS>Date: Sat, 14 May 2016 09:20:29 -0400
Subject: [PATCH 2/2] Add list of previously uploaded files to the edit files
 tab.

---
 app/views/curation_concerns/base/_form_files.html.erb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/curation_concerns/base/_form_files.html.erb b/app/views/curation_concerns/base/_form_files.html.erb
index c05aa14..63b2322 100644
--- a/app/views/curation_concerns/base/_form_files.html.erb
+++ b/app/views/curation_concerns/base/_form_files.html.erb
@@ -8,7 +8,7 @@
                          Filename
                          Display Label

-                     <% f.object.file_sets.each { |file| %>
+                     <% f.object.file_sets.each do |file| %>

                                  <%= file.solr_document.label %>
@@ -17,7 +17,7 @@
                                  <%= file.title %>

-                     <% } %>
+                     <% end %>

              </fieldset>

Closing projecthydra/sufia#2001

Related to https://github.com/projecthydra/sufia/issues/1943

Related also to https://github.com/projecthydra/sufia/issues/1707 in that the view should allow editing of file labels.

TBD: link to UI WG design for this.


Find a method of alerting new publications about ion channels of interest

In order to being aware of new publications about patch clamp experiments, new biological studies related to ion channels in C. elegans, etc in an automatic way, we need a procedure (something like a RSS feed reader with text mining abilities!). then we can augment it by some alerting (mailing) system.

@VahidGh Have you looked at Textpresso?  I wonder if there is something in there we could reuse?

@slarson, that's cool, especially has a separate section for C. elegans.

Google notifications will probably give us decent mileage.  Would be a nice simple project also for a newcomer to write a script to periodically search the Textpresso C. elegans section.


[BUG] When I try to comment linewise in visual mode, unrelated or extra code gets commented

Here is my config (using lazy):
return {
    "numToStr/Comment.nvim",
    keys = {
        {
            "<C-_>",
            function()
                require("Comment.api").toggle.linewise.current()
                require("rizhiy.keys").press("j", "n")
            end,
            desc = "Comment line",
        },
        {
            "<C-_>",
            function() require("Comment.api").toggle.linewise(vim.fn.visualmode()) end,
            mode = "v",
            desc = "Comment selection",
        },
    },
    opts = {
        mappings = {
            basic = false,
        },
    },
}

When I try to comment in visual mode (using C-/), different code rather than selection gets commented, sometimes the outer scope, sometimes completely unrelated code altogether.
This config was working previously, not sure what changed.
vim version:
NVIM v0.9.4
Build type: Release
LuaJIT 2.1.1692716794

Hmm, when I try to use default keymaps nothing happens.

I'm having similar behavior, it seems to me that it toggles the previous selection in visual/visual block mode, ignoring the visual line mode selection.

Surprisingly, I just had a similar investigation into how to get visual region.
Yeah, this plugin is also making the same mistake, so either they changed it recently or just common misconception.
Need to either change to . and v marks or modify function to exit visual mode before calling toggle.linewise():
{
    "<C-_>",
    function()
        vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes("<Esc>", true, true, true), "n", true)
        require("Comment.api").toggle.linewise(vim.fn.visualmode())
    end,
    mode = "v",
    desc = "Comment selection",
},

I just came to the same conclusion. I don't think this plugin is making a mistake so much as just expecting you to exit visual mode first. Trying to adjust the plugin to use . and v positions.

@Rizhiy for some reason that's not working here.
I have the following in my keybinds.lua:
vim.keymap.set('v', "«", function()
  vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes("<Esc>", true, true, true), "n", true)
  require('Comment.api').toggle.linewise(vim.fn.visualmode())
end)

(for some reason option-\ ends up as being « on my system, don't ask).
It still comments strange parts of the file in visual mode.


Error when running application in expo environment

I'm getting the following error when running the project in an expo environment with the first example usage code from readme.md:
2022-02-10 15:14:28.707610+1100 expoeject[5429:23036] [native]
Error setting property 'source' of RNTrimmerView with tag #5:
Exception thrown while executing UI block:
CALayer bounds contains NaN: [0 0; nan 20].
Layer:
<CALayer:0x6000018fc1e0;
  position = CGPoint (0 0);
  bounds = CGRect (0 0; 0 0);
  delegate = <UIView: 0x7faeb681d7f0;
    frame = (0 0; 0 0);
    layer = <CALayer: 0x6000018fc1e0>
  >;
  allowsGroupOpacity = YES;
>

Does anyone else have it working successfully in a bare workflow expo environment?
Your Environment

software
version

react-native-video-processing
^2.0.0

react-native
0.64.3

node
14.16.1

expo
~44.0.2

Xcode
13.2.1

I hadn't changed the source lines in the example code, changed them and it's working now.

@jsindos did you need to eject to get this running in expo?


Devnet memory leak in fullnode

Steps to Reproduce Issue
Just start the fullnode from the code source
Expected Result
The server should to be up and running
Actual Result
Server or service crash after 12 minutes

System Information

OS: debian 11.4
Compiler: rustic 1.62.1

@huitseeker do you think that your pull request is going to fix that ? : https://github.com/MystenLabs/sui/pull/3905

Experiencing the same.
See memory utilization stats log for sui process.
sui_mem_leak.log
.

This should be fixed in #3874

I think it's time to revisit our system requirements for fullnode.

Reopened related:
https://github.com/MystenLabs/sui/issues/1367


New version: Brave.Brave.Nightly version <IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #9812.

Log
Updated: <IP_ADDRESS> → <IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run


[PID/(Q)EAA - SD-JWT parameters ] trust chain

This PR aims to resolve the issue https://github.com/italia/eudi-wallet-it-docs/issues/259

Closes #259


Question regarding easy-peasy & keychain usage

Hello @HarishJangra ,
I have a project that uses OTP as the authenticator to log in into the app. Could you please show me the Login Navigation Flow and how do i use easy-peasy? i want to store the phone number and token to the local storage, while your initial boilerplate uses username and password instead. Kindly need your help asap. btw, thank you very much for the repo, it really helps cutting down some development time.

Hi @rajinpangkalpundai
You can definitely store anything in keychain due to library restrictions the keys in the store will be username and password only. You can refer to react-native-keychain for more info.
So You can store mobile or other info as username and password
and when you get the stored data here you can change like this
export const getLoginCredentials = async () => {
	try {
		const credentials = await Keychain.getGenericPassword();
		console.log("keychain get data ", credentials);
		if (credentials) {
			return {YOURKEYONE: credentials.username, YOURKEYTWO:credentials.password};
		}
		return false;
	} catch (e) {
		console.log("Cannot retrieve keychain data", e);
		return false;
	}
};

replace YOURKEYONE & YOURKEYTWO to your custom keys.

YOURKEYONE

i've checked react-native-keychain, and yes, it has limitation to only store in 'username' and 'password'. Another question is:
Is it possible to use AsyncStorage on 'Services' folder to save my login credentials and later use it to maintain the app state (session)?
Thank you for your quick reply, i really appreciate it.

Yes you can use it in whatever way you want.

Yes you can use it in whatever way you want.

Awesome! thanks for your help


Appstudio update test-custom-default-branch-szso

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-ntht-on-pull-request-
creating pipelinerun test-annotations-ntht-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-vchz-on-pull-request-
creating pipelinerun test-annotations-vchz-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nwtb-on-pull-request-
creating pipelinerun test-simple-build-nwtb-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ufgi-on-pull-request-
creating pipelinerun test-simple-build-ufgi-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wadk-on-pull-request-
creating pipelinerun test-simple-build-wadk-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-gcoj-on-pull-request-
creating pipelinerun test-annotations-gcoj-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-imul-on-pull-request-
creating pipelinerun test-simple-build-imul-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-yrqk-on-pull-request-
creating pipelinerun test-annotations-yrqk-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-lqle-on-pull-request-
creating pipelinerun test-simple-build-lqle-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nyrh-on-pull-request-
creating pipelinerun test-simple-build-nyrh-on-pull-request- in namespace build-e2e-tkye-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Add package.json nativescript platforms

#1

@enchev Also removed extension so it can work with Webpack

Ping @enchev , please update on NPM


ForEach-Object: Cannot convert argument "item", with value: "MicrosoftAdminPortals"

Hi everyone,
I have an environment where the report creation fails with following  error:
ForEach-Object: Cannot convert argument "item", with value: "MicrosoftAdminPortals", for "Add" to type "System.Guid": "Cannot convert value "MicrosoftAdminPortals" to type "System.Guid". Error: "Guid should contain 32 digits with 4 dashes (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx).""
How can I solve this please, any idea is welcome.
cheers

@jasoth @merill do you guys maybe have an idea?

I am facing the same issue. Is there a solution to this?

Please update to the latest version for the fix.


addition of many RL vehicles in a given scenario and activate render only after an instance

Hello,
I have added more number of RL vehicles in the example "cooperative merge" distributively. But I have noticed that, my agents which I have added, does not accelerate much when compared with 'human' vehicles. Because of this, there are many collisions which leads in teleporting and the agent doesn't train well. I am training using A3C. I have planned to train this further with SAC algorithm

I have tried with the sumocarfollowing parameters 'speed_factor', 'speed_dev' and EnvParamteres 'max_accel'. I first tried to train the agent with 'warmup_steps', hoping that RL vehicles will increase their speed afterwards. But they did not accelerate. Is it possible that I can make my RL vehicles run with the speed of human vehicles to avoid collision and perform better training?
Is there a way to set the render of the simulation only at the end of training or after certain training steps are completed? Because if I use [sim=SumoParams(render =True)]  , it renders from first training step and of-course it is known that training doesn't happen well when render is True.

I have my code as follows :
vehicles.add(
    veh_id='rl2',
    acceleration_controller=(RLController, {}),
    lane_change_controller=(SimLaneChangeController, {}),
    routing_controller=(ContinuousRouter, {}),
    num_vehicles=1,
    car_following_params=SumoCarFollowingParams(
        minGap=0.02,
        tau=0.2,
        sigma = 0.9,
        speed_mode="obey_safe_speed",
        speedFactor="normc(1,0.1,0.2,150)",
        speedDev="10",
    ),
    lane_change_params=SumoLaneChangeParams())
flow_params = dict(
    # name of the experiment
    exp_tag='cooperative_merge_Changed_rl_values',

    # name of the flow environment the experiment is running on
    env_name='TwoLoopsMergePOEnv',

    # name of the scenario class the experiment is running on
    scenario='TwoLoopsOneMergingScenario',

    # simulator that is used by the experiment
    simulator='traci',

    # sumo-related parameters (see flow.core.params.SumoParams)
    sim=SumoParams(
        sim_step=0.1,
        render=False,
    ),

    # environment related parameters (see flow.core.params.EnvParams)
    env=EnvParams(
        horizon=HORIZON,
        additional_params={
            'max_accel': 100,
            'max_decel': 3,
            'target_velocity': 10,
            'n_preceding': 2,
            'n_following': 2,
            'n_merging_in': 2,
            'warmup_steps': 200,
        },
    ),

    # network-related parameters (see flow.core.params.NetParams and the
    # scenario's documentation or ADDITIONAL_NET_PARAMS component)
    net=NetParams(
        no_internal_links=False,
        additional_params={
            'ring_radius': 50,
            'lane_length': 75,
            'inner_lanes': 1,
            'outer_lanes': 1,
            'speed_limit': 30,
            'resolution': 40,
        },
    ),

    # vehicles to be placed in the network at the start of a rollout (see
    # flow.core.vehicles.Vehicles)
    veh=vehicles,

    # parameters specifying the positioning of vehicles upon initialization/
    # reset (see flow.core.params.InitialConfig)
    initial=InitialConfig(
        x0=50,
        spacing='uniform',
        additional_params={
            'merge_bunching': 0,
        },
    ),
)
def setup_exps():

    alg_run = 'A3C'

    agent_cls = get_agent_class(alg_run)
    config = agent_cls._default_config.copy()
    config['num_workers'] = N_CPUS
    config['train_batch_size'] = HORIZON * N_ROLLOUTS
    config['gamma'] = 0.999  # discount rate
    config['model'].update({'fcnet_hiddens': [16, 16, 16]})
    config['lambda'] = 0.97
    config['clip_actions'] = False
    config['horizon'] = HORIZON

    # save the flow params for replay
    flow_json = json.dumps(
        flow_params, cls=FlowParamsEncoder, sort_keys=True, indent=4)
    config['env_config']['flow_params'] = flow_json
    config['env_config']['run'] = alg_run

    #flow_params = variant['flow_params']
    #flow_params = get_flow_params(variant)
    #flow_params['sim'].render = True
    #print(flow_params)

    create_env, gym_name = make_create_env(params=flow_params, version=0)

    # Register as rllib env
    register_env(gym_name, create_env)
    return alg_run, gym_name, config

if __name__ == '__main__':
    alg_run, gym_name, config = setup_exps()
    ray.init(num_cpus=N_CPUS+1, redirect_output=False)
    trials = run_experiments({
        flow_params['exp_tag']: {
            'run': alg_run,
            'env': gym_name,
            'config': {
                **config
            },
            'checkpoint_freq': 20,
            'max_failures': 999,
            'stop': {
                'training_iteration': 200,
            },
        }
    })

OS : Linux  18.04
Flow Version : 0.3.0
SUMO Version : v0_31_0-812-g1d4338ab80

Hi @pnp91, in response to your questions:

The RL vehicle might be colliding because your minGap value is too small. I would recommend taking it up to a larger value, say around 2. You can also consider increasing your tau value to around 0.5 or 1. Tuning thee values and possibly others may be key for you to avoid collisions.
To visualize the policy after training (with the renderer on), you will want to use the flow/visualizer_rllib.py method, see: https://flow.readthedocs.io/en/latest/visualizing.html

Hi @AboudyKreidieh :

I have already checked with these parameters and was not much visible difference. Hence I tried with speed_factor and speed_dev as well. But it did not help.
If I am correct, visualize is only for viewing one checkpoint and not from a particular point till end of training, right?. Yes I have already used 'visualize' after training is completed. But only for visualizing particular checkpoints and not to check from a specified training point till the end of training.

hi @pnp91:

You can avoid collisions altogether by setting speed_mode to "all_checks"; however, that may lead to some unrealistic behavior by vehicles on the main highway. Other values for the speed_mode, say 9, may also do the trick for you.
I don't think sumo supports rendering for only a subset of simulations, as you must call specific binaries that either include the gui or not before you actually run anything. You may want to consider initializing the states of the vehicles to something desirable and running the simulation with the renderer turned on from there.


onNext function for TearSheet not working in version 1.15

What package(s) are you using?

[X] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Describe in detail the issue you're having.

The TearSheet onNext function always rejects our promise in 1.15. This did not happen in 0.98 - we did not change any code, we only updated Carbon versions. I tried print debugging by adding print statements inside the onNext function but the print statements did not show up, it just always rejected.

Is this issue related to a specific component?

Yes the TearSheet component

What did you expect to happen? What happened instead? What would you like to
see changed?

We expected the TearSheet to allow us to print inside onNext and also for it not to always reject the promise.

What browser are you working in?

The latest version of Google Chrome

What version of the @carbon/ibm-products (or @carbon/ibm-cloud-cognitive)
package are you using?

Version 1.15

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

This is for Cloud Pak for Watson AIOps. We have an upcoming deadline within the next week. We are currently using an older version in the meantime.
Steps to reproduce the issue

Return a new promise inside the onNext prop for the Create TearSheet component with resolve and reject as input variables
Inside the promise, check if a React state external to the component is true. If it is not, then reject.
It will always reject regardless and if you try to print debug it will not print your statements.

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/carbon-design-system/ibm-cloud-cognitive/tree/main/examples/cloud-cognitive/codesandbox

Additional information

Shared code with @matthewgallo

Hi 👋
We are also having problems with onNext.
Here is my sandbox - https://codesandbox.io/s/createtearsheet-onnext-bvljgf?file=/src/App.js
In the first step you can enter a string and when you click next it should console.log that string. However, it logs undefined and then the if you go back and then click next again, it prints logs correctly.
This is stopping us from upgrading from 1.3.0 -> 1.5.0+

Hey @tomleah, I've created https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/1823 which I tested against how you had your code sandbox component structured and it works with this change. @petergramenides I suspect that this may fix the issue you were running into as well.

Great! Thanks @matthewgallo 🎉

This should be fixed in v1.11.2. As always feel free to reopen if you are still experiencing any issues! Thanks for your patience with this one @tomleah and @petergramenides

Thank you @matthewgallo! Also thank you @tomleah for reproducing.