#save is post(ing) NOT patch(ing)

tl;dr #save is doing a POST transaction NOT the PATCH transaction, and therefore is creating new content instead of saving existing updates.
This is generally not what most people think of when saving an existing object:
.2.4 :028 > c = BlockScore::Candidate.all.first
 => #<BlockScore::Candidate:0x007f9a4d354758 JSON:{
  "object": "candidate",
  "id": "56b913fc3565650003000d1a",
  "created_at":<PHONE_NUMBER>,
  "updated_at":<PHONE_NUMBER>,
  "livemode": false,
  "note": null,
  "ssn": "0000",
  "passport": null,
  "date_of_birth": "1992-03-05",
  "name_first": "Joany",
  "name_middle": null,
  "name_last": "Dooley",
  "address_street1": "6251 Consuelo Drive",
  "address_street2": null,
  "address_city": "New Keely",
  "address_subdivision": null,
  "address_postal_code": null,
  "address_country_code": "MR",
  "deleted": true
}>
2.2.4 :029 > c.name_middle = 'Jay'
 => "Jay"
2.2.4 :030 > c
 => #<BlockScore::Candidate:0x007f9a4d354758 JSON:{
  "object": "candidate",
  "id": "56b913fc3565650003000d1a",
  "created_at":<PHONE_NUMBER>,
  "updated_at":<PHONE_NUMBER>,
  "livemode": false,
  "note": null,
  "ssn": "0000",
  "passport": null,
  "date_of_birth": "1992-03-05",
  "name_first": "Joany",
  "name_middle": "Jay",
  "name_last": "Dooley",
  "address_street1": "6251 Consuelo Drive",
  "address_street2": null,
  "address_city": "New Keely",
  "address_subdivision": null,
  "address_postal_code": null,
  "address_country_code": "MR",
  "deleted": true
}>
2.2.4 :031 > c.save
 => true
2.2.4 :032 > c
 => #<BlockScore::Candidate:0x007f9a4d354758 JSON:{
  "object": "candidate",
  "id": "56ba2d2963616100030006e6", #<-- note: new ID
  "created_at":<PHONE_NUMBER>,
  "updated_at":<PHONE_NUMBER>,
  "livemode": false,
  "note": null,
  "ssn": "0000",
  "passport": null,
  "date_of_birth": "1992-03-05",
  "name_first": "Joany",
  "name_middle": "Jay",
  "name_last": "Dooley",
  "address_street1": "6251 Consuelo Drive",
  "address_street2": null,
  "address_city": "New Keely",
  "address_subdivision": null,
  "address_postal_code": null,
  "address_country_code": "MR"
}>

The effect of this is multifold:

The object being saved is not altered, so the changed data is NOT associated with the object ID changed
If save occurs for a NEW object everything appears fine. A subsequence change of the same Ruby Object will create a duplicate (assuming nothing altered in the interim)
If the object being saved is for example a Person with existing Question Sets, then the original stored object still has those questions sets and scores and the new object created has no question sets (it is new)
When invoked through the Member class, this causes duplicate question set entries to be generated is the member object is saved multiple times (Member.new(person, question_set))
If the client/customer is saving the referenced objects API Id in their own system for later reference (like sotring person.id after creation), and then they retrieve alter and save that via the API, the saved content is effectively lost to them, unless they retrieve the new ID (see quetion set info above for side effect).

My current plan is to fix this by...

Borrowing from the #update logic to do a patch
Only patch changed attributes that are updateable (stolen from same logic)
which requires (to be proper) a before and after view of the objects attributes (captured at retrieval - hash diff -> patch)

One concern is that setters and getters MUST update the attributes (I suspect this is already in place, but must be confirmed). If the attributes are readonly after retrieval this would make it interesting.
Am (VERY?) open to other suggestions.

As an aside I have reviewed the PHP and Python implementations which do not appear at present to be vulnerable to the same behavior. Python was actually manually tested, and PHP uses a unsaved attribute array to maintain the list of dirty attributes.

It appears this problem IS in fact a bug, but should be relatively easy to remedy on the assumption that Person and Company are not immutable by definition. They are not technically immutable, but there may be some reason (@backus?) that they should be considered immutable for audit trail purposes.

Yes, they should be considered immutable for audit trail purposes

@alainmeier  In that case then since they are not presently immutable, I will fix the #save issue removing the capability to alter them.

Sounds good :)

@alainmeier does it make any business sense for Question Sets to be alterable (aka #save able)?

Nope, doesn't make sense!

So from a business viewpoint .save should only be visible when editing candidates (matches the API docs). For all other objects the #save method should not be accessible publicly.

Wait hold on - I completely forgot about the pattern of:
c = Company.new(company_params)
c.save

Are you including this pattern in your understanding of #save?

@alainmeier  In truth that should be a one time event. However if you save an already #saved Company today you get ANOTHER Company created. Which is why save is not a truly good name for that. **In the API docs there is no option to do what you described, however it does work.
**
My question is what should happen when you save a persisted item. Adding to that confusion (not relevant to Person and Company which cannot be deleted) if you #save a deleted item then you end up with a new instance with same values but now no longer in a deleted state - a dup and undelete effect.

From a process concern, item #5 in my first comment above is relevant.

It would be simpler, however a regression, to eliminate #save fro Person and Company. But I stress it is a regression, since that undocumented API is accessible.

@alainmeier @backus IMHO to enforce a guideline of immutability, then #save should work either 0 times (use create!), a single time (on unpersisted non-deleted data), or generate an appropriate error response. This applies to Person, Company, and question set.
Allowing #save on persisted data (a dup effect), or deleted data (a dup and undelete effect) rather than enforcing creation of a new Ruby Object (perhaps from the hashed attributes of a retrieved company for example) is a very unsafe technical option. It is asking for misunderstanding.

After @backus and I discussed this on HipChat the implementation change is as follows:

#save will return false if deleted: true
#save! will return invalidrequest if deleted: true
#save will behave otherwise as is for Candidate which already allow edit and is documented as such in the API docs => patch

re: Person, Company and QuestionSet

#save and #save! will function as post (like  .create)  if id.nil?
#save will return false if id != nil
#save! will return invalid request if id != nil

This may mean some more special handling in the Member class.
The client effect is that if in the past a client did the following:
p = Person.retrieve('...')
p.name_last = 'xyz'
p.save

... what happened was

a post occurred,
a new person was created with a new id,
and the returned status was true.
The attributes from the original were in the new object, subject to what was altered.

In the proposed fix p.save will return false, no new object will be created. p will still contain the original object subject to change.
One possible variant of that would be to deep freeze the retrieved object (Person, Company or QuestionSet) and/or to not define the setters in Collection on the new object. That would prevent those objects from responding to change since they are immutable once retrieved


Added option to add file-relative include paths

Hey first of all thanks for this project, it has already saved me a lot of time and is probably the best atom c++ linter out there :+1:
I needed the option to give include paths relative to the currently editing file. At the moment one is not generally able to give include paths realtive to the current file if it is part of a bigger project, which can be really annoying.
So I added a check for pathnames starting with a "-" (dont know if this prefix ok, feel free to change it to whatever you like) to expand them relative to the current file.
Have to add that I am a total js noob, so if you want to include this feature (i would also understand if you do not see why it is needed) you should have a look at the code changes. I made only small changes and its working for me, though.

Glad to hear you're enjoying linter-gcc, :+1:  and this looks like a cool little feature. I will definitely merge your request, but I'd just like to play around with it first.

everything alright with it?

Expect to see this in the next release of linter-gcc, very soon. Thanks for your contribution!


"This site can't be reached" using Windows 10 and WSL2

I'm submitting a...
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[ ] Feature request
[ ] Documentation issue or request
[ ] Other: <!-- Please describe: -->

Current behavior

Create new folder.
Add new index.html
Click "Go live!"
Chrome opens http://<IP_ADDRESS>:5500/index.html but I get a This site can’t be reached. <IP_ADDRESS> refused to connect.

Expected behavior
I expect the site to be loaded correctly.
Environment
Browser:

- [x] Chrome (desktop) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: 5.6.0
- Platform:  Windows 10
- Visual Studio Code: 1.44.2

Others
WSL2 (Windows subsystem for Linux) Ubuntu 18.04
With the official WSL extension for VScode

(If I write npx http-server . in the VSCode bash terminal the site is served correctly on 8080.)

(Have also tried uninstalling and reinstalling the extension)

Did some more investigation. If I set "liveServer.settings.useLocalIp": true it works. I guess this has something to do with WSL2.
Any ideas why localIp might work but not else?

Seems to be a problem with WSL2 rather than this extension.
https://github.com/microsoft/WSL/issues/4636

I ran powershell as admin and ran "wsl --shutdown" and it worked for me once. But then it stopped working.


Workspace edit metadata needsConfirmation results in all checkboxes unchecked

The WorkspaceEdit that has metadata with needsConfirmation set to true shows the Refactor Preview which is great but all checkboxes are unchecked. I'd rather have all checkboxes checked.
I went through various git issues where showing Refactor View was first implemented. All screenshots show all changes check-marked. I also saw this one #144893 which also shows all changes check-marked and the request to check/uncheck a group of changes which would be useful.
However, I always have all changes unchecked and wonder how they could be all checked by default?
I'd appreciate a code snippet
For example i have this:
            const edit: VSCode.WorkspaceEdit = new VSCode.WorkspaceEdit();

            const folders = VSCode.workspace.workspaceFolders;
            if (!folders || folders.length === 0) {
                VSCode.window.showInformationMessage('Open a folder first');
                return;
            }
            const newFile = VSCode.Uri.joinPath(folders[0].uri, 'foo.txt');
            edit.createFile(newFile,
                { overwrite: true },
                { label: 'New text', needsConfirmation: true }
            );

            const initialContent = new TextEdit(new Range(new Position(0,0), new Position(0, 0)), "Hello World\nOne more line");
            edit.set(newFile, [initialContent]);

            VSCode.workspace.applyEdit(edit, { isRefactoring: true });

How can i make checkboxes checked for the above?

is:open label:ux

Maybe same as https://github.com/microsoft/vscode/issues/207578?

This is actually designed. Needs confirmation means the extension is unsure about this change and the user must explicitly accept it


exclude all COFF binaries from "symbol_presence" test

Some release 3.0 tests fail on CYGWIN:

symbol_presence
abort

Request is for first one.
Remark: In corrections for issue #13558 looks like that MINGW is excluded by accident (mistake). For good this is right decision.

For protocol details for failed tests:
....
01-test_symbol_presence.t .......... 1/?
# Number of lines in @nm_lines before massaging: 0
# Number of lines in @def_lines before massaging: 5377
# Number of lines in @nm_lines after massaging: 0
# Number of lines in @def_lines after massaging: 5373
# The following symbols are missing in cygcrypto-3.dll:
#   ACCESS_DESCRIPTION_free
#   ....
#   v2i_GENERAL_NAME_ex
not ok 2 - check that there are no missing symbols in cygcrypto-3.dll
01-test_symbol_presence.t .......... 3/? ---------------------------------------
# Number of lines in @nm_lines before massaging: 0
# Number of lines in @def_lines before massaging: 521
# Number of lines in @nm_lines after massaging: 0
# Number of lines in @def_lines after massaging: 517
# The following symbols are missing in cygssl-3.dll:
#   BIO_f_ssl
# ....
#   i2d_SSL_SESSION
not ok 4 - check that there are no missing symbols in cygssl-3.dll
# ------------------------------------------------------------------------------
#   Failed test 'check that there are no missing symbols in cygssl-3.dll'
#   at test/recipes/01-test_symbol_presence.t line 135.
01-test_symbol_presence.t .......... Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/4 subtests
....

...
01-test_abort.t ....................
test/aborttest.c:14: OpenSSL internal error: Voluntary abort
*** starting debugger for pid 20495, tid 2088
../../util/wrap.pl ../../test/aborttest.exe => 0
not ok 1 - Testing that abort is caught correctly
01-test_abort.t .................... 1/? ---------------------------------------
#   Failed test 'Testing that abort is caught correctly'
#   at test/recipes/01-test_abort.t line 16.
#          got: '1'
#     expected: '0'
01-test_abort.t .................... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests
....

The request to exclude "symbol test" is because test look like is oriented to ELF targets and nm utility from binutils.
NM is not suitable to this tests when is used COFF binary. Also test is skipped in 1.1.1 because it search for lib[foo].so  while used is cyg[foo].dll or lib[foo].dll.

Some resutls from nm command:
$ nm cygcrypto-3.dll  | grep ' [BDT] ' | grep ACCESS_DESCRIPTION_free
0000.... T ACCESS_DESCRIPTION_free
$ nm cygcrypto-3.dll  | grep ' [BDT] ' | grep v2i_GENERAL_NAME_ex
000.... T v2i_GENERAL_NAME_ex
$ LANG=C nm -DPg cygcrypto-3.dll
nm: cygcrypto-3.dll: no symbols
$ LANG=C nm -Pg cygcrypto-3.dll  | grep ACCESS_DESCRIPTION_free
ACCESS_DESCRIPTION_free T ...
$ nm cygcrypto-3.dll  | grep ' [BDT] ' | grep ACCESS_DESCRIPTION_free
00000005e03ce020 T ACCESS_DESCRIPTION_free
$ nm cygcrypto-3.dll  | grep ' [BDT] ' | grep v2i_GENERAL_NAME_ex
00000005e03d5230 T v2i_GENERAL_NAME_ex
$ LANG=C nm -DPg cygcrypto-3.dll
nm: cygcrypto-3.dll: no symbols
$ nm -Pg cygcrypto-3.dll |grep ' [BCSDT] '  | wc -l
7308
$ nm cygssl-3.dll | grep ASN1_OCTET_STRING_free
000.... I __imp_ASN1_OCTET_STRING_free
000.... T ASN1_OCTET_STRING_free
As is visible from above commands NM cannot be used for COFF binary.

For protocol results from 1.1.1 branch
../test/recipes/01-test_symbol_presence.t ..
1..4
# NOTE: developer test!  It's possible that it won't run on your
# platform, and that's perfectly fine.  This is mainly for developers
# on Unix to check that our shared libraries are consistent with the
# ordinals (util/*.num in the source tree), something that should be
# good enough a check for the other platforms as well.
ok 1 # skip Can't run 'nm -DPg ../../libcrypto.so' => 256...  ignoring
ok 2 # skip Can't run 'nm -DPg ../../libcrypto.so' => 256...  ignoring
ok 3 # skip Can't run 'nm -DPg ../../libssl.so' => 256...  ignoring
ok 4 # skip Can't run 'nm -DPg ../../libssl.so' => 256...  ignoring
ok
All tests successful.

This is fixed with #17119

Closed?


Join of two changelog_kv gets numerous incorrect intermediate results in 1.3.14 version Proton

what's wrong
In the test of two streams join, changelog(append-only) inner join changelog(versioned-kv), some data did not join, and the joined data's quantity kept increasing with larger data volumes and output numerous incorrect intermediate results even used lag_behandfunction.
How to reproduce
-- the left append-only stream
CREATE stream EDC_ITEM_TABLE
(
    `projectId` string,
    `envId` string,
    `envSiteId` string,
    `subjectId` string,
    `mandatory` int32,
    `filled` int32,
    `repeated` int32,
    `active` int32,
    `name` string,
    `itemOid` string,
    `updateTime` datetime64(3),
    `raw` string
);
-- the right version-kv stream
CREATE stream EDC_CRF_DATA_TIME_TABLE
                                (
                                    `projectId` string,
                                    `envId` string,
                                    `subjectId` string,
                                    `updateTime` datetime64(3),
                                    `raw` string
                                )
                                PRIMARY KEY (projectId, envId, subjectId)
                                SETTINGS mode = 'versioned_kv';

-- stream to store the join result
CREATE STREAM edc_item_active_result(`projectId` string, `envId` string, `subjectId` string, `name` string, `itemOid` string, `flag` bool, `updateTime` datetime64(3), `raw` string
                );

-- mv to join to get the results
CREATE MATERIALIZED VIEW etl_data_combine INTO edc_item_active_result AS
SELECT
  projectId, envId, subjectId, name, itemOid, (mandatory = 1) AND (active = 1) AND (filled = 0) AS flag, updateTime, raw
FROM
  (
  SELECT
    t1.projectId, t1.envId, t1.subjectId, t1.name, t1.itemOid, t1.mandatory, t1.active, t1.filled, t1.raw, t1.updateTime
  FROM
    changelog(EDC_ITEM_TABLE, projectId, envId, subjectId, itemOid) AS t1
  SETTINGS
    seek_to = 'earliest'
) AS temp
INNER JOIN changelog(EDC_CRF_DATA_TIME_TABLE) AS t2 ON (temp.projectId = t2.projectId) AND (temp.envId = t2.envId) AND (temp.subjectId = t2.subjectId) AND (temp.updateTime = t2.updateTime) AND lag_behind(1ms, t2.updateTime, temp.updateTime);

Error message and/or stacktrace

The data information of EDC_ITEM_TABLE:
run these query:
SELECT count(*) FROM table(EDC_ITEM_TABLE);
SELECT subjectId, count(*) FROM  EDC_ITEM_TABLE GROUP BY subjectId SETTINGS seek_to = 'earliest';

Here are the results in the txt file, which shows the counts of the results of the join should be 3000 and each subjectId with 5 join results.
issue_ITEM.txt
After the mv etl_data_combine was created, the counts of it kept increasing abnormally and exceeded the correct counts a lot.

Observing this case, I droped this mv and the counts of edc_item_active_result did not increase.
Checked the results in the issue_result_records.txt:
issue_result_records.txt
Lots of rows in EDC_ITEM_TABLE were not joined. Only part of rows are joined and the result counts of each subjectId exceeded 5 a lot.

Had tried many times with the right data but I could not reproduce the bug in 1.3.15.


Also im not sure why the FinalRank file looks like the whole file was replaced, it might be formatting or something im not sure.
The patch for FinalRank.ChangeLevel was the only thing actually changed in that file.

There also seems to be an ancient method called GetLevel in Plugin.cs which I rarely used
public static LevelContainer GetLevel(string id)
{
	foreach (AngryBundleContainer container in angryBundles.Values)
	{
		foreach (LevelContainer level in container.levels.Values)
		{
			if (level.field.data.uniqueIdentifier == id)
				return level;
		}
	}

	return null;
}

yeah no idea why I did not make that sort of method

ooooh I didnt even see that

epic fail
public static bool TryFindLevel(string id, out LevelContainer level)
{
    level = null;

    foreach (AngryBundleContainer container in Plugin.angryBundles.Values)
    {
        foreach (LevelContainer levelContainer in container.levels.Values)
        {
            if (level.data.uniqueIdentifier == id) // not level, levelContainer
            {
                level = levelContainer;
                return true;
            }
        }
    }

    return false;
}

I'm sorry.


customized display time of each logo

I find the logos switch too fast between each design right now, it feels a bit hectic / nervous.
I would like to see the option to give it a customizable display time. Let's say "play every logo for 2 / 5 / 8 / 10 / 15 seconds".
If not customizable, maybe just giving every logo a few extra seconds would make it feel more relaxed and nicer to watch.
But I love this screensaver! I don't know why the default ones of Apple are still so lame since there are so many possibilities.
Thanks for the efforts!

You can do that.

I'm finding that option doesn't seem to work for me. Each animation plays once and moves on. Catalina beta 19A536g.

Maybe there is another bug. At the beginning the time for each animation seems okay until "Apple's Original Version" comes. After that the animations are only played half and independent of "Repeat Each Animation". I will create an issue.


Consider moving current master to 2-2-stable

After #3 is merged it might make sense to put 3-x support into master and move the current master to 2-2-stable to be consistent with typical Spree versioning.

Branch is here: https://github.com/joshnuss/spree_stock_notifications/tree/2-2-stable


type is module

e2e test fail with This file is being treated as an ES module because it has a '.js' file extension and '/home/runner/work/wallet-provider/wallet-provider/package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.

now it fails with:
HOME=/root npm run test-extension
shell: sh -e {0}

<EMAIL_ADDRESS>test-extension
cd ./test-extension && npx playwright test

ReferenceError: __dirname is not defined in ES module scope


Hotfix/0.6.1

Coverage decreased (-1.0%) to 89.244% when pulling f56c6ec397756be4dac0668e3cf29cda6fd59db3 on hotfix/0.6.1 into a323d2c6c0b7f3609bb1872e7f1c719307c65e38 on master.


Team Tycoons Registration

Application for Registering Team to CODE CAMP 1.0
This pull request template helps you complete an application to the CODE CAMP 1.0 HACKATHON. Use the checklist below to verify that you have followed the instructions correctly.
(Put x inside [] like [x])
Select the type of Pull Request

[x] Registration Pull request.

[] Submission Pull request.
Checklist ✅

[x]I have read the instructions on the README file before submitting my application and gone through CODE_OF_CONDUCT.
[x] I have Submitted my team's details by making a folder of my team as instructed in How to send Pull Request page.
[x] I have used the Markdown file template to add my information for the Hackathon.
[x] I have given all the necessary details as mentioned in template index.md file and the details are correct and best to my knowledge.
[x] I understand that a reviewer will merge my pull request after examining it or ask for changes in case needed.
[x] I understand I should not tag or add a reviewer to this Pull Request.
[x] I understand the Details added to the template will be used as a means of communication at the time of result declaration.
[x] I have added the event to my calendar.

@rishiagrawal2609 , Please follow the instructions and join your team to our whatsapp group

I already joined the group
On Mon, 20 Jul, 2020, 4:07 pm Utkarsh Yadav<EMAIL_ADDRESS>wrote:

@rishiagrawal2609 https://github.com/rishiagrawal2609 , Please follow
the instructions and join your team to our whatsapp group
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ISTESRMNCR/CODE-CAMP-2020/pull/49#issuecomment-660948376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANF7LV52D7ICUZQXRHJYKCLR4QM57ANCNFSM4PCBCMVQ
.

ok done


Ranged hit enchant on warrior counts as melee hit.

Describe the bug
The added ranged hit from the  R19 Threatfinder Enchant gets counted as melee hit.
To Reproduce
Steps to reproduce the behavior:

Go to the Cataclysm Arms Warrior sim (https://wowsims.github.io/cata/warrior/arms/)
Add a ranged weapon.
Select R19 Threatfinder for the enchant.

Expected behavior
I'm not entirely sure the expected behavior is since there is no ranged hit stat for Arms in the simulator but, since I imagine ranged attacks aren't part of any serious rotation, the stats from the enchant should just be ignored.
Sim Links and Screenshots
Sim
Desktop (please complete the following information):

OS: Windows 11
Browser chrome
Version 126

Additional context
It's solvable by removing the enchant, but if you import your gear from an addon it may take a while to see where the mismatch in stats comes from.

Oh I see this has already been reported: https://github.com/wowsims/cata/issues/588


Use pkg_resources to parse requirements

Fixes #6153

Can one of the admins verify this patch?

/azp run python - identity - ci


additions for debian in readme and small fixes

i did this all while not thinking, be aware

looks good


Support HIP

I sent total PR #1094 .
I will send split PR as following list.

[x] Fix setup.py (#1121 , etc.)
[x] Support compiler on HIP (#1094)
[x] Fix header files (#1094)
[x] Use fmin and fmax for HIP environment #1116
[ ] BLAS
[x] RAND (#1094)
[ ] SPARSE
[ ] SOLVER
[ ] DNN
[x] CI
[x] rocThrust (#2666)

Hi,
Great work! Would like to talk more about rocm support. Can you share your email id? @gstoner

Hi.
Can i help for your testing codes before merging pull request ?

Thanks.
At first, I plan to fix some base functions. For example, setup.py and compiler.py.
I would like you to cooperate with the test after merging these PRs.

Okuta.
OK, I'll try them.

This site (https://gpueater.com/) provides virtual servers with AMD graphics cards (and images that come with ROCm installed). Perhaps those could also be useful for testing purposes.

Related:

[ ] FFT (#3892)
[ ] CUB (#3893)

@okuta @emcastillo I think this issue can be closed. The only component we haven't covered is hipSPARSE/rocSPARSE, for which we can issue a separate ticket to track.


Rebuilding AboutUs Section

I want to add following feature to the section:

An vector image as logo
Styling of content in about section

closes via #110


build: build docker images on release created

This PR corrects an error with the release wokrflow

Thanks for the pull request, @Ian2012!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

Hmm, the dev / local tests didn't seem to run on this?

@bmtcril changes only to the .github folder doesn't trigger the dev / local workflows as those changes doesn't affect the project

@bmtcril changes only to the .github folder doesn't trigger the dev / local workflows as those changes doesn't affect the project

Riiiiight I just approved that 😂

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


[tx] Allow [:db/tx :db/txInstant INSTANT] in transaction input

Datomic (and the Clojure implementation of Mentat) allow to specify the :db/txInstant that the transaction will be applied with by including a datom like
[[:db/add :db/tx :db/txInstant INSTANT]]

(Note that the :db/tx is a Mentat addition, although I think Datomic added an equivalent thing recently.)  This requires INSTANT to be strictly after the last transaction timestamp and strictly before the current transactor timestamp.
Adding this allows to import data with meaningful timestamps, for example.

I don't believe it's necessary that the timestamp be later than the last transact: multi-master replication demands the opposite. But it's a reasonable restriction to start with.

Let's make this ticket track also adding arbitrary assertions with the :db/tx entity, that is, handling things like
[[:db/add :db/tx ATTRIBUTE VALUE]]


txlog for key

Feature request
I can get the txlog for a table, but I'd like to be able to get a txlog or transaction summary for a given key of some table.

can we get a new issue for this? the 2.3.6 label reflects the functionality that went out.


Blacklist Curse of binding enchantment

Like we have config for blacklisting items but how can we blacklist items with enchantment like Curse of binding

Cannot even remove the item in creative mode like you can the vanilla armor slots. You have to use the "shift  click the red X"  clear to get rid of the binding items. For this you need a server op/owner to be online for if you pick up so your basically screwed if you play on a Hardcore world / server or a server that runs the "Dark RPG" mod pack where the mod pack is designed in a way where you cannot drop items upon death.

Added a config option to disallow them (on by default), and try something like /trinkets set head vanity 0 air if you've already put something in the slot


Nightly tests failing as of 9 days ago

Describe the bug
See here - our nightly tests are failing, in particular our map tests -- it seems to be related to the changing size of our map
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/usds/justice40-tool/actions/runs/1073597302 to see test logs from 9 days ago for the map tests
See below :

  1) Tests for the Explore the Map page
       Can focus on Lower 48 :
     AssertionError: Timed out retrying after 4000ms: expected 'http://localhost:8000/en/cejst#3.19/38.07/-95.87' to include '3.25/38.07/-95.87'
      at Context.eval (http://localhost:8000/__cypress/tests?p=cypress/e2e/map.spec.js:119:18)

  2) Tests for the Explore the Map page
       Can focus on Hawaii :
     AssertionError: Timed out retrying after 4000ms: expected 'http://localhost:8000/en/cejst#5.35/20.574/-161.438' to include '5.89/20.574/-161.438'
      at Context.eval (http://localhost:8000/__cypress/tests?p=cypress/e2e/map.spec.js:119:18)

  3) Tests for the Explore the Map page
       Can focus on Puerto Rico :
     AssertionError: Timed out retrying after 4000ms: expected 'http://localhost:8000/en/cejst#7.65/18.2/-66.583' to include '8.19/18.2/-66.583'
      at Context.eval (http://localhost:8000/__cypress/tests?p=cypress/e2e/map.spec.js:119:18)

This appears to be related to our map changing size
Expected behavior
These tests should pass every night
Possible Fix

run npm run cy:open
This will trigger the cy window, click on the failing test
check the URL for z/x/y numbers,
they should match the expected numbers in the test

We have the solution for this. We need to change expectation on the test.


AARs are exploded to the build directory which gets deleted by gradle clean

For me, this causes broken projects in Eclipse over and over.
Likely for that reason, the gradle eclipse plugin puts libraries (and its sources) into the local Gradle cache. It would be good if gradle-android-eclipse would use that approach too.

I'm working on this, see https://github.com/greensopinion/gradle-android-eclipse/wiki/2020-experiment


merge

Coverage increased (+1.8%) to 83.344% when pulling f4c7451e5351005a2fb6abe2085d6df2c116cb63 on rmillikin:master into cfbea60bfb26b445a2ebf70077d8c50ec1e0d6e0 on smith-chem-wisc:master.


Mcsema debugging guide

Add debugging information/tips/etc to the documentation.
Should discuss the awesome gdbinit settings, general tips on debugging translation errors and segfaults in translated bitcode.

Done.


Certain array access expressions are handled wrong (also documentation for "slicing" into lists doesn't seem to be right)

With this input:
[
    {
        "id": 2,
        "name": "Greg's Batch",
        "notes": null,
        "version": 16
    },
    {
        "id": 28,
        "name": "Test Batch - 2016-04-05T11:08:56.924",
        "notes": "Created on 2016-04-05T11:08:56.924",
        "version": 0
    }
]

And this code:
 Object o = from(json).get("[-1]");
 System.out.println(o.toString());

Returns a list of List of size two, with both elements null.
I think the culprit is line 41 in JSONAssertion.groovy
if (key =~ /^\[\d+\].*/) {
    expr = "$root$key"
} else {
    expr = "$root.$key"
}

This regex does not match [-1] and thus creates a groovy expression that tries to dereference an array.
The real regex should be something like ^[-?\d+(..-?\d+)?].* to account for negative numbers and list "slicing".
Additionally, if calls to GroovyShell is the way that list slicing is achieved, the documentation for list slicing on the README file is not correct.
The groovy documentation here shows that slicing is accomplished with .. character and not the : character.
Here are examples from the aforementioned Groovy documentation.
def sub = text[5..10]
assert sub == 'cheese'

text = "nice cheese gromit!"
x = text[-1]
assert x == "!"

def name = text[-7..-2]
assert name == "gromit"

Finally, this tool purports to evaluate Jaway JsonPath expressions and correctly evaluates the expression [-1:] per the JsonPath README file. This leads me to believe it is not using the com.jayway.restassured:json-path:2.9.0 to evaluate the expressions.

So I just realized the class I'm referring to is the rest-assured repo and not the json-path repo. I just assumed that "json-path" in rest assured dependend on the the json-path repo. I will close and re-open in the rest-assured repo.

So I just realized the class I'm referring to is the rest-assured repo and not the json-path repo. I just assumed that "json-path" in rest assured dependend on the the json-path repo. I will close and re-open in the rest-assured repo.


[core-http] Do not use type any

Some serialization methods use type any as a parameter, and as their return type, as discussed here: https://github.com/Azure/azure-sdk-for-js/pull/8687/files#r422707727
We should be using proper types everywhere. We should look for all of the use cases of the any type in the @azure/core-http library and replace them with their proper types.

@xirzec Am assuming this wont be a problem in v2 of core-http?
In which case, would it be worthwhile to pick this up?
cc @jeremymeng, @chradek

This particular code will live in core-client, and I was planning on cleaning it up when I port it there.


As a Project Coordinator, I want to see a Task list when I create or edit a project, so that I can complete forms in any order

Description:
When creating or editing a project, i.e. when creating a project revision, a large number of fields may be edited. To follow UX best practices, those fields should be divided in multiple forms: Overview, Managers, Contacts, Attachments, and later a form for the different schedules.
When multiple forms have to be filled, one possible UI pattern is the task list.
This user story would add a task list as shown below in the left side of the page when creating a project revision.

See the CIF Workflow Co-design for more examples.
A ProjectRevision component needs to be created to control the transitions between the different forms, and whether the task list will show a link to a page with a "Submit" button (later to be replaced to a link to the "Check Answers" page, see #226)
Acceptance Criteria:
Given that I clicked the button to start a project
When I am redirected to the project overview form, at /cif/project-revision/<id>/form/overview
Then I see the task list on the left, and the project overview form on the right
And the status of the "Project Overview" from is "Not started"
And the "Project Overview" form is highlighted in the task list.
Given that I filled the project overview form and it is valid
When I click the "Submit Project Overview" button
Then I am redirected to /cif/project-revision/<id>/check-answers
And the project overview form status is changed to staged/"Complete"
Given that I am viewing the /cif/project-revision/<id>/check-answers page
When I click the submit button
Then the project revision is submitted and I am redirected to the project list
check answers page is very simple with just a submit button, nothing else. To be completed in #226
Given that I am viewing the overview form
When I click on the "Add projects managers" link
Then I am redirected to /cif/project-revision/<id>/form/managers
When I click on the "Add projects contacts" link
Then I am redirected to /cif/project-revision/<id>/form/contacts
When I click on the "Add attachments" link
Then I am redirected to /cif/project-revision/<id>/form/attachments
Given that I am opening a form with a status of "Not Started"
Then the Submit <form name> and "Undo Changes" buttons are disabled
When I change a form input
Then the Submit <form name> and "Undo Changes" buttons are enabled, and the form status is changed to "In Progress"
Given that I am viewing a form with a status of "In Progress" and that one or several of the inputs are invalid
When I click the Submit <form name> button
Then the validation errors are displayed and the form status changes to "Attention Required"
errors can be displayed using the default rjsf/core and later refined in #200
Skip button and Discard Project revision to be covered in separate user stories
Development Checklist:

[ ] When starting a project revision, form_change records should only be created on the first edit
[ ] Add a new state for form_change records: staged
[ ]
[ ] Meets the DOD

Notes:

Personas:
Operations Team

Hey team! Please add your planning poker estimate with ZenHub @AlexZorkin @dleard @pbastia

We will be breaking down the work in separate pull requests: the routing, the component styling, ...

@suhafa you can find more wireframes in our workflow co-design Miro board, in the frame titled "#225: Support multiple forms using a task list"

Closing this, in favor of multiple, smaller user stories under a task list epic

(uploading just for reference)

Edited the position of buttons
Suggestion: Have 'optional' fields or not applicable categories under task list be collapsible dropdown, and have them collapsed if not relevant at that phase of the project to reduce visual load and direct user to focus on what tasks are important.


Document configuration for modified date predicate in Crayfish and RdfMappings

From https://groups.google.com/forum/#!topic/islandora/ah56IIO9hnQ
We need to update in the READMEs of both Milliner and Islandora to highlight that their configurations must match up for the modified date predicate.

DIG: Still necessary to update? Ask Tech call for help.

Related issue: https://github.com/Islandora/documentation/issues/1395


IMAGE region support

@danielhuppmann I think I resolved your comments. I followed your directions in renaming the mapping of IMAGE CEU + WEU towards "Europe (excl. Turkey)", while it is not perfect, as Ukraine, for example, is also not part of these IMAGE regions, while this file seems to suggest it is part of "Europe (excl. Turkey)".

Note that you can also include any timeseries data at the regional level "Europe (excl. Turkey)" in your submission to the scenario data, and those values will be merged (and override) any values computed from the automated region-processing.

Good to know - thanks!


Derivative Class Implementation

Feature 2; User Story A; Task 001
In order to implement symbolic calculations that are derivable, we need a working derivative class for all forms of the following derivable functions: polynomials, Trig functions (sin, cos, tan, csc, sec, cot), ln(x), a^x, e^x.

Have working code for the a^x and e^x and one step trig functions in attached file. Other methods require further time invested, as the derivatives and integrals for trigonometric derivations are proving quite tedious. I have found that in a nutshell, what we would have to do is find the tangent line of a point* on your function, take the slope, and then plot that slope as a point on your derivative. You'll do this in discrete steps (like incrementing x by .2 every time), so it won't be an exact solution, but you'll be able to gather enough information.


chore(main): release 0.7.6

:robot: I have created a release beep boop
0.7.6 (2022-09-23)
Bug Fixes

Redirect old airflow urls to /pipelines ones (1c33d44)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-frontend/releases/tag/0.7.6 :sunflower:


Overall Results

Hi,
I am not getting the final result so I am entering the text into excel(WHY!!). However, my results looks very good, I am getting around 90%.

Ya! the same as me

yes, same as me, when predict the ucf101 results on model "sports1m_finetuning_ucf101.model", I can get 90.9% accuracy! This is not as reported

What could be going wrong? or actually is there anything going wrong?

I have no idea.
@Mohamad73  And also, do you know what 'c3d_ucf101_finetune_whole_iter_20000_TF.model' is? Which dataset is this model pre-trained on? I can get around 58% accuracy when predict the ucf101 results.

I think it is not pre-trained on any big dataset such as Sports-1M. It is only trained on the UCF-101

Thanks for your answer. The accuracy is quite high, in dutran's paper, train from scratch only achieved ~45% accuracy. It is quite strange, I should check it again.

How are you getting the final accuracy? I am not getting it, I have to enter my data into excel to get the result?

The predict_c3d_ucf101.py outputs file 'predict_ret.txt'. It contains the predict_label and actual label, by processing the txt using python, pandas library, I can then get the result.

@laura-wang I got very low acc, would mid sharing the code you changed for finetune the network. What's more, can you share the sports1m_finetuning_ucf101.model? I can't download it.
Looking forward to any replies.
Thanks

I meet the same results. If I use the predict_c3d_ucf101.py, the accuracy is about 81.23% for the model which is trained from scratch and for the fine-tuned model, the accuracy seems to be 90.54% (89.34% for sports1m_finetuning_ucf101.model). In addition, when I use Random_clip_valid.py, the accuracy becomes 83.58% for sports1m_finetuning_ucf101.model. All the results I get seem to be higher than those in the tables.
I read the codes and it seems to be test on video clips. The highest clip accuracy is 79.87% and for random-clip, the best result is 76.6% as reported. I can not figure out why. Is it possible that we do the RIGHT thing and just get higher results?

@Mohamad73 @SiDinhThoi @laura-wang @Ai-is-light @BestJuly ，THe results @hx173149 obtained were tested on ucf-101 test split 1,and where your guys trained ,you then split dataset again,so your test.list must have many samples that have been in train _splitt_1(which is used to finetune the pretrained model on sports1m),another way model sports1m_finetuning_ucf101.model have seen your test sample.

@Mohamad73 @SiDinhThoi @laura-wang @Ai-is-light @BestJuly ，THe results @hx173149 obtained were tested on ucf-101 test split 1,and where your guys trained ,you then split dataset again,so your test.list must have many samples that have been in train _splitt_1(which is used to finetune the pretrained model on sports1m),another way model sports1m_finetuning_ucf101.model have seen your test sample.
Thank you for you explanation.
I agree with you for some part. When I use the split01 to train from scratch, the accuracy is very low. Actually, when I got very high results, the training split and the testing split has no overlaps. However, the question still lies to the samples because if we split the dataset by ourselves, e.g. sample id (not the id for categories) 0~4 for training, and id 5~6 for testing, although these 6 videos are different, the scenes in those videos are similar because the same person with the similar background doing the same cation. However, for the split01, there is no overlaps with the persons and the backgrounds in videos. That is why the results are so different.
And when I train from scratch with C3D on split01 training part, when testing, the accuracy is about 30%, and for resnet-18 with 3D convolutions, one researcher reported on CVPR'18 that if trained from scratch, the accuracy is about 42.4%. The result I've got seems more reasonable now.


Reduce command tree

Don't show commands to players that can't use them
This would really to reduce command load time for player from geyser

CommandWhitelist by YouHaveTrouble
You can also deny the permission, but if some reason, it doesn't. Use what I said above.


Failing on writing Unicode Strings

I get the following exception HiveError: String contains non-ASCII characters. whenever I write a Unicode string (currently, I am just trying to store some Arabic text as a String in the database).
Any thoughts on how to fix this? I'm happy to implement the fix if you provide some pointers.

Keys have to be ASCII. Could you share a key value pair which is not working?

Nevermind them. I made all the strings ascii and kept the values Unicode and it seems to work well.
Thank you for the quick feedback.

how did you do that?
am facing the same problem here whenever am trying to save Arabic string what should I do?

To create your key,
Use
String.fromCharCode(key)
eg:
var myKey=String.fromCharCode(key);
hiveBox.put(myKey,value);
then
while fetching your values
also retrieve your values by converting the non ASCII chars and pass as Key argument
hiveBox.get(myKey,value);
Result
//my values

how did you do that?
am facing the same problem here whenever am trying to save Arabic string what should I do?

Hive only accepts ASCII characters as entry keys. But this limitation does not applies to values.
box.put('non-ascii-key', '...');                 // will not work
box.put('ascii-key', 'non-ascii-value'); // will work

Use this
Make sure to
import dart:convert;
Convert your key to ascii code
Eg:
//your key is probably an unaccepted string
Var myKey= key.codeUnitAt(0);
Then,
hiveBox.put(myKey,values);
To retrieve your values Do the same thing,
Var myKey=key.codeUnitAt(0);
Then,
hiveBox.get(myKey);
//results your values

If it works for you, lemme know if it doesn't also lemme know with your error logs to help better

Anyways posting error logs will help to better understand the issue.


Save wallpapers in directories named by input?

Not really an issue, but I have a question.
The script saves all wallpapers in the Wallhaven directory (which is in your working directory). How could I save them in directories named based on what I input?
For instance, if I enter category, anime, swf, I want the wallpapers to save to anime/sfw/ [as in, sfw directory is inside anime directory] or just anime/.
If I enter latest, I want the wallpapers saved to latest/.
Or if I enter search, pokemon, I want the wallpapers to be saved to pokemon/

The following code will more or less do what you want.
If you search for pokemon, it will be saved to pokemon/
If you enter latest it will appear in latest/
If you enter category, anime , sfw it will be saved to anime-sfw/
Here is the modified code:
`########################################################
Program to Download Wallpapers from
alpha.wallhaven.cc

Author - Saurabh Bhan

Dated- 26 June 2016
Update - 11 June 2019
########################################################
import os
import getpass
import re
import requests
import tqdm
import time
import urllib
import json
BASEURL=""
cookies=dict()
global APIKEY
APIKEY = 'Enter Your APIKEY Here'
def category():
global BASEURL
global DIRECTORY
print('''
****************************************************************
Category Codes
all     - Every wallpaper.
general - For 'general' wallpapers only.
anime   - For 'Anime' Wallpapers only.
people  - For 'people' wallapapers only.
ga      - For 'General' and 'Anime' wallapapers only.
gp      - For 'General' and 'People' wallpapers only.
****************************************************************
''')
ccode = input('Enter Category: ').lower()
ctags = {'all':'111', 'anime':'010', 'general':'100', 'people':'001', 'ga':'110', 'gp':'101' }
ctag = ctags[ccode]
print('''
****************************************************************
                        Purity Codes
sfw     - For 'Safe For Work'
sketchy - For 'Sketchy'
nsfw    - For 'Not Safe For Work'
ws      - For 'SFW' and 'Sketchy'
wn      - For 'SFW' and 'NSFW'
sn      - For 'Sketchy' and 'NSFW'
all     - For 'SFW', 'Sketchy' and 'NSFW'
****************************************************************
''')
pcode = input('Enter Purity: ')
ptags = {'sfw':'100', 'sketchy':'010', 'nsfw':'001', 'ws':'110', 'wn':'101', 'sn':'011', 'all':'111'}
ptag = ptags[pcode]

BASEURL = 'https://wallhaven.cc/api/v1/search?apikey=' + APIKEY + "&categories=" +\
    ctag + '&purity=' + ptag + '&page='
DIRECTORY = ccode + "-" + pcode

def latest():
global BASEURL
global DIRECTORY
print('Downloading latest')
topListRange = '1M'
BASEURL = 'https://wallhaven.cc/api/v1/search?apikey=' + APIKEY + '&topRange=' +
topListRange + '&sorting=toplist&page='
DIRECTORY = 'latest'
def search():
global BASEURL
global DIRECTORY
query = input('Enter search query: ')
BASEURL = 'https://wallhaven.cc/api/v1/search?apikey=' + APIKEY + '&q=' +
urllib.parse.quote_plus(query) + '&page='
DIRECTORY = query
def downloadPage(pageId, totalImage):
url = BASEURL + str(pageId)
urlreq = requests.get(url, cookies=cookies)
pagesImages = json.loads(urlreq.content);
pageData = pagesImages["data"]
for i in range(len(pageData)):
    currentImage = (((pageId - 1) * 24) + (i + 1))

    url = pageData[i]["path"]

    filename = os.path.basename(url)

    os.makedirs(DIRECTORY, exist_ok=True)
    osPath = os.path.join(DIRECTORY, filename)
    if not os.path.exists(osPath):
        imgreq = requests.get(url, cookies=cookies)
        if imgreq.status_code == 200:
            print("Downloading : %s - %s / %s" % (filename, currentImage , totalImage))
            with open(osPath, 'ab') as imageFile:
                for chunk in imgreq.iter_content(1024):
                    imageFile.write(chunk)
        elif (imgreq.status_code != 403 and imgreq.status_code != 404):
            print("Unable to download %s - %s / %s" % (filename, currentImage , totalImage))
    else:
        print("%s already exist - %s / %s" % (filename, currentImage , totalImage))

def main():
Choice = input('''Choose how you want to download the image:
Enter "category" for downloading wallpapers from specified categories
Enter "latest" for downloading latest wallpapers
Enter "search" for downloading wallpapers from search
Enter choice: ''').lower()
while Choice not in ['category', 'latest', 'search']:
if Choice != None:
print('You entered an incorrect value.')
choice = input('Enter choice: ')
if Choice == 'category':
    category()
elif Choice == 'latest':
    latest()
elif Choice == 'search':
    search()

pgid = int(input('How Many pages you want to Download: '))
totalImageToDownload = str(24 * pgid)
print('Number of Wallpapers to Download: ' + totalImageToDownload)
for j in range(1, pgid + 1):
    downloadPage(j, totalImageToDownload)

if name == 'main':
main()`

Thanks!
For the directory names with search, how would I have spaces and special characters (slash, apostrophe, colon, etc.) converted to underscore?
E.g., search final fantasy, saved to final_fantasy. Search fate/stay night, saved to fate_stay_night.

To replace spaces and special characters with an underscore replace line 79 of the modified file with
DIRECTORY = re.sub('[^0-9a-zA-Z]+', '_', query)
Now only digits, uppercase and lowercase alphabet will remain, anything else will be replaced by an underscore.
Here is the updated modified code:
wallhaven-customfolders-fixedSearch.txt

Works perfectly! Thanks for your help! Keep up the good work!


pi_with_darts.png is missing from the img folder

The figure doesn't show in the exercise box.

thanks!


Add reference to build-info command

What content needs to be created or modified?

Add reference to build-info command
Describe the solution you'd like

Where should the new material be placed?

Dapr CLI reference
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos

https://github.com/dapr/cli/pull/745
https://github.com/dapr/dapr/pull/3342
Additional context

/assign


Changelog entries for v0.5

Coverage remained the same at 93.429% when pulling 16fba9e9413fb0d6c854965f5f3819af7c4eac39 on rel05 into 90ae6ee2686d67b4530eaf0ffe38bd8c89a8da66 on master.


Issues using !TupleReturn methods in expressions

!TupleReturn now properly works in a sane way for non-destructuring assignment, but it's still not wrapped in a table when used as an expression (e.g. print(tupleReturnMethod()); or let a = tupleReturnMethod()[0]). This brings up the type system discrepancies again, as discussed in #110. Note that for these cases, the return values should be wrapped with ({ ... }) as opposed to simply { ... }, because { ... }[1] is invalid syntax in lua.

I think this can be solved by moving the tuplereturn checking to the call expression transpilation and traveling up the AST to check if it is part of a destructing assignment. Will have a look probably tomorrow.

Alternatively, always wrapping the result in a table and using select for destructuring assignments might be easier, but impose a slight performance penalty.

Should be fixed in 3a92068cddd77a97794c294ba1fa41f37a401554


rewards mapping fix + test

The latest change to the rewards recording mistakenly mapped the reward eval function to the actions rather than their indices.

Oh! That's it exactly! Man, I hate that we have to do it this way.
This also need to handle batching. I can take care of adding that tomorrow.

Batching added. Again, great catch and thanks for the unit test!
This is exactly how coba got so many tests to begin with. I've just been adding new tests every time I find a bug :).
If your team needs it I released this tonight as a new minor version update (6.3.0) because my batching code quasi-broke backwards compatibility.
Also, not that it matters to you but it looks like a research team at Stanford is using Coba now so that's fun...
Also also, the SimpleEvaluation class is getting kind of messy at this point. A long time ago what is now SimpleEvaluation actually used to be three separate classes. The three classes was hard for new users because they would struggle to figure out which evaluation type to use. So, I folded them all into SimpleEvaluation where we try to guess what to do based on what we're given. Maybe it is time to start think about separating again... Or at the very least refactoring to clean up a little...

Great, thanks for the quick turnaround!
It's not super urgent to get the bugfix into a release as it only affects recording rewards which I am not relying on in any evaluations.
Great news about the Stanford lab! I am also looking forward to getting this into the hands of our data scientists and see what they come up with.
Yea, I can see the SimpleEvaluation class growing to become unwieldy. A simple first step could be to modularize the process function a bit into predict, learn and boilerplate components.


Suggest an alternative to the --all flag

I tried using the all option to create all manpages, but got
opts.on("--all", "Create a book with all man pages") do |dir|
ManBook.logger.info "Creating a book with all man pages"
raise "Not yet implemented"
end
As an alternative, I used
manbook $(apropos . | awk '{ print $1 }' | xargs)

After doing this and creating the toc, and zipping the file and changing the extension, it created a 40 meg monster. This took my desktop a few minutes to load, and think it would probably crash the kinde.

Thanks. Could you submit this as a pull request?


How do I get latest payment date?

I am using getSubscriptionTransactionDetails to get my subscription receipt every time app initializes. I only get purchaseTime as the date when I first subscribed. How do I get latest payment date?
I also cancelled my test subscription on Google Play but when I pick up latest details it still says "PurchasedSuccessful". Any ideas?

Hi @annelorraineuy.
I do not use subscriptions myself, so I rely on others to test here. I am unsure if there is any other date to go by than purchaseTime.
I'd recommend maybe saving details like this in your own backend API though, and lookup there.

@annelorraineuy look for "expires_date" in the last receipt in the array that gets returned. all fields described here - https://developer.apple.com/library/content/releasenotes/General/ValidateAppStoreReceipt/Chapters/ReceiptFields.html

@annelorraineuy sorry, thats just for iOS didnt realize you were using GooglePlay til just now

@temitope - yea for Google Play. Isn't this package just for Android billing, or does it support Apple IAP already as well?

oops :D @annelorraineuy you are entirely right. i am using both this and the iOS only one and didnt realize i was in the wrong repo almost misdirecting people with wrong info. sheesh. thanks 👍

@annelorraineuy Hi, have you find a solution for this? I am in the same situation. I have no idea how to check if the subscription is still valid or not. Maybe it would be worth it to add some use cases to this repo, so one can quickly get an idea on how to implement stuff :-) Thanks!

I checked the underlying docs, and they specify a autoRenewing-flag for checking cancelled subscriptions: https://github.com/anjlab/android-inapp-billing-v3#handle-canceled-subscriptions
I've added a new version 2.9.0 that exposes this flag, as well as docs on how to use it:

Call InAppBilling.getSubscriptionTransactionDetails(productId) and check the details.autoRenewing flag. It will be set to false once subscription gets cancelled.
Also notice, that you will need to call periodically InAppBilling.loadOwnedPurchasesFromGoogle() method in order to update purchase/subscription information from the Google-servers.

Like I've said previously, I do not use subscriptions myself, so I have been unable to test this. If someone else could try it out and give feedback, that'd be good. Not sure how "cancelled" subscriptions lines up with "unpaid" in this case.


Question about data preprocess

先决条件 / Prerequisites

[X] 该问题未被其他 issues 提出。
No previous issues have reported this problem.

发生了什么 / What happened
Thanks for your great work and ncie code!
In 'prepare_task_dataset.ipynb', it seems there is a bug in the following code:
'''
!pip install fair-esm
from tools import embedding_esm as esmebd
if len(needesm)>0:
tr_rep0, tr_rep32, tr_rep33 = esmebd.get_rep_multi_sequence(sequences=needesm, model='esm1b_t33_650M_UR50S',seqthres=1022)
#merge existing
feature_esm0 = pd.concat([feature_esm0, tr_rep0], axis=0).reset_index(drop=True)
feature_esm32 = pd.concat([feature_esm32, tr_rep32], axis=0).reset_index(drop=True)
feature_esm33 = pd.concat([feature_esm33, tr_rep33], axis=0).reset_index(drop=True)

#save
feature_esm0.to_feather(cfg.FILE_FEATURE_ESM0)
feature_esm32.to_feather(cfg.FILE_FEATURE_ESM32)
feature_esm33.to_feather(cfg.FILE_FEATURE_ESM33)

'''
Note that feature_esm0 has not been declared (If there is no existing features when I run the code for the first time), so I get a error as follow:

期望的现象 / What you expected
See  What happened'
如何重现此问题 / How to reproduce
See  What happened'
操作系统版本 / OS version
See  What happened'
浏览器或客户端版本 / Browser/client version
See  What happened'
其他环境 / Other environments
No response
其他 / Anything else
No response

Thank you for your interest in our work.
We apologize for the inconvenience caused. The issue you encountered is primarily due to the incompatibility between the defined ROOTDIR in the config.py file and your workspace. To resolve this, please manually specify the absolute project DIR that corresponds to your workspace.

Please note that we will leave this issue open for your feedback.

First of all, thank you for your quick and detailed reply, and I apologize for not noticing the GitHub issue response email earlier.
Secondly, regarding this issue, I have now realized that you have provided the processed data, and I am currently downloading the processed data.
As for the problem mentioned in the issue, I am still confused. I believe the issue I encountered is not related to ROOTDIR. Even after following your advice and modifying ROOTDIR to an absolute path, I encountered the same problem. Below, I will elaborate on my confusion in detail:
The error I encountered is that "feature_esm0" is not declared at this location：
feature_esm0 = pd.concat([feature_esm0, tr_rep0], axis=0).reset_index(drop=True),  and the only possible place where "feature_esm0" could have been defined is if ftool.isfileExists(cfg.FILE_FEATURE_ESM0):     feature_esm0 = pd.read_feather(cfg.FILE_FEATURE_ESM0)     feature_esm32 = pd.read_feather(cfg.FILE_FEATURE_ESM32)     feature_esm33 = pd.read_feather(cfg.FILE_FEATURE_ESM33)     feature_unirep = pd.read_feather(cfg.FILE_FEATURE_UNIREP)     feature_onehot = pd.read_feather(cfg.FILE_FEATURE_ONEHOT)     #caculate embedding list     needesm = full_snap_data[~full_snap_data.id.isin(list(set(feature_esm33.id)))]     needunirep = full_snap_data[~full_snap_data.id.isin(list(set(feature_unirep.id)))]     needonehot = full_snap_data[~full_snap_data.id.isin(list(set(feature_onehot.id)))]. However, it appears that when I first ran this code, the file cfg.FILE_FEATURE_ESM0 did not exist, which resulted in "feature_esm0" not being declared and caused the error I encountered. If cfg.FILE_FEATURE_ESM0 has been created in the previous code, I would greatly appreciate it if you could let me know the specific section of code where it was created.

One more question: when I ran task1.ipynb, an error occurred stating "No such file or directory: '/tmp/test_fasta_results.tsv'." In the repository you provided, the tmp directory is empty, and it appears that no code has been written to write any data to the tmp directory. I would greatly appreciate it if you could assist me in resolving this issue.

"I apologize for the delayed response; it was caused by an uninitialized variable. I have updated the code in the second code block of Section 7.1. If you have any additional questions, please address them in this issue."


WIP: Retry failing calls to devfile registry

/kind bug
What does this PR do / why we need it:
Retry the calls to devfile registry when call fails.
We have to discuss if this is the best solution, or if the problem can be fixed on the registry size.
With this patch, I was able to run the test suites several times without any failure, while seeing odo retry after errors.
Which issue(s) this PR fixes:
Fixes #5116
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

Not needed anymore as tests will run on IBM Cloud


Regex.replace does not set the index accurately

In this line in Regex.replace:
https://github.com/elm-lang/core/blob/1b28ce5ce8f30ddf549e5f454a772f7b623e09b4/src/Native/Regex.js#L90
there are two problems.
First, the variable i will inevitably be equal to -1 at this point. Thus, saying i - 1 is necessarily equivalent to -2, but obscure.
Second, the arguments object is not really an array, and does not support negative indexes. So, the index property is always set to undefined.
So, it would be better to replace this line with something like:
index: arguments[arguments.length - 2],

Here's an illustration of the problem, for use in elm-repl. (I would have written a failing test, but had some trouble getting the tests to run at all). The <internal structure> references below are the undefined results from match.index.
import Regex exposing (..)
findComma = regex(",")
replace All findComma (\match -> toString match.index) "a,b,c"

-- "a<internal structure>b<internal structure>c" : String

Yeah, pretty bad. Also reproducible on http://elm-lang.org/try, using this program:
import Graphics.Element
import Regex exposing (..)

findComma = regex(",")

main = Graphics.Element.show <| replace All findComma (\match -> toString match.index) "a,b,c"

Maybe you can revise your original post above to directly include that? Or better yet in pull request replacing this ticket? :smile:
Generally in the line of this comment by Evan.

To do a proper PR, I would want to write a failing test case that the PR fixes. However, I'm having trouble getting the tests to run at all -- I'll try harder at that when I have time.

Yeah, it's a pity that the master branch is not in shape to run tests at the moment. Maybe #446 or #447 would help you locally?

Ah, thanks, I will give those a try.

Consolidated regex stuff in #722. Follow along there!
Seems like a PR is possible.


Cannot Bulid ".openpublishing.build.ps1"

I'm try run .ps1 is error Please help

@kenakung Can you share what you are trying to do?  ".openpublishing.build.ps1" is a private repro.
Do you have a specific issue with the Microsoft Edge Enterprise documentation?  Thanks!

Closing issue as there has been no reply.


Teach find_button() to find tags of type

This is in addition to the other buttons it already knows how to find:
<input type="submit">
<input type="reset">
<input type="image">
<button>

This commit includes a new test to verify the new functionality.

 aloe_webdriver: Contribute upstream fixes needed by sites aloe tests

:+1:


crc table

Closes #185

I see that the workflow has failed because the secrets are not exposed to builds originating from forks. Should we reconfigure the workflow to skip SonarScanner unless the secrets are available? If you think this is sensible then perhaps we should fix it in this pull request.

Sure, thats acceptable. Not ideal but I don't see a clean way of doing it otherwise. I'll update this PR later.


adding api800 support for LIG

Description
API 800 Support for LIGs
Issues Resolved
NA
Check List

[x] New functionality includes testing.

[x] All tests pass for go 1.11.x + gofmt checks .

[x] New functionality has been documented in the README if applicable.

[x] New functionality has been thoroughly documented in the examples (please include helpful comments).
[X] New endpoints supported are updated in the endpoints-support.md file.

[X] Changes are documented in the CHANGELOG.

@mbfrahry Can you please review this PR, and let me know if any more changes need to be incorporated?

The change from TypeList to TypeSet is a little more nuanced than what is done in this PR. The two main shifts are that we don't care about ordering and hashes are used to identify the different sets

Hi @mbfrahry, I'm not sure I still understand when to use TypeSet and TypeList. Can you let me know on what basis do we decide this? Is there a way to determine if the response from oneview is ordered or unordered? Right now, I'm of the assumption that we can use TypeSet to all the attributes so that we don't need to do any ordering if required.

Hey @madhav-bharadwaj. So you're right that a TypeList is for ordered lists and a TypeSet is for unordered lists.
 Is there a way to determine if the response from oneview is ordered or unordered?

You'd have to ask the oneview api team about what lists are ordered vs unordered. You can determine manually by sending a large list to the api when creating a resource and then see if it comes back the same way on a read request.
When I wrote this provider years ago, I don't believe TypeSet was a type within Terraform so I built the hack into the provider that orders the unordered list from the oneview api. Now that you're swapping to TypeSet, that ordering doesn't need to happen anymore and can be removed.
Doing a d.Get() of a TypeSet is different than a TypeList as well because it's using a Hash now. So before it was d.Get("some_list.0.some_attribute") but now with a hash that will look like d.Get("some_set.12345678.some_attribute"). The loops in place will not work anymore because they are doing d.Get on i which isn't a hash.

Required changes have already been raised in this PR: https://github.com/HewlettPackard/terraform-provider-oneview/pull/68 and merged. So closing this PR.


Firewall - live view does not use description information from rule instead of "nat rule" as label

if a create a "outbound-rule" the "live view" always shows "nat rule" as label for the log entry and not the description of the triggered rule
To Reproduce
Steps to reproduce the behavior:

Go to 'firewall.' and create a nat outbound rule for p.e. a specific lan ip.
activate logging for that firewall rule
go to "firewall" -> "live view" and wait until the rule is triggered by the specific lan device
check the log entries
See error, that the description is "nat rule" instead of "description of triggered rule"

Expected behavior

If i create a firewall rule (outbound) the live view will show the description information from dedicated firewall rule as information in the column "label"

Describe alternatives you considered

if the rule has not an description it shows maybe furthermore the Label "nat rule"

Screenshots
rule config

live view

Environment
OPNsense 23.7.3-amd64

like suggested, create new one: https://github.com/opnsense/core/issues/6839


Livereload not working with modd

I'm trying to get a simple setup for an Elm single page app.
I've got this modd.conf
@shell = exec

src/**/*.elm {
	prep: elm-make src/App.elm --warn --output elm.js
}

elm.js index.html main.css {
	daemon: devd -m .
}

Any changes to the elm files successfully trigger a recompilation into elm.js. No problems here. The changes to elm.js, index.html and main.css are also sucessfully detected by modd:
>> sending signal hangup appears any time I change one of these.
My problem is that the livereload functionality somehow doesn't work. A breakpoint inside .devd.livereload.js at the onmessage callback shows that the websocket code never gets executed.
Livewatch does work fine whenever I run devd standalone, and the same breakpoint gets hit without a problem. I also tried a bunch of options for devd, but none changed the result.
Getting the same problem with Chrome and Firefox dev edition. I'm on Windows 7 x64 Pro.
If, however, I run devd in livewatch mode separately from modd (with the daemon line removed from the conf), I get whatever I wanted to achieve. But that seems a bit silly.
I'd be happy to provide additional info or help debugging. Thanks!

Hi there. Devd injects the livereload payload into the  section of the page. Does the page you're serving have one?

Yes, it does.

OK, could you please verify for us that the livereload payload is being injected into the page? Do that by viewing source, and looking for code like this in the header section:
<script src="/.devd.livereload.js"></script>

If the code is there, could you please look in the browser dev console and see if you see any issues with loading the payload, or with executing it (errors on console). If the code is NOT there, could you please show me what the initial section of the page looks like (that is, everything up to the body tag), so I can try to figure out why it's not there.

The code is there and the code is run (verified by running through the debugger). The console shows:
.devd.livereload.js:131 DevdReconnectingWebSocket attempt-connect ws://devd.io/.devd.livereload
.devd.livereload.js:147 DevdReconnectingWebSocket onopen ws://devd.io/.devd.livereload

Erm... has the separate modd calling devd w/livereload setup ever worked for anyone on Windows?
daemon.go calls Process.Signal
On windows this function is implemented in exec_windows.go
func (p *Process) signal(sig Signal) error {
	handle := atomic.LoadUintptr(&p.handle)
	if handle == uintptr(syscall.InvalidHandle) {
		return syscall.EINVAL
	}
	if p.done() {
		return errors.New("os: process already finished")
	}
	if sig == Kill {
		err := terminateProcess(p.Pid, 1)
		runtime.KeepAlive(p)
		return err
	}
	// TODO(rsc): Handle Interrupt too?
	return syscall.Errno(syscall.EWINDOWS)
}

🤔
So it seems that go on Windows doesn't handle anything else than SIGKILL...
(Which is weird, since, after testing with modd with daemon +sigterm:  devd -m . in the config, the devd subprocess * is* killed... so I must be wrong somewhere...)
Looking up the go issues on signal handling on Windows as well as MSDN documentation, it seems that signals basically don't work except for very specific cases. (I'm still confused as to what works and what doesn't, so I'll read up about the situation and add further comments here. I'll also get a go debugger set up.)

What's the conventional way to tickle a child process on Windows? Do a dummy write on a pipe?

Hello @lamarqua and @cortesi
On my fork, I've given modd the ability to send signals on Windows using "piped signals" (made up name). When necessary, modd can establish a pipe to the stdin of daemons it starts and will write the string representation of the signal it wishes to send as a line on that pipe.
https://github.com/frou/modd/commit/4ad97bbb7cf065d34f2e6134e86221c62dbc0172
I've also updated devd to be able to consume SIGHUP in that way to cause a livereload.
https://github.com/frou/devd/commit/faf340b584fa9df5a079f343bff1eaba5b007a2c
What you think?

@frou These sound like valuable additions. If you don't mind, could you lodge a PR on the modd project to begin with, and we can get the ball rolling?

@cortesi - Yep I will when I get a chance. For starters, it needs refined a bit because Go's string representation of the signal values has an odd mix of tenses, e.g. "quit" , "killed" , "interrupt" , "terminated"

I wonder if hacking up a custom signalling system for Windows is somewhat obsoleted by WSL now? WSL works very well, and modd + devd can run there with support for native signals. Thoughts, anyone?

That's probably part of a wider question in Windows developer land. Namely, is the native Windows CLI a lost cause? And is it unreasonable to expect all OSS-aware Windows developers to be WSL users? (AFAIK it's still an optionally-installed feature)

I wonder if hacking up a custom signalling system for Windows is somewhat obsoleted by WSL now? WSL works very well, and modd + devd can run there with support for native signals. Thoughts, anyone?

I might be a luddite but I still use Windows 7


Add a [VIEW LIVE POST] button to the UI

- Do you want to request a feature or report a bug?
Feature
- What is the current behavior?
None
- If the current behavior is a bug, please provide the steps to reproduce.
N/A
- What is the expected behavior?
After a post or pages is published will be nice to see a button [VIEW LIVE POST] or similar next to [Published] button with a blank target.

- Please mention your versions where applicable.

Netlify CMS version:
Browser version:

Node.JS version:
Operating System:
- Please link or paste your config.yml below if applicable.

This is related to https://github.com/netlify/netlify-cms/issues/372, but specifically targets published posts. This would require:

Design
Deriving the published entry path (can be changed by generator, build scripts, deployment platform, etc)
Determining whether the published entry has finished deploying (again, #372).

@erquhart You was asking about the red dot. That concept can be used in this issue.

It would be really helpful for non-technical users to get information about what's going on during the build to know if the post is already live. Depending (it seems) on how busy the Netlify bots are, we're seeing shorte and also very long build times. This is very confusing for authors since they are never sure if something went wrong or if it just takes more time.
Would something like the check for preview be possible for the live version - a check build status?

This actually should have been closed when #372 was closed - #2028 added a View Live button for published content: https://www.netlifycms.org/docs/deploy-preview-links/#preview-links-for-published-content
@marcojakob we should get a new issue open for that, and determine how best to do that using the backend (deploy preview links uses GitHub status checks).

Thanks @erquhart. I've opened #2324.


i like this util. has development been stopped?

i like this util. has development been stopped?

Development is paused the project hasn't seen much interest and I was getting bored with it.


Compilation of kaitai python module

@jpivarski how do you recommend we handle the compilation of the kaitai python module?
The simplest thing would be to just compile it locally and add the generated file to the repository.
However, we could also make it a part of the Python package, I found for example this repository (we do not need such a complicated system, but we could simplify it):
https://github.com/trailofbits/polyfile/blob/master/compile_kaitai_parsers.py

Compiling the Kaitai file-reader requires a Scala dependency, kaitai_struct_compiler, in addition to a normal CMake, make, gcc-or-clang compilers. For a Python package to provide that as a runtime capability, it could be distributed by conda-forge to get the normal compilers (compilers, make, and cmake), but there isn't a good way to get Scala through the conda-forge channel; there's a very old package in bioconda, but mixing channels is both a bad idea and can't be listed in the dependencies. On top of that, users would still need to get the kaitai_struct_compiler separately, and—until @ManasviGoyal's updates are integrated into the upstream kaitai_struct_compiler, users would have to get it from a non-standard source.
So unfortunately, I think the process of creating Kaitai file-readers, at least up to the generated C++ source code stage, has to be a specialized activity (until the above issues are resolved). Saving architecture-specific binaries in source control is not usually a good idea, but maybe the C++ source code files can be saved in source control. At least the diffs would be readable.
The Python extension is designed to dynamically pick up shared libraries with file-format specifics. The first step it takes is to load a so/dylib/dll shared library file representing one file format, produced by a KSY. So how about this (for now, at least):

KSY → C++ sources: performed by experts who have manually set up their environment.
C++ sources can go in a repository owned by a scientific collaboration, such as SDSS or CTA-observatory.
C++ sources → so/dylib/dll: part of routine deployment (continuous deployment from GitHub to PyPI?)
Loading so/dylib/dll at runtime: first step in the import of a Python library that reads the data format.

The Python packages can be arranged like this: generic awkward_kaitai is independent of file format; it has the generic code to load a so/dylib/dll as a Python class instance with a load method for reading raw data. Meanwhile, collaboration-specific libraries like awkward_kaitai_sdss and awkward_kaitai_cta can list awkward_kaitai as a dependency and carry the platform-specific so/dylib/dll in their wheel.
awkward_cpp is an example of a library that just carries platform-specific compiled code. It has a very thin Python wrapper around it to identify where the so/dylib/dll is, within its installation, and load it: https://github.com/scikit-hep/awkward/blob/main/awkward-cpp/src/awkward_cpp/cpu_kernels.py. In particular, importlib_resources.files(awkward_cpp) / "lib" / name finds the name of the bundled so/dylib/dll and ctypes.cdll.LoadLibrary(str(libpath)) loads it. awkward_kaitai also works by loading functions with C signatures through ctypes.

I am new to the project, I need to go step by step.
So for now I am experimenting just with plain kaitai, without awkward, see #1.
So, in this simplified case, I think the best thing would be to not commit the autogenerated code to the repository, but have Github actions generate it in order to run the tests.
Then, when we are ready to release a package on Github and PyPI, we can have a Github Action include the autogenerated file into the package, so that the user can skip the compilation step.


[minor issue] Warps and loading

Warp circles need a 1 second freeze player. some clients are not able to load interiors as fast as others as a result when they go through a warp point they fall through the map and depending on where that interior is located they appear back sometimes across the map from where they were.
freezeing the player for a second I believe would solve this as it would give the client a chance to fully load whatever interior they are being warped to

Is there a way to programmatically detect if the area is already loaded?
Other than fixing player's position in the air, we could also do something with the nativeui for the player to not see "the void". Perhaps some simple fullscreen overlay with css fade animation.

What about https://runtime.fivem.net/doc/natives/#_0x41CA5A33160EA4AB ?


Transfer memo gets cut off with Cyrillic letters

I'm not sure why, but transfer memo gets cut off when cyrillic letters are present.
String in memory doesn't get cut off, so it's probably something to do with DefaultByteWriter's method fun putString(value: String).
Here are examples of incorrect transactions:
expected: тест
actual: те
trx link
expected: тестируем testing<PHONE_NUMBER>
actual: тестируем testing 1
trx link

I reproduced the issue with the following tests
       on("transfer abi model with Cyrillic letters") {

            val transferArgs = TransferArgs(
                "memtripissu5",
                "memtripblock",
                "12.3040 EOS",
                "тест"
            )
            val transferBody = TransferBody(transferArgs)

            val output = transactionWriter.squishTransferBody(transferBody).toHex()

            it("should encode letters correctly as hex") {
                assertEquals("5034c6aeba9ba59200118da7ba9ba592a0e001000000000004454f530000000008d182d0b5d181d182", output)
            }
        }

        on("transfer abi model with Cyrillic letters 2") {

            val transferArgs = TransferArgs(
                "memtripissu5",
                "memtripblock",
                "12.3040 EOS",
                "тестируем testing<PHONE_NUMBER>"
            )
            val transferBody = TransferBody(transferArgs)

            val output = transactionWriter.squishTransferBody(transferBody).toHex()

            it("should encode letters correctly as hex") {
                assertEquals("5034c6aeba9ba59200118da7ba9ba592a0e001000000000004454f530000000025d182d0b5d181d182d0b8d180d183d0b5d0bc2074657374696e672031323334353637383930", output)
            }
        }

This is fixed in beta01.
The following tests verify the bug fix:

End to end transfer on Jungle net\
Byte writing


minor issue with service vm oneiso vm1  (trying to install to zvol)

The vm script appear to ignore the value of vm_dev_type  in the vm.conf file when booting install CD
using a command like service vm oneiso vm1
This line hard codes image in the device map
line 312:  echo "(hd0) $host_vmdir/$1/${1}.img" > $host_vmdir/$1/device.map

The workaround is to create vm1.img as a symlink to the right zvol.  Perhaps in the case of a non-img setting the mkvm.sh could create this link just as it creates a symlink to the iso.
with the symlink in place everything worked smoothly, thanks

This sounds like two other requests that I am finally testing:

  echo "(hd0) $host_vmdir/$1/${1}.img" > $host_vmdir/$1/device.map

  echo "(hd0) $vm_device" > $host_vmdir/$1/device.map

Please give that a try and see it works as expected.

Note that this will create a device.map with (hd0) set to the ISO in "iso" mode. This should be fine for setup but would probably not work if someone wanted to interrogate a configured hard drive from the live ISO.


COmpatability with Health Estimate module

Hi
You have a great little system her wich works well.
I'm trying to enhance it by using the "Health Estimate" module and getting a strange result.
In Health Estimate you can identify the object field to use for the health. i.e. actor.system.health.
This works fine (as does actor.data.system.health) for NPC's, but not for PC's ??
I've looked in the structure and cannot see any difference, but there must be something.
Can you help.
Thanks
Richard

Thanks
It was confused by trying to add temporary health.  When I turned this off,
it worked for npc and pc's.
Thanks
Richard
On Wed, 12 Oct 2022, 21:33 jsavko, @.***> wrote:

This system isn't updated to v10 yet which may be why it's giving weird
results. For PC stat try actor.system.health.value
—
Reply to this email directly, view it on GitHub
https://github.com/jsavko/vagabonds/issues/8#issuecomment-1276646153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJ3MWUWFPQ5A7QYSFV6V4DWC4HA5ANCNFSM6AAAAAAQ6ZDRHU
.
You are receiving this because you authored the thread.Message ID:
@.***>

How were you adding temporary health?

It was a setting at the bottom of health estimate configuration.
Set to default on.  Once I turned this off, it all worked .
I think as the Vagabond actor has no temp health, this was causing the
problem
Richard
On Fri, 14 Oct 2022, 15:36 jsavko, @.***> wrote:

How were you adding temporary health?
—
Reply to this email directly, view it on GitHub
https://github.com/jsavko/vagabonds/issues/8#issuecomment-1279020956,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJ3MWU3AGXR6YTJYJMGOPDWDFOXHANCNFSM6AAAAAAQ6ZDRHU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I will test with the module when I get some free time and see if I can get it enabled for you

Thanks
What Vagabond does need is a "max HP" input on the npc template, otherwise
they all default at 8.
Richard
On Fri, 14 Oct 2022, 15:41 jsavko, @.***> wrote:

I will test with the module when I get some free time and see if I can get
it enabled for you
—
Reply to this email directly, view it on GitHub
https://github.com/jsavko/vagabonds/issues/8#issuecomment-1279027045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJ3MWSSYKQ6COAVJN5OGRDWDFPJZANCNFSM6AAAAAAQ6ZDRHU
.
You are receiving this because you authored the thread.Message ID:
@.***>


chore: update dependencies

This PR will update all dependencies (including webpack for the docs page). It also updates react/prop-types, flow, eslint.
I added eslint-prettier to keep eslint and prettier in sync.
I ran all tasks without getting an error.
fyi the package-lock.json is intentional because I don't use yarn

fyi the package-lock.json is intentional because I don't use yarn

I believe the CI uses yarn. Feel free to use npm if you prefer.

I use travis in the vue-starter with npm, should be possible :-)


CQL Logic Improvements BONNIE-669

-Does not highlight statements that define populations that are not being considered.
-The logic for this is stopgap until we move it and have clause level highlighting.

What is a good alpha account and measure to test this with?

My questions were answered and I checked it out locally. looks good!

I rebased this on my local machine and noticed that the stratifications are broken. None of the population defining statements are being labeled. Looking into this now.

Upon further investigation, I found that uploading the measure again gets the population labels to come back for stratifications. I think something changed in the upload process for stratifications since the measure was loaded into the account I was testing with.

Looks good


Sprint12 Staging

Coverage decreased (-0.3%) to 80.156% when pulling 80b8277ca636b5cbf1fd04c2c65250b39b4d4abc on sprint12-staging into dce0f506dc07f27624703ddc30669503db7f72ef on master.


Would the IAO be the appropriate place to insert a "datum status" concept

This is from the European Nucleotide Archive website:
"The International Nucleotide Database Collaboration (INSDC) have developed a standardised missing/null value reporting language to be used where a value of an expected format for sample metadata reporting can not be provided." -

Would IAO be a good home for these guys?  I would actually propose a few changes, e.g. add one more category 'in process' if there was some commitment to deliver the information; as well a "recorded" status would in theory complete these rough metadata states, and a "datum status" concept to position them all under.  Right now the GenEpiO ontology has these terms but they are best positioned in a more general ontology.

Yes. See the term status instances already in IAO (ontology metadata). We might want to talk about the label for the class - datum status is rather broad. From an IAO point of view, status labels are information about what they are the status of.

Perhaps "datum retrieval state" then? I realize now that reporting that a datum instance is in some state like this would depend on a storage or intermediary system attempting to fetch a datum instance and finding the instance is in one of these metadata states instead.

I've put up a pull request for your consideratio

About term status as instances ... I get how one connects a class via "has curation status" to an instance like "requires discussion".  But I like the idea of keeping data request state a class since it is likely to be further subclassed in the future, e.g. "data obfuscated" could have subclasses "data obfuscated by randomization" and "data obfuscated by scaling" etc. But we could punn these to have owl:NamedInstance ids for those who want.
Either way, I think we can use "has quality" relation between "data item" instances and a "data request state"?  Or must we introduce a "has data request state" (domain data item, range data request state)?

can't use 'has quality' at the moment, since only independent continuants can have qualities, so a different property. There is an ongoing discussion advocating that generically dependent continuants can have dependents themselves, but consensus has not been reached.
I see the desire to be able to subdivide later. I'm not sure about whether I think its a good idea, since it is an easy path to moving things that might be more properly represented elsewhere.
For example, a more transparent way to encode obfuscation would be to have a data transformation type and represent that the data is the output of such a process.


Multiple floors indoor maps

As a user I want to be able to see multible floors and be able to switch between them.
As a developer I want to be able to import multible floors.

This should include displaying doors, bathrooms, staircases, elevators, etc. as icons/highlights.


Making Cockpit Platform Agnostic

Aim of this pull request is to start moving the platform-dependent behaviours and scripts out of the cockpit software, which should be a platform agnostic software package, and into the new openrov-pal (platform abstraction layer) package.
OpenROV-Dev/openrov-pal#1
Milestones

[x] Remove the linux/ directory and move those files to the PAL project
[ ] In PAL, restructure scripts into platform specific directories
[ ] In PAL, use system detection to automatically create soft links for core system functions to platform specific implementations
[ ] Replace any references to old scripts/script locations with new softlinks in Cockpit software

Opened as part of new branch in #71


removed inconsistent code calling convert_string_from_utf16

Coverage remained the same at 42.794% when pulling e7f4165f0b894da4c379b961a76f265c208f0bb7 on yitam:convert_string into c9b92b167931d839e84499a5dcc5a3dbb732ea0c on Microsoft:merged-codebase.

Coverage remained the same at 42.794% when pulling e7f4165f0b894da4c379b961a76f265c208f0bb7 on yitam:convert_string into c9b92b167931d839e84499a5dcc5a3dbb732ea0c on Microsoft:merged-codebase.


Add disk usage info to '{Start,Secure} Cluster'

Specifically:

This is listed as a known limitation of v1.1 and in v2.0, so we break
that content out into an include

From there, we use that include in the various places that make note
of that limitation (basically, the existing Admin UI docs)

Then, we can easily add the information to the various 'Start/Secure
a Cluster' pages by snarfing the include

Note: I tried, but was unable to add the included note to the "Start a
Cluster in Docker" pages due to
https://github.com/cockroachdb/docs/issues/2659

This change is

Also, you need to update your copy of master, rebase this branch against it, and then push again to pass TC:
git checkout master
git pull master
git checkout fixitday-local-cluster-disk-usage
git rebase master
git push --force origin fixitday-local-cluster-disk-usage

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d23267e5fca8cc734bee402e48d8d8d95f638bdc/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bac307f9f6d791f7fe7481319a733643aee8004b/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3e80ace3167360e8ef904bf33c24ffad6dada7f3/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/eacc8ede9d64d732df464a65a03f31043f6f2d97/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7076917f5ee7d9fefa58a25143497643c5131ac3/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/cbf62dceaaf498ca77c88523cf859506b561e7f7/

@jseldess I have a couple of questions:

Do you have further feedback?  I think I've addressed the issues you pointed out, but I'm happy to fix any other things you noticed

Should I ask for another review or is that not necessary since we are exposing information that already existed elsewhere in the docs, just in more places?

LGTM, @rmloveland!
Please add Fixes #<issue number> to your PR description so the related issue auto-resolves. And then merge away!


Application crashes when start writing AML script

First, this looks like a very promising and interesting product.
I have an issue with AML. Whenever I try to enter anything (it fails on the first keystroke) into Update your schema pane app crashes.

Hi @valdisz
Thanks for reporting, this is due to your grammarly extension which interract with Elm and make it break.
I'm looking for a fix but right now you can just disable it on Azimutt.

Great, tnx!


fabio does not start with multiple listen sockets

When starting fabio with multiple listen sockets it shuts down immediately with "bind address already in use"
./fabio -proxy.addr ':9999,:10000'

2017/04/28 07:19:14 [FATAL] listen: Fail to listen. listen tcp :10000: bind: address already in use
2017/04/28 07:19:14 [FATAL] http: Server closed
2017/04/28 07:19:14 [FATAL] ui: http: Server closed

git bisect identifies 5a23cb19dc64a30ee40c42bd3ec1dde289a91033 for #265 but I have a feeling that this just exposes the problem since errors are now fatal. v1.4.3 works.


Question-Suggestion RE: Recommending Downgrading to 9.2.0

Hi,
My question/suggestion relates to Step 22 on https://github.com/Plailect/Guide/wiki/Homebrew-Launcher-(Browser).
"All versions above 9.2.0 should do 9.2.0 Downgrade at this point, before proceeding"
Could this be expanded on some to include advantages/disadvantages and why it's recommended?
I'm on an o3DS with 9.9.0 w/ HBL set up and debating on whether I should downgrade and risk complicating things for myself since I do have some DSiware and a lot of DLC that I don't want to risk losing. Is it possible to continue without downgrading and what are the trade offs?
Extra information would really help. Thanks in advance!

Let me get this out of the way:
If you don't downgrade to 9.2 you can't install A9LH period.
This is due to there being no kernel exploits above 9.2.
Anyway..
Downgrade: Use a permanent entrypoint (A9LH) that allows instant bootup of CFW and homebrew but still lets you update. Best option.
Don't downgrade:
Be unable to use CFW completely. Also if you update (or don't) you have to deal with:
Entrypoints getting patched. So sometimes you might not even be able to use homebrew for a certain amount of time.
Since there's no CFW you can't boot HBL from the home menu.
You're limited to only "legit" CIAs which there aren't many of.
Frankenfirmware.(?)
Eshop, online games, etc not working.
No kernel so homebrew is MUCH more limited, and slower in some cases.
Without A9LH you won't be able to unbrick without using a hardmod.
That's all I can think of right now. There's probably other cons I didn't mention.
tl;dr downgrade to 9.2


Add opt_variables re-initialization routine.

Jenkins: Automatic build-test is triggered.

Jenkins: Automatic build-test is started.

Jenkins: Automatic build-test is successfully finished.


Deployment docs and/or assistance

As far as I can tell, there's no documentation surrounding how to actually deploy this server outside of an evaluation phase. This would make what I'm currently doing (attempting to deploy the API and UI to Cloud Run) a lot easier.
So far I've been winging it by using the script and then uploading Docker images to my Google Container Registry, which I can then deploy to Cloud Run. This worked in the past, but recently I've been getting this frustrating error in my deployed API instance:
2021-12-08 18:21:10.818 EST(node:13) UnhandledPromiseRejectionWarning: Error: Query engine exited with code 255
2021-12-08 18:21:10.818 EST
2021-12-08 18:21:10.818 EST at ChildProcess.<anonymous> (/node_modules/@prisma/client/runtime/index.js:27551:23)
2021-12-08 18:21:10.818 EST at ChildProcess.emit (events.js:315:20)
2021-12-08 18:21:10.818 EST at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)

This doesn't happen when I run the ./vrt-install.sh script or whatever locally, so I'm assuming it has something to do with my configuration. But without any guide whatsoever on how one might deploy this in the real world, I can't tell what I did wrong.
Other potentially relevant config (but how would I know that? these settings aren't documented anywhere!)

I'd love to be able to use this, but I'm just not sure how.

@tubbo thanks for the request
minimum required configuration for remote machine is to update URLs
you could find video of how to set up VRT via ssh https://www.youtube.com/watch?v=cPsHeKnqL2M
sorry, can't say anything looking on that error :(
make sure you are using latest container versions