Navigational properties

What about navigational properties? What is the behavior with those? Seems like EFCore completely ignores the convention when the property is configured a navigational property.

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

ID: 2e162696-b19f-d73f-8b1f-d1b4c3048b62
Version Independent ID: a83e4fb5-e4ae-a46f-7248-487ea2679ac5
Content: Backing Fields - EF Core
Content Source: entity-framework/core/modeling/backing-field.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

@orobert91 Navigation properties use the same convention. If this isn't working, then please file a new issue on https://github.com/aspnet/EntityFrameworkCore/issues including a runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

Clarify that navigation properties are by convention always set through the property even if the field has been discovered. This is because setting navigation properties is never considered to be part of entity materialization (because you have to have two materialized entities before a navigation can be set) but this is something that is more due to implementation than an intended conceptual choice.

I think you should consider writing to backing fields the default behavior, whether it's for materialization or setting a navigation. When developer declares a backing field, there's a high chance it's because the getter and setter contains other logic. Also, it keeps everything consistent and will not pollute the DbContext model creation with SetPropertyAccessMode(). Anyway, thanks for clarifying the docs.

@orobert91 We were discussing the same thing recently. I filed #12430

I am similarly confused as to how we can use navigation properties with backing fields.
My navigation getter is private but when it comes to querying the entity with LINQ I stumbled on a few set back.
My user have a navigation to an entity named "Account" and the entity account have a navigation to "Billing".
Using EF.Property to refer to the navigation property throw me an exception

The Include property lambda expression 'x => Property(x, "Account")' is invalid. The expression should represent a property access: 't => t.MyProperty'.

Using EF.Property for my Where condition tells me that it can only be used within LINQ queries, but it is being used inside one.

System.InvalidOperationException: The EF.Property method may only be used within LINQ queries.

var usersQuery = _userManager.Users
    .Include(x => EF.Property<Account>(x, nameof(x.Account))) // throws
    .ThenInclude(x => x.Billing)
    .Where(nameof(Account) x => EF.Property<Account>(x, nameof(x.Billing)).IsOwned); // throws

var result = await usersQuery.ToArrayAsync();

And while I can use the string method with a combination of nameof() to keep my type safety, ThenInclude is not an extension that exists within that context and Where require a lambda anyway.
I'm not sure what to do.
var usersQuery = _userManager.Users
     .Include(nameof(ApplicationUser.Account))
     // .ThenInclude(x => x.Billing) the method doesn't exists
     .Where(x => EF.Property<Account>(x, nameof(x.Billing)).IsOwned); // throws

var result = await usersQuery.ToArrayAsync();

The page on backing fields need more documentation.


Consider changing naming structure for clarity's sake

The current use of with-* or of-* is a little confusing. We should decide the most common use case for the functions and then decide a naming scheme that makes the most "grammatical" sense for that use case.

We could potentially use the notion of set instead of "with"


An unexpected error encountered when requesting SAS token for the Azure Storage Network bypass

Hi,
I am running into an issue when reading data from Blob into Azure SQL DW. My database was recently restored (Friday) and I was able to read data from Blob into ADW until last night. Now I am getting an error saying "An unexpected error encountered when requesting SAS token for the Azure Storage Network bypass."
Any thoughts to why I get this error? Did the SAS token expire, thus affecting my scoped credentials?

@toenie23 Thanks for the feedback. We are actively investigating and will get back to you soon.

@Adam-Smith-MSFT I have been working with Tony and I think his SAS token has expired and need to reconfigure his external data source with a new token but, I want to ensure that is the issue.

@toenie23 You can check if the SAS token has expired via the URI you are using:
https://myaccount.blob.core.windows.net/sascontainer/sasblob.txt?sv=2015-04-05&st=2015-04-29T22%3A18%3A26Z&**se=2015-04-30**T02%3A23%3A26Z&sr=b&sp=rw&sip=<IP_ADDRESS>-<IP_ADDRESS>&spr=https&sig=Z%2FRHIX5Xcg0Mq2rqI3OlWTjEg2tYkboXr1P9ZUXDtkk%3D
The "se" is the Expiry time that is set for the token. you can find more information in this link: https://docs.microsoft.com/en-us/azure/storage/common/storage-dotnet-shared-access-signature-part-1
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[Bug]: WITH (UPDLOCK) does not create a proper row lock

Contact Details
<EMAIL_ADDRESS>What happened?
This happens with the BenchmarkSQL, which is a TPC-C style benchmark. The exact same code works fine with a real MS-SQL Server 2017.
The (relevant) queries of the NEW_ORDER transaction are
SELECT d_tax, d_next_o_id
    FROM bmsql_district WITH (UPDLOCK)
    WHERE d_w_id = ? AND d_id = ?

UPDATE bmsql_district
    SET d_next_o_id = d_next_o_id + 1
    WHERE d_w_id = ? AND d_id = ?

INSERT INTO bmsql_oorder (
    o_id, o_d_id, o_w_id, o_c_id, o_entry_d,
    o_ol_cnt, o_all_local)
    VALUES (?, ?, ?, ?, ?, ?, ?)

These are all JDBC PreparedStatement constructs. But that should not make any difference.
WITH (UPDLOCK) is T-SQL's equivalent of the SQL standard FOR UPDATE. Babelfish seems to accept the syntax but with no effect on the actual locking behavior. Under enough concurrency there are instances of transactions getting aborted due to
2022-03-12 19:32:41.154 UTC [56660] ERROR:  duplicate key value violates unique constraint "bmsql_oorder_pkeybmsql_oorder3723443fbaae7a4abcf8aec1f6574105"
2022-03-12 19:32:41.154 UTC [56660] DETAIL:  Key (o_w_id, o_d_id, o_id)=(8, 7, 3088) already exists.
2022-03-12 19:32:41.154 UTC [56660] CONTEXT:  SQL statement "INSERT INTO bmsql_oorder (    o_id, o_d_id, o_w_id, o_c_id, o_entry_d,     o_ol_cnt, o_all_local) VALUES ("@P0", "@P1", "@P2", "@P3", "@P4", "@P5", "@P6")"
	PL/tsql function inline_code_block line 1 at SQL statement
	TDS Protocol: Message Type: RPC, SP Type: SP_EXECUTE, Phase: TDS_REQUEST_PHASE_PROCESS. Processing SP_EXECUTE Request

This indicates that more than one session is able to retrieve the same row version from bmsql_district WITH UPDLOCK before the UPDATE happens, which should not be possible.
Version
BABEL_1_X_DEV (Default)
Extension
babelfishpg_tsql (Default)
What flavour of Linux are you running into the bug?
Other
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I can also reproduce the problem with two manual SQLCMD sessions.  WITH (UPDLOCK) ...
GO

for the exact same row in two sessions. And both sessions do retrieve the same row version. Even if the first session does and UPDATE, the second is not blocked from doing the SELECT (and getting the old row version because the UPDATE hasn't been committed yet).
This is the PostgreSQL behavior of SELECT and UPDATE without using FOR UPDATE.

This is a known current limitation. As documented, all hints are currently ignored.

This is not really a hint but a request to acquire a row level lock for the selected data. T-SQL does not support the ANSI-SQL SELECT ... FOR UPDATE [NOWAIT] syntax. That makes the WITH (UPDLOCK) a necessary, basic feature.
Is there a suggested workaround for this?

I do not disagree on the semantics, but technically this is a 'table hint' in the T-SQL dialect. As for a workaround, the same effect can be achieved by doing a dummy update first.

Thanks Rob.
I will try that workaround for now. It will double the number of dead rows for those actions (and the TPC-C has 3 of them), so it may have a negative impact on performance.

You can also generate a key value this way by first performing the update (SET d_next_o_id += 1) and then use the SELECT to pick up the generated value. This seems a little more direct than first doing a SELECT with an update lock and then doing the update.

It would work even better if T-SQL had support for UPDATE ... RETURNING.
For now I went with the dummy update first and can run BenchmarkSQL without errors. See this commit. I may change it later and see if it makes a performance difference.
I am currently trying to put a fix into the index build OOM problem so that I can compare Babelfish against plain PostgreSQL.
Thanks again.

You can do it as follows in SQL Server, but Babelfish does not currently support this syntax:
DECLARE @v int
UPDATE bmsql_district
SET d_next_o_id = d_next_o_id + 1
@v = d_next_o_id + 1
WHERE [...]
SELECT @v    ==>  contains the generated key value

The purpose of the UPDLOCK is to serialize on the key generation. The additional attributed can be retrieved after that. There are multiple ways of doing this though.

Rob, one of the primary goals for Babelfish was to provide a migration path for legacy applications without the need of invasive code changes. Rewriting a simple SELECT ... WITH (UPDLOCK) followed by an UPDATE into a completely reversed query logic is the exact opposite of that original goal. Especially when the correctness of such change can only be tested under concurrent load.
In this context I consider BenchmarkSQL an example of a legacy application. It works perfectly correct on MS-SQL 2017. Trying to run it on Babelfish requires to change the query logic in multiple places. The problem isn't even raised prominently, since the change in behavior is silent. One only runs into duplicate key errors when running this under high enough concurrency.
There is another place in the TPC-C where the UPDLOCK is essential. That is the PAYMENT transaction for customers with bad credit. It has to have a lock on the CUSTOMER row when retrieving the credit status to determine if it actually needs the rather large and network expensive c_data column. And only when that happens it performs rather complicated string manipulation on that data and finally uses a different query to update the CUSTOMER row by providing the new, client application generated, data for that column.
So while I agree with you that there are many ways to work around this missing feature on a case by case base, it is a bug infested mine field and needs attention ASAP.

I do not disagree that this is a feature that may be used by existing applications and which Babelfish should ideally support. But I thought we were discussing possible workarounds here, hence my comments.

Table Hints are currently not supported. The behavior to give errors is controlled by the escape hatch babelfishpg_tsql.escape_hatch_table_hints.


factorio: update all versions

also adds libpulseaudio, as experimental already needs is and soon
stable will need it too.
Motivation for this change
update
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @Baughn @elitak

Looks good; built and ran fine for me.

@filalex77: you're right, i've fixed the hashes and updated to the latest release. also built on my own hydra now, to check automatically if i messed up.
@Baughn: i've tested this and we need alsa as well as pulse.

@GrahamcOfBorg build factorio factorio-demo factorio-experimental factorio-headless factorio-headless-experimental

Looks okay. Do I need to tell the bot somehow?

@elitak These checks are not required to merge - they're just a helpful reference; thus, if you tested it, one of the maintainers should merge it when possible.

I have a weird error popping up after installing the derivation :
Description of the error
After building & installing the game and trying to start a new campaign, I get the following popup in game and in the console :
106.474 Error MainLoop.cpp:1195: Exception at tick 5158719: Error while running reload_script:
filesystem error: copy_file "/nix/store/dmsmbfvhsfgybbldgsmfczy2mj8y9jkg-factorio-demo-0.17.79/share/factorio/data/base/campaigns/npe/locale/uk/npe.cfg" to "/home/sirc/.factorio/temp/currently-playing/campaign-locale/uk/npe.cfg" failed:
Permission denied [/nix/store/dmsmbfvhsfgybbldgsmfczy2mj8y9jkg-factorio-demo-0.17.79/share/factorio/data/base/campaigns/npe/locale/uk/npe.cfg] [/home/sirc/.factorio/temp/currently-playing/campaign-locale/uk/npe.cfg]

System
From the binary itself :
   0.000 2019-12-23 22:20:12; Factorio 0.17.79 (build 47865, linux64, demo)

Steps to reproduce

Download the nix file
nix repl
pkgs = import <nixpkgs> {}
factorio = pkgs.callPackages factorio.nix {}
:i factorio
:q
factorio (as non-root)
Try to launch a new campaign

Notes
Of course, the error seems to stem from filesystem permission issues but I launch it as a normal user (the person that owns the home directory) so is this the expected behavior ?
Of course, launching the game as root solves the issue as expected but should you have to launch the game as root ?

Of course, launching the game as root solves the issue as expected but should you have to launch the game as root ?

Absolutely not, and I'm just assuming that because why would a game ever need to have such privilege?

Permission denied [/nix/store/dmsmbfvhsfgybbldgsmfczy2mj8y9jkg-factorio-demo-0.17.79/share/factorio/data/base/campaigns/npe/locale/uk/npe.cfg] [/home/sirc/.factorio/temp/currently-playing/campaign-locale/uk/npe.cfg]

This is actually a common problem that happens with NixOS.
It sounds like factorio copies these files from the nix store to some temp directory and wants to write to them, but it doesn't chmod them and make them owned by the proper user. I bet you can fix this on your own by fixing the permissions, and this is a pretty reasonable bug report to submit (I make one for it all the time).

This has got some testing in https://github.com/NixOS/nixpkgs/pull/77288, so I don't think there's much issues. Will merge.

bet you can fix this on your own by fixing the permissions, and this is a pretty reasonable bug report to submit (I make one for it all the time).

I will look into how the game developers want bug reports then.

I bet you can fix this on your own by fixing the permissions.

I will try then. I have just no idea which user should be given the permissions.

@TheSirC You probably want to chown /home/sirc/.factorio/ to your user and group, and make sure all the files have the correct writ-ability.

@TheSirC The directory that the error pointed doesn't exist according to the output of that command temp/currently-playing/campaign-locale/uk/npe.cfg

@TheSirC @worldofpeace I recently stumbled upon a same error after upgrading factorio to 0.18.18. I did some quick experiments and found that game removes files in .factorio/temp/currently_playing after you hit the button on error message window (which is logical, because you return to main menu and dir is called currently_playing). If you check the directory just after you get the error, you would see than npe.cfg file. it will be in RR- mode.
As a terrible workaround I just remounted /nix/store and manually did chmod +rw for share subfolder.

Remounting the nix store rw compromises the nix store and should not be done.

Remounting the nix store rw compromises the nix store and should not be done.

It did seem quite...sketchy (?)
What would be the recommended way to circumvent the issue then ?

@TheSirC. I see three possible ways. Only the third one being correct, imho.

Somehow fix the permissions inside /nix/store. I guess it may be hard because AFAIK they're reset to 444 on last installation stage
Implement some weird hook using LD_PRELOAD or something like that. We should detect copy being created and fix permissions on the fly.
Solve the problem upstream, I would say it's perfectly ok to have master-copy of files read-only. If not, why to copy them in the first place? It's responsibility of Factorio to ensure paths it's going to write to have appropriate permissions.

If you must write to the /nix/store path, you could have the wrapper script place either a bind mount or overlayfs over the directories that need write permission. Otherwise, I agree that getting the devs upstream to fix the issue is the best course. They are pretty responsive once you reach them. A post on their forums usually gets their attention.


Strange new line characters appearing in some contexts causing environments to be unrecognized Airbrake Java 2.2.8

I've noticed that some of my errors end up having newline characters around some of the Strings in the Context:
{
  "environment": "\ncmy.server.com\n",
  "notifier": {
    "name": "\nairbrake-java\n",
    "url": "\nhttps://github.com/airbrake/airbrake-java\n",
    "version": "\n2.2\n"
  },
  "remoteAddr": "<ip>",
  "rootDirectory": "\nnull\n"
}

This only appears to be happening occasionally but when it does it means that these errors won't match the correct environments that I"m deploying to using deploy tracking.

Any ideas?

Thanks!

Should be fixed in https://github.com/airbrake/javabrake


Updated CN official translations

CN released 2.6.5 main story this week

@MetLee Had to change the accessibility popup for play store approval, please update the translation.
If you join the discord server, I can ping you there in the future.

@MetLee Had to change the accessibility popup for play store approval, please update the translation.
If you join the discord server, I can ping you there in the future.

Sorry I have no discord account. You can ping me on GitHub and I will receive a email immediately.

@MetLee Had to change the accessibility popup for play store approval, please update the translation.
If you join the discord server, I can ping you there in the future.

Which translations do I have to update? You just added <string name="discord" translatable="false">Discord</string> in the last commit.


ERROR: Can't open dynamic library

Describe the bug
The first line I get from executing the game from the terminal is this one:
"ERROR: Can't open dynamic library: addons/godotsteam/linux64/libgodotsteam.x86_64.so. Error: /home/mateus/.steam/debian-installation/steamapps/common/Where is the Princess/../lib/libgodotsteam.x86_64.so: cannot open shared object file: No such file or directory."
I have no folder named lib on the common directory. Is the path correct? Shouldn't the lib be inside the game directory instead of the common directory. I have seen other games that have a lib directory inside their game directory.
To Reproduce
Steps to reproduce the behavior:

Installed the GodotExtension version of Godotsteam
Exported the game
Uploaded the game to steam
Execute it

Expected behavior
I don't know, maybe I'm missing some step.
Desktop (please complete the following information):

OS: PopOS
Version 22.04 LTS

Version of Godot:
4.2.1
Version of GodotSteam:
GDExtension 4.8

Hey there!  What does your game's Steam installation folder look like?

Hi, I put the godotsteam folder inside the addons folder. I get autocompletion for the Steam singleton, so it seems it activates correctly.

Oh, I mean when you upload it to Steam and then the player installs it from their library, what are the contents of the Steam installation folder?

All good!  Glad it got solved; sometimes it helps to have a second set of eyes.


Support outputting Clang compilation database

See https://clang.llvm.org/docs/JSONCompilationDatabase.html for details about the format. Used by tools such as clangd to provide much better intellisense.

It's generated by the script, from the flags pushed to dbFlags

It's generated by the script, from the flags pushed to dbFlags

Ah gotcha, so up to the user of this library, makes sense :)


Remove block system; create a static homepage with banner

Banner should include a welcome message with number of experiments, biosources, assays, etc.
Make news populate from a JSON (maybe start with static at first)

"welcome to 4DN data portal. We host 0 4DN experiments from 0 different cell types, and 201 external experiments from 7 different cell types." Needs #132

Finished homepage and added static routing; made a banner that presents 4DN and external experiment numbers & biosources number.


Small code improvements in facet component

backporting "Debugging FacetComponent I noticed two little things which can be improved"

 Files changed 5,000+
Not good. Ok maybe I put it wrong, but it's cherry-picking as described in dev docs.


"Remove additional files at destination" should not delete other WebJobs/web services in the same Azure App Service

It does not make sense that enabling "Remove additional files at destination" for a WebJob/web service should delete other WebJobs/web service in the same Azure App Service.

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

ID: 4b648a9b-9a8d-423f-94c9-cb56ec458273
Version Independent ID: efe57b89-fbac-8a5c-4607-102ebfd57b54
Content: Azure App Service Deploy task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-rm-web-app-deployment.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Why not?  Don't those other things consist of files at the destination?

Maybe? I suppose thats a question for Microsoft as to how they have implemented the app service sandbox
Its still unintuitive

I guess I don't understand why you would enable "delete additional files at destination" unless you understood what additional files existed at the destination.

It’s unexpected that by default, files other than the ones that were ever part of a given project would exist at a projects publish destination
The fix to this should probably be to not deploy WebJobs to the same destination as the web app

Sounds pretty reasonable when you put it that way.


Call next() instead of sending status 404 if no response in available

It will be useful to call next() instead of immediately sending status 404 to client. By calling the next function, it becomes possible to describe the processing to be performed as a fallback when apex cannot respond to the request (for example, when the client cannot accept JSON-LD response).
app.get(routes.actor, apex.net.actor.get, (req, res) => {
	// fallback
});

If no next handler is defined, express.js will automatically send 404 status to client. As such, this fix probably won't require major changes from users of the library.

Sorry, this change was not relevant to fallback.
For future reference, fallback route can be defined as following code.
app.get(routes.actor, apex.net.actor.get)

app.get(routes.actor, (req, res) => {
	// fallback
})


Add support for LISTEN and NOTIFY

I'm not all that comfortable with libpq so give this extra scrutiny. 🙊
I used this example heavily to help me determine how to receive notifications using libpq. I also purposefully broke convention and used a closure in listen instead of creating a onNotification property off Database since a database could be listening to multiple channels simultaneously.
Couple of things that I wanted to point out for feedback:

What to do with errors caught in listen since the closure in try background can't throw? Using fatalError right now but not sure if that's the correct approach.
The name of the Notification struct. It's not really meant to be used outside the listen closure so it's maybe okay to keep named as is even though it collides with Foundation's own Notification.

Codecov Report

Merging #29 into master will increase coverage by 0.01%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   93.36%   93.37%   +0.01%
==========================================
  Files          11       12       +1
  Lines        1748     1812      +64
==========================================
+ Hits         1632     1692      +60
- Misses        116      120       +4

Impacted Files
Coverage Δ

Sources/PostgreSQL/Notification.swift
100% <100%> (ø)

Tests/PostgreSQLTests/PostgreSQLTests.swift
95.72% <100%> (+0.22%)
:white_check_mark:

Sources/PostgreSQL/Database.swift
89.13% <85.71%> (-1.5%)
:x:

Continue to review full report at Codecov.

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

@natebird Made the changes you requested and also found a slight performance increase using string concatenation instead of interpolation when constructing the SQL statements, especially for large NOTIFY payloads.

@tanner0101 @LoganWright - do you know why CircleCI isn't responding to PRs on this repo?

Circle only responds to PRs which are started from this repo, not external forks.
I don't know why it's so.
The typical workaround for me was to push the other branch here and start a new PR - you should be able to do this with your permissions.

@vzsg I figured out the issue. It is a setting in Circle. @johnbona can you push an empty commit to this branch again to kick Circle?

It's not going to help, as this PR is still originated from johnbona:feature/listen-and-notify.

Sorry, I couldn't resist :)

Yay!

@johnbona can you review this again? The tests are failing.
Now that we have a 2.0 Beta release for this repo we can merge this PR into the 2.0 release.
Thanks!

@natebird Should be good now!


Add star indexing workflow

As part of the genetic demulitplexing, we need a STAR genome index. We will use this for bulk mapping to find SNPs.
This PR adds a workflow to create that index, putting the result with the other indices we have created at: s3://nextflow-ccdl-data/reference/homo_sapiens/ensembl-104/star_index/
Currently the index is based on the full gtf file; it might be a bit smaller if we reduced it to the gtf file that only includes the genes in the transcriptome gtf; I may test this.
Otherwise, this should be a pretty simple workflow. My only annoyance is that the input files had to be decompressed on disk :-(.
THe other question was the naming of the index output directory. I named it just with the Assembly, but I could add some suffix, like .star_idx if that seemed like something worth doing.

Currently the index is based on the full gtf file; it might be a bit smaller if we reduced it to the gtf file that only includes the genes in the transcriptome gtf; I may test this.

Nope, doesn't make a difference. Unless there is another reason, I will stick with the default Ensembl file for simplicity.


Implement solution to create a release for a repo

The solution will implement the methods and UI/X from the other issues in this epic.
Create a FAB (floating action button) to allow the user to create a release for a repo.  This button will eventually be used for other actions such as PDF creation.
DoD:
User can create a release of a repo.

@birchamp one clarification... do you want one button to replace both functions, namely:

add a book to the workspace
release a repo (resource)

That would follow a common mobile motif. For example, the "plus" sign in this screenshot, which presents a list of things to do when clicked:

I'm not sure.  I was wondering about that myself.   If we're going to use the tab view like the mockups show then we could have the main function (create Book) as the upper right button, create user button on the users tab, create assignment on the task tab.

Here is a screen shot of where the repo release page is invoked from within the nav drawer:

To Do:

As soon as the resource is selected, then two things must be done:
a. Verify that the master branch has a valid manifest. If it does not then the release will not show in the catalog, which is the whole point of doing a release.
b. If there is a valid manifest, then get the latest release version to show to the user.
Insert between the dropdown and version text entry an information status. It will be one of the following:

Cannot release: manifest is not valid
No existing releases. This will be the first one.
Last release was: vXXX

How to determine if a manifest is valid for a resource repo:
TBD

How to identify latest release...
Case 1: There is no release yet.
The API:
curl -X 'GET' \
  'https://qa.door43.org/api/v1/repos/es-419_gl/es-419_tn/releases?draft=false&pre-release=false&page=1&limit=9999' \
  -H 'accept: application/json'

This returns a body of []. I.e., an empty array. Thus no releases exist yet.
Case 2: There are existing releases.
The API:
curl -X 'GET' \
  'https://qa.door43.org/api/v1/repos/unfoldingword/en_tn/releases?draft=false&pre-release=false&page=1&limit=9999' \
  -H 'accept: application/json'

Snippet of returned body:
[
  {
    "id": 875880,
    "tag_name": "v58",
    "target_commitish": "master",
    "name": "Version 58",
    "body": "This is version 58 of the unfoldingWord® Translation Notes for the Bible.
  }
  ... elided
]

Since the results are conveniently returned in last-in-first-out order, then results[0].tag_name will yield the latest version, namely, v58

Next up... using the form inputs and the user clicks to create the release...
Sample URL for POST:
https://qa.door43.org/api/v1/repos/es-419_gl/es-419_tn/releases
Here is sample JSON for the POST.
{
  "tag_name": "v1.0.0",
  "target_commitish": "master",
  "name": "v1.0.0",
  "body": "First release of es-419 Translation Notes",
  "draft": false,
  "prerelease": false
}

There are three expected response statuses:

201: success
404: not found
409: json format error

@mandolyte
I actually think it is best to go by the catalog API v5 endpoint rather than the Gitea v1 endopint which has validated the release to be catalog-worthy:
So:

no catalog release:
https://qa.door43.org/api/catalog/v5/search/es-419_gl/es-419_tn

has catalog release:
https://qa.door43.org/api/catalog/v5/search/unfoldingword/en_tn

This doesn't require draft and pre-release set to false as by default it is only production releases. I'm suggesting this because anyone can make a release with even bad data, but can't get into the catalog unless it it has been validated (of course we currently only validate the manifest.yaml file, but in the future there might be more things to check)


Add feature: API

Asked on original Wekan Gitter by @RaviRokkam_twitter
RaviRokkam wanted to integrate website with Wekan, so if task is added to website, it should be added as a card to Wekan.
This would require API to access Wekan data.
For those interested in this, please add all existing previous info as listed in 2) of https://github.com/wefork/wekan/wiki/FAQ

Related Python-based project:
https://github.com/LINKIWI/wekan-api

LINKIWI's wekan-api works but seems to have read-only access. So adding a card is not included.

Discussion moved back to https://github.com/wekan/wekan/issues/166


fallback avatar image added

Empty avatar circle for profiles without profile image looks like error with fetching profile image, so I put placeholder here.
before:

after:

Thanks, this does need to be done. However, I'd really like to deterministically generate images based on pubkey so we don't have a ton of undifferentiated orange person icons. I believe this is what Damus does with the robot avatars, although I hate those. This can be done similarly to how relay colors are calculated. Maybe masking the coracle logo with a color palette generation algorithm:

What do you think about it?

This has been merged, with a rebase and some modifications. In particular, I re-used the hsl color generation from relays, which makes it much easier to create a pretty color palette. Thanks!


Add --clean option

Specifying the --clean argument should make sure that the output directory only contains resources and content that belongs to the datasources being built.
It might either delete the output directory (if it already exists), or implement a removal of any existing, but unused, resources.

Maybe the cleaning should actually be the default, and the argument should instead be --dont-clean to avoid having it delete the unused files?


Stormblessed: Blacklist miningchamp.com

Stormblessed requests the blacklist of the website miningchamp\.com. See the MS search here and the Stack Exchange search here.

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

There are actually 5 posts on MS. All of them are TP. The post that was FP had previously unnoticed undisclosed affiliation.

However, it is used in another post by a high rep user. Given that some of the posts are TP due to undisclosed affiliation, I'm concerned that future posts will be FP, so I'd prefer to leave this on the watchlist for now.
I've custom flagged both of the most recently posted answers with this domain. Neither is a great answer and both have spam-like issues. Let's wait to see how those flags are resolved.

Both of my flags are still pending.

Approved by double-beep in Charcoal HQ


Leverage Dict-based aux_operators

What is the expected enhancement?
Terra has just added support for Dict-based aux_operators in its Eigensolver interfaces (https://github.com/Qiskit/qiskit-terra/pull/6870). This can greatly improve the handling within the property framework.

I believe the aforementioned PR also slightly changed the list-based interface by removing one level of list-nesting. This went unnoticed until the CI on the main branch failed last night, presumably because the terra unittests do not cover the aux_operators interface? :thinking:

Okay some more observations:

the NumPyEigensolver has (since its first commit) returned the auxiliary operator eigenvalues as a tuple: (value, 0). current code. It is not 100% clear to me, why it is done this way. I also did not find documentation on this matter. Neither in the algorithm itself nor in the Result object. The only suggestion I have would be that the 0 is a place-holder for when we will have standard-deviation information on this eigenvalue (which in the numpy case is always 0).
in order to match this interface (that each aux_op_eigenvalue is actually a tuple) the VQE used to wrap each evaluated aux_op into a list here which is no longer the case in the new code here

There are two solutions:

Remove the tuple style from NumPyMinimumEigensolver (strongly disfavored)
Change the VQE to also return tuples (not simply wrap the eigenvalue into a list, imo) and document the return type properly for the future.


How to use react-scripts in combination with electron?

I try to combine a create-react-app started with react-scripts with electron. Is it possible to setup my electron project to run from localhost in development mode? And how to keep node functionality like fs and __dirname alive through webpack build process?

https://medium.freecodecamp.org/building-an-electron-application-with-create-react-app-97945861647c

if you want your renderer process to use actual node or electron APIs, you'll need something like this

For two package.json architecture (one for js, one for native modules), you should be able to ignore src/node_modules.
Here is my updated config-rewired.js
// @flow
const path = require('path');

const ignorePath = function (exclude = [], config) {
  const rule = config.module.rules[0];
  if (!rule) {
    console.log('js related rule not found');
    return config;
  }
  rule.exclude = exclude.concat(rule.exclude || []);
  return config;
};

module.exports = function override(config, env) {
  // Make it run in electron renderer process
  // If we want electron start, we will set cross-env BROWSER=none
  if (process.env.BROWSER === 'none') {
    delete config.node;
    config.target = 'electron-renderer';
  }

  config = ignorePath([path.resolve(__dirname, 'src/node_modules'), path.resolve(__dirname, 'src/~')], config);

  if (env === 'production') {
    console.log('⚡ Production build with Optimization.');
  }

  return config;
};

Btw, if you need to use native modules, I had to include them using remote like so:
const odbc = require( "electron" ).remote.require( "odbc" );


feat(toast): allow two rows label in Toast

Closes #16

Can the code be a mixin that is reused in HeaderTitle in packages/core/src/Dialog/Dialog.tsx as well? Something like this:
const twoLinesClamp = styled.css`
  /* stylelint-disable-next-line value-no-vendor-prefix */
  display: -webkit-box;
  /* stylelint-disable-next-line property-no-vendor-prefix */
  -webkit-line-clamp: 2;
  /* stylelint-disable-next-line property-no-vendor-prefix */
  -webkit-box-orient: vertical;
  /* stylelint-disable-next-line property-no-vendor-prefix */
  -moz-box-orient: vertical;
  overflow: hidden;
  white-space: pre-wrap;
`

...
export const ToastLabel = styled(Typography).attrs({
  variant: 'chip-tag-text',	  variant: 'chip-tag-text',
})<{	})<{
  readonly hasCloseButton: boolean	  readonly hasCloseButton: boolean
  readonly isError: boolean	  readonly isError: boolean
  readonly hasEmphasis: boolean	  readonly hasEmphasis: boolean
}>`	}>`
  height: 100%;	  height: 100%;
  ${twoLinesClamp};
  ...

This way we will have stylelint-disable-next-line only in one file.

Suggestions:

More descriptive name than twoLinesStyle
A comment with a description above for intellisense info in VS Code
twoLinesStyle  is exported in index.ts file so it could be imported from practical-react-components-core and reused in the application

Suggestions:

More descriptive name than twoLinesStyle
A comment with a description above for intellisense info in VS Code
twoLinesStyle  is exported in index.ts file so it could be imported from practical-react-components-core and reused in the application

Fixed all your suggestions. Changed the name to your original suggestion twoLinesClamp because the css property name is line-clamp. @anius

Suggestion:
/**
 * A styled-components mixin that truncates two-lines text with ellipsis.
 *
 * An example:
 * ```tsx
 * const Div = styled.div`
 *   ${twoLinesClamp};
 * `
 * ```
 */

or similar


Add a screenshot in the readme

Adding a screenshot in the readme will give people most of the information they need on if this is something they want or not. If it looks good and continues to be maintained, I'll add a link to it in the line_profiler README itself.

Thanks for your kind words!
Screenhots are definitely planned (see #7) but I'll wait a bit so that the interface is more stabilised. Soon(TM)


SemanticARCOpts fix - remove deadEndBlocks checks

SemanticARCOptVisitor::performGuaranteedCopyValueOptimization was
converting this SIL
%borrow = begin_borrow %copiedValue
%copy = copy_value %borrow
%borrowCopy = begin_borrow %copy
end_borrow %borrow
end_borrow %borrowCopy
destroy_value %copy
// something something
unreachable

into
%borrow = begin_borrow %copiedValue
%innerBorrow = begin_borrow %borrow
end_borrow %borrow
end_borrow %innerBorrow
// something something
unreachable

Dead-end blocks are simply irrelevant for this
optimization. Unfortunately, there were multiple layers of attempted
workarounds that were hiding the real problem, except in rare cases.
Thanks Nate Chandler for reducing the test.

I have a working branch that removes dead-end blocks nonsense completely from OSSA and further cleans up the code. But this targeted bug fix is urgently needed.

@swift-ci test


Sprint 3 staging

added to the read.me file about genomic test directory updates

Sprint 3 finnished!


Fresh install not launching - r1 container restarts over and over.

Describe the bug
Tried to update to the new version of r2r using 'pip install -U r2r'. The r1 container never starts, it just loop restarts. Tried deleting the containers and re-pulling. Tried deleting the conda environment and starting completely fresh with a new r2r install.
To Reproduce
Steps to reproduce the behavior:

On Mac
pip install r2r
r2r serve --docker --full
r2r never fully starts up - terminal hangs because the r1 container never hits 'healthy' or whatever.

Expected behavior
Containers all start and dashboard comes up in a browser window
Desktop (please complete the following information):

OS: MacOS 15.1
Browser: Safari - not the issue since the container never starts
Version: latest using 'pip install r2r'. Tried 'pip install -U r2r' also. r1 container says prod:latest.

Additional context
Log/error from the r1 container:
2024-10-17 17:33:43 2024-10-18 00:33:43,588 - ERROR - root - Error creating providers, pipes, or pipelines: value too large to convert to int
2024-10-17 17:33:43 ERROR:    Traceback (most recent call last):
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/starlette/routing.py", line 693, in lifespan
2024-10-17 17:33:43     async with self.lifespan_context(app) as maybe_state:
2024-10-17 17:33:43                ^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/contextlib.py", line 210, in aenter
2024-10-17 17:33:43     return await anext(self.gen)
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/app_entry.py", line 22, in lifespan
2024-10-17 17:33:43     r2r_app = await create_r2r_app(
2024-10-17 17:33:43               ^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/app_entry.py", line 62, in create_r2r_app
2024-10-17 17:33:43     return await builder.build()
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/assembly/builder.py", line 198, in build
2024-10-17 17:33:43     providers = await self._create_providers(
2024-10-17 17:33:43                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/assembly/builder.py", line 147, in _create_providers
2024-10-17 17:33:43     return await factory.create_providers(*args, **kwargs)
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/assembly/factory.py", line 361, in create_providers
2024-10-17 17:33:43     or self.create_orchestration_provider(self.config.orchestration)
2024-10-17 17:33:43        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/main/assembly/factory.py", line 129, in create_orchestration_provider
2024-10-17 17:33:43     orchestration_provider = HatchetOrchestrationProvider(config)
2024-10-17 17:33:43                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/app/core/providers/orchestration/hatchet.py", line 22, in init
2024-10-17 17:33:43     self.orchestrator = Hatchet(
2024-10-17 17:33:43                         ^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/hatchet_sdk/hatchet.py", line 193, in init
2024-10-17 17:33:43     self._client = new_client(config, debug)
2024-10-17 17:33:43                    ^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/hatchet_sdk/client.py", line 44, in from_environment
2024-10-17 17:33:43     return cls.from_config(config, debug)
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/hatchet_sdk/client.py", line 64, in from_config
2024-10-17 17:33:43     conn: grpc.Channel = new_conn(config)
2024-10-17 17:33:43                          ^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/hatchet_sdk/connection.py", line 50, in new_conn
2024-10-17 17:33:43     conn = start.insecure_channel(
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/grpc/init.py", line 2117, in insecure_channel
2024-10-17 17:33:43     return _channel.Channel(
2024-10-17 17:33:43            ^^^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "/usr/local/lib/python3.12/site-packages/grpc/_channel.py", line 2074, in init
2024-10-17 17:33:43     self._channel = cygrpc.Channel(
2024-10-17 17:33:43                     ^^^^^^^^^^^^^^^
2024-10-17 17:33:43   File "src/python/grpcio/grpc/_cython/_cygrpc/channel.pyx.pxi", line 504, in grpc._cython.cygrpc.Channel.cinit
2024-10-17 17:33:43   File "src/python/grpcio/grpc/_cython/_cygrpc/arguments.pyx.pxi", line 76, in grpc._cython.cygrpc._ChannelArgs.cinit
2024-10-17 17:33:43   File "src/python/grpcio/grpc/_cython/_cygrpc/arguments.pyx.pxi", line 42, in grpc._cython.cygrpc._ChannelArg.c
2024-10-17 17:33:43 OverflowError: value too large to convert to int
2024-10-17 17:33:43
2024-10-17 17:33:43 ERROR:    Application startup failed. Exiting.

Can you try updating with r2r update instead of pip install -U r2r' and try again—I believe that your environment might be in an odd state, as this was an issue in the docker compose file that has since been patched.
If that doesn't work, can you look for the compose.full.yaml and change all instances of HATCHET_CLIENT_GRPC_MAX_RECV_MESSAGE_LENGTH to 134217728 (or anything less than 2^32.)

Can you try updating with r2r update instead of pip install -U r2r' and try again—I believe that your environment might be in an odd state, as this was an issue in the docker compose file that has since been patched.
If that doesn't work, can you look for the compose.full.yaml and change all instances of HATCHET_CLIENT_GRPC_MAX_RECV_MESSAGE_LENGTH to 134217728 (or anything less than 2^32.)

Thank you. I installed an old version, got it up and running where I could log in and interact (installed 3.2.10), then ran 'r2r update' - same issue, same log error.
Looked in the compose.full.yaml and found these lines:
  # Hatchet
  - HATCHET_CLIENT_TLS_STRATEGY=none
  - HATCHET_CLIENT_GRPC_MAX_RECV_MESSAGE_LENGTH=${HATCHET_CLIENT_GRPC_MAX_RECV_MESSAGE_LENGTH:-13421772800}
  - HATCHET_CLIENT_GRPC_MAX_SEND_MESSAGE_LENGTH=${HATCHET_CLIENT_GRPC_MAX_SEND_MESSAGE_LENGTH:-13421772800}

  Why is it there twice? Oh well, changed both of the negative numbers to -134217728 and it worked. I'd like to understand whats going on as well as why it's negative when you gave me a positive number to put there. Also why is it there twice?

  Anyway, thanks @NolanTrem. Working for now but prob should fix the repo :). appreciate you!

It's not negative, it's just the formatting that Docker Compose uses to say "use the HATCHET_CLIENT_GRPC_MAX_RECV_MESSAGE_LENGTH variable otherwise default to 134217728"
We also do the same other places, such as:
image: ${R2R_IMAGE:-ragtoriches/prod:latest}
- OPENAI_API_KEY=${OPENAI_API_KEY:-}
The issue here is that Hatchet (which is written in Go) is casting these to an int and not an int64. The original number was more than 32 bits, causing an issue there. This worked in prior releases, but something changed in Hatchet that caused it to break.
It's definitely possible that you'd get this error later on with send, so you're correct to change both.
I believe it is only broken on one release, and it seems to be correct on main.

I believe it is only broken on one release, and it seems to be correct on main.

Sorry for being slow but what do you mean here? Shouldn't the pip install and update pull from the main branch? You can feel free to close the issue with your next comment. Appreciate you again.

No worries at all! You're 100% right—the pip package that you have should match what's on main… but honestly pip package management can be a pain, and sometimes just doesn't work. I did an r2r update and saw that I was on the latest, then vimed into the compose.full.yaml and saw that this is the correct (< 32 bit number) so I think it's good (maybe? hopefully?) and that your pip environment might be pointing to the wrong thing... Glad it's all working for you now, and feel free to reach out if you have any questions!


[CLOSED] Fetch tag for manuals when publishing

 Issue by tommyp
Friday 31 July 2015 at 15:21 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/532

In order for Manuals to appear on topic pages we need to fetch the topics in Content API and send those to Rummager when publishing. This PR adds a TagFetcher class which gets the tags and modifies the Manual before saving and publishing it.
The last commit has some shoddy stubbing at a random step. This looks a bit weird but I couldn't figure out where to stub this anywhere else that would result in my tests passing.
Paired on this with @KushalP. Ticket.

tommyp included the following code: https://github.com/alphagov/specialist-publisher/pull/532/commits

 Comment by danielroseman
Friday 31 July 2015 at 17:06 GMT

I don't know anything about this story, but given we are just finishing up a big piece of work to untangle specialist-publisher from content-api it's a shame this adding another dependency on it. Is there any other way to get this data?

 Comment by bradwright
Monday 03 August 2015 at 08:42 GMT

We have a ticket to remove all connections with Content API from this publisher, which this PR will block. Can we get the topics from anywhere else?

 Comment by KushalP
Monday 03 August 2015 at 09:46 GMT

@danielroseman the different options that we looked at for querying this data can be found here: https://gov-uk.atlassian.net/wiki/display/FS/HMRC+manuals+update+-+29+JUL+2015
We opted to query the Content API here because it would be easier to throw away in the near future and make sure that Rummager has the correct data.

 Comment by bradwright
Monday 03 August 2015 at 09:49 GMT

From some conversations I've had, it seems like this is (regrettably) the only viable solution that doesn't create more work later. So let's go with this, but make sure it's documented as tech debt on the Wiki.

 Comment by KushalP
Monday 03 August 2015 at 09:52 GMT

but make sure it's documented as tech debt on the Wiki

Are there any good examples of how we're doing this at the moment?

 Comment by bradwright
Monday 03 August 2015 at 09:52 GMT

You're going to need to reconcile this PR with https://github.com/alphagov/specialist-publisher/pull/533, though.

 Comment by bradwright
Monday 03 August 2015 at 09:54 GMT

Are there any good examples of how we're doing this at the moment?

See the Whitehall app page.

 Comment by KushalP
Monday 03 August 2015 at 10:04 GMT

You're going to need to reconcile this PR with #533, though.

Yup. @tommyp and I have discussed and we're okay with dealing with the merge conflicts on this PR if #533 is likely to get merged soon. I'll chat to @elliotcm about this.

 Comment by elliotcm
Monday 03 August 2015 at 10:07 GMT

We're expecting to merge https://github.com/alphagov/specialist-publisher/pull/533 some time today.

 Comment by tommyp
Tuesday 04 August 2015 at 08:41 GMT

Fixed the commits based on @mikejustdoit's comments.

 Comment by mikejustdoit
Tuesday 04 August 2015 at 14:32 GMT

:+1: I'm happy with this, but does it need rebasing or something before merging?

 Comment by mikejustdoit
Tuesday 04 August 2015 at 14:31 GMT

I think I'd prefer if new features like fetching tags lived in their own services, but

@KushalP's explained why that isn't necessarily beneficial in this case, and
I don't think we've figured out a nice way to use multiple services like that yet

 Comment by KushalP
Tuesday 04 August 2015 at 14:32 GMT

I think I'd prefer if new features like fetching tags lived in their own services

That's one of the things being discussed in the early tagging discussions I mentioned :rocket:

 Comment by tommyp
Tuesday 04 August 2015 at 14:38 GMT

I think I'd prefer if new features like fetching tags lived in their own services

Further to @KushalP's reply, this App will go through some major refactoring sooner or later.


[sc-22645] Add Active Directory Adapter Catalog

This is an initial version of the adapter catalog:
Shortcut story

sc-22645

Tagging @marc-sgnl to identify a minimal set of attributes for this SoR. Not a blocker for this PR as this can be done in a follow up PRs as well.

Tagging @marc-sgnl to identify a minimal set of attributes for this SoR. Not a blocker for this PR as this can be done in a follow up PRs as well.

Ack np - I'll wait until we merge this and then create a PR

I feel we should create a catalog per distribution, It'll be easier to maintain the catalog going ahead.
WDYT @r-rakshith?


age 1 MSE simulations

Hi all - what do we want to do about including the closed-loop simulation results on the performance of an age 1 index that we presented to the JMC last May? Will this be an additional appendix?
Otherwise, since it's an area that nobody is currently editing, I could write a quick summary describing that little progress was made by the JTC on operating model develoment due to workload and other committments; but that there is the development of a spatial operating model occurring at UBC that we think will be useful for future analyses. I can put this in the UNRESOLVED PROBLEMS AND MAJOR UNCERTAINTIES section.
N.

I think we agreed a while back to not have an MSE appendix.
I won't touch the "**NGT suggest next two paragraphs be removed" parts then, so hopefully we can easily merge.

Though I've tweaked some of the earlier paragraphs, and the merge stuff checks conflicts by line numbers. Maybe just write some text and just post it here and I'll copy it in, since it won't be long will it....

Roger that. It'll be brief anyway

Hi Nathan,
We could report the age-1 MSE investigations, but may as a late addition
once we have the rest of the document complete. I included a couple of
sentences regarding this in the acoustic data section, but it probably is
better to put in a more results oriented section.
I am starting to think that we don't need to report on our progress on the
MSE in the assessment document, but we should report on that status
somewhere. I feel that the assessment document should be results and
analysis. Maybe a short separate document that can be presented and given
to the SRG and JMC about our progress?  This can be something we do after
the 8th, but before hte SRG meeting.
I haven't looked at your paragraphs yet.
Allan
On Wed, Feb 3, 2016 at 10:27 AM, minnowslayer<EMAIL_ADDRESS>wrote:

Roger that. It'll be brief anyway
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/36#issuecomment-179390576
.

I was going to write something like the following:
Due to other assessment commitments, the JTC made modest progress on MSE development.  We presented the closed-loop simulations evaluating the effect of including potential age 1 indices that were not yet complete in February 2015 at the May 6-7 JMC meeting in Victoria, B.C. The simulations showed that: introduction of age-1 survey results in lower catch, lower P(<B10 $%$) and a lower AAV but that comparable results in terms of catch and could be achieved using higher precision of age-2+ survey or alternative harvest control rules; the simulations assumed that an age-1 survey design with consistent, effective, and numerous sampling was used. In addition, the JTC has been communicating with a Ph.D. student at the University of British Columbia that has built a spatial operating model for Pacific hake: she presented her spatial operating model at the JTC’s December 2015 meeting; once her simulation analysis using it is complete and the results published, it could be a useful tool for the Pacific hake MSE.
But i think a separate document might be easier. Including some description of the results that I have above invites presenting the full set of results which i don't think we want. Perhaps we could flag this as a discussion for tomorrow's call?

Sounds good to me.
On Wed, Feb 3, 2016 at 11:14 AM, minnowslayer<EMAIL_ADDRESS>wrote:

I was going to write something like the following:
Due to other assessment commitments, the JTC made modest progress on MSE
development. We presented the closed-loop simulations evaluating the effect
of including potential age 1 indices that were not yet complete in February
2015 at the May 6-7 JMC meeting in Victoria, B.C. The simulations showed
that: introduction of age-1 survey results in lower catch, lower P(<B10
$%$) and a lower AAV but that comparable results in terms of catch and
could be achieved using higher precision of age-2+ survey or alternative
harvest control rules; the simulations assumed that an age-1 survey design
with consistent, effective, and numerous sampling was used. In addition,
the JTC has been communicating with a Ph.D. student at the University of
British Columbia that has built a spatial operating model for Pacific hake:
she presented her spatial operating model at the JTC’s December 2015
meeting; once her simulation analysis using it is complete and the results
published, it could be a useful tool for the Pacific hake MSE.
But i think a separate document might be easier. Including some
description of the results that I have above invites presenting the full
set of results which i don't think we want. Perhaps we could flag this as a
discussion for tomorrow's call?
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/36#issuecomment-179409738
.


Size analysis preflight task

#245  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


Chore: convert build system to webpack

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

几个命令都跑了一遍，没什么问题了。


Translation of 2.mdx

Translation of 2.mdx (chapter1/2) + update of ToC.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Clicking "Join Meeting" just opens another Mattermost window

Hi,
I just entered "meet.jit.si" as server and can start calls now. But when I click on the "Join Meeting" button, it opens a new browser window with the Mattermost instance again with the Jitsi-URL appended: https://mattermost.foo.bar/meet.jit.si/ instead of https://meet.jit.si/
What could I do here? If I enter https://meet.jit.si as server, it doesn't work at all with this log message:

2018-09-18T11:28:43.736156Z app[...]: {"level":"error","ts":1537270123.7358959,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"jitsi","error":"connection is shut down"}

Hey @philiplb the url should have https://meet.jit.si as the url, and there have been a few issues that I have been observing, related to the plugin crashing due to the way that golang is converting errors. This was causing a whole host of issues, and I have just released a new version that should fix most of them. Please let me know if this works for you!

Hi, thank you for the very fast reaction. Unfortunately the new version of the plugin fails to start (Mattermost 5.3.0):

2018-09-18T13:53:59.256317Z app[...]: {"level":"error","ts":1537278839.2560217,"caller":"app/plugin.go:70","msg":"Unable to activate plugin","plugin_id":"jitsi","error":"unable to copy webapp bundle directory: jitsi: stat plugins/jitsi/static: no such file or directory","errorVerbose":"stat plugins/jitsi/static: no such file or directory\nunable to copy webapp bundle directory: jitsi\ngithub.com/mattermost/mattermost-server/plugin.(*Environment).Activate\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/plugin/environment.go:183\ngithub.com/mattermost/mattermost-server/app.(*App).SyncPluginsActiveState\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/app/plugin.go:68\ngithub.com/mattermost/mattermost-server/app.(*App).InitPlugins.func2\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/app/plugin.go:141\ngithub.com/mattermost/mattermost-server/app.(*App).InvokeConfigListeners\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/app/config.go:134\ngithub.com/mattermost/mattermost-server/app.(*App).UpdateConfig\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/app/config.go:46\ngithub.com/mattermost/mattermost-server/app.(*App).EnablePlugin\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/app/plugin.go:206\ngithub.com/mattermost/mattermost-server/api4.enablePlugin\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/api4/plugin.go:183\ngithub.com/mattermost/mattermost-server/web.Handler.ServeHTTP\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/web/handlers.go:131\ngithub.com/mattermost/mattermost-server/web.(*Handler).ServeHTTP\n\t:1\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux.(*Router).ServeHTTP\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux/mux.go:162\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers.recoveryHandler.ServeHTTP\n\t/home/ubuntu/jenkins/workspace/msr/mattermost-platform-release/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers/recovery.go:78\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers.(*recoveryHandler).ServeHTTP\n\t:1\nnet/http.serverHandler.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2694\nnet/http.(*conn).serve\n\t/usr/local/go/src/net/http/server.go:1830\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:2361"}

I would love to know how I uploaded version 0.2.0 as the release, but I verified that I have uploaded the correct files this time. You should be able to download them again from release 1.0.1, and they should work correctly now.

Works now like a charm, thank you very much!


self.ocr_df.text_size is none

Hi, I'm processing a bunch of images with img2table but some of them hit the error below. The error seems to occur on small images like below which is a silly example but I was wondering if there could be a fix for this error? Thanks!

Hi, this error is on my end when no text is detected by the OCR.
I just patched it on the main branch and will include it in the next release.


FlexTable example does not work

Hi, FlexTable example doesn't work
And why have these widgets not moved to druid repo? are they not complete?
the trait bound `Align<_>: druid::widget::widget::Widget<_>` is not satisfied
the following other types implement trait `druid::widget::widget::Widget<T>`:
  <AdvancedSlider as druid::widget::widget::Widget<f64>>
  <Box<(dyn PrismWidget<T> + 'static)> as druid::widget::widget::Widget<T>>
  <Box<(dyn druid::widget::widget::Widget<T> + 'static)> as druid::widget::widget::Widget<T>>
  <Canvas<T> as druid::widget::widget::Widget<T>>
  <CanvasWrap<W, T, F> as druid::widget::widget::Widget<T>>
  <ComputedWidget<T, U> as druid::widget::widget::Widget<T>>
  <DisablePrismWrap<W, U, P> as druid::widget::widget::Widget<T>>
  <DynamicSizedBox<T> as druid::widget::widget::Widget<T>>

the trait bound `Label<_>: druid::widget::widget::Widget<_>` is not satisfied
the following other types implement trait `druid::widget::widget::Widget<T>`:
  <AdvancedSlider as druid::widget::widget::Widget<f64>>
  <Box<(dyn PrismWidget<T> + 'static)> as druid::widget::widget::Widget<T>>
  <Box<(dyn druid::widget::widget::Widget<T> + 'static)> as druid::widget::widget::Widget<T>>
  <Canvas<T> as druid::widget::widget::Widget<T>>
  <CanvasWrap<W, T, F> as druid::widget::widget::Widget<T>>
  <ComputedWidget<T, U> as druid::widget::widget::Widget<T>>
  <DisablePrismWrap<W, U, P> as druid::widget::widget::Widget<T>>
  <DynamicSizedBox<T> as druid::widget::widget::Widget<T>>

Hey @graymind75,
In regards to your first question, I suspect that you are pulling in two different versions of druid. If you do cargo tree -i druid you'll see if there is more than 1 version, and what other crates are pulling it in as a dependency. We pin druid-widget-nursery to a specific version of the master branch (now called main IIRC), and so you need to do the same in your project. Otherwise there are two druid::widget::widget::Widget traits (which is not desirable and at least to me is very confusing).
For your second question, the reason that druid hasn't grown much recently is that more focus has been going into piet-gpu and the xilem architecture. These are both hopefully going to be the foundation for a better druid in the future, but they are still in an experimental/development state, where the design space is being explored and breaking changes happening regularly. For more information read this great blog post: https://raphlinus.github.io/rust/gui/2022/07/15/next-dozen-guis.html
Hope this is helpful.


Add new challenge type: requireUserCountWithinMovesAndMinWaitTime

Apparently the requireUserCountWithinMoves() challenges can always be completed with some ridiculous solutions. This could be ok for one or two challenges, but the most interesting challenge would probably be one that requires both efficiency and speed - requiring a number of transported passengers with a limited number of moves while not letting them wait long.

Also add challenge requireUserCountWithinMovesAndTime due to https://github.com/magwo/elevatorsaga/issues/66


Edit Distance in python

Rules:-
1)Follow me and add star in this repository
2)No c++/c file will be accepted
3)Comment for assign this task to you
Follow all rules you will get hacktoberfest-accepted tag

Hi could I work on the following issue

@shubham7668 could I work on the following issue ?

Done @rising-star2712
Lets contribute
Make sure to follow all rules for hacktoberfest-accepted tag


Create digit_dp_cpp

A very nice example to get command over digit dp

This is a web application so adding C++ code doesn't do much.
Please convert your code to javascript and create the needed files:

algorithm/dp/<your-algorithm>/desc.json, containing algorithm information such as description
folder with the implementation name, usually algorithm/dp/<your-algorithm>/basic
and code.js and data.js files inside algorithm/dp/<your-algorithm>/basic

Take a look at other algorithms for examples.


Copy files from src to dst using python

Title/Heading
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generates no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

PR needs to fix the following issues:

Template unfilled (Title and description)
Each task need to have their own PR.
Not followed contribution Guidelines
Use of external library not needed, use native file handling in python

you were assigned this issue for the Python directory but you are pushing a palindrome file with this. Please make fresh PR with Template filled and required file only

sure will do


⚠ Panel - Hydeos has degraded performance

In c53fa55, Panel - Hydeos (https://pterodactyl.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1900 ms

Resolved: Panel - Hydeos performance has improved in f55927e after 14 minutes.


installation instructions

I can't seem to get this program to work.
i'm stuck at:

Configure gtfs-editor application.conf
cp conf/application.conf.template conf/application.conf

this copies the template to a new application.conf file. Is it necessary to configure or can I use the default options. Do i need a database? Do I need a webserver? Do I need to place gtfs-editor into www folder of apache?

Install dependencies
[path to play1.2.5]/play dependencies

I have no idea what to put here. I just get this error: play FAIL formats: can't open input file `dependencies': No such file or directory
what is pretty logical since "dependencies" is not a jvm application.

No idea...
can someone please help me getting this launched ?
using ubuntu 16.04 LTS

I am also looking for these answers.


CliRunner prompt input echos twice on Python 2

import click
from click.testing import CliRunner

@click.command()
def cli():
    prompt("value")
    click.echo("done")

runner = CliRunner(echo_stdin=True)
result = runner.invoke(cli, input="test")
print(result.output)

Python 3 output (correct):
value: test
done

Python 2 output:
value: testtest
done

I went back and this appears to be the case in every version since 1.x. The issue seems to be with the visible_input function in CliRunner.isolation. It uses input after that has been wrapped in EchoingStdin. It appears to work in Python 3 due to some weird lookup behavior, but that appears to be an accident.  There are no tests for echo_stdin and prompt at the same time.
It didn't show up in the docs because the docs runner overrode isolation with different code that had visible_input pull from the original input and only replaced sys.stdin with the EchoingStdin wrapper, rather than both with the wrapper. While porting the docs runner to pallets-sphinx-themes, I removed that code in favor of simpler subclassing, and noticed that the docs had double input, which led me to discover the full issue.

Taking a look at this

So this feature has proven to be a little more complex than it would seem. The reason behind this bug is that the EchoingStdin was not being set up properly for Python 3 as it's intended functionality of printing anything being read is being overrode by io.TextIOWrapper:
if self.echo_stdin:
    input = EchoingStdin(input, bytes_output)
input = io.TextIOWrapper(input, encoding=self.charset)

So some re-ordering of this can reproduce the double print for 3.7 as well:
input = io.TextIOWrapper(input, encoding=self.charset)
output = io.TextIOWrapper(
    bytes_output, encoding=self.charset)
if self.echo_stdin:
    input = EchoingStdin(input, output)
sys.stdout = output

Then removing the forced print will lead to what I would believe to the the expected functionality:
def visible_input(prompt=None):
    sys.stdout.write(prompt or '')
    val = input.readline().rstrip('\r\n')
-     sys.stdout.write(val + '\n')
+     sys.stdout.write('\n')
    sys.stdout.flush()
    return val

With echo_stdin=True both Python 2 and Python 3 output:
value: test
done

and echo_stdin=False outputs:
value:
done

Assuming this is the desired functionality, this causes some breaks in unit tests, which to be honest are a little confusing as to what their intentions are based on how they are structured. Further investigation leads to a question of the actual purpose to the echo_stdin flag is, as there are also a hide_input and prompt options available to commands.

This can be closed, support for py2 was removed.


Does not work with flutter 1.7

I verified a couple of time. Everything works for flutter 1.5 but in 1.7 i keep getting the "boolean expression must not be null" whenever make a parse server request. if i switch back to flutter 1.5, it works. I tried both the 1.0.21 an 1.0.22 versions.

I am using in version 1.7. Flutter and lib version 1.0.21 and 1.0.23 with no problem.
How are you starting Parse in code?
An await is required.
See the sample code.
https://github.com/phillwiggins/flutter_parse_sdk/blob/master/example/lib/main.dart

you are right. i didn't await for parse init. Thanks a million!!


Unhandled Exception: LateInitializationError: Field '_receivePort@1710301534' has not been initialized.

Platform:

[ ]  Dart
[X]  Flutter Android or iOS
[ ]  Flutter Web

IDE:

[ ]  VSCode
[X]  IntelliJ/AS
[ ]  XCode
[ ]  Other, which one?

split-debug-info and obfuscate (Flutter Android or iOS) or CanvasKit (Flutter Web):

[ ]  Enabled
[X]  Disabled

Platform installed with:

[X] pub.dev
[ ] GitHub

Output of the command flutter doctor -v below:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.2.2, on Microsoft Windows [Version 10.0.19042.867], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[!] Android Studio (not installed) // OBS: Android Studio is installed but not recognized by this command for some reason
[√] IntelliJ IDEA Ultimate Edition (version 2021.1)
[√] Connected device (3 available)

The version of the SDK (See pubspec.lock):
5.1.0

I have the following issue:
When I try to reinitizalize the Sdk with different parameters, I get the following error:
E/flutter (13798): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: LateInitializationError: Field '_receivePort@1710301534' has not been initialized.
E/flutter (13798): #0      IsolateErrorIntegration._receivePort (package:sentry/src/isolate_error_integration.dart)
E/flutter (13798): #1      IsolateErrorIntegration.close (package:sentry/src/isolate_error_integration.dart:26:5)
E/flutter (13798): #2      Hub.close (package:sentry/src/hub.dart:234:27)
E/flutter (13798): #3      Sentry._init (package:sentry/src/sentry.dart:104:17)
E/flutter (13798): <asynchronous suspension>
E/flutter (13798): #4      Sentry.init (package:sentry/src/sentry.dart:55:5)
E/flutter (13798): <asynchronous suspension>
E/flutter (13798): #5      SentryService._initializeClient (package:itini/core/services/sentry_service.dart:139:5)
E/flutter (13798): <asynchronous suspension>

Steps to reproduce:

Initialize SDK;
Close SDK;
Initialize SDK;

Actual result:
LateInitializationError is thrown.
Expected result:
No errors thrown.

Thanks for filing this issue.
What exactly is your use case that you need to init, close and init the SDK again?

In my case, I didn't find a proper way to edit the environment property without reinitializing the SDK (please, tell me if there is an easier way to do so). But once I worked on a project that we had two Sentry projects: one for production and one for staging. So, I think in the last case a reinitialization to change the DSN would be necessary.

@denrase Does this happen in the Sentry app, too?
I imagine it should, if you're toggling the SDK in the settings.
I'm also kinda suspect that one shouldn't use the appRunner callback (and thus don't use the IsolateIntegration) if you're toggling the SDK.

@ueman I'm not using the appRunner parameter to run my application.

@ueman I believe NativeSdkIntegration can happen the same if close throws, _options might not be inited yet.
can you check that too? thanks

@ueman Just tied to repoduce this in the Sentry Mobile app by disableing and enabling multiple times form the Settings. Not seeing this error in the logs so far.
Close
D/Sentry  ( 5473): UncaughtExceptionHandlerIntegration removed.
D/sentry-native( 5473): shutting down backend
D/Sentry  ( 5473): NdkIntegration removed.
D/Sentry  ( 5473): EnvelopeFileObserverIntegration removed.
D/Sentry  ( 5473): AppLifecycleIntegration removed.
D/Sentry  ( 5473): ActivityLifecycleIntegration removed.
D/Sentry  ( 5473): AppComponentsBreadcrumbsIntegration removed.
D/Sentry  ( 5473): SystemEventsBreadcrumbsIntegration remove.
D/Sentry  ( 5473): TempSensorBreadcrumbsIntegration removed.
I/Sentry  ( 5473): Closing SentryClient.
D/Sentry  ( 5473): Shutting down

Re-Enable
D/FlutterSecureStoragePl( 5473): StorageCipher already initialized
I/Sentry  ( 5473): Initializing SDK with DSN<EMAIL_ADDRESS>I/Sentry  ( 5473): GlobalHubMode: 'true'
I/Sentry  ( 5473): sentry-debug-meta.properties file was not found.
D/Sentry  ( 5473): UncaughtExceptionHandlerIntegration enabled: true
D/Sentry  ( 5473): default UncaughtExceptionHandler class='com.android.internal.os.RuntimeInit$KillApplicationHandler'
D/Sentry  ( 5473): UncaughtExceptionHandlerIntegration installed.
D/Sentry  ( 5473): ShutdownHookIntegration installed.
D/Sentry  ( 5473): SendCachedEventFireAndForgetIntegration installed.
D/Sentry  ( 5473): NdkIntegration enabled: true
D/Sentry  ( 5473): Started processing cached files from /data/user/0/io.sentry.mobile.app/cache/sentry
D/Sentry  ( 5473): Processing dir. /data/user/0/io.sentry.mobile.app/cache/sentry
I/sentry-native( 5473): using database path "/data/data/io.sentry.mobile.app/cache/sentry/.sentry-native"
D/Sentry  ( 5473): Processing 0 items from cache dir /data/user/0/io.sentry.mobile.app/cache/sentry
D/Sentry  ( 5473): File /data/user/0/io.sentry.mobile.app/cache/sentry/outbox is not a File.
D/Sentry  ( 5473): File /data/user/0/io.sentry.mobile.app/cache/sentry/.sentry-native is not a File.
D/Sentry  ( 5473): Processing file: /data/user/0/io.sentry.mobile.app/cache/sentry/session.json
D/sentry-native( 5473): starting backend
D/Sentry  ( 5473): File '/data/user/0/io.sentry.mobile.app/cache/sentry/session.json' doesn't match extension expected.
D/Sentry  ( 5473): NdkIntegration installed.
D/Sentry  ( 5473): Finished processing cached files from /data/user/0/io.sentry.mobile.app/cache/sentry
D/Sentry  ( 5473): Registering EnvelopeFileObserverIntegration for path: /data/user/0/io.sentry.mobile.app/cache/sentry/outbox
D/Sentry  ( 5473): EnvelopeFileObserverIntegration installed.
D/Sentry  ( 5473): SendCachedEventFireAndForgetIntegration installed.
D/Sentry  ( 5473): AnrIntegration enabled: false
D/Sentry  ( 5473): enableSessionTracking enabled: true
D/Sentry  ( 5473): Started processing cached files from /data/user/0/io.sentry.mobile.app/cache/sentry/outbox
D/Sentry  ( 5473): enableAppLifecycleBreadcrumbs enabled: true
D/Sentry  ( 5473): Processing dir. /data/user/0/io.sentry.mobile.app/cache/sentry/outbox
D/Sentry  ( 5473): SU isn't found on this Device.
D/Sentry  ( 5473): Processing 0 items from cache dir /data/user/0/io.sentry.mobile.app/cache/sentry/outbox
D/Sentry  ( 5473): Finished processing cached files from /data/user/0/io.sentry.mobile.app/cache/sentry/outbox
D/Sentry  ( 5473): AppLifecycleIntegration installed.
D/Sentry  ( 5473): ActivityLifecycleIntegration enabled: true
D/Sentry  ( 5473): ActivityLifecycleIntegration installed.
D/Sentry  ( 5473): AppComponentsBreadcrumbsIntegration enabled: true
D/Sentry  ( 5473): AppComponentsBreadcrumbsIntegration installed.
D/Sentry  ( 5473): SystemEventsBreadcrumbsIntegration enabled: true
W/Sentry  ( 5473): Current session is not ended, we'd need to end it.
D/Sentry  ( 5473): SystemEventsBreadcrumbsIntegration installed.
D/Sentry  ( 5473): enableSystemEventsBreadcrumbs enabled: true
W/pool-9-thread-1( 5473): type=1400 audit(0.0:43): avc: denied { read } for name="version" dev="proc" ino=4026532035 scontext=u:r:untrusted_app:s0:c156,c256,c512,c768 tcontext=u:object_r:proc_version:s0 tclass=file permissive=0 app=io.sentry.mobile.app
D/Sentry  ( 5473): TempSensorBreadcrumbsIntegration installed.
D/Sentry  ( 5473): enableSystemEventBreadcrumbs enabled: true
D/Sentry  ( 5473): Serializing object: {"sid":"2c5fd4e0-3d21-420d-8a18-97bfeb1cf099","did":"e8f915b2-e54e-4dba-96dd-26b03e76e9b3","started":"2021-06-29T12:47:26.918Z","status":"exited","seq":1624970870058,"duration":23.14,"timestamp":"2021-06-29T12:47:50.058Z","attrs":{"release":"io.sentry.mobile.app@1.0.2+22","environment":"debug"}}
D/Sentry  ( 5473): Serializing object: {"sid":"0c9710b6-c793-47dd-8664-a0740e07a208","did":"e8f915b2-e54e-4dba-96dd-26b03e76e9b3","init":true,"started":"2021-06-29T12:47:50.050Z","status":"ok","timestamp":"2021-06-29T12:47:50.050Z","attrs":{"release":"io.sentry.mobile.app@1.0.2+22","environment":"debug"}}
[sentry] [debug] release<EMAIL_ADDRESS>D/Sentry  ( 5473): Serializing object: {"sid":"0c9710b6-c793-47dd-8664-a0740e07a208","did":"e8f915b2-e54e-4dba-96dd-26b03e76e9b3","init":true,"started":"2021-06-29T12:47:50.050Z","status":"ok","timestamp":"2021-06-29T12:47:50.050Z","attrs":{"release":"io.sentry.mobile.app@1.0.2+22","environment":"debug"}}
D/Sentry  ( 5473): Adding Envelope to offline storage: /data/user/0/io.sentry.mobile.app/cache/sentry/6f354802-660f-4a64-a1e2-d15b1da2383c.envelope
D/Sentry  ( 5473): Envelope sent successfully.
D/Sentry  ( 5473): Discarding envelope from cache: /data/user/0/io.sentry.mobile.app/cache/sentry/6f354802-660f-4a64-a1e2-d15b1da2383c.envelope
D/Sentry  ( 5473): Envelope flushed

Re-Close
D/Sentry  ( 5473): UncaughtExceptionHandlerIntegration removed.
D/sentry-native( 5473): shutting down backend
D/Sentry  ( 5473): NdkIntegration removed.
D/Sentry  ( 5473): EnvelopeFileObserverIntegration removed.
D/Sentry  ( 5473): AppLifecycleIntegration removed.
D/Sentry  ( 5473): ActivityLifecycleIntegration removed.
D/Sentry  ( 5473): AppComponentsBreadcrumbsIntegration removed.
D/Sentry  ( 5473): SystemEventsBreadcrumbsIntegration remove.
D/Sentry  ( 5473): TempSensorBreadcrumbsIntegration removed.
I/Sentry  ( 5473): Closing SentryClient.
D/Sentry  ( 5473): Shutting down


Closes #212 | Implement dataloaders for Thai_Alpaca

Please name your PR after the issue it closes. You can use the following line: "Closes #ISSUE-NUMBER" where you replace the ISSUE-NUMBER with the one corresponding to your dataset.
Closes #212.
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Hi @yana-xuyan, considering that instruction tuning is conducted by using an instruction and a response, probably we can join the instruction and the input in the dataset into text_1 in the t2t schema (used for Tasks.INSTRUCTION_TUNING)? (As suggested by @sabilmakbar at #242.)

Example 1: text_1 = f"Context: {input}\n\n{instruction}" and text_2 = output.
Example 2: If input is null, then text_1 = instruction and text_2 = output.

What do you think? @jen-santoso @yana-xuyan

I think this sounds good. Waiting for the modifs @yana-xuyan

Hi, but without “f”, {_DATASETNAME}_source will get errors. Do you mean this way of constructing names is not allowed?
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Jennifer Santoso @.>
发送时间: Saturday, January 6, 2024 1:13:28 PM
收件人: SEACrowd/seacrowd-datahub @.>
抄送: Yan XU @.>; Mention @.>
主题: Re: [SEACrowd/seacrowd-datahub] Closes #212 | Implement dataloaders for Thai_Alpaca (PR #248)
@jen-santoso commented on this pull request.

In seacrowd/sea_datasets/thai_alpaca/thai_alpaca.pyhttps://github.com/SEACrowd/seacrowd-datahub/pull/248#discussion_r1443628296:

+_SEACROWD_VERSION = "1.0.0"

+class ThaiDatabricksDollyDataset(datasets.GeneratorBasedBuilder):

"""Thai Alpaca Dataset"""

SOURCE_VERSION = datasets.Version(_SOURCE_VERSION)
SEACROWD_VERSION = datasets.Version(_SEACROWD_VERSION)

SEACROWD_SCHEMA_NAME = "qa"

BUILDER_CONFIGS = [

   SEACrowdConfig(

       name=f"{_DATASETNAME}_source",

       version=SOURCE_VERSION,

       description=f"Thai-Alpaca source schema",

remove the starting f (from make check_file)
The same goes for line 51
―
Reply to this email directly, view it on GitHubhttps://github.com/SEACrowd/seacrowd-datahub/pull/248#pullrequestreview-1807246162, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJGALO6RGZHB4JIYUDO7A73YNDMPRAVCNFSM6AAAAABBFFXDGOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMBXGI2DMMJWGI.
You are receiving this because you were mentioned.Message ID: @.***>

Hi, but without “f”, {_DATASETNAME}_source will get errors. Do you mean this way of constructing names is not allowed?

I think it's on the description field which you don't use any variable reference

Hi, but without “f”, {_DATASETNAME}_source will get errors. Do you mean this way of constructing names is not allowed? 获取 Outlook for iOShttps://aka.ms/o0ukef
________________________________ 发件人: Jennifer Santoso @.> 发送时间: Saturday, January 6, 2024 1:13:28 PM 收件人: SEACrowd/seacrowd-datahub @.> 抄送: Yan XU @.>; Mention @.> 主题: Re: [SEACrowd/seacrowd-datahub] Closes #212 | Implement dataloaders for Thai_Alpaca (PR #248) @jen-santoso commented on this pull request.
________________________________ In seacrowd/sea_datasets/thai_alpaca/thai_alpaca.py<#248 (comment)>:
+_SEACROWD_VERSION = "1.0.0"

+class ThaiDatabricksDollyDataset(datasets.GeneratorBasedBuilder): + """Thai Alpaca Dataset""" + + SOURCE_VERSION = datasets.Version(_SOURCE_VERSION) + SEACROWD_VERSION = datasets.Version(_SEACROWD_VERSION) + + SEACROWD_SCHEMA_NAME = "qa" + + BUILDER_CONFIGS = [ + SEACrowdConfig( + name=f"{_DATASETNAME}_source", + version=SOURCE_VERSION, + description=f"Thai-Alpaca source schema", remove the starting f (from make check_file) The same goes for line 51 ― Reply to this email directly, view it on GitHub<#248 (review)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJGALO6RGZHB4JIYUDO7A73YNDMPRAVCNFSM6AAAAABBFFXDGOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMBXGI2DMMJWGI. You are receiving this because you were mentioned.Message ID: @.***>

Sorry for the ambiguous note.
Only on the description line (lines 44 and 51).
The ones on the name (lines 42, 49) and schema (line 52) are OK

Hi, thank you for the changes, tested and LGTM!


Provide context to SimpleHttpOperator#response_check

Hi,
I have a sequential DAG defined as follows:

An HTTP Operation performs a POST against some endpoint which returns an UUID

POST /process
34ac3fe0-5798-11e5-885d-feff819cdc9f

An HttpSensor checks some other endpoint using the previous obtained UUID until it reaches a certain state (Defined in the response body)

GET /process/34ac3fe0-5798-11e5-885d-feff819cdc9f
OK

To provide the UUID to the second step I'm using the XCom utility over Jinja. The HttpSensor is configured as follows:
getStatus = HttpSensor(
    ...
    endpoint="process/{{ ti.xcom_pull(key='uuid', task_ids='getUuid') }}",
    ...
)

The tricky part comes when I try to provide XCom with the UUID value resulting from the first POST operation. The only place (as far as I know) which has access to the response object outside the SimpleHttpOperator is the response_check function (Configured by parameter when creating the Operator as a lambda or a defined function). Since the response_check function only receives the response object when its called during execute I have no reference to the Task Instance to call TaskInstance#xcom_push. I have modified the SimpleHttpOperator into ExtendedHttpOperator to provide context as follows:
def execute(self, context):
    ...
    if self.response_check:
        if not self.response_check(response, context):
            raise AirflowException("Response check returned False.")

With this modification I can define a check function as follows:
def check(response, context):
    processId = response.content
    context['ti'].xcom_push(key='uuid', value=processId)
    return True

...

startProcess = ExtendedHttpOperator(
    task_id='getUuid',
    method='POST',
    endpoint='process',
    response_check=check,
    ...
)

The DAG works as expected, sharing the POST UUID information with the following HttpSensor node in the graph. I would like to know if this is the right way to go when sharing information between DAG nodes and, if so, it would be nice to improve the SimpleHttpOperator to provide context to the response_check function just as PythonOperator does (Based in a provide_context parameter).
Please note that these are my very first steps with Airflow so maybe I'm doing something really wrong. Thanks in advance!

This is still fairly unmapped territories, so operators or function arguments you might need or best practices have not been defined yet.
One easy way to leverage more out of the xcom framework would be for SimpleHttpOperator to return the response's data, which will create an xcom entry (any operator's execute return value gets "xcommed"). This would become available wherever SimpleHttpOperator is used.
It also seems like posting + sensing is a common use case and it's nicer when it's executed atomically as one task instead of 2. So there could be a HttpPostThenSense of some kind, or you can do this in a very simple PythonOperator as well.
Now I'm guessing you are interfacing with a specific system that exposes a web API. So say if that system was something called "AmazonBackups", you could write a specific AmazonBackupOperator with the perfect interface for it.

^ +1 to HttpPostThenSense. When dealing with REST APIs, the POST is often an async trigger, and I need to confirm whether the job completed successfully or failed / killed, by continuously sensing with a GET.
We've defined it as a Plugin operator, for our particular pattern of POST/GETs, but I believe a generic solution will be very helpful too. xcom is excellent for doing ad-hoc communication, but in this case it's more convenient to define and treat Post+Sense as a single operator.

HttpPostThenSense might be harder to use than writing your own operator actually is. I'm happy reviewing a PR that does this though.


Bazel test flake //test/core/end2end:h2_compress_test@large_metadata@poller=poll-cv

Seen at #16161
exec ${PAGER:-/usr/bin/less} "$0" || exit 1
Executing tests from //test/core/end2end:h2_compress_test@large_metadata@poller=poll-cv
-----------------------------------------------------------------------------
D0801 19:58:05.760473086      18 test_config.cc:391]         test slowdown factor: sanitizer=3, fixture=1, poller=1, total=3
E0801 19:58:05.767974480      18 cq_verifier.cc:273]         cq returned unexpected event: OP_COMPLETE: tag:0x1 OK
E0801 19:58:05.768142227      18 cq_verifier.cc:280]         expected tags:
tag(101) GRPC_OP_COMPLETE success=1 test/core/end2end/tests/large_metadata.cc:170
*******************************
Caught signal SIGABRT
test/core/end2end/h2_compress_test(backtrace+0x61)[0x502241]
test/core/end2end/h2_compress_test[0xd6a604]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x11390)[0x7f6ff8fbd390]
/lib/x86_64-linux-gnu/libc.so.6(gsignal+0x38)[0x7f6ff7f6a428]
/lib/x86_64-linux-gnu/libc.so.6(abort+0x16a)[0x7f6ff7f6c02a]
test/core/end2end/h2_compress_test(_Z9cq_verifyP11cq_verifier+0x77e)[0x6c716e]
test/core/end2end/h2_compress_test[0x6555ce]
test/core/end2end/h2_compress_test(_Z14large_metadata24grpc_end2end_test_config+0x16a)[0x653dba]
test/core/end2end/h2_compress_test(_Z18grpc_end2end_testsiPPc24grpc_end2end_test_config+0x460a)[0x58fc1a]
test/core/end2end/h2_compress_test(main+0x18e)[0x58acee]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0)[0x7f6ff7f55830]
test/core/end2end/h2_compress_test(_start+0x29)[0x4b5ce9]
Aborted (core dumped)

poll-cv was removed in #18298 (the problem itself seems to be what was diagnosed as poll-cv specific by @yashykt in #18272)


I Cant train

Hello, I put some of my photos in data/cityscapes/images/train, data/cityscapes/images/val, data/cityscapes/images/train, data/cityscapes/label/train_labels and data/cityscapes/val_labels.
(Absolute path to png = D:\GraduateWork\PIDNet\data\cityscapes\images\train\0010.png,
to its label = D:\GraduateWork\PIDNet\data\cityscapes\labels\train_labels\0010.png, and so on)

Then, i did some lst files: train.lst and val.lst
But when i try to start the train, i get an error: NotImplementedError: Got <class 'NoneType'>, but expected numpy array or torch tensor.

As far as i understand, the error is in the files .lst, im most likely pointing the wrong path's. But I don't understand what exactly I'm doing wrong.
I tried full paths, for example 1 row of train.lst file:
D:\GraduateWork\PIDNet\data\cityscapes\images\train\0010.png    D:\GraduateWork\PIDNet\data\cityscapes\labels\train_labels\0010.png

tried paths, starts from a folder data/:
cityscapes/images/train/0010.png    cityscapes/labels/train_labels/0010.png
Or including it:
data/cityscapes/images/train/0010.png    data/cityscapes/labels/train_labels/0010.png
Tell me please what am I doing wrong? How to correctly write paths to images in .lst files?
PS im using pidnet_large_cityscapes.yaml , everything inside is right:
DATASET:
DATASET: cityscapes
ROOT: data/
TEST_SET: 'list/cityscapes/val.lst'
TRAIN_SET: 'list/cityscapes/train.lst'
NUM_CLASSES: 19

I think you should not include the 'DATASET.ROOT' and 'DATASET.DATASET' in the list of paths. So in your example, it should be like this:
images\train\0010.png     labels\train_labels|0010.png
I hope it helps


[Feature] Add Makefile Support for gcc for the project structure

Prepare the base makefine for Gcc support

Make ma makefile module for easy configuration
Allow easy confuguring and extending
Keep in mind you'll need to add additional features like doxygen supoort linting support testing support etc.


Support configuring the parser w/ relevant server settings?

My understanding is that some PG server options can change how parsing works (e.g., allow_nulls and standard_conforming_strings). Does libpg_query support configuring the parser with these settings, to match the server? Can/should it?

@ethanresnick Today there is no mechanism to modify these settings, mostly since there are very limited use cases for setting them.
Do you have a reason to want to set them / what's the use case?

Do you have a reason to want to set them

No, I was mostly just curious; my thought was that a user of this library who did have those settings applied on their server would likely want to match them in libpq_query, as it seems like having a guarantee of consistent parsing between the server and wherever libpq_query is running could be an important security property, and part of the reason people would want to use this library

+1 for standard_conforming_strings. If you want to parse like pg, you need to be able to match pg's alternative parsing modes.

@lfittl It's been quite a while since I worked with this library, but, at the time I opened the issue, I was using my own fork of a then-current Javascript wrapper, and calling its parseQuery/parseQueryAsync methods. I was not using query fingerprinting.

I'm using it from Node — import { parse } from 'pgsql-parser'; — to parse out separate SQL statements in a text block.

Thank you!


[HW] Enrique A Perez Delgado

Name: Enrique A Perez Delgado
Email<EMAIL_ADDRESS>Linkedin Profile: My LinkedIn Profile
Attach the homework screenshots below for both step II and step III:

Well done, Enrique! Thanks!

Here's your badge -> https://api.badgr.io/public/assertions/WC9ohuglRquQTSZKEr6d5w?identity__email=eaperezdelgado%40gmail.com


Add ignoreNonDOM to jsx-a11y/no-autofocus

What is to be discussed?
At Styleguide we implement components with the autoFocus feature without using the native HTML attribute. So it's safe to ignore this rule for components created by developers.
Describe the solution you'd like
Add this argument to the de jsx-a11y/no-autofocus rule.
How we do this today?
We are ignoring this rule to be able to use the autoFocus property of our components.

I really would an opinion of an accessibility expert, since using autofocus may reduce the actions needed to do something but I'm not sure about which impacts this has on accessibility :/

Are you ignoring the rule in a per line basis or did you add an override in the .eslintrc?
Nevertheless, I think linting only native DOM elements should be enough, since a component could have any kind of behavior related to a autofocus prop (which is something debatable, of course).
More opinions on this matter would be great!

@kaisermann I only saw ignoring per line but it's very common.
@rerissondaniel About accessibility, autofocusing an element being a bad or good practice really depends on the context you are using it. For example, using autoFocus at a form may be confusing but using in a Modal to focus on the content instead of the close button(the first element inside it) can be good. You can see the example of the modal here https://www.w3.org/TR/wai-aria-practices/#keyboard-interaction-7

@rerissondaniel About accessibility, autofocusing an element being a bad or good practice really depends on the context you are using it. For example, using autoFocus in a Form may be confusing but using in a Modal to focus on the content instead of the close button(the first element inside it) can be good. You can see the example of the modal here https://www.w3.org/TR/wai-aria-practices/#keyboard-interaction-7

That's nice, thank you!

I recently implemented a property in the Styleguide <Alert> to allow the developer to choose to enable autofocus when the element appears in the screen.
I named it focusOnOpen since autoFocus was not allowed.
During this process I also implemented another property that manipulated the focus when the alert closed, and it turned out yield in a terrible accessibility experience.
It is really hard to tell when we should or shouldn't use autoFocus, so I think it is good to have that eslint rule there to prevent us from forgetting it.
Maybe we do want to ignore it everywhere we use, so we are explicitly stating we know what we are doing.

@felippenardi What do you think about changing it to be a warning instead of an error?


IP-5633/add eventhub + eventhubnamespace resource. add previously uncommited …

…containerregistrytask resource.

Could you remove me from the CODEOWNERS file next time you edit? :)