Update active-directory-accessmanagement-simplerulegroup.md

Hi @RobdeJong, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (rodejo). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


chore: update README.md

Signed-off-by: 荣顶<EMAIL_ADDRESS>
Details
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current PR for duplication.


feat(frontend): Landing page

What core or extra feature does this satisfy?
This is a core feature.
A brief description of how the website works and the goal it serves to achieve.
Describe the feature

[ ] Have a paragraph to describe what this website does. (Leek)
[ ] Mission statement (Leek)

Additional context
Landing page is different from main page in that it is static and just describes our page. This is one of the page that contains a navbar to navigate between the pages.
Recommended/suggested endpoints (please modify as you see fit when working on the issue):

Landing Page HTML - localhost:3000/

Moved issue


Improve union case serialization & added tests

I prefer this way of serialization of union cases. It's well usable from other languages like C#.

The major change in here is that union cases are now serialized as objects with fields.
This can be done because the tuple of a union case can contain field names and if they don't contain a field name it is always there because the compiler autogenerates these.

I will consider this change. Though it's a breaking change (and not a breaking bugfix) to current logic. So such change in logic should be either under config flag or introduced in major release.

I did this minimal change during company time. I'm not using it anymore there but I'm willing to polish it in my private time, however this will take some time


Add warning about your own SMTP

@digitaldiina Please feel free to amend the text to improve my English.


Fill in some missing tilemap location reference pages

Put in the missing reference pages for tilemap and sprites API updates.

[x] Add new tilemap location API pages.
[x] Add a page for destroyAllSpritesOfKind()
[x] Add a Location object page describing it's props.
[x] Update linking in location pages.
[x] Set help paths for reference pages.

Note: A new page is included for tiles.Location but there is no way to link context help for it to this block-

This is the same as the situation with the sprite properties too.
Closes https://github.com/microsoft/pxt-arcade/issues/4590

Not sure what you mean about no way to link context Help?  Looks like it's pointing to https://arcade.makecode.com/pkg/libs/device#tiles-location-number-%40get%40


cache build assets for component

Solves #220

Super simple implementation. Tested things manually as well, and things work nicely :)


Bug: MP Systems displays a blank screen with View/Edit

Issue:
When a user wishes to View/Edit MP Systems and clicks on View/Edit, then the system displays a blank screen.
Steps to Recreate:

Login to ECMPS and access the MP Module
Select and checkout Barry
Select System from the Sections dropdown and click on View/Edit for any of the existing records
You will notice that the screen turns blank (Kyle and Jaso are able to reproduce with the below screenshots, but Vishnu is unable to)

Notes:
We also see an internal server error when we click View/Edit/Create for Qualifications and suspect that it happens when the Sys ID/Comp ID and the Location ID are not defined/passed.
Blank Screen for Systems:

Errors for Qualification:

Systems and Qualification create and edit as expected


How to install?

How to deploy this code into my webserver? Thank you for your amazing work!

I signed up to vercel and i easily deployed my own monokai pro generator. Thank you for your amazing work! please continue!


Code changes re. Keyboard controls #3484

Description
This relates to discussion Keyboard controls #3484
In working this though there are some code changes in SmoothControl.ts that you can have if you want them.
These extend the keyboard controls from orbit changes to orbit, pan, step size & preset views and also remove the null hit behavior.
The removal of null hit behavior should really be conditional on an exposed state, but I didn't know how to do that.
They are contained entirely within SmoothControls.ts.
Keyboard controls are:
Arrows - orbit
Shift + Arrows - pan
Ctrl + Arrows - orbit at small step (ratio from unmodified set as a const, but could ideally be passed in)
Ctrl + Shift + Arrows - pan at small steps
Pressing Shift displays the same spot cursor as used on mouse-pan. Shift now behaves as a defacto 'about to pan' indicator. It also accidentally solved a user comment that they didn't really know where it was orbiting around.
Keys 1, 2, 3, 4 (and Numpad 1, 2, 3, 4) set theta to 0, 90, 180, 270 degrees at current phi and radius
Keys 7, 8, 9 (and Numpad 7, 8, 9) set phi to +45, 0 and -45 degrees at current theta and radius
This gives the ability to 'square up' a view which works in conjunction with the orbit keys.
Only experimental testing has been done. No automated test.
Live Demo
Video clips in  Keyboard controls #3484
Version

model-viewer: v1.12.0

Browser Affected

[ 1 ] Chrome, version: xx.x.xxxx.xx
[ 1] Edge
[ 1] Firefox
[ 0] IE
[ 0] Safari

OS

[ 0] Android
[ 0] iOS
[ 0] Linux
[ 0] MacOS
[ 1] Windows

AR

[ ] WebXR
[ ] SceneViewer
[ ] QuickLook

Please open a PR so I can review these changes; it sounds like this could be quite useful. Thanks!

No problem, but I've fallen at the first hurdle - I don't know what I'm doing!
If PR is pull request then what do I choose for base & compare?
If there is a noob PR raising 101 then send me that way. In the words of an unnamed poet there are non so blind as those that have read the documentation a thousand times.

This is a good place to get started: https://docs.github.com/en/get-started/quickstart/github-flow
Once you have a fork, make a branch, make changes, and push it, when you come to our Pull Requests tab, Github will have a green button like "do you want to make a PR for the branch you just pushed"? Their default behavior should work just fine.


💡 [FEATURE] - Add DevOps Communities Section to Resources

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
The current DevOps resources does not include a section for communities where individuals can engage with peers, ask questions, and share knowledge. Connecting with communities is crucial for professional growth and staying updated with industry trends.
Explain your solution
Add a new section titled "Communities" to the DevOps resources. This section will include links to various DevOps communities such as forums, subreddits, LinkedIn groups, chat platforms, and more.
Any alternative approaches/features
No response
Additional Context
No response

Good idea, I assigned it to you

Didnt noticed someone already got the issue and raised a PR, maybe y ou can add resources under it, thanks


Option to import other react component inside the deepchat

Hi,
I was wondering if there is any easy option to import custom react/jquery component inside deep-chat. Like Image gallery/carousel/
example image is mentioned below.

Hi @anup-singh.
Apologies for the late reply as I have been very busy with my work. The answer to your question is yes, as long as it is an actual custom element.

I will be closing this issue as there has been no further activity.


Teams PowerShell Preview Notes

Please consider adding notes on each of these cmdlets indicating if they are currently only in the preview module.

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

ID: 6804423b-8e08-5916-9355-6bebe8bb08b0
Version Independent ID: aa669763-6470-cc61-ea8d-52aee886f986
Content: MicrosoftTeamsPowerShell Module
Content Source: teams/teams-ps/teams/teams.md
Service: teams-powershell
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Hello @BrianMcNeese Brian, how are you?
That information was added here Microsoft Teams PowerShell Release Notes - Cmdlet availability recently.
Thanks for your feedback.

@get-itips Thank you very much for the contribution and sharing this explanation. @BrianMcNeese Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


keep bidi support as non-default behavior

Arabic script (used also in many languages) needs to have both bidi and  font-shaping support in order to have it read properly
translate-shell has a pseudo facility to display the arabic script in terminals that don't support bidi, and this is the default behaviour when choosing language targets to be arabic or farsi. however as of 2019 most terminals has bidi and shaping support provided because of the newer Gtk VTE features. now  pseudo bidi in translate shell makes that scripts unreadable in that terminals as bidi-and-shaping process is done twice actually.
currently user can disable translate-shell bidi via -no-bidi option, maybe should make the enabling bidi as the option (-with-bidi) and disabling pseudo bidi by default as most of mac and linux terminals now support it already out of box.

currently user can disable translate-shell bidi via -no-bidi option,

One trick to disable translate-shell's bidi by default, is to add
  :no-bidi true

in the configuration file.
I can confirm that newer VTE does bidi transformation automatically, however this may not be generally true for older distros, and there are also a lot of non-VTE emulators which do not have this (according to https://gist.github.com/XVilka/a0e49e1c65370ba11c17). So, a more proper default behavior would be to detect if the terminal emulator has embedded bidi support and perform bidi transformation only if needed (so as not to transform the right-to-left text twice).

I can confirm that newer VTE does bidi transformation automatically, however this may not be generally true for older distros, and there are also a lot of non-VTE emulators which do not have this (according to https://gist.github.com/XVilka/a0e49e1c65370ba11c17). So, a more proper default behavior would be to detect if the terminal emulator has embedded bidi support and perform bidi transformation only if needed (so as not to transform the right-to-left text twice).
I am afraid that terminal emulators don't provide any info or flags that indicate bidi is provided in there
maybe should suggest it to VTE's developers as many other TUI apps have similar issues (including vim and emacs)?

trans will stop doing its own BiDi transform for terminals based on newer VTE versions. Specifically, it checks if the env variable $VTE_VERSION is greater than 5703 (since vte 0.57.3 is the first version that implements RTL/BiDi support according to this).
If anyone encounters RTL rendering problems under non-VTE terminals, feel free to open a separate issue.


Access to fetch at 'https://login.windows.net/consumers/oauth2/v2.0/authorize?client_id=

Access to fetch at 'https://login.windows.net/consumers/oauth2/v2.0/authorize?client_id=
(redirected from 'https://rewards.bing.com/') from origin 'chrome-extension://agabinmnkdikcfpoheccgnolecngccgm' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
This pops up in developer console from errors tab

You may need to visit https://rewards.bing.com/ and log in.


Can't Start

What should I do?

hi there, could you more thoroughly describe the process you went through? We're not sure how to fix the issue if we are unable to recreate the bug.
Here are some things you can try in the meantime:

Change your region
Regenerate your Auth Token

hi there, could you more thoroughly describe the process you went through? We're not sure how to fix the issue if we are unable to recreate the bug.
Here are some things you can try in the meantime:

Change your region
Regenerate your Auth Token

I tried but problem didn't solved :(

If the problem still persists, you could try to update to the newest version here. (Make sure to delete the old version first)
You may have to input your auth token again in the mod menu after the upgrade.

If the problem still persists, you could try to update to the newest version here. (Make sure to delete the old version first)
You may have to input your auth token again in the mod menu after the upgrade.

I installed the new version of the mod and tried to put the game in a new directory, created a new token, tried other locations, changed my internet source, checked the necessary mods, but unfortunately I couldn't use it. Same problem again :/

One other possible fix would be restarting your Host Network Service:

Press Win+R, type services.msc
In the list of services, find "Host Network Service"
Right click on it and press restart
Try the mod again

Thanks Its working! :)

Thanks again 👊🏿


Issue/132 create controller for pokemon and trainer for spring mvc

Resolves #132

Please fix to use javatime:format in place of my:localDate (now that it's merged into master it should be a straightforward change).


Can't load .env variables into .conf

This line generates the .conf from the template, but doesn't seem to load any of the sourced .env variables -- nginx -t reports them as invalid...
# Fill your config options into nginx.conf.template to create nginx.conf
envsubst \
    "$(printf '${%s} ' $(bash -c "compgen -A variable"))"\
    < "$CONFIG_DIR/nginx.conf.template" \
    > "$CONFIG_DIR/nginx.conf"

I am trying that, through trial and error, but an example of what it should look like would help a lot. I've successfully added a few like this at the top of the .conf file:
env PROJECT_DIR=/opt/wiki;
Where the variable value is an uninterrupted string. But variables like this throw errors:
env CACHE_REQUESTS=GET HEAD POST;
Error is:
nginx: [emerg] invalid number of arguments in "env" directive in /etc/nginx/nginx.conf:23
nginx: configuration file /etc/nginx/nginx.conf test failed

If leave out the troublesome variables, I get further validation errors like:
invalid max_size value "max_size=$CACHE_SIZE" in /etc/nginx/nginx.conf:114
nginx: configuration file /etc/nginx/nginx.conf test failed

I recommend following nginx docs to resolve any individual issues with nginx config keys: https://www.nginx.com/blog/nginx-caching-guide/
As mentioned in those docs, the format for that config key is max_size=10g not max_size=10GB.


Import error when using skvideo.io

I am trying to import skvideo.io in python 3.6.3 but getting an import error. I installed skvideo through pip and it was installed successfully.
The is the error I am getting
 Traceback (most recent call last): File "", line 1, in import skvideo.io File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\skvideo__init__.py", line 3, in from .utils import check_output, where File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\skvideo\utils__init__.py", line 10, in from .edge import canny File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\skvideo\utils\edge.py", line 8, in import scipy.ndimage File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\ndimage__init__.py", line 161, in from .filters import * File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\ndimage\filters.py", line 37, in from scipy.misc import doccer File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\misc__init__.py", line 67, in from scipy.interpolate._pade import pade as _pade File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\interpolate__init__.py", line 175, in from .interpolate import * File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\interpolate\interpolate.py", line 20, in import scipy.linalg File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\linalg__init__.py", line 186, in from .misc import * File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\linalg\misc.py", line 5, in from .blas import get_blas_funcs File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib\site-packages\scipy\linalg\blas.py", line 196, in from scipy.linalg import _fblas ImportError: DLL load failed: The specified module could not be found.

Did you install scipy?
On Oct 18, 2017 4:14 AM, "Hasan Abdelghaffar"<EMAIL_ADDRESS>wrote:

I am trying to import skvideo.io in python 3.6.3 but getting an import
error. I installed skvideo through pip and it was installed successfully.
The is the error I am getting
Traceback (most recent call last): File "", line 1, in import skvideo.io
File "C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib
site-packages\skvideo__init__.py", line 3, in from .utils import
check_output, where File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\skvideo\utils__init__.py",
line 10, in from .edge import canny File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\skvideo\utils\edge.py", line
8, in import scipy.ndimage File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\ndimage__init__.py",
line 161, in from .filters import * File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\ndimage\filters.py",
line 37, in from scipy.misc import doccer File
"C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib
site-packages\scipy\misc__init__.py", line 67, in from
scipy.interpolate.pade import pade as pade File
"C:\Users\Hasan\AppData\Local\Programs\Python\Python36\lib
site-packages\scipy\interpolate__init.py", line 175, in from
.interpolate import * File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\interpolate\interpolate.py",
line 20, in import scipy.linalg File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\linalg__init__.py", line
186, in from .misc import * File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\linalg\misc.py", line 5,
in from .blas import get_blas_funcs File "C:\Users\Hasan\AppData\Local
Programs\Python\Python36\lib\site-packages\scipy\linalg\blas.py", line
196, in from scipy.linalg import _fblas ImportError: DLL load failed: The
specified module could not be found.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scikit-video/scikit-video/issues/58, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIAHzVmx8o53v23orjxVB1uqlEs5Hi4Uks5stcFrgaJpZM4P9ZlT
.


500 internal server error when passing a array of numbers into day attribute

I was trying to make this work for the custom meeting finder page on our site nvintergroup.org/meetings/
While testing I kept getting 500 errors and I eventually found that it was caused by trying to pass in an array of numbers into the day attribute.  For the Meeting Guide API they accept it in this format, but for this pdf builder it does not.  You might want to make that distinction, as well as any other attributes between the APIs that may not line up anymore.
Thanks again for creating this, once I sorted it out it's working great!

we should fix it so it supports day as an array - thanks for spotting that!

Hi @dashmurphy!
I'm the one of the maintainers for PDF. I'm going to explore this and get it fixed. Would you be able to send me a sample of the feed that was throwing the error or a url?
Thank you!


Publish for ScalaCheck 1.14

It's already that way in master, but there's no published release which supports ScalaCheck 1.14.

Looks like 1.14 is published for 2.12 but not 2.11.

@LeifW as "com.github.alexarchambault" %% "scalacheck-shapeless_1.14" % "1.2.0-1" exists, I guess this can be closed?


bugfix for crane-agent create nodeResourceTsp

What type of PR is this?
bugfix for crane-agent create nodeResourceTsp

/LGTM


Uncontrolled Usage

What are your thoughts on allowing the datepicker to be used as an uncontrolled element, https://facebook.github.io/react/docs/forms.html#uncontrolled-components?
Would allow you to;
var moment = this.refs.datepicker.getValue();

Instead of listening on the event for changes.

Hmm, interesting approach. I guess I initially made it a controlled input to allow for masking/styling of the input while typing (--/--/----), but haven't gotten to implementing that yet. What's the reason for you to make the datepicker uncontrolled?

This will just give it an option to be used in both controlled and uncontrolled. Our main requirement was to be able to get the value out of the datepicker and there was no link to the date input.

Thanks! I've merged the PR to master, will perform a release later today.

Thanks Remon for your great component :+1:


Examples? such as a devise example

I wondered if there should be an examples directory that had some different example deployments. It might be "handy" for new implementations? For example. We had an "existing" rails application that uses "devise" for user authentication / management. We just wanted to add the ability for it to be the IdP. It turns out that is super easy, once you have fought it long enough ;)
Devise Example

https://github.com/plataformatec/devise

app/controllers/saml_idp_controller.rb
class SamlIdpController < SamlIdp::IdpController
  before_filter :authenticate_user!, except: [:show]

# override create and make sure to set both "GET" and "POST" requests to /saml/auth to #create
  def create
    if user_signed_in?
      @saml_response = idp_make_saml_response(current_user)
      render :template => "saml_idp/idp/saml_post", :layout => false
      return
    else
      # it shouldn't be possible to get here, but lets render 403 just in case
      render :status => :forbidden
    end
  end

# NOT USED -- def idp_authenticate(email, password) -- NOT USED

  def idp_make_saml_response(found_user) # not using params intentionally
    encode_response found_user
  end
  private :idp_make_saml_response
end

config/routes.rb
(add the following)
# SAMLv2 IdP
  get '/saml/auth' => 'saml_idp#create'
  post '/saml/auth' => 'saml_idp#create'
  get '/saml/metadata' => 'saml_idp#show'

config/initializers/saml_idp.rb
Add this file per the README. Note that it does require customization.

You must set "base" and it should probably be something like base = Rails.configuration.x.saml_idp_base which should be set as config.x.saml_idp_base = https://www.example.com/ in your config/environments/${RAILS_ENV}.rb.
The SSL KEY should probably be in secrets.yaml (and not in version control).
The config.name_id_formats must be set (watch out, the example has a # before the =)
Also, for config.service_provider.persisted_metadata_getter, see Issue #16 just in case that hasn't been fixed yet.

I love this idea! Even just a Devise example would be a great start for a ton of people I would think.
If you want to submit a PR that adds this directory and outlines how to do this within one file or a set of files, that'd be great.

I am thinking a directory of markdown docs, sort of like above, provide a fairly simple way of sprinkling documentation and code together into a single file. I might add Prerequisites or something?  I had to go through a lot of stack traces to figure out what was going wrong. It kept trying to call ".persistent" against my User object, and I couldn't figure out why :). Anyhow, I will see if I can carve off another half hour or something to polish this up.

That'd be Great, thanks a bunch for anything you can contribute!

Would also be nice to provide examples on how to test this with rspec/minitest.  Running in circles trying to write tests for the IDP controller.

Did any of these examples ever materialise? I'm finding it difficult to understand the configuration, especially of the initialiser in which the examples seem to be strangely commented.
At the moment my service provider is never trying to get fresh metadata, even though there is no persisted metadata. Validation of the request then fails due to missing acs_url. I've added a base, defined the service provider name and metadata URL, and tried to set the name_id.formats but couldn't fathom the syntax.
Appreciate any help or examples!
-Matt

Also note that the Devise Example above only works if using the SAML Http redirect protocol.  If using the HTTP post method this does not work, as you will never be redirected back to the sp.

@TJM : Did you solve the ".persistent" undefined method for the User model object in Devise?

@rayson1223 - I will admit that I am the "IT" guy, and I was just part of the project to help with the SAML stuff. I know "it works" for us, but use it at your own risk. The only place I see "persistent" in our initializer is:
  config.name_id.formats =
    {                         # All 2.0
      email_address: -> (principal) { principal.email },
      transient: -> (principal) { principal.id },
      persistent: -> (p) { p.id },
    }

There is also some stuff further down about metadata, but I think its unrelated.

What is the reason of rendering that template?
render :template => "saml_idp/idp/saml_post", :layout => false ?

I've found an issue with this type of Devise usage. When you use:
before_action :authenticate_user!

To redirect incoming SAML auth requests to Devise's session controllers, you will easily exceed the cookie size limit if the SP is sending a signed authn request. For example, I'm using devise + omniauth-saml on the SP side with the following rather ordinary settings (Gitlab uses similar settings):
security: {
  authn_requests_signed: true,
  logout_requests_signed: true,
  logout_responses_signed: true,
  want_assertions_signed: true,
  metadata_signed: true,
  embed_sign: true,
  signature_method: XMLSecurity::Document::RSA_SHA256,
  digest_method: XMLSecurity::Document::SHA256,
}

If authn_requests_signed is enabled, then before redirecting to Devise's sign in page, the URL containing the SAMLRequest parameter will fail to be stored in the session for use as return URL. This will be too large to be persisted in a session cookie jar.
Since saml_idp required signed logout requests to support SP-initated SLO, I've found that you have to at least exclude the logout action from authentication:
before_action :authenticate_user!, except: %i[logout]

I've not yet found a solution for enabling signed authn requests apart from switching to another type of session storage on the IdP.

@vipera were you able to figure anything out. I've got the same issue I believe.

@JohnKacz I have currently set authn_requests_signed to false in the SP configuration. This "fixes" the issue by shortening the SAMLRequest payload. My IdP is currently used only by internally developed SP applications, so I'm permitting unsigned requests from those sources.
Alternatively, you could look into using Memcached or Redis as the session store within the saml_idp application. Then you should not run into this problem as you can store more data in the session store. I understand that CookieStore has its benefits and that this might not be practical if you're not already using either for other caching.
So basically I didn't manage to get to the bottom of it, I worked around it. I didn't have the time to tackle it properly, but if anyone has any pointers I'd be glad to try and make it work with cookie-stored sessions.

@JohnKacz, @vipera
I'm not sure why authn_requests_signed is affecting failure of the non authenticated SAML request. We need to do more investigation about this issue. If you guys could provide more information like logs or whatever to help to speed up the investigation that would be great.
About cookie store issue is cookie only allows to store 4kb but the SAML request is too bug to store it. Instead of that you guys can store SP initiated SAML request URL (not request itself) for SAML request and let "devise" redirect back to SP url again after successfully logged into your IdP.

I had the same issue and wasn't able to get devise to help so I modified the validate_saml_request before action and set the SAMLRequest to the session on post and then pulled it back out on get.
def validate_saml_request
      if request.get?
        params["SAMLRequest"] = session["SAMLRequest"]
      elsif request.post?
        session["SAMLRequest"] = params["SAMLRequest"]
        store_location_for("user", "#{request.base_url}/saml/auth") if current_user.blank?
      end

      super
    end

I'm on a forked branch of 0.9.0

@mrobock your code has a hint. I would like clarify one thing who also looking for some solution.
SAML has also specification GET request (not only POST), people who uses your way need to be aware of SP is only support POST request.
And I think really don't need to override the method since you have full management of your own session controller, or application controller of a Rails app.


Plugin configuration with environment variable not updated

Describe the bug
I have jenkins with datadog plugin.
My jenkins instance is dockerized on kubernetes cluster and deployed by chart helm.
I configure the following env var of my container :
DATADOG_JENKINS_PLUGIN_REPORT_WITH : DSD
DATADOG_JENKINS_PLUGIN_TARGET_HOST : ip of my kubernetes node, dynamically resolved
DATADOG_JENKINS_PLUGIN_TARGET_PORT : 8125
If i deploy a fresh jenkins without plugin configuration, the configuration is ok, i can see it via jenkins administration UI.
If i redeploy my jenkins and change the value of DATADOG_JENKINS_PLUGIN_TARGET_HOST , the configuration is not updated.
And another thing, if i configure the first time manually on admin panel UI and then i set environments and restart the container, the configuration on the UI is not updated with the environment values.
To Reproduce
Steps to reproduce the behavior:

Deploy jenkins from scratch with datadog plugin on docker with env (DATADOG_JENKINS_PLUGIN_REPORT_WITH...)
Go to the admin datadog panel and check the configuration is ok
Change the value of the env (DATADOG_JENKINS_PLUGIN_TARGET_HOST) and redeploy the container
Go to the admin datadog panel and see the configuration of target host has not been updated

Expected behavior
The jenkins datadog plugin configuration is updated with correct value.
Maybe the behaviour is normal but in that case, the plugin documentation is not clear about that : https://docs.datadoghq.com/integrations/jenkins/
Environment and Versions
jenkins 2.176.3 with datadog plugin 1.1.2 dockerised and hosted on kubernetes (aks cluster)

Hey @gregoryboue
That behavior is not expected, environment variables should take precedence over the local configuration. I'll try to reproduce but I assume that if you delete the org.datadog.jenkins.plugins.datadog.DatadogGlobalConfiguration.xml in the jenkins root dir before starting jenkins you should see the environment variables being used correctly again, correct?
Seems that it can be a small bug in the configuration loading logic, thanks for reporting 👍

I was able to reproduce and it was indeed an issue with the local configuration taking precedence over environment variables. I have merged a PR to fix this, we'll release a new version of the jenkins plugin soon with that change 👍

@FlorianVeaux Great, thanks you !!


Improve notification on no node found situation

In the case of no connection:
do not dispay the full screen modal - but instead rely on the network-connectivity-bar to change states from green to orange and display 1 non-invasive popup that says "we're going to keep trying to connect to a good node for you. in the mean time, your experience may be degraded and we would recommend against making any transactions".
also: prevent notification popup spam like here:

Duplicate of https://github.com/cosmos/voyager/issues/1511.

This is split out from that issue.

Oh, sorry.

Change naming to "Node is not reachable"
Add option to retry or use mocked version

Working on this in #1533

Done in #1533


ListItem - trailingAccessoryContent with Button causes jank

We've been seeing jank in our list of items (with hundreds of items), and tracked it down to the trailingAccessoryContent. Removing trailingAccessoryContent makes the list scroll very fast. Our trailingAccessoryContent is also a single Fluent button, so it's really nothing super intense, and I was also able to repro with a vanilla Material button with not much in it. Interestingly, using a text view of a constant string ("hi") is also very fast.
From the Profile Trace via AS, it says the "measureAndLayout()" step is taking a long time, which is what also led me down this investigation path.
< Fluent Version 1.30.0 >
Behavior

Case 1 - TrailingAccessoryContent with Fluent Button - Jank

trailingAccessoryContent = { Button(onClick = { }, text = "hi") { }}

Case 2 - TrailingAccessoryContent with material Button - Jank

trailingAccessoryContent = { androidx.compose.material3.Button(onClick = { /*TODO*/ }) { }}

Case 3 - TrailingAccessoryContent with Text("hi") - No Jank

trailingAccessoryContent = { Text("hi")}

Case 4 - No TrailingAccessoryContent - No Jank

Relevant code / code in question - ListItem, line 411
            if (trailingAccessoryContent != null && textAlignment == ListItemTextAlignment.Regular) {
                Box(
                    Modifier
                        .padding(
                            top = if (trailingAccessoryContentAlignment == Alignment.Top) padding.calculateTopPadding() else 0.dp,
                            bottom = if (trailingAccessoryContentAlignment == Alignment.Bottom) padding.calculateBottomPadding() else 0.dp,
                            end = padding.calculateEndPadding(LocalLayoutDirection.current)
                        )
                        .fillMaxHeight(),
                    contentAlignment = trailingAccessoryAlignment
                ) {
                    trailingAccessoryContent()
                }
            }

@caltseng I can confirm that the jank is present in the latest verison of fluent for list item with a trailing accessory item. Interestingly, it even persists when I use a text element with a colored box around it, but does not persist when using only a Text element. The jank is also present when a leading accessory item is used.
I also investigated the logic for alignment of the trailing item, as you suggested, and making changes to it does not help eliminate the jank either. Going to investigate this further. Please let me know if you have had any new insights about the same.


[BUG] Dropdown component not in UI layer

Details:
Jotunn Version: 2.10.4
Jotunn Submodule(if applicable): -
Repeatability(Consistent(100%), Inconsistent(50%), Rare(1%), Unknown(==1)):  100%
Problem Description:
The Dropdown component of the gameobject created by GUIManager.CreateDropDown has the incorrect layer set. It is 0, where 5 (UI Layer) is expected.
Expected Behaviour:
var dd = GUIManager.Instance.CreateDropDown( ... )
dd.GetComponent<Dropdown>().gameObject.layer == 5
Actual Behaviour:
var dd = GUIManager.Instance.CreateDropDown( ... )
dd.GetComponent<Dropdown>().gameObject.layer == 0
Link to your server's and client's LogOutput.log
It's not visible in any logs, but a user of my mod has posted a screenshot here: https://github.com/SpikeHimself/XPortal/issues/11

Additonal findings:
Same as above applies to the dropdown's "Template" child, i.e.
var dd = GUIManager.Instance.CreateDropDown( ... )
dd.transform.Find("Template").gameObject.layer == 0  // Should be 5

Changing both of these values to 5 (i.e. GUIManager.UILayer) solved the issue for my mod's user.

Thanks for the report. It's actually interesting that it behaves like this, since the dropdown is initally created with Unity's own utility class DefaultControls.CreateDropdown() that even seems to set the layer correctly for some children.
Agreed, it makes sense that the controls are always spawned under the correct UI layer. As far as I can tell you already fixed it in your mod but feel free to open a PR if you want to provide a fix in Jotunn, too. Otherwise I will put it on my to-do list and get to it when I get to it.


[BUG] Heroku Connection Refused

Describe the bug:
There is an issue with Heroku and opening a port for sending an email.
To Reproduce:
Steps to reproduce the behavior:

Go to Time Off form
Fill the form out
Click submit
See error

Expected behavior:
There should be a page showing successful submission of a form
Actual Behavior:
An error appears saying the connection is refused
Screenshots:

Environment:
Heroku
Possible Fix
Contact Heroku to open a port OR use a third party add on or pluggin

This issue has been closed.


Why inappwebview does not support desktop

I noticed that the existing webview packages can only be used as widgets on mobile phones, and can only be used as new windows on the desktop. Why can't they be used as widgets on the desktop? Is it because Flutter itself does not support it?

Exactly, the Flutter team is working on it. Maybe we will have that with Flutter 4 🤷🏻♂

Exactly, the Flutter team is working on it. Maybe we will have that with Flutter 4 🤷🏻♂

Thank you. I don't know much about ios development. I want to ask a question. Can this effect be achieved by embedding web pages through WKWebView in macos?

InAppBrowser and HeadlessInAppWebView are already implemented on macOS, check the API Reference to see what is supported on macOS


[Request] Changed expand direction of bottom page select dropdown.

Changed expand direction of bottom page select dropdown to be 'top'. It seems easier to read.
Preview

Nice found! Closing this PR and merging this into next commit


bower-components-path default

bower-components-path should first check for a .bowerrc in the project root or use bower_components as the default

Going to leave this up to the user or calling generator as parsing bowerrc may get messy


Remove preflight checks as part of crc daemon

Why do we have preflight checks for crc daemon this is always block in case crc daemon is started as standalone process after #3088 and #3072. Also crc daemon doesn't need to have any checks before it can run.
After following PR merge if dev member try to do following then it will fail to debug the daemon api.
> .\crc.exe daemon
INFO Checking if running in a shell with administrator rights
INFO Checking Windows 10 release
INFO Checking Windows edition
INFO Checking if Hyper-V is installed and operational
INFO Checking if crc-users group exists
INFO Checking if current user is in Hyper-V Admins group
INFO Checking if Hyper-V service is enabled
INFO Checking if the Hyper-V virtual switch exists
INFO Found Virtual Switch to use: Default Switch
INFO Checking if vsock is correctly configured
INFO Checking if daemon task is installed
exit status 1

the daemon starts slowly due to these checks; the tray has a delay of 4 to 6 seconds for this. after the removal of these preflights for the daemon we can check if the tray can start quicker,

crc daemon doesn't need to have any checks before it can run.

Why? When is the daemon going to validate the OS is setup properly for crc use?

crc daemon doesn't need to have any checks before it can run.

Why? When is the daemon going to validate the OS is setup properly for crc use?

@cfergeau do we really need that? Daemon is separate process and doesn't need any setup to run. It is not depend on preflight to run correctly right? crc setup and crc start require preflight check because these command depend on it not the setup like  other subcommand version, oc-env ..etc?

the daemon can run independent of a crc setup. the actual check happens when you perform a crc start.

Yes, this is the background info that was missing, that the needed checks will be ran as part of crc start anyway.

After following PR merge if dev member try to do following then it will fail to debug the daemon api.

Fwiw, the right skip option can be set for this. This is not a very compelling reason for removing the checks ;)

It is removed with https://github.com/code-ready/crc/pull/3088/commits/458237512bd1d22a2fce3e44f5c24ff279d9a475


Implement open_channel message

Adds type and implements codec.
Closes #110

Codecov Report

Merging #149 into master will increase coverage by 0.03%.
The diff coverage is 97.10%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
+ Coverage   95.00%   95.04%   +0.03%
==========================================
  Files         141      143       +2
  Lines        3745     3814      +69
  Branches      435      439       +4
==========================================
+ Hits         3558     3625      +67
- Misses         93       94       +1
- Partials       94       95       +1

Impacted Files
Coverage Δ

packages/wire/lib/messages/OpenChannelMessage.ts
96.96% <96.96%> (ø)

packages/wire/lib/MessageType.ts
100.00% <100.00%> (ø)

packages/wire/lib/flags/OpenChannelFlags.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


babelify dependency

Hi!
Is this project really need babelify package as a production dependency?
PR by @oliverwoodings: Fix babelify dependency. Add support for react-bootstrap

This was done to stop downstream projects using browserify throwing errors about babelify not being found. Actually I think we can avoid this entirely by just changing the test setup (since we don't need browserify for anything but testing). Thanks for pointing it out!

See #25

Thanks! :+1:


Create support for undo and redo commands

Codecov Report

Merging #285 into master will increase coverage by 0.21%.
The diff coverage is 40.25%.

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


Fix TODO double check attachment

Summary

Fix TODO double check attachment
The device element on AttachVolume response returns dummy value, so it could not check attachment device by DescribeInstanceAttribute.
Instead, check by InstanceUniqueId and VolumeUniqueId

Review

[x] All checks have passed

Thanks! It works!! 🚀


[SILGen] Fix leak in thunk for async throwing swift conformance to ObjC requirement with optional completion.

Fix a leak when emitting the native to foreign thunk for an async function which fulfills an Objective-C protocol requirement which can be fulfilled with either a value or an error via a nullable completion.
Previously, the SIL in question used to look like this:
  %maybe_completion = ...
  try_apply %impl..., normal success, ...

success(%value):
  switch_enum %maybe_completion...
      case some!enumelt: invoke
      case none!enumelt: ignore

ignore:
  br join

invoke(%completion):
  %some_value = enum Optional, some!enumelt, %value // consumes %value
  %guaranteed_some_value = begin_borrow %some_value
  %none_error = enum Optional, none!enumelt
  apply %completion(%guaranteed_some_value, %none_error)
  end_borrow %guaranteed_some_value
  destroy_value %some_value
  br join

join:
  destroy_value %maybe_completion
  ...

which leaks %value on the codepath through ignore.
Note that %value is consumed by the enum instruction, but %completion is invoked with %guaranteed_some_value, a guaranteed value.  So there is no need to consume %value in invoke.
Here, %value itself is borrowed and forwarded into an enum instruction whose result is passed to %completion:
  %maybe_completion = ...
  try_apply %impl..., normal success, ...

success(%value):
  switch_enum %maybe_completion...
      case some!enumelt: invoke
      case none!enumelt: ignore

ignore:
  br join

invoke(%completion):
  %guaranteed_value = begin_borrow %value
  %guaranteed_some_value = enum Optional, some!enumelt, %guaranteed_value
  %none_error = enum Optional, none!enumelt
  apply %completion(%guaranteed_some_value, %none_error)
  end_borrow %guaranteed_some_value
  br join

join:
  destroy_value %maybe_completion
  destroy_value %value
  ...

Because an argument scope was already being created and a cleanup was already being pushed for %value, nothing more is required to fix the issue than to reorder the enum and the borrow.
rdar://119732084

@swift-ci please test

@swift-ci please test linux platform

@swift-ci please test windows platform

@swift-ci please test


keep date, time, on the same line when in a map

keep date, time, on the same line when in a map, fixes #207
Requirements

[x] Entry in CHANGELOG.md was created
[x] Functionality is covered by newly created tests

Pull Request Test Coverage Report for Build 2f0a8bb9b791d42e270799a83744590b7c652e18-PR-208
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build a373121a75a735315800ae0698d4ca37ed1a0f04:
0.0%

Covered Lines:
121

Relevant Lines:
121

💛  - Coveralls


Dependency injection by incident

Regard the demo in this repo about GithubProvider
public enum GitHub {
    case Zen
    case UserProfile(String)
    case UserRepositories(String)
}

extension GitHub : MoyaTarget {
...
}

Normally, we would pass the MoyaTarget instance to the MoyaProvider (the enum Github in this case, but it could be struct, class, whichever conforms to MoyaTarget), like this
let GitHubProvider = MoyaProvider(Github.Zen)

but the way you use generic,
let GitHubProvider = MoyaProvider<Github>()

I find this interesting because we can leverage generic and still can call methods on the MoyaTarget, without passing the instance into MoyaProvider

The MoyaTarget never gets passed the the provider directly. You call provider.request(Github.Zen). Could you elaborate on what the issue is?

Well, it gets passed in to the Initializer – the MoyaProvider is a generic type on MoyaTarget, so it needs to know what the type is. I'm not sure if MoyaProvider(GitHub.Zen) would work, but I haven't tried it. There's no initializer that it could call – or is this a Swift thing I'm not aware of?

@petester42 @ashfurrow this is not an issue. Maybe I misunderstand the code

@onmyway133 issues are for questions and clarifications, too! I want to make sure that we're on the same page. Let's start over:
What is happening that you don't expect? What is not happening that you do expect?

I just misunderstood the code. Now I'm realize that MoyaProvider uses generic on MoyaTarget, and indeed, MoyaTarget instance is passed into MoyaProvider 's request method

Yup! And you can only pass enum values of the correct type to the request method, since they're compile-time checked 🎉

Closing, as I think this is sorted :+1:


Will this project be updated?

@lettertwo, or @johnetrent as another member of this project's organization, are there any plans to update this project?

Given the status of this repo (no update in 2 years) I have forked and made some design changes & fixed deprecations which I think are a big improvement https://github.com/DeedMob/react-load-image

version 3.0.0 is on npm.


Go mod file is invalid in v2

https://github.com/prometheus/prometheus/blob/v2.21.0/go.mod
Is wrongly declaring the module. It's missing a v2 from its end. Which makes the build fail:
<EMAIL_ADDRESS>invalid version: +incompatible suffix not allowed: module contains a go.mod file, so semantic import versioning is required

Hi :wave:
Yes, and there is lot's of reasons why this is this way. You read all here: https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/prometheus-developers/F1Vp0rLk3TQ/F412gRw0CAAJ
TL;DR: It's as easy as putting GIT-SHA or v2.21.0 as version of module, run for Prometheus go mod tidy and it will work. The reason why it's confusing is that we introduced Go modules while the project was in v2 version already.
At the end, it will look something like github.com/prometheus/prometheus v1.8.2-0.20200819132913-cb830b0a9c78
And this is fine, it's not v1.8.2 it's might actually higher version - you can see the date and actually commit hash cb830b0a9c78. This is how Go module works right now and we are looking for good solution here.
Hope that helps :hugs:

@bwplotka
This would help if I would directly require Prometheus. But it's coming in as a dependency from something.

Hm. I would contact the dependency author and put desired Prometheus version in "replace" section of go mod file. See https://github.com/thanos-io/thanos/blob/830ac35a57602ae890dc195095bf58b831854bb7/go.mod#L80

Now to find, who the heck is the dependency which brings it in.... :))

Thanks, this worked btw->
replace github.com/prometheus/prometheus => github.com/prometheus/prometheus v1.8.2-0.20190424153033-d3245f150225

Should be the right version...

You can use go mod why github.com/prometheus/prometheus

go: finding module for package github.com/prometheus/prometheus
# github.com/prometheus/prometheus
(main module does not need package github.com/prometheus/prometheus)

:D


Have overlays look for container div with default id

Helper libraries such as single-spa-vue, single-spa-angular, and single-spa-react all implement a default dom element getter now that appends a div to the dom with an html id that follows a certain convention.
Single-spa-inspector could check if that html id exists when looking for a div to append overlays to. This would be awesome because then single-spa users don't even have to do export const devtools from their apps. See related https://github.com/CanopyTax/single-spa-react/issues/51.

ooh that's a great idea. Should be pretty easy as well.
I'll add it in when I get back while also taking a look at implementing some override behaviors in the extension.

@frehner any update on this? I am keen on seeing this in. If you can't get to it soon I might have some time for it.

Ah I forgot about it.

I might have some time soon to work on this as well.


Update authenticate-user.js

Would it be better?
Fixes #[replace brackets with the issue number that your pull request addresses].
Changes proposed in this pull request:

[list out summary of changes here]
[list out summary of changes here]
[list out summary of changes here]
[etc]

Note: pull requests without proper descriptions may simply be closed without further discussion. We appreciate your contributions, but need to know what you are offering in clearly described format. Thanks! (you can delete this text)
cc @HospitalRun/core-maintainers

@fooksupachai if you run ember test locally on your changes, you will see the following error:
not ok 250 Chrome 56.0 - ESLint - helpers/authenticate-user.js: should pass ESLint
    ---
        actual: >
            false
        expected: >
            true
        stack: >
                at Object.<anonymous> (serve://dist/assets/tests.js:11392:12)
        message: >
            helpers/authenticate-user.js should pass ESLint.
            5:1  - Always use `const` when making variables from Ember properties (ember-suave/require-const-for-ember-properties)
            5:1  - Unexpected var, use let or const instead. (no-var)
            5:23  - Unexpected whitespace before semicolon. (semi-spacing)
        Log: |

The reason for this is that the changes you made do not comply with the style guides HospitalRun uses:
https://github.com/dockyard/styleguides/blob/master/engineering/ember.md
and
https://github.com/dockyard/styleguides/blob/master/engineering/javascript.md
This change reverses those style changes explicitly chosen and therefore I am going to close it.


message : "Assertion Failed: A helper named "moment" could not be found"

Thanks for the ember-moment. I don't understand why this helper is no default in Ember?
I installed ember-moment with command: ember install ember-moment. When i use helper {{moment date_created 'LLL'}} or {{moment date_created 'DD MM YYYY'}} ember gives an error message: "Assertion Failed: A helper named "moment" could not be found". But the {{moment-format date_created 'LLL'}} works correctly.
Information about use the ember-moment helper I read on the page https://github.com/stefanpenner/ember-moment#helpers. Is this incorrect?

What version of ember moment are you using

+1
    "ember-moment": "6.1.0"

The moment helper is only on 7.0.0.  Which is in beta but hopefully soon I'll release.

7.0.0 was just released


4 Mods and this one is the problem.

I'm attempting to use the BitsAndChisels mod with 3 mods, one of which is FabricAPI. The other 2 are ImmersivePortals and the Pehkui mod. The game kept crashing until I removed the BitsAndChisels mod and here is the error log. Note, I am using Fabric 1.16.4 and BitsAndChisels 1.1.5.
The game crashed whilst initializing game
Error: java.lang.RuntimeException: Could not execute entrypoint stage 'main' due to errors, provided by 'imm_ptl_core'!
Exit Code: -1
crash-2021-04-09_16.12.00-client.txt

I updated to 1.16.5 and it fixed the issue. Doesnt matter


Dealing with Null responses

Hi everybody,
I hope this is the best place to post this, if not my apologies.
We are owning a Web Api both client and server side, and we developed a method that can return a null answer in some cases

The associated Json documentation

However, when the client is generated with Nswag :

There is a mandatory null check in case of a 200 response
A possible 204 answer is not taken into account.

I did try to add a 204 attribute server side, and this does add a 204 status check on the server side, but within the 204 check, it will nevertheless throw an exception

I am starting to get a bit out of ideas and I have to admit i am not so familiar with the swagger/swashbuckle/Nswag ecosystem.I did try to google it a bit and i found those (one year) old answers but I was wondering if there was any workaround at the moment?
https://github.com/RicoSuter/NSwag/issues/1602
We do not partcularly need to return a 204 answer in case of a null content (ASP does it automatically), so if there is a way to return a null 200 response and to suppress the null check for this method, I will take it.
So far, we are writing a partial class client with a specific 204 handling case, but we need to do some tweaking everytime we refresh the client. This works, but this is not ideal.

Thank you very much for your help,
Kind regards

I think this is just a continuation of https://github.com/RicoSuter/NSwag/issues/2995


Added Group ping

Pretty straightforward

@dignifiedquire Fixed with updated comments. RFM?

LGTM

Thanks, @dignifiedquire!


Remove migration info from Windows 8 to 8.1

The [GridView] class described here is for the UWP platform which is exclusive to Windows 10. However, there is a short paragraph dedicated to porting an app from Windows 8 to Windows 8.1.
More than 3 years into Windows 10's life, I guess it is reasonable to assume that no one is porting apps from Windows 8 to Windows 8.1 or to Windows 10 any longer. Hence, any such reference to Windows 8 (8.1) should be removed from this UWP documentation as it is unnecessary information.

@jwmsft : PR passed the acceptance checklist. Please evaluate further and approve/merge if appropriate. Thanks!

See conversation on PR #784


Fixed a problem where recess would fail if directories exist inside the ...

...lib/lint or the lib/compile directories. This breaks for SVN and CVS users who keep recess in their repos.

 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.

We are no longer maintaining this project and as such are arching all PRs.


ci(tools): clean up github actions

Gather all lint checks into one action and parse them with a problem-matcher. This way we make good use of the pnpm run lint script.
There are issues with setup-node default problem-matcher for eslint so we include a custom one.

See this issue for the problem matcher: https://github.com/actions/setup-node/issues/96#issuecomment-1471014244


Build failure using Ninja generator

With Xcode 10 and 10.2, this demo or any one line swift code will fail to build. Xcode generator works.
The error message:
ninja: Entering directory `out/'
[1/1] Linking Swift executable Test0
FAILED: Test0 CMakeFiles/Test0.dir/src/main.swift.o Test0.swiftmodule
: && /usr/bin/swiftc -output-file-map CMakeFiles/Test0.dir/output-file-map.json -incremental -emit-executable -o Test0 -emit-module -emit-module-path Test0.swiftmodule -emit-dependencies    ../src/main.swift    && :
:0: error: unable to load output file map 'CMakeFiles/Test0.dir/output-file-map.json': No such file or directory
ninja: build stopped: subcommand failed.

Need to add in
if(NOT XCODE_VERSION VERSION_LESS 10.2)
set(CMAKE_Swift_LANGUAGE_VERSION 5.0)
elseif(NOT XCODE_VERSION VERSION_LESS 8.0)
set(CMAKE_Swift_LANGUAGE_VERSION 3.0)
endif()
Ref: https://github.com/Kitware/CMake/blob/master/Tests/SwiftOnly/CMakeLists.txt

If you have to do that, it sounds like you are using the Xcode generator.  How did you configure the project?

This issue seems to be caused by running cmake in a directory other than the build directory (and CMake placing files relative to the current rather than build directory). For example,
cd build
cmake -G Ninja ../ -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_TESTING=YES

works (although the tests still fail), but this doesn't:
cmake -G Ninja -B build -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_TESTING=YES

I wonder why the -B option doesn't work.  Does that work for other builds and not just swift?  If so, that might just be a bug in CMake that needs to be addressed.  But, yes, I believe that you are correct about it being run from the wrong directory.  I'm going to close this out.


Run rake fmt and fix Lint and format text build

Fixes a build introduced in #2. Will rebase there to pick up this fix.

Fixes a build introduced in #2. Will rebase there to pick up this fix.


Catch only specific error

Here we use a blanket catch statement.
Problem: It swallows all errors
Solution: Catch only specific errors

can we use multi catch or if statement with catch like in C#?

@aloisdg I will probably use an if statement.

Removing help wanted for now as I am refactoring the code.


creates unresponsive blank page in electron

On this line:
workbook.xlsx.writeFile(path)

This happens in  the electron 0.37.8 release package on Windows 10 and OsX (10.11.4):

window turns white / blank
code following the line is not run
the window title remains and the page can be reloaded (Ctrl + r), so I guess no new BrowserWindow was opened

Oddly enough this works fine in dev mode.
The same happens when I try to use XLSX (https://github.com/SheetJS/js-xlsx) instead on the corresponding line:
XLSX.writeFile(wb, path, { bookType: 'xlsx', bookSST: true, type: 'binary' })

So I guess it will probably be an issue of electron more than exceljs or XLSX. Unfortunately the issue (https://github.com/electron/electron/issues/5174) was closed by electron devs as being a third party issue.
I would be very grateful for any pointers / help.

@zcbenz helped me find a solution:
run workbook.xlsx.writeFile(path) in a separate process using child_process.fork (https://nodejs.org/api/child_process.html#child_process_child_process_fork_modulepath_args_options)


Makefile updates

Makefile: build and clean targets, no -v in install
Added build and clean targets to follow Makefile idioms. Removed verbose flag from install target

ok to test


[Parallel testing] PDOException: There is already an active transaction

Laravel Version: 8.33.1
PHP Version: 8.0.3
Database Driver & Version: MySQL 5.7.32

Description:
Greetings, we want to test multiple database interactions, so we need isolated tests and a way to create / migrate / seed replicas of our mysql database (disk storage). We are using Sanctum for authentication.
I've been trying to use the new parallel testing functionality and read your different docs and issues about it.
My conclusions are:

the DatabaseTransactions trait is needed, the others are "replaced" in whole or in part by ParallelTesting
the new ParallelTesting::setUpTestDatabase() is exactly what I need to seed the databases at creation
no test db definition required as ParallelTesting will already create copies of the form name_test_1, name_test_2
without --recreate-databases, seeding won't be called again as it is only called after its test db is created

My seeds are a bunch of factories, DB::unprepared.
Running art migrate:fresh and art db:seed manually then art test without --parallel, tests work fine. However when running with --parallel --recreate-databases, the seeding does not work and I stumble upon this error:

PDOException: There is already an active transaction

EE                                                                  2 / 2 (100%)

Time: 00:14.187, Memory: 20.00 MB

There were 2 errors:

1) Tests\Feature\AuthTest::testAuthLogin
PDOException: There is already an active transaction

[root]/vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:137
[root]/vendor/laravel/framework/src/Illuminate/Database/Concerns/ManagesTransactions.php:113
[root]/vendor/laravel/framework/src/Illuminate/Foundation/Testing/DatabaseTransactions.php:21
[root]/vendor/laravel/framework/src/Illuminate/Foundation/Testing/TestCase.php:129
[root]/vendor/laravel/framework/src/Illuminate/Foundation/Testing/TestCase.php:90

2) Tests\Feature\AuthTest::testAuthGetUser
Illuminate\Database\QueryException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'company_id' cannot be null (SQL: insert into `users` (`company_id`, [cut]))

[root]/vendor/laravel/framework/src/Illuminate/Database/Connection.php:678
[root]/vendor/laravel/framework/src/Illuminate/Database/Connection.php:638
[root]/vendor/laravel/framework/src/Illuminate/Database/Connection.php:472
[root]/vendor/laravel/framework/src/Illuminate/Database/Connection.php:424
[root]/vendor/laravel/framework/src/Illuminate/Database/Query/Processors/Processor.php:32
[root]/vendor/laravel/framework/src/Illuminate/Database/Query/Builder.php:2882
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php:1547
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php:1052
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php:1017
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php:858
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Factories/Factory.php:275
[root]/vendor/laravel/framework/src/Illuminate/Collections/Traits/EnumeratesValues.php:234
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Factories/Factory.php:278
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Factories/Factory.php:236
[root]/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Factories/Factory.php:230
[root]/tests/Feature/AuthTest.php:73

FAILURES!
Tests: 2, Assertions: 0, Errors: 2.

Providers/AppServiceProvider.php
<?php

namespace App\Providers;

use Illuminate\Support\Facades\Artisan;
use Illuminate\Support\Facades\ParallelTesting;
use Illuminate\Support\ServiceProvider;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        // Executed when a test database is created...
        ParallelTesting::setUpTestDatabase(function () {
            Artisan::call('db:seed');
        });
    }
}

tests/CreatesApplication.php
<?php

namespace Tests;

use Illuminate\Contracts\Console\Kernel;

trait CreatesApplication
{
    /**
     * Creates the application.
     *
     * @return \Illuminate\Foundation\Application
     */
    public function createApplication()
    {
        $app = require __DIR__.'/../bootstrap/app.php';

        $app->make(Kernel::class)->bootstrap();

        return $app;
    }
}

tests/TestCase.php
<?php

namespace Tests;

use Illuminate\Foundation\Testing\DatabaseTransactions;
use Illuminate\Foundation\Testing\TestCase as BaseTestCase;

abstract class TestCase extends BaseTestCase
{
    use DatabaseTransactions;
    use CreatesApplication;
}

migrations/*.php
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Support\Facades\DB;

class ImportDbTables extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        DB::unprepared(file_get_contents(base_path().'/database/dump.sql'));
    }
}

phpunit.xml
<?xml version="1.0" encoding="UTF-8"?>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:noNamespaceSchemaLocation="./vendor/phpunit/phpunit/phpunit.xsd"
         bootstrap="vendor/autoload.php"
         colors="true"
>
    <testsuites>
        <testsuite name="Feature">
            <directory suffix="Test.php">./tests/Feature</directory>
        </testsuite>
    </testsuites>
    <coverage processUncoveredFiles="true">
        <include>
            <directory suffix=".php">./app</directory>
        </include>
    </coverage>
    <php>
        <server name="APP_ENV" value="testing"/>
        <server name="BCRYPT_ROUNDS" value="4"/>
        <server name="CACHE_DRIVER" value="array"/>
        <server name="MAIL_MAILER" value="array"/>
        <server name="QUEUE_CONNECTION" value="sync"/>
        <server name="SESSION_DRIVER" value="array"/>
        <server name="TELESCOPE_ENABLED" value="false"/>
    </php>
</phpunit>

Example test
<?php

namespace Tests\Feature;

use App\Models\User;
use Laravel\Sanctum\Sanctum;
use Tests\TestCase;

class AuthTest extends TestCase
{
    public function testAuthLogin()
    {
        $user = User::factory()->create([
            'email' =><EMAIL_ADDRESS>            'password' => 'test_1',
        ]);

        $response = $this->postJson('/auth/login', [
            'email' =><EMAIL_ADDRESS>            'password' => 'test_1',
            'device_name' => 'test_1'
        ]);

        $response->assertOk()->assertJsonFragment([
            'status' => 'Success',
            'message' => null
        ]);
        $this->assertArrayHasKey('data', $response);
        $this->assertArrayHasKey('token', $response['data']);
    }
}

Any help would be greatly appreciated!

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

We are experiencing similar problem in our pipelines.
Running multiple parallel tests, which are using transactions often causes PDO exception.
I've debugged it havely and have suspicion that the cause is shared db session when running transactions from command line.
Parallel tests create separate processes per file, so if one file is smaller than the other the process is finishing when the other process is in transaction mode at the time.
PHP process, when closing clears all open connections by default and it looks like it clears transactions as well.
I still cannot create a single code to have it replicated all the time, but for our app almost every attempt ends with PDO exception for inactive transaction.


Fix mill build warnings about deprecated usages.

Feature Request
Description of Problem:
Since upgrading to mill 0.11.x some used types are deprecated, address these deprecations.
[#0] [warn]     Jvm.runLocal("mdoc.Main", cp, dirParams)
[#0] [warn]     ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:48:9: value Jvm in package modules is deprecated (since Mill 0.11.0-M9): use mill.util.Jvm
[#0] [warn]         Jvm.runLocal("mdoc.Main", cp, dirParams ++ Seq("--watch"))
[#0] [warn]         ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:56:26: type Sources in package define is deprecated (since Mill after 0.11.0-M8): Use mill.define.Target instead.
[#0] [warn]   def docusaurusSources: Sources
[#0] [warn]                          ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:57:22: type Sources in package define is deprecated (since Mill after 0.11.0-M8): Use mill.define.Target instead.
[#0] [warn]   def compiledMdocs: Sources
[#0] [warn]                      ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:77:[19](https://github.com/finos/morphir-scala/actions/runs/5615735487/job/15216636429?pr=265#step:5:20): value Jvm in package modules is deprecated (since Mill 0.11.0-M9): use mill.util.Jvm
[#0] [warn]     val process = Jvm.spawnSubprocess(
[#0] [warn]                   ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:153:14: value Jvm in package modules is deprecated (since Mill 0.11.0-M9): use mill.util.Jvm
[#0] [warn]     val p2 = Jvm.spawnSubprocess(
[#0] [warn]              ^
[#0] [warn] /home/runner/work/morphir-scala/morphir-scala/project/modules/docs.sc:169:5: value Jvm in package modules is deprecated (since Mill 0.11.0-M9): use mill.util.Jvm
[#0] [warn]     Jvm.runSubprocess(

Hi Damian. I made PR for this. Could you review it?
https://github.com/finos/morphir-scala/pull/287


FAP and FIR  not landing in filter_list under Experiment() after cast from xml

Working on branch fir_fap_filters

@kkappler Try using branch fix_issues I updated experiment to take in FAP and FIR filters.

@kkappler Can you add example stationxml's that have FAP and FIR filters to the data to the fix_issues branch

@kujaku11 I added two files, one from parkfield with FIR and another one that Tim used to test the ability to embed a FAP.

FIR is making it into the experiment.surveys[0].filter_list
@kkappler will check if/whynot these are being applied and comment here

They are landing in the filter list


[Mobile] - Disable button when no answer is selected or when answer has already been submitted

PR to address https://github.com/rightoneducation/righton-app/issues/523. Further comments/video capture has been added to the PR.

I think for the sake of tomorrow's build, I'm going to merge this PR. Let me know if there are any issues though, and I'll revert and correct


Windows: Command Failed at startup when the username contains ampersands

This happens as soon as you open the app:
Command failed: C:\Users\Foo & Bar\AppData\Local\Temp\9DB4.tmp.bat 'C:\Users\Foo' is not recognized as an internal or external command, operable program or batch file. The system cannot find the path specified. (code 1

This can be more easily reproduced by placing drivelist in a directory containing an ampersand and executing the example file:
git clone https://github.com/resin-io-modules/drivelist
move drivelist "Foo & Bar"
cd "Foo & Bar"
node example.js

This file is executed by child_process.execFile(). Simple tricks like quoting the whole path don't work. I also tried escaping the ampersand with ^&.
I did make it work by setting the execFile cwd option to the directory name of the script and then simply executing the basename, however that won't play well with asar archives. And to put another constraint to this: .execFile is the only spawning function we can use, since its the only one that is patched by Electron.

Blimey, sounds fairly nasty. Do we know if this has affected any users in the wild? I guess the nature of this bug (and the related ones) means that Etcher would fail before it gets the chance to log any errors?

FWIW, if I try replicating your example above on Linux (in /tmp), it works okay as long as I do npm install after doing cd "Foo & Bar". These all work:
node example.js
cd ..
node ./Foo\ \&\ Bar/example.js
node Foo\ \&\ Bar/example.js
node "Foo & Bar"/example.js
node "Foo & Bar/example.js"
cd ~
node /tmp/Foo\ \&\ Bar/example.js
node "/tmp/Foo & Bar/example.js"
node /tmp/"Foo & Bar"/example.js

and as we're not using any 'special' characters or variables here, these all work identically with double or single cream... I mean quotes! :wink:

Blimey, sounds fairly nasty. Do we know if this has affected any users in the wild? I guess the nature of this bug (and the related ones) means that Etcher would fail before it gets the chance to log any errors?

Yeah, I discovered this one through Sentry :/ The error happens after the app is started, so Corvus does get a chance to report it.

I've been attempting to dig into this some more, using drivelist as a nice & small test case...
(in the hope that this might also help with #1376 )
On Linux, it looks like I can use any crazy characters in the pathname that I like, and everything "just works" :
cd /tmp/Foo\ \!\"£\$%\^\&\*\(\)\'\\\:\;@#~\`\|\\\[\]\{\}\?\<\>-_\=+\$\{PATH\}\ Bar/
$ node example.js
[
  {
    "device": "/dev/sda",
    "description": "SanDisk SD8SN8U1",
    "size":<PHONE_NUMBER>168,
    "mountpoints": [
      {
        "path": "/"
      },
      {
        "path": "/home"
      },
      {
        "path": "/boot/efi"
      },
      {
        "path": "/var/lib/docker/aufs"
      }
    ],
    "raw": "/dev/sda",
    "protected": false,
    "system": true
  }
]

Without all the crazy backslash-escapes auto-created by tab-completion, the actual directory name looks like
/tmp/Foo !"£$%^&*()'\:;@#~`|\[]{}?<>-_=+${PATH} Bar

and from my home-dir, I can run any of:
node '/tmp/Foo !"£$%^&*()'\''\:;@#~`|\[]{}?<>-_=+${PATH} Bar/example.js'
node /tmp/Foo\ \!\"£\$%\^\&\*\(\)\'\\\:\;@#~\`\|\\\[\]\{\}\?\<\>-_\=+\$\{PATH\}\ Bar/example.js
node "/tmp/Foo "\!"\"£\$%^&*()'\\:;@#~\`|\\[]{}?<>-_=+\${PATH} Bar/example.js"
node "/tmp/Foo "'!'"\"£\$%^&*()'\\:;@#~\`|\\[]{}?<>-_=+\${PATH} Bar/example.js"

with identical results. (apparently the ! causes problems (history expansion) in double-quoted strings in an interactive terminal, but not in shell scripts )
On Windows, these are the illegal filename-characters: \ / : * ? " < > |, and here's what I found from my investigations:

Folder
node example.js
node folder/example.js
folder/win32.bat

(blargh, 3am now so I'll finish these notes off tomorrow...)
Possibly relevant / useful: https://github.com/sindresorhus/opn/pull/47

Thanks a lot for helping me out with this.

I'm beginning to wonder if using cmd /c ... with .execFile might be a workaround?)

The problem is still asar archives. The only way to execute a file inside an asar is to pass its path as the first argument to .execFile (since Electron understands that, and will handle extraction). Any other variations means that the file will not execute.

There is a solution though, which will actually make my work on the CLI way easier: inline the drivelist scripts in JavaScript files that manually extract them out and execute them.
The logic put there can execute the file using my cwd workaround, and then it all should work. What do you think?

The problem is still asar archives. The only way to execute a file inside an asar is to pass its path as the first argument to .execFile (since Electron understands that, and will handle extraction). Any other variations means that the file will not execute.

Ahhh, thanks. I didn't realise that the .execFile restriction was due to the way that Etcher dynamically unpacks things from the asar.

There is a solution though, which will actually make my work on the CLI way easier: inline the drivelist scripts in JavaScript files that manually extract them out and execute them.

Huh? That sounds hideous :-/ But I guess it might work, as long as we don't run into permissions / threading / forking issues.
However I wouldn't want to see the drivelist scripts 'manually' embedded inside other Javascript files, as that would make them totally unmaintainable. We should continue to edit the platform-specific scripts as standalone files, and then have some step of the build-process that then embeds them inside another JS file (e.g. as base64 or something).

Check https://github.com/resin-io-modules/drivelist/pull/164. I created a small script that creates a JSON file out of them, so we can still keep separate for developing purposes. I'll test this thoroughly before merging.

Remind me to look into this again sometime next week ;-)

Not sure if this is the same or related error but didnt want to open up another issue if it was. Getting this on startup. The wierd thing is, I havent updated Ethcer recently that Im aware of but it started doing it all of a sudden. Uninstalled and reinstalled the latest from the site did not resolve.
Picture of error

@pizzapants That indeed looks like a different problem. Could you please open a new issue, including as much information as possible? Thanks.


Merging initial version into main branch

Merging initial version into main branch

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

recheck


[CI/CD] Move binary resources to another location

We currently have around 1.2GB of images, sounds, and fonts in the app/resources and app/original_resources directories, which results in slower git operations and longer workflow durations in CI.
We should find a way to isolate these binary files from the core repo to keep things lean and fast. One solution here could be simply moving the resources to another repo (e.g. open-duelyst/resources), configuring .zip and .tar.gz releases for that repo, and then having the core repo pull those in at build time.

We have app/original_resources (uncompressed images, etc.) and app/resources (compressed images). It should be easy to move original_resources to their own repo for starters.

I think moving them to a new LFS enabled repo and having a process to pack / unpack into a compressed zip is a good idea. We may however need to nuke the history on this repo to get back the size savings.
We can start by creating the asset repo first and go from there. I should be able to help with this over the weekend.

Hey @marwanhilmi, assigning this to you since I don't have permissions to create new repos in the open-duelyst Github org.

You can use git filter-branch to purge art assets and preserve code history if you want.


Oct 19 - 23 - Take your first steps with Python

Event Information

Category
Details

Reactor Topic
Coding, Languages, and Frameworks

Monthly Topic (if applicable)
Open Source

Weekly Topic (if applicable)
Python

Workshop Duration
60 mins

Title
Take your first steps with Python

Description
Are you just interested in Python and want to learn how to get started coding with it? Come to this one hour workshop where you will learn about how Python is used, how to setup your development environment with VS Code, and write your first programs in Python. After you take your first steps, we'll share many resources for you to continue learning on your own pace.

Learn Content
https://docs.microsoft.com/learn/paths/python-first-steps/ (Modules 1-3)

Workshop(s) details

Date
Local Start Time
Location
Speaker
SharePoint ID
Platform

Oct 20
6:30 PM
Bangalore
TBD
11635
TBD

Oct 20
4:00 PM PST
US/Canada
TBD
11637
TBD

Oct 21
7:30 PM
Sao Paulo
TBD
11643
TBD

Oct 21
7:30 PM
Shanghai
TBD
11644
TBD

Oct 21
5:00 PM
London, Stockholm, Tel Aviv, Abu Dhabi
TBD
11645
TBD

Event Checklist
10 Weeks Prior

[x] Reactor Content Team (RCT) opens a new issue for an upcoming local workshop filling in the Event Information and Workshop details tables. RCT member who owns content (content owner) this week will be the assignee.
[x] RCT and content owner discuss details of the event in the comments of this issue. RCT adds a confirmed-event label and local-digital-event tag to the issue and adds event to the Local Digital Events project. Issue will appear in the Upcoming Streams column
[x] RCT adds a ready-for-marketing-edit label so the RCT can review title and description

8 Weeks Prior

[ ] RCT does a review of title and description and removes the ready-for-marketing-edit label
[ ] RCT moves the issue to the Ready for SharePoint column
[ ] RCT informs the Reactor Marketing Team of the new event information
[ ] RCT confirms the speaker
[ ] RCT provides speaker with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT moves the issue to the Content Preparation In Progress column

5 weeks Prior

[ ] Content owner opens a PR with the new content for the workshop on the customer-facing Reactor repository and adds the content-development and local-digital-event tag to the PR.
[ ] RCT reviews, approves, and merges the changes
[ ] RCT shares content with the vendor/instructor
[ ] RCT provides vendor/instructor with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT ensures that instructors have received an invite to the workshops and that PM team works with instructors to get setup in the platform (e.g. Teams, Teams Live Event)

1 Week Prior

[ ] Connect vendor/instructor with the PM(s) for the locations where this event has been marketed
[ ] Check-in with vendor/instructor that they are prepared for the workshop
[ ] RCT moves the issue to the Ready to Stream column

During and After Stream

[ ] Instructor (and optional moderator) run the stream
[ ] _(Optional)_RCT submits recording to be uploaded to the Reactor YouTube and to be made available on LearnTV
[ ] RCT moves issue to the Stream Complete column

@KimberlyBolton1 in case you didn't see this one - there is another one :)
These have been entered into Dynamics and just need to be updated when final details come through.

@meaghanlewis  Hi Meaghan, here's the edit. Can you give me some prereqs to include--knowledge level and what they should have installed before they come? I took a stab but need your help.
Take Your First Steps with Python
Are you interested in learning how to code with Python? This one-hour workshop will teach you how Python is typically used and how to set up your development environment with Visual Studio Code. You will write your first Python programs and walk away with the resources to continue learning on your own.
Prerequisites: This workshop recommends that you have a basic knowledge of a programming language other than Python. To follow along, install  Python (https://www.python.org/downloads/), Visual Studio Code (https://code.visualstudio.com/Download), and create an Azure account ((https://azure.microsoft.com/en-us/free/).

Thank you @KimberlyBolton1! ✅

Bangalore, Sao Paulo, Shanghai, and London/Stockholm/Abu Dhabi/Tel Aviv are all published without speaker data (9/2)

Events delivered and attendance data recorded. Closing issue.


How to swap just two items, using drag and drop function

Hi.
I want to swap just two items. basically the grid example rearrange items between two items.
any idea?

Hi. Sorry, this library does not support such behavior and I do not have a plan to support that.

UPDATE: Okay, I will try to implement it. Some others are requesting this feature. (#269)

@Card2Me, @TapanHp I have just made a solution of this issue. I added the setItemMoveMode() to RecyclerViewDragDropManager, it allows to configure the behavior as "swap" mode. This new feature will be included into the next release, but you can try it now using JitPack.

Tip: try develop branch library via JitPack.
repositories {
    maven { url "https://jitpack.io" }
}

dependencies {
    compile 'com.github.h6ah4i:android-advancedrecyclerview:c9763e000fa8e80e6410f4a9d500e21b0e58a7e5'
}

What you did man ? Its awesome that's what i needed ..  Thanks for making this functionality available. Your library was and will be so helpful , Thanks.

@TapanHp Hi. I have just merged the develop branch and released it as v0.9.2. It includes the setItemMoveMode() method. Thanks 👍


Better error for _normalize_index

Based on https://github.com/theislab/scanpy/issues/593, the error now tells you which values were not valid.

Great! Thank you!


Configuration: Add note for verbose option

Summary
Indicate what the default is if only one test file is being run.
Fixes https://github.com/facebook/jest/issues/9205

Codecov Report

Merging #9279 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9279   +/-   ##
=======================================
  Coverage   64.92%   64.92%
=======================================
  Files         278      278
  Lines       11893    11893
  Branches     2932     2933    +1
=======================================
  Hits         7722     7722
  Misses       3542     3542
  Partials      629      629

Continue to review full report at Codecov.

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

@SimenB All of them? It's hard for me to know exactly how long this has been true. Possibly from the start, but who knows.


Feat/error handling

Summary
Adds SSL error handing to the SDK.
NOTE: merging this PR will kick off the major release!

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


Bluetooth: Proprietary HCI Vendor Specific Event

This PR adds support for two features:

HCI vendor specific (HCI VS) event handler
Proprietary HCI vendor specific event handler

HCI vendor specific event handler
The Zephyr HCI vendor specific event carries a controller-defined subevent (HCI VS subevent); the subevent code is located in Event parameter 0 and the subevent data is located in Event parameter 1.
This PR adds an event handler for the BT_HCI_EVT_VENDOR event code which is then delegated to a subevent handler (similar to how LE Meta Events are handled). It is then possible for future HCI extensions, like HCI Mesh Ext, to be hooked into the HCI vendor specific event handler.
Proprietary HCI vendor specific event handler
As a proprietary controller vendor, we need support for an application to react on proprietary HCI vendor specific events. To solve this we have added a new Zephyr HCI extension called "HCI Prop Ext", that adds support for a HCI VS subevent with event code 0xFF (not to be confused with the BT_HCI_EVT_VENDOR event code itself). This new HCI VS subevent, called "HCI Prop Report", carries a proprietary blob of data, that can be delivered to the application via a registered callback.
The PR also adds a test that runs the Host standalone with a stubbed test controller that only supports a very limited HCI command set.

First of all, I like the general idea
Some quick thoughts, for which I (just as @jhedberg) reserve the right to change my mind

The name "proprietary" is kind of confusing in a context where we already have something called "vendor specific". I don't have any good names to suggest right now, but I'll give it some thought. I might just be slow, but the naming threw me off for a while.
If we decide to use sections for defining VS event handlers, do we need to think about the case where there are multiple "subscribers" to the same (sub-)sub-event? That makes it trickier to make a decision whether something is "handled" or not. Then void return makes sense for the callbacks, since one handler for a given event/opcode should not affect any of the other handlers.

Yeah i know the name is confusing, it's kinda like saying it is "vendor-specific HCI vendor-specific events". Normally as a vendor you would have complete control over the Host/Controller and you don't need this abstraction, but in our case we have some proprietary logic in our controller implementation that we need to expose to the application.

Perhaps we could allow any HCI event to capture this way not only vendor specific, obviously this could break the stack if not handled properly but since this will probably only be used by those knowing the consequences Id guess that allow for more flexible way to customize the stack. Id name this BT_HCI_USER though, which could then gate APIs such bt_hci_cmd_send* and custom event handlers to users outside of the stack.

Based on your input, i think i will make a new PR, with something along the lines that @jhedberg  suggested earlier.
Registering a handler for HCI vendor specific events, and then in the handler return success/failure in finding a proprietary subevent handler, then the hci_core can look at it's own handlers for any enabled HCI extensions.
Pseudo code-ish it would be along the lines of this:
app_vendor_specific_event_cb(buf) {
    check buf for proprietary subevent
    return 0 if handled else -EINVAL
}

hci_vendor_event(buf) {
    err = app_vendor_specific_event_cb(buf)
    if (err)
        default_vendor_specific_event_handler(buf)
}

With regards to the BT_HCI_USER idea that @Vudentz suggested, i do like it, but i'm going to leave it out to begin with and then maybe we can add it afterwards.

With regards to the BT_HCI_USER idea that @Vudentz suggested, i do like it, but i'm going to leave it out to begin with and then maybe we can add it afterwards.

Regarding this idea, I'd leave it out for now since it has the potential of breaking the proper functioning of the stack.

Closing this in favor of https://github.com/zephyrproject-rtos/zephyr/pull/16106


Improved Search Functionality

wofi's default search is a bit annoying so I added: -I --matching multi-contains --insensitive
Here's an example of what I mean:

Here I searched 'no'. No results came up despite their definetely being an emoji entry with exactly the word 'no' in it. But its not even shown within 16 results...
This is what I was looking for:

After the fix, and searching 'no':

First result...
much better...

Thank you for the PR. Few things

-I no need for it if we are just listing emoji, as they are text not image.
--insensitive → Not a big deal, we can leave it outside in order to keep it simple.
--matching multi-contains → I've tested it and while it's true it's more precise in that particular scenario, it's less precise in other scenarios. For example writing "sweat" will find 💦 while with our current method would find 😅.

Because that terrain is a bit subjective, something we could do instead is exposing a $WOFI_EMOJI_ARGS so users can pass their custom wofi options instead of having to fork the project.

Also, the emoji are automatically populated from emojilib.

-I was an oversight, I didn't intend to include it.
I agree, that the best solution would be to allow the users to pass their own args.
Side note:
I still find that its more accurate, actually... --matching multi-contains --insensitive
In the case of 'sweat':
Searching before change.

Searching after change.

Notice the word sequence:

'sweat' occurs sooner here than in the following example:

that is a flaw with the emoji list not the search function.


Change to Readdirnames for some cases

There was a couple uses of Readdir/ReadDir here where the only thing the return value was used for was the Name of the entry. This is exactly what Readdirnames returns, so we can avoid the overhead of making/returning a bunch of interfaces and calling lstat everytime in the case of Readdir(-1).
https://cs.opensource.google/go/go/+/refs/tags/go1.20.4:src/os/dir_unix.go;l=114-137

Usually the fd after open is not used elsewhere, it looks better to wrap it as a Readdirnames function, but I'm not sure where this public tool function would be better placed in that package.
func Readdirnames(dir string) (names []string, err error) {
    	f, err := os.Open(dir)
	if err != nil {
		return err
	}
	defer f.Close()

	return f.Readdirnames(-1)
}

Just a little thought, this pr looks great

@Iceber Yea.. it does get a lil redundant, especially in the loadShims case as I'd prefer to close the fds once we have our info immediately. But I fall into the same thought, where would this go? Somewhere under pkg might make sense, maybe in ioutil, but so far that's been focused on custom reader/writer implementations. I think this is at just the right amount of use where I think we're okay without a wrapper, but if anyone feels strongly I'll peruse around for somewhere to place one


Named parameters don't work in sqlc with MySQL

What happened?
Version:
sqlc version
v1.15.0

Running sqlc generate on this query:
SELECT *
FROM table
WHERE CASE WHEN TRUE THEN `status` IN ( @HELP::int[] ) ELSE FALSE END;

Fails to parse, with:
queries.sql:72:47: syntax error near "::int[] ) ELSE FALSE END;

Trying to use ANY like the docs say doesn't work, it was unclear if this was postgres or mysql syntax at first, but for some reason the docs seem to hard code postgres syntax, and the playground only supports postres:
SELECT *
FROM table
WHERE CASE WHEN TRUE THEN `status` = ANY(@HELP::int[]) ELSE FALSE END;

Fails to parse, with
queries.sql:72:47: syntax error near "@HELP::int[]) ELSE FALSE END;"

(it's weird it fails to parse on a different place, not the ANY which is invalid mysql syntax
Ok, let's try with arg() instead
SELECT *
FROM table
WHERE CASE WHEN TRUE THEN `status` IN ( sqlc.arg('statuz') ) ELSE FALSE END;

So sqlc generate passes here, except it generates a null string, not a slice
func (q *Queries) DoTheThing(ctx context.Context, statuz sql.NullString)

ok the docs tell us to append ::int[] as a type hint, so
SELECT *
FROM table
WHERE CASE WHEN TRUE THEN `status` IN ( sqlc.arg('statuz')::int[] ) ELSE FALSE END;

Crashes sqlc once again:
queries.sql:72:61: syntax error near "::int[] ) ELSE FALSE END;"

Ok, not sure what's going on, named arguments don't seem supported. Let's try them in a different place:
SELECT *
FROM table
WHERE CASE WHEN sqlc.arg('test') THEN `status` IN ( ? ) ELSE FALSE END;

Now we have three problems:
sqlc generate
unsupported reference type: <nil>

Broken error message
But exit status code of sqlc is successful
Generates what appears to be broken sql: ... WHERE CASE WHEN ? and there is no test argument added to the generated golang code

Ok maybe it's missing a type hint?
SELECT *
FROM table
WHERE CASE WHEN sqlc.arg('test')::bool THEN `status` IN ( ? ) ELSE FALSE END;

Except that crashes the parser again:
queries.sql:72:36: syntax error near "::bool THEN `status` IN ( ? ) ELSE FALSE END;"

Should we consider the mysql as supported in sqlc, or something in beta/experimental mode? It does not appear usable, and the docs don't mention how to use MySQL syntax, unless I'm missing something obvious

I see this is at partly a dupe of https://github.com/kyleconroy/sqlc/issues/695 - where slices aren't supported in MySQL. I'm not sure about the CASE WHEN clause and the additional parse errors

MySQL is fully supported in sqlc. Many of the queries in your bug report aren't valid, hence the compilation failures. First, the shorthand CAST operator (::) isn't supported by MySQL. You'll need to use the full CAST syntax instead. MySQL also does not support array types, which is covered in https://github.com/kyleconroy/sqlc/issues/695.
Feel free to open up a new issue if you run into other issues.

helo @kyleconroy
UPDATE users
SET
    first_name = IF(CAST(sqlc.arg(set_first_name) AS boolean) = true, CAST(sqlc.arg(first_name) AS varchar), first_name),
    last_name = IF( CAST(sqlc.arg(set_last_name) AS boolean)  = true, CAST(sqlc.arg(last_name) AS varchar), last_name),
    avatar = IF( CAST(sqlc.arg(set_avatar) AS boolean)  = true, CAST(sqlc.arg(avatar) AS TEXT), avatar),
    updated_at = now()
WHERE id = ? and deleted_at is null;

why i igot error
sqlc/query/users.sql:24:61: syntax error near "boolean) = true, CAST(sqlc.arg(first_name) AS varchar), first_name),"

I feel that SQLC is only suitable for PostgreSQL.

and i found issue with @ for sqlc.arg() on mysql

I feel that SQLC is only suitable for PostgreSQL.

i'm using it with sqlite. sqlc for sqlite is still buggy but already good enough. hope it'll be as good as for postgres soon.

-- name: UpdateAuthorName :exec
UPDATE author
SET name = CASE WHEN CAST( sqlc.arg('set_name') AS CHAR )
    THEN sqlc.arg(myname)
    ELSE name
    END;

const updateAuthorName = `-- name: UpdateAuthorName :exec
UPDATE author
SET name = CASE WHEN CAST( sqlc.arg('set_name') AS CHAR )
    THEN ?
    ELSE name
    END
`

not working


Fix github permissions for user "alevkoy"

User @alevkoy has been added as a collaborator under the Emulation section.  However this user appears to be stuck with only contributor/read only access.

@alevkoy has been added to the "collaborators" team.


Update to .NET 6 (preview 3)

GitHub is loading the .NET 5 clrjit.dll instead of the .NET 6 DLL on Windows


Support 'full' arguments

This commit extracts "full arguments" from the features branch. I think we can merge it and then move on to extracting and merging getters  followed by eval.


Add ability to pass Faraday config block to Client

Add test that error messages are included in Faraday response body
Add ability to pass Faraday config block to client

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?

This is such interesting timing. I was thinking of adding something along these lines, actually, because so far I was monkey patching this in the initializer openai.rb!
OpenAI.configure do |config|
  config.access_token = Rails.application.credentials.open_ai_key
  config.request_timeout = 300
end

module OpenAI
  module HTTP
    def connection_with_timeout(multipart: false)
      original_connection(multipart:).tap do |conn|
        conn.use TimeoutMiddleware, timeout: @request_timeout
      end
    end

    alias_method :original_connection, :conn
    alias_method :conn, :connection_with_timeout
  end
end

Obviously, having such a block to configure the Faraday object is an improvement.
FYI: this was because I was still observing time outs

This is such interesting timing. I was thinking of adding something along these lines, actually, because so far I was monkey patching this in the initializer openai.rb!

OpenAI.configure do |config|

  config.access_token = Rails.application.credentials.open_ai_key

  config.request_timeout = 300

end

module OpenAI

  module HTTP

    def connection_with_timeout(multipart: false)

      original_connection(multipart:).tap do |conn|

        conn.use TimeoutMiddleware, timeout: @request_timeout

      end

    end

    alias_method :original_connection, :conn

    alias_method :conn, :connection_with_timeout

  end

end

Obviously, having such a block to configure the Faraday object is an improvement.
FYI: this was because I was still observing time outs

Ah sweet - could you test this PR with that use case? Would be much appreciated 🙏