[🐞] Assets in qwik city are loaded from a page's folder name instead of public

Which component is affected?
Qwik City (routing)
Describe the bug
When I use an asset from the public folder in a page, e. g.:  in the flower page from the qwik starter script, I get an error in the browser, that the resource was not found in that directory: GET http://localhost:5173/flower/favicon.svg 404 (Not Found)
I'm using qwik 0.14.1 and and qwik-city 0.0.127
Reproduction
https://stackblitz.com/edit/qwik-starter-apamp4?file=src/routes/flower/index.tsx
Steps to reproduce
I used the starter script: npm create qwik@latest and added in the flower page an image element, so the flower page looks like this:
<>

      <input
        type="range"
        value={state.number}
        max={50}
        onInput$={(ev) => {
          state.number = (ev.target as HTMLInputElement).valueAsNumber;
        }}
      />
      <div
        style={{
          '--state': `${state.count * 0.1}`,
        }}
        class={{
          host: true,
          pride: loc.query['pride'] === 'true',
        }}
      >
        {Array.from({ length: state.number }, (_, i) => (
          <div
            key={i}
            class={{
              square: true,
              odd: i % 2 === 0,
            }}
            style={{ '--index': `${i + 1}` }}
          />
        )).reverse()}

    </>

System Info
System:
    OS: macOS 13.0.1
    CPU: (8) arm64 Apple M1
    Memory: 67.83 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.13.0/bin/yarn
    npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
  Browsers:
    Chrome: 107.0.5304.110
    Firefox: 107.0
    Safari: 16.1

Additional Information
Hower!
When i add the folder flower in the public folder and copy the asset inside, it works, the asset can be found. Is this intended this way?

Use "/favicon.svg" urls prefixed with / link to the root of the site, whereas if you don't it is relative to the current page.

Ah! It works! Thank you so much!


do not stub imports that are part of the target world

This PR makes sure that if an import is present in the target world, it does not get stubbed out. I went with a different approach than what was suggested in the issue comment as I noticed the module only contained the wasi imports if they are a part of the target world and was simpler than having to prepare the wit. Let me know if I am missing something.
fixes #105

There are two ways that imports can be included in the output binary:

Imports are pulled in by the engine itself
World imports being converted into imports by the splicing code

The stubbing code here acts on the binary that is the union of the above. Therefore the stubbing code cannot know for a given import if it was used by the engine or if it is used by the target world.
Therefore saying that features only stub out when the final binary doesn't use them excludes engine imports pulled in that we do want to stub. For example libc builtins that get pulled in that are unused.
This means that this PR risks outputting a binary with extra engine imports that were supposed to be stubbed. I think you will find that the "pure component" case of no features is thus no longer working with this PR, and that pure components have imports again.
This list of these engine bulitins that should always be stubbed can be determined by building a binary against any non-WASI world. The problem is that this list should be excluded against any WASI imports of the world being targetted. And thus we need to parse the target world to do the exclusion as I described in https://github.com/bytecodealliance/ComponentizeJS/issues/105#issuecomment-2110865818.
I'm open to counter arguments if you can prove any of the above otherwise.

I just checked this PR and can confirm the pure component case is still okay, but the edge cases are still not quite consistent I think. To dig into the nuances:

Preview1 builtins are always stubbed (stub_preview1())
It just so happens that filesystem, CLI and sockets are not at all used by StarlingMonkey builtins, so as a result we will never have this ambiguous union case with world imports. If StarlingMonkey builtins ever bind to these subsystems in future we will get these ambiguity cases.
This then leaves random, CLI, clocks and HTTP as cases where having the feature disabled will still stub out the world. This results in the case where you must set features: ['random'] if you are binding to a world that uses random.

So the consequence is that for example having HTTP disabled by default will result in not being able to output to an HTTP world, unless you explicitly enable the HTTP feature as well.
Right now, features is something that relates to the engine itself, and gets unioned with the target world. It seems like it would be odd to have to align the features usage with the exact target world usage.
For example consider if we add support for websockets in future. This is a "websockets" feature. But whether websockets is enabled or not shouldn't affect weather I can target a world that uses websockets.

I think the above tracks in my head. I am happy to take a stab at using an approach similar to splicing if that is the recommended approach.

Thanks for being open to that! It's just a few lines of code. An example of walking the exports of a WIT world is here - https://github.com/bytecodealliance/ComponentizeJS/blob/main/crates/spidermonkey-embedding-splicer/src/lib.rs#L143. Where instead of exports, imports can be used. Those strings can then be directly used as the exclude strings when performing stubbing.

I have updated the code to resolve the target world imports and use that to conditionally stub imports. I wanted to check if this is the desired approach. If this is on the right approach I will rebase the PR.

Thinking about this some more: I suppose an alternative here would just be to stub the WASI imports before doing the splicing, ie before the engine gets passed in here - https://github.com/bytecodealliance/ComponentizeJS/blob/main/src/componentize.js#L52.
Then any world imports would be fully additive without needing to go through the world exclusion process.
This also seems like it would work better with composition workflows where right now as soon as something is used it is no longer stubbed.
Sorry for being slow on this one, took me a minute to think it through.

No worries. I will take a stab at updating it to the above approach tomorrow.
On another note,  Working on this also made me realize we can potentially solve #104 similarly. We could use @babel/parser and @babel/traverse to parse the source file and stub any of the imports not in use by the guest code but part of the target world.

Definitely share in more detail what you're thinking about the stubbing in that issue. Note that we already use this lexer to analyze imports and exports in https://github.com/bytecodealliance/ComponentizeJS/blob/main/src/componentize.js#L124.

Is the suggested idea of calling stubWasi before spliceBindings something like the following?
  const features = []; // Intionally  leaving out the part of checking for enabled features.
  const stubbedEngine = stubWasi(await readFile(engine), features);

  let { wasm, jsBindings, importWrappers, exports, imports } = spliceBindings(
    sourceName,
    stubbedEngine,
    witWorld,
    maybeWindowsPath(witPath),
    worldName,
    false
  );

When using that to build the example I get the following,
Error: calling the Reactor initialization function

Caused by:
    0: error while executing at wasm backtrace:
           0: 0x7fbf - <unknown>!<wasm function 20>
           1: 0x76b206 - <unknown>!<wasm function 11512>
    1: wasm trap: wasm `unreachable` instruction executed
file:///Users/karthik_ganeshram/Work/wasm-tools-contrib/ComponentizeJS/src/componentize.js:219
    throw new Error(err);
          ^

Error: Failed to initialize the compiled Wasm binary with Wizer:
Wizering failed to complete
    at componentize (file:///Users/karthik_ganeshram/Work/wasm-tools-contrib/ComponentizeJS/src/componentize.js:219:11)
    at async file:///Users/karthik_ganeshram/Work/wasm-tools-contrib/ComponentizeJS/example/componentize.js:7:23

Node.js v21.7.3

I will look into it more.

Right, stubbing can't happen before Wizering. And there is some trickiness around environment variables and FS access here, where those subsystems would need to be specifically stubbed as Wizer stubs. Difficult to come up with a consistent model here! Perhaps the approach you implemented here is in fact a good one for now.
I can review and land if you like in the mean time, before we explore alternatives further.

It would be great to review and land this, it would help with the usage of extension libraries in JS to provide node APIs until we get them directly from starlingmonkey. I can fix the above suggestion and also remove the unwraps in a bit.

This should now be ready for a review.


Add no-gas mode to kevm

Adding <use-gas> cell.
Adding --no-gas flag that will set the <use-gas> cell to false when used.

TO DO:

[ ] Test in kontrol
[x] Update expected output for tests.

@anvacaru please post performance numbers here, comparing against master. Also with Kontrol.
Also, we need to have a duplicate of each rule in optimizations.md, where each rule mentions the <use-gas> cell and whether it is using it or not.

@ehildenb I created a new branch in Kontrol (kevm-no-gas) in which I use this version of KEVM.
This is the performance table.
I'll create a similar one for the Kontrol nogas branch.
| Test                                                                       | kevm-no-gas time  | v0.1.92 time | (kevm-no-gas/v0.1.92) time
|----------------------------------------------------------------------------|-------------------|--------------|----------------------------
| src/tests/integration/test_foundry_prove.py::test_foundry_remove_node      | 25.43             | 48.81        | 0.52099979512395
| src/tests/integration/test_foundry_prove.py::test_foundry_auto_abstraction | 22.65             | 28.1         | 0.8060498220640568
| InitCodeTest.test_init()                                                   | 164.2             | 190.05       | 0.8639831623257036
| src/tests/integration/test_foundry_prove.py::test_foundry_merge_nodes      | 54.6              | 62.22        | 0.8775313404050145
| src/tests/integration/test_foundry_prove.py::test_foundry_resume_proof     | 74.99             | 82.83        | 0.9053483037546782
| AssumeTest.testFail_assume_true(uint256,uint256)                           | 64.83             | 71.58        | 0.9056999161777033
| AssumeTest.test_assume_false(uint256,uint256)                              | 71.45             | 78.76        | 0.9071863890299644
| InitCodeTest.testFail_init()                                               | 127.18            | 139.01       | 0.9148982087619597
| BMCLoopsTest.test_countdown_concrete()                                     | 23.3              | 25.29        | 0.9213127718465798
| SetUpDeployTest.test_extcodesize()                                         | 119.39            | 113.31       | 1.0536581060806636
| Storage.sol                                                                | 61.53             | 54.18        | 1.1356589147286822
| ERC20.sol                                                                  | 59.78             | 51.31        | 1.165075034106412
| AssertTest.test_assert_true()                                              | 75.01             | 52.74        | 1.4222601441031475
| TOTAL                                                                      |<PHONE_NUMBER>999998 | 998.19       | 0.9460523547621192

@ehildenb here is the profile for kontrol. I could not generate the profile for evm-semantics proofs because some of the mcd proofs are currently failing.
| Test                                                                       | kevm-no-gas time   | master time        | (kevm-no-gas/master) time
|----------------------------------------------------------------------------|--------------------|--------------------|----------------------------
| AssertTest.testFail_expect_revert()                                        | 67.48              | 320.21             | 0.2107367040379751
| src/tests/integration/test_foundry_prove.py::test_foundry_merge_nodes      | 55.16              | 74.85              | 0.7369405477621911
| src/tests/integration/test_foundry_prove.py::test_foundry_auto_abstraction | 22.69              | 29.05              | 0.7810671256454389
| AssumeTest.test_assume_false(uint256,uint256)                              | 71.94              | 85.69              | 0.8395378690629012
| InitCodeTest.test_init()                                                   | 164.83             | 195.06             | 0.8450220444991285
| SetUpDeployTest.test_extcodesize()                                         | 119.02             | 139.75             | 0.8516636851520573
| BMCLoopsTest.test_countdown_concrete()                                     | 22.98              | 26.98              | 0.8517420311341735
| AssumeTest.testFail_assume_false(uint256,uint256)                          | 44.81              | 52.11              | 0.8599117251966993
| AssumeTest.testFail_assume_true(uint256,uint256)                           | 65.12              | 74.75              | 0.8711705685618729
| src/tests/integration/test_foundry_prove.py::test_foundry_resume_proof     | 76.65              | 85.63              | 0.8951302113745184
| InitCodeTest.testFail_init()                                               | 127.44             | 141.61             | 0.8999364451663017
| Empty.sol                                                                  | 52.51              | 50.03              | 1.0495702578452928
| Storage.sol                                                                | 61.6               | 53.84              | 1.1441307578008915
| ERC20.sol                                                                  | 60.82              | 51.59              | 1.1789106415972086
| TOTAL                                                                      | 1013.0500000000002 | 1381.1499999999999 | 0.7334829670926404

There are changes to the <gas>, <callGas>, and <refund> cells made in driver.md, state-utils.md, and kevm-ltl.md. Should these be addressed as well by this PR?

I tested one Kontrol-based test against the Kontrol master, Kontrol no-gas, and KEVM no-gas (this PR), with the following results:
ExpectRevertTest.test_expectRevert_returnValue
                         Total         Pre         Eff    Steps
	master:		104.00	 	33	 71.00	   5772
	nogas:	 	 94.36	 	33	 61.36	   3406
	kevmnogas:	 93.97	 	33	 60.97     3763

where Pre denotes the pre-processing time and Eff denots the effective time of the test. It's interesting that even though the number of steps is higher for KEVM-no-gas than for Kontrol no-gas, the test time is lower, possibly due to failed matchings of higher-priority rules in the latter.

Higher priority rules will always introduce a slowdown because the rules have to be tried in order.


Consider decoupling feature releases from plugin releases

Feature Description
Consider decoupling feature releases (Unsplash, Coverr, Tenor) from plugin releases with the goal of:

Pragmatically in the short term, allow us to release with stable if there are non-ui blockers on our side before the Monday release cutoff.
Allow us to disable a provider if there is a prolonged outage or other issue.

There isn't a javascript API / HTTP endpoint available for backend feature discovery, so other teams have been creating their own API endpoints.
Approaches considered

As a start, a feature endpoint can be added media.proto with a list of enabled features ('unsplashTab', 'tenorTab', 'coverrTab').
Later, we can add integrate the backend feature tooling to return enabled features (avoid hard coding).
Alternatively (or later), a new API can be added to provide a feature flag / experiments endpoint for Story Editor generally.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Better handling for cases where the API is not available makes sense, but I am wary of adding any feature flag API endpoints. That's a huge change and is ultimately a product decision. Also, and perhaps most importantly, we first need to ensure that this does not violate WordPress.org guidelines.

Better handling for cases where the API is not available makes sense, but I am wary of adding any feature flag API endpoints. That's a huge change and is ultimately a product decision. Also, and perhaps most importantly, we first need to ensure that this does not violate WordPress.org guidelines.

I mentioned this idea in standup as a pragmatic way to release sooner if we're still waiting for non-functional parts before Monday, and to provide a turn-down option later once the release is in an environment that we don't have control over.
It's probably too late for the Monday cutoff, but an implementation for media3p tabs wouldn't need to be baked in forever (so this wouldn't have to be a one-way door solution). If the WordPress guidelines allow it, a general-purpose way of toggling feature flags would be a good practice for feature development in Story Editor going forwards.
I had a read over the WordPress.org guidelines, which part do you see is relevant to feature hiding?

I had a read over the WordPress.org guidelines, which part do you see is relevant to feature hiding?

I was specifically referring to this proposed item:

Alternatively (or later), a new API can be added to provide a feature flag / experiments endpoint for Story Editor generally.

I'm wary about doing that. The guidelines are sometimes a bit vague, so I don't wanna make assumptions. We'd need to reach out to the plugins team first before doing anything in this regard.
Some sections that caught my eyes:

Trialware is not permitted
Plugins may not contain functionality that is restricted or locked, only to be made available by payment or upgrade. Functionality may not be disabled after a trial period or quota is met
Plugins may not send executable code via third-party systems.
Executing outside code within a plugin when not acting as a service is not allowed
Serving updates or otherwise installing plugins, themes, or add-ons from servers other than WordPress.org’s

Again, I don't wanna make assumptions. We just need to clarify these things before investing time in something like this.

Anyhow, we've now shipped, so I don't consider this a priority for now. Let's close it and consider it a "maybelater"

I got confirmation from the plugins team that this would be OK, but only on an opt-in basis by the user. And of course with all the code being already in the plugin (delivering executable code to the plugin from an external service would be a no-no)


eccodes: 2.35.0 -> 2.36.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/eccodes/versions.
meta.description for eccodes is: ECMWF library for reading and writing GRIB, BUFR and GTS abbreviated header
meta.homepage for eccodes is: https://confluence.ecmwf.int/display/ECC/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.36.0 with grep in /nix/store/vi4v5yqdc5wyld23ih2kcdcg9c5f8nz8-eccodes-2.36.0
found 2.36.0 in filename of file in /nix/store/vi4v5yqdc5wyld23ih2kcdcg9c5f8nz8-eccodes-2.36.0

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath

cdo
eccodes
gnudatalanguage
python311Packages.eccodes
python312Packages.eccodes

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vi4v5yqdc5wyld23ih2kcdcg9c5f8nz8-eccodes-2.36.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 eccodes https://github.com/r-ryantm/nixpkgs/archive/1c33830be92033f7b4547c8be7ee5f38a1a393ec.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1c33830be92033f7b4547c8be7ee5f38a1a393ec#eccodes

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vi4v5yqdc5wyld23ih2kcdcg9c5f8nz8-eccodes-2.36.0
ls -la /nix/store/vi4v5yqdc5wyld23ih2kcdcg9c5f8nz8-eccodes-2.36.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

5 packages built:

cdo
eccodes
gnudatalanguage
python311Packages.eccodes
python312Packages.eccodes

Maintainer pings
cc @knedlsepp for testing.

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

5 packages built:

cdo
eccodes
gnudatalanguage
python311Packages.eccodes
python312Packages.eccodes


canShareViaEmail on iPad

Hi there, I love your plugin, it's amazing...
window.plugins.socialsharing.canShareViaEmail(... returns False on my iPad2 iOS 7.1.2 device. It's flawless on Android.
Why is this? Is there something I'm missing?
Thanks!

Thanks, and sorry it took me a while. A long while even.
The canShareViaEmail function will return true if the native iOS emailclient has been properly configured to send email. At least that's what the iOS documentation (of the function I'm using) tells me.
If you have any concern please post the javascript code here as well.


failed to open stream: No such file or directory

I've setup your script like written in the README.
But no matter where i set the download folder or as which user ever i execute it, the script always returns the attached error.
root@debian:~/laracasts-downloader# php start.php
====================================
Starting Collecting the data

209 new lessons and 296 episodes. 17.904s elapsed with 3.25Mb of memory usage.

====================================
Authenticating

Successfull!

====================================
Downloading Lessons

Download started: email-verification-in-laravel . . . . Saving on lessons folder.
ERROR: Unable to open /path_to_folder/laracast/lessons/0001-email-verification-in-laravel.mp4 using mode r+: fopen(/path_to_folder/laracast/lessons/0001-email-verification-in-laravel.mp4): failed to open stream: No such file or directory

Any idea whats going wrong?
Thanks for support!

Hi, do you have write permissions on the lessons folder?

Hi,
yes i had write permissions, path was created as root (incl. lessons and series folder) and script was also started as root.
Now i've deleted the sub-directories lessons and series et voilà it's running like a charm. Weird...

Download started: email-verification-in-laravel . . . . Saving on lessons folder.
Elapsed time: 109.354s, Memory: 4.00Mb
Current: 1 of 209 total. Left: 208

Thank you very much for the amazing script!

Really weird!
Good to hear it works! Have fun!


Additional auto fixables

extends list of auto-fixable array/object types
auto-fix Unions where one side is string/union of strings
add generic equal class auto-fixables
auto-fix Unions where both sides contain only 1 atomic (sub)type
make docblock usage configurable via psalm config

Fix: https://github.com/orklah/psalm-strict-equality/issues/3

Yeah sorry, it's on my list, I'll get to it soon

Oh. I just realized you didn't see my first round of comments because I didn't publish them :( sorry. I thought you responded to it earlier but it was just a response to my new issue.
You'll see duplicate comments or outdated comments, you can resolve them directly

It's really became a complex piece of code. I'd be way more at ease if we could make some kind of automated check that everything is correct. Not necessarily a whole test suite, but some script we can run to check whether the plugin is doing its job

Done! Make sure to report any error while running it, the lack of tests on those plugins makes this kind of change random :p


create extensions that records AWS requests into JSON files in the data directory

The goal of this extension is to demonstrate how you can intercept requests and response and do something with them.
a request/response format could look like this:
{
  "request": {
    "service": "sqs",
    "operation": "CreateQueue",
    "payload": {
      "QueueName": "my-queue"
    }
  },
  "response": {
    "status_code": 200,
    "payload": {
      "QueueUrl": "http://localhost:4566/000000000000/my-queue"
    }
  }
}

we could write each request/response doc into an NDJSON file.
to achieve this we need to:

use the     def update_response_handlers(self, handlers: CompositeResponseHandler): extension method to inject a new response handler
that response handler needs to access context.service, context.operation, context.service_request (if it exists), context.service_response (if it exists)`
create the request/response JSON
append it to a file

can close now with https://github.com/thrau/localstack-aws-log-viewer


Complementary Module

Hi @RamblingCookieMonster
I have been using this module with great success on PS 5.1 so thank you for your hard work. I did however run into issues when trying to use on Linux in PS 7.0. I can go into detail on why but to suit my purposes (and as a learning experience) I have created a new module to complement PSRabbitMQ, focusing on Consuming.
It is yet very simple and a work in progress but I thought it might be worth sharing via the PSGallery. So far I have named it PSRabbitMq.Consumer  as it is intended to be used with your module.
Would you be happy for me to publish such a module? As far as i know you haven't copyrighted the name but I felt that I would like to have your approval out of courtesy.
You can find my module here. All feedback and questions welcome.

sounds awesome, thanks for the heads up  : D


Draggable songs in Queue

It would be better if the app allowed to correct the order of the songs in the queue by dragging and dropping them.
The easiest solution is to use an npm package for achieve this functionality.

There's already a base for this, it just needs some cleaning up and it should work properly

I thought that this functionality is broken, but seems like it's working properly.

There's already a base for this, it just needs some cleaning up and it should work properly

@JustCallMeQuincy  Did you find something wrong with it?

Works perfectly in 2.0.0-stable!


top level organization issues

There are several issues with the top-level organization.

The top-level organization shows z3 topics on the main landing page. The z3 topics should (eventually?) only show up when selecting the z3 tutorial (reflecting that rise4fun is more than z3 even though current activities are z3 centric).
When selecting the z3 tutorial the page is undefined. What is the right link? Where should it be added?
The side bar shows z3 sections in an indeterminate order. The order in the upper task bar is according to a json file, but the order in the left side bar is neither logical, nor alphabetical.

The top-level organization shows z3 topics on the main landing page. The z3 topics should (eventually?) only show up when selecting the z3 tutorial (reflecting that rise4fun is more than z3 even though current activities are z3 centric).

There's a bit of a confusion here. This web site should be all about Z3; but built in the "rise4fun" template. Eventually we'll have hosted content for each tool stored in separate locations.
rise4fun really refers to a project template that will allow to create an interactive tool web site easily

I am still confused because the web site is called https://microsoft.github.io/rise4fun
It contains the button to get to the z3 tutorial, but the rest of the landing page is very specific to z3.

For example, it contains the header with "Strategies" "Fixedpoints".
It also contains the footer with references to z3.
Should we just put the entire page into https://microsoft.github.io/z3guide?

We need rename the repo to z3guide AND update some config flags in the docusaurus.config.js

You make the decisions, I don't have all context anyhow. But who executes? I think it would be for @rlisahuang to rename repo.

The one-repo-one-tool concept sounds reasonable. On top of everything, we would need a separate landing page for all rise4fun tools, where the current z3 tutorial will be linked from there.

I will do the renaming after the two interviews today. A bunch of changes are needed and I want to avoid breaking anything before we show the preview version to our interviewees.

We can rename the repository to update rise4fun to z3guide or something... Yes it's not rise4fun, it's all about z3 right now.
The landing page is most stale and has not been updated at all.
Sent from Outlookhttp://aka.ms/weboutlook

From: Nikolaj Bjorner @.>
Sent: Saturday, July 2, 2022 1:22 PM
To: microsoft/rise4fun @.>
Cc: Peli de Halleux @.>; Comment @.>
Subject: Re: [microsoft/rise4fun] top level organization issues (Issue #30)
I am still confused because the web site is called https://microsoft.github.io/rise4funhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmicrosoft.github.io%2Frise4fun&data=05|01|jhalleux%40microsoft.com|ada657b2efbb44b7505c08da5c688a2c|72f988bf86f141af91ab2d7cd011db47|1|0|637923901317457248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=dx7Kdz232UnZiA552QvJa8A0ry3CoRe7VVB0HzJTBC0%3D&reserved=0
It contains the button to get to the z3 tutorial, but the rest of the landing page is very specific to z3.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F3085284%2F177015093-6f214076-dbb8-4301-8115-271dd3383c53.png&data=05|01|jhalleux%40microsoft.com|ada657b2efbb44b7505c08da5c688a2c|72f988bf86f141af91ab2d7cd011db47|1|0|637923901317457248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=tPWYptmXDP26v8coZbSSTHUTcVeWfvoTgynD3yNDmXg%3D&reserved=0
For example, it contains the header with "Strategies" "Fixedpoints".
It also contains the footer with references to z3.
Should we just put the entire page into https://microsoft.github.io/z3guidehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmicrosoft.github.io%2Fz3guide&data=05|01|jhalleux%40microsoft.com|ada657b2efbb44b7505c08da5c688a2c|72f988bf86f141af91ab2d7cd011db47|1|0|637923901317457248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LrPLcDoGA7%2B93nLHnR4u%2BlMdSfagF3qNkhlh8aqv30w%3D&reserved=0?
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Frise4fun%2Fissues%2F30%23issuecomment-1172955482&data=05|01|jhalleux%40microsoft.com|ada657b2efbb44b7505c08da5c688a2c|72f988bf86f141af91ab2d7cd011db47|1|0|637923901317457248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=xdy50My4DuVwXzguIGfZQSfUqp%2F6E9vub23Fa7IdG%2Bg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKO74FDLU36CXAFYQYDVSCQHBANCNFSM52ONFDXA&data=05|01|jhalleux%40microsoft.com|ada657b2efbb44b7505c08da5c688a2c|72f988bf86f141af91ab2d7cd011db47|1|0|637923901317457248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Pf2iVf8Wg%2FOxoP8nMAnPdxK5fu%2F8l0QXGEqFrjylUt0%3D&reserved=0.
You are receiving this because you commented.Message ID: @.***>


TEST3 not accessible

'We're very sorry, but something went wrong'.
https://test3.freebmd.org.uk/
Not currently accessible.

Apparently it hasn't been set-up yet. @lemon-ukgen can you assist?

Issue is with us not lemon; we have not placed any code in the production folder for freebmd2; hence there is nothing to run. What do we want to put there and why and how urgent

@PatReynolds @richpomfret @DeniseColbert @AlOneill
Have activated BMD2 test3. Have activated registration into technical syndicate. Have cleaned out the freereg users from the userids. Have cleaned out the refinery pages of FreeREG pages. Only current members are myself, vino, pat and richard.
I suggest that all members of the current and future documentation team who need to work on refinery register and work on BMD test3

Hi @Captainkirkdawson thanks for this.  Having first work on documentation done on test3 makes sense.  I just tried going into test3.freebmd.org.uk Got an error (screen says it was logged).

@PatReynolds I broke the first law of programming. I did not test a late day code change. Fixed a spelling error. test3 is running now

Just tested sign up, and works fine.  One thing: do we need to de-mandate the Transcribers Agreement, as some volunteers may not want to do that (or perhaps more simply, say we'll reset this to not-subscribed manually)?

@Captainkirkdawson Oh no — not my day. Was told registration was successful and got email but on changing password got told that "Reset password token is invalid".
BTW, two observations: success message is magenta, so looks like a link (suggest teal or green — both in our CSS); message appears way down the page and would be easy to miss.

@AlOneill sorry that was my mistake; I changed your role on seeing your registration. Doing that invalidated the token. Have sent you another
WRT rest of message I am at a loss as to what, where and when these comments apply

@Captainkirkdawson TY. Have now set password successfully.
My comments apply to the refreshed Transcriber Registration page that appears on submission of form.

@AlOneill could that be raised please in the registration story whichever that should be 110 144 or 145. In this story all I have done is to make a production test version for people to use and to make sure that it works (clearly I failed initially). I will do nothing with it on this story and it does not affect this story.

Fair point — thought I would mention before I forgot: not a screen one sees often!
Raised issues in #144

Success; however, no Refinery button:

That is because all registration is as transcriber. I have to change your role and have done so.


Rename phpunit.xml to phpunit.xml.dist

This will allow for users to have a local configuration for unit testing.

Looks good. Thanks!


chore nereids support window function but some cases does not open yet (#33098)

Proposed changes
cherry-pick #33098

run buildall


Re-use Input component on Login and AddDevice screen

Hi @Pessina , I was testing the PR, and noticed that bug is introduced when selecting email domains, and it would be great if it can be addressed. I've made the recording of the issue, so you can check that out. Thanks!


Can you create nodejs:10-latest

Hi,
This package looks good.
npm install is failing with current nodejs:9 and nodejs:8
can you create a version 10 with npm 6?
I've have to do a workaround for now which is not suitable as a PR.
https://github.com/stephenmoloney/docker-jitsi-meet/tree/fix/bin-webpack-not-found

Yes, absolutely. I am using them in my private repos, so let me just sanitize them and them push them up to Github. It might take a day for Dockerhub to build the images as many are dependent on the base images of Debian/Alpine/Ubuntu which forces a rebuild of all. Otherwise you could build a copy locally for the time being while the automated build process completes.

Thanks . Yes, I'm using a manual build for the moment of nodejs containers.

@tiredofit any change to get your published Docker images tagged? Took me quite some time to find this issue. In my case the Kubernetes cluster nodes were upgraded, which caused the images to be pulled again from Docker which finally installed the broken image version. This would not have been happen if the images are tagged 😄

Hi, I tag my private images, but haven't been able to think about getting them tagged on Github/Dockerhub properly. But yes I will rebuild my workflow for the public Github images from not just :latest

Sounds great! Thank you

Hi @gruberro
I am 75% of the way done with tagging all images, so that you don't end up in a pickle again. All repositories that simply had tracked one version of a software package are tagged with :latest, the the verison number from my CHANGELOG.md. -
The repos that were once tracking multiple software versions out of master are being split to their own branches. :latest will always track whatever I deem as the most recent package release with the distro of my choice (usually Alpine), and there will also be a versionnum:latest e.g 5:latest or 5-alpine:latest as they are now. The difference is that if you look on the Dockerhub you will also see tags that look like 5_alpine-2.0 again, pulled from the CHANGELOG.md in that branch.
Thanks for the kickstart to get this done, I've been meaning to for a while on Github but it was just a matter of hearing voices requesting and carving out some time in my schedule to get this done. I hope this helps.

Hi @tiredofit!
That sounds great. I'm following the images I'm using and will update as soon as they are tagged!
Thank you very much! That helps a lot!!


Please add support of Angular 4 RC

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x ] feature request
Current behavior:
The current version of Ionic uses Angular 2.x which produces js bundles with larger size.
Expected behavior:
The current version of Angular 2.x produces js bundles with larger size. Whereas Angular 4 introduced a new View Engine in which they made some changes which reduce the size of the generated code for components by more than half.
Changelog: https://github.com/angular/angular/blob/master/CHANGELOG.md
View Engine Design Docs: https://docs.google.com/document/d/195L4WaDSoI_kkW094LlShH6gT3B7K1GZpSBnnLkQR-g/preview
Though Angular 4 is still in RC, but I believe its the right time to add support for it in Ionic 2. Since its mostly aligned with Angular 2.x so it shouldn't be an issue to add its support.
The only thing I see is that they have pulled Animations into their own package and if Ionic uses Angular 2 Animations (which I hope), it will require some updates to the animation parts.

Thanks for the issue! We are actively working on updating to Angular 4. There are a few steps we have to complete before we can drop this version in. 🙂

Sounds great :)

Are the "few steps" documented anywhere so that people can either pitch in or decide whether their apps are likely to run into problems?

I'd also like to know the impact of this change in the existing projects and also whether this is going to be optional.

Hello, thanks for using Ionic! We are now running Angular 4 on the master branch and it will be in the next release. We will have details about this update in the changelog for the next release. Thanks! (:


Nightly builds enhancements

Describe the feature
Copied from the comment:

The name format should be async-profiler-3.0-hash-arch.tar.gz (hash should go before arch, that is, be a part of the full version number).
PROFILER_VERSION should be set accordingly to 3.0-hash, so that asprof -v returns the version number with a commit id.
Include artifacts from build/jar (async-profiler.jar and jfr-converter.jar) when publishing a release.

Use Case

Improve developer experience.
Provide up-to-date binaries for JFR converter and async-profiler Java API.

Proposed Solution
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

We could run same tests (in MacOS, arm etc.) in all PRs, without updating the releases.


Intro to state in Compose: Android Basics with Compose

URL of codelab
Cómo usar la función calculateTip().
Error in translatation. In the first step the code is not written down in TipTimeScreen(). It is written down in EditNumberField().
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #85, #83, #82, #80, #78.

@TheDani1
Thank you for reaching out to us.
Duplicate of #97


Error handling

errors should at least result in the appropriate HTTP response status code

suggestions:

400 - bad request (in case of input error)
502 - bad gateway (in case of a missing mesos scheduler)

I add basic validation for submitted tasks using javax api.
Bad gateway error is produced if the response of the implementing framework has not successful "2xx" status code.
This implementation needs more testing (see #36 )


update np min to 122

Adoption of #9093

Merged in main post merge of #9105

gpuci run tests

Seems there's a problem with the docs build, so moved back to "Waiting on author".

Ah, the problem is that ufunc_db no longer imports numpy_version.

gpuci run tests

I ran this through the Build-Farm as numba_smoketest_cpu_yaml_117 and it passed fine. 🍏


Replace moment.js with Day.js

Hey guys,
thanks for keeping the datepicker alive.
As moment.js has been discontinued as of lately (https://momentjs.com/docs/#/-project-status/): is there any chance to switch to Day.js (https://day.js.org/)?

Hello @MacGyer! Sounds intriguing, but I guess it would require a whole refactoring of both tempusdominus-bootstrap (this repo) as well as its core tempusdominus and a new major version bump.
If you wanna start implementing this feature in new branch, you are welcome and I am willing to help and contribute as well, but I think it will require some time and effort because the overall codebase of this forked project is quite old and tightly coupled to Moment.
Let me know! Thank you!

BTW: Sorry for replying only now, but GitHub didn't send me a notification of this issue.

Hi @tonix-tuft,
thanks for getting back. Unfortunately, I am more a back end coder (PHP, SQL etc.) and not so much into front end dev (i. e. JS) :(
As I have seen there might be some interesting development in the original repo: https://eonasdan.com/posts/state-of-my-picker

I am closing this issue for now, because Eonasdan (the original maintainer of tempusdominus-bootstrap) has came back and maybe this integration will happen in the upstream project. We'll see what happens.
Cheers!


[bug] Selection criteria for Selection tool

The criteria for selection is "if the selection area touches even one pixel of the object it gets selected", and it is inconsistent with some other apps like Inkscape which there it is "if the selection area include all the pixels of the object"
Can we have an option at least that we can specify which selection rule the app would work with?
To me, the latter seems to have a lot of usecases for overlapping objects.
On a separate note, Selection tool keyboard shortcut is set to "V", is there any reason why it is not "S"? (similar to other tools which is basically the first letter?)
Thanks

You can hold control or command for the alternative selection mode. By default it selects anything the selection box is touching, but in the alt mode it selects only shapes that are entirely contained in the box.
We may bring the "swap default selection mode" feature up from the v1 project as a preference, depending on demand!


Pingy Web Scaffold

Hey @dansup, I created a Web Scaffold for Pingy CLI out of your Bulma templates, see here: https://github.com/pingyhq/pingy-scaffold-bulma
If you'd like to take ownership of the repo or anything like that then let me know.
You can close this issue :)

@davej Looks great, good work! I will add a "related projects" section to the readme and add this when I get a chance.


pipeline fails:  std::bad_alloc ERROR

I followed the instructions but the pipeline is failing with this message:
label: hierarchy (with options)
List of 4
 $ echo      : logi FALSE
 $ fig.height: num 8
 $ fig.width : num 8
 $ fig.cap   : chr "Hierarchy of immune cell types used for mapping cell types between methods and datasets."

terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
/bin/bash: line 2: 76154 Aborted                 (core dumped) Rscript -e "bookdown::render_book('index.Rmd')"
[Fri Jul 26 16:52:30 2019]
Error in rule book:
    jobid: 0
    output: results/book/index.html, results/cache/.dir, results/figures/schelker_single_cell_tsne.pdf, results/figures/spillover_migration_chart.jpg, results/figures/spillover_migration_all.pdf, results/tables/mixing_study_correlations.tsv, results/tables/spillover_signal_noise.tsv
    conda-env: /home/azubair/projects/immune_deconvolution_benchmark/.snakemake/conda/151e8d15
    shell:

    touch results/cache/.dir
    rm -f results/book/figures && ln -s ../figures results/book/figures
    cd notebooks && Rscript -e "bookdown::render_book('index.Rmd')"

        (exited with non-zero exit code)

I checked that memory usage doesn't go up at all, so I'm not inclined to think that it is because I am out of memory.
Any ideas why this could be happening? Thanks.

Could still be a memory issue... How much RAM do you have?
Also, what operating system?
On Fri, Jul 26, 2019, 23:57 Asif Zubair<EMAIL_ADDRESS>wrote:

I followed the instructions but the pipeline is failing with this message:
label: hierarchy (with options)
List of 4
$ echo      : logi FALSE
$ fig.height: num 8
$ fig.width : num 8
$ fig.cap   : chr "Hierarchy of immune cell types used for mapping cell types between methods and datasets."
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
/bin/bash: line 2: 76154 Aborted                 (core dumped) Rscript -e "bookdown::render_book('index.Rmd')"
[Fri Jul 26 16:52:30 2019]
Error in rule book:
jobid: 0
output: results/book/index.html, results/cache/.dir, results/figures/schelker_single_cell_tsne.pdf, results/figures/spillover_migration_chart.jpg, results/figures/spillover_migration_all.pdf, results/tables/mixing_study_correlations.tsv, results/tables/spillover_signal_noise.tsv
conda-env: /home/azubair/projects/immune_deconvolution_benchmark/.snakemake/conda/151e8d15
shell:
touch results/cache/.dir
rm -f results/book/figures && ln -s ../figures results/book/figures
cd notebooks && Rscript -e "bookdown::render_book('index.Rmd')"

    (exited with non-zero exit code)

I checked that memory usage doesn't go up at all, so I'm not inclined to
think that it is because I am out of memory.
Any ideas why this could be happening? Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grst/immune_deconvolution_benchmark/issues/28?email_source=notifications&email_token=ABVZRVZPEMMAMLIUUJHV2DTQBNXMBA5CNFSM4IHHOOFKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HBZZZXA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVZRV43JBFTNFCA24HPHKDQBNXMBANCNFSM4IHHOOFA
.

I'm using a Ubuntu 16.04 with 62 GB RAM. I didn't change the config.R file so should be using 2 cores.
Thanks!

Hi Asif,
Gregor is travelling right now, so I will try to answer. It looks in your
log like the script tried to allocate (reserve) ca. 75 GB of RAM which
failed on your computer. My guess ist that you did not see memory usage go
up because it could not be allocated in the first place. Once Gregor will
be back he can give you a more qualified comment or maybe some hints to
reduce memory usage.
Best,
Markus
On Mon, Jul 29, 2019 at 7:16 PM Asif Zubair<EMAIL_ADDRESS>wrote:

I'm using a Ubuntu 16.04 with 62 GB RAM. I didn't change the config.R
file so should be using 2 cores.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grst/immune_deconvolution_benchmark/issues/28?email_source=notifications&email_token=AAKWBXPOMCYLV4I7STX3GOLQB4QVTA5CNFSM4IHHOOFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BMKIY#issuecomment-516080931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKWBXMMJ7XVXJL5NFVTIQLQB4QVTANCNFSM4IHHOOFA
.

--
Dr. Markus List
Head of the Research Group on Big Data in BioMedicine
Chair of Experimental Bioinformatics
TUM School of Life Sciences Weihenstephan
Technical University of Munich (TUM)
Freising-Weihenstephan, Germany
e-mail<EMAIL_ADDRESS>tel:       +49 8161 71 2761
www:   http://biomedical-big-data.de
orcid:   https://orcid.org/0000-0002-0941-4168
twitter: @itisalist https://twitter.com/itisalist

Thank you, Markus.
Yes, it is strange that it tried to allocate 75 GB of RAM. I remember Gregor mentioning somewhere that we need 12 gigs per core and I thought I would be fine.
Of course, it would be great if Gregor could comment on this.
Thanks!

Hi Asif,
can you double check that the cores are really set to 2 in your config.R?
In an earlier version (few commits earlier) I had the default value higher.
I successfully ran the pipeline on Google colab with 12GB of ram, so in
principle you should have more than enough.
Best,
Gregor
On Tue, Jul 30, 2019, 22:00 Asif Zubair<EMAIL_ADDRESS>wrote:

Thank you, Markus.
Yes, it is strange that it tried to allocate 75 GB of RAM. I remember
Gregor mentioning somewhere that we need 12 gigs per core and I thought I
would be fine.
Of course, it would be great if Gregor could comment on this.
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/grst/immune_deconvolution_benchmark/issues/28?email_source=notifications&email_token=ABVZRV3X74E2KBJULE5NQ73QCBQQRA5CNFSM4IHHOOFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3EOSOY#issuecomment-516483387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVZRVYU5NCHVSNATZPOSQ3QCBQQRANCNFSM4IHHOOFA
.

Sure, will do. Thank you for your reply.

just wondering, @grst  any thoughts on releasing a docker image of this pipeline?

Personally, I'm not a big fan of docker... freezing the conda packages is (should be) another way of ensuring reproducibility.
In practice, it seems to me that neither of the two systems does a perfect job. I personally had trouble getting to run various docker containers and in particular memory issues could still occur.

Hi @grst - I tried running this pipeline on the cluster and had a bit more luck there.
However, the pipeline still fails and the error message is the same as the one here - https://github.com/icbi-lab/immune_deconvolution_benchmark/issues/24#issuecomment-502604687
>>> Running timer
## Enter batch mode

## Loading immune gene expression

## Removing the batch effect of /lsf_tmp/88991367.tmpdir/RtmpNmCJKH/filec3ec84ae9e0

Found2batches
Adjusting for0covariate(s) or covariate level(s)
Fitting L/S model and finding priors
Finding parametric adjustments
Adjusting the Data

Quitting from lines 570-591 (_main.Rmd)
Error in { : task 1 failed - "Recv failure: Connection reset by peer"
Calls: <Anonymous> ... withCallingHandlers -> withVisible -> eval -> eval -> %do% -> <Anonymous>
In addition: Warning messages:
1: Transformation introduced infinite values in continuous x-axis
2: Removed 3407 rows containing non-finite values (stat_bin).
3: In forceAndCall(1, FUN, newX[, i], ...) :
  closing unused connection 5 (http://raw.githubusercontent.com/ebecht/MCPcounter/master/Signatures/genes.txt)
4: In EPIC::EPIC(bulk = gene_expression_matrix, reference = ref, mRNA_cell = mRNA_cell,  :
  mRNA_cell value unknown for some cell types: CAFs, Endothelial - using the default value of 0.4 for these but this might bias the true cell proportions from all cell types.

Execution halted
[Wed Oct 30 12:42:32 2019]

One thing I did notice is this message at the start:
(base) [azubair@nodecn062 immune_deconvolution_benchmark]$ snakemake --use-conda
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided cores: 1
Rules claiming more threads will be scaled down.
Job counts:
	count	jobs
	1	book
	1

Don't understand why snakemake is using 1 core when I've specified 8 in the config ?! Do you think I need to pass number of cores to snakemake as well ? This is my specification in the config:
(base) [azubair@nodecn062 immune_deconvolution_benchmark]$ head notebooks/config.R
config = new.env()

registerDoMC(8)

Yes, I you will have to specify --cores. It defines the max number of cores snakemake has available.  (See https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html#threads).
Recv failure: Connection reset by peer sounds like something went wrong when downloading a file from the internet (EPIC fetches the signatures matrices directly from github). Do the cluster nodes have internet access? Otherwise it could just be bad luck and work if you re-run it.

Hi @grst  - I tried this again. I'm pretty sure that compute node has internet access as I clone the repository on the node itself. However, I am still getting the above error. Do you think the URL might have changed?

Hm, this is weird.
The CI still runs through, so MCP-counter seems to be able to fetch the files on the test server.
Maybe you try to get to run MCP-counter outside the pipeline to figure out what the actual problem is.

Sure, I'll try this again.
Also, @grst - I noticed the cell_type_map from the particular commit in innube_deconvolution_benchmark is different from the one included in the immunedeconv package (on the master branch). is this intentional? The package cell_type_map breaks some code in benchmark pipeline.

Yes, that's intentional (related to the discussion in https://github.com/icbi-lab/immunedeconv/issues/14). immunedeconv >= 2.0 is incompatible with the pipeline.

Hi @grst - Looking closely it seems the problem is with TIMER and not EPIC. Would you know why this might be happening? Thanks!

The error seems to happen in lines 570-591 of the intermediate markdown file (_main.Rmd):
Quitting from lines 570-591 (_main.Rmd)

Can you find the corresponding chunk and post it here?

Hi @grst . Apologies for getting back into this after so long.
This is the corresponding chunk (along with line numbers)
  569 ```{r, cache=TRUE, message=FALSE, echo=FALSE, warning=FALSE, results='hide'}
  570 timer_indications = rep("OV", ncol(schelker_ovarian$expr_mat))
  571 all_results_bulk = foreach(method = config$deconvolution_methods,
  572                            .final = function(x) {setNames(x, config$deconvolution_methods)}) %do% {
  573   deconvolute(schelker_ovarian$expr_mat, method, indications=timer_indications) %>%
  574     mutate(method=method) %>%
  575     mutate(source="bulk")
  576 }


Implemented FT 16

Implemented FT 16
JIRA Issue For FT 16: https://applab.atlassian.net/browse/NMS-181

please merge


Catch Database Outage Errors

Addressing issue #138.
Fixes issue where the process_tasks scheduler crashes when failing to retrieve tasks from a database. Now logs a warning that the database was unreachable and tries again later.

Thanks @jtimmons !


API - Branches with "/" cannot be accessed.

Gogs version (or commit ref): <IP_ADDRESS>7
Git version: /
Operating system: Docker
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[X] Not relevant

Log gist (usually found in log/gogs.log):

Hi,
There seems to be a problem in the API for branches, it is not possible to retrieve the information for a single branch if it has a "/" in it's name eg feature/dns :
http://gogs.local/api/v1/repos/Development/olomanagement/branches => shows the branch list with the branch but;
http://gogs.local/api/v1/repos/Development/olomanagement/branches/feature%2Fdns
does not give any result.
Thanks.

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.


occurence detail: mobile: occurrence drawer cannot be scrolled

On touch devices (or at least on my test iphone) the occurrence drawer content cannot be scrolled vertically.

This seem to be a well known issue in reakit/issues/469

It isn't the fix I had hoped for, but disabling modal on the reakit dialog solves it. Luckily it seems that are wanting to fix this issue. For now it has to be non-modal


Bump Git tags for kube-rbac-proxy, cri-tools, troubleshoot and govmomi

Bumping versions for some supplementary projects.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


1335 raster coverage conversion

@mjstealey The default call should be adequate. If a specific version number is needed, it is 0.1.3. Thank you.

+1


fix: npm natives module crash

ref: https://github.com/gulpjs/gulp/issues/2162#issuecomment-384380989

Thank you for the PR. I'm having problems reviewing diff on package-lock.json, so I regenerated it locally - does it fix the original problem?

Thank you for the PR. I'm having problems reviewing diff on package-lock.json, so I regenerated it locally - does it fix the original problem?

Sure, that's what I meant.


Questions about offsets

Hi ! Excellent work on making this implementation of Deformable Convolution.
When I used DCN to replace some of my backbone conv to train an network for object tracking problem, I found that simply replaced the conv to DCN didn't improve performance as expect and it decreased for about 6-8% on both precession and succese.
When I check the offset value during training, offset mean can reach about 100 or even larger.And the responds feature were much more ambiguous than normal convolution.
So is there any solutions to fix or contain the offsets during learning, so the receptive field can be limited to a fixed area?

好的，谢谢你
| |
@.***
|
|
@.***
|
---- 回复的原邮件 ----
| 发件人 | @.> |
| 日期 | 2022年04月12日 15:43 |
| 收件人 | @.> |
| 抄送至 |<EMAIL_ADDRESS>|
| 主题 | Re: [chengdazhi/Deformable-Convolution-V2-PyTorch] Questions about offsets (#69) |
你好，在可变形卷积的使用上我遇到了同样的问题。请问你最后怎么修改的？或者只是单纯地不使用了？
不太记得了，印象里是没查出为什么
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Non-latin fonts are incorrect

Hello! Thank you for a wonderful script!
Very cool thing, though - thank you!

This version of the script is not working (24 Dec 2016)
Russian fonts look horrible on http://hearthstonelabs.com/sunwell. (Maybe use BenguiatBoldITC and FranklinGothicURWConMed?)
How can I render a large number of cards on one page? (Do not write the text by hand?)

vs.


fix(cloudcommon): missing set_meta opslog

这个 PR 实现什么功能/修复什么问题:
修正：设置metadata的操作日志未记录

是否需要 backport 到之前的 release 分支:

release/3.4
release/3.6
release/3.5
release/3.7

/area util
/cc @zexi @yousong

Codecov Report

Merging #9914 (d9fbc78) into master (daaa708) will increase coverage by 2.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9914       +/-   ##
==========================================
+ Coverage    5.97%   8.05%    +2.08%
==========================================
  Files         849     107      -742
  Lines      132183   12991   -119192
==========================================
- Hits         7897    1047     -6850
+ Misses     123585   11824   -111761
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.05% <ø> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/db/tablespec.go

pkg/util/rbacutils/policygroup.go

pkg/compute/models/inter_vpc_network_vpc.go

pkg/compute/models/syncutils.go

pkg/mcclient/options/vpc.go

pkg/util/samlutils/types.go

pkg/keystone/models/groups.go

pkg/cloudcommon/db/list.go

pkg/keystone/models/fernetkeys.go

pkg/hostman/system_service/system_service.go

... and 730 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #9914 (d9fbc78) into master (daaa708) will increase coverage by 2.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9914       +/-   ##
==========================================
+ Coverage    5.97%   8.05%    +2.08%
==========================================
  Files         849     107      -742
  Lines      132183   12991   -119192
==========================================
- Hits         7897    1047     -6850
+ Misses     123585   11824   -111761
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.05% <ø> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/db/tablespec.go

pkg/util/rbacutils/policygroup.go

pkg/compute/models/inter_vpc_network_vpc.go

pkg/compute/models/syncutils.go

pkg/mcclient/options/vpc.go

pkg/util/samlutils/types.go

pkg/keystone/models/groups.go

pkg/cloudcommon/db/list.go

pkg/keystone/models/fernetkeys.go

pkg/hostman/system_service/system_service.go

... and 730 more

Continue to review full report at Codecov.

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


Document creating pre-compiled schemas for CSP with ajv-cli

What problem do you want to solve?
Easily understanding how to use ajv with a Content Security Policy that doesn't include unsafe-eval.
What do you think is the correct solution to problem?
The README.md should contain a section on how to use ajv-cli to create and use pre-compiled schemas. I found how to do this eventually, but only after lots of googling.
Will you be able to implement it?
Nope.

There is the part about  ajv-cli: https://github.com/ajv-validator/ajv/blob/master/README.md#command-line-interface
The structure definitely needs to be improved - there is a recent ticket about it.
It would be great if you could suggest the amendments based on your experience.


Adding the tool box to git

Merging Toolbox from RuwanY.

Merged.
On Tue, Apr 14, 2015 at 9:36 AM, Ruwan<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/wso2-dev/device-cloud-appliances/pull/1
Commit Summary

Adding the tool box to git

File Changes

A bam-tool-boxes/sensorDataSummary/IoT_Devices_Analytics.tbox
https://github.com/wso2-dev/device-cloud-appliances/pull/1/files#diff-0
(0)
A bam-tool-boxes/sensorDataSummary/WSO2DevicePlatform-2.0.0.zip
https://github.com/wso2-dev/device-cloud-appliances/pull/1/files#diff-1
(0)
A bam-tool-boxes/sensorDataSummary/db_configs.xml
https://github.com/wso2-dev/device-cloud-appliances/pull/1/files#diff-2
(20)
A bam-tool-boxes/sensorDataSummary/readme.xml
https://github.com/wso2-dev/device-cloud-appliances/pull/1/files#diff-3
(7)

Patch Links:

https://github.com/wso2-dev/device-cloud-appliances/pull/1.patch
https://github.com/wso2-dev/device-cloud-appliances/pull/1.diff

—
Reply to this email directly or view it on GitHub
https://github.com/wso2-dev/device-cloud-appliances/pull/1.


Update 2000-01-01-clip-rect.md

Cliprect is broken for JPG export, should make devs aware of it.

Correction, it's also broken for PNG, didn't realize it until after I brought it in Photoshop, the same excess space is replaced by transparent pixels (where it would be filled by black pixels in a JPG).


Improve logic in ibtracs wind speed reading

Currently ibtracs reader uses only wmo_wind if any values exist. Would be good to incorporate info from usa_wind if the points match wmo_wind at the reported intervals.

closed by commit 9020fb59ea1f5963f0533f644719ea40b22098ac


Add support for Audius.co embeds

Audius.co is a newer, high quality streaming service.
This pull request adds support for Audius embeds like so <ReactPlayer url='https://audius.co/embed/track?id=88754&ownerId=275&flavor=card' />
Tests have been included. Thank you in advance for your consideration.

I'm normally apprehensive about adding bytes to the package that won't be used, but:

it looks like a cool platform
people can use lazy players
I'm planning a rewrite that should cut the bundle size down
you clearly put a lot of effort into this

So I would be up for adding this. However, it seems to be broken? https://audius.co/embed/api.js returns a 404 for me, and I can't find any docs about the player API.

Thank you so much for getting back with me! I'm glad you're ultimately interested in getting it integrated.
From my understanding there was scheduled maintenance around the time you tested that embed link. It should be available now.
Also, there is no embed api documentation yet, but I will make submit a PR for the documentation. It is extremely similar to Soundcloud's Widget API, if that is helpful.

Hello @cookpete ! I wanted to see if there was anyway I could help move this forward and get this merged in. I believe everything should be compatible with your latest release.

Do I see something wrong here or is this just a plain embed with no controls? At least in the demo it is ignoring every command and does not report any value at all


Add ramineni to approvers

What this PR does / why we need it:
This PR is add ramineni to approvers list.
My contributions include :
https://github.com/search?q=is%3Apr+author%3Aramineni+org%3Akubernetes
https://k8s.devstats.cncf.io/d/13/developer-activity-counts-by-repository-group?orgId=1&var-period_name=Last decade&var-metric=contributions&var-repogroup_name=SIG Cloud Provider&var-country_name=All
And also I'm the author of the multiple features in CSI to get the driver in-par with intree and one of the maintainer of gate to make sure its always green :)
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/assign @lingxiankong @adisky
/cc @dims @hogepodge

/approve

/lgtm
welcome and it's great to have you :-)


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 4162284f724921b3022a652074b03b641fdfd454
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Update CONTRIBUTORS.md

Description
A comprehensive description about the Pull Request
Updating contributors List
Reference Issue number #(issue_no) #276
issue_no must refer to the issue related to this PR, visit here
Type of change
Choosing one or more options from the following as per the nature of your Pull request.
NOTE: Check boxes using [x]

[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ x] Documentation Update

Checklist:

[ x] I have named my files and folder, according to this project's guidelines.
[ x] My code follows the style guidelines of this project.
[ x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[ x] I have commented on my code, particularly in hard-to-understand areas.
[ x] I have created a helpful and easy to understand README.md, with problem description and my name.
[ x] I have included a requirements.txt file (if external libraries are required.)
[ x] My changes do not produce any warnings.
[ x] I have starred this repository.
[ x] I have added a working sample/screenshot of the script.
[ x]

I have checked for trailing spaces in file names and none have them.
[ x] I have added (your-name)[your-github-profile-link] in CONTRIBUTORS.md

@alvinuday Contributors.md now will be taken care off by maintainers. The process is now automated.


Add "file exists" parser

This parser checks that the given argument is the path of a file that exists.

Can you please add a few test cases for this parser?
Thx :-)

Done


windows: swap executable

fix #14

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 5 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

selfupdate.go
0
1
0.0%

swap.go
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
0

Relevant Lines:
305

💛  - Coveralls


checkbox and radio with colors

Hi all,
it could be interesting to use this lib : http://flatlogic.github.io/awesome-bootstrap-checkbox/demo/
to color checkbox or radio in survey

@xibman You may do it right now, using custom widget feature: http://surveyjs.org/examples/jquery/custom-widget-tagbox.html
You may add a new property "isAwesome:boolean", and use it in the isFit function.
Thank you,
Andrew

@andrewtelnov , Interesting.  How, though, would one go about integrating something like iCheck (http://icheck.fronteed.com/) to style the checkboxes and radios?

Also, I've noticed that the labels for checkboxes and radios don't have the 'for' attribute set.  So, if we wanted to style the checkboxes and radios purely with CSS, the technique of hiding the checkbox or radio itself wouldn't work, right, as making the label control the checkbox or radio element requires that the 'for' attribute be set?

@SlowburnAZ I have added in our TODO list the task to discuss using the "for" attribute.
And yes, we are going to make more examples with custom-widget feature. I personally want to make an example with select2. I have added the iCheck widget in our list of examples we have to create.
Thank you,
Andrew

@andrewtelnov Awesome.  Thanks!  Would love to be able to use that library.

@SlowburnAZ About "for" attribute. I have looked at the code. I definitely had to do it early, so sorry for the delay.
We are using the following rendering:

<input [radio or check] /> Text for check or radio item

"for" attribute is needed when label is separated from the input. For example:
Text for check or radio item
<input [radio or check]  id="MyInputId"/>
In our case, when input tag inside the label, "for" attribute doesn't do anything and doesn't provide any help.
PS: You may find interesting this discussion:  https://github.com/surveyjs/surveyjs/issues/281
We are going to show, how to use the most popular JavaScript widgets instead of the current rendering.
Thank you,
Andrew

@SlowburnAZ Here is the example with icheck widget: http://surveyjs.org/examples/jquery/custom-widget-icheck.html

@andrewtelnov In looking at the "renderAs", I could simply leave that as "standard", correct?  Meaning, I could not add the "icheck" renderAs property to metaData, not add the renderAs prop to the questions, and then in the "isFit" function, return true if renderAs = "standard", correct?

@andrewtelnov I've run into a quirk with the Matrix questions and using iCheck:  If you select an answer for one of the matrix questions, all answers of that type are then selected for all questions in the matrix.
Is there any way to pass the actual Matrix question whose radio button was clicked into the "afterRender" function?  Seems that currently only the entire question's  element is being passed, so every input with the same value is being selected.

@SlowburnAZ Yes, you may do not use "renderAs" property, but use isFit as:
isFit : function(question) { return question.getType() === 'radiogroup'; }
So, it will use iCheck for all radiogroup questions.
Let us check, how does it work for matrix. We have not tried it yet.
@dmitrykurmanov Could you try it out?
Thank you,
Andrew

@andrewtelnov ok

vue version: https://plnkr.co/edit/hTu6MYsWgObUUPmxfG1G?p=preview

@dmitrykurmanov Much appreciated, sir.

@SlowburnAZ you are welcome!


Migrate system module to Ballerina latest syntax

Purpose
$subject.

@Bhashinee there are some changes left in test files.


Cursor not centered in KDE

Hi, I tried your patch with the cursor but it doesn't work on KDE.
When I open inventory the block in the middle is actually hovered but cursor isn't there.
I tested this on Hyprland it works fine there. Maybe some kwin problem?

Seems like this has been fixed, works for me after https://github.com/BoyOrigin/glfw-wayland/commit/86eacdc9e70ef4b63dff8fe58042831bd50e6bb7.


It does not work instagram!

Stop working instagram Social feed js
I tried all the options with tokens and client ID.
Work stopped after the upgrade seems instagram.
before everything worked
demo GitHab instagram does not work

Does anyone have a working example - social feed js instagram?
As I understand it, no one works

is working great for me
I will not post here a working example because ID and token are things i don't want to share, but this is a screen shot of one instagram feed: http://grabilla.com/06708-50f5cb89-cd8b-4baf-baeb-2d736aa6c8ec.png
please read other post in the issues tab because we have already spoke about how to proceed to make it working,

Who can throw a working version with instagram can without keys and tokens?

And the problem is only with instagram, everything else works

I have it working with Instagram right now. My Insta profile is public, though. And you need permission to access the account in addition to the tokens and stuff.

And if I need to bring a photo of another account?
Now in the presence of the token and the client id can only get photos from your account, and I need to someone else (before work)

And what you have chosen in the preparation of a public token of the list?

Whomever has access to the Insta account will see a notice to give permission to the app. You can give your client/friend instructions on what to do.

The token will only work once the account gives permission from inside the account itself.


Some suggestions on adding children...

I think it would be better if pixi just had the same naming and interface for adding and removing children as the DOM: insertBefore, appendChild, removeChild. addChildAt isn't very useful at all imo, and makes code very prone to breakdown... insertBefore is much more useful and makes less prone to breakdown.
Naming: I have enough things to remember without having to remember that pixi.js uses addChild and removeChild instead of appendChild and removeChild.

Closing, this is unproductive.


fix: infobar: close icon button should be round

SUMMARY:
Updates the InfoBar close icon button to be round

JIRA TASK (Eightfold Employees Only):
ENG-39767
CHANGE TYPE:

[X] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[X] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
Pull the pr branch and run yarn and yarn storybook. Verify the InfoBar stories behave as expected.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c7ee085). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Current head 5348c09 differs from pull request most recent head 677c893. Consider uploading reports for the commit 677c893 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #500   +/-   ##
=======================================
  Coverage        ?   84.87%
=======================================
  Files           ?      784
  Lines           ?    16297
  Branches        ?     5622
=======================================
  Hits            ?    13832
  Misses          ?     2397
  Partials        ?       68

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add GitHub action for cloning an openQA job mentioned in a PR description

This GitHub action allows to clone a job when creating a PR by mentioning it in the PR description via @openqa: Clone https://….
Related ticket: https://progress.opensuse.org/issues/130934

Marking as ready again as I was able to test it and it works. Of course I only tested it so far on my fork but I guess that should suffice for now (see https://github.com/Martchus/os-autoinst-distri-opensuse/pull/2 for an example; I cancelled the cloned job manually). Later (or next week) I'll provide a PR to use it in the openSUSE distribution.

In addition to that, I suppose it would also make sense to treat OPENQA_HOST not as secret. Otherwise the log looks like this:
 Cloned '***/tests/3957590' into:
 - ***/tests/3959032

Or was there a reason why we even treat the hostname as a secret?


Create Weekly FW Mgr/Firewall backup schedule in ECF Assembly

Describe the Issue
We need to back up Firewall and Firewall Mgr via SCP Regularly
Additional Context

For Assembly only
role/policy to allow S3 bucket access
Firewall Manager and 1/2 Firewalls
Enable SCP server and create user in Windows Server
In GAIA, Schedule Job to kick off the system backup weekly locally (This Appliance)
Initialize GAIA access to SMS and Firewall (Public Key related)
Create a housekeeping script to clean up the backup in Windows Jump Server
Create a script in FW to clean up local backup

Acceptance Criteria

[x] GAIA portal access no problem
[x] SSH access via putty no problem
[ ] System backup schedule run regularly
[x] System backup upload to S3 bucket
[x] error message in the job log if the job failed

A set of pre-defined requirement that need to be met in order to mark the user story as “done”.  See below for team agreement details.

can manually launch system backup in GAIA.  SSH to SMS and Security Gateway and save configuration to text file.  SCP to transfer local copy in jump server to s3 bucket completed.

source code and documentation checked in to enterprise github repo.


[4.2] [ClangImporter] Handle lookup tables with many entries

Cherry-pick of #17265 to the 4.2 branch. Reviewed by @ahoppen.
rdar://problem/40828315

@swift-ci Please test


fix: add hover effect to cards

fixes #48
added a hover effect to the template to the "Our services include" section.
https://github.com/gauravsingh1281/Rentalog-Rent-Management-Web-App-For-Landlords/assets/66733074/ea4c16b0-844c-445b-9301-dd4846f22173

@pratikshelar546 Thank you for your contribution 😇

thanks for the merge 🙌 excited to contribute more in future❤


the same database for desktop and android

Hi all,
Obviously it should be discussed somewhere before but I was unable to find it :-/
Suggestion: make android app able to use the same data file as desktop version. This would let to live without exports/imports, just synchronization of data file (e.g. using cloud).

Yeah, you're right, this has been discussed a lot previously in several issues here on GitHub.
Unfortunately, it's not going to be happening any time soon due to feature incompatibility between the apps, and also to avoid data corruption

This is almost possible now, if you do an XML export and overwrite the gnucash file you imported, you basically get this. Just need an option to save to a particular file instead of also to an app-specific directory.

Merge into #567
Issue will be tracked there

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain ownyourbits.com has exceeded the max emails per hour (3/3 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o8.sgmail.github.com ([<IP_ADDRESS>]:26742)
by skywalker.onesite.co with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1cVaJD-0015aH-IN
for<EMAIL_ADDRESS>Mon, 23 Jan 2017 03:49:03 -0500
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=vlEPM/Vwd8zz4kzdgj5rI9riOf4=; b=QMDjL1WwZrH5V8dt
hxhyVISSm+X2sNLvyw1hFO/aQP+vMWITLwuJgnwDvXr/oTQtt1OgfWS2e/C4mHDG
3CIrTdjduipSdEsgx7w25l121joRVFv3rqj8s1Lrppy+qc942Z1GbKcKY0gHIaC2
JNpXl3pTE2bHcrUk8AIiQHzLuBw=
Received: by filter0942p1mdw1.sendgrid.net with SMTP id filter0942p1mdw1-12088-5885C353-4E
2017-01-23 08:48:19.67059267 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0003p1iad1.sendgrid.net (SG) with ESMTP id 6QFkDq-fTpmMcWZGQZfmXQ
for<EMAIL_ADDRESS>Mon, 23 Jan 2017 08:48:19.731 +0000 (UTC)
Date: Mon, 23 Jan 2017 00:48:19 -0800
From: Ngewi Fet<EMAIL_ADDRESS>Reply-To: codinguser/gnucash-android<EMAIL_ADDRESS>To: codinguser/gnucash-android<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [codinguser/gnucash-android] the same database for desktop and
android (#375)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5885c35396a05_3e553fcf29971138565961";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: codinguser
X-GitHub-Recipient: nachoparker
X-GitHub-Reason: subscribed
List-ID: codinguser/gnucash-android <gnucash-android.codinguser.github.com>
List-Archive: https://github.com/codinguser/gnucash-android
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AUWsXH9ak1r8JEfkZEGr6R3M-Ig5Nb-Hks5rVGlTgaJpZM4F35QL
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: bZZJyvIOioDBIv7or3Raig/qRxFLJCSVN5+R3lL3f3h30LJsXKYqLlKbqNudjqAo9mis44ihhOtuut
muAseDgphQvx8D/0EWxMcPAtefP6vgnxtgfKyrAlTTovv9v98wMFRWQi5OFrGx5AhgUy/f6TKcKBF1
GeeUqldxVUWmwPx0al/Zp660eZF4zDib6dfvZp/Wlj4laD6e9ZAXtxMAE9QOPphrrCQRJerkFVLebh
k=
----==_mimepart_5885c35396a05_3e553fcf29971138565961
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #375.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/codinguser/gnucash-android/issues/375#event-932457503
----==_mimepart_5885c35396a05_3e553fcf29971138565961
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #375.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5885c35396a05_3e553fcf29971138565961--


Getting "Process didn't exit successfully: target/debug/rusted (signal: 11)" when using into_raw_mode()

Hello Ticki,
Im trying to use libterm on macosx to play around with terminal UI stuff and Im getting that error on this line:
let mut stdout = stdout().into_raw_mode().unwrap();
You got any clue on what could be going on?
Thanks!

I debugged with gdb and got this:
Program received signal SIGSEGV, Segmentation fault.
0x00000001000048be in rusted::raw::W.IntoRawMode::into_raw_mode (self=<error reading variable: Cannot access memory at address 0x95f8>)
at /Users/user/.cargo/git/checkouts/libterm-cfb8814c8a0f5aca/master/src/raw.rs:67
67	        let (mut ios, err) = get_terminal_attr();

Great catch. Unfortunately, I don't have a Mac OS X computer, so I need you to give me a stack trace. Signal 11 is SIGSERV (segfault), which is definitely not intended. Also, can you give me your Rust version number and head commit?
Thanks
/ticki

Oh, you already answered my request ;P

rustc 1.7.0 (a5d1e7a59 2016-02-29)

What happens if you switch to nightly?

I wish I knew how to do that lol. I'll look into it.

Check out the Rust book, it got instructions. There is also multirust, which manages Rust versions.

Okay, I changed to nightly and got the exact same error on gdb.

You're running the simple example, right?

And you're on the latest master?

Yes and yes

Let me know if you need any more info

Unfortunately, I am not able to reproduce it, so it is near to impossible for me to fix it.

Do you have a gitter chat or something we can chat in real time about this? I can try and find a way to fix it.

I'll have to go for a lecture in half an hour, so I can't do it right now, but if you're on IRC, we can chat, when I get home again.

I am in IRC yes. What channel are you in?

I don't have time right now, but in a few hours, I'll get on mozilla IRC (username is ticki).

Cool, I'm there all the time, nickname is nkr

Ok, thanks.

I'm on the chat now.

This is fixed now.


It is possible call shouldRefreshToken when expire with a expire date then call it?

Hi @lucascerci, not sure what you mean by "when expire with a expire date", could you elaborate a little bit more?

today we doing like this
const shouldRefreshToken = error =>   error.response.status === 401;
when the request return 401 we refresh token but its not cool let the system loose the permission to get again, i thinked in something like:
const shouldRefreshToken  = when token.expire(date) is about to expire, call the refresh again. im doing this already with the code and timeouts but yet its not perfect

Oh, I see. Basically when you know the date & time when token will expire, then refresh it just before it expires, not when you get the first 401 response, right?
Hm, you cannot really do that with the current refresh-fetch package, because shouldRefreshToken is called only when there is an error from request, not before every request.
But it's actually a simpler setup, which can be implemented in just a few lines without the need of refresh-fetch package I think. Something like:
import merge from "lodash/merge";

let token = null;
let tokenExpiresIn = null;

const refreshFetch = (url, options = {}) => {
  let refreshTokenPromise = Promise.resolve();
  if (shouldRefreshToken()) {
    refreshTokenPromise = refreshToken();
  }

  let optionsWithToken = options;
  if (token !== null) {
    optionsWithToken = merge({}, options, {
      headers: {
        Authorization: `Bearer ${token}`,
      },
    });
  }

  return refreshTokenPromise.then(() => fetch(url, optionsWithToken));
};

const shouldRefreshToken = () => {
  if (tokenExpiresIn === null) {
    return true;
  }

  const now = new Date().getTime();
  const willExpireInAMinute = tokenExpiresIn < now + 60000;

  return willExpireInAMinute;
};

const refreshToken = () => {
  // Do the refreshing and set token & tokenExpiresIn
};

By now im using your package as a second possibility to user stay logged in. Its working like this but i will soon when i get back to it try to do like your example with promises, ty, i will close it now
//first timout after loggin or reload page
const refreshTokenWithExp = () => {
if(retrieveToken() != null) {
setTimeout(() => {
refreshToken()
}, calculateTimeToRefresh() - 10000); //10000 = 10 seconds before token expire
}
}
refreshTokenWithExp();
 const saveToken = token => {
Cookies.set(COOKIE_NAME, token, {domain: process.env.REACT_APP_BASE_COOKIE_URL})
//when the system save a new token, its called again a refresh token with the calculate time to refresh
setTimeout(() => {
refreshToken()
}, calculateTimeToRefresh());
};`


Segment Violation when using fullhook.so with FireFox on Ubuntu 16.04 (xenial)

Fullhook.so and basehook.so work as described on Ubuntu xenial with applications socat and vi, but not with FireFox.

I suspected that this was caused by the large number of threads that FireFox starts, but the cause is in the following line:
char *p = (result < strlen(s_fullmagic)) ? NULL : strnstr(buf, s_fullmagic, result);

and more specifically in the comparison:
(result < strlen(s_fullmagic))

This is because result is of type ssize_t, which is signed, but strlen() is of type size_t which is unsigned (at least in this case). I am surprised that this hasn't bitten me more (no warnings about this and I'm sure I've done it before). When result is -1, the comparison fails because it is compared to strlen() as a large unsigned value. The correct way to compare size_t and ssize_t then is:
(result < (ssize_t) strlen(s_fullmagic))

The change was made in src/fullhook.c and src/halfhook.c and will be included in v1.2.2.

Closed with release of v1.2.2. (Note: While readhook is now playing well with firefox, upon the actual overflow, a reverse-shell is nevertheless not created. The attempt appears to be trapped by the firefox sandboxes but isn't yet clear. A new issue for further exploration will be created.)


Kg/api format

Fixes a formatting issue.

This PR isn't going to pass CI because when you compiled the protos this file got deleted: api/interservice/cfgmgmt/service/cfgmgmt.pb.client_mock.go. I had that problem, and I was able to workaround it be deleting everything in the bin/ directory of automate and recompiling the protos.

Thanks for letting me know!

You might also be able to fix it by checking that file back in (ie, undo the deletion) but it'll just keep happening when you compile the protos.

i told kimberly i would work on getting pr rebased and green on CI so we can merge.
the commits are being real funky so i've decided to go the path of least resistance: grab the actual changes from this branch and start a new branch fresh from master with just those changes (proto) and recompile all the things. will hopefully have a new pr up soon


[Request] Hosts file version

I don't wanna just filter the sites, but block them fully. Could we get an autogenerated file that's in hosts file format, using <IP_ADDRESS>?
(There are apparently simple scripts to generate such files, but I don't know any particular ones. Github Community to the rescue!)

So, I've just created a hosts file using <IP_ADDRESS>, so it should work just fine (tested on my os' hosts file). Test it out and tell me how it goes!

Bravo, and I luv the comments!
Thank you, this is just what is needed. Bye-bye AI!


prep commits for "Change stream field in composebox to be a dropdown"

Prep commits for #23355

@timabbott would love a review on this to unblock the next PR! Thanks :)


Implement rocket reservation

Added:

[x] reserveRocket action to reserve a rocket
[x] cancelReservation action to cancel the reservation of a rocket
[x] A reserved badge that shows when a rocket is reserved
[x] Implementation of display of 'Reserve Rocket' button or 'Cancel Reservation' button, depending on whether the rocket is reserved or not

STATUS APPROVED 👍
Your project is complete. There is nothing else to say other than is time to merge it.
Congratulations
Happy coding


Implemented RemoteRequestHistory into retrieval of possible Twitter handles, merged keywords lists

Implemented RemoteRequestHistory into retrieval of possible Twitter handles to check whether the search has been done on a candidate before (#568). Fixed a bug preventing the status message parameter from being added to the RemoteRequestHistory object. Merged keywords lists from Google and Twitter searches into one common list.

👍


Deprecate RSSLink

I have added disableKinds = ["RSS"] to my configuration file, which does indeed disable RSS accross the site.
However the following snippet of HTML is still output to the page, even though its wrapped in the conditional check. I would have expected this to be false and for the HTML to not output.
{{ if .RSSLink }}
    <link href="{{ .RSSLink }}" title="{{ .Site.Title }}" type="application/rss+xml" rel="alternate">
{{ end }}

This then causes a 404.

You need to do something a la:
https://github.com/gohugoio/hugoDocs/blob/master/themes/gohugoioTheme/layouts/_default/baseof.html#L25
RSSLink will be deprecated and eventually removed.

Ah, thank you for the quick response 👍

The warning in 0.55 is slightly misleading (and this deprecation is not mentioned in the release notes):
WARN 2019/04/09 21:04:47 Page's .RSSLink is deprecated and will be removed in a future release. Use the Output Format's link, e.g. something like:
    {{ with .OutputFormats.Get "RSS" }}{{ . RelPermalink }}{{ end }}.

When I use the recommended snippet I get this:
parse master failed: template: index.html:19: function "RelPermalink" not defined

It looks like the snippet in the warning message needs to be changed to this (no space between the . and RelPermalink):
{{ with .OutputFormats.Get "RSS" }}{{ .RelPermalink }}{{ end }}.


Update DPG: Bisa Health

Complimentary to #602

Bisa Health initially submitted with several gaps in the DPG standard indicators which need to be addressed before we can proceed with the review:

[ ] Clear Ownership
[ ] Platform Independence
[ ] Adherence to Standards & Best Practices
[ ] Adherence to Privacy and Applicable Laws
[ ] Do No Harm
[ ] Protection from Harassment

I have contacted the person who made the submission and will update once they come through.


Backgroud opacity not backwards compatible

Same holds true for the Text Component's opacity option but text still shows up

Bugs cased by:

#338
#334


Possible hiccup during jubilinux flashing

While flashing my Edison from Ubilinux to Jubilinux it timed out waiting for the reboot after repartitioning
Rebooting to apply partition changes
Now waiting for dfu device 8087:0a99
Timed out while waiting for dfu device 8087:0a99
Running it again, successfully flashed the device.

Were you watching it on console at the time? Did it give you a Ctrl-D prompt?

The device rebooted to the previous install, verified by logging in and checking configs i had there(so same all around)

An additional note, I rebooted again just in case too.  Then just ran flashall.sh again


Misspell linter does not detect many typos

I'm not sure why this happens, this issue is just so that it is documented.
See the comments in this PR for a couple examples: https://github.com/EduardGomezEscandell/GoWSL/pull/11

Closing as we can't really do much about it


Remove the deprecated custom output wrapper.

git grep utputWrapper does not show anything anymore after this change.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4079/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5402/
Test PASSed.


Replace is room available

Noticed this was deprecated I think I got the logic down to something that would resemble the current functionality.

Maybe I'm just blind but I can't find the method Room.isAvailable() in the docs. Did you mean this?


Selected and setSelected can't be the same Constant (SDK 49)

selected is expecting a string while setSelected is expecting "undefined"

I know what you are saying.
I am aware that it should work like that.
But I am trying to tell you that it doesn't work. I tried to put the variable as selected and the function which updates the variable as setSelected but it throws me an error.
It is complaining that the Variable is a String.
I Provided you with the code:
export default function (){
    analytics().logAppOpen()
    const [selected, setSelected] = useState('+91');

    return (
            <View style={styles.container}>
                <CountryCodeDropdownPicker selected={selected} setSelected={setSelected} setCountryDetails={}/>

And the error:
TS2322: Type 'Dispatch<SetStateAction>' is not assignable to type 'Dispatch<SetStateAction>'.   Type 'SetStateAction' is not assignable to type 'SetStateAction'.     Type 'undefined' is not assignable to type 'SetStateAction'.

Hey Oliver,
As I am checking your code, I see that you are passing nothing to the prop setCountryDetails, which expects a setState function.
It is the one throwing the error. If you don't have to use country details, simply don't pass that prop. It will solve the issue.
So, the modified code should look like:
export default function (){
    analytics().logAppOpen()
    const [selected, setSelected] = useState('+91');

    return (
            <View style={styles.container}>
                <CountryCodeDropdownPicker selected={selected} setSelected={setSelected} />

Hey removing setCountryDetails doesn't change the Issue.
To show you my Problem I made a Screenshot just to not confuse you anymore:

also:

Have you tried compiling the code. It think it is the warning thrown by eslint for typescript in the IDE or something like that.
I think after compiling it should work perfectly. As the package has around 1k users and no one has raised any complaint like this.


Fix Menu Alignment

Fix centering of menu items. Menu is not vertically centered

Issue fixed in branch feature-menu-fixes 9/27


User is still hard coded in auth.js

Look as well to see if we can protect the client secret.

fixed


Not and Issue just ask some question :)

Hai
Currently I'm learning about vue3 + typescript + quasar, and I found your amazing repository, I like your design pattern, the way you make src/models and src/service is easy to understand
I have some question

Why you make interface ProjectModel and ProjectDTO, what the reason ?
My background is backend developer (laravel), I have a simple api for blog : post, post_category, tags can you help me make a frontend crud about it

Is there any other way other than issues to communicate with you ?
Thanks :)

Hi) glad to hear that))

It seemed to me that it would be a good idea to separately describe how the data would look specifically for the request API - they may differ. For example, in ProjectDTO we do not have a users field, because this is managed from the backend and cannot be changed through the frontend, etc. Or are you asking about something else?
Of course, how exactly can I help you? Are there any difficulties or questions?

discord - Selithrarion(Sergey)#5794
telegram - selithrarion09


Fix the example with Expression Language

@stof said in https://github.com/symfony/symfony-docs/pull/17905:

When evaluating the expression, the ExpressionLanguage does not have access to private properties of the object, as it runs from its outside. And contrary to Twig, ExpressionLanguage does not have the magic . operator that tries to find a getter when it cannot use the property.

I then improved the documentation :).

Thank you @seb-jean.