Assertions within repeated forEach loops throw implicit Selector resolution errors

What is your Scenario?
I'd like to execute the same assertion in a forEach loop to test the contents of every item in a list, update the list in some way, then execute another loop with the same Selectors to check the elements have changed as expected.
What is the Current behavior?
The test reports that it passes, but throws implicit Selector resolution errors, e.g.:
Fixture
 ✓ list reverses successfully

 1 passed (2s)
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.

What is the Expected behavior?
The test should actually pass without throwing errors.
What is your public website URL? (or attach your complete example)
Minimal test case here:
https://github.com/mikejamesthompson/testcafe-selector-resolution-errors
What is your TestCafe test code?
import { ClientFunction, Selector, t } from 'testcafe';

const results = ['1', '2', '3'];
const resultsReversed = results.concat([]).reverse();

fixture`Implicit Selector resolution in async forEach`
  .page`http://localhost:8000/index.html`

test('list reverses successfully', async () => {
  const listItems = Selector('li');
  const iterator = Array(3).keys();

  Array.from(iterator).forEach(async (i) => {
    console.log('Loop 1:', i);
    await t
      .expect(listItems.nth(i).innerText).eql(results[i]);
  });

  const reverseListItems = ClientFunction(() => {
    const ul = document.querySelector('ul');
    const li = document.querySelectorAll('li');
    const f = document.createDocumentFragment();
    for (let i = li.length - 1; i > -1; i -= 1) {
      f.append(li[i]);
    }
    ul.append(f);
  });
  await reverseListItems();

  Array.from(iterator).forEach(async (i) => {
    console.log('Loop 2:', i);
    await t
      .expect(listItems.nth(i).innerText).eql(resultsReversed[i]);
  });
});

Your complete configuration file
N/A
Your complete test report
Note that logging from the first loop only appears, which suggests the second loop doesn't execute at all despite the test apparently passing.
> testcafe chrome:headless ./test.js

 Running tests in:
 - Chrome 101.0.4951.41 / Linux 0.0

 Implicit Selector resolution in async forEach
Loop 1: 0
Loop 1: 1
Loop 1: 2
 ✓ list reverses successfully

 1 passed (2s)
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.
Cannot prepare tests due to the following error:

Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.

Screenshots
No response
Steps to Reproduce
Use the repo:
https://github.com/mikejamesthompson/testcafe-selector-resolution-errors
TestCafe version
1.18.6
Node.js version
v16.0.0
Command-line arguments
testcafe chrome:headless ./test.js
Browser name(s) and version(s)
Chrome 100
Platform(s) and version(s)
macOS 10.15.7
Other
No response

Please do not use the forEach loop with promises that should be awaited. The forEach loop does not wait until all promises are completed. This approach will not work. Please refer to the following StackOverflow question: https://stackoverflow.com/questions/37576685/using-async-await-with-a-foreach-loop. It explains the problem in detail.


Implement discounted cash flow analysis

Includes the various types of free cash flow (FCF, UFCF, LFCF, etc)

Should include editable data.table for the variables that get pulled in. Use calendar input to select years? I believe it can be modified to only include years.


EIP-7685: exclude empty requests in requests list

What was wrong?
The following PRs include breaking changes in the behavior of requests for devnet-5:
https://github.com/ethereum/execution-apis/pull/599
https://github.com/ethereum/EIPs/pull/8989
How was it fixed?
Remove requests elements from the requests list if the byte array is empty.
Also do not remove the request_type byte in the return list of the T8N result since it's now necessary to calculate the requests commitment.
Cute Animal Picture

I see that the tox issue is from running our latest generated blockchain tests, which makes sense since we are altering the expected hash in the header of any Prague test, but to generate a new set of tests we need this PR.
It's an interesting dependency problem.

Can you generate and release new tests before this is merged, or should I force merge this?

Can you generate and release new tests before this is merged, or should I force merge this?

I think our option could be to combine all required branches into a temporary forks/prague branch in some of our forks, make the release by pointing to that, then re-run CI in this PR.

This has now been pointed at the devnets/prague/5 branch. Tests will be disabled in that branch until EEST generates prague-devnet-5 fixtures.


Add opengraph balises

Theses opengraph's balises should be added :
og:title
og:type
og:image
og:url

If the page is an article :
article:published_time
article:author
article:section OR article:tag

Ref : http://ogp.me/

og:title : title of the page
og:type : article or website
og:image : featured image if it's an article, or a logo if not available
og:url : url of the page
article:section OR article:tag : Add Section on the frontmaker or use tags


Abstract routes for implementing layouts

How to I implement multiple layouts that are parents for sub set of routes by default?
E.g: I have some routes for pre-login users that has one layout theam and some routes for post-login users with different layout theam.

+1

+1

To better understand – you want the same URL, but totally different content based on app state?

https://github.com/angular-ui/ui-router/wiki/Nested-States-and-Nested-Views#abstract-states

Yes, I know UI router has a thing called "Abstract Routes."
I'm curious what the use case is.

Sorry for the hasty reply. I was in a meeting. I use them quite a bit to
serve as containers/groupings for child states. They allow you to set a
base path, template and logic that is used by all child states without
allowing that state to be accessed directly (you can only access a concrete
child state). A good example of this is unauthenticated and authenticated
states. The two templates are very different, and one important piece of
logic is as well (do not allow unauthenticated users access to
authenticated states). But you wouldn't  want to have 'authenticated' and
'unauthenticated' in your URL.
Another use case is for child resources. Say I have a state at the path
clients/1. In the abstract state I request data from the server for that
client so that all child states have the client data. I could also create a
template that adds to the breadcrumb or adds a 'Clients' heading to the
page. I could create a child concrete state that has the same path as the
abstract state which lists details about the client. Since this has the
same path as the abstract state, this can be thought of as the default
state. I can then create another child state at path clients/1/friends to
list the clients friends.
On Thu, 30 Apr 2015 5:54 pm Brian Ford<EMAIL_ADDRESS>wrote:

Yes, I know UI router has a thing called "Abstract Routes."
I'm curious what the use case is.
—
Reply to this email directly or view it on GitHub
https://github.com/angular/router/issues/203#issuecomment-98011320.

here I am referring Layout as a abstract component (with a html template with ng-viewports and optional controller)
all my routes (pages)   will inherit one of the available Layouts
Layout will never be associated with a route  since it is an abstract component.

+1

+1

+1
I'm using this feature from ui-router and want to upgrade my angular 1.5.x based application to angular latest version 5.x or 6.x but latest angular router still doesn't support this feature.
Is there any plan to implement this feature?


Add more Codec functions

Currently, there are two codec functions: functions(x) and function2(x) but only fucntion(x) is being used for encryption/decryption.
Design more mathematical functions, preferably four more: where encryption occurs sequentially function-> function2 -> function3 -> function4 -> function5 and Decryption occurs reverse-sequentially function5-> function4 -> function3 -> function2 -> function
This ensures that the overall process becomes more chaotic and the original and encrypted images less alike.

Assigned to @arulthileeban


Does it check automatically

I'm just curious as to to how it works. I installed the extension. I see it takes time, but I'm unsure if there are steps I need to take to initiate a check and see if there are any issues.

You just open a HTML file.


Various pxe fixes

What this PR does / why we need it:
ignition-fetch.service is starting too soon with the default configuration, it should start after network-online.target
systemd-networkd-wait-online.service should not start only if /run/networkd/initrd/neednet exists^1
when fetching the squashfs, retry multiple times and retry on all errors
Which issue(s) this PR fixes:
Special notes for your reviewer:

Multiple tests were successful, ignition files were fetched correctly (used to fail with newest systemd before the fix)

@5kt I'll leave it to you to merge when you're confident its good
thanks!


The Version 3.34 LTR cannot be downloaded on my windows laptop. Error code 522

URL
https://download.qgis.org//downloads/QGIS-OSGeo4W-3.34.11-1.msi
Type(s) of Problems
Broken Link
Summary
The Version 3.34 LTR cannot be downloaded on my windows laptop. Error code 522 (Host Error) appears every time (firefox, edge, chrome)

The same happens with version 3.38. Connection timed out Error code 522

Same on Mac files all versions

Today it could have been downloaded.
But afterwards, when opening QGIS, this error has been displayed:
"Open Sans font installation failed"
Error transferring https://github.com/google/fonts/raw/main/ofl/opensans/OpenSans[wdth,wght].ttf - server replied: Not Found
What can I do?

@EP23EP

"Open Sans font installation failed"
Error transferring https://github.com/google/fonts/raw/main/ofl/opensans/OpenSans[wdth,wght].ttf - server replied: Not Found

Does the issue still occur? I think you should report it at https://github.com/qgis/QGIS if it's related to QGIS Desktop


work out how the redirects would work

I'm trying to run an instance of seamlessaccess.org (https://github.com/TheIdentitySelector/thiss-js)
Separately I'm looking at how I log in to Outlook at SURF:
https://outlook.office.com automatically redirects me (without user interaction) to
https://login.microsoftonline.com/ which shows an account selector
I choose<EMAIL_ADDRESS>and I'm redirected to
https://login.surf.nl which automatically redirects me (again without user interaction) to
https://tiqr.surf.nl for MFA
Then a few more quick redirects (same path back?) before getting back to https://outlook.office.com


Having some trouble getting this setup and working

Can you show the starting html setup for the demo page? I can't seem to make this plugin work out of the box like your example shows.
I'm trying to develop a mobile app type of website that has a fixed menu bar at the bottom that I want the priority nav to attach to. This app is built in ember.js and the resulting output from ember without trying to init priority-nav looks like this

And when I initialize the plugin the initial ul element has no items in it and they have all moved into the dropdown I presume it is, but the demo site shows they should still be in the main ul before priority-nav attempts to move them. So I'm not sure what's happening here.
 The  element has to be a block element. And the ul inside the nav has to be inline-block or something that doesn't behave like a block element.

Yes the css is loaded. The nav is a nav element and the ul is set to inline-block

Mmh, weird. I can't tell you much more without seeing the actual code. Can you create a codepen?

I will see if I can port it over without the remaining context of the app

I have it setup here
http://codepen.io/anon/pen/MaRpyN
This is pretty much what I'm seeing in my app

@Jordan4jc The ul is not inline-block. It also has width: 100% on it. This causes the nav & ul to be the same width. The plugin checks how much space there is between the ul & the nav. Once its equal it will move the last li item to the dropdown element. Resulting in repeating the same action untill there are no li element's left.
So set it to inline-block and not to flex & remove the width property.
Like this: http://codepen.io/gijs/pen/PPgpyX

When I bring the codepen window down to around 400px it removes all but one of the icons and the menu button doesn't open anything (though I believe that may just be a CSS issue?), what is happening with all the icons being removed instead of just the one at a time when the window is small enough

Yes thats the default behavior. You can disable it by settings the breakpoint to 0. Take a look at the options :) https://github.com/gijsroge/priority-navigation#options
The toggle works, but this requires css to hide/show based on the classes the plugin provides while toggling.

Cool it's starting to take shape finally. I see that where you mention it needs to be inline-block in the readme you say something about flex. How does all of that work? Also I'm trying to get the menu button to do something like this image
![Menu Example]
(http://drive.google.com/uc?export=view&id=0B-eKT0Z_vxfHWDNqWks4VWJCOGM)
Preferably where the menu is all the way on the right of the screen at all times (floating) and the items can be flexible within the UL or something like that. Do I need to have a wrapper to add the flex display to?

Yes that is very much doable. Instead of me explaining it fiddle around with this -> http://codepen.io/gijs/pen/PPgpyX?editors=010 it should explain itself. If you have any questions just ask :)

Thanks for your help. Unfortunately I don't think I'm going to be able to make your awesome plugin work for my edge case. I have been fiddling with it and not really getting the results I want.
Ideally the menu toggle button would be positioned to the right all the time, and the navigation items would flex with space around to fill in the remaining space minus the menu button so nothing got hidden behind it.

Indeed. You can't have a fullwidth menu with this setup. It has to calculate the remaining space between the parent and the menu to move its items when needed. (you can with some media querie tinkering but that requires you to manually remove the space between on a certain width)
I added some colors to make it extra clear. It looks at the green color and check if there is still space for the menu. If there is no green left it will move one item to the dropdown.
http://codepen.io/gijs/pen/PPgpyX?editors=010


New feature generation script

Everything solved :)

You could also add a metadata.json in the root feature-generation directory, so that both scripts are created when using it, right?

On Sun, Jul 07 2019, mmerce wrote:

You could also add a metadata.json in the root feature-generation
directory, so that both scripts are created when using it, right?

could add -> must add :)
(everything in a top-level subdir must be a package)

Done!


JWT token revocation does not work in IS 5.6.0

Tested with a WUM updated pack of IS 5.6.0 (timestamp<PHONE_NUMBER>980).

Configure JWT access tokens by following https://docs.wso2.com/display/IS560/Self-contained+Access+Tokens
Create an OAuth app and obtain a JWT access token
Introspect the JWT token
https://docs.wso2.com/display/IS560/Invoke+the+OAuth+Introspection+Endpoint
Revoke the JWT token
https://docs.wso2.com/display/IS560/OAuth2+Token+Revocation
Introspect the JWT token again.

It will still return the response indicating the token is active. The expected behaviour is that the token needs to be identified as inactive.

Not reproducible in IS 5.7.0 (latest WUM updated pack) onwards.


[name].min.[extension]

Hi, thanks for your project!
I think that brackets-clientlibs-combine is a simple and great plugin.
I try to write this (like brackets concat plugin) in my clientlibs.combine, but obviusly doesn't works:
{
"combineOnSave": true,
"minify":true,
"combine": [
{
"output": "public/js/home.min.js",
"files": [
"resources/assets/script/component/layout.js",
"resources/assets/script/page/contact.js"
]
}
}
Is there some way to minify automatically the output?

Currently there is no support for minification of output files.  Please feel free to extend this plugin and create the feature yourself.

Would be great to add this feature : minify and combine in one operation.
Thanks.


Document check-coverage thresholds usage

If I use the following .istanbul.yml file:
check:
	global:
		statements: 90
		lines: 90
		branches: 90
		functions: 90

then I never get a fail, even though my coverage is 70%.
However, if I run istanbul check-coverage --statements 90 then the test fails as expected.
I discovered this usage only thanks to #192. It seems important to document the usage of check-coverage (and possibly fix the YAML config loading, but I'm not sure I understood it properly…).

Are you actually running the check-coverage command when you use the istanbul.yml file?
Note that setting the thresholds does not automatically run the check coverage command for you. It only makes it easier to put all your thresholds in one place.

Thanks for providing this information. I understood this on my own after some time. The main issue here is really about documenting the feature usage.

why is this still not documented?

check-coverage:
  statements: 50,
  branches: 50

doesn't work
reporting:
  check-coverage:
    statements: 50,
    branches: 50

doesn't work
what's the proper syntax?

check:
    global:
        branches: 100
        functions: 100
        lines: 100
        statements: 100

Works!  Make sure file is .istanbul.yml and not istanbul.yml as well.

(and you call check-coverage; the definitions given by @niftylettuce are there only to configure the command, not to run it)


insertion sort using C

This program use insertion sort method to sort an array.

You can follow me on GitHub and give this repo. a ⭐


Update image??

How can I update image with react table 2. I have a component get base64 from image and I want to show it to update image but react-table-next-editor don't have Type.IMAGE. Can someone help me solve this problem.
Thank you everyone so much!! ♥

@lockhaicttv You'd be better off creating a custom edit component and writing a custom wrapper:
{
   dataField: 'image',
   formatter: (cell) => <MyCustomImageEditingComponent image={cell} />
}

Tks so much


Add python 3 compatibility

Hi,
since I want to use this library with python 3 I added support for it and added py3 to the list of runtimes in tox.ini.
Three fixes were necessary:

Imports: mainly to do with the new urllib submodules
String/bytes difference: had to add a few encode/decode
Import path in the vendor'ed mulitencode module

Please review this and give me feedback. All tests pass.
I'd really like to get this merged!

Hi @lenniboy and thank you for this PR!
I'm sorry it has taken so long for me to look at it. It's looking good to me at a quick glance, and it's good motivation for me to get the gocd-cli to support Python 3 as well. :)
I was about to merge this in but hit a snag with VCR and Python 3. It's starting to get a bit too late to get it merged in tonight, but I'll do my best to get it through this week!
 I'll have to do some manual testing before I feel comfortable clicking the big release button.
…really need to get that integration suite going :P

Good stuff!
If there is anything I can help out with, let me know!

28 days later and this is finally merged :smile:
I've tagged and pushed a new release v0.9.0 with Python 3 support. Thanks again @lenniboy!


adafruit-nrfutil: remove nose and modernize

Description of changes
Removes nose and just uses pytest's compatibility layer for tests.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Uhh, by the way, this should apparently be lib.licenses.unfree…

sigh good ol' licensing issues. I'll update it.

I misspoke slightly, it’s at least lib.licenses.unfreeRedistributable. Sort of limits the utility of having this packaged, but hopefully upstream can resolve it.


DOC: indicate functions with multiple valid solutions in csgraph

Issue #17734 arose because an implementation change in Dijkstra's algorithm led to a change in returned solution, in a case in which multiple valid solutions were possible; the consequence was breaking test cases in user code. This problem is common in algorithms in csgraph. The purpose of this PR is to improve documentation to reduce confusion by adding a note for each function that could have multiple valid solutions.
This is my first pull request.
I did my best to review all of the algorithms and judge which had multiple valid solutions; I may have made a mistake on a couple but the stakes are not that high and the original algorithm (Dijkstra's) is now correctly noted. I only put notes on individual functions and not the whole library because I'm imagining that someone facing a similar issue in the future would first look for help on the individual function they're using.
[skip ci]
I can add CI later for docs if needed.

I didn't mean for CI to run. I had originally written,
[skip ci] I can add CI later for docs if needed.
as one line but CI ran anyway. Should these have been on separate lines? Is there any way to cancel it? Because this is just a draft and a doc change, I don't think full CI is needed.

Thank you for the PR @AlexSzatmary and for thinking about the CI.
The skip message has to appear in the commit message itself. It seems that it's only present in the PR description (which does nothing.)
So new commit messages should look like:
DOC: update some docs.

[skip actions] [skip cirrus]

You still want CircleCI to run, so you only need [skip actions] [skip cirrus]

Hi! I want to make sure that I'm not holding up this PR. Is there anything I need to do now? The check that failed did not seem related to this issue.

I think it just needs some review. @ivirshup can I interest you with that?

I'm not super available for the next couple weeks, so am afraid I can't take a super close look.
It looks fine at a glance, but I haven't checked to make sure that (a) the added statement is true for each function or (b) whether there are other functions where this should be added.
I do wonder if this could be noted more centrally, like in the module docs.

No worries :)

I do wonder if this could be noted more centrally, like in the module docs.

This might be a good alternative 👍

Thanks for the review. It would definitely help if someone could double-check that the correct functions are noted, as suggested by @ivirshup .
As I noted in my first post in this PR,

I only put notes on individual functions and not the whole library because I'm imagining that someone facing a similar issue in the future would first look for help on the individual function they're using.
I'm not sure it's necessary but I'm open to adding a general note about this issue in csgraph, probably in the reference:
https://scipy.github.io/devdocs/reference/sparse.csgraph.html

Friendly ping @ivirshup would you have the bandwidth to check if this is applied to the correct functions?

Thanks, @jjerphan for the review and suggestions. I have no strong opinion on your two main suggestions:

Don't mention change with Python version. I'm using language suggested by @perimosocordiae . I suppose that Python version change could affect things, but it would be unlikely.
Use a warn directive. I'm simply not familiar with the norms here.

For either point, I think the stakes are low and it's best to not overthink. This PR addresses a narrow issue—exact results could change unexpectedly when updating SciPy version but I can't think of why that would matter outside of needing to change reference cases in a unit test. The user needs to do nothing different to get a result that is as valid as before. I think this is worth doing because it could help someone when they're troubleshooting in this exact circumstance.
I'd like opinions from others and will implement the consensus.


[DRAFT] Import wasm support from snapchat repo

We need to provide a generator with the changes, I guess
My current plan is to switch to a CMake fetch_content way of working.
This will fetch a build artifact from a PR, which can be used until the change in the generator has been released.
Please be a little bit patient. This might look like a lot of what is happening, but this will set the base for a maintainable workflow for the following years. And I wish that the WASM functionality would be part of it.
I will work as quickly as possible and help as much as my day job and family situation allow me to do.

btw the CMake part to get the generator looks atm like this to me
set(Djinni_URL "https://github.com/cross-language-cpp/djinni-generator/releases/download/current-latest/djinni")
set(Djinni_DownloadPath "${CMAKE_BINARY_DIR}/djinni")

# get the generator and make it executable
if(NOT EXISTS ${Djinni_DownloadPath})
    message(STATUS "Downloading djinni from ${Djinni_URL}")
    file(DOWNLOAD ${Djinni_URL} ${Djinni_DownloadPath})
    file(CHMOD ${Djinni_DownloadPath} PERMISSIONS OWNER_EXECUTE OWNER_WRITE OWNER_READ)
endif()

# run djinni to see the command exists and works
execute_process(COMMAND ${CMAKE_BINARY_DIR}/djinni --version RESULT_VARIABLE result OUTPUT_VARIABLE output ERROR_VARIABLE error)
# print the result so we see what's going on and what works or not
message(STATUS "\$?: ${result}")
message(STATUS "djinni --version: ${output}")
message(STATUS "Error: ${error}")

This gives me the latest developer release (current main branch)
For getting the build artifact from a pipeline, this needs to change a bit,
since the pipeline will upload a zip of the generator, and fetch_content might be the better choice for handling that
I will check and report back.
Something like this will be the (from me) recommended way of using the generator.
Of course, with a checksum check.

Downloading a pipeline artifact is not as straightforward since they are not publicly available.
it only works as an authenticated user, which is not that awesome, but solvable
https://docs.github.com/en/rest/actions/artifacts?apiVersion=2022-11-28#download-an-artifact
I don't know right now what the most user-friendly way would be.
Getting the generator from a PR for a PR in this repo
I will stay on it since this is a trivial problem that is only complicated due to the GH infrastructure.
And it's been bothering me since the repo split.
(Sorry for abusing this PR to brainstorm for a solution, But I see this now as a good motivator and use case to invest more energy in that situation finally)

I just checked. Downloading a build artifact works
ARTIFACT_ID=1049780945

curl -L \
  -H "Accept: application/vnd.github+json" \
  -H "Authorization: Bearer $TOKEN" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  https://api.github.com/repos/cross-language-cpp/djinni-generator/actions/artifacts/$ARTIFACT_ID/zip --output djinni.zip

The artifact_id can be read from the download link,
The $TOKEN, I can add one to the CI settings, but it will expire from time to time,
Then unzip djinni.zip, give it x permission, and everything is fine.
(on Windows rename to bat)
I make settings on the repos to make it possible to generate fine-graded personal tokens that only are allowed to download a build artifact. That's good since that means I can add my token to the CI
However, I have to think about how to most elegant enable that in a ci run of the support lib.
But it is not a problem technically, it's just a question of what the 'best way' would be.
The dynamic properties of the artifact ID generation make that a little bit inelegant, but at least there is a way
Maybe there is even a chance to get the latest artifact to a PR, that would be nice, will continue look at the github API docs
It's nice to finally have a reason to dig into that situation :-)

I just noticed that the test-suite unit tests for wasm/ts are not being run on the snapchat CI. I hope we can get it running here.

The unit tests are running locally. So all that remains (??) for this PR to complete the basic wasm support, I think, would be to run the wasm tests on the CI - which as I wrote above, snapchat-djinni is not doing currently - and add some kind of simple example which uses it.

Thanks a lot, Alexis, this is now in an exciting state! I am really happy and thankful for the process.
About the open points:
run wasm tests on CI (and examples)
It seems that would require a headless chrome instance?
Not sure how to verify the examples. They are defacto also tests, but would need GUI.
I know it's possible to run both, Android and iPhone simulator/emulator headless, but would need to look that up.
It would be awesome to have that, but this requires more investigation. And it will drastically extend CI times. But soooo nice to have.
documentation
That might be the easiest part, I will happily contribute it, once I had time stepping through the additions.
generator build:
I am on the task of providing different ways for getting the generator into the support lib builds
The conan way is dead, and I will add a CMake download.
That can be the latest developer release, the latest tagged release, or, and this is what is missing, an artifact id
(artifact from an action as attached here: https://github.com/cross-language-cpp/djinni-generator/actions/runs/6964234110)
It's unfortunate that getting the artifact is on github more complex than required, but there is a way.
DjinnniModule.ts/js files
Could they be 'generated' by the generator? As far as I seen, those are just one liners, one for js one for ts, or do I miss something?
The Python bindings would require the same, but in the Python case the generated code would need adoption depending on the IDL input. So it's not static. I guess the js/ts is static and never changes?

as far as I see no real blocker, but some decisions need to be done, and some time for implementation need to be found, want might be the bigger problem, but I have that now on the top of my prio list

I started with a Python script to get the generator from a PR; it works locally with an access key in the environment. See #77
Unfortunately, GitHub requires such a workaround, but at least we can do it.
python ../get-generator-from-pr.py 157 will download the last successful build from the pr here: https://github.com/cross-language-cpp/djinni-generator/pull/157.
We need to setup the CI/CD env, so it has a key and then check that this temporary solution will not go into main,
but when everything is ready, merge generator PR and then switch to the latest developer release of the generator.
I hope to be able to do that, but my calendar is very stressful at the moment, so no promise, but I will do my best.

WASM/TS tests are now running in a headless chrome instances, and failing test output is captured from the browser and dumped to the terminal.

DjinnniModule.ts/js files
Could they be 'generated' by the generator? As far as I seen, those are just one liners, one for js one for ts, or do I miss something?

They can be created by the generator. This was my preferred approach, but in the generator integration tests we generate multiple sets of sources from djinni IDL, and for each DjinniModule.ts/js was being created - leading to problem with the cmake. Not sure how best to work this, but am tempted to make it so that whether or not to generate the DjinniModule.ts/js files can be specified as a command line parameter, and thus we can make sure only one set gets generated.

This looks pretty nice now!
I will have to give it a closer look, though.
Besides providing a unified way to get the wanted generator in a PR here, I also want to check that all the samples work and see if they can be used. Also, should they stay here, or should we move them into the https://github.com/cross-language-cpp/djinni-example-cc project (which should be revitalized as the demo project that utilizes all various generators)
Super exciting, the outcome of that PR could be not only a new language generator but also severe infrastructure improvements and modernizations.
I have a super busy week ahead, so please be patient if I am not immediately available for further actions in the upcoming days.

I have a super busy week ahead, so please be patient if I am not immediately available for further actions in the upcoming days.

I'll probably slow down a bit. I'm keen to get this merged, and I think most of the hard work is done, but I also have a lot on my plate.

I am sorry, but looking at my calendar, it is very likely my review must wait until the Christmas holidays.
Also, adding the generator selection in the pipeline is easy to do but requires some time.
I want to prioritize these tasks, and I will try to do it.
Thank you for your understanding and patience.

Had today time, tried various things,
deciding between the generator from the latest release, or the pre-release is possible.
Getting a generator from a PR is technically possible, practically not because atm I see now way to pass the access token to the PR. And strangely, github wants an access token when I want to download a build artifact via the API
See
https://github.com/cross-language-cpp/djinni-support-lib/pull/77#issuecomment-1868088924
Need to think a bit, maybe we can live with pre-release and merge the PR on the generator, of we have some other idea.
Other people do some crazy workarounds to skip this limitation, not sure if I want to adopt one of those

FYI:
I hope there should soon be a way to get the support lib, and generator workflow synced, and it will be possible to show a supporting lib works with a new generator without having a generator released.
Details: https://github.com/cross-language-cpp/djinni-generator/discussions/94#discussioncomment-7987560


major fixes

Turns out, there was no way that the links and photos could have worked properly during the demo no matter what, just because of how things were set up to begin with. @calisio did an awesome job, everyone killed it just being able to get up what we did in time for the demo :muscle:
PHOTO of freshly seeded local data that @calisio and I loaded in yesterday.

Description
This PR fixes lots of unnecessary duplication of fields between courses and enrollments (we were storing provider_platform_id on the enrollment but not the course?), and we were storing the provider_course_id on both the course and the enrollment, and naming it two different things? (provider_resource_id and provider_course_id?) There was no consistency with types, as provider_platform_id was a string and didn't actually reference the table for a relationship. Several other fields were of the wrong type, provider_course_id was an integer in one table and a string in the other.
A lot of these changes were hard to catch until the front-end showed up. Honestly I noticed something at 11:30 after trying to get the photos to show up locally, and ended up going down a rabbit hole of bugs. Here we are 11 hours and many coffees later :P
Some notes on the assorted changes:
Enrollments/Courses:
I'm not against having a links field on the enrollments, but for the current use where its simply being used as the image URL for the course, I think that belongs on the course and not the enrollment (as it can be used for many courses), and in the future if we need many or multiple links for the enrollments, i think they would be better off in their own table, instead of storing JSON which is prone to syntax errors, and causes json parsing/serialization to happen 3 different times, we are casting or asserting JSON in our PHP code, that means its deserializing it, then mysql does a syntax check to assert the json syntax. If this was raw json we were storing that came directly from somewhere like canvas, then I think that would make sense, but i wasn't able see a links field or any corresponding field that's coming directly from canvas.
Actions/CanvasServices:
As these are RPC's, I felt that they belong in the /actions URI, and pulling data out of the path makes less sense for actions, which can have a well-defined JSON body outlining the data/actions being requested. The action to grab the courses required the User ID, which are agnostic to users, unlike enrollments (I left a user specific Courses action in case it's needed, but also added one that grabs all the accounts courses). Also they did not account for the fact that CanvasServices is (was) a direct 1/1 of canvas's API calls, so it was feeding our User ID's to canvas as if they were canvas user ID's. I altered the behavior of canvasServices to work directly with our User ID's from now on.
Also, only the root level API key is allowed to query all enrollments for a user. Not even the Site admin key that is currently hooked up to the prod account was able to call that funciton for another user. However, I found a work-around by grabbing the courses and fetching the enrollments one-by-one.
note: All these changes required editing various parts all over the code-base that depended on the existing structures (tests, requests, resources, etc). I'm sure I made a mistake somewhere and I'm sure I'm forgetting something about a change I made. Please let me know if you'd like further explanation.
Tests all pass, build looks good locally :+1:

wtf these tests are absurd i just want to go to sleep. literally haven't failed locally one time. the gh runner must use some ancient libsqlite.so


[DROOLS-1175][DROOLS-1242] - jbpm simulation adjustments

@psiroky @tsurdilo this should fix the problems on master with failed tests of jbpm simulation

+1


Сan not scroll list of layers in Google Chrome

Leaflet version I'm using: 1.0.2
Browser (with version) I'm using: Chrome & FF
OS/Platform (with version) I'm using: Windows 10

I can not scroll list of layers in layers control. Error in chrome 55, but all is well in Firefox 50. View gif.  In Leaflet 0.7.3 works fine.
Chrome

Firefox

This might be related to the troublesome PointerEvent logic of #5180.
Just in case, can you try with Leaflet 1.0.3?

Screen from http://leafletjs.com/examples/layers-control/example.html, which is used 1.0.3

Can confirm. I've left a playground at https://playground-leaflet.rhcloud.com/vifa/edit?html,output

According to https://dom.spec.whatwg.org/#dispatching-events , step 18.1, the behaviour that Chrome>=55 exhibits is the right one: if any DOM parent of a element runs preventDefault() on the event while it is bubbling up (which is steps 12-13), an event flag is set, and that flag must (or should) be checked at the end of the browser's event handling logic.

A simple example like at http://jsbin.com/mefojojugi/edit?html,output displays the proper behaviour in other browsers, so I don't know which parts of Leaflet made this work until now. :confused:

I think this is beacause scroll propagation is disabled only for not-touch browsers:
https://github.com/Leaflet/Leaflet/blob/v1.0.3/src/control/Control.Layers.js#L166
But in last chrome version L.Browser.touch is true for desktop version.
This is sad as this flag is used in many places. For example zoom control is now enlarged in desktop Chrome (see gifs in issue)

Good catch! I was able to make a cleaner PR.
Regarding L.Browser.touch in Chrome, do read https://github.com/Leaflet/Leaflet/issues/5266 and https://github.com/Leaflet/Leaflet/issues/3944


python311Packages.file-read-backwards: 3.0.0 -> 3.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.file-read-backwards is: Memory efficient way of reading files line-by-line from the end of file
meta.homepage for python311Packages.file-read-backwards is: https://github.com/RobinNil/file_read_backwards
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.1.0 with grep in /nix/store/f87kx0bvihi77q3svy2z8v0rjk8r01hg-python3.11-file-read-backwards-3.1.0
found 3.1.0 in filename of file in /nix/store/f87kx0bvihi77q3svy2z8v0rjk8r01hg-python3.11-file-read-backwards-3.1.0

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

home-assistant-component-tests.file
python311Packages.file-read-backwards
python312Packages.file-read-backwards
tribler

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f87kx0bvihi77q3svy2z8v0rjk8r01hg-python3.11-file-read-backwards-3.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.file-read-backwards https://github.com/r-ryantm/nixpkgs/archive/132b9976d9c2f527d7cb7428877e2e516db1044c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/132b9976d9c2f527d7cb7428877e2e516db1044c#python311Packages.file-read-backwards

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f87kx0bvihi77q3svy2z8v0rjk8r01hg-python3.11-file-read-backwards-3.1.0
ls -la /nix/store/f87kx0bvihi77q3svy2z8v0rjk8r01hg-python3.11-file-read-backwards-3.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

6 packages built:

home-assistant-component-tests.file
python311Packages.file-read-backwards
python311Packages.file-read-backwards.dist
python312Packages.file-read-backwards
python312Packages.file-read-backwards.dist
tribler

Maintainer pings
cc @j0hax for testing.

Result of nixpkgs-review pr 309757 run on x86_64-linux 1

6 packages built:

home-assistant-component-tests.file
python311Packages.file-read-backwards
python311Packages.file-read-backwards.dist
python312Packages.file-read-backwards
python312Packages.file-read-backwards.dist
tribler


[Event Request] Report 5851 "Suggest Item Standard Cost" - OnInsertStdCostWkshBeforeInsert

Please add one new Publisher to Report 5851 "Suggest Item Standard Cost":
    [IntegrationEvent(false, false)]
    local procedure OnInsertStdCostWkshBeforeInsert(var ToStdCostWksh: Record "Standard Cost Worksheet"; RoundingMethod: array[3] of Code[10]; AmtAdjustFactor: array[3] of Decimal;)
    begin
    end;

Change Function "InsertStdCostWksh":
    local procedure InsertStdCostWksh(No2: Code[20])
    begin
        with ToStdCostWksh do begin
            Init;
            Validate("Standard Cost Worksheet Name", ToStdCostWkshName);
            Validate(Type, Type::Item);
            Validate("No.", No2);

            ....
            ....
            ....

*           OnInsertStdCostWkshBeforeInsert(ToStdCostWksh, RoundingMethod, AmtAdjustFactor);     //new

            if not Insert(true) then
                Modify(true);
        end;
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42366.


Paper to the project?

How can I cite this repo, could the author provide the paper link for their great work?

Yes, we will incorporate it into our further work soon. Thank you!


[WIP] CSV Dataset

The end user will be able to upload and display csv data.
Using the dataset uploader:

The csv data is saved in Girder as a file

10 rows are saved in the metadata:

The end user can view the csv file in a tabular format by clicking on the table icon in the dataset panel (in this step, the file is being retrieved from Girder and parsed with Papa parse):

Thanks @essamjoubori .  I will take a look tomorrow.
Then once we finish #257 and merge it, I'll help you get this one merged.

@mgrauer since #257 is merged, can we get this one in master?

We'll need to rebuild this once #301 goes in.

Also will want to change _isCsvFile to be based off of mimetype, which we can get in the client through the file API.

We'll need to rebuild this once #301 goes in.

+1,

Also will want to change _isCsvFile to be based off of mimetype, which we can get in the client through the file API.

Sounds like a small change. If not, I would suggest we create an issue for it and merge this one post #301 merge

Yes, that would be a small change in addition to the CSV work, I wanted to track it here.
One other small change as a part of this, document 'csv' as a dataset_type in the docs.

This will really be going in with #312.  No, really.


Adding in support for Python 3.13

Description
This adds support for Python 3.13. I think I've got all the references, but there's probably somewhere else that I've overlooked.
Shout-out to London Django Meetup for the Hacktoberfest Hackathon inspiration. CC @adamchainz et al.
Testing notes
I ran this under docker (python:3.13-slim) locally and make test and make coverage passed all the tests. Would have also tried with a virtualenv but pyenv didn't have 3.13 (non-RC) at the time of testing.
Checklist:

[ x ] I have added the relevant tests for this change.
[ x ] I have added an item to the Pending section of docs/changes.rst.

Thanks! The failures are probably due to psycopg2 not having 3.13-compatible wheels yet. (https://github.com/psycopg/psycopg2/pull/1729)
I added some comments.

I've probably overstepped the mark by removing 3.8 as it's end of life, but as the next version release looks to be a major one I thought it would be compatible with that. Let me know if that's a problem and I'll reinstate it.

I've probably overstepped the mark by removing 3.8 as it's end of life, but as the next version release looks to be a major one I thought it would be compatible with that. Let me know if that's a problem and I'll reinstate it.

On a pedantic day I'd like this to happen with a separate pull request. I think supporting 5 minor Python versions at the same time is enough and I'm therefore fine with it. CI isn't free energy wise after all.

Thank you for your contribution!


Run cargo install --locked --git https://github.com/ZcashFoundation/zebra --tag <new-tag> zebrad from our README before/on release tagging

Motivation

We want to make sure that our install instructions are currently 'green' when we update them for a new release.
Open Questions

This might depend on the latest release tag, should this test only run when a tag is pushed?
Should this run every time the README.md file is changed?

Related Work

Please add your planning poker estimate with ZenHub @dconnolly


How do I return TLV from a custom command? (CON-764)

All of the examples only show sending TLV. How do I return it?
For example the ViewScene command is using AddReponse(). How do I do this in esp-matter?
    response.sceneName.SetValue(CharSpan(scene.mStorageData.mName, scene.mStorageData.mNameLength));
    Span<Structs::ExtensionFieldSet::Type> responseEFSSpan(responseEFSBuffer, deserializedEFSCount);
    response.extensionFieldSets.SetValue(responseEFSSpan);

    ctx.mCommandHandler.AddResponse(ctx.mRequestPath, response);

@jonsmirl The TLV types for commands are defined for different clusters and their commands in connectedhomeip, e.g The Response type for ViewScene command has kGroupID and kSceneID reference
So, The TLV type for a custom command needs to be defined in similar way

How do I do AddResponse in the esp-matter model?

The ESP-Matter command callback API is designed to provide an abstraction to users, The TLV Writer's handling is expected to be done internally from connectedhomeip, Which is why it's not implemented currently in esp-matter

This needs to be done using Matter APIs. I have implemented now using those APIs.


fix(material-experimental/mdc-tabs): update theming api

Project import generated by Copybara.
This change originates from cl/404320409

Re-pushed and seems to be okay now


Possible bug

Hello,
First of all, congratulations for this work. It is very useful for the research community and it has potential for exploring.
I think I have detected a small bug, but significant in the code. I don't know if I am right or I have misunderstood something. Please let me know as soon as possible.
The bug is located in the file DSNetwork.cpp. In the implementation of the function void DSwitch::ConfigDS(int input_src, int output_mode), in line 17 is this:
config_rds = input_rds;
Would not be this?
config_input = input_rds;
Because as I see it, you are configuring who is the father of the tree (the input of the current node) and therefore, depending on the input_src variable, you connect config_input either to input_lds or input_rds.
Is this correct?
Thank you very much ,
Francisco
email<EMAIL_ADDRESS>

Hi, thanks so much for your interesting in mRNA. For the first question yes, you are correct, it should be config_input = input_rds.
For the second question, it is a good question but I thinks you have a little bit miss understood on it. Our principle is to find out a best trade-off point for the efficiency. We do not prune all the strategies that generate partial sums, what we prune is the strategy which generate too much partial sums that cause the third reconfiguration of the network. (Usually we hope using two network configurations is able to perform the computation of the whole layer).
In the example layer you provide, you can see, even though there is partial sums in mapping strategy 1, its run time is the fastest among all the strategies, this is because it generates the highest utilization rate.
Actually, we are going to update mRNA soon, in the new version, the searching space will be enlarged. And more realistic cases will be considered, for example, the overhead of reconfiguring the networks, bandwidth of the main-memory an so on.
Really appreciated for your advise and question. Please let me know for the further problem inside the code base. mRNA is still under construction. Thanks.
Yours, sincerely.
Best regards.
Zhongyuan.
----- 原始邮件 -----
发件人: "Francisco Muñoz Martinez"<EMAIL_ADDRESS>收件人: "georgia-tech-synergy-lab/mRNA"<EMAIL_ADDRESS>抄送: "Subscribed"<EMAIL_ADDRESS>发送时间: 星期四, 2019年 6 月 06日 下午 5:44:14
主题: [georgia-tech-synergy-lab/mRNA] Possible bug (#1)
Hello,
First of all, congratulations for this work. It is very useful for the research community and it has potential for exploring.
I think I have detected a small bug, but significant in the code. I don't know if I am right or I have misunderstood something. Please let me know as soon as possible.
The bug is located in the file DSNetwork.cpp. In the implementation of the function void DSwitch::ConfigDS(int input_src, int output_mode), in line 17 is this:
config_rds = input_rds;
Would not be this?
config_input = input_rds;
Because as I see it, you are configuring who is the father of the tree (the input of the current node) and therefore, depending on the input_src variable, you connect config_input either to input_lds or input_rds.
Is this correct?
Thank you very much
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/georgia-tech-synergy-lab/mRNA/issues/1

Hi @zzy82518996 thank you so much for your quick answer. I really appreciate it. Regarding with your answer: yes, it is possible a missunderstood a little bit. Now, everything is clear.
There is something else I would like you ask you in addition.
-How do you deal with the loops outside the tile in MAERI? Because I see that mRNA generate this code with that purpose:
Loops outside the tile:
R/T_X = 1
S/T_Y = 1
C/T_C = 15
C/T_K = 1
C/T_N = 1
C/T_OX = 56
C/T_OY = 56
R%T_X = 0
S%T_Y = 0
C%T_C = 0
K%T_K = 0
N%T_N = 0
X'%T_OX = 0
Y'%T_OY = 0
However,  what you would need is another tile (T_R,T_S,....) to deal with this. Am I wrong? If it is like that, why mRNA does not search the best mapping for this remaining loop?

Also, when partial sums are generated and consequently, mRNA generates two different configurations for one mapping in the profiling file, why don't mRNA generate two different tile code in the MAERI configuration file? I mean, as I see the output and the code of mRNA, no matter that there are two configurations in the profiling file, that anyway in the MAERI configuration file you only indicate one tile code like this:

MAERI Configuration:
Layer variables:
X = 224
Y = 224
C = 60
K = 1
N = 1
X' = 56
Y' = 56
R = 11
S = 11
Mapping kernel (tile):
T_X = 11
T_Y = 11
T_C = 4
T_K = 1
T_N = 1
T_X' = 1
T_Y' = 1
Virtual Neuron :
VN_Size = 484
Num_VN = 1
wouldn't you need another code like this for the second configuration?
Thank you so much and I hope have explained myself well.
Best regards,
Francisco Muñoz

Dear Francisco Muñoz:
Thanks for your questions.
For your first question I think you are right, we do not perform mapping space exploration for the outer loops in this version. In current version, we choose an loop order which we think is able to generate the best on-chip memory locality. But we are developing the mapping exploration for the outer loops.
Essentially, the order of the outer loops influences the data transform pattern between the off-chip and on-chip memory. Furthermore, we also consider deploying DNN onto multiple distributed MAERI accelerators. We think this problem is the combination of mapping exploration and data placement and the search space will get much larger. Will let you know our new progress and welcome for your new constructive questions and suggestions.
For the second question, you are also right, we indeed need to generate two configuration files. Actually, we only generate the config file for convolution process but not for partial sums adding process. Our config file is a intermediate representation, the actual config code is generated in MAERI tool chain which is implement outside mRNA. Because we think partial sums adding do not have the concept of T_R, T_s and so on, because it is just the parallel adding process. Right now, the complete tool chain from Tensorflow to machine code is not totally automatic. The interface between mRNA and back-end code generator needs some handwriting work.
So, if you need the tool chain on the hardware side, you can send to email to Hyoukjun<EMAIL_ADDRESS>to ask for the code. We have made the tutorial in HPCA 2019 and ISCA 2018. You can find them on the conference website or professor Tushar Krishna's website.
Thanks again for your constructive questions. And welcome to communicate with us for the further discussion on the mapping space exploration for DNNs.
Yours, sincerely.
Best regards.
Zhongyuan.
----- 原始邮件 -----
发件人: "Francisco Muñoz Martinez"<EMAIL_ADDRESS>收件人: "georgia-tech-synergy-lab/mRNA"<EMAIL_ADDRESS>抄送: "zzy82518996"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>发送时间: 星期五, 2019年 6 月 07日 下午 10:07:01
主题: Re: [georgia-tech-synergy-lab/mRNA] Possible bug (#1)
Hi @zzy82518996 thank you so much for your quick answer. I really appreciate it. Regarding with your answer: yes, it is possible a missunderstood a little bit. Now, everything is clear.
There is something else I would like you ask you in addition.
-How do you deal with the loops outside the tile in MAERI? Because I see that mRNA generate this code with that purpose:
Loops outside the tile:
R/T_X = 1
S/T_Y = 1
C/T_C = 15
C/T_K = 1
C/T_N = 1
C/T_OX = 56
C/T_OY = 56
R%T_X = 0
S%T_Y = 0
C%T_C = 0
K%T_K = 0
N%T_N = 0
X'%T_OX = 0
Y'%T_OY = 0
However,  what you would need is another tile (T_R,T_S,....) to deal with this. Am I wrong? If it is like that, why mRNA does not search the best mapping for this remaining loop?

Also, when partial sums are generated and consequently, mRNA generates two different configurations for one mapping in the profiling file, why don't mRNA generate two different tile code in the MAERI configuration file? I mean, as I see the output and the code of mRNA, no matter that there are two configurations in the profiling file, that anyway in the MAERI configuration file you only indicate one tile code like this:

MAERI Configuration:
Layer variables:
X = 224
Y = 224
C = 60
K = 1
N = 1
X' = 56
Y' = 56
R = 11
S = 11
Mapping kernel (tile):
T_X = 11
T_Y = 11
T_C = 4
T_K = 1
T_N = 1
T_X' = 1
T_Y' = 1
Virtual Neuron :
VN_Size = 484
Num_VN = 1
wouldn't you need another code like this for the second configuration?
Thank you so much and I hope have explained myself well.
Best regards,
Francisco Muñoz
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/georgia-tech-synergy-lab/mRNA/issues/1#issuecomment-499898526

Hello again @zzy82518996 ,
Thank you so much for your answers, and also thank you for being so open to questions and suggestions.
The idea of using distributed MAERI accelerators looks quite interesting and in my opinion, I don't think the space search will increase too much. Most of the layers in a DNN has a large number of output channels (i.e., k). I think you could leverage this to scatter the k-dimension over the different accelerators and broadcast the inputs. I believe that would be one of the best mappings. You might think also that when distributing the load into the accelerators, you are going to use just coarse-grained division and therefore you can either distribute the k dimension as I explained or the n dimension. It would depend on the shape of the layer. Don't you think so?
With the tool chain in the hardware side, do you mean the MAERI configuration code generator? or the MAERI code itself?
If it's the MAERI itself, I have downloaded the MAERI code, but I am having many problems to get the bluespec license. Do you know how can I get it? I have email them several times but I have no answer at all.
Yours,
Francisco Muñoz Martinez


Urgent Action For A Safer Future! 18th October

"We must secure the safest possible future for people and the planet. That means taking urgent action"
Antonio Guterres
UN Secretary General
Caption
Our planet's future hinges on the actions we take today.  Together, we can secure a safer future for all.
#ClimateActionNow
#SaveOurPlanet
#FossilFreeFuture
#ActOnClimate
P.S the text in bold is meant to be on the poster.
@clentafrica/wg-design please assign someone this issue. Thank you

Hey @erioc,
It's hereeee

@SDauphin15 and @yigakpoa please schedule this post for tomorrow. Thank you

Scheduled for FB & Instagram


Check for Libre Baskerville printing issue

Observed behaviour

The Font "Libre Baskerville" causes problems in the application in print. I have included this font in a Word document and the generated PDF can not be printed on any printer - also the pure Word-document. Is there a solution?

Expected behaviour
We have a check for whatever that problem is

No problem with Word for Mac version 16.16.11 (the attached PDF prints without problems). Maybe a Word for Windows issue?
Libre Baskerville.pdf

Having the same issue on Word for mac version 16.49, MacOS 10.15.7. It prints fine on my HP P4015, but not my 9050dn.
Font sizes ≥ 31pt seem to print correctly, though. Bizarre.

(the attached PDF prints without problems).

Not for me, looks like a hot mess.


Fixed Keyboard Dismissal Issue on Signup/SignIn Screen

Description
This PR fixes issue #107, where the keyboard on the signup and signIn screen could only be dismissed by pressing the back button. Now, the keyboard will close when the user taps anywhere on the screen outside of the input fields, improving the user experience.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Screenshot of Successful Build:

Screenshots / Screen Recordings
https://github.com/user-attachments/assets/17ef2b9d-da8b-4f9b-a631-c23279fee091

Please accept it under hacktoberfest

Thanks @AshiqYadav.


Make maps and characters using Blender geometry nodes

Describe the status of the issue.

[ ] proposed
[X] draft
[ ] rejected
[ ] accepted
[ ] deprecated
[ ] superseded by

List the deciders of the issue.
V-Sekai,fire,saracenone
Describe the tags of the issue.
V-Sekai,blender,geometry,
Context and Problem Statement
We want to make procedural geometry.
Describe the proposed option and how it helps to overcome the problem or limitation
Make maps and characters procedurally due to cost or scope limits.
Describe how your proposal will work, with code, pseudo-code, mock-ups, or diagrams
Use blender geometry nodes.
Positive Consequences
Most scope can be accomplished per unit of work.
Negative Consequences
Less quality due to procedural techniques.
Option graveyard

Do everything individually.

If this enhancement will not be used often, can it be worked around with a few lines of script?
Not a few lines of code.
Is there a reason why this should be core and done by us?
It's not core. Would be coo to research.
References

V-Sekai
https://github.com/marian42/wavefunctioncollapse
https://ludwig-ai.github.io/ludwig-docs

License of the contribution
Copyright (c) 2022 V-Sekai contributors.
Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.  IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Needs to be tracked in Godot Engine org or Blender gltf export org.


Bugfix: Temporarily disable some E2E tests

Checklist
General

[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Description

This PR temporarily disables some currently failing E2E tests, until I figure out, why they fail on the bamboo setup, but run fine on my local setup. I will try to fix the tests ASAP.
Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

Screenshots

@TheZoker The error in the user management tests seems to be that it can longer find the #ngb-typeahead-0 element because (at least on TS1) it is called #ngb-typeahead-0-0. Maybe Cypress is stricter regarding that after the major version upgrade. Changing the element id should fix it.


(Tianbin)Week 1 review ticket

Individual GitHub pages (3 points)
GitHub page for Test Prep study, requiring 5.3, 5.4 notes and actions( See "GitHub pages actions" at the bottom of each TPT page on cspcoders).
GitHub page that documents key learnings and code snippets using GitHub and replit.
InfoDB lists
InfoDB loops
Fibonacci - (For Fibonacci(n)  print complete list 0 - n  and not just the nth term)

Individual GitHub pages (3 points)

[ ] GitHub page for Test Prep study, requiring 5.3, 5.4 notes and actions( See "GitHub pages actions" at the bottom of each TPT page on cspcoders).

[ ] GitHub page that documents key learnings and code snippets using GitHub and replit.
InfoDB lists
InfoDB loops
Fibonacci - (For Fibonacci(n)  print complete list 0 - n  and not just the nth term)


fix: additional blank line is needed between markdown and html.

Thanks for the contribution, however this was fixed at some point when the examples were refactored.


Add list of all different optimisations

All plugins should (and already do) highlight their own optimisations, but it would be useful to have a list which mentions all different things that can be done, and how to do them: either through a plugin, or through stringification settings.

Maybe later.


Add back dart2js dump-info

Was accidentally removed from the build output on last update.

I just pushed an updated gh-pages branch.


Fix #399, Deprecate decompress

Describe the contribution
Fix #399
Removes untraced capabilities to decompress libraries or
apps on load.  For non-startup apps, FS can be used.  For
startup, recommendation is to compress/decompress as part
of boot/startup vs individual applications/libs at load.
Fix #291
By deprecating decompress it can be moved to the FS app.
Fix #135
Static code analysis issues in decompress now N/A
Testing performed
Steps taken to test the contribution:

Standard build with and without code deprecated, SIMULATION=native, ENABLE_UNIT_TESTS=true

Expected behavior changes
No longer automatically decompresses apps/libraries as part of load
System(s) tested on

Hardware: cFS Dev Server 3
OS: Ubuntu 18.04
Versions: Master bundle + this commit

Additional context
FS app updates will add decompress functionality, and decompress is planned to be implemented as a replaceable library
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

Forced push comments/doxygen updates to mark API's as deprecated

Rebased on master

CCB 2020-04-29 : APPROVED

@skliper can you resolve the conflicts?

@astrogeco conflicts resolved


Added LastGid for multiple Tileset Support

Adds Ease for Developers Wanting to use multiple tilesets instead of doing, Saving Large amounts of code so now people can determine it by something simple like the example below:
for(int i = 0; i < map.Tilesets.Count; i++)
{
    if(Gid >= map.Tilesets[i].FirstGid && Gid < map.Tilesets[i].LastGid)
    {
        return i;
    }
}

Thanks very much for this patch.
Generally I have tried to follow the pattern of the TMX file, so would prefer that tilecount be saved to the file rather than creating a new equivalent field.  As you've shown, it's easy enough to compute if needed.
Also, no need to append your name to the code, the copyright statement is only there to preserve the software license.  Authorship can be tracked by git (e.g. git reflog -s -n).

Thank you for your reply
I will recommit all of your recommended changes, but what should the null resolve to, or should it not be nullable, As theoretically it could not resolve to 0 naturally.
Sent by Outlook for Android
On Wed, Aug 26, 2015 at 4:51 PM -0700, "Marshall Ward"<EMAIL_ADDRESS>wrote:
Thanks very much for this patch.
Generally I have tried to follow the pattern of the TMX file, so would prefer that tilecount be saved to the file rather than creating a new equivalent field.  As you've shown, it's easy enough to compute if needed.
Also, no need to append your name to the code, the copyright statement is only there to preserve the software license.  Authorship can be tracked by git (e.g. git reflog -s -n).

Reply to this email directly or view it on GitHub:
https://github.com/marshallward/TiledSharp/pull/18#issuecomment-135207459

Good question!  Libtiled does not even read this number.
I think it needs to be nullable though, since older versions of Tiled do not generate this field, and I think C# will crash if it's missing.  But I can take care of it.
Thanks very much for the patch!


export backup to S3

I don't know about the others, but I would like to use this backup solution with a scheduled lambda function. It would be sweet to have the possibility to export to a S3 bucket. And then it would also make sense to allow restoring from an export in a S3 bucket.

An even better option would be to export to memory (string should be sufficient, looking at the code).
The user of the library can then decide where to send the backup to (file, S3, database, etc).
For @mg98 each AWS Lambda container has a temp folder which can be used to write the data to and read the data again to then send it to S3.

Each Lambda function receives 500MB of non-persistent disk space in its own /tmp directory.
https://aws.amazon.com/lambda/faqs/

An even better option would be to export to memory

Yeah, I think that would be useful. Although, when looking at scalability, a good implementation of a S3 export could allow exports of data sets that, because of their size, exceed memory or disk limitations of the execution environment.

This would be nice. It's also pretty easy to chain the two together using the aws cli:
set -e
backup_name=$(date "+%Y-%m-%d")
cognito-backup-restore backup --pool all --use-env-vars --directory "$backup_name"
aws s3 cp "$backup_name" "s3://YOUR-BUCKET/$backup_name" --recursive

This would be nice. It's also pretty easy to chain the two together using the aws cli:
set -e
backup_name=$(date "+%Y-%m-%d")
cognito-backup-restore backup --pool all --use-env-vars --directory "$backup_name"
aws s3 cp "$backup_name" "s3://YOUR-BUCKET/$backup_name" --recursive

I don't think this works due to https://github.com/rahulpsd18/cognito-backup-restore/issues/39


Change Match Whole Word button to only contain lowercase letters

Currently, the Match Whole Word looks very similar to the Match Case button - both contain a small sample of mixed-case letters. This means users (or at least this user!) often has to think about which button does what each time the button is clicked.
It would be clearer if the Match Whole Word button only had lowercase (or uppercase) letters because then case sensitivity wouldn't be implied by the fact that multiple cases are present in the image.

I agree! very confusing.
Also I don't understand why Whole-Word doesn't have a full rectangle border??
These 3 borders are actually making me doubt that this will pick the entire word.

I was glad I found this issue as I was just about to create a new one with just this request. I also think that the whole word icon should not have any upper case letters, all lowercase so that you are only concentrating on the lines. I also suggest that only having an underline, not an overline, Not sure if that would be clearer but I feel that the overline is used to draw attention to letters which are different height, which for a lot of letters usually is the difference between lower and upper case letters. I feel that just an underline would more clearly highlight the matching of the full word. Thanks.

Any update on this? I have to do a double-take each time.
In addition to @andycastles's suggestion, worth trying out two lines on the left/right instead of overline/underline, since that's technically what we're looking for.

Just coming here to say I can never remember which one is the "match case" icon because they both contain mixed case. I was going to file an issue, but I'm glad to see I'm not the only one confused here.

Today will be a good day!


multiple 'Module not found: Error: Can't resolve' issues when trying to run storybook

Issue details
I just installed storybook for my project following the slow setting for React.
When I run 'npm run storybook' I get a lot of errors about unresolved symbols (which I don't get when I run the project on the app.
Steps to reproduce
Follow the step in the slow setup documentation: https://storybook.js.org/basics/guide-react/
in the terminal type 'npm run storybook'
=> lots of error
I tried the solution outlined in https://github.com/storybooks/storybook/issues/1503 (run 'sudo npm run storybook'), but that did not work.
 I used the create react native script to create my project.
At the end it asks to replace the content of my app.js, which I did.
When I start the simulator I get an error:
Syntax error

Even though I don't have any "," in my App.js
I'm actually going to eject my app, so i'll retry the gtstorybook command then

After ejecting the app and rerunning the command I got it working.
Thanks for putting this tool together

I had the same issue,
just add NODE_PATH=src before start-storybook, at least it works for me:
"scripts": {
  "storybook": "NODE_PATH=src start-storybook -p 3001 -c .storybook -s ./public"
}

It worked for me the first time.
After commit, not working any more.


Ability to add more extension types

I use the extension "Rainbow CSV", which adds some custom Language Mode types into VSCode, i.e. CSV (pipe). When swapping from CSV to CSV (pipe), the "Edit as csv" ability disappears. Would it be possible to add a setting where we could add additional Language modes?

I don't think this is possible... I need to set the language names hard coded in the package.json.
It looks like this:
{
  "when": "editorLangId == csv",
  "command": "edit-csv.edit",
  "group": "navigation"
},

and you cannot use js inside "when" (as for as I can tell).
When I look into the "Rainbow CSV" extension package.json I only see "csv (pipe)" as a missing language (maybe the others come from other extensions?).
The only way to solve this would be to hard code all language ids (from the img above) in this extensions package.json.
What do you think?

They're all from Rainbow CSV, I tested removing the extension and and all of those extensions in the screenshot were also removed. Seems like a hassle just to enable compatibility with one extension, so I'll continue swapping back to the standard CSV language mode when I need to edit. Thank you!

Nah, it should be pretty easy to add these modes. Just to be sure, you need to manually switch to e.g. CSV (pipe) or does Rainbow CSV/vscode automatically selects this when you open a file that is pipe delimited?
I'll add the modes in the image next weekend (probably). Probably there are other that have the same problem but didn't create an issue ;)

There's an option in Rainbow CSV that will swap automatically when it detects a delimiter type that you've configured. If it's not in the list it'll keep the Language Mode as standard CSV. Thank you!!

Ah ok, didn't know about this. So I'll not touch the list of supported language modes and close this.
Thanks for the investigation!


Update logic for detecting ASP.NET 5 web applications for RC1 release

Fixes #1810

This won't break beta8 and earlier, right?

I don't think so. As long as there is a wwwroot folder, which has been there for a while. The only time it'll break beta8, I think if someone changed the name of wwwroot in their project.json, but according to them, that was very rare for people to do that.


fix(toUnicode): Swap 'uo' (Aunkarh+Hora) to Unicode compliant 'ou' (Hora+Aunkarh)

Summary

As mentioned in the Unicode standard, the combination should be U+0A4B + U+0A41
Test
Tests were added. Output renders fine on iOS 15.3.1 on an iPhone 13.

Duration

Nice, where did you find that? What is this book?!


Dev4

Why:

Closes #785
What's being changed:
Cpp code for finding factorial of a number via recursion

@SRR-CODER, please link all issues for the programs that you did.
Thank you!

#834 #838 #736 #759 #785

#834 #838 #736 #759 #785 @victoriacheng15

Please link them in the first comment with Closes keyword. Like you have done with issue 785


Options, Text Editor, Advanced - Visual Studio (Windows): Clarify meaning of "Box Selection"

The sentence is not incorrect, but you really need to read this three times plus the purple box to understand this.
The option naming is really unfortunate, because it gives the impression that you enable some new behavior while it doesn't.
It should better be named like

Use simple box selection
Use legacy box selection
Use classic box selection

even better to reverse the checkbox and name it the other way:

Use new multi-caret selection

(enabled by default)
But at the side of the docs, at least that weird sentence could be changed:

This option is unselected by default, which provides for the following multi-caret selection behavior that's new in Visual Studio 2022

"Select this option to disable the new multi-caret..."  is just a very simple example of what's understandable :-)

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

ID: e121ed12-639f-15d5-b1f6-b73e59c0d2c1
Version Independent ID: e121ed12-639f-15d5-b1f6-b73e59c0d2c1
Content: Options, Text Editor, Advanced - Visual Studio (Windows)
Content Source: docs/ide/reference/options-text-editor-advanced.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@softworkz Thanks for your feedback; we appreciate it. We've augmented the text on the Docs page, based on your suggestion. However, to influence how the label is phrased in the product itself, please use the Suggest a Feature feedback mechanism that's designed for product-specific feedback. We'll close this Docs issue now, with our thanks to you for your help with improving the documentation!

Thanks for fixing the docs. Though, please understand that I don't agree that "Suggest a Feature" is the right channel for such matters.
First of all: this is not about a feature - it is about clarity of UI and usability.
Such things are more of a "goes without saying" category. "Suggest a Feature" is about real features and a site where votes are counted to determine which features to implement or not. And unfortunate naming of an option is not something for which much votes are to be expected.
I think for such kind of things, you should have internal channels to communicate. I mean, when even the doc-writers have a hard time to properly explain an option, this should be reason enough for the product/feature owner to re-think about it.
I mean look how ridiculous this is: For example, you create a  product with a button labeled:
"Do not press this button if you don't want to happen what doesn't happen when you don't press the button"
And then, when customer complain that they do not understand the button, you say to the customer:
"Hm, yea, the label is not ideal, but you will first need to start a voting campaign to see who many other customers will tell us that they don't understand the label from which we already know that it can't be understood" :-)

@softworkz -- thank you for your thorough and clear follow-up.


AttributeError: 'Dataset' object has no attribute 'asstr'

I cannot load cached or even newly saved h5ad files with anndata 0.8.0.  I am continually getting:
AttributeError: 'Dataset' object has no attribute 'asstr'
Is this related to #691
h5py version is 3.6.0

Can you provide a full traceback? It would also be good if you could show h5py.__version__ in the same python session where you see this error.

Python 3.8.13 (default, Mar 28 2022, 11:38:47)
Type "copyright", "credits" or "license" for more information.
IPython 8.3.0 -- An enhanced Interactive Python.
import numpy as np
import pandas as pd
import scanpy as sc
import scanpy.external as sce
import scvelo as scv
import seaborn as sns
import loompy
import os, sys
import velocyto as vcy
import os
os.environ["R_HOME"] = "/bin/R"
import infercnvpy
import pyreadr
sc.settings.verbosity = 3             # verbosity: errors (0), warnings (1), info (2), hints (3)
sc.logging.print_versions()
sc.settings.set_figure_params(dpi=600, dpi_save=600, color_map='OrRd')
sc.settings.set_figure_params(dpi=100, dpi_save=100, color_map='OrRd')
sc.settings.n_jobs = 56
os.getcwd()
os.chdir('/mnt/2TBWDMuDev2Underside20620/JabadoTaylor/')  ##Cerebellum Development
SN006 = sc.read('/mnt/2TBWDMuDev2Underside20620/HeidelbergCerebellumLooms/SN006.loom', var_names='var_names', obs_names='obs_names', cache=True, validate=False)
reading from cache file cache/mnt-2TBWDMuDev2Underside20620-HeidelbergCerebellumLooms-SN006.h5ad
Traceback (most recent call last):
Input In [1] in <cell line: 29>
SN006 = sc.read('/mnt/2TBWDMuDev2Underside20620/HeidelbergCerebellumLooms/SN006.loom', var_names='var_names', obs_names='obs_names', cache=True, validate=False)
File ~/anaconda3/lib/python3.8/site-packages/scanpy/readwrite.py:112 in read
return _read(
File ~/anaconda3/lib/python3.8/site-packages/scanpy/readwrite.py:734 in _read
return read_h5ad(path_cache)
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/h5ad.py:236 in read_h5ad
d[k] = read_dataframe(f[k])
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/h5ad.py:301 in read_dataframe
return read_elem(group)
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/specs/registry.py:183 in read_elem
return _REGISTRY.get_reader(type(elem), get_spec(elem), frozenset(modifiers))(elem)
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/specs/methods.py:524 in read_dataframe
index=read_elem(elem[idx_key]),
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/specs/registry.py:183 in read_elem
return _REGISTRY.get_reader(type(elem), get_spec(elem), frozenset(modifiers))(elem)
File ~/anaconda3/lib/python3.8/site-packages/anndata/_io/specs/methods.py:327 in read_string_array
return read_array(d.asstr())
AttributeError: 'Dataset' object has no attribute 'asstr'

Regarding the h5py, it came back as 2.10.0 in Spyder, which is probably causing the problem.  I'm going to clear Anaconda and reinstall everything.  And that was after I explicitly imported it with import h5py...otherwise, the version command didn't work.

After reinstall, all is fine and h5py is >3.3 in spyder.


Upgrading to the Clever v2.1 API

Clever has deprecated their v1 API so now the /me API calls are defaulting to v2.0. In order to get personal information about the user, such as email and name, we need to make follow up API call to the "canonical" URL given in the /me response. Here is more information about the "canonical" link: https://dev.clever.com/v2.1/docs/data-model#links
Why aren't we upgrading to a 3.0 API?
I wanted to get this change out as fast as possible, and there are probably more changes required to migrate to the 3.0 API.
Testing Done
Successfully created Student and Teacher accounts using a Clever "Instant Login" link. I was also able to successfully log into those accounts using the same link after logging out.
Video
https://user-images.githubusercontent.com/1372238/213958718-bec3f92b-150f-4e0f-b693-6820bdc6c461.mov
Follow up work

Update the code-dot-org/code-dot-org repo to depend on this version of the omniauth-clever gem.

Looking great. We should keep in mind that the Schema may change in future Clever API upgrades, and we may need to change how we handle the data.

Yes I will create a JIRA item to upgrade to the v3 API.


[FEATURE] Animate drawing of lines from left to right

I would like to change the line drawing animation so that it happens slowly from the left to the right instead of bottom to top.
Here's an example of what I would like to accomplish:
http://bl.ocks.org/markmarkoh/raw/8700606/

Closing as duplicate of #3232


Static analysis with coverage

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the feature
Today we don't have information regarding coverage in SonarCloud.

Findings at the moment:

I need you help regarding PR https://github.com/MockBukkit/MockBukkit/pull/1104. I've been trying to integrate the coverage report with SonarCloud, for us to have that information in the dashboard, but after some research regarding this topic, i found out that SonarCloud has two types of scans, Automatic Analysis and CI Analysis.
At the moment we have been using Automatic Analysis to fill the information in the dashboard, but this has some limitations where on of them is "Code coverage information is not supported." (Source: https://docs.sonarsource.com/sonarcloud/advanced-setup/automatic-analysis/#prerequisites ).
In order to have coverage in our SonarCloud dashboard we need to migrate to CI Analysis, which was done in the PR mentioned above, but was not enough. For this development to be completed we need to deactivate Automatic Analysis in the Administration Dashboard (Ref: https://docs.sonarsource.com/sonarcloud/advanced-setup/automatic-analysis/#deactivating-automatic-analysis), due to conflicts between Automatic Analysis and CI Analysis (https://docs.sonarsource.com/sonarcloud/advanced-setup/automatic-analysis/#conflict-with-ci-based-analysis).

@4everTheOne
Is this fixed now / can we close this issue?

Hello @Thorinwasher. It's not yet resolved:


Jenkinsfile changed by UpgradeToJava17 / RemoveTrailingWhitespace recipes

What version of OpenRewrite are you using?
I am using:

OpenRewrite v8.11.1
Maven/Gradle plugin v5.15.4

How are you running OpenRewrite?
I am using the Maven plugin, and my project is a single module project. I'm running the plugin via Maven CLI.
mvn org.openrewrite.maven:rewrite-maven-plugin:5.15.4:dryRunNoFork -Drewrite.recipeArtifactCoordinates=my.company:migration-recipes:1.0.0-SNAPSHOT -Drewrite.activeRecipes=my.company.MigrationRecipe

What is the smallest, simplest way to reproduce the problem?
My project contains the following Jenkinsfile, among other code:
@Library("jenkins-shared-libs@my_tag") _

// artifactory standard pipeline
artifactory()

What did you expect to see?
No changes to Jenkinsfile.
What did you see instead?
[WARNING] These recipes would make changes to Jenkinsfile:
[WARNING]     my.company.MigrationRecipe
[WARNING]         my.company.ProjectTypeAMigrationRecipe
[WARNING]             my.company.CommonRecipe
[WARNING]                 my.company.Java17Recipes
[WARNING]                     org.openrewrite.java.migrate.UpgradeToJava17
[WARNING]                         org.openrewrite.java.migrate.Java8toJava11
[WARNING]                             org.openrewrite.java.migrate.JavaVersion11
[WARNING]                                 org.openrewrite.java.migrate.UpgradeJavaVersion: {version=11}
[WARNING]                         org.openrewrite.java.migrate.JavaVersion17
[WARNING]                             org.openrewrite.java.migrate.UpgradeJavaVersion: {version=17}
[WARNING]             my.company.BestPractices
[WARNING]                 org.openrewrite.java.format.RemoveTrailingWhitespace

diff --git a/Jenkinsfile b/Jenkinsfile
index ed6e76b..2c86077 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -1,4 +1,4 @@ org.openrewrite.config.CompositeRecipe
-@Library("jenkins-shared-libs@my_tag") _
+@Library("jenkins-shared-libs@my_

What is the full stack trace of any errors you encountered?
No stack trace available

Are you interested in contributing a fix to OpenRewrite?
I'm already contributing on other issues.

Replicated indeed with a test
@Test
void jenkinsFileWithLibrary() {
    rewriteRun(
      groovy(
        """
          @Library("jenkins-shared-libs@my_tag") _

          // artifactory standard pipeline
          artifactory()
          """,
        spec -> spec.path("Jenkinsfile")
      )
    );
}

What's interesting here is that it fails whenever the variable name assigned to the library occurs on the same line. If you remove _ from my_tag the test passes; same thing if you replace the last _ with foo. It's not specific to _ either. It fails just the same if you replace _ with a.

I did some digging on this one, and it seems that the problem is here: https://github.com/openrewrite/rewrite/blob/main/rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java#L2072. I think it should consider the scope of the quotes.

Any updates on this one?

No updates from my side, although I've also been off for a couple weeks, with some more travel ahead. It seems you've pinpointed the issue already. Are you considering any code changes there already? Or should we tag other folks while I'm out?


clientconfig: modify unit tests

Tests for clientconfig should be modified to ensure the UserDomain and ProjectDomain mappings work correctly.

Implemented in #8


Fee calculation before trading

The fees are not taken into consideration before deciding to enter or exit a trade, which can lead to negative trades.
The current positive spread calculation is (pseudo code below):
long entrySpread = shortPrice/longPrice - 1;
long exitSpread = 1 - shortPrice/longPrice;

The spread is the percentage difference between the long and short price.
To take the fees into account we need to replace the shortPrice and longPrice with the effective shortPrice and longPrice with the equivalent price including the fees. The fees will decrease the price of sell orders and increase the price of buy orders.
//BUY
long entryEffectiveLongPrice = longPrice * (1+longFee);
long exitEffectiveShortPrice = shortPrice * (1+shortFee);

//SELL
long entryEffectiveShortPrice = shortPrice * (1-shortFee) ;
long exitEffectiveLongPrice = longPrice * (1-longFee);

The effective spreads are:
long effectiveEntrySpread = shortPrice * (1 - shortFee) / (longPrice * (1 + longFee)) -1;
long effectiveExitSpread = 1 - shortPrice * (1 + shortFee) / (longPrive * (1 - longFee));

As expected,  with 0 fees the formulas are the same and increasing the fees decrease the spread (making it harder to enter or exit trades).
Example for a trade entry:
longPrice = 1000
shortprice = 1010
shortFee = 0.0026
longFee = 0.005
effectiveShortPrice = 1010 * (1 - 0.0026) = 1007.374
effectiveLongPrice = 1000 * (1 + 0.005) = 1005

spread = 1010/1000 -1 = 0.0100
effectiveSpread = 1007.374 / 1005 -1 = 0.0022

The spread limits would need to be lowered as they don't need to compensate for the fees anymore.

Ok this makes sense to me. I can put a branch together and see how it goes.

Ok this makes sense to me. I can put a branch together and see how it goes.

The fix should make sure to take the decision to trade based on the final effective price. The 'effective price' is the price you pay for 1COIN, after fee, step size rounding, scale rounding etc


Provide for JSON or VPack content types, set headers accordingly.

feature/#153

provide support for both JSON and VelocyPack media types
test using velocypack content type will get a velocypack response from ArangoDB

Nice changes! I don't why I had in mind that this would be tied to the serialization. I like the approach you took there.


Google service account credentials usage doesn't match README instructions

Hi there! The README says the Google service account credentials should be placed at ~/.config/gspread/service_account.json, which is where gspread looks by default. However, line 17 of auto_auto_archive.py specifies a credentials file in the same directory as the script, so the script won't run unless you drop service_account.json in the same directory.
So it seems that either auto_auto_archive.py should be changed to call gspread.service_account without a filename parameter; or the README should be changed to specify that service_account.json be created in the application root. If you have a preference, I can open a pull request and change it either way.

Closed with pull request #9


codet5-base.jackson-databind.mid-6917.idx-41820.2.buggy

ckeansed the code , corrected indentation and errors

Indentation fixes. LGTM!


I am confused about the code

https://github.com/uwdata/fast-kde/blob/f94f0215f2a8e2c36944883eb71c639b8a292844/src/deriche.js#L133-L136
https://github.com/uwdata/fast-kde/blob/f94f0215f2a8e2c36944883eb71c639b8a292844/src/deriche.js#L193

I think that the number of parameters mismatches between the definition and usage.
Does 'c.sum_causal' and 'c.sigma' matter?

Thank you!

Thanks for asking! In the past there were multiple initialization methods, so the signature has some of the remnants of that.
Meanwhile, I just updated the dericheInitZeroPad method to include missing error correction loops (see the latest version), which makes use of the additional parameters.


Suggestions to facilitate contributions on this repository

In the context of Etalab helping the 10% projects to become more collaborative, here are some suggestions:

[x] Add a licence
[x] Add installation instructions to the README.md
[x] Complete the README.md with directions on how/why to contribute.
[x] Add a CONTRIBUTING section in the README.md (or a CONTRIBUTING.md file.)
[ ] Add tags to the GitHub page

Mhh... the checklist seems to suggest it is not really completed :)


Adds updating, tons of code, and better names

This merge adds the initial ability to update the add-in from GitHub.  This should make it much easier to deploy the code to users without them having to check the Releases to see if a new version is out there.  The current version number is included in the bUTL code.
There is a large amount of new code that has been added.  This code was accumulated over the last year or so.  There have also been a couple of other additions/bug fixes for other parts of the code.  Unfortunately, given all that has changed in this commit, it's nearly impossible to tell which changes are in which category.
Finally, I went through and gave much more descriptive (and proper) variable names to the majority of names.  The exceptions are in the Testing/RandomCode modules.  There are a couple of items that still need cleaned up, but things are much better now.

@RaymondWise, you can sync against master now.  I am done making major changes.  I will be working on local branches and pushing those to GitHub (but not master).  This will make it easier to see what I am working on.

I'm apparently terrible at git, I reforked so we're on the same page.

I have a bear of a time dealing with remote repos at times.  Fortunately, I haven't had to sync a fork yet.  For future reference, it should have been possible to fetch/checkout based on a remote repo pointing to this one.  Note sure if upstream gets created automatically, but it could be added if not.  See: https://help.github.com/articles/syncing-a-fork/


Bugfix/gh actions mpi

Pull Request Test Coverage Report for Build 659945463

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.491%

Totals

Change from base Build 659833416:
0.0%

Covered Lines:
1836

Relevant Lines:
2938

💛  - Coveralls


Create a supabase.auth.admin.getUserByEmail function

Feature request
Is your feature request related to a problem? Please describe.
On server-side, I want to check whether a user with a certain email exists before doing a signup. Since for safety reasons, signups on client side return fake "success" data, even though the user already signed up with this email. (https://github.com/supabase/supabase-js/issues/296)
Describe the solution you'd like
I would like to have an additional method to not only list all users and get one by Id, but also get one by email.
So there is supabase.auth.admin.getUserById(1) but not<EMAIL_ADDRESS>Describe alternatives you've considered

Using listUsers(), pretty inefficient but would do the trick
Signing up a user on the client

Hey yes this is a good suggestion. Though, we would probably use something like listIdentitiesForEmailAddress since email addresses do not always uniquely identify users.


cloneNode(DeepCopy) + Nested ngRepeat => Fails

I encountered an issue where cloneNode wouldn't clone the content of a table :
This table uses ng-repeat on every row (tr) and inside every row, on every cell (td).
I tried to clone it and got all of my rows, but not any of my rows contains a cell !

A plunker showing what you are trying to do would be mostly helpful


fix: commitlint 오류 수정

:tada: This PR is included in version 1.0.0-next.10 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: