Tests

90% code coverage reached

Coverage increased (+6.04%) to 90.65% when pulling 6e6d5ff6675ac3d93ad81c825ec1f89311783425 on tests into eadd03db1ecd73445e594e1dd3163ebc31ddd596 on master.

:clap:


Added token authentication

It is now possible to restrict access to the dashboard using a request token by defining  the OPCACHE_DASHBOARD_TOKEN environment variable.

This seems like a great idea to protect the risky features. What do you think about making it "secure by default" so that requests are rejected unless the token is present and an admin can disable the token by setting the value to an empty string? As currently built this requires extra effort to enable the security feature. IMO it should work in reverse where disabling the security requires extra effort.


Fix size param

Hello.  Thanks for sharing this -- it works well.
I could not get it to respect maxsize 1G but it seems to respect size 1G.  According to the Ubuntu man page it also looks like size may be the correct parameter here.
http://manpages.ubuntu.com/manpages/hardy/man8/logrotate.8.html

Hi there,
We use log rotate 3.8.7 which supports maxsize: http://serverfault.com/questions/474941/how-to-rotate-log-based-on-an-interval-unless-log-exceeds-a-certain-size
size only rotates at most once per day, so maxsize is the correct config setting here.
I'm surprised it didn't work for you, as we have tested this image ourselves. It runs every 5 minutes. Can you provide a reproducible set of commands?

Thanks for the follow-up.  I've just tested again with maxsize 1G and it works great.  I must have missed something the first time.  Haha :wink:


Add support for managing Redis dnfmodule on EL 8+

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

I can't seem to see what I am missing to make the requires relationship work in testing:
  1) redis on centos-8-x86_64 without parameters with manage_dnf_module true is expected to contain Package[redis dnf module] that comes before Package[redis]
     Failure/Error: it { is_expected.to contain_package('redis dnf module').with_ensure('6').that_comes_before('Package[redis]') }
       expected that the catalogue would contain Package[redis dnf module] with that comes before Package[redis]
     # ./spec/classes/redis_spec.rb:98:in `block (6 levels) in <top (required)>'

  2) redis on centos-9-x86_64 without parameters with manage_dnf_module true is expected to contain Package[redis dnf module] that comes before Package[redis]
     Failure/Error: it { is_expected.to contain_package('redis dnf module').with_ensure('6').that_comes_before('Package[redis]') }
       expected that the catalogue would contain Package[redis dnf module] with that comes before Package[redis]
     # ./spec/classes/redis_spec.rb:98:in `block (6 levels) in <top (required)>'

  3) redis on redhat-8-x86_64 without parameters with manage_dnf_module true is expected to contain Package[redis dnf module] that comes before Package[redis]
     Failure/Error: it { is_expected.to contain_package('redis dnf module').with_ensure('6').that_comes_before('Package[redis]') }
       expected that the catalogue would contain Package[redis dnf module] with that comes before Package[redis]
     # ./spec/classes/redis_spec.rb:98:in `block (6 levels) in <top (required)>'

  4) redis on redhat-9-x86_64 without parameters with manage_dnf_module true is expected to contain Package[redis dnf module] that comes before Package[redis]
     Failure/Error: it { is_expected.to contain_package('redis dnf module').with_ensure('6').that_comes_before('Package[redis]') }
       expected that the catalogue would contain Package[redis dnf module] with that comes before Package[redis]

To address the failing unit tests that are unrelated -- https://github.com/voxpupuli/puppet-redis/pull/452


Memory leak using pyarrow.Table and to_pandas method

Describe the enhancement requested
I'm trying to put dataframes of different types in multiprocessing shared memory. When I'm using dataframes that contain python types I've no problem, but when I use pyarrow types I've problems closing the Shared Memories. I've a function to put a dataframe to the Shared Memory and another to receive this dataframe. I have two problems when I'm using Arrow types like string[pyarrow] inside a dataframe:
When I call the close on the sm_put and after receiving the dataframe I call the close on the sm_get I get instantly an error, that causes the code to stop executing: sm_get.close() -> self.close() -> self._buf.release() -> BufferError: memoryview has 1 exported buffer.
When I call the close on the sm_put, and I don't call the close on the sm_get, I've no problem instantly, but I get a warning after a while (that surprises me because I don't call any function self.close()): self.close() -> self._buf.release() -> BufferError: memoryview has 1 exported buffer.
My functions to put the dataframe to the shared memory and to receive the dataframe from shared memory are these:
def put_df(data, logTime, logSize, usingArrow=False):
    if(usingArrow):
        table = (pyarrow.Table.from_pandas(data)).combine_chunks()
        record_batch = table.to_batches(max_chunksize = sys.maxsize)[0]
    else:
        record_batch = pa.RecordBatch.from_pandas(data)
    # Determine the size of buffer to request from the shared memory
    mock_sink = pa.MockOutputStream()
    stream_writer = pa.RecordBatchStreamWriter(mock_sink, record_batch.schema)
    stream_writer.write_batch(record_batch)
    stream_writer.close()
    data_size = mock_sink.size()

    sm_put = SharedMemory(create=True, size=data_size)
    buffer = pa.py_buffer(sm_put.buf)

    # Write the PyArrow RecordBatch to SharedMemory
    stream = pa.FixedSizeBufferWriter(buffer)
    stream_writer = pa.RecordBatchStreamWriter(stream, record_batch.schema)
    stream_writer.write_batch(record_batch)
    stream_writer.close()

    del stream_writer
    del stream
    del buffer
    if(usingArrow):
        del table
    del record_batch

    sm_put.close()
    return sm_put.name

def get_df(sm_get_name, logTime, logSize, usingArrow=False):
    sm_get = SharedMemory(name = sm_get_name, create = False)
    buffer = pa.BufferReader(sm_get.buf)

    # Convert object back into an Arrow RecordBatch
    reader = pa.RecordBatchStreamReader(buffer)
    record_batch = reader.read_next_batch()

    # Convert back into Pandas
    if(usingArrow):
        data = record_batch.to_pandas(types_mapper = pd.ArrowDtype)
    else:
        data = record_batch.to_pandas()

    del buffer
    del reader
    del record_batch

    sm_get.close()
    sm_get.unlink()

    return data

if __name__ == '__main__':
    names = [ "0", "1"]
    types = { "0": "string[pyarrow]", "1": "string[pyarrow]" }
    df = pd.read_csv(open(file, "r"), index_col=False, header=None, names=names, dtype=types, engine="c")
    sm_get_name = put_df(df)
    data = get_df(sm_get_name=sm_get_name)]

I'm using pyarrow 11.0.0 and pandas 1.5.3 and Ubuntu

You do need to close sm_get but you can't safely close it until after you are done using it.  Note that the to_pandas conversion could very well be zero-copy.  This means that data may reference portions of the underlying buffer.  This means you can't close sm_get until you've finished using (and deleted) data.

Yes that was the key. I've solved that calling del on data before closing the shared memory. The problem that I've now is that when I try to del the table is not deleted instantly and the memory reserved it's not released.

Do you have a repeatable example?  I find that python's garbage collector is difficult to predict in complex cases.  For example:
def destroy_table(table):
  del table

This method will never actually delete table because there will be at least one additional reference to it...
table = ...
destroy_table(table)
print(table) # table is still valid

So I think we will need an updated example to understand what is not working.


Get Apache packages from archive.apache.org instead of mirror

While building an image on top of this using some of the defined environment variables (APR_UTIL_VERSION, etc.), I noticed that the mirror.ventraip.net.au links for the pinned versions are no longer valid, since the mirror only has the latest version.
Hence, I changed the links to point to archive.apache.org which should (hopefully) be stable for much longer.
Also, the images wouldn't build from scratch from the current master, because the rm for /var/lib/apt/lists was failing out with a "No such file or directory." I added a -f to fix that.
Testing: Images built without errors from the build.sh script.

Apache people must have changed their procedures. The archive only used to have old versions and not the latest. It was always really annoying as I couldn't find a stable URL that would always work.

The archive only used to have old versions and not the latest.

Hm. If that was the case some time in the past, the archive now says “This is simply a copy of the main distribution directory with the only difference that nothing will be ever removed over here.”  I verified that it does contain the latest stable releases.
So hopefully downloading the pinned versions from there should be okay, even if they change in the future.

So you know, it is very difficult for me to update the images based on this repo that are on Docker Hub these days. This is because Docker Hub has for a while now blocked building with an automated build, any Dockerfile which tries to run the setcap program even if it is only to allow a program to bind to port 80.
I can make the changes to the repo, but I don't have a simple way of pushing new images to Docker Hub with newer versions of Apache and mod_wsgi. This is because my home Internet connection is too slow. I hesitate in trying to set up some CI service which can do it as this particular image isn't something I really do anything with now as I have better images have been working on and using instead, which aren't restricted from being built by Docker Hub and which use better practices in how they work.
If interested in those images and using then, can maybe drop an email to mod_wsgi mailing list and I can start discussing them there.

Seems Docker Hub may have lifted the restriction on use of setcap. Am updating images to latest versions of everything.

Perfect, thanks a lot!


[PROD] Allow List Update Email cleanup

Summary
When a CLA manager receives an email notification that the Allow list has been updated, the email includes a html> tag at the beginning.

Receive an email that the allow list has been modified.
Observe the email contains an html tag missing the opening symbol and the formatting may need to be cleaned up a bit.

Screenshots

Environment
Please complete the following information:
PROD

I think this email has been resolved - need to confirm.

@nirupamav

Tested and observed that there is blank tag in the "reach out to us for support" link and also email needs to be formatted

Need to convert the API call from v2 to v3.

Observed that the issue is fixed


Use flow-bin to typecheck locally, update ESLint to 2.0

Using local version of the Flow make it easy to have project specific version instead of relying on the user to have the correct version installed globally. For example, React Native uses an older version of Flow, while I have the latest version installed. Now I cannot typecheck the code because my version doesn't match the .flowconfig.
Test plan (required)
Run npm run lint and npm run flow to run eslint and flow.
cc @bestander @mkonicek

@facebook-github-bot import

Can you clarify the "use local tools"? Your explanation doesn't make sense in the context of how npm works. It might make sense for flow, but not for jest or eslint.

@cpojer Updated!

Waiting for @cpojer to approve this.
I'll reimport, because this touches npm deps, I'll need to update internal npm deps with a script.

Yeah I think this makes more sense.

Thanks for doing this!
Please hold on with merging this until Travis tests are back to green to be safe.

Added tag so that we come back to this next week

@satya164 maybe we can remove global flow install in Travis then?
https://github.com/facebook/react-native/blob/master/.travis.yml#L18

@bestander done!

e2e tests are still broken so we can't merge this yet, thanks for the patience.

@facebook-github-bot import

shipping...


Add mini.pick pickers to mini.extra for listing and toggling breakpoints, scopes and frames of nvim-dap.

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.pick, mini.extra
Description
Hi echasnovski! Is it in scope to add pickers to leverage nvim-dap functionality to toggle breakpoints, scopes and frames to mini.extra just like how fzf-lua does with commands like dap_breakpoints, dap_variables and dap_frames. And if you are ok with including these in mini.extra and are ok with me contributing, I can probably try raising a PR for these pickers if can get them to work to mini.extra.

Thanks for the suggestion!
I don't think including anything debug/breakpoint/DAP related to 'mini.nvim' makes sense before there is a relatively straightforward, organized, and documented way to deal with them. Either as 'mini.nvim' module or Neovim's built-in functionality (like with vim.lsp, for example). At the moment, I don't think any of these will happen, I am afraid.
Closing as not planned. But if you end up implementing any of them, would you mind making it somewhere public and linking it here? Maybe they will be useful for others.

Thanks for the suggestion!
I don't think including anything debug/breakpoint/DAP related to 'mini.nvim' makes sense before there is a relatively straightforward, organized, and documented way to deal with them. Either as 'mini.nvim' module or Neovim's built-in functionality (like with vim.lsp, for example). At the moment, I don't think any of these will happen, I am afraid.
Closing as not planned. But if you end up implementing any of them, would you mind making it somewhere public and linking it here? Maybe they will be useful for others.

Sure, I will make sure to post it in discussions.

Ok I just did some research on this and based on this comment from nvim-dap author itself https://github.com/mfussenegger/nvim-dap/discussions/810#discussioncomment-4623606, there is currently an internal api like require('dap.breakpoints').get() but the cavecat is that it is internal and not meant for external use and its arguments can change anytime without notice since it is internal. Also about scopes and frames there is already dap.ui.widgets for that and they can be used to show both scopes and frames using the following remaps.
lua	 Nmap("<leader>ds", ":lua require('dap.ui.widgets').centered_float(require('dap.ui.widgets'.scopes)<CR>", "Scopes") Nmap("<leader>df", ":lua require('dap.ui.widgets').centered_float(require('dap.ui.widgets').frames)<CR>", "Frames")
The first one is for scopes and the second one is for frames.
As per breakpoints we have require("dap").list_breakpoints() which will populate quickfix list with all the breakpoint lines.
We also have require("dap").clear_breakpoints() to clear all breakpoints.
So the clever way to deal with specific bulk breakpoint toggling is probably to delete unecessary items in the quickfix list and just use some command like cdo and call the toggle_breakpoint function on that line is probably the way to go.
Since the quickfix list is populated when we call list_breakpoints() method, we can probably view it using mini.pick using something like MiniExtra.pickers.list({scope = "quickfix"}).

Yeah, this looks similar to what is done in 'ibhagwan/fzf-lua'.

Yeah, this looks similar to what is done in 'ibhagwan/fzf-lua'.

Yeah either way, what you told initially stands right since dap.breakpoints module is not a complete one yet since the author wanted to add support for something called data breakpoints and will then probably work on providing a widget for toggling breakpoints. So since the api is not stable, there is probably no point in writing a picker as of now for it.


fix: connectionless OOB offers/requests and service parsing

Fixes #2907.
Modifications were made in an earlier PR to the handling of the service decorator which caused some issues in the OobProcessor when it was handling connectionless offers/requests. These issues should now be corrected.
It would also seem that connectionless interactions are not covered by our integration tests as a side note. I validated these fixes using this minimal reproducible example setup in my AME repo (committed version is testing against 0.11.0; I ran locally against an image built from main): https://github.com/Indicio-tech/acapy-minimal-example/pull/86
cc: @nodlesh @swcurran

I've got some test issues to sort out. Bad mocked values.

Superseded by #2910


"Connection aborted" but application does not terminate.

I receive a message "Connection aborted" but application does not terminate.
Why doesn't aborting the connection cause the program to terminate?
How to correctly reconnect or terminate the program with this error?

It should be trying to reconnect.  If you want it to do something different post a PR or fork the library.  I haven't used this or maintained it in years.
there are at least two other options to try if you don't like this library.
https://www.npmjs.com/package/@microsoft/signalr
https://www.npmjs.com/package/signalr


Revert "Roll engine 8fff8da38da2..d2ec21221e29 (8 commits)"

Reverts flutter/flutter#55522
Same reason as last revert, broke everything. IIRC @liyuqian revert the engine change that caused this, so we might just need to wait?
TBR: @flar @iskakaushik
To fix the broken devicelab tests.

Landing on red to fix build

Yup @liyuqian reverted the change. We need to wait for that to roll. Sorry about this.

No worries!


doc: document all functions in provider-base(7)

Fixes #13358

[x] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to master, thanks for the feedback.


Ch fixing api tests 141393735

Fix failing and incomplete API tests.

Coverage increased (+0.2%) to 79.886% when pulling 01f418d5cf07b854a555785a8ce83d56d0224d1f on ch-fixing-api-tests-141393735 into 03330632b8fbd4e718a9eabec6bc504d125c325d on develop.

Coverage increased (+0.2%) to 79.834% when pulling b508805b6a0820859da3d1bd7a6afbcc1ff8ae68 on ch-fixing-api-tests-141393735 into 03330632b8fbd4e718a9eabec6bc504d125c325d on develop.


Dispatching _FAILED actions from source promises

I see in the new docs it recommends the following pattern:
class UserAPI extends Marty.HttpStateSource {
  getUser(userId) {
    var url = 'http://jsonplaceholder.typicode.com/users/' + userId;

    return this.get(url).then(function (res) {
      if (res.ok) {
        return res;
      }

      throw res;
    });
  }
}

class UserQueries extends Marty.Queries {
  getUser(userId) {
    this.dispatch(UserConstants.RECEIVE_USER_STARTING, userId);

    return this.app.userAPI.getUser(userId)
      .then(res => this.dispatch(UserConstants.RECEIVE_USER, userId, res.body)_
      .catch(err => this.dispatch(UserConstants.RECEIVE_USER_FAILED, userId, err);
  }
}

However, in my testing, I noticed I was getting both UserConstants.RECEIVE_USER and UserConstants.RECEIVE_USER_FAILED dispatched in the failure case (possibly because of the throw not happening until the this.get(url).then() callback?).
I assume this is not the expected behavior, because I was under the impression that only one of the two actions (RECEIVE_USER or RECEIVE_USER_FAILED) should be dispatched.
I worked around it with the following function, but it seems a bit unnecessary:
export function wrapPromise(promise) {
  return new Promise((resolve, reject) => {
    promise.then(res => {
      throwIfError(res);
      return resolve(res);
    }).catch(err => {
      return reject(err);
    });
  });
}

Is it possible that the
this.dispatch(UserConstants.RECEIVE_USER, userId, res.body)

call is throwing? It seems like that could potentially lead to both actions getting dispatched.
I think in general it'd be safer to write those constructs as .catch(...).then(...).

Nevermind, this seems to have been an issue with a previous version of Marty (pre-0.10) and after further testing does not seem to repro.
Thanks all!


[Bug]: Call to a member function getKey() on string

What happened?
hello after i click on cancel i always get this error message "Call to a member function getKey() on string" any help how can i fix it
image : https://drive.google.com/file/d/1RLlBa3GeJaWzgtMHR2EzzeLgvD_5OPgp/view?usp=sharing
What did you expect to happen?
cancelling order
Version
3.5.x
What browser are you seeing the problem on?
No response
Relevant log output
No response

Fixed https://github.com/tastyigniter/ti-ext-cart/commit/832e73fbb9b6474a9cddacc63d1847c80d95a141


test: added e2e test that verifies if rebuild uses cached dependencies

Description
Added an e2e-test that verifies that a rebuild of the same component uses cached dependencies from jvm-build-service.
I also had to add a test that checks if all artifactbuild and dependencybuilds are complete and I added a ComponentInitialBuildAnnotationKey constant and edited files that use it.
Issue ticket number and link
https://issues.redhat.com/browse/STONEBLD-61
Type of change

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

How Has This Been Tested?
ginkgo -v --label-filter "jvm-build" ./cmd -- --config-suites="$PWD/tests/e2e-demos/config/default.yaml"

Checklist:

[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("[test_id:01] DEVHAS-62 devfile source")
[ ] I have updated labels (if needed)

/retest-required

/retest-required

@tnevrlka the test that the CI job is currently failing on won't pass. it's a known issue, currently being investigated

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/lgtm


Browse bug

Steps to reproduce

Go browse
Lastest manga
Open any manga chapter
Exit the manga and scroll down

Expected behavior
Should load the next page of the lastest manga
Actual behavior
Nothing happen
Crash logs
No response
Neko version
2.12.10
Android version
Android 9
Device
Oppo A12
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to the newest version Latest.
[X] I have filled out all of the requested information in this form.

Sorry for some reason i post the video here ,here link https://media.discordapp.net/attachments/569823838305648640/1092692299919675452/Record_2023-04-04-14-06-37_e93d65b045a4d38b75e3588ffeb6de8f.mp4

https://user-images.githubusercontent.com/79831076/229781790-4a6539ad-4c05-4892-815d-a3df1981e58a.mp4

Look like the bug happen to if you bookmark or change category in lastest manga

The bug happen even if you tap on any manga on latest upload and popular new title

This same issue exists while searching manga as well. I was able to reproduce it by these two methods

opening a chapter and scrolling down
searching for something that gives high result count and scrolling down.

The browse tab just hangs when it happens(not able to hide library entries or change view for results). I have to restart the app or go to another tab to fix it.
I'm using Android 13 on Pixel 7.


Design changes proposals

Some suggestions to improve Tagger interface.

[x] To achieve adaptiveness add bottom bar, move Web Services, Tag Actions and Apply buttons there, and also add a toggle button to control flap.
[x] Hide rigth pane for empty folders.
[x] Show StatusPage in right pane when nothing is selected, instead of showing empty tags entries.
[x] Move advanced search help to yelp docs.
[x] Avoid using EntryRow for read-only file properties: ActionRow with label in suffix can be used for duration and file size, and for fingerprint we could not show fingerprint but add a button to copy it to clipboard.
[x] Maybe move search entry to bottom bar? Or at least add separator between it and the files list.
[x] Improve album art, to show that the user can click to change it. Change cursor on hover, and add little edit icon in the corner of image.

https://github.com/NickvisionApps/Tagger/releases/tag/2023.7.1-beta2


Sparteo Bid Adapter: Add Sparteo documentation

🏷 Type of documentation

[x] new bid adapter

📋 Checklist

[X] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
[x] For new adapters check submitting your adapter docs

PR in Prebid.js : https://github.com/prebid/Prebid.js/pull/10582

please put quotes around the string params. backticks are just highlight markdown.

Hello @bretg,
I've made the changes.
Geoffray


Silent/Very Silent Install results in application opening after installation

SQL Operations Studio Version: 0.25.4

Steps to Reproduce:
Run the below command
sqlops-windows-setup-0.25.4.exe /VERYSILENT
After installation SQL OPs application opens on the users desktop automatically.
Looking in the log file you see the following final entry
2018-01-25 16:06:58.659   Run as: Original user
2018-01-25 16:06:58.659   Type: Exec
2018-01-25 16:06:58.659   Filename: C:\Program Files\SQL Operations Studio\sqlops.exe
Having looked at the Win32 installer at:
https://github.com/Microsoft/sqlopsstudio/blob/master/build/win32/code.iss
I believe adding the "skipifsilent" flag to the line:
[Run]
Filename: "{app}\{#ExeBasename}.exe"; Description: "{cm:LaunchProgram,{#NameLong}}"; Tasks: runcode; Flags: nowait postinstall; Check: WizardSilent

would resolve this issue and give a true silent install. Documentation entry for SkipIfSilent from http://www.jrsoftware.org/ishelp/index.php?topic=runsection
skipifsilent
Valid only in a [Run] section. Instructs Setup to skip this entry if Setup is running (very) silent.

@corivera I've reactivated this issue because I think we reverted the change due to auto-update bug.  Please close again if I'm incorrectly reactivating.

I can confirm on 0.30.4 with /silent

@Matteo-T I think this is related to the issue you saw with SQL Ops auto-starting after silent install.  There's a conflict with our auto-update mechanism and this switch that we'll need to resolve.

THanks @kburtram - I totally forgot about this one. Sorry for the dupe. :-(

Have you tried passing the response file along with the Silent/Very Silent switch?

As mentioned in https://github.com/microsoft/azuredatastudio/issues/11652 /MERGETASKS=!runcode option. Let us know if that doesn't work.

Closing out this older issue with the workaround suggested in #11652.  Please let us know if you're still impacted by this bug.


Some albums not appear in iOS version

Hello, first congrats for the library. Currently we are experimenting an issue on iOS version where some albums are not returned by the PhotoGallery.listAlbums.
This is our way to reproduce:

Pick a photo from your camera image_picker
Save in an album with gallery_saver GallerySaver.saveImage(url, albumName:"Album test"), if the album doesn't exists it will be created and visible from the native gallery of the phone.
With photo_gallery, the albums created by the GallerySaver package on android are readable and on iOS are not returned by PhotoGallery.listAlbums

Let me know if we can help you in some way

Have you found any solution to fetch all the albums including the missing ones?

@seeebbii Have you found any solution to this? I have been stuck on this for days

@seeebbii Have you found any solution to this? I have been stuck on this for days

unfortunately, nope.


404 Page issue

🐛 Bug Report
There is an issue with 404 pages with the deployed app on Vercel. xyz.com/404 is working perfectly and static translations are working well. Unfortunately, when you're trying to access a "real" 404 page like /asdasd translations are visible only for a fraction of a second and then fallback keys are shown.
To Reproduce
Create a Next.js app, and then create a custom 404.tsx page with the getStaticProps translations fetcher. Then try to access a non-existing page.
export async function getStaticProps (pageContext: NextPageContext) {
	const locale = pageContext.locale
	return {
		props: {
			...(await serverSideTranslations(
				locale,
				config.languageNamespaces,
				nextI18NextConfig
			))
		}
	}
}

Expected behavior
Not disappearing translations on 404 pages.
Your Environment

runtime version: i.e. node v16
i18next version: i.e. 22.4.9
deploy: vercel

If it works locally, please ask in the vercel/next.js repo.


Arthur/clojure ndarray api random merged

Reopened from PR #14750
Description
Split random functions in their own generated namespace.

ndarray_random_api
symbol_random_api

It renames the following functions to be as close as possible to the scala names:

-random-randint into randint
sample-randint into randint-likefor instance and put them under ndarray-random-api namespace.

TODO

Unify random and sample functions (mu vs loc and sigma)

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

cc @kedarbellare

Overall looks fantastic! What a great addition :)
Just a couple questions on the inclusion of some test files

sorry merged by mistake

@Chouffe - Actually instead of reverting it, I'll just open a new PR with those two file removed, Since everything else is good


Cursor is jumping when delete symbols from the middle of value

Problem:
When use value property, that goes from state variable, and delete characters from the middle of value, cursor is jumping to last character.
Please, take a look at the attached gif:

OS: Android
Version: 2.0.1
How to reproduce:
Example of code:
const App = () => {
  const [value, setValue] = useState('');

  const onChangeText = useCallback(
    (formatted: string) => setValue(formatted),
    [],
  );

  return (
    <SafeAreaView>
      <ScrollView>
        <View>
          <TextInputMask
            value={value}
            onChangeText={onChangeText}
            mask={'+1 ([000]) [000] [00] [00]'}
            style={styles.input}
          />
        </View>
      </ScrollView>
    </SafeAreaView>
  );
};

Repo to reproduce: https://github.com/abramovslava/jumping-cursor-react-native-text-input-mask

This is probably due to autocomplete behavior, try switching it off and see if solves your problem:
<TextInputMask autocomplete={false}, autoskip={false} />

This is probably due to autocomplete behavior, try switching it off and see if solves your problem:
<TextInputMask autocomplete={false}, autoskip={false} />

I've done these, but the problem still persists.

Sorry for late answer...
@martintreurnicht i can confirm that even with autocomplete and autoskip properties issue still persists. I`ve updated repo with these properties.

@abramovslava Ok i see, you're using version 2. That version is no longer supported, it was created by the previous maintainer, please upgrade to the latest version. I can confirm that it no longer happens on the latest

This still happens on v3.1.4

still jumping on v3.1.4

any update for this issue ?
version 3.1.4 on android, this issue still jumping, but on 3.1.2 this issue was resolved
but, on ios get the other error

Same issue on 3.1.4


Drop wpt fork

https://github.com/web-platform-tests/wpt/pull/43824 was merged and somehow we don't crash the runner for other 404s.
I have listed three remaining files to ignore in the wpt-harness.js now. I will add those as todo to #57
  RESULTS:
  - # pass: 3856 (-2)
  - # fail: 1226 (+10)
  - # type error issues: 63 (+1)
  - # files ignored: 3 (-5)

The drop in # pass is due to wpt/webaudio/the-audio-api/the-waveshapernode-interface/waveshaper.html not running, it also 404s on the buffer loader. I will try to get that one, and the other 404s fixed in the upstream repo as well. This can be merged in the meantime

Cool, thanks!

somehow we don't crash the runner for other 404s.

Actually I installed in the XHR mock in some previous commit, maybe it solved some issues
https://github.com/ircam-ismm/node-web-audio-api/blob/3837a264fc2764ce780700b4a92721947a2d1413/.scripts/wpt-harness.mjs#L50


Require cffi >= 1.4.1

Previous versions of cffi didn't know about common
types or any of the Windows types when using an out-of-line
module.  Version 1.4.0 if cffi should fix contain a fix however:
https://bitbucket.org/cffi/cffi/issues/228/bool-type-as-callback-argument-not-working
Rather than requiring 1.4.0+ we're using 1.4.1+ because of another issue:
https://cffi.readthedocs.org/en/latest/whatsnew.html#v1-4-1

Current coverage is 94.74%

Merging #45 into master will decrease coverage by -1.43% as of d000344

Powered by Codecov. Updated on successful CI builds.


Relax checking of test case

Similar to https://github.com/rubocop-hq/rubocop-minitest/pull/99, I'm suggesting to identify test case as a method call, whose name starts with assert_ or refute_. This will automatically work with custom assertions and rails' specific assertions.
https://github.com/rubocop-hq/rubocop-minitest/blob/85abdbea1853df82928daa90c29bb6203f418919/lib/rubocop/cop/mixin/minitest_exploration_helpers.rb#L70-L72
If sounds good, will submit a pr.

Yeah, I think it's worth a try. Can you open a PR?


Azure SQL DB

Cannot find how to send you a simple email. Usually that is what Contact Us provides. I want info on obtaining an Azure SQL DB. How much will it cost? We already have Azure Blob Storage account. Please respond -<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cc1b9f94-0697-7726-d678-1439a30005a0
Version Independent ID: 13c3c83f-6033-d428-2b81-d2558cc71a13
Content: Monitor and track Azure free service usage
Content Source: articles/cost-management-billing/manage/check-free-service-usage.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @amberbhargava
Microsoft Alias: banders

@stevio-stack - Thanks for bringing this to our attention. We will investigate it further and update you shortly.

@stevio-stack Hi there. Check out https://docs.microsoft.com/azure/azure-sql/database/cost-management for general information. For estimated costs, see https://azure.microsoft.com/pricing/calculator > Databases > Azure SQL Database and then select details about your needs. You'll get a monthtly cost estimate there.
#please-close


⚠ otzivi-tut has degraded performance

In c34910a, otzivi-tut (https://отзывы-тут.рф/) experienced degraded performance:

HTTP code: 200
Response time: 17541 ms

Resolved: otzivi-tut performance has improved in c690b6d.


Receiver not getting notification about new friend requests and messages

Hi Markpairdha,
Thanks for Sharing this code! And Congrats on amazing work you have done for learners like me.
I'm undergrad final year student, during this quarantine trying to build a simple chat app, and I've achieved almost everything, I wanted to achieve for my small project for learning purpose
Here is link of Question, I asked on SO: https://stackoverflow.com/questions/62363813/chat-app-receiver-is-not-getting-notification-about-friend-requests-and-message
As you can see in above posted question, I clearly mentioned:-
I'm able to send Friend Request and Messages. Receiver also able to receive Friend Request and Messages. But receiver is not getting Notification about Friend Requests and new Messages.
I believe you can help me, hence I decided to write you. Thanks - Ridihika

I would like to work on this issue @markpairdha

hi, I have the same issue. plz help

Plz help asap!!!!

The same issue


Translocations not called?

Hello,
After running breakmer with a sample with known translocations, I have and output folder with a subfolder for each exon, which contains sorted reads in bam files, (tree below):

├── TRAF3_EXON_12
│   ├── TRAF3_EXON_12_contig89_reads.bam
│   ├── TRAF3_EXON_12_contig89_reads.sorted.bam
│   ├── TRAF3_EXON_12_contig89_reads.sorted.bam.bai
│   ├── TRAF3_EXON_12_contig89_svs.out
│   ├── TRAF3_EXON_12_discreads.out
│   └── TRAF3_EXON_12_svs.out
├── TRAF3_EXON_3
│   └── TRAF3_EXON_3_discreads.out
├── TRAF3_EXON_6
│   └── TRAF3_EXON_6_discreads.out

Each of the _svs.out files gives the coordinates of indels which look like they can be found in the bam files. It seems that the _discreads.out files contain the structural variants (large inserts and translocations) that I am looking for but there is no sorted bam file to go with these.
The documentation seems to indicate that there should be a bam file output as well as a folder for each variant - is this correct? If so, what is preventing these from being generated?
I have no error messages in my log -- it seems to complete properly (below):

breakmer.assembly.assembler - INFO - Initiating kmer xxxxxxxxxxxx, found in 16 reads
breakmer.assembly.contig - DEBUG - 0 kmers left to check
breakmer.assembly.contig - INFO - Contig done with contig seq xxxxxxx Supported by 2 read(s).
breakmer.assembly.contig - INFO - Read IDs: xxxxxxxxxxxxxxxx
breakmer.assembly.assembler - INFO - Contig did not meet the read count threshold 2, with 2 or contig length (151) < readLen (151)
breakmer.processor.target - INFO - Resolving structural variants from 0 kmer clusters
breakmer.processor.target - INFO - Writing TRAF3_EXON_9 discordant read cluster result file /mnt/work1/users/xxxxxxxxxxxxxxxxxxxx
breakmer.processor.analysis - INFO - Writing "Breakmer.JJN3.14Jun16_onlyMAX.NF.TRAF" aggregated results files: all result - xxxxxxxxxxx"_svs.all.out and filtered results - /mnt/work1/users/xxxxxxxxxxxxxxxxxxxxxx_svs.out
breakmer.processor.analysis - INFO - Writing "xxxxxxxxxxxxx" aggregated results file /mnt/work1/users/xxxxxxxxxxxxxxx_discreads.out
breakmer.processor.analysis - INFO - Analysis complete in 2489.59

Thanks for any insights!

Hi, Matt,
Thank you for your quick response. I have a couple of additional questions.
I downloaded again the 0.0.6 version, which has the
sv_assembly_mm2.py
sv_assembly_py
sv_caller.py
sv_processor.py
utils.py
breakmer.py
all in the root directory, with a drawimage folder with a draw_trl_plot.R
file.
The documentation for this here: http://ryanabo.github.io/BreaKmer/
does not seem to apply in the same way.
When I put in the command
python breakmer.py prepare_reference_data -c
/path/to/config/file/kmer_region.config
I get the following error
Usage: breakmer.py [options]
breakmer.py: error: option -c: invalid integer value:
when I look at python breakmer.py --help
informs me that the -c option is to specify translocation threshold, not to
take a config file:
-c TRL_SR_THRESH, --trl_sr_thresh=TRL_SR_THRESH
                    Split read support threshold for translocations

                    [default: 2]

I attempted to drop the "-c", to put the relative path instead of the
absolute path, but this does not seem to work. (and I get different errors)
Thanks for your help,
Signy
On Wed, Jun 15, 2016 at 7:27 AM, Matt Ducar<EMAIL_ADDRESS>wrote:

Hi,
It looks like you are using the unstable code in the master branch or one
of the prototype releases. We've identified several issues with this code
and recommend everyone use the release 0.0.6 version of BreaKmer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ccgd-profile/BreaKmer/issues/29#issuecomment-226160357,
or mute the thread
https://github.com/notifications/unsubscribe/AR0PsMkEyg5AY2sxZDuiPQWuA50HSnVsks5qL-GOgaJpZM4I11Jf
.

I also tried using the breakmer.cfg ( prepare_reference_data --c
/path/to/config/breakmer.cfg) from the B2M example data set from the
previous version, which it will still not take
Also, do you have an example data file I can try this on?
Thank you!
Signy
Thanks!
On Wed, Jun 15, 2016 at 4:56 PM, Signy Chow<EMAIL_ADDRESS>wrote:

Hi, Matt,
Thank you for your quick response. I have a couple of additional
questions. I downloaded again the 0.0.6 version, which has the
sv_assembly_mm2.py
sv_assembly_py
sv_caller.py
sv_processor.py
utils.py
breakmer.py
all in the root directory, with a drawimage folder with a draw_trl_plot.R
file.
The documentation for this here: http://ryanabo.github.io/BreaKmer/
does not seem to apply in the same way.
When I put in the command
python breakmer.py prepare_reference_data -c
/path/to/config/file/kmer_region.config
I get the following error
Usage: breakmer.py [options]
breakmer.py: error: option -c: invalid integer value:
when I look at python breakmer.py --help
informs me that the -c option is to specify translocation threshold, not
to take a config file:
-c TRL_SR_THRESH, --trl_sr_thresh=TRL_SR_THRESH
                    Split read support threshold for translocations

                    [default: 2]

I attempted to drop the "-c", to put the relative path instead of the
absolute path, but this does not seem to work. (and I get different errors)
Thanks for your help,
Signy
On Wed, Jun 15, 2016 at 7:27 AM, Matt Ducar<EMAIL_ADDRESS>wrote:

Hi,
It looks like you are using the unstable code in the master branch or one
of the prototype releases. We've identified several issues with this code
and recommend everyone use the release 0.0.6 version of BreaKmer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ccgd-profile/BreaKmer/issues/29#issuecomment-226160357,
or mute the thread
https://github.com/notifications/unsubscribe/AR0PsMkEyg5AY2sxZDuiPQWuA50HSnVsks5qL-GOgaJpZM4I11Jf
.

Hi, again, Matt,
Are you able to clarify for me which version of breakmer I should be using?
Again, the 0.0.6b version, which has the
sv_assembly_mm2.py
sv_assembly_py
sv_caller.py
sv_processor.py
utils.py
breakmer.py
all in the root directory, with a drawimage folder with a draw_trl_plot.R
file. and runs everything at once
whereas the 0.0.6-ccgd version has these modularlized in separate folders
with the need to put in the prepare_reference_data, the start_blat_server
and the run commands separately.
I cannot get either of these versions to output a bamfile for the
translocation events for me.
Thanks,
Signy
On Wed, Jun 15, 2016 at 4:56 PM, Signy Chow<EMAIL_ADDRESS>wrote:

Hi, Matt,
Thank you for your quick response. I have a couple of additional
questions. I downloaded again the 0.0.6 version, which has the
sv_assembly_mm2.py
sv_assembly_py
sv_caller.py
sv_processor.py
utils.py
breakmer.py
all in the root directory, with a drawimage folder with a draw_trl_plot.R
file.
The documentation for this here: http://ryanabo.github.io/BreaKmer/
does not seem to apply in the same way.
When I put in the command
python breakmer.py prepare_reference_data -c
/path/to/config/file/kmer_region.config
I get the following error
Usage: breakmer.py [options]
breakmer.py: error: option -c: invalid integer value:
when I look at python breakmer.py --help
informs me that the -c option is to specify translocation threshold, not
to take a config file:
-c TRL_SR_THRESH, --trl_sr_thresh=TRL_SR_THRESH
                    Split read support threshold for translocations

                    [default: 2]

I attempted to drop the "-c", to put the relative path instead of the
absolute path, but this does not seem to work. (and I get different errors)
Thanks for your help,
Signy
On Wed, Jun 15, 2016 at 7:27 AM, Matt Ducar<EMAIL_ADDRESS>wrote:

Hi,
It looks like you are using the unstable code in the master branch or one
of the prototype releases. We've identified several issues with this code
and recommend everyone use the release 0.0.6 version of BreaKmer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ccgd-profile/BreaKmer/issues/29#issuecomment-226160357,
or mute the thread
https://github.com/notifications/unsubscribe/AR0PsMkEyg5AY2sxZDuiPQWuA50HSnVsks5qL-GOgaJpZM4I11Jf
.

Hi Signy,
The version of Breakmer you should use is under the "Releases" section.  You should use the latest release v0.0.6 called "CCGD_Profile_Production".  It sounds like you are cloning the repo and trying to run from there.
Thanks for pointing out the issue with the documentation.  The description at http://ryanabo.github.io/BreaKmer/ wasn't set up by our team and doesn't apply to the latest release.  We need to improve the documentation, but won't have the opportunity for a while.
-Matt


Make sure the project is packaged properly and can be installed

The command pip install git+PACKAGE_URL.git should work for another user.

Yes , checked working like a charm.


minor updates to README.md; removed dead link to credits and an unnec…

…essary period
Signed-off-by: Mark Cohen<EMAIL_ADDRESS>Description
Makes the README a little nicer and triggers pr_stats against main.
Issues Resolved
N/A
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #65 (86a60d4) into main (579f205) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #65   +/-   ##
=========================================
  Coverage     65.52%   65.52%
  Complexity      172      172
=========================================
  Files            29       29
  Lines           879      879
  Branches        119      119
=========================================
  Hits            576      576
  Misses          254      254
  Partials         49       49

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


feat(docker): healthcheck

contains docker healthchecking as mentioned in #103

@milesmcc is anything still missing?

Pretty much looks good! Just to confirm, what's the difference between the two o/O flags (in -o /dev/null -O /dev/null)?

-O,  --output-document=FILE      write documents to FILE
[..]
-o,  --output-file=FILE          log messages to FILE

Which means -O is for output file downloaded using wget and -o is for wget errors/logged messages.

Cool, thanks

Hi,
It's not good, my container doesn't start because healthcheck is ko:
[root] ~ # docker exec -ti shynet bash -c 'wget -o /dev/null -O /dev/null --header "Host: ${ALLOWED_HOSTS%,*}" "http://<IP_ADDRESS>:$PORT/healthz/?format=json"'; echo $?
1

With verbosity:
[root] ~ # docker exec -ti shynet bash -c 'wget --header "Host: ${ALLOWED_HOSTS%,*}" "http://<IP_ADDRESS>:$PORT/healthz/?format=json"'; echo $?
Connecting to <IP_ADDRESS>:8080 (<IP_ADDRESS>:8080)
wget: server returned error: HTTP/1.1 400 Bad Request
1

Why the host header is important here?

and ALLOWED_HOSTS is being cut on first colon
${ALLOWED_HOSTS%,*} on ALLOWED_HOSTS=domain1.com,domain2.com,domain3.com should evaluate to domain1.com
Without setting HOST header it wasn't working at all.
I've two domains in allowed hosts and it works for me.
Could you modify healthcheck in docker-compose to show everything more verbose?
EDIT: Yes. It's my fault. It actually evaluates to this:
root@big-boy:~# ALLOWED_HOSTS=domain1.com,domain2.com,domain3.com
root@big-boy:~# echo ${ALLOWED_HOSTS%,*}
domain1.com,domain2.com

I've not checked more than 2 domains. Quick fix will be send as MR in a few minutes. Sorry ;)

Should be fixed now ;)
Could you please check my branch?


Introduce SourceBuilder utility

This PR continues my work to simplify the way we create/use utilities for generating source code that I started when I introduced the Dependencies struct. The SourceBuilder struct is simply a collection of references to a Source, Interface, and Dependencies so that its methods can encapsulate all of the logic required for source generating, like marking requirements for intrinsics and looking up type info.
As a simple group of references, SourceBuilders can be created and destroyed easily whenever required and so don't tie things up in borrows any longer than needed.
This reduces our reliance on mem::take and mem::replace which we were using to swap Sources into and out of a primary slot that all of our utilities assume the current source is in. In the future, I expect this will allow us to manage Sources more cleanly/separately and stop relying on the mem utilities.
Note: this SourceBuilder utility is Python-specific, but the pattern is not. So while it shouldn't move to gen-core it may make sense for other generators to adopt this pattern.

I love it! It is much nicer to read and keep all the benefits.  I do wonder how this could be made to be more generalizable, but that is the same issue for all generators.

Seems reasonable to me, thanks for this!


add missing virtual destructor to base class IAllocator

add missing virtual destructor to base class IAllocator and also add destructor to CGBMAllocator and free gbm_device

you added a commit from another mr #2 here

you added a commit from another mr #2 here

opsie, fixed.


Gracefully shutdown when process is killed

Describe the feature
Currently any signal sent to the node just kills the node with no clean up.
If you CTRL-C a running instance, make a few changes, and start the node again, you will most likely end up trying to peer with nodes you've peered with before, but they have not yet evicted you from their peerset. This leads to you connecting to nodes that think you are already connected, the connection will fail.
The CLI should handle appropriate signals using the ctrlc crate. The following should happen (ideally):

Disconnect from all peers and stop connecting to new ones
Try to wait for the current executing stage to finish

As a note here, the header and bodies stage will be designed to hang until they get all the data they want. For this reason, we should provide a reasonable timeout before we simply discard what the stage has been doing.

Additional context
No response

supersede by #1315


Implicit resolution is poor when using erased implicits

It's desirable to be able to use indexed types, and have the indexes be erased when extracting. For example, vectors can be defined as
module Vector

open FStar.Ghost

type vector (a:Type) : nat -> Type =
    | Nil : vector a 0
    | Cons: #l:erased nat
            -> hd:a
            -> tl:vector a (reveal l)
            -> vector a (reveal l + 1)

Inference fails when trying to use Cons, however:
let v1 : vector a 1 = Cons () Nil

(Error 189) Expected expression of type "Vector.vector (*?u5*) _ (FStar.Ghost.reveal (*?u6*) _)"; got expression "Vector.Nil" of type "Vector.vector (*?u8*) _ 0"
1 error was reported (see above)

The following succeeds, but fails if the _ is changed to Nil:
val fold(#a #state:Type)(#l:erased nat):
    (state -> a -> state)
    -> state
    -> vector a (reveal l)
    -> Tot state (decreases (reveal l))
let rec fold #_ #_ #_ f s = function
    | Cons hd tl ->
        fold f (f s hd) tl
    | _ -> s

In order to have this sort of thing work correctly, the options I can see are to either automatically attempt to erase implicit, pure terms in the same way that implicit types are erased (so that the user doesn't need to use erased in code similar to the example given), or to make F*'s implicit resolution strategy controllable through tactics.
Both would be somewhat desirable - it's very easy to generate inefficient extracted code accidentally if implicit type indexes are not erased, and having control over F*'s implicit resolution strategy would give Coq-style type classes "for free".
If anyone else faces this issue, a partial workaround is to use an erased type as the index, and then define another type which erases the index, like so
type vector' (a:Type) : erased nat -> Type =
    | Nil : vector' a (hide 0)
    | Cons: #l:erased nat
            -> hd:a
            -> tl:vector' a l
            -> vector' a (hide (reveal l + 1))

type vector (a:Type) (n:nat) = vector' a (hide n)

let v1 : vector unit 1 = Cons () Nil

val fold(#a #state:Type)(#l:erased nat):
    (state -> a -> state)
    -> state
    -> vector' a l
    -> Tot state (decreases (reveal l))
let rec fold #_ #_ #_ f s = function
    | Cons hd tl ->
        fold f (f s hd) tl
    | Nil -> s

In order to have this sort of thing work correctly, the options I can see are to either automatically attempt
to erase implicit, pure terms in the same way that implicit types are erased (so that the user doesn't need
to use erased in code similar to the example given)

Implicits and erasure are orthogonal issues. Nothing prevents your code from being computationally dependent on an implicit argument. So, erasing it is not viable in general.
As for letting tactics control the way type inference works: it sounds like an interesting direction, but also quite challenging. At least in the current way things work in F*, I don't see a natural way to have tactics interpose on the strategy for inferring implicits. I've added it to a new issue on Meta-F* wish list: #1419
Finally, I like your vector' solution. Would you like to elevate the example to a wiki page describing a style for erased indices? If so, then I would suggest closing this issue.

Nothing prevents your code from being computationally dependent on an implicit argument. So, erasing it is not viable in general.

That's not necessarily the case. It's not generally possible (or desirable) to erase implicits, however we can attempt to erase implicits (of course, we could attempt to erase explicit arguments too). Where a function parameter is not used in the body of a function, it can be erased such that the type becomes generic (or Obj.t), and the parameter is replaced with a unit in extracted code, when instantiating implicits in order to invoke said function. As far as I understand this kind of erasure can be performed freely in pure code, and since it would improve performance of extracted code and remove the need to handle erasure explicitly in many cases, I think it's worth considering.

Would you like to elevate the example to a wiki page describing a style for erased indices?

A wiki page for erased indices is a good idea. The workaround I provided is fairly fragile and I would not advocate it as a general solution. In particular, the workaround fails if vector' a l is replaced with the equivalent vector a (reveal l) in the val declaration for fold. I'm not so sure why this is - adding an unfold qualifier to vector or triggering SMT patterns for vector a (reveal l) == vector' a l seems to have no effect. Do you have any suggestions regarding improving this workaround?

The style is a bit heavy, in the following Cons branch works but Nil gives a typing failure: Expected vector a (reveal l) but got vector' a (hide 0). My guess is we are not using SMT when checking the well-typedness of patterns, hence this fails, will need to see more.
let rec fold #a #state #l f s = function
    | Cons hd tl ->
        let l: erased pos = hide (reveal l) in
        fold #_ #_ #(elift1 #pos #nat (fun n -> n - 1) l) f (f s hd) tl
    | Nil #a  -> s

My guess is this is the relevant code: https://github.com/FStarLang/FStar/blob/master/src/typechecker/FStar.TypeChecker.TcTerm.fs#L1595.
@nikswamy, is it unsound to use smt (and force the guard) there instead of using teq_nosmt?

Regarding controlling implicits through tactics, it's certainly feasible to do something in the same vein as Coq, Agda and Lean, which have a special kind of implicit which is to be solved by typeclass resolution, instead of unification. Having a single kind of implicit would be very weird in my eyes, and I think it would bring some pain. (For instance, would the tactics run after unification fails? I think that's only detectable at the top-level.)
Such special implicits could look something like val f : ##(tau : t1) -> t2, where if we need to instantiate f with an implicit, we call tau (in the proper context) to provide a witness of t1. Having typeclasses is then just adding some syntax to create record types and their members, and some default resolution tactic that searches the context for things marked as instances. Of course these arguments could also be given explicitly.
I'm unsure on whether allowing to annotate with whatever tactic can be accomplished (how would unification work?). But adding a "typeclass" kind, as other languages do, should be doable.
Also, note that doing typeclasses in this way hinders canonicity, which is the property that there is a single instance for a given class+type. Not having it means one can potentially have two (ways of building) instances for Ord t, which are reversed, and some program points use one and some the other. While this would be quite bad in languages like Haskell, it's usually not so bad in dependently-typed ones.

Related to #1844

The original examples work as is now, my guess is because of automatic hide/reveal coercions. Adding a testcase for it and closing.


sane service: mention the lp group for printer+scanners

Motivation for this change
If a scanner is also a printer, it may be in the "lp" group and require that group to be used.
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks!


What's happened to Youtube repo?

Hi Michael,  after a 4 year absence from teaching Java I went to revisit your excellent videos to find that your youtube account doesn't exist? What's happened to your videos. Are they available somewhere else or ???
regards
Gordon

Happy days, looks like the channel is back :)


Add Zipkin tracing tags for EVCache calls

Context
We want to add observability to EVCache client calls by creating Zipkin spans and adding relevant tracing tags.
Description
This PR adds a evcache-zipkin-tracing subproject to the EVCache repo that contains a EVCache call listener which creates Zipkin spans using the Brave client.

@smadappa Let me know if the changes look good.

Looks good to me. Just added one comment. Another thing possibly is to squash and merge so that it is easy to review what went in to one commit.


Support 0->1 depth range in GLSL

In order to support the reverse-depth-buffer optimization, I would like to support a 0->1 depth range, even in GLSL. Given that the glClipControl extension isn't universally supported, and given that folks will still likely use the default depth range in OpenGL, we should support both depth ranges somehow. Also, OpenGL ES and WebGL do not support glClipControl at all. Perhaps the best way to accomplish this is conditional compilation -- if DEPTH_RANGE_ZERO_TO_ONE is defined, we will not transform the depth range. Otherwise we will, as we do today.
We should think about how such a preprocessor define could be utilized in other intrinsic functions that could allow you to write code that is portable to both depth range schemes.
@tgjones Any thoughts here?

With DEPTH_RANGE_ZERO_TO_ONE, did you mean that would be a C# #define?
A more discoverable API might be to have it as a property on the [VertexShader] attribute? It could be a DepthRangeTransform enum, defaulting to the current behaviour.
I'm trying to think of other intrinsic functions that could make use of this and coming up blank. That's probably just because it's late :) It does seem like a useful piece of metadata to have.

With DEPTH_RANGE_ZERO_TO_ONE, did you mean that would be a C# #define?

In my mind, it would be a preprocessor condition that is used by not defined in the shader code, e.g.:
gl_Position = output_.Position;
#if !DEPTH_RANGE_ZERO_TO_ONE
gl_Position.z = gl_Position.z * 2.0 - gl_Position.w;
#endif

By default, nothing would be change, but you could do something like this on the C# side:
string defines = string.Empty;
if (GraphicsDevice.IsDepthRangeZeroToOne)
{
    defines = "DEPTH_RANGE_ZERO_TO_ONE";
}

byte[] shaderBytes = LoadShader("myShader.vert", defines); // Add defines into the text and return bytes
``

> A more discoverable API might be to have it as a property on the [VertexShader] attribute? It could be a DepthRangeTransform enum, defaulting to the current behaviour.

I'm trying to think of a way that you can write your shader in a "portable" way, such that it will work under both depth range schemes. If we had a more robust "configuration system", perhaps you could just tell ShaderGen to spit out both versions of the shader -- but that's much more complicated.

> I'm trying to think of other intrinsic functions that could make use of this and coming up blank.

I think there might need to be some kind of builtin function that does depth comparisons for you, based on whether or not that define is set.


Add warning for running the binary on MacOS

I think using images is better but since there are no images in the assets currently, I thought it was better not to. Or should I use the github URL to put the images? though I think this functionality will be blocked in the future or maybe I'm just making that up 🧐

@mauromorales thanks! I think that's ok for now, we can add images later on, I usually use github images to not clutter the git content. maybe we can add a <details> section so to expand on how to configure the binary for mac?


Add-SPDistributedCache  has identical parameters for SP SE, SP 2019, SP 2016, but the article is scoped only for SPSE,

Add-SPDistributedCache  has identical parameters for SP SE, SP 2019, SP 2016, but the article is scoped only for SPSE, please add SP 2019 and SP 2016 as additional product versions for this article.

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

ID: 1cfdc3ce-d41e-7110-94f0-158493149ab9
Version Independent ID: 778d4ad6-de3a-4e9a-3b7d-339a9181b809
Content: Add-SPDistributedCacheServiceInstance (sharepoint-server)
Content Source: sharepoint/sharepoint-server-ps/sharepoint-server/Add-SPDistributedCacheServiceInstance.md
Product: sharepoint-server-itpro
GitHub Login: @Techwriter40
Microsoft Alias: pamgreen

@WayneEwington this actually looks to be a systemic problem -- I checked a handful of cmdlets and they've all been changed to SPSE only, even though they apply to 2010+.

I'm not the owner of the on-premises PowerShell cmdlets. Adding @TroyStarr to investigate

Hey folks, this is by design.  We've split the SharePoint Server PowerShell cmdlet documentation into two separate repositories.  One repository is for SharePoint Server Subscription Edition (which we expect to evolve over time with additional cmdlets and parameters) and the other is for SharePoint 2010 - 2019.  You can switch between them using the Version selector on the left side of the articles.  That said, I think we can make that clearer in the Version selector options so I'll follow up with the team to ask them to do that.

@TroyStarr could we rephrase "Latest" to 2010-2019 or something akin to that? This is confusing.

Yep, agreed. :-)

Yep, agreed. :-)

Hello @TroyStarr Troy, are you able to make @tseward 's suggested change? Thanks for clarifying

Hi @get-itips, we're discussing our options with our PowerShell repo contacts. There are some complications because the SharePoint Server (2010 - 2019), SharePoint Online, and SharePoint PnP PowerShell articles are currently grouped together as the single "Latest" choice in the version selector. No ETA yet on when we'll have a solution for moving forward.

Hi @get-itips, we're discussing our options with our PowerShell repo contacts. There are some complications because the SharePoint Server (2010 - 2019), SharePoint Online, and SharePoint PnP PowerShell articles are currently grouped together as the single "Latest" choice in the version selector. No ETA yet on when we'll have a solution for moving forward.

Thanks for the detailed update ;)

Because the appropriate team members are aware of this issue and are working on a solution, we're closing this issue.
Thank you, all!


Demonstrate how to make changes to taxonomy in runtime

Demonstrate how Fybrik taxonomy that reside in /tmp/taxonomy/taxonomy.json to be updated during run-time.
Tasks

[ ] Write an overview describing the role of the taxonomy in the fybrik architecture and the default taxonomy provided in https://fybrik.io/dev/concepts/taxonomy
[ ] Describe which actors/systems should update the different parts of the taxonomy in https://fybrik.io/dev/concepts/taxonomy
[ ] Assemble a sample git repo that will contain a sample taxonomy to be used in the documentation.
[ ] Expand custom taxonomy document with how to compile and deploy the new taxonomy for a running deployment.(and change the title of the page to taxonomy)
[ ] Document how to add a new element to the taxonomy. For example, new interface, new policy action, etc.

@ronenkat Different taxonomies have different owners as is described in https://docs.google.com/document/d/1sfdcNI6PTIYWsSIznpNqJNQCdVaVFchYB2vljzUrlew/edit?usp=sharing.  Thus, the mechanisms for update and the actors who do those updates would be different.
Some taxonomies would need to be updated via a connector. Example: If there is a new purpose used in policies, I would expect that the policy manager connector would have a mechanism to update purpose in the taxonomy.  If the geographies are managed by the data catalog, then it's connector would update them in the taxonomy.    Our connectors need to provide mechanisms for doing this.  These I believe have been yet to be defined/designed, and thus I suggest adding the relevant tasks.
On the other hand, modules are something managed by Fybrik.  It's the IT admin who deploys them, and who determines the capabilities available to the control plane for creating data planes.  Thus, a mechanism via which the admin can add these capabilities needs to be provided.  This mechanism also needs to be defined.
Technically, I would suggest the initial focus be on the modules.  I would specifically add tasks for new modules that provides (a) a new capability (b) new format and/or protocol (c) new transform action.

@Mohammad-nassar10 Some ideas for a taxonomy example page.
Example 1: Add new intent
Using the quick start demo...
0. Submit a fybrikapplication.yaml that has intent=marketing.  This should fail validation and no FybrikApplication CRD will be created, because no marketing in taxonomy

Governance Officer writes a new policy (in OPA) - If intent is marketing and data is tagged as PII, then redact
Governance Officer adds a new intent value = marketing in application taxonomy
Redeploy fybrik helm charts with taxonomy
Submit a fybrikapplication.yaml that has intent=marketing.  Now it should pass validation and only the data that isn't personal should be returned

Example 2: Add new transform

Write a new redact capability in the arrow flight server and call it redactY - just copy and paste the current redact and change the redact value from XXXXX to YYYYY
Add redactyAction to the actions in the arrow flight module yaml
Write an OPA policy that says - If read and data is PII then redactyAction - i.e. new type of redact
Redeploy the arrow flight module yaml
Redeploy fybrik helm charts with taxonomy
Run the quickstart notebook scenario.  Redacted data should show YYYYY

@roee88 @Mohammad-nassar10 Do these example scenarios make sense to you?

Keep in mind that the default allows all values. You probably want to start with "deploy with example taxonomy" (we have one in samples) and then from there move to your scenarios.

As we discussed @Mohammad-nassar10, let's simplify the examples to focus only on the taxonomy specific tasks.

Submit a fybrikapplication.yaml that has intent=marketing. This should fail validation and no FybrikApplication CRD will be created, because no marketing in taxonomy
Add a new intent value = marketing in application taxonomy
Redeploy fybrik helm charts with taxonomy
Submit a fybrikapplication.yaml that has intent=marketing. Now it should pass validation and only the data that isn't personal should be returned

Example 2: Add new transform

Deploy a fybrikmodule yaml that has a redactyAction in its list of actions.  This should fail.  If webhooks are deployed, then an error will be returned.  If not, an error will appear in the fybrikmodule.status
Add redactyAction to the module taxonomy as a new action
Redeploy fybrik helm charts with taxonomy
Deploy the fybrikmodule yaml from step #1.  It should succeed now


Would removing (or moving) Num::from_str_radix() be ever considered?

Related: https://github.com/rust-num/num-traits/issues/178#issuecomment-657421414
Quoting num-traits (this comment itself is super-old),
// FIXME: The standard library from_str_radix on floats was deprecated, so we're stuck
// with this implementation ourselves until we want to make a breaking change.
// (would have to drop it from `Num` though)

The problem: Num requires implementing from_str_radix and tons of other traits in this crate require Num. So, e.g., to be able to make use Signed, you are blocked by having to implement from_str_radix, whether you want it or not. Implementing it properly for a user-defined number type (e.g. a fixed decimal) is a non-trivial task, especially given that it won't actually be used - in most cases it would be simply a waste of time.
This method is not used anywhere in the crate and it's likely it's not used by anyone anywhere at all, simply because better options exist, and the provided implementation suffers from rounding errors.
If from_str_radix() is dropped from Num:

Num becomes implementable for various user-defined number types; as of right now, it's inaccessible because of that method
Num no longer has any methods or associated types whatsoever, it becomes a neat marker trait with some supertraits
As a consequence, various traits like Signed become unlocked for user-defined number types as well

Related thread on internals re: why std::num::from_str_radix is (was) bad. If FromStrRadix was dropped from standard library, why can't it be dropped from here as well?
Finally, if the current implementation has to be retained somewhere, one could move it into a new FromStrRadix trait.

If FromStrRadix was dropped from standard library, why can't it be dropped from here as well?

That's the messy history of the num crate -- these methods were still #[unstable] in std, so they were deprecated in 1.4 and removed in 1.7. But when copied to num, we don't have such stability mechanisms, so they became de-facto stabilized with everything else here.
I don't intend to make breaking changes to num-traits, since the traits have a lot of cross-crate API interaction for our users. I'm thinking I might finally raise to 1.0 with a compatible semver trick to make that position clear.
On Num::from_str_radix in particular, I think we could relax the radix requirements. It's barely specified now, "Convert from a string and radix <= 36," but the integer implementations also enforce the lower bound, assert!(radix >= 2 && radix <= 36), so they panic otherwise. I suggest we codify that, but also allow that implementations may just return Err for any other unsupported radix. Thus a new type could choose to only support radix = 10 like a normal parse, or they might even return an unconditional Err.

If that's the preferred way, then why would int implementations panic and not return Err instead?
Also (just thinking aloud), if returning an unconditional Err is now completely valid, why wouldn't it be a default implementation with some default assoc type as well?

why would int implementations panic and not return Err instead?

We're just forwarding to the inherent methods from the standard library. We could pre-check the radix for an error ourselves, but that still has to fit into a ParseIntError. There's also no direct way to construct that type, so we'd have to fake-parse some other string to get the error we want.

Also (just thinking aloud), if returning an unconditional Err is now completely valid, why wouldn't it be a default implementation with some default assoc type as well?

That would be nice, but that feature is still incomplete and unstable: https://github.com/rust-lang/rust/issues/29661

I suggest we codify that, but also allow that implementations may just return Err for any other unsupported radix.

This would allow me to implement Num for fixed-point numbers in the fixed crate. Currently I cannot because I don't want to implement a method that does not fulfil the trait's specification, but updating the documentation to explicitly allow returning Err on unsupported radices will solve it in a backwards-compatible way.

@cuviper IIUC, this doc clarification/update wouldn't be considered a breaking change and thus can be done prior to 1.0 bump?

I think it's not a breaking change, but I suppose that's debatable. I would not suggest that any existing Num implementations should reduce their radix abilities, or at least that's a breaking change consideration for that type. I think it's OK that new Num implementations may be limited, just as new implementations of Add may have unique overflow limitations, for example.
But my earlier point was that 1.0 would not have any breaking changes either, just reflect the unchanging status quo.


update authn to use cimi user resource

Update the authentication resources and workflows to use the CIMI user resource, rather than pulling user information directly from hsqldb.
Remove the hsqldb configuration, dependencies, and code from the repository.

The core of this task is a reimplementation of the namespace com.sixsq.slipstream.auth.utils.db, using the clojure-based resources rather than the user object from hsqldb. The three main functions are:

Find/set roles for the user (only the super user role from the user resource is important)
Find the password digest for a particular user for internal authentication
Create a user on a valid authentication when the user resource doesn't yet exist

The last feature may be removed in favor of explicit registration. This would simplify the logic of the authentication and reduce the "magic" in the overall user on-boarding process.

This was implemented as part of #1239 and merged with https://github.com/slipstream/SlipStreamServer/pull/1312.  The code cleanup in that namespace is needed though.


Use .cc or .cpp, but not both

Also, .cpp conflicts with the c preprocessor, so i would go for .cc or .cxx, but i have even seen .c++

Fixed by 5be993.


Added Docker and .env.*.example

Added optional Docker capabilities to this project with docker-compose
Written down infos on how to execute it on README.md
Added .env.server.example and .env.client.example, taken from here

I forgot to add a note on why this would be useful.
Well, consider that many developers use Windows, and that redis isn't properly supported on that platform. You may also want to have a clean and no-hassle way to deploy your project somewhere else and leverage the built-in devops utility that docker-compose provides out of the box

Merged, thanks man (I used a similar Docker setup while developing the up)


Add a concept drift algorithm for regression tasks

Concept drift is the measure of model performance degradation. For binary classification, we use the DDM algorithm. We want to implement a concept drift detection algorithm for regression tasks as well.
The concept drift algorithm will go inside this if-else condition in concept_drift.py.
ADWIN is a good algorithm to get started (but feel free to implement your own).

Hi @vipgupta,
Is this issue still relevant? I see you have sourced in the drift algorithms from river in the codebase.

Hi @parthpankajtiwary! Yes, this is solved. Thanks for pointing this out :)


Nixos/boot: fixing yubikey dynamic linking

Motivation for this change
After changes in the yubikey-personalization package, a dynamically-linked library could not be found. This PR fixes this.
The bug is relatively serious: it is luckily caught by tests at updates (causing all NixOS updates in the last n months to fail silently for an unknown n), but any mistake there could potentially make encrypted partitions undecipherable.
Things done

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

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hi ! Is this still an issue ? If so - what would we need to do to get this one merged ? Thanks !

Nope, not an issue anymore, thanks.


feat: --metafile to provide a local metafile to skip bundling

This adds an option --metafile that can be used to provide the path to a local metafile instead of installing and bundling in the tool itself. Probably did bad things with the code, but happy about feedback.
closes #2

Implemented, see this: https://github.com/fabiospampinato/banal/issues/4#issuecomment-1850756549


Rename servlet.py to env_servlet.py.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #286

PR #287   👈

This stack of pull requests is managed by Graphite.


fix - run_transformation  - removed operator ParseDbtProfilesFile and…

… added their functionality to DebussyDbtRunOperator
doing this on separete operators dont work when airflow is running on a cluster, so we moved to one task to work on clusters

PR validated for squad!


[CUML] Benchmarking notebook

I have created a reusable benchmark notebook for the cuML algorithms. It outputs an image of chart to demonstrate the speedup (or slowdown) from the corresponding algorithms in sklearn. I will submit a PR for this.

Hey cjnolet, can you link to the PR that has these benchmarking notebooks when you do them?

PR submitted, and merged into Notebooks extended.


Refactor the guide to user registration flow

The guide to the user registration flow needs some attention. It needs to follow our style guide.
https://www.kubeflow.org/docs/components/central-dash/registration-flow/
Also consider how best to flow from this guide to the more indepth guide on multi-user isolation mentioned in issue https://github.com/kubeflow/website/issues/1613. Consider whether this page becomes redundant or should be merged into one of the new pages we may create.

I'll take a look at this one.

Thanks @simmscg! Does that mean you're also up to tackling https://github.com/kubeflow/website/issues/1613? I think they may go hand in hand.

Oh gosh, TBA. I'm not sure if #1613 will fit into the sprint time that's left. In saying that, #1614 looks like it needs a lot of extra context, so I think you're right. I'll do a bit more reading.

After talking with Sarah, I'm going to keep these both on unassigned -- there are a few steps that need to happen before they get tackled, and they may not happen before the doc sprint finishes.

I'll take this one on.


Azure DevOps Demo Generator issue: Pipeline is not valid due to missing task

Error while creating build definition: The pipeline is not valid. A task is missing. The pipeline references a task called '4fa3aac0-0020-4832-9089-9979c9dd3e69'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version 0.*, job 'Phase_1', step 'Bower_3'.)

Kindly install the necessary extensions from marketplace. These are required for the template to provision the project correctly.

#626


Bugfixes

Updating the README to include steps on how to onboard to your Stripe account

Removing the existing transactions and Stripe account from the DB
Adding the 10% marketplace fee into the payment so that you can see it on your Stripe Account

LGTM, with the exception of removing the stripeUserId in the database.json

LGTM


Trouble with implementation

Hi,
I am trying to use this gem but am unsure of what to put in the controller. I am trying to integrate with Quickbooks and their identity url is
https://openid.intuit.com/OpenId/Provider
Could you provide a quick example for authenticating?

@cwoolum If you still need it - after nearly 3 years - version 1.2.0 was just released providing a configurable identity_url.


Ci reorganization

Codecov Report

Merging #136 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #136   +/-   ##
=======================================
  Coverage   80.75%   80.75%
=======================================
  Files          10       10
  Lines         291      291
=======================================
  Hits          235      235
  Misses         56       56

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 028c7ab...4580b19. Read the comment docs.


Add aside as a block element

Register  as a block element - this element is used for documentation builders such as Slate within their Markdown templates

Thanks for this fix!
Cheers,
Thomas.

Thanks @tzi! Any timeline for an updated version? (I've had to modify the file in my vendor folder for now!)

It's published under the v2.3.0 version
https://github.com/Elephant418/Markdownify/releases/tag/v2.3.0
We dropped support of hhvm and PHP5.3, so it's a minor release.


Feature request: flush results for each 'page'

Hi,
I've noticed that process of downloading messages is split into 500 messages chunks.
Could you add export flushes to csv, to have results for each 'page' of processed data without waiting for the whole process to finish?
Regards,
Lury

Yes, this is needed.  I have over 614 thousand emails in my Gmail account and this will take forever to process all my emails.  It would be great if it could flush the results after 500 messages have been processed.

This is a great idea. I have almost 400k emails in my inbox, and I had to let it run overnight.
I'll need to think about the architecture of the state file, so this might take a bit to implement.


Default value to False for ShowPeoplePickerSuggest

Docs Build status updates of commit d39f008:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@JohanFreelancer9

@get-itips @tseward @dariomws

@tseward

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 3566c4e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Clean up filter design functions

Fixes #416
Fixes #415
Fixes #417

Codecov Report
Attention: Patch coverage is 88.57143% with 4 lines in your changes missing coverage. Please review.

Project coverage is 81.49%. Comparing base (53e7731) to head (9c1a05b).

Files
Patch %
Lines

src/sdr/_filter/_polyphase.py
70.00%
3 Missing :warning:

src/sdr/_filter/_delay.py
75.00%
1 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #418      +/-   ##
=================================================
+ Coverage          81.33%   81.49%   +0.15%
=================================================
  Files                 63       63
  Lines               3798     3771      -27
=================================================
- Hits                3089     3073      -16
+ Misses               709      698      -11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


TSV Downloads of Table Contents

This introduces a new API endpoint and corresponding React component that allows for a user to download a data TSV based on the current parameters used in a query.
I wanted to get feedback on the approach before hooking it up to every table. It also will require some visual polish, I'm not sure how to constrain the size of the MUI loading spinner or error banner, so the button increases in size when either is present. (And the LoadingButton appears to be in beta/not shipped with the default library)
The <TableDownloader/> component takes two props: tableName which is the name of the table you want to download. This is registered in the  table_resolvers dictionary in the table downloads controller server side.
This determines which Resolver will get used to generate the TSV. (Basically which top level query is this table using).
Then, the resolver in question will need to implement two methods: table_headers and to_row(object:). This instructs the resolver how to build the individual TSV rows. For genes here I've implemented the same columns we show on the frontend, but this gives us the flexibility to do other things too. We could include columns like ids, or urls to the individual records that we may not want to show visually in the table.
The second prop on the <TableDownloader> is vars This is simply the same data structure you pass to the gql query used to generate the table in the first place. This ensures the TSV is scoped in the same way as the table as the same params will be passed into the resolver. In this examples, it's the gene names used in the initial search query.
Finally, because the TSV can be arbitrarily large, the entire thing is streamed to the client by the server rather than being built up in memory and sent as one large chunk.
Its not a huge change in terms of lines of code, but its pretty dense, so I'm happy to walk through it start to finish on a call!

per meeting today, @jsstevenson or @katiestahl would you be able to look through this PR as well just to make sure everything checks out?


composer update failed to v3.40

Hi, i try to composer update my laravel app, and i get this issue
Updating vlucas/phpdotenv (v3.3.3 => v3.4.0): Loading from cache    Failed to execute unzip -qq  '/Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/vlucas/phpdotenv/e5b6ed41efa8748d01157c4a3fe078cf' -d '/Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/composer/19329fc5'

[/Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/vlucas/phpdotenv/e5b6ed41efa8748d01157c4a3fe078cf]
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/vlucas/phpdotenv/e5b6ed41efa8748d01157c4a3fe078cf or
        /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/vlucas/phpdotenv/e5b6ed41efa8748d01157c4a3fe078cf.zip, and cannot find /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/vlucas/phpdotenv/e5b6ed41efa8748d01157c4a3fe078cf.ZIP, period.

    The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)
    Unzip with unzip command failed, falling back to ZipArchive class

this make in the end
PHP Fatal error:  Uncaught ReflectionException: Class config does not exist in /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php:790
Stack trace:
#0 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(790): ReflectionClass->__construct('config')
#1 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(667): Illuminate\Container\Container->build('config')
#2 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(615): Illuminate\Container\Container->resolve('config', Array)
#3 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(767): Illuminate\Container\Container->make('config', Array)
#4 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(1225): Ill in /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 790

Fatal error: Uncaught ReflectionException: Class config does not exist in /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php:790
Stack trace:
#0 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(790): ReflectionClass->__construct('config')
#1 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(667): Illuminate\Container\Container->build('config')
#2 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(615): Illuminate\Container\Container->resolve('config', Array)
#3 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(767): Illuminate\Container\Container->make('config', Array)
#4 /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php(1225): Ill in /Users/arywibowo/Sites/laravel-enterprise-app-setup/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 790

and when i downgrade to v3.3.0, its working fine.

i'm sorry, it seems the problem on my local machine, composer issue. i'm close this


Locale handling should be improved

The use of Locale and DefaultLocale in djunits is not optimal. The following issues should (at least) be repaired:

use the standard Java names for the Locale files, such as en-US for the US locale and nl-NL for the Dutch locale.
always format the numbers according to the rules of the Locale.
remove the DefaultLocale class. Use the en_US locale files for prefixes when the used Locale does not have prefix files. Still format the numbers according to the used Locale.
add unit tests using different Locales, for reading (parsing) quantities as well as outputting (formatted) quantities.

Note that the DefaultLocale in Java is set for the entire JVM. This means that our Locale for displaying units can use a static variable. See https://www.oracle.com/technical-resources/articles/javase/locale.html. Note that every time when a formatted scalar, vector or matrix is printed the code needs to check whether the locale has changed, and load the corresponding resource bundle if it has. The Localization class can handle the reading of the resource bundles.

Resource bundles have moved to a subfolder 'locale' in /src/main/resources (/resources in generated jar files), and names of the resource bundles have been shortened (no need to start with 'locale' anymore).

UnitSystem points to a static resource bundle loaded as a UnitLocale. Unit did not do this yet, which is quite strange. Let's define a static localization also in the Unit, so it can print names, abbreviations, etc. in a localized form as well. A few extra methods will be created for localized unit strings.

There are two categories of Locales: those for DISPLAY and those for FORMAT. The FORMAT one takes care of digits, decimal separator and thousands separator. The DISPLAY one takes care of formatting for a user interface in a local language. Therefore, we clearly have to use just the DISPLAY category of the current default Locale. This means that we can, e.g., use Dutch symbols with English formatted numbers,such as 14.2 km/u. Here, the Locale.Category.DISPLAY is nl_NL or nl, whereas the Locale.Category.FORMAT is en_US or en.

This test code now finally works (it was commented out in the unit tests because it never worked as intended):
        Locale.setDefault(Locale.US);
        assertEquals("Duration", DurationUnit.MINUTE.getLocalizedName());
        assertEquals("h", DurationUnit.HOUR.getLocalizedDisplayAbbreviation());
        assertEquals("hour", DurationUnit.HOUR.getLocalizedTextualAbbreviation());
        Locale.setDefault(new Locale("nl", "NL"));
        assertEquals("Tijdsduur", DurationUnit.MINUTE.getLocalizedName());
        assertEquals("u", DurationUnit.HOUR.getLocalizedDisplayAbbreviation());
        assertEquals("uur", DurationUnit.HOUR.getLocalizedTextualAbbreviation());

Localized methods are available and working.
Unit tests have been added and work properly.
Implications for printing and parsing scalars will be picked up in Issue #3 and Issue #4.


Do something on record deletion?

Is there a feature to run a command when we click on the "delete" button?

The reason I need it is because I want to also delete the record from a related table.

Checkout laravels model observers. They can hook into events like deleting so you can do more

You can extend this method if you wish to do it in the controller:
https://github.com/Laravel-Backpack/CRUD/blob/master/src/app/Http/Controllers/CrudController.php#L216
Or you have access to the Eloquent events (which is a better idea for most operations anyway):
https://laravel.com/docs/5.3/eloquent#observers
In the above, you have access to both deleting and deleted events, depending on when you want to tap into the delete action.

@zschuessler Thanks for that. How can I extend the method on my own controller? By calling it on my controller the same way?
    public function destroy()
    {
          //my code
    }

@madd86 you would create a function in your controller with the same name.
Then after your custom code you can call parent::destroy()
However I'd recommend the Observer route as it's more universal and less hacky

The issue is that the deleting observer doesn't work because according to taylor "Yes, this is basically expected. In order to fire the events we would have to pull the models first and then call the events with the model instances. Then call the delete query."
https://github.com/laravel/framework/issues/2536


CI tests are very slow

We should check that parallelism in the tests isn't thrashing GitHub actions, and also that the test cases we have are suitable. The timing of each test is produced in the CI logs (currently dominated by the GA and bootstrap tests).

Thanks, some tests can probably be simplified (GA-bootstrap); 👍