Improve operatibility with latest module systems

Add support to allow package to be used with nodenext
Remove @types/cheerio dependency as cheerio ships with its own types

Background context: I was trying to use tablejson inside my project which uses the moduleResolution as nodenext and the package complained with typescript errors. On investigation, I found out that @types/cheerio was no longer needed with the latest cheerio package as it supplied its own types. Adding the .js extension while importing modules, although unintuitive, is a more future proof way while maintaining backward compatibility.
Let me know if this can be included.

I'm currently preparing a hybrid package that can cope with the needs of both worlds. I'll add this to master in the next few days. Thanks for your PR. I'll leave it open and let you check the changes.

Hi.
I published version 4.0.0 which should be able to copy with the need of both worlds. Could you please give this a shot and get back to me in any case of an error.
Thanks

I tried using the latest version got the folllowing error.
error TS7016: Could not find a declaration file for module 'tabletojson'. '/working-directory/tabletojson/dist/lib/mjs/Tabletojson.js' implicitly has an 'any' type.
  There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The 'tabletojson' library may need to update its package.json or typings.

1 import { Tabletojson as tabletojson } from 'tabletojson';

Can you please check?

Like displayed in the issue here: https://stackoverflow.com/questions/58211880/uncaught-syntaxerror-cannot-use-import-statement-outside-a-module-when-import I would like to ask you to create a small project I can fork which has the problems. It is clear, that typingsand typesare equivalent and can both be used.
I tested it in a test project I created, everything worked.
But as far as I see this is a problem which is not as easy to be solved:
https://github.com/microsoft/TypeScript/issues/52363#issuecomment-1659227177
and
https://gist.github.com/andrewbranch/79f872a8b9f0507c9c5f2641cfb3efa6
So, as far as I can see, the problem is the module resolution with nodenext. For now I cannot see a solution for this behavior despite the fact that I would have to create a second library for nodenext.

Thanks @maugenst for reverting back. Here is a link to a toy project. Simply run npm run compile to reproduce the issue.

I'll have a look and get back to you...

With version 4.0.1 I changed the type exports. I tested it with your project and it worked. Could you please verify?

Thank you. It works with my project. Appreciate your prompt response!
Strange that "types" and "typings" are not synonymous as claimed by the documentation. Must be a tsc bug with nodenext resolution.
I'll close the pull request.


[SUGGESTION] Move to MediaWIki

I firmly believe that contributions and moderation would be significantly easier to I were to move uranometrical/notes to (a potential fork of) MediaWiki.
Localization and maintaining the wiki itself would become significantly easier, and a familiar environment would be more user-friendly.

@Metacinnabar

Looks promising, I would prefer if we found a nice looking theme however.


incompatible (can't compile) with IDEA-141.157.3

error in HaxeStructureViewFactory because:
TreeBasedStructureViewBuilder.createStructureViewModel() is removed in this commit
error in HaxeRegularExpressionImpl because:
HaxeRegularExpressionImpl.supportsExtendedHexCharacter  & supportsNamedGroupRefSyntax is added in this commit & this commit

IDEA-141 is 14.1, right?  We haven't gotten Idea 14 solid yet.
Thanks for the report and the research.

yes, it is 14.1.
I have maked a PR to branch masterIdea14.
is branch masterIdea14 for IDEA 14.x or just for 14.0?

is branch masterIdea14 for IDEA 14.x or just for 14.0?

As long as it compiles for 14.0, we can put any 14.x changes there.  If they are 14.1 specific, then we will need to add another branch.  However, is it possible, with the your changes, to keep the old signatures intact, and call the new signatures?  I'm not sure whether we can just drop the @Override on the new functions, but it's worth a try.  At any rate, if we can keep the old signatures, then this change, anyway, can remain 13.x compatible as well.

I think you should add another branch for idea 14.1 because:

RegExpNamedGroupRef is added in rev 5d585f5 which is only included since idea/140.361 (expand & see the tag list of commit 5d585f5 ) (idea 14 is 138.* & 139.* - see Release Notes )
since HaxeRegularExpressionImpl implement RegExpLanguageHost, it must implement method supportsNamedGroupRefSyntax(RegExpNamedGroupRef) in all IDEA from idea/140.370+ (see tag list of commit 733be824 - this commit is where the method being added)

That means: If we support IDEA 14.1 (idea/140.370+) => must implement RegExpLanguageHost.supportsNamedGroupRefSyntax(RegExpNamedGroupRef) => RegExpNamedGroupRef must existed => IDEA must be idea/140.361+ => can't support IDEA 14.0 in the same code base!


Rate limit reached for 10KTPM-200RPM...

Expected Behavior
Expected GPT Engineer to do it's thing!
Current Behavior
It read my prompt. Asked a dozen questions which I answered in detail. Then:
Nothing more to clarify.
INFO:openai:error_code=rate_limit_exceeded error_message='Rate limit reached for 10KTPM-200RPM in organization org-[orgid here] on tokens per min. Limit: 10000 / min. Please try again in 6ms. Contact us through our help center at help.openai.com if you continue to have issues.' error_param=None error_type=tokens message='OpenAI API error received' stream_error=False
Failure Information (for bugs)
This, I believe is the relevant portion of the stack:
File "c:\python310\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "c:\python310\scripts\gpt-engineer.exe_main.py", line 7, in
sys.exit(app())
File "c:\python310\lib\site-packages\gpt_engineer\main.py", line 64, in main
messages = step(ai, dbs)
File "c:\python310\lib\site-packages\gpt_engineer\steps.py", line 154, in gen_clarified_code
messages = ai.next(messages, dbs.preprompts["use_qa"])
File "c:\python310\lib\site-packages\gpt_engineer\ai.py", line 37, in next
response = openai.ChatCompletion.create(
File "c:\python310\lib\site-packages\openai\api_resources\chat_completion.py", line 25, in create
return super().create(*args, **kwargs)

Is there a way to rate limit the app? This is the only thing I was using it for.
My prompt was pretty detailed. About 150 lines.

Same thing here. I spent an hour writing a detailed prompt then another one answering 15 questions. Then ... CRASH
INFO:openai:error_code=rate_limit_exceeded error_message='Rate limit reached for 10KTPM-200RPM in organization org-******* on tokens per min. Limit: 10000 / min. Please try again in 6ms. Contact us through our help center at help.openai.com if you continue to have issues.' error_param=None error_type=tokens message='OpenAI API error received' stream_error=False
Traceback (most recent call last):

  File "/Users/robert.stone/.pyenv/versions/3.11.1/bin/gpt-engineer", line 8, in <module>
    sys.exit(app())
             ^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/gpt_engineer/main.py", line 64, in main
    messages = step(ai, dbs)
               ^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/gpt_engineer/steps.py", line 154, in gen_clarified_code
    messages = ai.next(messages, dbs.preprompts["use_qa"])
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/gpt_engineer/ai.py", line 37, in next
    response = openai.ChatCompletion.create(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/openai/api_resources/chat_completion.py", line 25, in create
    return super().create(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/openai/api_resources/abstract/engine_api_resource.py", line 153, in create
    response, _, api_key = requestor.request(
                           ^^^^^^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/openai/api_requestor.py", line 298, in request
    resp, got_stream = self._interpret_response(result, stream)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/openai/api_requestor.py", line 700, in _interpret_response
    self._interpret_response_line(

  File "/Users/robert.stone/.pyenv/versions/3.11.1/lib/python3.11/site-packages/openai/api_requestor.py", line 763, in _interpret_response_line
    raise self.handle_error_response(

openai.error.RateLimitError: Rate limit reached for 10KTPM-200RPM in organization org-******* on tokens per min. Limit: 10000 / min. Please try again in 6ms. Contact us through our help center at help.openai.com if you continue to have issues.

Anyone found solution for this?
I tried using sleep for upto 5 mins... Still not working...

seeing same issue ^^  :(

Same. Need to solve or at least know if I break into smaller components will it work.

Doesn't seem like an issue with gpt-engineer but openai, happy to hear suggestions will close otherwise

I rotated my openai api key and that seemed to fix the issue...at least for now

I rotated my openai api key and that seemed to fix the issue...at least for now
@snlamm
Hey Can you please elaborate? Rotate the openai key like how?

The problem is that your prompt and clarifications are too extensive for your OpenAI plan. Depending on your OpenAI plan, your token limit can be around 10K/min and if you have written a long prompt or the application has too many lines of code generated, you can easily get over this limit.
On Azure OpenAI for gpt-4-32k you have a token limit of 60K/min and can use 32K token per request by default. I think OpenAI also enabled such sizes for enterprises now.
The only thing gpt-engineer could do is already warn upfront that your prompt is probably too too long, but I don’t think this is worth the effort at the moment, because the OpenAI error already explains the issue.


feat: provide table options everywhere and add title to api tables

Hey, I just made a Pull Request!
Add tableOptions to all tables and additionally title to API tables.
This will align the interface to the the Backstage provided tables over the whole project.
We required this as we like to make all tables exportable as CSV as we found this useful to have any kind of ad hoc reporting.
:heavy_check_mark: Checklist

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

FIY @Rugvip


[Bug]: BetterF3 mod not coloring debug menu, also debug menu becoming stuck on screen

Describe the Bug
I don't get any crashes or error messages when I load the latest version of Forge with BetterF3 (as I have been with other mods that broke recently), but when I load into my worlds and press F3, the debug menu is not colorized as it's supposed to be, and it also does not go away when I press F3 again, so it remains on my screen until I disconnect from my worlds.
Additional note: this appears to be happening on both my singleplayer worlds and on multiplayer servers (not sure if any difference would be expected, but point is I'm not seeing any).
Steps to Reproduce
Enter an SP world or an MP server and press F3.
Expected Behavior
Debug menu comes up but is not colored. Debug menu also not disappearing on subsequent F3 presses.
Screenshots

Modloader Type
Minecraft Forge
Modloader Version
41.0.85
Mod Version
1.3.1
Log Output
No response
Additional Context
No response

Try going into settings and in General Settings turn "Disable Mod"  to "No"

Closed do to probable duplicate of #99.


plot gene names on DEG heatmap if they fit, also consolidate some heatmap code

fixes #92

Hey, looks like this finally succeeded! @mattjmeier let me know if this is ok to merge


Progress Indicator Component

Is your feature request related to a problem? Please describe.
Progress indicator component for multi-stepped form interface.
Describe the solution you'd like
Please add a progress indicator component for multi-stepped form interface.
Describe alternatives you've considered
Alternatives - Carbon Design System Progress Indicator
Additional context
None.

Hi there. Please use our AHA Ideas board to request and vote on new features.


KimTay Database

ADD ISSUE DESCRIPTION HERE
I am trying to connect to KimTay database through Github to do my cengage labs for my class Intro to SQL at Front Range Community College in Westminster, CO. However it does not let me choose the KimTay database so I can attemp to do my tasks for my labs. Please HELP!
Version: 1.94.2
Commit: 384ff7382de624fb94dbaf6da11977bba1ecd427
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
Embedder: codespaces

/causedByExtension
Please use proper feedback channel for your course.

could you show me a step by step process please of this extension

This is issue tracker for VS Code editor itself, I doubt that anyone here knows course/setup specifics.
As said above you should use proper contacts for your course.


Handle OIDC errors

I try http://localhost:3000/openid-connect/redirect_uri?error=access_denied&state=123  (that's my exported port), all I get is an entry in the nginx error log:
==> /var/log/nginx/error.log <==
2020/03/16 09:44:43 [error] 393#393: *17 # _oauth2_openidc_redirect_uri_handler: invalid request to the redirect_uri: code parameter could not be found [error=access_denied&state=123], client: <IP_ADDRESS>, server: , request: "GET /openid-connect/redirect_uri?error=access_denied&state=123 HTTP/1.1", host: "localhost:3000"

I'd imagine some handling for error response. I'm not sure what it ought to be though.

yes, the module is in beta right now, error handling is certainly something that needs to be added/improved


SqlGenerator caches statement thereby disabling dynamic NamingStrategies [DATAJDBC-208]

Jens Schauder opened DATAJDBC-208 and commented

Issue Links:

DATAJDBC-358 CrudRepositories with dynamic table names
("is duplicated by")

While the caching is a good thing by default an alternative that does not cache the SQL statements to allow for dynamic name generation would be nice.

While the caching is a good thing by default an alternative that does not cache the SQL statements to allow for dynamic name generation would be nice.

Closing this, since it was more a theoretical idea, than based on actual demand.


How to populate getAccount silently

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[X]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
I am trying to silently populate UserAgentApplication account object silently without requiring a login (loginPopup, loginRedirect). MSAL is a secondary feature so i do not want to require a login.
If i call acquireTokenSilent without a loginHint it will error.
If i hard code the loginHint everything works as expected. Is it possible to allow the library to traverse all logged in accounts? Similar how to the login/logout interfaces support multiple accounts.
My use case is sort of specific in that all i want to know is if there is any active account so i can show a signout button, but it would be a nice feature of the Library to be able to get all those accounts.
Examples:

"How do I use MSAL with Vue.js"
"How do I SSO between tabs?"
"How do I use MSAL to protect my custom Web API?"
"How can my app support multiple AAD tenants?"
"When will my scenario be supported?"
"When will this framework be supported"

@arsnyder16 You can silently SSO to an existing AAD/MSA session by calling acquireTokenSilent with a loginHint and your clientId as the only scope. If there is an existing session, an ID token for that session will be acquired and getAccount will be populated. If not, it will through an error (e.g. login_required) that your app will need to handle.
We are working to add this a new top-level API, ssoSilent, in #1166.

@jasonnutter
Yea I am trying to avoid tracking a loginHint, my best guess is to store something in local storage from the last known session and try to use that. That won't be super reliable.
If i am understanding correctly ssoSilent will lift this requirement of having a loginHint?

now that i look at the details of the PR, i don't exactly see the use case for that method. All it is doing is passing the clientId for you but you already had to initialzie the UserAgentApplication with the correct clientId anyway.
I guess its just to avoid confusion with calling acquireTokenSilent when you really just want to login?

Just to elaborate on my use case since the PR does have a discussion around requiring loginHint.
My use case is really to just try and identify any open session so i can show the sign out button, but i can see down the road the need to show an identity chooser directly in my UI. For example if i am showing a list of files from the users one drive. They might be using their personal and business identities. It would be a better experience for me to have a drop down control so they can easily switch between accounts. Currently i would need to show a popup window and have the user choose through the Microsoft interface.

@arsnyder16 Correct, ssoSilent is convenience method for the use case when your application is confident that the user already has an active AAD session (i.e. you already sent them through the login flow using something other than MSAL.js). The loginHint is required to disambiguate if there are multiple active sessions.
Can you provide more details on your use case (e.g. is there a server component)? You mention MSAL is secondary, can you clarify what you mean by that?

@jasonnutter What if there is only 1 active session? It seems logical to that it could be retrieved without a login hint
I think it would be a good feature to allow the client to the library to retrieve all the active sessions. So they can build interfaces and constructs based on them. As I mention above

@jasonnutter  For more context.
We have a commercial application that has both web and desktop(mac/win) interfaces, similar to Office. We are integrating with different cloud repositories, onedrive/google drive/dropbox etc. So a user may choose to open one of our files from onedrive or they may choose google drive. They may also do a combination of both.  For our desktop components we are using embedded CEF to handle some of these integration pieces so that we can share as much logic as possible with the web versions. When opening the desktop application you may already have a valid cookie (choosing yes on the do you want to stay signed in page) and be able to create implicitly authorize with MSAL/OneDrive/Microsoft Graph.
We would like to supply our users with an interface to show them what account are currently active, it may be a list that is combination of different providers  maybe a google identity, a onedrive for business identity and a onedrive personal identity. Within that list we want to allow them to log out or see more details of the individual identities. In order to build all this it would be very helpful if msal would supply a way to get all active sessions and furthermore switch and log out of individual ones.   I am sort of piecing it all together currently, but with some enhancements to MSAL.JS it could be a little more straight forward.

@arsnyder16

What if there is only 1 active session? It seems logical to that it could be retrieved without a login hint

The problem is that your application cannot know for sure that there is only one currently session.
We are planning on adding multiple account support to MSAL (ETA next quarter, I believe), which I believe will solve this use case.

@jasonnutter Great! Yea that will be beneficial. Is there a work item that i can follow?

@arsnyder16 You can follow #1082

Closing issue


Browser connectivity not working

Hi, I tried your connect-browser.html and it is not working

Well a little bit more to go on would be nice. It works perfectly on my machine

WebSocket connection to 'wss://sense-demo.qlik.com/app/%3Ftransient%3D' failed: Error during WebSocket handshake: Unexpected response code: 403
That is always on my browser's console everytime I run the HTML


Excel module [Create a drop-down list] has video link duplicate of [Fill data automatically]

Category

PROVISIONING NOTE: If you are having issues with the provisioning service while trying to provision M365 learning pathways please do not post that issue to this repo as we cannot help you here. Please post all questions and issues for the provisioning service to https://github.com/SharePoint/sp-dev-docs/issues.

[ ] Question
[x ] Bug
[ ] Content

Question / Expected or Desired Behavior
In Home > Excel > Intro to Excel > Create a drop-down list, the video should fit the module title.
Observed Behavior
In Home > Excel > Intro to Excel > Create a drop-down list, the video is "Fill data automatically"
Steps to Reproduce
In the MLP Viewer, navigate to Home > Excel > Intro to Excel > Create a drop-down list.
Please note: This is a copy of issue #119 https://github.com/pnp/custom-learning-office-365/issues/119, which was closed with the message "Content has been refreshed closing this issue.". Upgrading to <IP_ADDRESS> changes nothing - the behavior is exactly the same as before and the bug has therefore not been removed.
Submission Guidelines
Thanks for your contribution! Sharing is caring.

To make sure there's clarity, the version of the web part has nothing to do with the content, they are completely decoupled.  This is a content issue and I'll reassign it as I was told this was fixed with the content refresh that happened at Ignite 2019 which coincided with the solution going GA as well as the version 3.0.0 web part.


nginx issue in windows 10

Creating dockerizedmagento_solr_1
Creating dockerizedmagento_fullpagecache_1
Creating dockerizedmagento_sessions_1
Creating dockerizedmagento_cache_1
Creating dockerizedmagento_mysql_1
Creating dockerizedmagento_php_1
Creating dockerizedmagento_nginx_1
ERROR: for nginx  Cannot start service nginx: oci runtime error: rootfs_linux.go:53: mounting "/mnt/sda1/var/lib/docker/aufs/mnt/6dc10df5fa48c5d8753491fe169d0e25d2b7eb109b8870a44cac784fadf573e0/etc/nginx/nginx.conf" to rootfs "/mnt/sda1/var/lib/docker/aufs/mnt/6dc10df5fa48c5d8753491fe169d0e25d2b7eb109b8870a44cac784fadf573e0" caused "not a directory"
ERROR: Encountered errors while bringing up the project.

I would say this is related to #22.
Docker and Windows is tricky and I will not be able to help you with that.


feat: the suffa tactic

The suffa tactic.
suffa tac runs the tactic sequence tac and returns a Try this: suggestion
of the form suffices [target_after_tac] by tac; assumption.
For example
example {m n : Nat} (h : m = n) : 0 + m = n := by
  suffa rewrite [Nat.zero_add]
  assumption

suggests the replacement
example {m n : Nat} (h : m = n) : 0 + m = n := by
  suffices m = n by
      rewrite [Nat.zero_add]
      assumption
  assumption

See this thread as well as this message.

I don't like the name suffa. I have no idea what it stands for (so a beginner won't either).
Also are you sure that a tactic sequence consumer is the correct approach here? What about an "Insert suffices here" code action on any tactic? Then

We don't have to learn yet a new tactic (which would be especially hard to learn since it is not present in the final code)
We don't need to type anything more to edit the code


Re-enable Organization Management UI unit tests

Is your suggestion a test requirement/change or improvements of the test framework? Please describe.
The identity-apps unit test was failing after to the addition of Organization management test suite.
Hence, it was temporarily disabled in https://github.com/wso2/identity-apps/pull/3048/commits/be4f845bf15a854a7d7d688cefa70b81236c69ec.
Describe the solution you would prefer
Fix the mock issues and re-enable the suite.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


JSDoc comments being removed

This is all comments in general but pertains to JSDoc as it's not a big deal if other comments are removed. tsup uses esbuild for building and they remove all comments, and have no plans to change this - so I will need to switch to another tool (probably just rollup on it's own)
If using rollup should use rollup cleaning plugin and make sure jsdoc is whitelisted

No longer relevant


GenerateLineupException: Can't generate lineups. Following constraints are not valid: total_players,positions_MVP

is there a limit on how many playersgroup the MMA optimizer can handle?
doesnt like this code until i comment out a few of my groupings, then it takes off.
see code below
fight1= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Cory Sandhagen', 'TJ Dillashaw')], max_from_group=1, min_from_group=1)
optimizer.add_players_group(fight1)
#fight2= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Raulian Paiva', 'Kyler Phillips')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight2)
fight3= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Darren Elkins', 'Darrick Minner')], max_from_group=1, min_from_group=1)
optimizer.add_players_group(fight3)
fight4= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Maycee Barber', 'Miranda Maverick')], max_from_group=1, min_from_group=1)
optimizer.add_players_group(fight4)
#fight5= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Randy Costa', 'Adrian Yanez')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight5)
#fight6= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Punahele Soriano', 'Brendan Allen')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight6)
fight7= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Nassourdine Imavov', 'Ian Heinisch')], max_from_group=1, min_from_group=1)
optimizer.add_players_group(fight7)
#fight8= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Jordan Williams', 'Mickey Gall')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight8)
#fight9= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Andre Ewell', 'Julio Arce')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight9)
#fight10= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Sijara Eubanks', 'Elise Reed')], max_from_group=1, min_from_group=1)
#optimizer.add_players_group(fight10)
fight11= PlayersGroup([optimizer.get_player_by_name(Nickname) for Nickname in ('Hannah Goldy', 'Diana Belbita')], max_from_group=1, min_from_group=1)
optimizer.add_players_group(fight11)
#sandhagen = optimizer.get_player_by_name("Cory Sandhagen")
#optimizer.add_player_to_lineup(sandhagen)
#sandhagen.max_exposure = 1
#sandhagen.min_exposure = .66
#yanez = optimizer.get_player_by_name("Adrian Yanez")
#optimizer.add_player_to_lineup(yanez)
#yanez.max_exposure = 1
#yanez.min_exposure = .50
#soriano = optimizer.get_player_by_name("Punahele Soriano")
#optimizer.add_player_to_lineup(soriano)
#soriano.max_exposure = 1
#soriano.min_exposure = .50
lineups = optimizer.optimize(n=500)
#optimizer.set_min_salary_cap(59300)
for lineup in lineups:
print(lineup)
optimizer.export(r"C:\Users\brand\Desktop\PYTHON\UFC\UPLOAD.csv")
import pandas as pd
df = pd.read_csv(r"C:\Users\brand\Desktop\PYTHON\UFC\UPLOAD.csv")
optimizer.print_statistic()
would be so much easier if fanduel included their opponent in the csv

You set min_from_group for each group, which means that at least one player should appear from each group, but you have more groups than roster slots, you can try to remove min_from_group.

Cant decypher my error after i fixed the above issue @DimaKudosh
KeyError                                  Traceback (most recent call last)
 in
68 lineups = optimizer.optimize(n=50)
69 optimizer.set_min_salary_cap(90)
---> 70 for lineup in lineups:
71     print(lineup)
72 optimizer.export(r"C:\Users\brand\Desktop\PYTHON\UFC\UPLOAD.csv")
~\anaconda3\lib\site-packages\pydfs_lineup_optimizer\lineup_optimizer.py in optimize(self, n, max_exposure, randomness, with_injured, exposure_strategy)
385         constraints = [constraint(self, players_dict, context) for constraint in rules]
386         for constraint in constraints:
--> 387             constraint.apply(base_solver)
388         previous_lineup = None
389         for _ in range(n):
~\anaconda3\lib\site-packages\pydfs_lineup_optimizer\rules.py in apply(self, solver)
294     def apply(self, solver):
295         if not self.with_exposures:
--> 296             self._create_constraints(solver)
297
298     def apply_for_iteration(self, solver, result):
~\anaconda3\lib\site-packages\pydfs_lineup_optimizer\rules.py in _create_constraints(self, solver)
328                     combinations_variables[group_name] = solver_variable
329                 for group_players, group_min, group_max in sub_groups:
--> 330                     variables = [self.players_dict[p] for p in group_players]
331                     if group_min is not None:
332                         if not stack.can_intersect:
~\anaconda3\lib\site-packages\pydfs_lineup_optimizer\rules.py in (.0)
328                     combinations_variables[group_name] = solver_variable
329                 for group_players, group_min, group_max in sub_groups:
--> 330                     variables = [self.players_dict[p] for p in group_players]
331                     if group_min is not None:
332                         if not stack.can_intersect:
KeyError: None
-What should i be looking for?

Looks like you add a not-exist player to the group because your script failed when iterating over your group and found None. get_player_by_name  can return None if a player with provided name doesn't exist.

good catch @DimaKudosh .  Closing issue now.  Thanks


Fix task deletion

Hi @goodchai0
Have made the changes to fix the issue for deletion where every time the last element was getting deleted instead of the selected element
Can you please review the pr
Issue link:- https://github.com/goodchai0/day-planner/issues/31
Screencast for the changes
https://www.loom.com/share/de4dc2f74616499fa1be3f9475f00730

Hi @goodchai0  can you take a look t the above PR

Star the repo

Yes done  that


Missing method.

.address appears not to be in the sdk. What is available is the .getSmartAccountAddress()
const biconomyAccount = new BiconomySmartAccount(biconomySmartAccountConfig)
const biconomySmartAccount =  await biconomyAccount.init()
console.log("owner: ", biconomySmartAccount.owner)
console.log("address: ", biconomySmartAccount.address)
https://docs.biconomy.io/docs/Biconomy AA Stack/Account/initialization

Hey @iamonuwa in the latest version of the the sdk the method is .getAccountAddress()
With the method you are mentioning this is referring to Account V1 https://docs.biconomy.io/docs/category/smart-account-v1


github.repos.get({owner: 'octokit', repo: 'node-github'}) is sending wrong accept header

The request currently sends Accept: application/vnd.github.drax-preview+json
const GitHub = require('github')
const nock = require('nock')
nock('https://api.github.com', {
  reqheaders: {
    'accept': 'Accept: application/vnd.github.v3+json'
  }
})
  .get('/repos/octokit/node-github')
  .reply(200, {
    full_name: 'octokit/node-github'
  })

const github = new GitHub()

github.repos.get({owner: 'octokit', repo: 'node-github'}).catch(console.log)
// Nock: No match for request
// {
//   "method": "get",
//   "url": "https://api.github.com/repos/octokit/node-github",
//   "headers": {
//     "host": "api.github.com",
//     "content-length": "0",
//     "user-agent": "NodeJS HTTP Client",
//     "accept": "application/vnd.github.drax-preview+json"
//   }
// }

Try it on https://runkit.com/gr2m/59b7290cfadd4e00123218df
The reason is this line:
https://github.com/mikedeboer/node-github/blob/f06dd9fc310c0f8e6893793ed8ce6335f7cde2a6/lib/routes.json#L511
Can you tell why the "/repos/:owner/:repo" route is in both the application/vnd.github.drax-preview+json and also the application/vnd.github.scarlet-witch-preview+json group? Looks like a bug to me?

for reference, here are all accept headers by route as they are implemented today
https://gist.github.com/gr2m/46b593a662638aafeba373132b8fa48c

There's an overlap for that endpoint for getting additiional preview-only content in the response when providing either the scarlet witch or drax accept header. Unfortunately it doesn't look like you can include both and it just picks the first accept header. So the endpoint can optionally include info for License preview or Codes of Conduct.

Okay, so that is a problem with the current implementation, right? Not easy to fix? I’d suggest we leave the issue open until we revise the current generation of APIs

Can you tell why the "/repos/:owner/:repo" route is in both the application/vnd.github.drax-preview+json and also the application/vnd.github.scarlet-witch-preview+json group? Looks like a bug to me?

Youʻre actually right, ivʻe been having this problem to but it is werid that


Using Snabbdom without a module system

I am working on a project that is not using webpack, babel or even npm. I write everything in plain old JS, use a C-style preprocessor and a global namespace, filled with long, scoped function names, as if i were programming in C. There are some benefits to creating JS this way, compared to using a fancier toolchain (instant builds, smaller bundles, less time spent on tools, easier to optimize). Actually, it is beginning to seem better in almost every way, and the only things i miss are some webpack plugins, and some modern syntax, but i digress.
I'm using Snabbdom, and in order to get it into my project, i need to get a lot of stuff that i would not need otherwise; nodejs, npm, webpack+plugins, all to build a file that holds Snabbdom, and takes its exports and puts them on the window object.
It would be nice if there was a single-file bundle somewhere that i could just download and use, as if today was ten years ago. Or is there already? Anyone else interested in something like this?

Main snabbdom.js dist file is already packaged as a UMD module with identifier snabbdom, i.e. it will be registered as global variable snabbdom if you are not using any bundler or packaging system.
You can copy its content right in your browser console and see for yourself.

Indeed that seems to do exactly what I want. I'm not that familiar with UMD or it's relatives so i never would have guessed from the dist files themselves. Thanks for the quick reply!

This is what i did to make it work:

Downloaded dist/snabbdom.js, as well as all the modules
Removed the embedded sourcemaps at the bottom of each file
Found out that...

h is available as snabbdom.h
modules available as global variables are snake_cased, like snabbdom_eventlisteners, etc.

Works great.


Access Method

Access Method
Description
There seems to be a significant need for determining the permissions a user has
on files outside of the current user. Access acts as a wrapper for os.access by
extending the ability to specify a user and/or group along with a file to test permissions against.
Requirements

[x] Allow users to specify a file to test against
[x] Allow users to specify a mode to test against (R_OK, W_OK, X_OK)
[x] Enable user to specify either a user or a group to test against.
[x] Successfully enumerate a users or groups permission on a specific filet.

Run / test configuration

Run Config

access(test-file, mode, user=user, group=group)
Either user or group must be specified only one can be left as default.

Owner Tests

Make a user the owner: chown <user> test-file

Owner and read chmod u+r test-file
Owner and write chmod u+w test-file
Owner and execute chmod u+x test-file

User's Group Tests & All group Test

Make a user's group the owner: chown :<user's-group> test-file
To test other groups other than the user's group, enumerate a group they're apart of  (e.g, adm) and add that group to the file: chown :<enumerated-group> test-file

Group and read chmod g+r test-file
Group and write chmod g+w test-file
Group and execute chmod g+x test-file

Other Tests

Other and read chmod o+r test-file
Other and write chmod o+w test-file
Other and execute chmod o+x test-file

Merged after successfully testing this. Thanks alot @Nib0x62s!
I made some minor changes to consolidate the flag generate and checking logic in commit b02fffa43f32c10f3e5f82c3423f45f816d7c0c8.


Typo in github repo description

http://reduxjs.org -> http://redux.js.org
Cheers! :)

Thanks! Not sure who changed that...


Fix issue with selected_articles field in article slider component

Fetching "selected_articles" config should return a string, which will be given over to "explode". But default value is an empty array. That results in an error.

Hey @dimitri-koenig,
thank you for your pull-request.
We will merge as soon as possible.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-26460
Please use this issue to track the state of your pull request.

 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.

thanks for your contribution @dimitri-koenig  👍🎉💙


add select

This is a small addition to make it easy build a new NamedTuple with a subset of the original keys:
select(nt::NamedTuple, k::Symbol) = nt[k]
select(nt::NamedTuple, k::NamedTuple) = select(nt, keys(k))
select(nt::NamedTuple, ks) = namedtuple(ks)((nt[k] for k in ks)...)

See https://github.com/JeffreySarnoff/NamedTupleTools.jl/issues/7#issuecomment-518396085

Codecov Report

Merging #8 into master will decrease coverage by 1.57%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   48.88%   47.31%   -1.58%
==========================================
  Files           1        1
  Lines          90       93       +3
==========================================
  Hits           44       44
- Misses         46       49       +3

Impacted Files
Coverage Δ

src/NamedTupleTools.jl
47.31% <0%> (-1.58%)
:arrow_down:

Continue to review full report at Codecov.

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

Sure, I'll get something together

thank you -- merging


Fix TypeScript syntax in docstring

This is a very very small pull request 😉

@sedrubal Thanks for the Fix, but types.py is a generated file. Also, this fix needs to go into all languages like python, rust, and C#, So fix needs to go into the meta model:
https://github.com/microsoft/vscode-languageserver-node/blob/fb0a3d48b9aa1c52d527f623164b0b4f11115d5a/protocol/metaModel.json#L6567
The metamodel is generated from actual definitions here:
https://github.com/microsoft/vscode-languageserver-node/blob/fb0a3d48b9aa1c52d527f623164b0b4f11115d5a/types/src/main.ts#L168-L181


Explicit export of ContractFunction and SignatureTemplate

When using cashscript from an ESM module, compiler can not find ContractFunction and SignatureTemplate.
We can be explicit about them.

JS can be weird 😅
Thanks for the contribution, will release it in 0.7.4 shortly.


Use DisplayLink to prevent crashes in willResignActive

We've experienced frequent crashes while tracking routes, when the app goes into background, right after willResignActive is called.
This change instantiates DisplayLink to control rendering, preventing the crash. It is based off what Mapbox does, but more minimal to not require copying various 6.x changes and only touch the relevant parts, mostly what's required to instantiate and use CADisplayLink.
After this change no more crashes have been observed.

Hi @reezer and thanks for the PR, this is important fix and these seem to be similar crashes as noted in issue #70.
There seems to be some problem with this PR when tested with actual device (in my case iPhone 12). If you run the SDK demo app, put the app to background (don't close it) and bring it back to foreground, moving the map doesn't work anymore. I managed to fix this by taking few more changes from the related Mapbox PR. I'll try to make change request to your branch.

@reezer I created the following PR to your branch: https://github.com/reezer/maplibre-gl-native/pull/1
It includes few more changes from the Mapbox PR and seems to work correctly in my testing.

@juhieta I will look into this as soon as I have time. In fact we also had problems when the app come back into foreground and I have some local changes for displayLink as well.
Also I locally tried to remove the Snapshotting-Code, since that also seems to be another source of crashes and removal was at least discussed in mapbox bugs as well. So far this seems to be a way of working around those issues.

Had a chance to look at it. Thanks.
Yes, I looked at the changes in Mabox PR 432, but crashes still seems to happen to some people with recent versions of mapbox, which is why I removed the snapshots locally. So far this seems to work, but it still requires more testing.

Thanks, and yes I'll also test this more now. I did actually remove the snapshots too as they caused some unwanted side effect for our case (if base map is changed to something very different, you can get a snapshot of completely different map when bringing to foreground). Maybe in future the snapshots should be made optional at least (and possibly disabled by default).

However, as this limits the opportunities for the app to take map view snapshots, this can cause the map to "jump" more often when brought to foreground. Therefore, another PR to disable the snapshotting completely could be considered after this is merged.

I now created new PR #83 as a suggestion to disable map snapshotting for iOS.

It looks like this change broken ios UI tests - platform/ios/platform/ios/Integration Tests/MGLMapViewPendingBlockTests.m, testSetCenterCoordinatePauseRendering because it is using MGLMapView pauseRendering selector which was removed.
Exception	NSException *	"-[MGLMapView pauseRendering:]: unrecognized selector sent to instance 0x7fe5da035200"	0x0000600003ea3120

Thanks @petr-pokorny-1, I can have a look!

This should fix it @petr-pokorny-1: #88

Btw the ios-ci -builds for PRs seem to fail instantly with error:
No runner matching the specified labels was found: macos-11.0
For example, https://github.com/maplibre/maplibre-gl-native/actions/runs/860113172

Btw the ios-ci -builds for PRs seem to fail instantly with error:
No runner matching the specified labels was found: macos-11.0
For example, https://github.com/maplibre/maplibre-gl-native/actions/runs/860113172

I see - 11.0 should be available but it isnt for some reason. I have set it to macos-10.15 and launched CI build again


macOS: If editor_doc_cache.res exist in the "cache/godot" , project will crash while opening

Tested versions
Godot 4.2.1 stable
Yesterday I closed the project as usual, but today I can't open it anymore
can not reproduce
System information
MacOS 12.7.4 -Godot 4.2.1 stable -Vulkan API 1.2.231 - Forward Mobile - Using Vulkan Device #0: Intel - Intel HD Graphics 5000
Issue description
editor_doc_cache.res is created by which project do not matter.
Delete "cacha/godot" and create an empty project ,that project can be opened many times without crashing. But once you try to open the problematic project, it will crash.
Delete editor_doc_cache.res then open the problematic project again, it won't crash.

Changing "rendering/renderer/rendering_method" in ProjectSetting from Mobile to GL Compatibilitythe can also avoid crash.
the log with godot -e -v --path[problematic project path]:
......
......
......
Loaded system CA certificates
Creating VMA small objects pool for memory type index 1
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.2.1.stable.official (b09f793f564a6c95dc76acc654b390e68441bd01)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] 1   libsystem_platform.dylib            0x00007ff807436dfd _sigtramp + 29
[2] 2   ???                                 0x00007fba2c6393c0 0x0 +<PHONE_NUMBER>68000
[3] void rx::mtl::GetMatrixUniformMetal(unsigned int, unsigned int*, unsigned int const*, bool)
[4] void rx::mtl::GetMatrixUniformMetal(unsigned int, unsigned int*, unsigned int const*, bool)
[5] void rx::mtl::GetMatrixUniformMetal(unsigned int, unsigned int*, unsigned int const*, bool)
[6] Ref DirAccess::_create_builtin()
[7] Ref DirAccess::_create_builtin()
[8] Ref DirAccess::_create_builtin()
[9] RenderSceneBuffersRD::get_use_taa() const
[10] RendererSceneCull::Instance::dependency_deleted(RID const&, DependencyTracker*)
[11] Gradient::get_color_at_offset(float)
[12] Ref DirAccess::_create_builtin()
[13] ItemList::get_v_scroll_bar()
[14] ItemList::get_v_scroll_bar()
[15] ItemList::get_v_scroll_bar()
[16] CallableCustomExtension::default_compare_less(CallableCustom const*, CallableCustom const*)
[17] CanvasItem::get_canvas_item() const
[18] ItemList::get_v_scroll_bar()
[19] CallableCustomExtension::default_compare_less(CallableCustom const*, CallableCustom const*)
[20] Node::get_tree() const
[21] Node::get_tree() const
[22] Node::get_tree() const
[23] Node::get_tree() const
[24] Node::get_tree() const
[25] Node::get_tree() const
[26] Node::get_index(bool) const
[27] void rx::mtl::GetMatrixUniformMetal(unsigned int, unsigned int*, unsigned int const*, bool)
[28] RendererCompositorRD::_create_current()
[29] 29  dyld                                0x000000011219d52e start + 462
-- END OF BACKTRACE --
Steps to reproduce
can not reproduce
Minimal reproduction project (MRP)
can not reproduce , move the problematic project to another Mac(MacOS 10.15.7-Godot 4.2.1 stable) , it will not crash while opening anymore.

Might be related to https://github.com/godotengine/godot/issues/89678.
CC @bruvzg @RandomShaper

I was not able to reproduce https://github.com/godotengine/godot/issues/89678, but if it's crashing if resource exist, it might explain it (old resource for might be incompatible in some way, or is it created by the same Godot version?).


libuv: update to v1.18.0

Hi all, I've noticed that libuv has been updated to v1.18.0 yesterday.
Thank you all for bringing us this useful tool.

 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.:x: lcdtyph sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Change order of components (post_reaction.jsx) to enable hovering for e2e test

Summary
While working on https://github.com/mattermost/mattermost-server/issues/12299 , I was not able to configure Cypress so that it would hover on 2 of 3 of the post menu icons (post reaction icon & comment icon). After reviewing the code, it turned out to be a problem with how the <button> element was not nested within the <OverlayTrigger> component, but was instead the parent of said component, which in turn resulted in Cypress not being able to hover over the post reaction icon.
I changed the nesting order by putting the <button> element inside the <OverlayTrigger> component.
Ticket Link
This PR was not a listed issue.
Related Pull Requests
https://github.com/mattermost/mattermost-webapp/pull/3894
https://github.com/mattermost/mattermost-webapp/pull/3893

@josephk96 See comment on https://github.com/mattermost/mattermost-webapp/pull/3893


Delete foreach Glob.Directories() fails

Hi,
On version 1.1.1 this code
foreach (var dir in Glob.Directories(sourcePath, "**/bin")) {
    Directory.Delete(sourcePath + dir, true);
}

causes error

System.IO.DirectoryNotFoundException: Could not find a part of the path '...'.

Is this by design or bug?
Here is stacktrace:
   at System.IO.Enumeration.FileSystemEnumerator`1.CreateDirectoryHandle(String path, Boolean ignoreNotFound)
   at System.IO.Enumeration.FileSystemEnumerator`1..ctor(String directory, EnumerationOptions options)
   at System.IO.Enumeration.FileSystemEnumerable`1..ctor(String directory, FindTransform transform, EnumerationOptions options)
   at System.IO.Enumeration.FileSystemEnumerableFactory.DirectoryInfos(String directory, String expression, EnumerationOptions options)
   at System.IO.DirectoryInfo.InternalEnumerateInfos(String path, String searchPattern, SearchTarget searchTarget, EnumerationOptions options)
   at System.IO.DirectoryInfo.GetDirectories(String searchPattern, EnumerationOptions enumerationOptions)
   at System.IO.DirectoryInfo.GetDirectories()
   at GlobExpressions.TraverseOptions.GetDirectories(DirectoryInfo root)
   at GlobExpressions.PathTraverser.Traverse(DirectoryInfo root, Segment[] segments, Int32 segmentIndex, TraverseOptions options)
   at GlobExpressions.PathTraverser.Traverse(DirectoryInfo root, Segment[] segments, Int32 segmentIndex, TraverseOptions options)
   at System.Linq.Enumerable.SelectManyIterator[TSource,TCollection,TResult](IEnumerable`1 source, Func`2 collectionSelector, Func`3 resultSelector)+MoveNext()
   at System.Linq.Enumerable.ConcatIterator`1.MoveNext()
   at System.Linq.Enumerable.SelectManyIterator[TSource,TCollection,TResult](IEnumerable`1 source, Func`2 collectionSelector, Func`3 resultSelector)+MoveNext()
   at System.Linq.Enumerable.ConcatIterator`1.MoveNext()
   at System.Linq.Enumerable.OfTypeIterator[TResult](IEnumerable source)+MoveNext()
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at ConsoleApp1.Program.Main(String[] args)

Did it literally say Could not find a part of the path '...'. or was the a specific path it was unable to find?

I want to add that calling ToList before iteration fixes it.

Oh I see, the files being iterated are being simultaneously deleted. Let me look at reproducing.

A prerelease version is available here that should fix the issue.
https://www.nuget.org/packages/Glob/1.1.2-CI-20190306-040219
Please let me know the results and I will push a 1.1.2 release.

Yes. This fix works.

Closed by 1.1.2. Thank you


Unable to find '../../scripts' in expected paths

Issue Type

Documentation Report

Your Environment
Vagrant 1.8.6
VirtualBox 5.1.6r110634
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides

Your OS

macOS El Capitan 10.11.6

Full console output
TASK [Run configured post-provision ansible task files.] ***********************
 [WARNING]: Unable to find '../../scripts' in expected paths.

Summary
The documentation at https://github.com/geerlingguy/drupal-vm/blob/2997dc9a3db2108f14ac1e3bb0b6d1540a5beef3/docs/extras/scripts.md shows the path for post provision tasks as ../../scripts/post-provision.yml, but this fails with the above error.
When I update the path to be ../scripts/post-provision.yml it appears to read the correct file (although this is currently failing on me since updating drupalvm, but that's another issue.

@kiwimind - Are you using Drupal VM standalone, or as part of some other tool (e.g. BLT)? If used standalone, I believe the pathing would be correct, if the vagrant commands are run inside Drupal VM's folder. But if with a tool, the path might be different.
I haven't confirmed this to be the case, but I do vaguely remember having to change that path when used with BLT at least.

@geerlingguy Thanks for your prompt response.
No, not using any other tools with this.
Have updated to the latest Drupal VM alongside my custom config file, trying to add in both a post provisioning task and script.
It's working with the amended path to the post-provision.yml file.
Can investigate further if it helps when back in the office on Monday.
Thanks again.

@kiwimind - Sounds good, I'll try to test sometime too, when I get a chance!

Confirmed—this was incorrect, so I updated the docs. Thanks, @kiwimind, for the error report!

https://github.com/ansible/ansible/issues/14036

@oxyc ++ — I knew it was working when we merged it originally :P

@geerlingguy no worries, thanks for the swift response.


Install failed

When I install the custom node of animatediff, it shows the error like below:

The only required dependency is opencv-python, which also required by many other custom nodes (ImpactPack, comfyui_controlnet_aux,...).
Can you try:

Install via Comfy Manager again
Or follow these steps to install the repo manually:

cd root/ComfyUI
.\venv\Scripts\activate.bat # for Windows
source venv/bin/activate # for Linux

cd custom_nodes
git clone https://github.com/ArtVentureX/comfyui-animatediff
pip install -r comfyui-animatediff/requirements.txt

I'm having slighly different trouble.
After installing the node and restarting comfyui, it hangs with these messages
Building wheel for flash_attn (setup.py) ... started
Building wheel for flash_attn (setup.py) ... still running
Building wheel for flash_attn (setup.py) ... still running
and will continue like that forever it seems.
Same happens when i install flash_attn manually via pip, but without the additional output.
Steps mentioned above didn't help, requirements are already installed and don't even include flash_attn.


only one element slide

Is it possible to customize your plugin so that when switching slides only one element is added.
example

i want this

Hey @valeriy-efimov, you can set the perPage prop to 1! https://ssense.github.io/vue-carousel/api/#perPage

@quinnlangille not working
https://jsfiddle.net/suobpda2/
if press right button, display  "3 5 6" but i want to see "2 3 4" if again press "3 4 5" etc

Ah I misunderstood your request, for that you'll need to add scrollPerPage=false, like this -> https://jsfiddle.net/oLvr8wem/

@quinnlangille thank you very much!


Added .gitignore, trust me it's good for you

 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.:x: assafnativ sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Most of that .gitignore is useless for a non-Python project.
Excluding only .exe, .dll, .obj  and .pdb would have the same effect.

@sylveon I would also add .vs, *.ipdb, *.ipch, *.iobj. , Debug, Release, all the other configuration/platform mixes.

Ok, I've removed the Python stuff

You forgot the A there in that commit message.


How to mount a network resource inside a container and allow rw on the resource?

Hi,
assuming the container is on the same lan (bridged network), is there a way to mount a network resource (nfs or cifs) directly from inside the container?
Here is that I would like to do:
lxc create ubuntu:xenial c1
lxc exec c1 bash
mount -o username=SAMBAUSER,password=SAMBAPASSWD,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/share /mnt
And be able to read and write to /mnt from inside the container.
I tried to mount the network resource from the host and then issue a bind mount without success.
lxc config device add c1 MyShare disk source=/mnt path=/mnt
Thank you!

On Sun, May 08, 2016 at 01:57:26PM -0700, typhoe wrote:

Hi,
assuming the container is on the same lan (bridged network), is there a way to mount a network resource (nfs or cifs) directly from inside the container?
Here is that I would like to do:
lxc create ubuntu:xenial c1
lxc exec c1 bash
mount -o username=SAMBAUSER,password=SAMBAPASSWD,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/share /mnt
And be able to read and write to /mnt from inside the container.
I tried to mount the network resource from the host and then issue a bind mount without success.
lxc config device add c1 MyShare disk source=/mnt path=/mnt

This looks reasonable, can you elaborate about what "without success"
means?

Thank you!

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/issues/2005

A straight mount inside the container isn't going to work due to nfs and cifs not being mountable by unprivileged users, but mounting on the host and bind-mounting into the container should work.
Though you will most likely have to pass uid= and gid= to your host side mount too to set owner uid and gid which make sense in the container.

Sorry I took so much time to follow up my first post. And thank you for your answers.
In the following, lxc is my host (ubuntu xenial 64 bits, up to date on the 13th of mai 2016) and c1 is the container.
The network resource I use is a cifs share from a Synology NAS.
Here are the tests I made:
Check CIFS mount from host (uid 1056 is my samba user on my nas)
root@lxc:~#
mount -o username=***,password=***,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/downloads /mnt
ls -lh /mnt
total 0
drwxrwsrwx 2 root users 0 mai    8 13:27 test
echo "foo" > /mnt/test/foo.file
ls -lh /mnt/test/foo.file
-rw-r--r-- 1 1056 users 4 mai   11 15:04 /mnt/test/foo.file
#=> So, network cifs share seems to work perfectly from the host
umount /mnt
Check mounting network resource without any special configuration
root@lxc:~#
lxc init ubuntu:xenial c1
Creating c1
lxc start c1
lxc exec c1 bash
root@c1:~#
apt-get -y install cifs-utils
<...........>
The following NEW packages will be installed:
cifs-utils keyutils libavahi-client3 libavahi-common-data libavahi-common3 libcups2 libldb1 libpython-stdlib libpython2.7 libpython2.7-minimal libpython2.7-stdlib libtalloc2  libtdb1 libtevent0 libwbclient0 python python-crypto python-ldb python-minimal python-samba python-talloc python-tdb python2.7 python2.7-minimal samba-common samba-common-bin  samba-libs
<...........>
mount -o username=***,password=***,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/downloads /mnt
#=>
mount error(13): Permission denied
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs)
root@lxc:~#
tail -f /var/log/syslog
May 13 16:27:17 lxc kernel: [442532.401588] audit: type=1400 audit(1463149637.775:62): apparmor="DENIED" operation="mount" info="failed type match" error=-13 profile="lxd-c1_</var/lib/lxd>" name="/mnt/" pid=13085 comm="mount.cifs" fstype="cifs" srcname="//nas/downloads"
Check mounting network resource with apparmor unconfined
root@lxc:~#
lxc stop c1
lxc delete c1
lxc init ubuntu:xenial c1
lxc config set c1 raw.lxc lxc.aa_profile=unconfined
lxc start c1
lxc exec c1 bash
root@c1:~#
apt-get -y install cifs-utils
mount -v -o username=***,password=***,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/downloads /mnt
#=>
mount error(1): Operation not permitted
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs)
#=>Nothing in various logs... /var/log/syslog /var/log/lxd/lxd.log /var/log/lxd/c1/lxc.log
Check mounting network resource with container priviledged
root@lxc:~#
lxc stop c1
lxc delete c1
lxc init ubuntu:xenial c1
lxc config set c1 security.privileged true
lxc start c1
lxc exec c1 bash
root@c1:~#
apt-get -y install cifs-utils
mount -v -o username=***,password=***,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/downloads /mnt
#=>
mount error(13): Permission denied
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs)
#=>Nothing in various logs... /var/log/syslog /var/log/lxd/lxd.log /var/log/lxd/c1/lxc.log
Check mounting network resource with apparmor unconfined and container priviledged
root@lxc:~#
lxc stop c1
lxc delete c1
lxc init ubuntu:xenial c1
lxc config set c1 security.privileged true
lxc config set c1 raw.lxc lxc.aa_profile=unconfined
lxc start c1
lxc exec c1 bash
root@c1:~#
apt-get -y install cifs-utils
mount -o username=***,password=***,iocharset=utf8,file_mode=0777,dir_mode=0777 -t cifs //nas/downloads /mnt
root@c1:~#
ls -lh /mnt/
total 0
drwxrwsrwx 2 root users 0 May 11 13:04 test
echo "bar" > /mnt/test/bar.file
ls -lh /mnt/test/bar.file
-rw-r--r-- 1 1056 users 4 May 11 14:29 /mnt/test/bar.file
#=>Seems ok! And if I change umask, I can create file with 666 rights.
So it is a solution. Not the best I think, but a solution.
@stgraber Is it possible to allow the container to mount network resource but stay unpriviledged? So just allow the mount permission? How? And same, is it possible to create a container with only the apparmor right to mount resource?
Possibly using a profile?
Thank you very much!

See my comment in #2025...

@JohnstonJ Interesting!
That's not a solution to mount the resource inside the container, but I'll try by mounting on the host and sharing the resource with the container.
And at least, It doesn't need unprivileged or messing with apparmor.
Thank you!

@typhoe Nope. Only filesystems which are user namespace aware can be mounted inside an unprivileged container. That's a kernel enforced policy, not something we're doing with some kind of security profile in userspace, so there's no way for you to allow it.
If doing a privileged container, then the mount restrictions are applied through apparmor rather than enforced directly by the kernel, in which case, you can set raw.apparmor to "mount," which would allow all mounts.

Ok, thank you @stgraber for the clarification, I'll try the solution offered by @JohnstonJ  then to stay with an unpriviledged container approch!

so it is safe to assume that an unprivileged container can never mount an nfs share directly?
i'm mounting nfs in the host and only giving each container a subfolder as their own mount entry point. container a must never be able to access container b's files. so thats the case right?

Right now, that's correct. It's not a limitation imposed by LXD but by the Linux kernel. The NFS filesystem isn't marked as being safe for unprivileged users to mount.


[Documentation] Improve transaction-api.md

Added note to enable transaction support in standalone mode.

/pulsarbot run-failure-checks

Hi @truongninh can you re-run the tests? After the tests pass successfully, we can merge this PR.

/pulsarbot run-failure-checks

Close this PR since transaction doc has been udpated.


[Deviantart] It´s possible to speed up the process of download?

I think the process is too low when we have a lot of images. In my case it starts downloading 24 per page, and delay 3 minutes for start with the next page. So just imagine download a big collection

Download speed is always limited by the server, so I don't think there is much that can be done here..

@Hrxn is right, the speed you get with DA is very variable. You can try 2 threads (I wouldn't try more than that, but that's me) since it's a little faster even if DA's end is busy; I've done many folders which have had hundreds of items each, so I know from experience.
If you drop the timeout to 12000 milliseconds, meaning 12 seconds, you might get some boost, at least I seem to have, versus 60000 (60s) or even bigger, where it took even 15 minutes between pages. Seemingly that timeout is how long RipMe waits for reply and re-tries when nothing happens, which'd explain why it takes so long.
The bottom line: don't try too many threads and too low timeouts cuz although DA still hasn't IP-banned me for ripping tons of stuff, they could get wise anytime and suddenly put a range of rippers behind a firewall, which could end uglily. 2 threads and 12s sound like the reasonable max to me to keep using.

@Hrxn You´re right. Sound logical for me.
@rautamiekka Yes. Is configured by default with: 50 Maximum threads and 60000 milliseconds of timeout. I didn't use your recommended configuration. But I tried with a similar,  and there isn´t a significant diference for speed up the velocity. I´m donwloading with 24 threads per page, I think is the max. capacity for DA use. Because only 24 images are hosted per page. And about timeout. I´m not sure, but I used 10000 ms.
In shot, There´s not much you can do about the spped up. I have 20Mbps, and according to my calculations. It will take me for over 30 hours to download 17000 images of my collection.

^ 24 threads is just asking for trouble, from both DA and your connection, and if you've 20Mbps down you're not giving it much per thread.

Looks like this issue is addressed as "by design".
Closing as client-side / by-design / wontfix.


fix build CI errors on automl pycocotools dependency install

fix build CI errors on automl pycocotools dependency install

Codecov Report

Merging #138 (fcf7623) into main (341f1d6) will decrease coverage by 40.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #138       +/-   ##
===========================================
- Coverage   75.01%   34.99%   -40.03%
===========================================
  Files          27       27
  Lines        1789     1789
===========================================
- Hits         1342      626      -716
- Misses        447     1163      +716

Flag
Coverage Δ

unittests
34.99% <ø> (-40.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 18 files with indirect coverage changes


Add content to Built in Icon Support page

https://github.com/softlayer/sl-ember-components/wiki/Built-in-Icon-Support

@SpikedKira this is in the v0.11.0 milestone, just making sure we don't forget about it.

Not forgotten.  Unsure if still needed based on the current content of the readme file.  Do we still need to add content here?  If so, what should be included?

@SpikedKira We had a conversation about the content, where we were going to talk about the reasoning things were/had to be approached they way they were.  Think this was to include us to trampling on other classes, but not certain that was the specific topic or if it was limited to that.  When we discussed it I did not capture any of my thoughts.

I have added some content to the Wiki. Please let me know if anything needs to be addd/modified. https://github.com/softlayer/sl-ember-components/wiki/Built-in-Icon-Support

@azizpunjani Great, thanks!


Build error bigtable/spark/build.sbt

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
The issue you're having must be related to a file in this repository. We are unable to provide assistance for issues unrelated to samples in this repository.
Please include as much information as possible:
In which file did you encounter the issue?
bigtable/spark/build.sbt
Did you change the file? If so, how?
No
Describe the issue
sbt clean assembly fails to build the uber jar
[error] lmcoursier.internal.shaded.coursier.error.FetchError$DownloadingArtifacts: Error fetching artifacts:
[error] https://repo1.maven.org/maven2/org/apache/avro/avro-mapred/1.11.0/avro-mapred-1.11.0-hadoop2.jar: not found: https://repo1.maven.org/maven2/org/apache/avro/avro-mapred/1.11.0/avro-mapred-1.11.0-hadoop2.jar

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

@jhambleton @vii long shot, but you're the last humans who've touched this sample.
Any idea what's happening here? I can reproduce on my end.
I tried looking through the dependency tree (sbt dependencyTree) to see where that 404 dependency was coming from, but I don't even see any instances of "avro-mapred". Not sure where to go from here.

It looks like build.sbt is heavily outdated. The current version of Dataproc is 2.2 making the setup of the script too old. Someone from @cloud-native-db-dpes, please have a look at this issue.


Generic IOC image generation should require main branch of ibek-support

This issue is about generic IOCs and therefore could be in https://github.com/epics-containers/ioc-template but I'm puting it here because it's really an issue with management of this repo.
Because ibek-support is a submodule of every generic IOC and submodule versions are defined by commit hash only, there is a very likely scenario that individual branches of the this repo will proliferate on a per-generic-IOC basis. The only motivation to merge into main is when you want to re-use changes to a support module definition in a different generic IOC. At that point you may be in a world of merge conflict pain. giles as already experienced this when he is the only person working on the repo at present.
@GDYendell has suggested the following fix.

Change the generic IOC CI such that it will fail and not release push image to the container registry if it determines that the commit hash that it is using is NOT in the main branch. This therefore requires that the developer merges and pushes their changes before pushing the generic IOC source.
Allow that this check is switched off if the generic IOC tag that kicked off the CI is a beta tag

@coretl care to comment on this?

Sounds reasonable


Suggestions

These are suggestions regarding how weapon effects could change depending on how high attributes are, instead of only damage dealt changing..

Fire duration could be proportional to the attribute.
Poison effect could be differentiated from fire by dealing more damage over more time instead of being almost the same dps as fire. Could be done by reducing the level of poison to 1 and making the poison duration 6 seconds for every point of poison.
Cold damage could use the new freezing mechanic instead of the slowness effect. Every attack would add (34+attribute) TicksFreezing to the target (10 consecutive hits with a cold damage attribute of 1 would freeze the target in 10 hits). The cap for TicksFreezing should be raised to a higher number tho (2400 seems reasonable, it's roughly 1 minute of freezing with freezing damage. Remember TicksFreezing tick down by 2 every game tick)
The lightning strike chance could be proportional to the attribute. Chance could be (0.05*attribute)
The explosive attribute instead of boolean could be numerical like the others and explosion power could be proportional to the attribute. I would suggest explosionPower=(3x+16)/19}

Also a config option to disable certain attributes from doing extra damage would be appreciate :)

Thanks for suggestions
New update (also backported to 1.16): https://github.com/spoorn/SpoornWeaponAttributes/commit/7f19a4220b1b527901af7d783308db04c9393789
Added expression type strings for various durations, and freeze mechanic.
Didn't change lightning strike or explosive though as those 2 effects are more impactful/unique than the others and I want them to be a flat enhancement rather than scaling to prevent over/underpowered weapons.
You should be able to disable extra damage by setting maxDamage to 0

If you have more ideas feel free to suggest, or even add them yourselves via a pull request

👍 👍


Bio-Formats 7.0.1 updates

Updates to bump Bio-Formats version and add release announcement for 7.0.1.
This PR will fail the builds and wont be ready for full review until BF artefacts have been released

Release text is identical to that on the BF version history and release notes draft document. Also updated the link to TILL Photonics due to an outdated broken link

Text reads fine 👍 - didn't see any typos or other issues


Mast: Watch qaprovider.com

Mast requests the watch of the watch_keyword qaprovider\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

qaprovider\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Team 15 SRS Review: Health/Safety & Security Requirements too Abstract

HS2: The system shall not affect the users' physical and mental health
Rationale: The system must not harm the users' health and safety

In what specific manner can the system end up impacting user health and safety? Specify that perhaps the system will not force the user into positions that could potentially strain/pull muscle groups depending on their current level of flexibility. Since most people live sedimentary lifestyles, sudden activity could do more harm than good.
Section 4.3, SR1 and SR2 specify user data security, yet Section 4.6 states:

There's no user login mechanism because its deemed unnecessary for the system's core functionality

If there is no way for users to create accounts, thereby create profiles for themselves, how would your system even keep track of user data for you to "encrypt and transmit" or "collect"? I understand there is more than one method by which one can take advantage of user data (For example: Utilizing users' motion tracking data to train AI models on), but the most commonly understood way is through user accounts. Some degree of clarity would help these parts not seem as conflicting.
Team 15 TA: @Lyuyang0130

Resolved in:
4a2917a


Python 3 support

First shot for Python 3 support: pass 2to3 on the codebase.
Resulted in an importable and working(?) module. Tests fail on name resolution over api.genepeeks.com and I guess this is not a fault of the module. The changes also broke Py2.7. May be possible to have it work by using futures module as a compatibility layer. Idk if is worth, as Py2.7 is EOLed for almost an year already.
That said, I have no way to know if this is working. My target REST API is Django Rest Framework, that does not honor .filter() without plugins.
Owner/maintainer, please enlighten the next steps needed here.
(and thanks for the existing code)

@alanjds Thank for submitting this PR, I think Python 3.X support would be a wonderful thing to add! GenePeeks used to be a company I worked for, but the API is sadly no longer around... So I would add some mocking around the API or using something like responses would satisfy that part of the tests.
As for supporting Python 2.7, I'd definitely like to do it, if for nothing else than just on principal (I'm sure some people will be using it 10 years from now still). This is a small library so I imagine it won't be a huge overhead in the maintenance. I will take a pass on the tests and get back to you with some insights on where we should go from here!


Ambiguity between snow_depth and snow_height

Hi ,
Thank you for the great work!
In hourly, there is one more parameter one can access is snow_depth instead the parameter name is changed to snow_height, which actually creates the confusion because i real world both are different.

working on it,will publish a second version of the librairy soon

issue fixed


Logo Proposal

Good day Sir @christiandelbianco  , I am a graphic designer and I would like to contribute by designing a logo if you will allow me. I think it will give more visibility to your nice project. It will be my gift for free to the project. I am hoping for your positive feedback Sir. :)
Thanks and best regards!
-Tobaloidee

Thanks for the positive feedback Sir, I will start my design now..will post it here asap.. :)

Hi Sir, here's my design concept.. If you have suggestions or modifications in mind please dont hesitate to tell me. :)

yellow version
icon

logotype

blue version
icon

logotype

Nice!
I will use the yellow one! At least, the old logo color is the same of the new one 🥇
By the way, do you have any idea if is possible to include this logo in GIT Repository page?
What i mean is: instead of current BOOK picture you can see on left side of "christiandelbianco/monitor-table-change-with-sqltabledependency", put your logo.
Can you sent me the logotype image splitted in two files?

THANKS

Sir, I have uploaded the Logo files & updated the Readme in this PR #73
you can also download the other variations of the logo in this linked drive of mine
LOGO FILES
Thanks so much!! I am so glad that you are accepting my contribution. best regards Sir!
-Tobaloidee

Hi, can you give me your profile home page ? I'd like to add it on Contributors section

JC Abuyot
https://github.com/Tobaloidee
Thanks Sir! Best regards!

Hi Tabaloidee,
I hope you do not offend if i ask you a thing.
The original logo of SqlTableDependency was this one.
Also, people that download using Microsoft Visual Studio tool, are used to see this logo:

So, i want to ask you if you can "improve" the old one, without change it too much, in order to be "back compatible" with the old picture.
Regards

Surely Sir, I will make a new concept and design and will show you again here asap..I saw that logo before i did the design that's why i came up with the yellow color.

here Sir, actually this is my first concept in mind before..


Usage needs to be edited in Readme

This is a small thing, but it could be important. The Usage section of README.md gives the expanded syntax as
npm audit --json | audit-export --folder <folder_path> --name <file_name.html> --title <HTML_file_title>
... but in source/index.js, it looks like the filename parameter should be --file, not --name .
My suggestion would be to keep --file, and edit the Readme.

Hey @rdepew, thank you!
I haven't noticed it 😅
It's now fixed and was also added a --help function to the tool. Released now on version 2.1.0.

Thanks for the quick edit! I like the --help option in 2.1.0.


Add and tweak barcode tests

Part of #3554
Description

#3554 is about addressing TODO comments. There were several in spec/models/barcode_item_spec.rb about adding tests.

Some tests were covered by what already existed in the "filters" context; we deleted those empty describes (and their TODO comments) and added a comment pointing readers to that context
We added a new test for by_item_partner_key but since it's also used in filtering, we put it in that context too
We made the existing test for by_base_item_partner_key more robust by adding a test on the number of returned items
Also, the description of described "global" didn't match behaviour, so we updated that.

Paired with @mgriffin ❤
Type of change

Test improvements

How Has This Been Tested?

CI

@sampart actually, we're planning on revamping / removing the global barcode feature. It's probably not worth it to mess with barcode tests until that's done. Sorry!

No worries!


claim short code vs full name

i'm rather confused by the tables, they seem to show a bunch of short code for type, but when i inspect the tokens i receive i see many have the long type.
EG: the docs say oid but i see http://schemas.microsoft.com/identity/claims/objectidentifier  , maybe this is a config in my tenant that's using the full uri?  but since it's an option these docs should make note of it somehow...if i want to use that value across tenants am i supposed to look for both claims and use the one i find?  if so the part about identifying users should mention that somehow

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

ID: 39e50f53-203a-35d3-794f-73b27be0fde0
Version Independent ID: f2af0873-d2ef-6df2-d3c2-8c953d392097
Content: Microsoft identity platform ID tokens - Microsoft identity platform
Content Source: articles/active-directory/develop/id-tokens.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

@drdamour , Thanks for reaching out. We will investigate and update this thread.

@drdamour I believe you are receiving a SAML token format in your application which uses long type names like http://schemas.microsoft.com/identity/claims/objectidentifier instead of oid . The article mentions ID token which uses the JSOPN web token (JWT) format and the JWT token will always contain short type name oid  . Now coming to second part of your query .

"if i want to use that value across tenants am i supposed to look for both claims and use the one i find? if so the part about identifying users should mention that somehow"

If your application is reading the token and is not sure that the token I think the token payload immediately gives the idea whether its a JSON web token or a SAML token and accordingly the logic can be build in the app to obtain Object ID by searching for oid in case of JWT token and http://schemas.microsoft.com/identity/claims/objectidentifier in case of SAML token. Since the article is about oAuth ID tokens and they use JWT format hence only oid is mentioned here in the article.  If your application is built in ASP.net core , the OpenID Connect middleware converts some of the claim types when it populates the claims collection for the user principal:

oid > http://schemas.microsoft.com/identity/claims/objectidentifier
tid > http://schemas.microsoft.com/identity/claims/tenantid
unique_name > http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name
upn > http://schemas.xmlsoap.org/ws/2005/05/identity/claims/upn

Please check the following documentation for more information .
Working with claims based identity in Azure AD
Claim sets and restricted Claim types for SAML and JWT tokens
Thank you .

@shashishailaj its not a saml token, this is an openid connect identity token 100%.  My app is secured with easy auth and that doesnt work with saml.  And if i change my id token in app reg to add optional claims those show up in my token. If i change saml claims they do not show up.  So you are incorrect about it being saml
please reopen


Type 'ErrorObservable' is not generic.

ERROR
node_modules/ng2-markdown-to-html/src/app/markdown-to-html/markdown-to-html.service.ts (25,39): Type 'ErrorObservable' is not generic.)
i am getting this error with  @angular/cli 1.0.0-beta.32.3
while using the ng build -prod --aot --verbose

Can you tell me which version of angular + angular-cli your are using please? I'll look into it ...

I'll update angular-cli .. comin back to you!

same here... when i compile with --aot

@pmcalabrese Thx for the insight i'll make the fix tonight

@jfcere no problem 👍

I updated Angular-CLI to 1.0.0-beta.32.3 and solved the typing error when compiling with AoT.
Please upgrade to version 0.0.9 to get the fix.
I'll close the issue, don't hesitate to reopen if there is anything!


"Batogu" is listed in the drop-down menu

Description:
"Batogu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Batogu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Batogu, Brăila (Cireșu)".


Demo for writing test

can you add an example service for demonstrating on how to write test.

ttps://github.com/zeromicro/zero-examples/pull/20

https://github.com/zeromicro/zero-examples/pull/20


Add info about neovim

Not only neovim, but also vim. https://github.com/vimpostor/vim-tpipeline/issues/2#issuecomment-794430475
No need to use this plugin at all.

Yea, it seems it works natively in vim version 8.2.2345 and later. Thanks for letting us know. I'll update the readme.


Record with 'interface' key produces undefined result when used with ports

I have a port that looks like this:
port simplelink_init : { interface : String, callsign : String } -> Cmd msg

However when I use it like so:
case msg of
  InitAction(InitPage.SetLoopback) -> (Main(MainPage.init), simplelink_init { callsign = "KI7EST", interface = "loopback"})

On the other end of the port the 'interface' key is undefined. It looks like the generated JS is adding an extra $:
case 'SetLoopback':
    return {
        ctor: '_Tuple2',
        _0: _user$project$Main$Main(_user$project$MainPage$init),
        _1: _user$project$Main$simplelink_init(
            {callsign: 'KI7EST', $interface: 'loopback'})
    }

If I change they key to 'foo':
port simplelink_init : { foo : String, callsign : String } -> Cmd msg

Then the generated JS looks correct and everything comes through fine:
case 'SetLoopback':
    return {
        ctor: '_Tuple2',
        _0: _user$project$Main$Main(_user$project$MainPage$init),
        _1: _user$project$Main$simplelink_init(
            {callsign: 'KI7EST', foo: 'loopback'})
    }

From what I could tell interface isn't a reserved keyword and the compiler didn't yell at me. This is elm version 0.18.0 on Windows and repo with repro branch is https://github.com/vvanders/simplelink/blob/elm_repro/ui/elm/Main.elm
Not sure if this is expected or not but it tripped me up so I thought I'd mention it here so someone else doesn't hit the same thing.

This and #1510 are triggered by words in the jsReservedWords list

SSCCE:
port module Main exposing (main)
import Html exposing (..)
main =
    program
        { init = ( 0, out { default = "hello" } )
        , update = \_ m -> ( m, Cmd.none )
        , view = \_ -> text "Check the Console"
        , subscriptions = \_ -> Sub.none
        }
port out : { default : String } -> Cmd msg

<html>
<head>
</head>
<body>
  <script>
    var app = Elm.Main.fullscreen()
    app.ports.out.subscribe(function(msg) { console.log(msg)})
  </script>
</body>
</html>

Tracking in #1685 now. Thank you for the report!


upgrade laterals and boundary conditions

Also there are no changes added to CHANGES.rst.
the changelog has now been updated


Styling issue

Version and Environment Details
Operation system: Windows 10
IDE name and version:  Intellij 2019.1.3
Plugin version: 2.2.0
Look at this picture:

When I remove the gql tag it looks like this:

Adding gql tag somehow adds some extra indentations. I'm using 2 spaces for indention both in typescript and graphql code style settings.

Hi Ali.
Thanks for using the plugin.
The plugin wraps the JavaScript/TypeScript formatter to be able to also format the injected GraphQL.
If you configure your JavaScript indentation to match your TypeScript indentation you should see the right indentation.
Let me know if that works for you.
Best regards,
Jim.

Yes, it works when JS and TS both have same indentation.


Add more tests for TaskService.

Coverage increased (+1.3%) to 62.585% when pulling 8bc50d4e3764892fd1629722bf22c1eaee8be3a9 on dwarfcrank:test-improvement into dc63253678b1a801eb809d8b6dc3cf23b0da781e on tdd-pingis:master.


Adding support for parsing directories, including optional recursion

Addressing issue #67
fyi @renatoi

CLA is valid!

Coverage remained the same at 100.0% when pulling 46ea85914cb3c9b65b1fe5877cd6eb499ed315d9 on cli-dir into 60d418d1d0e6023e8e0d1216c11222b98b280526 on master.

LGTM


The sample transfer function didn't work with .bashrc

When trying to use the provided function when queried without parameters in command line, I had the following error:
-bash: /root/.bashrc: line 20: syntax error near unexpected token `;;'

It comes from the "|tee /dev/null,;;, I removed a ; and the function works correctly.
My zsh-fu is not good enough to know if it's required by zsh.

@slurdge you are right (at least in bash), I'm not sure even about sone extra commas. I will fix the front-end as soon as I've time


Add-Incident occured a bug

Missing argument 9 for CachetHQ\Cachet\Commands\Incident\ReportIncidentCommand::__construct(), called in Cachet/app/Http/Controllers/Dashboard/IncidentController.php on line 121 and defined

Whoops. My bad.


Update nix config to use python 3

it is python 3

Can you update the title to something slightly more descriptive? Something like Update nix config to use python 3

Changed title


Str to DateTime converter

I have suggestion...
If we convert any types to specific type in StrField, InitField and so on, than would be nice to convert string representation of datetime in DateTimeField and DateField. I have found python library called dateparser (https://dateparser.readthedocs.org/en/latest/). I've tried it -> https://gist.github.com/boonya/47ac31b2d74122b459fa and it works but I am not sure that we really need this one )

Hey @boonya ,
You know, I'm not sure that it can be applicable to the current conception of the library, cause domain_models conception is about creation of some kind of easy-to-use & easy-to-integrate domain modelling system, but not a data converter / filter / parser / processor / serializer etc...
BTW, dateparser is awesome lib! Thanks for bringing it!

I close this this ticket, but feel free to reopen it, if you have any other ideas / suggestions about it.


Send :worker_idle signal

I've been playing recently with Celluloid and wanted to use it for a side project. I wanted to build my queue consumer library on Celluloid's pool but found it limiting. I wanted to have fixed size pool of workers processing given task with block and wait for idle ones to process another buffered tasks. After analyzing pool_manager code it turned out that small modification which is subject of this PR would satisfy my needs.
Here's sample code which uses this feature:
require 'celluloid/autostart'
require 'logger'

class Processor
  include Celluloid
  attr_accessor :logger

  def initialize
   @logger = ::Logger.new(STDOUT)
  end

  def process!(i)
    logger.debug("Processing task i=#{i}")
    sleep(10)
    logger.debug("Finished task i=#{i}")
  end
end

class Worker
  include Celluloid

  def run!
    pool = Processor.pool(size: 5)
    100.times do |i|
      pool.wait :worker_idle unless pool.idle_size > 0
      pool.async.process!(i)
    end
  end
end

Worker.new.run!

I'm not sure if calling pool.idle_size is thread safe and quite don't know how to test my addition.
@tarcieri what do you think? Does what I'm doing make sense? Maybe I can achieve what I want to do with other Celluloid primitives? (pool + condvar?)

Some thoughts on this:
The entire PoolManager needs to be rewritten.  Context:

“95% of the time standard synchronous RPCs will work - but not all the time, that's why it's nice to be
able to open up things and muck around at the message passing level.”
― Joe Armstrong

A lot of the work @halorgium did would enable it to be rewritten now. Here's what he was working on in that regard:
https://github.com/halorgium/celluloid-coordinator

Some thoughts on this:
The entire PoolManager needs to be rewritten.  Context:

“95% of the time standard synchronous RPCs will work - but not all the time, that's why it's nice to be
able to open up things and muck around at the message passing level.”
― Joe Armstrong

A lot of the work @halorgium did would enable it to be rewritten now. Here's what he was working on in that regard:
https://github.com/halorgium/celluloid-coordinator

Thanks Tony. I wasn't aware of any work in this area, @halorgium - are you actively working on this now a days? celluloid-coordinator looks like just an spike of an idea.


TCL files have an error

When i tried to use the azure theme for my project, tkinter gave me an error saying:
_tkinter.TclError: can't read "I(box-disabled)": no such element in array

I think i figured it out, for some reason, there is no image for 'box disabled', so the code using 'box-disabled' is trying to reference something that doesnt exist, which is why tkinter prints out an error.

I have just got this issue as well. Is it a new patch or smth?

there must be something wrong with the newer versions, i have an old version thats working, i can share the google drive link for you

Would be nice, thank you. Also do you know if I can still use ttkthemes in parallel with this? I have sort of theme selection panel.

Dont really know, but I think you could integrate the theme into the package if you know some TCL and some handling with packages.

Fixed with 965b527


Programmatically create a blocking task executor with a custom thread size

Currently, the number of threads of a blocking task executor could be configured only by Flags.numCommonBlockingTaskThreads().
If a user who avoid using flags, he or she has to manually copy the factory code.
I think we can add a new builder method to ServerBuilder that takes the number of threads.
public ServerBuilder blockingTaskExecutor(int numThreads) {
    return ...;
}

I am looking at this issue.
While reviewing the code, it looks to generateScheduledExecutorService, armeria supports one method that using SchedueldThreadPoolExecutor from java.util.concurrent package.
It turns out that ThreadFactory is used for spawning threads. One parameter 'threadNamePrefix' is required for this. Do you have any idea on it ?
Current usage is only for common pool (static name).

How about armeria-server-blocking-tasks? Because it is only used for Armeria server.

One question I have is: do we really need a dedicated builder method in ServerBuilder? Would it be better providing a utility class for this purpose, like we did in EventLoops?

like we did in EventLoops?

Maybe ThreadFactories? That sounds good.

do we really need a dedicated builder method in ServerBuilder?

The number of LINE internal standard blocking thread pool is 500. Many users copy and paste the following code since they want an explicit configuration than flags.
https://github.com/line/armeria/blob/964b7a5b7485d7d575f206dcd9a32117f56438f7/core/src/main/java/com/linecorp/armeria/common/CommonPools.java#L39-L45
I thought, if this pattern is frequently used, a dedicated builder method with sensible defaults looks good for convenience.

@trustin @ikhoon
public final class BlockingTaskExecutorBuilder {

    private final String threadNamePrefix;
    private int numThreads;
    private long keepAliveTime;
    private boolean allowCoreThreadTimeout = true;

    public static BlockingTaskExecutorBuilder builder(String threadNamePrefix) {
        return new BlockingTaskExecutorBuilder(requireNonNull(threadNamePrefix, "threadNamePrefix"));
    }

    public BlockingTaskExecutorBuilder numThreads(int numThreads) {
        checkArgument(numThreads >= 0, "numThreads: %s (expected: >= 0)", numThreads);
        this.numThreads = numThreads;
        return this;
    }

    public BlockingTaskExecutorBuilder allowCoreThreadTimeout(boolean allowCoreThreadTimeout) {
        this.allowCoreThreadTimeout = allowCoreThreadTimeout;
        return this;
    }

    public BlockingTaskExecutorBuilder keepAliveTime(long keepAliveTime) {
        checkArgument(keepAliveTime >= 0, "keepAliveTime: %s (expected: >= 0)", keepAliveTime);
        this.keepAliveTime = keepAliveTime;
        return this;
    }

    public ScheduledExecutorService build() {
        final ScheduledThreadPoolExecutor blockingTaskExecutor = new ScheduledThreadPoolExecutor(
                numThreads,
                ThreadFactories.newThreadFactory(threadNamePrefix, true));
        blockingTaskExecutor.setKeepAliveTime(keepAliveTime, TimeUnit.SECONDS);
        blockingTaskExecutor.allowCoreThreadTimeOut(allowCoreThreadTimeout);
        return blockingTaskExecutor;
    }

    private BlockingTaskExecutorBuilder(String threadNamePrefix) {
        this.threadNamePrefix = threadNamePrefix;
    }
}

This proof of concept is what you expected ?

like we did in EventLoops?

ThreadFactories? That sounds good.

EventLoops seems to mean EventLoopGroups
https://github.com/line/armeria/blob/4edffc8c11bd74665d652a1fd108d6a0307b6733/core/src/main/java/com/linecorp/armeria/common/util/EventLoopGroups.java#L44

@ikhoon Oh,, I didn't know that.
Let me look at that and create another code snippet to sync common sense.

@ikhoon @trustin maybe this snippet could be your common sense ?
public final class BlockingTaskExecutors {

    private static final String DEFAULT_THREAD_NAME_PREFIX = "armeria-common-blocking-tasks";
    private static final int DEFAULT_KEEP_ALIVE_TIME_SECONDS = 60;

    public static ScheduledExecutorService newBlockingTaskExecutor(int numThreads) {
        return newBlockingTaskExecutor(numThreads, true, DEFAULT_KEEP_ALIVE_TIME_SECONDS,
                                       DEFAULT_THREAD_NAME_PREFIX, true);
    }

    public static ScheduledExecutorService newBlockingTaskExecutor(int numThreads,
                                                                   String threadNamePrefix) {
        return newBlockingTaskExecutor(numThreads, true, DEFAULT_KEEP_ALIVE_TIME_SECONDS,
                                       threadNamePrefix, true);
    }

    public static ScheduledExecutorService newBlockingTaskExecutor(int numThreads,
                                                                   boolean useDaemonThreads) {
        return newBlockingTaskExecutor(numThreads, useDaemonThreads, DEFAULT_KEEP_ALIVE_TIME_SECONDS,
                                       DEFAULT_THREAD_NAME_PREFIX, true);
    }

    public static ScheduledExecutorService newBlockingTaskExecutor(
            int numThreads, boolean useDaemonThreads, boolean allowCoreThreadTimeout) {
        return newBlockingTaskExecutor(numThreads, useDaemonThreads, DEFAULT_KEEP_ALIVE_TIME_SECONDS,
                                       DEFAULT_THREAD_NAME_PREFIX, allowCoreThreadTimeout);
    }

    public static ScheduledExecutorService newBlockingTaskExecutor(int numThreads,
                                                                   boolean useDaemonThread,
                                                                   int defaultKeepAliveTime,
                                                                   String theadNamePrefix,
                                                                   boolean allowCoreThreadTimeOut) {
        final ScheduledThreadPoolExecutor blockingTaskExecutor = new ScheduledThreadPoolExecutor(
                numThreads,
                ThreadFactories.newThreadFactory(theadNamePrefix, useDaemonThread));
        blockingTaskExecutor.setKeepAliveTime(defaultKeepAliveTime, TimeUnit.SECONDS);
        blockingTaskExecutor.allowCoreThreadTimeOut(allowCoreThreadTimeOut);
        return blockingTaskExecutor;
    }

    private BlockingTaskExecutors() {}

}

Sounds good to me all in all. How about introducing a builder API so we don't have to have too many overloaded methods?


Action cleanup

Applied mkpr on .github/workflows/ci.yml
sed /      - run: npm ci/d

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

npm package (@latest dist-tag)
v1.11.62
GitHub release

Your semantic-release bot :package::rocket: