PermissionOverrideActionImpl doesn't inherit denied and allowed values from a PermissionOverride

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report

I created a lock command to take away write perms and whatnot from channels,
the unlock works by doing upsertPermissionOverride on a GuildChannel with a role as param
Recently I started getting strange behaviour and while debugging today I found that on the PermissionOverrideAction that the upsert returns, the allow and deny values were 0.
https://github.com/DV8FromTheWorld/JDA/blob/development/src/main/java/net/dv8tion/jda/internal/requests/restaction/PermissionOverrideActionImpl.java#L55-L62
see how there's no permission flags being set in this constructor or by methods called by this constructor
Expected Behavior
PermissionOverrideAction returns the actual current values of the existing permissionoverrides for the permissionholder that was passed to create it.
Code Example or Reproduction Steps
context.textChannel.upserPermissionOverride(roleWithSomeDeniedFlagsOnTextChannel).getDeny()

^^^ Will return 0 instead of the actual override flags when run in a channel were the bot has perms ofcourse.

Code for JDABuilder or DefaultShardManagerBuilder Used

N/A
Exception or Error

N/A

It currently returns the unused deny field, which is only updated if you make changes. This is by design, but we could argue to return getCurrentDeny() instead, which will return either the original or the updated value.

I'd be happy with any of the following:
option 1:
return the current permissions (might break some bots)
optionally add new methods that retain the current behaviour of getDeny and getAllow
option 2:
add methods getCurrentDeny/getCurrentAllow to PermissionOverrideAction
option 3:
Keep it as is and change the documentation to properly reflect what you're gonna get.

^^ I did some keyboard shortcut by accident

Oh some bonus side-effects,
When doing upsertPermissionOverride, all previous permissions are cleared except for the ones your changed.
This thus violates Creates a new override or updates an existing one. as it doesn't update anything, it just does a hard override like putPermissionOverride would do

When doing upsertPermissionOverride, all previous permissions are cleared except for the ones your changed.

That is not true, since they are applied to the current values in the update method.
https://github.com/DV8FromTheWorld/JDA/blob/development/src/main/java/net/dv8tion/jda/internal/requests/restaction/PermissionOverrideActionImpl.java#L285

fix in https://github.com/DV8FromTheWorld/JDA/commit/ae39561567d068efff707c3e433117f1d2b75df3
this can be closed then I assume ?


Redirect users away from mbtace.com

We want all users, including search engine robots, to use the canonical
mbta.com hostname. The exception is mTicket, which uses a special
hostname that we detect to flag the request for special handling; there
is a carveout for that special hostname.
Summary of changes
Asana Ticket: mbtace.com being indexed/viewed by public

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

One of those rare cases where Plug is a good solution :D
I was thinking this ticket could be satisfied by some changes to the nginx.conf, but this is probably more maintainable.

@ryan-mahoney yeah my preference is to keep it in the code, kind of have everything together in one place where all the devs can see and touch it.


SEPS525 supported?

Hello,
Is it possible to add support for SEPS525 display?
I'm using this display in a VMU mod with Pi Zero, but the refresh rate is killing the experience...
Datasheet: http://www.newhavendisplay.com/app_notes/SEPS525.pdf
The mod: https://sudomod.com/forum/viewtopic.php?t=6917
I hope something could be done about this...

The display does look like it could be supported within current driver design. I don't unfortunately have the display (or even if I did, unfortunately not able to spend time to work on adding support at the moment).
If someone has the display and is able to find a spec sheet or reverse engineer the bus, pull requests would be welcome!

As far as I read the bus is very well described in the attached datasheet?
Unfortunately my programming skills are not enough to pull this off, that's why I asked here...
Since the bus i documented I could try, but I would need some guidelines how to start.

Oh right - indeed, the spec sheet documents the protocol. In general the porting process is as follows:

Write the initialization sequence for the display. E.g. ILI9341 has it here
Write the pixel cursor update sequence protocol, e.g. for ILI9341 here.
Write any needed display-specific color conversion e.g. here
Address any other display-specific aspects that the display is incompatible with against the existing driver (e.g. partial window sets, DMA, or something else)

Thanks for the fast answers! I'll try to find time to dig into this.


CWV scores improvements

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #231
Test URLs:

Before: https://main--icicidirect--aemsites.hlx.live/research/equity

After: https://cwv--icicidirect--aemsites.hlx.live/research/equity

[ ] New Blocks introduced in this PR
If yes, please provide details below

Block Name
Documentation

For e.g. carousel
doc-link

Major pending issues that can't be fixed without affecting the UX/theme of the page are:

Color contrast due to orange theme of the page - the recommended color combination that would fix the contrast ratio would be black text with orange background

The space between the header primary buttons to fix the touch target gap. Inceasing the space between the buttons would move the buttons in 2 rows which is different from the original page
Alt text missing on the chatbot image, since it is loaded in a lazy way from external source, alt could not be added on the images


Update arc to 0.11.0 and azcli to 0.3.0 - Stable

Updating extension gallery of stable build to have new arc-0.11.0 and azcli -0.3.0 for the Azure Arc enabled Data Services release today.
I will merge in this PR after this one is approved and merged in to bump the version numbers of the extensions in their respective package.json: https://github.com/microsoft/azuredatastudio/pull/17932

Can you please add this to https://github.com/microsoft/azuredatastudio/pull/17934? They don't need to be separated


Integrate with Hue automatically

Currently, this integration requires the user to obtain the Hue API key and add it manually to the sensor.py file (see ReadMe). The current aim is to pull the Hue API key from the HA config and use the value in the sensor.py file. This is being worked on actively, but my knowledge of HA is limited overall. I will keep this issue open and updated regularly with progress. Please contribute if you know how to do this.
Thank you!

This issue was solve with the latest update to Sensor.py. Hue Bridge is now configured automatically provided you have added your Hue bridge successfully to HomeAssistant. Manual config is still possible. Please view the README for more info.


Use ConfigLoader.default_configuration for default

When the config file path is same as the default config file,
use ConfigLoader.default_configuration instead of create a new config.
Because some plugins add their default config values to
ConfigLoader.default_configuration. So the Cops of the plugins expect
that default config values exists.
Fix https://github.com/rubocop-hq/rubocop-rails/issues/127

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

@hanachin Sorry for the slow response! The changes look good, but you'll have to rebase on top of master.


AI Service not available - no error logs

I added a basic camera - rtsp://admin:password@<IP_ADDRESS>:554
Why am i getting "AI Service not available" when I try to access AI? There are no error logs in docker compose.
How can I debug this?

I have the same error, did you solve it?

No, I just moved on


Remove the setup counter from the KVMachine ready status

What this PR does / why we need it
Cluster-API takes the reason and the message from the infra-machine (in our case, the KubeVirtMachine) Ready condition, into the cluster-api Machine's InfrastructureReady condition. Currently the text is something like "0 of 2 completed". This is not very informative.
This PR changes the behavior so now the KV Machine Ready condition will include both the reason and the message from the failed condition.
That way, the cluster-api InfrastructureReady condition, will also contain the same reason and message. This will make the actual issue, if exists, to more accessible to the user.
Release notes:
Remove the setup counter from the KV-Machine ready status

/ok-to-test


Make sure we can add resources with same name. Ie no overwrite

This will fix issue #31

Thank you for the very fast response! After applying the fix, the parts are no longer overwritten.
As it stands, this is most probably a BC break as the old behavior is explicitly mentioned in the docs:

If the same $name is used twice the first resource will be overwritten.

Thank you. I missed that... I wonder why I would write that.
I need to do some more research before I can merge this. It seams to be the right thing to do but Im not sure..

So, here is what I concluded:
$builder->addResource('foo', 'content');

// Same as
$builder->addResource('whatever', 'content', ['content-disposition'=>'form-data; name="foo"']);

One could remove the first parameter of addResource and put it in the options array as a header (like the example above) or a new "name" key. But neither are not preferable since 99% of the cases you would never use the third parameter of addResource (options array). You would only care about the first two.

Resource names SHOULD be unique. If same name is used one SHOULD use the array syntax. Ie "foo[0]", "foo[1]" etc. But this is not a hard requirement (MUST) and I think we should be flexible rather then opinionated.
I vote for merging this PR in the current state and release 0.2.0.

The difference between 0.1.x and 0.2.0 would be that all resources you add to the builder will be added to the stream. You cannot overwrite. Though, resources may have the same POST name and some servers cannot handle that.

Sounds great! Thank you for taking a look at this and fixing it so quickly!

I agree with you.
I just wanted to mention that one could remove the first parameter because it is redundant and not strictly needed. But I think we should keep it since it is way more convenient and covers 99% of the cases. I expect it to be an edge case to configure your own headers.


Expand Referenced Tweets

Currently it leaves referenced_tweets list alone, the column in the CSV ends up like this:
[{"type": "replied_to", "id": "1380226330034372610"}]
[{"type": "quoted", "id": "1380226330034372610"}]
[{"type"": "retweeted", "id": "1261081519566675969"}]

but we could expand this into separate columns:
referenced_tweets.replied_to
referenced_tweets.quoted
referenced_tweets.retweeted

and by extension, type column should be a list like ["reply"] or ["retweet","reply","quote"] if it's a quote tweet that's a reply to someone that was then retweeted. type should also be __inferred_tweet_type or something to indicate where this field is coming from.

The CSV would endup looking like:
referenced_tweets.replied_to,referenced_tweets.quoted,referenced_tweets.retweeted,
,,1261081519566675969,

referenced_tweets.replied_to,referenced_tweets.quoted,referenced_tweets.retweeted,
1380226330034372610,,,

etc

Followup: https://twittercommunity.com/t/usersretweets-problem-with-json-normalize-to-flatten-the-nested-json/157713


Add setter for JProperty name property

Currently there is no way to change JProperty name after creation, but looking into source code I do not see any technical limitation. In our case we would like to modify name if JProperty (we have little DSL language build on top of JSON and we are using Json.net to work with it).
Will it be possible to add setter Property name property?
James, please let me know if you ok with it? I will submit PR including unit tests.

I'd rather it was read-only. The property name is cached in a dictionary on a JObject for lookup performance for example and I'd have to raise an event to notify the JObject. Easy enough to remove and readd a new JProperty with the new name.

I tried to Remove and Add, but it is not working as I expecting. Here is example to illustrate problem.
var json = @"{
    ""prop1"": {
        ""foo"": ""bar""
    },
}";

var obj = JsonConvert.DeserializeObject<JObject>(json);

var property = obj.Property("prop1");
var value = property.Value;

property.Remove();
obj.Add(new JProperty("prop2", value));

Debug.Assert(((JProperty)value.Parent).Name == "prop2", "Problem!");

You get that result because a value will be copied if it already has a parent.
var json = @"{
    ""prop1"": {
        ""foo"": ""bar""
    },
}";

var obj = JsonConvert.DeserializeObject<JObject>(json);

var property = obj.Property("prop1");
var value = property.Value;

// remove value so it has no parent
property.Value = null;

property.Remove();
obj.Add(new JProperty("prop2", value));

Assert.True(((JProperty)value.Parent).Name == "prop2", "Problem!");


drm command line options do not work as documented

Seems like the --match and --exclude options are doing the opposite of what the documentation says.
Running drm --dryrun -v --name --match file_* has no output (other than "Walking ...") indicating that no files are deleted.
Running drm --dryrun -v --name --exclude file_* prints
Type=REG File=/p/lustre1/gonsie/file_00000007
Type=REG File=/p/lustre1/gonsie/file_00000008
Type=REG File=/p/lustre1/gonsie/file_00000016
Type=REG File=/p/lustre1/gonsie/file_00000017
Type=REG File=/p/lustre1/gonsie/file_00000022
Type=REG File=/p/lustre1/gonsie/file_00000023
Type=REG File=/p/lustre1/gonsie/file_00000031

meaning that those are deleted (which is the opposite of what exclude should do).
Also, the final status "Walked 353 files in ..." is misleading. Those are the items that matched the regex, not the total number of items analyzed.

Yeah, this is a touchy option, because it takes POSIX regex syntax (from regexec) rather than shell regex.  Also, you have to be sure the shell is not interpreting things before it gets passed to the command.  I think the following would be the right syntax:
drm --dryrun -v --name --match 'file_.*' /p/lustre1/gonsie/*

Note the single quotes to prevent the shell from expanding the "*" and the added "." to tell the regex that you want any character repeated, rather than zero or more of "_".  Also, I added the /p/lustre1 argument to provide the paths to be walked, from which to then match or exclude items.
We should think about this interface, since the syntax mix up is easy to do, and as you have shown, it can lead to exactly the wrong result.

@gonsie @adammoody
Added a note to the drm page about not using shell here: https://mpifileutils.readthedocs.io/en/latest/drm.1.html


add PLOOC header files

this will fix include errors:
../mqttclient/mqttclient.h:39:10: fatal error: plooc_class.h: No such file or directory
#include "plooc_class.h"
^~~~~~~~~~~~~~~
compilation terminated.

thanks for yous commit!


Is possible in google docs?

Is possible in google docs?

Yes it is possible, Check this sample https://github.com/Autodesk-Forge/model.derivative-nodejs-google.drive.viewer


Develop - feedback rendered

Checklist before merging

[x] Documentation updated
[x] Config.json changes reflected in source/get_data.validate_config
[x] Version number updated in pyproject.toml
[x] Completion script updated
[x] Completion script updated in commands/info.completion()
[x] Changelog updated in commands/info.changelog() (only empty changelog after merge into master)

[!WARNING]
Publishing workflow is disabled, because it will be changed to trigger on a release/tag instead of merged PR


🔐 Amend IAM Roles to include lakeformation:GetDataAccess permission

User Story
AP users need to be able to access their data based on their IAM permissions even when specific datasources are managed in LakeFormation. This needs to be a prerequisite to enrolled existing datasources into LakeFormation and sharing them for use in QuickSight, so that QS MVP does not block other platform business.
Value / Purpose
LakeFormation allows a simpler and more granular control of data and is a preferred way of managing permissions to Glue resources than IAM.
Useful Contacts
Julia Lawrence Michael Collins James Sott
User Types
No response
Hypothesis
If we add lakeformation:getdataaccess permission to current IAM roles, we will not cause interruption in service once LakeFormation is live.
Proposal

Update all alpha roles to include the lakeformation:getdataaccess permissions using terraform datacall.
Update all app roles in a similar way
Review AP service IAM roles that deploy or interact with Glue resources to include this permission. For roles managed in code, this needs to be applied via code.
Communicate with AP users on what we're doing and why as well as provide troubleshooting instructions to fix errors that might arise.

Additional Information
Updating alpha and app roles is fairly straight-forward but the risk is around missing a role or updating it manually when it is in code. It's key that we communicate this out broadly so that users whose roles were missed will be able to get themselves fixed quickly.
Definition of Done

[ ] Comms drafted and sent out
[ ] Alpha roles updated
[ ] App roles updated
[ ] Other roles updated
[ ] Support guidance created

This isn't needed until we start transitioning to full LakeFormation. Things should work as is in hybrid mode.

This isn't needed until we start transitioning to full LakeFormation. Things should work as is in hybrid mode.


Triage finschia-sdk#196

The subject of the triage:

Finschia/finschia-sdk#196

This PR needed to be abandoned. Because cosmos is using sonarcloud instead of code coverage.
The related PR:

cosmos/cosmos-sdk#11856
cosmos/cosmos-sdk#13944

Note:
We need more information about pros and cons of sonarcloud.


Adding code Coverage report section

This Pull Requests add codeCoverage report section in Maven Task.  This currently adds only Jacoco tool.

approve or close


Deprecation Warning from macros/schema_tests/not_null_proportion.sql

Describe the bug
dbt run on a project that has dbt-utils installed will trigger a Deprecation Warning.
Steps to reproduce
Set up a simple dbt project with dbt-utils installed.
Expected results
dbt run gives no warnings.
Actual results
dbt run produces a Deprecation Warning.
Screenshots and log output
(env) emily@Emily-Rockman jaffle-shop % dbt run
Running with dbt=0.21.0-b1
* Deprecation Warning: The "packages" argument of adapter.dispatch() has been
deprecated. Use the "macro_namespace" argument instead.
Raised during dispatch for: test_not_null_proportion
For more information, see:
https://docs.getdbt.com/reference/dbt-jinja-functions/dispatch
Found 2 models, 4 tests, 0 snapshots, 0 analyses, 353 macros, 0 operations, 0 seed files, 0 sources, 0 exposures

13:22:41 | Concurrency: 4 threads (target='dev')
13:22:41 |
13:22:41 | 1 of 2 START table model dbt_testing.my_first_dbt_model.............. [RUN]
13:22:41 | 1 of 2 OK created table model dbt_testing.my_first_dbt_model......... [SELECT 2 in 0.09s]
13:22:41 | 2 of 2 START view model dbt_testing.my_second_dbt_model.............. [RUN]
13:22:42 | 2 of 2 OK created view model dbt_testing.my_second_dbt_model......... [CREATE VIEW in 0.06s]
13:22:42 |
13:22:42 | Finished running 1 table model, 1 view model in 0.29s.

Completed successfully

Done. PASS=2 WARN=0 ERROR=0 SKIP=0 TOTAL=2

System information
The contents of your packages.yml file:
packages:
  - package: dbt-labs/dbt_utils
    version: 0.7.2

Which database are you using dbt with?

[x] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.21.0-b1

Additional context
This is related to the recent changes in Cherry Pick: not null proportion schema test #411.  macro_namespace should be used instead of packages in macros/schema_tests/not_null_proportion.sql.   See docs for more info.
Are you interested in contributing the fix?

Good call @emmyoop, thanks for opening!
@jasnonaz Apologies for not catching the outdated dispatch usage in #375/#411. This should be as simple as replacing this line:
https://github.com/dbt-labs/dbt-utils/blob/c7be76a2715d897755390e2a9a29d70026ae73c3/macros/schema_tests/not_null_proportion.sql#L2
With:
{{ return(adapter.dispatch('test_not_null_proportion', 'dbt_utils')(model, **kwargs)) }}


Injection CSS with Browser-sync

First, congratulation for the course, it's great.
Second, I'm having problems to figure it out why my browser is reloading on inject css. I tried to find help on the plugin docs, it's seems to be updated and it's a little bit different of your implementation, so, no success.
I copied and paste the exercise gulp files, but it's keep not working, the browser is reloading when I change the less file.
Thanks in advanced !

I have the same problem.
versions:
gulp --v
[16:09:25] CLI version 3.8.11
[16:09:25] Local version 3.9.0

browsersync: 2.7.13

@johnpapa is there a version that we can use and will work with your course?

I'm having the same issue.
I was using Safari as my default browser. Changed to Chrome, but issue still exists.
Has anyone found the solution?

@annabellor versions gave me a clue. I went back to the course and checked which versioni @johnpapa  installed when he ran npm install --save-dev browser-sync and realized it the current version at the time was 1.8.2.
So I went back and reverted my project to use 1.8.2 and voila! Injection of the CSS started working.
I guess it is an issue with the current version of browser-sync.

Hmmmm. I see this issue related here https://github.com/BrowserSync/browser-sync/issues/717

It appears that the latest version of browser-sync isn't paying attention to the '!' + config.less line in options.files, and the refresh is getting triggered by the **/*.*.  Updating options.files to only include the files to listen to seem to fix the problem.
files: [
	config.client + '**/*.js',
	config.client + '**/*.html',
	config.client + '**/*.jpg',
	config.client + '**/*.png',
	config.client + '**/*.gif',
	config.temp + '**/*.css'
]


PersistentDictionary max file size

Hi all,
I've been trying to work out how to somehow configure the PersistentDictionary to limit the max file on disk. I've looked at DatabaseConfig.cs and played around, but no luck.
Any suggestions?
Cheers

Ah! I should have endured for another 5 minutes. I was able to limit the filesize using a combination of the DatabasePageSize and DatabaseMaxPages properties of the DatabaseConfig linked above.
e.g.
var config = new DatabaseConfig();
config.DatabasePageSize = 4096;
config.DatabaseMaxPages = 2000;

var persistentDictionary = new PersistentDictionary<string, string>(
                directory: "Test",
                customConfig: config);


Win7 Import Task Fail : Format Invalid

When I try to import task xml, I receive an error.

Win7 Home Prem:SP1

@AMNegron Sorry for your inconveniences, the version of the release XML file is 1.4 and you need 1.2 for Windows 7. I did a quick rework of the original file. Use this in the meantime until I create the next release which will contain a legacy file. Thanks for reporting!
Here the file you need:
Network_Share_Reconnecter.zip

@AMNegron v1.3.6 released which includes the new task file.
/closed


[New game]: Bunny is Lost!

🎮 Game Request
Game Logic:
In "Bunny is Lost!", players control a bunny escaping a forest fire by jumping over obstacles and collecting carrots to maintain health. The game starts with a "Run!" button and features a main loop updating the bunny's movement, health, and collision with enemies like birds. The game ends if the bunny's health drops to zero or it gets caught by a bird.

Point down the features
Game points Features:

Dynamic scaling for different devices
Procedurally generated background music
Interactive controls via keyboard or touch
Collectible carrots for health
Challenging obstacles and enemies
Visual health and jump indicators
Endgame feedback on bunny’s fate

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @divyansh-2005 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Please assign me this task so that I can work on it as soon as possible under GSSOC'24.

/assign

/assign

Hey @divyansh-2005 ! , This issue is already assigned to @shivan2004. 💗
You can work on other issues 🚀
If there is anything wrong you can contact us on Discord🕹👀

Issue was created my me. Kindly assign me, I have the code ready for the game.

Hey @divyansh-2005 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


[WIP] Save given and family name separately on CFPs

Fixes #997

There are several TODOs:

improve UI wording (see attached image)
eventually, we should datamigarate our data, so we don't have to do checks like first_name.endswith(last_name) [may be out of scope for this one?]

The test you're mentioning is from AssopyUser.name, right? I'd leave removing that outside of the scope, I'd need to confirm production data has been fixed.

Thanks for you work on hits, @aleszoulek !

The changes from this MR have been moved to https://github.com/EuroPython/epcon/pull/1147 and additional tests have been added. Thanks for your work on this, @aleszoulek !


Glowsticks, flares and lighting behaviours

This PR adds these things:

LightingBehaviourComponent - It stores various lighting behaviour definitions and allows you to apply them to the point light component on the parent entity.

ExpendableLightComponent - Used on expendable things like glowsticks and flares to handle their various icon states, sound states, and lighting states.

Right now i have these functioning LightingBehaviours:

Pulse (the chosen property interpolates between a min and max value using user's choice of nearest/linear/cubic).
Color cycle (light cycles through a list of colors using nearest/linear/cubic).
Fade (property interpolates from min to max value using nearest/linear/cubic).
Randomize (property interpolates from current value to a random value chosen between min and max values using nearest/linear/cubic).

Things that would be nice to have:

The ability to chain behaviours... So if behaviour A is told to play behaviour B on completion then it handles that logic on its own.

The idea is to make this thing as modular and extensible as possible so it can do some heavy lifting on entities that have point lights that require various states/behaviours. Instead of hand-writing behaviours for fading out and such, you'd just get the lighting behaviour component to handle it.
You have pretty granular control over how the light behaves with all of the various parameters. You can also have more than one of these behaviours defined in the prototype and you can toggle specific behaviours on and off using their given string ID (and even have several active at the same time). This means you have a pretty flexible system for giving a point light different states and behaviours that can be quickly managed via code and tweaked via yml for quick iterating.
Gifs made during the creation process (a bit old but you get the idea):
Toggle: https://cdn.discordapp.com/attachments/310555209753690112/747295854678769736/stre.gif
Random brightness: https://cdn.discordapp.com/attachments/310555209753690112/747295476906328124/rand.gif
Flicker: https://cdn.discordapp.com/attachments/310555209753690112/747294794983669770/flik.gif
Color sequence smoothed: https://cdn.discordapp.com/attachments/310555209753690112/747278967093985430/fdf.gif
Color sequence smoothed with rainbow colors: https://cdn.discordapp.com/attachments/310555209753690112/747280579824844800/ffff.gif
Using a random brightness behaviour on a flare: https://cdn.discordapp.com/attachments/310555209753690112/748025062640320622/test7.gif
Example of how to use it:
(a bunch of other components for whatever entity this is)
...
    - type: PointLight
      enabled: false
      color: red
      radius: 10.0
      energy: 10.0
    - type: LightBehaviour
      behaviours:
        - !type:RandomizeBehaviour # makes it bright and flickery like a campfire
          id: flickery_mode
          interpolate: Nearest
          minDuration: 0.02
          maxDuration: 0.06
          startValue: 5.0
          endValue: 10.0
          property: Energy
          isLooped: true
        - !type:FadeBehaviour # shrinks the radius over the course of 10 seconds
          id: fade_out
          interpolate: Linear
          maxDuration: 10.0
          startValue: 10.0
          endValue: 2.0
          property: Radius
        - !type:ColorCycleBehaviour # alternating red and blue light, police car style
          id: get_down_on_the_fucking_ground
          interpolate: Nearest
          maxDuration: 0.5
          isLooped: true
          colors:
            - red
            - blue

And then to make use of these definitions from the YML you'd just do this on the clientside:
myLightBehaviourComp.StartLightBehaviour("get_down_on_the_fucking_ground"); // turn on the police lights

myLightBehaviourComp.StopLightBehaviour("get_down_on_the_fucking_ground"); // turn off the police lights

myLightBehaviourComp.StartLightBehaviour(); // turn on all 3 of the behaviours. probably looks like shit but you get the idea.

myLightBehaviourComp.StopLightBehaviour(); // turn off all

Things to discuss

Should this be moved elsewhere? Like perhaps a visualizer since all the behaviour is clientside? (My opinion is no, because visualizers have some restrictions that would make it really awkward to use this).
Is anything missing/done in a bad way? The property system doesn't have any safety rails but there's only so many properties of lights that people want to animate (to be specific, Radius, Energy, Rotation, Color and Enabled seem like the only properties of interest).

Alright this is good for review now. 🙏

Things i need to do:

Make the expandable light component use a visualizer for the iconstates
Refactor lighting behaviour to use a custom animator instead of this monstrosity it has become
Attribute the sprites to me since i did em

Submodule needs an update in order for the checks to pass. Otherwise i think i'm done making changes.

Related to #1129


Moving generated styles to a css folder and changing all routes related with fonts due to this change

So now, dist folder will look like...

REVIEWER: @javierarce

:cool: :cool: :cool:

Much appreciated buddy! :)


Error: could not open STL file '/home/longu/Dexterous Manipulation with Reinforcement Learning/mj_envs-master/mj_envs/hand_manipulation_suite/assets/../../../dependencies/Adroit/resources/meshes/forearm_simple.stl'

Hi! I'm wondering how to solve this error
Error: could not open STL file '/home/longu/Dexterous Manipulation with Reinforcement Learning/mj_envs-master/mj_envs/hand_manipulation_suite/assets/../../../dependencies/Adroit/resources/meshes/forearm_simple.stl'
when I execute
python utils/visualize_demos.py --env_name relocate-v0
It seems that I have finished all previous steps in the "Getting started", and now I get stuck.
I also tried to drag DAPG_hammer.xml to the MuJoCo simulate, and the same error occurred.
~/.mujoco/mujoco200/bin$ ./simulate
MuJoCo Pro version 2.00
Error: could not open STL file '/home/longu/Dexterous Manipulation with Reinforcement Learning/mj_envs-master/mj_envs/hand_manipulation_suite/assets/../../../dependencies/Adroit/resources/meshes/forearm_simple.stl'

All right, I solved it by downloading the files in mj_envs/dependencies/Adroit/
If you download the mj_envs directly, no files exist in directory mj_envs/dependencies/Adroit/


Compilation issue on Example program of TF Lite

System information

OS Platform and Distribution : Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:Desktop
TensorFlow installed from (source or binary): Source https://github.com/tensorflow/tensorflow.git
TensorFlow version: Release 1.15.0
Python version:
GCC/Compiler version (if compiling from source): gcc (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Default compilation of example folder (Hello World) leads to missing header file reference and shows error.
Provide the exact sequence of commands / steps that you executed before running into the problem

Install Ubuntu 18.04 OS
Download git using command "git clone https://github.com/tensorflow/tensorflow.git"
Install build essentials using command "sudo apt install build-essential"
Install curl library using command "sudo apt install curl"
Issue command "make -f tensorflow/lite/micro/tools/make/Makefile test_hello_world_test"

Any other info / logs
make -f tensorflow/lite/micro/tools/make/Makefile test_hello_world_test
g++ -O3 -DNDEBUG -std=c++11 -g -DTF_LITE_STATIC_MEMORY -fno-rtti -DTF_LITE_DISABLE_X86_NEON -I. -Itensorflow/lite/micro/tools/make/downloads/ -Itensorflow/lite/micro/tools/make/downloads/gemmlowp -Itensorflow/lite/micro/tools/make/downloads/flatbuffers/include -Itensorflow/lite/micro/tools/make/downloads/kissfft -c tensorflow/lite/micro/kernels/logical.cc -o tensorflow/lite/micro/tools/make/gen/linux_x86_64/obj/tensorflow/lite/micro/kernels/logical.o
In file included from ./tensorflow/lite/kernels/internal/reference/binary_function.h:18:0,
from tensorflow/lite/micro/kernels/logical.cc:16:
./tensorflow/lite/kernels/internal/common.h:24:10: fatal error: fixedpoint/fixedpoint.h: No such file or directory
#include "fixedpoint/fixedpoint.h"
^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
tensorflow/lite/micro/tools/make/Makefile:257: recipe for target 'tensorflow/lite/micro/tools/make/gen/linux_x86_64/obj/tensorflow/lite/micro/kernels/logical.o' failed
make: *** [tensorflow/lite/micro/tools/make/gen/linux_x86_64/obj/tensorflow/lite/micro/kernels/logical.o] Error 1

I've tried locally and am not seeing the same issue you see.  I'm wondering if one of the downloads may have failed for you.
Please try running
make -f tensorflow/lite/micro/tools/make/Makefile clean_downloads
make -f tensorflow/lite/micro/tools/make/Makefile clean
make -f tensorflow/lite/micro/tools/make/Makefile test_hello_world_test

Hi Nat,
Its working fine after Executing mentioned steps. I believe it should be download failure of Header file "tensorflow/lite/micro/tools/make/downloads/gemmlowp/fixedpoint/fixedpoint.h"


Saturation Functions Patch

Patch RegionOutput() to correctly handle detection of Region 4 when provided (T, p) state point on the saturation curve from a saturation function that provides the desired vapor/liquid state.  Closes #19.
In addition:

[x] Exception for non-saturation function on saturation curve changed to "out_of_range".
[x] Mathcad wrapper catches a "saturated" condition correctly in all TP wrapper functions.
[x] Additional error flag in Mathcad wrapper added for "UNKNOWN" exception thrown.

Tested in MinGW and VS compilers for correct behavior of saturation functions.

@ibell - Better?

Yes, exactly - you should always wrap branches in conditionals unless it is on one line : https://google.github.io/styleguide/cppguide.html#Conditionals


Adding implementation for Hedera ID Change backwards compatibility.

Signed-off-by: Nikolay Nedkov<EMAIL_ADDRESS>Detailed description:
Which issue(s) this PR fixes:
Fixes #633
Special notes for your reviewer:
Checklist

[ ] Documentation added
[x] Tests updated

You have not merged main properly. Things that have been committed in main show as changes..


Missing permission: cluster:admin/opendistro/ad/detector/info

When creating a detector in Anomaly Detection, I get and error that I don't have permissions for "[cluster:admin/opendistro/ad/detector/info]".
I have a role containing these cluster permissions:

cluster:admin/opendistro/ad/detector/delete
cluster:admin/opendistro/ad/detector/jobmanagement
cluster:admin/opendistro/ad/detector/run
cluster:admin/opendistro/ad/detector/search
cluster:admin/opendistro/ad/detector/stats
cluster:admin/opendistro/ad/detector/write
cluster:admin/opendistro/ad/result/search
cluster:admin/opendistro/ad/detectors/get

When I go to add the missing permission to the role, the GUI says it doesn't exist (can't auto-complete in the Cluster Permissions field).
What gives? Do I need to run securityadmin.sh again, and will that override my current configuration?

I was able to add the permission to my role through the API, but I still think the permission should be auto-completeable.
Also, why is one of them called /detectors while the rest are /detector?

The API didn't work as documented, though. I used the PATCH _opendistro/_security/api/roles/<role> API, which for some ungodly reason wants a raw JSON array without enclosing {} braces.
I sent:
PATCH _opendistro/_security/api/roles/anomaly_detectors
[
  {
    "op": "add", "path": "/cluster_permissions", "value": ["cluster:admin/opendistro/ad/detector/info"]
  }
]

But this removed all the other permissions. I had to include them all in the API request.


Complete all descriptions of functions. Add a test of CompressedAirSystem.

Complete all descriptions of functions. Add a test of CompressedAirSystem.

good job


Collapse’s aria-expanded attribute never leaves true state.

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
When a Collapse trigger is clicked, it should update the trigger’s aria-expanded attribute from "true" to "false" and vice versa. The test case shows that only the change from "false" to "true" happens. The attribute never gets updated again when the trigger is clicked again.
Note that the change from "false" to "true" doesn’t happen either if the div.collapse is removed. This might be by design, but I think this is a bug, too. There might be nothing to display at the time the trigger is clicked, but IMHO, the element still got expanded and should reflect that.
The reason I care about this is because I show an icon to display if a collapsible element is currently collapsed. Similar to the demo, I update the icon in CSS based on the trigger’s aria-expanded attribute.
Reduced test cases
https://stackblitz.com/edit/stackblitz-starters-zhsmta?file=index.html
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Chrome, Firefox
What version of Bootstrap are you using?
5.3.3

Thanks for reporting an issue @mwaibel-go
I just looked at your StackBlitz reproducible example.
Maybe I'm wrong, you'll tell me, but It seems that changing the class name from .collapse to anything else actually works (sounds like it works in our docs for instance):

  Collapse trigger

aria-expanded = true

aria-expanded = false

-
to be hidden

+
to be hidden

That could be explained by the fact that the .collapse class changes during the animation and the change of states. Targetting with data-bs-target the class .collapse is maybe to avoid, and it would be better to target an id (or another "safe" class name).

It seems that changing the class name from .collapse to anything else actually works

You’re right. When I debugged this, I didn’t notice that the .collapse class gets removed from the target elements for the duration of the transition.
What seems to happen is this (quoting collapse.js, ll. 183–191)
    this._element.classList.remove(CLASS_NAME_COLLAPSE, CLASS_NAME_SHOW)

    for (const trigger of this._triggerArray) {
      const element = SelectorEngine.getElementFromSelector(trigger)

      if (element && !this._isShown(element)) {
        this._addAriaAndCollapsedClass([trigger], false)
      }
    }

So now I know how to work around my specific issue, but I still think it’s odd to skip _addAriaAndCollapsedClass when there are no elements to collapse. As I said earlier, the element did get (un)collapsed, even though there’s no visible change in the UI.
First the .collapse class gets removed on the target element. This causes the selectorEngine to not find the element anymore (because the data-bs-target attribute specifies elements with the collapse class). Therefore element is null in this snippet, which causes the _addAriaAndCollapsedClass invocation to be skipped.

I'm glad the workaround was helpful! I'll keep this issue open so we can explore it further. Thanks for your detailed response 🙏.


How to create splash screen ?

I have seen few samples they have added few lines in native code. Since I don't find any native code. How can I achieve it in create-react-native-app ?

have a look here: https://docs.expo.io/versions/latest/guides/configuration.html#loading

Hi! As indicated in the docs, adding native code is only possible if you eject from CRNA or use react-native init. Thanks!

also @mathieudutour is correct that you can use that loading api. we are also currently working on an improve api for splash screens, which will land in sdk22, so check out our release notes at blog.expo.io for that in the next couple of weeks


bump fresco to 2.3.0

Summary
Bump Fresco to 2.3.0.
Changelog
[Android] [Changed] - Bump Fresco to 2.3.0
Test Plan

Hmm actually, test_android failure seems related:
java.lang.NoClassDefFoundError: Could not initialize class com.facebook.imagepipeline.platform.KitKatPurgeableDecoder

Can you double check?

It looks like failing to load native libraries, and I'm not good at Buck to resolve the issue. Could you please ask someone in FB for help.

I'll find time to investigate next week, but expect some delay.

I maintain a lot of repos and I dislike it when people just "bump" things, at the same time, I am sort of powerless to help on this one and I actually tried with a related PR :-), because I don't have access to the internal systems/logs. Any chance of investigation and merge on this one? Other than buck-related things this should work...

@dulmandakh entropy is starting to take hold on this one, with a conflict

@dulmandakh entropy is starting to take hold on this one, with a conflict

Sorry I was quite swamped with the new architecture work for RNTester, I’ll try to pick this up again early next week. Any help figuring out the Buck issue would be very useful as well.

Sorry I was quite swamped with the new architecture work for RNTester, I’ll try to pick this up again early next week. Any help figuring out the Buck issue would be very useful as well.

I wonder if the failure above will self-resolve after resolving conflicts and re-pushing here. I say that because I noticed "KitKat" in the error and I believe minSdkVersion is 21 now, KitKat is 19. One can hope

I wonder if the failure above will self-resolve after resolving conflicts and re-pushing here. I say that because I noticed "KitKat" in the error and I believe minSdkVersion is 21 now, KitKat is 19. One can hope

Yeah that’s possible, I was just thinking about that. @dulmandakh wanna try rebasing?

Yeah that’s possible, I was just thinking about that. @dulmandakh wanna try rebasing?

Got it, will do soon

Got it, will do soon

So I looked into this a bit. This PR was days before the commit to bump test to run SDK 21: https://github.com/facebook/react-native/commit/7dcecb2907a2f4046ed4729ae183f30516247074
So to save time, I'm going to merge this PR and we'll fix forward any breakage after.

So I looked into this a bit. This PR was days before the commit to bump test to run SDK 21: https://github.com/facebook/react-native/commit/7dcecb2907a2f4046ed4729ae183f30516247074
So to save time, I'm going to merge this PR and we'll fix forward any breakage after.

This seems to pass: https://app.circleci.com/pipelines/github/facebook/react-native/7775/workflows/1b8515a7-bb5b-44ce-a404-674d04b828da/jobs/184957
So we're all good to go.

This seems to pass: https://app.circleci.com/pipelines/github/facebook/react-native/7775/workflows/1b8515a7-bb5b-44ce-a404-674d04b828da/jobs/184957
So we're all good to go.

Just in time for fresco 2.4.0 :-)

Just in time for fresco 2.4.0 :-)


GraphSCCFun has wrong sig

Steps to reproduce
Try to use GraphSCCFn
Expected behavior
The result structure doesn't have topOrder.
Actual behavior
It should.
Root cause
This is wrong.
Should use new docs.
Should update upstream sml-libs and then pull it in.

Fixed in b742479abd83ceab5cbcd52be943306aca77938f which uses the latest sml-libs which includes this commit


Simplify EXIF transpose

replace EXIF transpose method with ImageOps.exif_transpose, below is it's source code:
def exif_transpose(image, *, in_place=False):
"""
If an image has an EXIF Orientation tag, other than 1, transpose the image
accordingly, and remove the orientation data.
:param image: The image to transpose.
:param in_place: Boolean. Keyword-only argument.
    If ``True``, the original image is modified in-place, and ``None`` is returned.
    If ``False`` (default), a new :py:class:`~PIL.Image.Image` object is returned
    with the transposition applied. If there is no transposition, a copy of the
    image will be returned.
"""
image_exif = image.getexif()
orientation = image_exif.get(ExifTags.Base.Orientation)
method = {
    2: Image.Transpose.FLIP_LEFT_RIGHT,
    3: Image.Transpose.ROTATE_180,
    4: Image.Transpose.FLIP_TOP_BOTTOM,
    5: Image.Transpose.TRANSPOSE,
    6: Image.Transpose.ROTATE_270,
    7: Image.Transpose.TRANSVERSE,
    8: Image.Transpose.ROTATE_90,
}.get(orientation)
if method is not None:
    transposed_image = image.transpose(method)
    if in_place:
        image.im = transposed_image.im
        image.pyaccess = None
        image._size = transposed_image._size
    exif_image = image if in_place else transposed_image

    exif = exif_image.getexif()
    if ExifTags.Base.Orientation in exif:
        del exif[ExifTags.Base.Orientation]
        if "exif" in exif_image.info:
            exif_image.info["exif"] = exif.tobytes()
        elif "Raw profile type exif" in exif_image.info:
            exif_image.info["Raw profile type exif"] = exif.tobytes().hex()
        elif "XML:com.adobe.xmp" in exif_image.info:
            for pattern in (
                r'tiff:Orientation="([0-9])"',
                r"<tiff:Orientation>([0-9])</tiff:Orientation>",
            ):
                exif_image.info["XML:com.adobe.xmp"] = re.sub(
                    pattern, "", exif_image.info["XML:com.adobe.xmp"]
                )
    if not in_place:
        return transposed_image
elif not in_place:
    return image.copy()

We'll do a little testing and merge if it's okay.

Original Training is BUGFree， but I didn't test the input with rotate， someone test out bug of that kind input should connect to this PR


Same Age and Data Problem

Describe the bug
A clear and concise description of what the bug is.
the age and traving data is same in this cluster.

This is because the patients in question are roughly similar in description.


(🎁) add an upgrade command

prior art:

poetry: https://github.com/MousaZeidBaker/poetry-plugin-up
npm: https://docs.npmjs.com/cli/v10/commands/npm-update

> uv upgrade
upgraded 10 dependencies in 0.001 seconds 🚀

this will update the version pins in the pyproject.toml file
why? when i want to update all the dependencies in a project, it can be tedious to manually search for and update each and every dependency. additionally, for application projects (not libraries), i like to be able to squizz the pyproject file for an easy overview of which dependencies are actually installed, not just some possible range

I don't think uv sync can bump the pinned versions of every dependency to the latest available version, which I believe is what the issue creator is asking about. uv sync just downloads modules that match the pin, it doesn't change the pin.

Ahh, yes uv sync updates the environments but not the install specs.
Would it be recommended to not pin exact dependencies in the pyproject.toml - instead to use the lock file for tracking that?
pyproject.toml should track compatibility and uv.lock should track exact versions for deployments/reproducibility.

I suspect a lot of teams have a similar workflow to my team, which most project tooling has poor support for.
In the pre-Dependabot days, once a month or so we'd open up the project file on one screen and pypi on the other, then manually bump all the versions to the latest we could find. Then we'd install everything, read release notes, check what broke, and perhaps go back a version or two for some of the dependencies (which is why unpinning all the deps wouldn't work). After that we'd generate a new lock file.
Having an upgrade command would remove the manual labor of having to look up all the versions and editing the file for each individual dependency.

uv upgrade-interactive would also be great, like yarn (v1):

Use uv sync -U to update packages

Although I will note that it would be nice to be able to see which packages are outdated, similar to pip list --outdated or the poetry command which does the same thing.

One other thing to note, you can use uv run pip list --outdated right now as a workaround

Although I will note that it would be nice to be able to see which packages are outdated,

That's tracked in #2150.

Like @KotlinIsland said, especially for applications it's really nice to be able to browse narrow ranges of your top level dependencies within pyproject.toml and have a tool automatically bump them for you so you don't have to go to PyPI and look at each top level dependency to see what the latest version is.
For more prior art, see npm-check-updates

https://www.npmjs.com/package/npm-check-updates

It has a nice output UI:

I agree this would be a really helpful feature, especially useful in PRs for tracking version increments. Its a lot easier to see these in pyproject.toml than a lock file

I'm also a big fan of yarn's yarn upgrade-interactive for interactively updating dependency versions in package.json and would love to see similar uv functionality for updating dependency versions in pyproject.toml.

I use this tiny script. Works like a charm.

Is this intended to handle the following use case?
dependencies = [
    "httpx==0.27",
]

uv upgrade
dependencies = [
    "httpx==0.28",
]

(where 0.28.1 is the latest version of httpx as listed in uv tree --outdated)
Would it require something like uv upgrade --outdated [<package>] or uv upgrade --override [<package>] or similar?
Currently, even an ugly hack like
uv tree --outdated -d1 | grep "latest" | sed -E 's/.*── ([^ ]*) .*latest: v(.*)\)/\1==\2/' | xargs -I {} uv lock --upgrade-package {}
isn't working, because uv lock --upgrade-packages still attempts to satisfy the existing constraints. And the uv remove/uv add workaround doesn't address handling different dependency groups.

@wgordon17

[The] uv remove/uv add workaround doesn't address handling different dependency groups.

You might be interested in @KotlinIsland’s improved version, as this should handle dependency groups.


How to build only libcrypto.a (and not libssl.a)?

There used to be a make build_crypto AFAIK.
But on OpenSSL 1.1, build_crypto seems to be gone.
What is the solution?

Just "make libcrypto.so" (or libcrypto.a) directly -- the new build system produces a single global makefile that knows how to build everything.

@kaduk
make libcrypto.a did not work:
In file included from crypto/aes/aes_cbc.c:10:
include/openssl/aes.h:13:11: fatal error: 'openssl/opensslconf.h' file not found

Just make however works just fine. The problem is I waste time building libssl.a
Any ideas?

Sorry, no.  Perhaps post to the openssl-users mailing list; see https://mta.openssl.org for details.

openssl/openssl.conf.h not found

Hmm, that feels like a bug (missing dependencies on generated headers).  Once you manually make the three-ish generated headers, the build should work okay.
So,
make include/openssl/opensslconf.h
make crypto/include/internal/bn_conf.h
make crypto/include/internal/dso_conf.h
make libcrypto.a

@levitte any idea why this stopped/isn't working?
# Empty DEPEND "indices" means the dependencies are expected to be built
# unconditionally before anything else.
DEPEND[]=include/openssl/opensslconf.h crypto/include/internal/bn_conf.h \
         crypto/include/internal/dso_conf.h

They are collected in the Makefile variable GENERATED_MANDATORY, which is used as dependencies for the target build_generated, so in essence, you need to do this when you're starting fresh:
make build_generated libcrypto.a

As soon as enough has been made, that work-around isn't needed any more, because dependency generation will have registered exactly which object files depend on those header files, and they will be regenerated as needed.
The reason we have this situation is a catch 22.  We could make all object files in crypto/ and ssl/ depend on those header files explicitely...  the trouble is that if any of them changes, everything gets rebuilt, not just the fewer particular files that do depend on them (I mean, imagine changing the BN conf...).  The other method is to generate dependencies first, but since we do that automatically as part of the build, it's a moot point.  So we have the workaround with build_generated...

Ah, silly me only tried make generated and didn't look for make build_generated

Perhaps we should pick out a list of "public" top-level targets like libfoo.a/so and put explicit dependencies on build_generated in the makefile template?

For openssl-1.1.0f,
make include/openssl/opensslconf.h
make crypto/include/internal/bn_conf.h
make crypto/include/internal/dso_conf.h
make libcrypto.a

works fine. Are you saying that for openssl-1.1.0f you would recommend
make build_generated libcrypto.a
make libcrypto.a

instead? Thanks.

Yes, make build_generated && make libcrypto.a is the pretty way to spell the above, that I had forgotten existed.

@snap9: Well, since build_generated depends on include/openssl/opensslconf.h, crypto/include/internal/bn_conf.h and crypto/include/internal/dso_conf.h, either works.  build_generated is simply a comfortable shortcut that will always work, even if the list of globally generated header files changes.
Side note: it's not necessary to do an extra make libcrypto.a ;-)

@kaduk, we already have a list of "public" targets in INSTALL.  The build_ targets are more of the internal sort, targetted at developers that "know their thing", and with the option of changing over time.

This works, thanks! You are geniuses.

@levitte @kaduk What is the correct way to run the tests for only libcrypto.a? Is it make test build_generated libcrypto.a or just make test libcrypto.a?

There is no such separation. The test target depends on all programs being built, and they in turn depend on both libraries, collectively. We simply haven't thought of dividing it per library...

Is it same for Windows(not mingw/cygwin/msys)?

Yes


[Release-1.25] Enable dependabot

Signed-off-by: Derek Nola<EMAIL_ADDRESS>

Proposed Changes
Adds dependabot on a weekly schedule for release-1.25 branch

Types of Changes
CI

Verification

Testing

Linked Issues

User-Facing Change

Further Comments

Would we want to do the updateCLI stuff as well? Or would that be a separate pr?

Would we want to do the updateCLI stuff as well? Or would that be a separate pr?
That's the plan, I'm just going to make it a separate PR.


Add initial scheduler support

Add the ability to choose schedulers, at least for a subset of the pipeline
configurations. Allows configuring for text to audio and audio to audio
in the sidebar. Currently not used for interpolation, aka the riffusion
pipeline.
Topic: schedulers_v0

Reviews in this chain:
└https://github.com/riffusion/riffusion/pull/88 Add initial scheduler support

#
head
base
diff
date
summary

0
8856252b
75c67e1e
diff
Jan 14 22:53 PM
3 files changed, 86 insertions(+), 15 deletions(-)


Safe wrapper for eth promiscuous mode

Hello, this is my attempt at #246. This works locally, I could add wrappers for wifi as well for the sake of symmetry, though I have no use for them.
lmk

This should be fine I think. Want me to make a wrapper for wifi as well?

This should be fine I think. Want me to make a wrapper for wifi as well?

If the Wifi driver actually does have such a mode and you could expose it that would be great.


Allow for export samples together with onnx

This change enables exporting sample inputs and outputs for transformers (together with onnx). It could be run separately or as part of training/evaluation.

@natuan we want to remove export to onnx from the training flows and training scripts and keep this only in the onnx export script. This enables us to keep a separation of responsibilities and concerns that make testing and code changes in the future much easier.
Additionally, is there any way we can add in the samples export to the export_onnx function and add it to the CLI arguments? Everything we push up now should have the ability to be run from the CLI along with the Python APIs

I'll close this PR and try to move sample generation to export_onnx


No such file or directory: 'checkpoints\fish-speech-1.4'

Self Checks

[X] This template is only for bug reports. For questions, please visit Discussions.
[X] I have thoroughly reviewed the project documentation (installation, training, inference) but couldn't find information to solve my problem. English 中文 日本語 Portuguese (Brazil)
[X] I have searched for existing issues, including closed ones. Search issues
[X] I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
[X] [FOR CHINESE USERS] 请务必使用英文提交 Issue，否则会被关闭。谢谢！:）
[X] Please do not modify this template and fill in all required fields.

Cloud or Self Hosted
Self Hosted (Source)
Environment Details
Windows 11
Steps to Reproduce

Install latest

git clone https://github.com/fishaudio/fish-speech
cd fish-speech
call install_env.bat

Try and run it (with start.bat)

Traceback (most recent call last):
  File "D:\FishSpeech\fish-speech\fishenv\env\lib\threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "D:\FishSpeech\fish-speech\fishenv\env\lib\threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "D:\FishSpeech\fish-speech\tools\llama\generate.py", line 912, in worker
    model, decode_one_token = load_model(
  File "D:\FishSpeech\fish-speech\tools\llama\generate.py", line 677, in load_model
    model: Union[NaiveTransformer, DualARTransformer] = BaseTransformer.from_pretrained(
  File "D:\FishSpeech\fish-speech\fish_speech\models\text2semantic\llama.py", line 387, in from_pretrained
    config = BaseModelArgs.from_pretrained(str(path))
  File "D:\FishSpeech\fish-speech\fish_speech\models\text2semantic\llama.py", line 82, in from_pretrained
    with open(path, "r", encoding="utf-8") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'checkpoints\\fish-speech-1.4'

Looks like the code still has multiple references to 1.4 that should be 1.5, ie

And the models are gated now?  Why?
Cannot access gated repo for url https://hf-mirror.com/fishaudio/fish-speech-1.5/resolve/main/gitattributes.
Access to model fishaudio/fish-speech-1.5 is restricted and you are not in the authorized list. Visit https://huggingface.co/fishaudio/fish-speech-1.5 to ask for access.

I agreed to the access.  Run it again.  Still fails.
It makes it MUCH easier for people if you do not restrict your models.
Do you have a series of steps I should follow once I agree to access to then use the model with the local UI?
Thanks
✔ Expected Behavior
Not to crash
❌ Actual Behavior
It crashes :)

plz use huggingface-cli login

plz use huggingface-cli login

OK, that fixes the "cannot access gated repo" issue, but I still get the other error.
FileNotFoundError: [Errno 2] No such file or directory: 'checkpoints\fish-speech-1.4'

Please check your file folder / fix the code by change 1.4 to 1.5.

Please check your file folder / fix the code by change 1.4 to 1.5.

But shouldn't the dev fix the code in the repo?  It is going to fail for anyone who tries it.

same here, it seems that your github is still referencing on 1.4, I've tried to replace some thext by 1.5 but there are so many enties in different files!
please help :)

ok managed to start the api server you need to in Visual Studio Code look for "1.4" in the all directory and replace all by "1.5" but I will no do a PR as I still have many warnings

So can the dev(s) please fix the 1.4 references now?

I see that some of the 1.4 refs were fixed.  I can now get the UI launching, but there are still refs to 1.4.
When I select the 1.5 models instead, http://<IP_ADDRESS>:7862 does not open.  "Firefox can’t establish a connection to the server at <IP_ADDRESS>:7862"


 Remember to configure the API endpoint:
`tk env set environments/default --server=https://<IP_ADDRESS>:6443`
Errors occurred while initializing the project. Check the above logs for details.

Can you help me setup the repository? Thanks in advance!

Is it possible to have a documentation with kubernetes version support? And drop notice? That was difficult to understand that the support for this version has been dropped.

Hey
We usually follow the supported kubernetes versions, 1.23 is out of support since March 2023 - you can use an older version of the library which still contains the 1.23 libs.

@xvzf, perhaps consider adding it to the readme, possibly including instructions on how to use an older version?
Otherwise, you can close this issue. Thank you for your response.

You can look at the commit history, browsing each commit should surface quickly when the last version of 1.23 was created, took me 1min: https://github.com/jsonnet-libs/k8s-libsonnet/tree/5c83a9a2b08fa055cfc62d9f59270161dd729596

@Duologic Upon exploring the repository, I encountered an older version of Kubernetes and faced an issue during the Tanka init process. As I have not previously contributed to the repository, I am unfamiliar with the code. I believe that adding documentation to explain which versions are deprecated or located in older commits would save time for all users and make the installation of the jsonnet k8s library easier. While navigating the commit history is straightforward, understanding how the repository functions, browsing through commits, and manually modifying the jsonnet bundler lock file can be challenging for new users.


Some chars are not rendering inside code blocks correctly - &ampquot; &amplt; &ampgt;

Some chars are not rendering inside code blocks correctly:
var mystring = "some text";
renders as;
var mystring = &ampquot;some text&ampquot;;
var lst = new List();
renders as;
var lst = new List&amplt;string&ampgt;();

corrected title

Ignore me!
This was caused by Jade escaping.
Had to change #{markdown} to !{markdown} to unescape it.


[Keycloak Error] User cannot be authenticated.

Hi,
With the example "laravel-keycloak-web-guard-example-master", after enter login and password, I have this error :
Vizir\KeycloakWebGuard\Exceptions\KeycloakCallbackException
[Keycloak Error] User cannot be authenticated.
Laravel log content :
[2023-06-19 09:17:34] local.ERROR: [Keycloak Service] ID Token already expired.
[2023-06-19 09:17:34] local.ERROR: [Keycloak Error] User cannot be authenticated. {"exception":"[object] (Vizir\KeycloakWebGuard\Exceptions\KeycloakCallbackException(code: 0): [Keycloak Error] User cannot be authenticated. at /Users/christophe/Documents/Sources/generation-ia/laravel-keycloak-web-guard-example-master/vendor/vizir/laravel-keycloak-web-guard/src/Auth/Guard/KeycloakWebGuard.php:147)
I tried to logout and logon, but the error was the same
Thanks in advance

@generationia  Sorry, but you are in wrong repo.

I'am sorry


Repo sync for protected CLA branch

The pull request is created from master637606694183248408 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


MethodBase.IsFamilyAndAssembly docs outdated

Describe the bug
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/private-protected
now exists, rendering the docs at https://docs.microsoft.com/en-us/dotnet/api/system.reflection.methodbase.isfamilyandassembly?view=net-5.0 incorrect.
Expected behavior
The note that "The Visual Basic and C# languages cannot define methods with MethodAttributes.FamANDAssem visibility; that access level appears only in the C++ example." is outdated.  Instead, the docs should refer to private protected.
The same issue is present at https://docs.microsoft.com/en-us/dotnet/api/system.reflection.fieldinfo.isfamilyandassembly?view=net-5.0#System_Reflection_FieldInfo_IsFamilyAndAssembly too

This issue was moved to dotnet/dotnet-api-docs#4400


Dapr bot - read members from GitHub Team

Today Dapr bot maintains a hard-coded list of members who can trigger privileged actions. This proposal is to read the list from groups dapr/approvers-dapr, dapr/maintainers-dapr, and dapr/maintainers.

/keep-alive

This should be re-opened.

/keep-alive


Getting the data from the wrong location. AddRecipe.

https://github.com/natac13/vegan-recipe-app-redux/blob/add/custom-middleware-for-firebase/app/components/addRecipe/AddRecipe.js#L64-L67
This data is coming from the form where it should be coming from the state.recipe immutable Map like in EditRecipe.js https://github.com/natac13/vegan-recipe-app-redux/blob/add/custom-middleware-for-firebase/app/components/editRecipe/EditRecipe.js#L48-L59
However In EditRecipe I am stringifying the recipe to add to Firebase. This will be changing to send the immutable map recipe through the dispatch(action([immutableMap])) instead of the string version. I will then intercept the action with the middleware and convert the action.recipe from addRecipe() and action.oldRecipe/action.updatedRecipe from updateRecipe() actions creators.

Closing as this is the same as #7


Jupyter Notebook experience in VSCode.

I can't replicate the experience with jupyter-notebook on VSCode in different machines. I have 2 Windows machines a windows server and my pc (both uses Remote Linux for development; I use a Remote Linux machine in Server for development and WSL2 on pc).
The versions of VSCode, Python & Jupyter extensions are the same in both the machines. The jupyter-notebook in the server provides a better experience (as shown in extension details, mainly it supports vim), which I cannot get on my pc.
About VSCode.
Version: 1.58.0 (user setup)
Commit: 2d23c42a936db1c7b3b06f918cde29561cc47cd6
Date: 2021-07-08T06:54:55.083Z
Electron: 12.0.13
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

Jupyter Extension: v2021.8.1013163132
Python Extension: v2021.6.944021595
Snap of a .ipynb file in Server (which supports VIM & that I want to replicate)

Snap of the same in my pc (which don't support vim).

I could not figure out what I am missing.

Same here. Only on the insider build of vscode I see the new interface.

The new interface is currently available to all VS Code Insiders users and a fraction of VS Code Stable users. To opt into the new interface via your settings.json file, please see instructions here: https://github.com/microsoft/vscode-jupyter/wiki/Native-Notebook-Support-in-VS-Code#enabling-the-new-experience

Thanks! Happy Developing!


Python package fails when building Docker image based on Alpine Linux

I'm running into an issue including Tink in Docker images based on Python / Apline Linux. I get the following error when building the image:
Step 1/2 : FROM python:3.7-alpine3.11
 ---> b478b156c249
Step 2/2 : RUN pip install tink
 ---> Running in d7c3c4f809fa
Collecting tink
  Downloading tink-1.4.0.tar.gz (120 kB)
    ERROR: Command errored out with exit status 255:
     command: /usr/local/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-58y_vri6/tink/setup.py'"'"'; __file__='"'"'/tmp/pip-install-58y_vri6/tink/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-o48j7siw
         cwd: /tmp/pip-install-58y_vri6/tink/
    Complete output (1 lines):
    Could not find bazel executable. Please install bazel tocompile the Tink Python package.

To reproduce, try to build the following Dockerfile:
# Fails on Alpine 3.11
FROM python:3.7-alpine3.11

RUN pip install tink

The default Python image works fine.
# Succeeds on base python image
FROM python:3.7

RUN pip install tink

Unfortunately there are no binary wheels available for Alpine Linux and you will have to build the package from source.
The reason for that is that Alpine Linux is incompatible with the (current) manylinux standard as it uses musl libc, therefore it's not possible to distribute binary wheels for Alpine at the current stage.


Update needed on sqlfluff version

Because dbt-superset-lineage (0.2.1) depends on sqlfluff (>=0.8.2,<0.9.0)
and no versions of dbt-superset-lineage match >0.2.1,<0.3.0, dbt-superset-lineage (>=0.2.1,<0.3.0) requires sqlfluff (>=0.8.2,<0.9.0).
So, because seed depends on both dbt-superset-lineage (^0.2.1) and sqlfluff (^1.4.1), version solving failed.

@one-data-cookie @mrshu @sweco could you review and merge it?

@MuslimBeibytuly Just out of curiosity, was the sqlfluff already present in your environment when you tried to install dbt-superset-lineage?

@mrshu yup,sqlfluff with higher version was already there


Support multiple architectures in CUPY_NVCC_GENERATE_CODE

Close #3317.

Jenkins, test this please

Jenkins CI test (for commit 2942ef4fb62f9b4480444ce1fdeeaae66c2c8fc3, target branch master) succeeded!


Jarryd/265/refactor accounts page

Description
Refactors accounts page to show tabbed lists instead of buttons. Adds pagination to rewards and unbondings.
closes: #265

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[X] Targeted PR against correct branch
[X] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[X] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[X] Re-reviewed Files changed in the Github PR explorer


The OWL build available from the CHEBI FTP site is non-standard

The chebi.owl available here ftp://ftp.ebi.ac.uk/pub/databases/chebi/ontology/
is potentially confusing to users. It uses an annotation vocabulary that is non-standard and unique to CHEBI.
I recommend you use the standard OWLAPI obo<->owl translation. You can use the latest version of Protege, or a command line tool such as

Robot - https://github.com/ontodev/robot/
owltools/Oort - https://github.com/owlcollab/owltools

If you do this we can also simplify the OBO central build pipeline (at the moment we are forced to make our own CHEBI build using this jenkins job - http://build.berkeleybop.org/job/build-chebi/ )
Reported by: cmungall

@cmungall, I have tried the owltools as a dependency, but the owl file exported doesn't have any major changes, can you please elaborate bit more on what is "Standard annotation vocabulary", and the things that are confusing the users.  Also if you can provide an example to look, would be helpful.

@adekker2  check this

These are your annotation properties:
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#InChIKey"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#InChI"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#xref"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#altId"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#Synonym"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#SMILES"/>
    <owl:AnnotationProperty rdf:about="http://purl.obolibrary.org/obo#Definition"/>`

This is the official mapping
http://owlcollab.github.io/oboformat/doc/obo-syntax.html#5.6
There are some other things you should switch as well, for example, you should either use the standard part-of object property (BFO_0000050), removing your own axioms about it; or if the CHEBI one is truly different best to choose a different label so as not to confuse.
Is the code that makes the conversion in github? I could point to specific problems
@dosumis is often at the EBI and can provide local help if need be

Unless you have expressiveness outside of OBO, it should be to sufficient to make the OWL version using the OBO dump of ChEBI and convert using Oort or Robot.

Thanks @dosumis.
@cmungall, we generated a OWL file using Robot (converted from OBO to OWL). Its available in the public ftp, can you please take a look and let us know if there are anything missing.

@cmungall do you want us to generate the future OWL file using Robot library. Can you please confirm.

Correct. Tool is here https://github.com/ontodev/robot/
robot convert -i chebi.obo -o chebi.owl

Thanks @cmungall, thats how i generated this one, shall do it for the future releases too.


Add waitlist sign up link to homepage

Add the URL (https://forms.gle/w3vyujNdsAuLsUHR9) as a button to the following pages:

Home page (index.html)
Students page

We have a outcomes page (https://careersincode.org/outcomes/) that has 3 buttons on it we can leverage to add a second button.

@AndrewWladis  Looks good!


Target specific examples

Problem
Building system depending libraries with examples that vary based on the OS (e.g. embedded-hal integrations or USB functionality) causes example tests to constantly fail if you write examples to fail when running cargo test.
Proposed Solution
Allow for examples to be scoped to specific targets and possibly allow for cross compilation.
Notes
No response

Thanks for the report! This seems effectively a duplicate of #9208.


[Bug] zendesk__ticket_metrics.first_reply_time_business_minutes ignore schedule

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
Comparing the data collect via Fivetran vs the Zendesk report UI; it appears our business schedule is not taken into consideration when calculating the value.
Seems related to: https://github.com/fivetran/dbt_zendesk/issues/53
Relevant error log or model output
N/A

Expected behavior
Values in zendesk__ticket_metrics.first_reply_time_business_minutes should consider our schedules and reduce the minutes to first reply.
dbt Project configurations
# Name your project! Project names should contain only lowercase characters
# and underscores. A good package name should reflect your organization's
# name or the intended use of these models
name: "binti_data_platform"
version: "1.3.2"
config-version: 2

# This setting configures which "profile" dbt uses for this project.
# We default to using local, in the CI and Fivetran, this will be overidden by passing the --profile flag,
# I.E. dbt run --profile
profile: "local" # pass in the --profile flag to overide this when running dbt run

# These configurations specify where dbt should look for different types of files.
# The `source-paths` config, for example, states that models in this project can be
# found in the "models/" directory. You probably won't need to change these!
model-paths: ["models"]
analysis-paths: ["analysis"]
test-paths: ["tests"]
seed-paths: ["data"]
macro-paths: ["macros"]
snapshot-paths: ["snapshots"]

target-path: "target" # directory which will store compiled SQL files
clean-targets: # directories to be removed by `dbt clean`
  - "target"
  - "dbt_packages"

vars:
  # Configurations for the dbt fivetran github package
  github_source:
    # This is the project Fivetran dumps data into
    github_database: my-gcp-project
  # GCP project where Fivetran Jira connector loads data
  jira_database: my-gcp-project

  zendesk:
    using_organization_tags: False

  zendesk_source:
    # This is the project Fivetran dumps data into
    zendesk_database: my-gcp-project
    using_organization_tags: False

# Configuring models
# Full documentation: https://docs.getdbt.com/docs/configuring-models
models:
  +database: |
    {%- if target.name == "fivetran" -%}
      my-gcp-project
    {%- else -%}
      {{ target.database }}
    {%- endif -%}
  github:
    github__pull_requests:
      +schema: marts
  zendesk:
    zendesk__ticket_metrics:
      +schema: marts
  binti_data_platform:
    marts:
      +schema: marts
      jira:
        +materialized: table
      github:
        +materialized: table

Package versions
packages:
  # Source Github Models
  # https://github.com/fivetran/dbt_github_source
  - package: fivetran/github_source
    version: [">=0.6.0", "<0.7.0"]
  # Transformational Github Models
  # https://github.com/fivetran/dbt_github
  - package: fivetran/github
    version: [">=0.6.0", "<0.7.0"]
    # Source Jira Models from Fivetran
  - package: fivetran/jira_source
    version: [">=0.6.0", "<0.7.0"]

  # Transformational Zendesk Models
  # https://hub.getdbt.com/fivetran/zendesk/latest/
  - package: fivetran/zendesk
    version: [">=0.10.0", "<0.11.0"]
  - package: fivetran/zendesk_source
    version: [">=0.8.0", "<0.9.0"]

What database are you using dbt with?
bigquery
dbt Version
1.3.1
# Choose the version of dbt that will run the project.
# Supported versions are listed in https://fivetran.com/docs/transformations/dbt/setup-guide#prerequisites.
# If the version is not specified, dbt 0.18.0 will be used.
dbtVersion: "1.3.1"

jobs:
  # Give each job a unique name. The name will be displayed in the Fivetran
  # dashboard once your jobs are imported.  Job names must be unique.
  - name: Weekly Github Issue Metrics
    # The name of the target that will be used when running the job. If it's not
    # specified, the target will be named 'prod'.
    targetName: fivetran
    # Define when this job should run, using cron format. This example will run
    # every day at 12:00pm (according to your warehouse timezone). For help with
    # cron formatting, visit https://crontab.guru/.
    schedule: 0 0 * * 0
    steps:
      - name: Install Deps # Give each step in your job a name. This will enable you to track the steps in the logs.
        command: dbt deps # Enter the dbt command that should run in this step. This example will run all your models. For a list of available commands visit https://docs.getdbt.com/reference/model-selection-syntax/.
      - name: Update Github Models
        command: dbt run --select +github__pull_requests +github__pull_request_reviews
      - name: Run Tests
        command: dbt test --select +github__pull_requests +github__pull_request_reviews
  - name: Daily Jira Metrics
    targetName: fivetran
    schedule: 0 0 * * *
    steps:
      - name: Install Deps
        command: dbt deps
      - name: Update Jira Models
        command: dbt run --select +jira__field_change_history +jira__issues +jira__sprints
      - name: Run Tests
        command: dbt test --select +jira__field_change_history +jira__issues +jira__sprints
  - name: Daily Zendesk Metrics
    targetName: fivetran
    schedule: 0 0 * * *
    steps:
      - name: Install Deps
        command: dbt deps
      - name: Update Zendesk Models
        command: dbt run --select +zendesk__ticket_metrics
      - name: Run Tests
        command: dbt test --select +zendesk__ticket_metrics

Additional Context
I spot-checked the data collected vs the data in the Zendesk reports.
Are you willing to open a PR to help address this issue?

[ ] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[X] No.

HI @KaoruDev thanks for opening this issue and taking a look at the previous issues that may relate to the same result you are encountering currently.
I believe you are correct that the Issue you linked does seem to similar to what you are encountering. If that is the case, I wanted to direct you to our DECISIONLOG where we detail how we calculate the business metrics for reply time for Zendesk tickets. In particular I wanted to highlight how this will in fact differ from Zendesk's built reports.
The main reason is that Zendesk will only take into consideration the latest schedule that the ticket was assigned. However, this does not tell an accurate picture of time worked on the ticket, especially if the ticket changed hands and schedules. Therefore, our package uses every schedule (and the duration in that schedule) the ticket was associated with when calculating business metrics. We feel strongly that this is a more accurate way of reporting and have decided to intentionally not adjust the package to fit the Zendesk reporting as we do not believe it tells the full story of the ticket. The DECISIONLOG linked will include an example and how you can see that it will differ from the Zendesk reporting.
While this is not something we will adjust within the official version of the package, if you do wish you configure the logic to be custom on your end then I would encourage you to take a look at this post on how to override the logic of one of our models while still leveraging the package as a whole. This should allow you to make the adjustments I included in Issue #53 for the relevant model and get your desired outcome.
Please let me know if this does not answer your initial question. Thanks!

Good to know, thank you for the quick response! I'll close for this for now.


enforcer: fix directory rule behaviour

AppArmor ** or * rule doesn't include the directory itself. Taking an example of ps and proc. ps tries to access the proc directory itself before reading the underlying directories. When we save allow ps -> /proc/, it didn't include the /proc/ directory itself so we update the directory rule to include both directory itself and the recursive pattern identifier.
Now we would use /proc/{,}. {} signifies alternation and we provide it with to either match recursively (**) or the dir itself (empty string)
BPFLSM kernel space refers to /proc/ as /proc since everything is a file. So we added a additional file rule from userspace when we need to deal with Directory Rules.

Codecov Report

Merging #926 (4eb1f01) into main (3ee3ccd) will decrease coverage by 1.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #926      +/-   ##
==========================================
- Coverage   39.19%   37.48%   -1.71%
==========================================
  Files          31       31
  Lines        9829     9829
==========================================
- Hits         3852     3684     -168
- Misses       5471     5661     +190
+ Partials      506      484      -22

Impacted Files
Coverage Δ

KubeArmor/enforcer/appArmorTemplate.go
100.00% <ø> (ø)

KubeArmor/core/dockerHandler.go
0.00% <0.00%> (-47.48%)
:arrow_down:

KubeArmor/common/common.go
39.38% <0.00%> (-3.87%)
:arrow_down:

KubeArmor/core/kubeArmor.go
47.56% <0.00%> (-2.44%)
:arrow_down:

KubeArmor/enforcer/runtimeEnforcer.go
34.68% <0.00%> (-1.16%)
:arrow_down:

KubeArmor/core/kubeUpdate.go
38.19% <0.00%> (-0.23%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Great finding @daemon1024!
So now if we want to recursively include proc dir, then we would be using proc/{**} in the policy rule?


Trying to parse new sentences gives runtime error

When I run the parser with the default sentences in the best_models/raw_sentences.txt, it parses correctly. But when I make any change to those sentences or add any new sentence, it gives runtime error:
return self.layer_norm(outputs + residual), attns_padded
RuntimeError: The size of tensor a (51) must match the size of tensor b (38) at non-singleton dimension 0
using pytorch version: 1.8.1+cu101
python version: Python 3.7.10
Any idea about the issue? Can't we use the pretrained model (swbd_fisher_bert_Edev.0.9078) to parse our own sentence?

When I run the parser with the default sentences in the best_models/raw_sentences.txt, it parses correctly. But when I make any change to those sentences or add any new sentence, it gives runtime error:
return self.layer_norm(outputs + residual), attns_padded RuntimeError: The size of tensor a (51) must match the size of tensor b (38) at non-singleton dimension 0
using pytorch version: 1.8.1+cu101
python version: Python 3.7.10
Any idea about the issue? Can't we use the pretrained model (swbd_fisher_bert_Edev.0.9078) to parse our own sentence?

For using the pre-trained model to label your own sentences, please check this repo!

For using the pre-trained model to label your own sentences, please check this repo!

Thanks for the pointer! Could you please add license information for this repo?


doesn't create .tmp folder

Issue:
If .tmp folder doesn't exist
stream.js:74
throw er; // Unhandled stream error in pipe.
^
Error: ENOENT: no such file or directory, open 'C:\pg\PogoBot-Telegram.tmp\MTg0NDY3NDQwNzM0NDAyOTIwNjY=.png'
at Error (native)
OS: Windows 10 Pro 64bit
Commit: 6148bd596dd24bdc54fc81f16b9c11e287719ada

Fixed in a644047ee: images are no longer saved to disk, they are sent as buffers so .tmp won't grow huge and is no longer needed.


[CT-1762] [Bug] generate_schema_name_for_env() breaks generic tests in prod job.

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When using target set to prod and using generate_schema_name_for_env(), generic tests fail for the models that have schema set because tests are referencing the target schema, not the custom schema.
If you add the custom schema to the generic test, the test picks up the custom schema but the compiled sql is broken.
Expected Behavior

Tests should use the same schema as the models under test
Setting schema on test should not break the compiled sql

Steps To Reproduce

Set your job target to be prod and command to be dbt test
Add the following macro as mentioned in https://docs.getdbt.com/docs/build/custom-schemas#an-alternative-pattern-for-generating-schema-names

{% macro generate_schema_name(custom_schema_name, node) -%}
    {{ generate_schema_name_for_env(custom_schema_name, node) }}
{%- endmacro %}

Define schema in the dbt_project.yml

models:
  my_dbt:
    # Config indicated by + and applies to all files under models/example/
    staging:
      database: MYDB
      +materialized: view
      model_folder_1:
        schema: custom

Create tests in models/staging/model_folder_1/schema.yml

version: 2

models:

  - name: stg_groups
    description: "Groups"
    columns:
      - name: group_id
        description: "The primary key for this table"
        tests:
          - unique
          - not_null

Run the job which has dbt test command
Inspect the failed test artifact sql and notice that the schema is not the custom schema

select
      count(*) as failures,
      count(*) != 0 as should_warn,
      count(*) != 0 as should_error
    from (

select
    group_id as unique_field,
    count(*) as n_records

from MYDB.target.stg_groups
where group_id is not null
group by group_id
having count(*) > 1

    ) dbt_internal_test

Try setting schema in models/staging/model_folder_1/schema.yml

version: 2

models:

  - name: stg_groups
    description: "Groups"
    schema: custom
    columns:
      - name: group_id
        description: "The primary key for this table"
        tests:
          - unique
          - not_null

Run the job which has dbt test command again
Inspect the failed test artifact sql and notice that the schema is the custom schema but the sql template was broken

select
    group_id as unique_field,
    count(*) as n_records

from MYDB.custom.stg_groups
where group_id is not null
group by group_id
having count(*) > 1

Relevant log output
No response
Environment
- OS: dbtCloud
- Python:
- dbt: 1.3.2

Which database adapter are you using with dbt?
No response
Additional Context
No response

@cjcdoomed Were you able to get to the bottom of what was happening here? I was surprised when I initially read the description & reproduction case, so I want to make sure this isn't a thorny edge case yet to be uncovered/understood.


feat: add LawfulAlternative class

Mostly for completeness, the main use case is Option. Another would be a NonDet monad.
I only included "linear" alternative laws (everything is evaluated the same number of times on both sides) since these are the only ones we can expect for a control-type monad. For data-type monads, there are additional distributive laws like ((a <|> b) >>= f) = ((a >>= f) <|> (b >>= f)) that might be expected to hold. These probably belong in a MonadPlus extension, or similar.
Migrated from closed PR lean4#1999

This is an old PR. It's evident that everyone has managed to live without it for a while. I think it needs a quick review to see whether it should be closed, revived, or whether something else might be better.

I'm in favour.
I would like to get to the point that we have a nice NonDet monad!

Closing due to age. Some parts are rescued in new PR.


Unspecified Platform value 'windows-arm-v7' added to the targetPlatform valueset.

Hi
With a build.gradle script defining the platform dimension as
platforms {
	win32 {operatingSystem "windows"	architecture 'x86' }
	win64 {operatingSystem "windows"	architecture 'x64' }
	linux32 {operatingSystem "linux"	architecture 'x86' }
	linux64 {operatingSystem "linux"	architecture 'x64' }
	macos {operatingSystem "macosx"		architecture 'x64' }
}

the targetPlatform valueset contains the unwanted/unspecified windows-arm-v7 which implies a exception when the platform name is controled by the script.
Execution of this build.gradle script
build.gradle.txt
C:\Users\fandre\Documents\git\gradle\subprojects\docs\src\samples\native-binaries\prebuilt>C:\ProgramFiles\Gradle\gradle-4.5-rc-1\bin\gradle clean build
Platform=platform 'linux32'
Platform=platform 'linux32'
Platform=platform 'linux64'
Platform=platform 'linux64'
Platform=platform 'macos'
Platform=platform 'macos'
Platform=platform 'win32'
Platform=platform 'win32'
Platform=platform 'win64'
Platform=platform 'win64'
Platform=platform 'windows_x86'
Platform=platform 'windows_x86'
Platform=platform 'windows_x86-64'
Platform=platform 'windows_x86-64'
Platform=platform 'windows_arm-v7'

FAILURE: Build failed with an exception.

* Where:
Build file 'C:\Users\fandre\Documents\git\gradle\subprojects\docs\src\samples\native-binaries\prebuilt\build.gradle' line: 11

* What went wrong:
Could not determine the dependencies of task ':linkMainReleaseExecutable'.
> Unknown architecture: arm-v7

Hi @zosrothko
This looks like a limitation with the way prebuilt libraries are created.  We don't allow prebuilt libraries to declare anything about themselves WRT flavors/build types/platforms, so we configure them for all possibilities.  In this case, we also configure them for all of the "default" platforms that we would normally include in Gradle.
We need to make prebuilt libraries, system libraries and libraries built by other tools more of a first class thing. We're going to be working on that soon (e.g., #424).
My recommendation (as a workaround) would be to more lenient on which platforms you're expecting.  If you don't set a particular platform's shared library/static library/etc fields, you'll get an error if Gradle tries to use them later.
Thanks!

@big-guy I had already applied your recommendation as a workaround. Thanks for having taken time for answering to this issue.


Added Tools section

Description
Added a Tools section for GitHub, VS Code, Sublime Text, NPM and Linux Terminal.
Fixed # (issue) (e.g. Fixed #10)
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

[x] I have pulled the latest changes in the repository
[x] I have squashed my commits
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code

@shriya030 Can you please elaborate about the first point? I edited every logo to look consistent. Had a talk with @abhijeet007rocks8 regarding that.
And I am searching for extra resources, I'll add them whenever I find those.

@shriya030 Can you please elaborate about the first point? I edited every logo to look consistent. Had a talk with @abhijeet007rocks8 regarding that.
And I am searching for extra resources, I'll add them whenever I find those.

If those images you are using are available online use their links if they are custom then please do mention them.

They are custom made, where should I mention this?

@shriya030 The images are custom made so I guess using direct images is the only solution.

@shriya030 The images are custom made so I guess using direct images is the only solution.

@abhijeet007rocks8 @joshi-kaushal On every page we have used links of the images so it's better to use links here too so that there should be uniformity

@shriya030 The images are custom made so I guess using direct images is the only solution.

@abhijeet007rocks8 @joshi-kaushal On every page we have used links of the images so it's better to use links here too so that there should be uniformity

@shriya030 I understand your point of uniformity.
But How can links  be used when images are not hosted online ??

@shriya030 The images are custom made so I guess using direct images is the only solution.

@abhijeet007rocks8 @joshi-kaushal On every page we have used links of the images so it's better to use links here too so that there should be uniformity

@shriya030 I understand your point of uniformity.
But How can links be used when images are not hosted online ??

You can use the images from google. These are just logos.

hello @shriya030, can you please merge this PR regardless of the LGMSoC event? I have done everything mentioned.

@shriya030 please check, it's been almost a month.