Qt configuration does not seem respected on Linux

It looks like neither "maxconnections" nor "addnode" are respected on Linux in the ~/.config/MintCoin/MintCoin-Qt.conf file. Research and fix!

It turns out that "maxconnnections" and "addnode" are configuration that is applicable to both the command-line daemon and the GUI, so are stored in ~/.MintCoin/MintCoin.conf. The ~/.config/MintCoin/MintCoin-Qt.conf file is only for GUI-specific options, although confusingly the GUI program will leave a single assignment of every option that it does not understand in there, so:
addnode=<IP_ADDRESS>
addnode=<IP_ADDRESS>
addnode=<IP_ADDRESS>

Will become:
addnode=<IP_ADDRESS>

And the others removed, without warning.
I guess this is probably okay, and not worth fixing as this behavior falls out of the Qt configuration model. But maybe some documentation explaining it would be in order.

Okay, this is addressed in the doc-updates branch:
https://github.com/shane-kerr/Mintcoin-Desktop-Wallet/tree/doc-updates

I think this one is sufficiently addressed in the doc-updates and on my 2.0 branch.


HDDS-2106. Avoid usage of hadoop projects as parent of hdds/ozone

Ozone uses hadoop as a dependency. The dependency defined on multiple level:

the hadoop artifacts are defined in the  sections
both hadoop-ozone and hadoop-hdds projects uses "hadoop-project" as the parent

As we already have a slightly different assembly process it could be more resilient to use a dedicated parent project instead of the hadoop one. With this approach it will be easier to upgrade the versions as we don't need to be careful about the pom contents only about the used dependencies.
See: https://issues.apache.org/jira/browse/HDDS-2106

Thanks @arp7 and @adoroszlai the review. There are no more warnings (missing parts are also migrated from pom.xml) and the integration test failures are not related.
Will merge it soon.
For the records: this is just the first step. As we have this brand new parent pom, later we can simplify the hadoop-ozone/pom.xml and hadoop-hdds/pom.xml as many common parts can be moved to pom.ozone.xml


Docsy template: not-yet-localized pages missing

This is a Bug Report

Problem:
~With the (draft) Docsy version of the Kubernetes website,~ pages and sections that are only partially localized have fallback hyperlinks to the English localization originals, but only in
Compare the main section of https://kubernetes.io/it/docs/concepts/architecture/ to the left navigation on the same page - there are more links in the left nav than in the page body.
Proposed Solution:
Improve localization for page body. This may mean working with upstream Docsy.
Comments
Issue description adjusted; the left nav links were previously also missing.
/priority backlog

/remove-lifecycle stale

/remove-lifecycle stale

/kind bug

/area web-development

/remove-lifecycle rotten
This is an important fix to prevent readers of the localized page from missing the untranslated pages.

Duplicated by https://github.com/kubernetes/website/issues/27572

/triage accepted
/lifecycle frozen
/priority important-longterm

This is fixed.
/close


Pathway Spring Cleaning

There appears to be a large number of old and possibly unused branches that exist in Pathway. I would like to get back into the codebase and start designing functionality, except that I have no idea where to start due to two pressing issues:

I have no idea what incarnation/branch of Pathway has the most recent code. Where are we working? What are the feature branches for? Is Master up-to-date such that I could fork off of it?
Our bug tracker is currently a mess, cluttered with Old and New and things that should probably be Iceboxed. We probably need to do some spring cleaning here as well.

Master is currently completely up to date with everything that isn't in active dev.
The following feature branches are currently ahead of master:

api-dev: forked off of master 4 days ago, currently just contains a prototype of the Scala internal FileHandle API
test-refactor: contains all of the work I've been doing on the Great Test Embetterment project.
wish: I added an initial skeleton for a Wish to this branch several months ago. Besides that, it is several commits behind master.

The gh-pages branch is just used by Travis for publishing the API docs to GitHub Pages, so we don't need to mess with that. The io-refac branch contains changes to IO that have been merged into master for a while, it can (and probably should) be deleted.

What's the status on this issue? Is this closable?

Okay, I went ahead and deleted the io-refac and test-refactor branches. I'm not sure if we care about any of the code in the api-dev and wish dev branches.

It looks like we can close this now that the test-refactoring stuff is merged in and we've pruned a number of the stale dev branches.


[STORM-1377] Adds lock when launching a test cluster

There seems to be a race that results in certain tests accessing the wrong test nimbus server, and that is causing failures.  This gates attempts of any of the tests to talk to a nimbus cluster.

storm-core jdk8 had 1 failure each on netty-unit-test and nimbus-test
!storm-core jdk7 had an apparent crash while testing storm-sql-core
Will close and re-open to kick another build.

+1

+1

I am not sure this will solve the problem—the jdk8 build failed even on this branch. I have reproduced this on a local, single-core cluster.  Closing until I can find a more reliable work-around.
I am suspecting an unexpected exception is being thrown here, and it is not actually a locking issue.
Closing until I can figure this out.


readd npm run start compatibility because you dumbasses removed it 💀

…krolled2:

also please make proper commits i CANNOT read this commit history at all my god


Create plugin documentation

We should have a proper documentation of our plugins:

[ ] Add a list of plugins we maintain to README
[ ] Link to the pynxtools-plugin gh topic in the README
[ ] Write a tutorial/how-to on dealing with plugins and how to create a plugin/reader (based on https://github.com/FAIRmat-NFDI/pynxtools-plugin-template)
[ ] Add a list of extra installs

This should also be reflected in the https://github.com/FAIRmat-NFDI/pynxtools-plugin-template, i.e.,

Link to main repo
Link to pynxtools-plugin topic

#231 resolved here!


i run st2web  but faild  who can give me some suggest plz. -)

everything is ok until last step .


Handle pasted text to create entities

Hello
While pasting the content, we can get the pasted content in onPasteRawText but then again editOnPaste handles the pasted data. My usecase is something like this:
I have a input where when a user types a comma separated values, I convert them into entities(immutable) and decorate them to look like this which I have done.

But the same steps need to be done to the pasted data. I might be missing something on how to handle the pasted data.
Any help would be appreciated.
Thanks

Hmm. onPasteRawText could possibly be a canceling handler, i.e. handlePastedText. Then you would be able to return true if you handle the paste yourself.

Yes. Something like handleBeforeInput is a cancelling event.

I would be fine with this. Would you like to submit a PR to replace onPasteRawText with handlePastedText?

Sure. I will do the changes and submit the PR.

@hingermayank thanks for adding handlePastedText. Are there are any examples of how to use it?

Found an example of handlePastedText for anyone searching in the future: https://github.com/facebook/draft-js/issues/416#issuecomment-221639163


feat(context): move service map to context top-level

The graphback property has been removed from GraphbackContext.
export interface GraphbackContext {
-  graphback: {
-    services: GraphbackServiceConfigMap
-  }
+  graphback: GraphbackServiceConfigMap
}

Now you can reach the Graphback services like so:
- context.graphback.services.Note.findBy(...);
+ context.graphback.Note.findBy(...);

This is another enabler PR for #1637

Services as key can be easily changed in createContext method. Prisma do not even care to namespace services and mount every obj to context

Let's merge. I will push templates and play with the changes

Not ready for a merge

I have adjusted the code and updated the description. Now you can call Graphback services like this:
context.graphback.Note.findBy(...)

Services as key can be easily changed in createContext method.

Not true, the SchemaCRUDPlugin relies on the existence of services to load the services.

Thanks @machi1990


Support for MARL

Do you plan supporting environments for Multi-Agent RL in the near future?
This would be a key feature in my choice for a RL library.

Hi,
we are interested in multi-agent but there is no clear timeline, although did some analysis on what would be needed. Since this is a wide field, any specific semantics/algorithm you have in mid?

I had in mind a similar API to the one used in RLlib:

Having a MultiAgentEnv where the obs_dict, rew_dict, done_dict, info_dict = step(action_dict) method receives a dictionary with unique agent string ids as keys and the corresponding agent's actions as values. It would also return similar dictionaries for the observations, rewards and termination signals.
The environment can also optionally return a mega state to train centralised value functions.
Multiple agents can share the same policy model or not.

In terms of algorithms,  it would be great if non MARL-specific algorithms like the PPO would work out-of-the-box in Multi-Agent environments. I feel that it be more important to have the necessary structure to interact with MultiAgentEnv than to implement MARL-specific algorithms. Ideally the specific MARL algorithms could be implemented by users of the library once they need them.

Ok, the environment is not hard to implement, the thing I am still thinking about and which potentially needs a bit more planning is how multiple agents are coordinated.
I agree in first instance one could do just a collection of independent agents. The main issue is that RLgraph is not centered around Ray the same way RLlib is, so would have to decide what entity holds these agents.
I thought about a MultiAgentCoordinator which wraps a collection of agents and is then used in the Ray executors even in single-agent cases (and in the single-threaded executor), why leaving individual agents as is. Sharing the same policy would then require syncing. Alternatively (and requiring much larger changes), is to allow individual agents to hold multiple policies and include sharing in the build process (similar to RLlib's agents actually being containers for collections of policies).


[Feature]Deej remember setting after reboot

Is it possible to make deej remember the settings after a reboot?,  eg if I have the microphone muted or volume set to a specific level it should automatically remember that setting during the next startup.

Hi there @oddife, thanks for writing.
If you're using potentiometers, which is what deej supports, the behavior you're describing is just a consequence of leaving them at their absolute positions. At startup, deej just sets all volume to whatever potentiometer readings it gets from serial - if it didn't do that, it wouldn't be doing its primary function which is to translate analog readings to app volumes.
If you're using rotary encoders (which deej doesn't officially support), these aren't absolute as they can rotate infinitely to either direction. In this case it's up to you to take care of storing their current values to persist between reboots of the Arduino hardware. I've seen some people use an EEPROM for this, but I haven't done this myself and I'm not sure what the actual lifespan of a solution like this is (since you'd be writing to this EEPROM quite a lot).
At any case, this is consciously out-of-scope for the project because encoders aren't officially supported.

thank you for the replay, yes I'm using encoders as I needed to mute the inputs with that switch, EEPROM would be a bad idea so I was thinking could the software make a file that would write the values once they are changed and reload it every time the software starts.

As I mentioned, this is out-of-scope for deej because it doesn't include support for encoders. You can still implement it for yourself in a fork, but this isn't going to be a core feature.


feat(cheatsheet): link to jamdocs.org

Since jamdocs.org is now live, let's link to the appropriate site there instead.
--> https://jamdocs.org/interface/00-cheatsheet/

👌
Thinking of it, we should probably also add a link to the docs to the settings page in Jam. Tracked https://github.com/joinmarket-webui/joinmarket-webui/issues/428.


Update cl arg --intervention to pass multiple interventions to simulation

Update --intervention arg to enable multi-intervention simulations in the future, such as Boiler Upgrade Scheme + Boiler Ban
As not all interventions we plan to implement will result in a subsidy seen by households (e.g. boiler ban), the logic used in household.get_total_heating_system_costs() is amended to anticipate this (previously, a £0 subsidy would only be assigned if model intervention attribute was empty)

Thanks for adding the test.


Issue on eBanking and mobileBanking payment preference

Describe the bug
Throws ImageStreamListener error while navigating to eBanking or mobileBanking tab and whole app freezes.
To Reproduce
Steps to reproduce the behavior:

Enable PaymentPreference.eBanking or PaymentPreference.mobileBanking preferences using KhaltiScope.of(context).pay(preferences: []).
Tap on eBanking or mobileBanking tab.
ImageStreamListener exception thrown from image.dart

Device Information (please complete the following information):

Device: Android emulator (android 11.0)

Additional context
Error may have been thrown as there are some thumbnails images missing for few banks. Proper inclusion of image thumbnail may help.

This is KPG-v1 and this version of SDK is no longer maintained and already in the deprecation schedule.
Please upgrade to KPG-v2 checkout version
https://github.com/khalti/checkout-sdk-flutter


Docs: consistent usage of CSS sections in /helpers, /layout

Description
Following https://github.com/twbs/bootstrap/pull/38175, this PR suggests more modifications in the way we display CSS and Sass info globally in our docs:

Most of the time, I've integrated when not already there a surrounding "CSS" section
I've systematically used the plural form for the section titles (even if they contained only 1 element):

"Sass mixin" → "Sass mixins"
"Sass loop" → "Sass loops"
"Sass map" → "Sass maps"

So it means that I also changed the URLs impacted by these modifications
In order to keep the same spirit of "Sass variables", "Sass mixins", "Sass loops":

"Utilities" → "Sass utilities API"
"Utilities API" → "Sass utilities API"

To summarize, here's the new structure:
## CSS
### Variables
### Sass variables
### Sass maps
### Sass mixins
### Sass loops
### Sass utilities API

Note
Regardingsite/content/docs/5.3/layout/css-grid.md, I decided to let it as is because the content is really linked to a limitation and doesn't seem to fit our regular CSS section.

Hope I haven't broken too much links in the process 😇
Motivation & Context
Docs headings global consistency around CSS and Sass.
Type of changes

[x] Refactoring (non-breaking change)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] My change introduces changes to the documentation
[x] I have updated the documentation accordingly
(N/A) I have added tests to cover my changes
[x] All new and existing tests passed

Live previews

https://deploy-preview-38219--twbs-bootstrap.netlify.app/

@mdo I've updated this branch by going one step further in this refactoring for consistency. Love to hear your thoughts about it. Re-requesting a review (final this time).


[Loc] update to sql-database-projects

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27586

Relevant Lines:
60910

💛  - Coveralls


Suddenly Super Smash Brothers Ultimate error message

Super Smash Brothers Ultimate suddenly shows an error message when launched. Worked fine with dvr-patches before but not anymore

I don't recall any new update being released recently so i think it's on your end. Did you install any new game mod or sysmodule ?

I have one mod installed and some sysmodules but it worked with them just fine too

Also in the tesla overlay sysdvr is constantly stuck at switching when switching from usb to OFF

Appearently it's the modpacks fault. I deleted it and everything works again... Even after deleting the dvr-patches it still didnt work but without the modpack it does again so it's not the patches's fault


squat_analysis: add body transforms dictionary

Adding body transforms to get trunk lean for squats. Not sure if this should only update the utilsKinematics.py file for squats, or if it's OK for this to be different than the one used in other analysis folders.

Thanks @antoinefalisse!


updation of batter and pitcher

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #3989

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

Thank you @msifran ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @msifran, you can't open another pull request before completing the previous one 😀
You already have the following 1 open pull request(s):
#3992

Thank you  @msifran , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[🐞] Qwik City's useLocation doesn't detect the origin scheme

Which component is affected?
Qwik City (routing)
Describe the bug
We are using the useLocation for to set the canonical url like in the Qwik City starter. Once we put the Qwik app behind a reverse proxy like AWS Cloudfront or behind ngrok or simply a load balancer, the canonical url is with http instead of https.
In our RouterHead we have:
export const RouterHead = component$(() => {
  const head = useDocumentHead();
  const loc = useLocation();

  return (
    <>
      <title>{head.title}</title>
      <link rel="canonical" href={loc.url.href} /
...

Reproduction
https://registry.jolimont.rosa.be/fr/search/
Steps to reproduce
Live example

Go to the reproduction url https://registry.jolimont.rosa.be/fr/search/
The url is with https but the canonical url is with http.

To reproduce locally

Install the qwik starter.
Serve it via ngrok.
The canonical url in the page is with http instead of https.

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.20.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.4.2 - /usr/local/bin/npm
    pnpm: 8.6.10 - /usr/local/bin/pnpm
  npmPackages:
    @builder.io/qwik: ^1.1.4 => 1.1.4
    @builder.io/qwik-city: ^1.1.4 => 1.1.4
    undici: 5.22.1 => 5.22.1
    vite: 4.3.5 => 4.3.5

Additional Information
It's unclear for me what is the best way for Qwik to detect the origin and scheme. Client side it is easy but it isn't on the server side. I would expect Qwik City to rely on the origin header, but it's not a mandatory header.
I would rather not rely on the ORIGIN parameter int he vite config https://qwik.builder.io/docs/guides/static-site-generation/#origin because we want to serve the same app under different domains.
Maybe we should be able to pass the origin to the renderToStream function? I would prefer a good default but I don't mind handling the logic if needed.

Looked into it, and I can't think of a way for the QwikCity to know what the original protocol was. It is not something that is reliably sent over to the server. (Unless you know of a way)
May I suggest this workaround?  When generating URLs just generate them without the protocol, that way they will default to the same protocol as the current page, which will solve the issue.
{location.url.toString().substring(location.url.protocol.length)}

I don't fully get your idea.
I've investigated a bit more. My understanding is that we cannot trust the headers in general. We need to rely on the host.
Once we use a reverse proxy we can't rely on the host anymore but we can setup the reverse proxy to forward the headers. The reverse proxy adds X-forwarded-* cookies with the original host.
We can even tell expresse.js to handle this with http://expressjs.com/tr/guide/behind-proxies.html
app.set('trust proxy', true)
req.hostname will come from X-Forwarded-Host
I'm going to try this out.

My understanding is that there are two problems:

Forwarding of the host: This is done through the Host header, and it already works. But Host header does not contain the protocol (http/https) only the host. such as my-host.com
Forwarding of the protocol. Was the original request http or https. This information I don't know how to get.

So my suggestion is to just do //server.name/path by skipping http: the URL in the browser will default to whatever the current protocol scheme is.

Ahh https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Proto is what I was missing. Let me dig into it further.


README references QRScanner

The README references QRScanner but instead it should be Keychain

Fixed in version 1.0.4


Regression: the trait bound for<'de> arrow::datatypes::Schema: serde::de::Deserialize<'de> is not satisfied

Describe the bug
Arrow 24 does not support de-serializing schemas with the flow code anymore.
To Reproduce
let schema: Result<arrow::datatypes::Schema, serde_json::Error> =
                serde_json::from_reader(schema_file)

The same code used to work with previous arrow versions. See https://github.com/domoritz/csv2parquet

Have you enabled the serde feature on arrow_schema?

No. Do I need to make arrow schema an explicit dependency?

I think you may need to, yes. I didn't plumb a feature flag through when I split the crates apart. This wasn't intentional, but I think I would rather leave it as is, less default features the better even if it was accidental 😅

Thank you. Adding arrow-schema = { version = "24.0.0", features = ["serde"] } fixed it.
Having to update multiple crates concurrently is a bit tricky since dependabot cannot do it right now. It would be awesome if you could make it so that the parquet crate automatically imported arrow and arrow schema at some point.
I'll close this issue since my issue is solved. Thank you @tustvold!


Difficulty level

I think that during a game it could be useful showing the difficulty level chosen.

good idea - i've added this in v2.1.13 - will be shown both on current turn and on final scoring screen


[5.2] fix docblock of Routing

It is strange that StyleCI didn't fix that before

Why should StyleCI fix this? It's not a static analysis tool, it just fixes code style.

In terms of fixing the 2 spaces thing, we plan to add that later on, and fix all that across the entire framework.

OK, let me revise the PR, and fix the docblock but not the space problem.
BTW, @GrahamCampbell is it a good idea to refactor some function of Routing to new file?

BTW, @GrahamCampbell is it a good idea to refactor some function of Routing to new file?

I don't think so?


Pack Booleans

With this type, it packs the buttons compound type into 10 bytes instead of the preferable 2 bytes, or even 1.25 (though this would probably be much slower). Could you change this? I didn't see any plans for backwards-compatibility in the spec.
const Format = new Type({
  version: 'uint',
  frames: [{
    buttons: {
      A: 'boolean',
      B: 'boolean',
      X: 'boolean',
      Y: 'boolean',
      L: 'boolean',
      R: 'boolean',
      ZL: 'boolean',
      ZR: 'boolean',
      plus: 'boolean',
      minus: 'boolean',
    },
    sticks: {
      L: {
        x: 'uint',
        y: 'uint'
      },
      R: {
        x: 'uint',
        y: 'uint'
      }
    }
  }]
});

Hello @leo60228 ,
Currently you can implement this yourself on top of this lib, using a Buffer and them extracting the bits:
function getBit(buffer, n) {
  return buffer[n>>3] >> (n & 0x7) & 1
}

We could implement a 'bit' type in the future that could group multiple fields into a single byte, but this isn't straight forward as it seems at first.


Overriding the broadcast event name

Hi folks,
I'm using Pusher to broadcast events to a React front-end. It's working spot on, however we're exposing full class names in our JS in order to listen for that event. Is there anyway to override the event name from say App\Events\MySomethingEvent to app.myevent?
Cheers!

Apologies, just did some digging around and found broadcastAs(), couldn't find that in the docs though?


WIP: Add instructions for setting up gitsign

What does this PR do
Now that Chainguard Enforce is running on this repository, this now adds
instructions to the README.

@JAORMX thanks for this, although we have a separate doc for development, if you can drop these notes at the top of that doc -https://github.com/metal-toolbox/alloy/blob/main/docs/README-development.md

@joelrebel will do. Thanks!

@JAORMX if theres no other changes to this, lets have it merged

I forgot about this, sorry about that.


Detect transaction rollbacks

Apologies if this is already a feature/documented, but I couldn't find it.
I'm trying to assert that an database insert/modification did not happen because it was rolled back.
Tracker.history.insert shows rolled back inserts, so I'm wondering if there is an alternative way to determine if something was rolled back?

Yes you can, by inspecting the tracker.history.transactions field, which contains all transactions + it statuses.
Check the test code


OSS local files crawl error

Dear all,
Any body plz let me know i have thousands of files that are indexed now through localfiles crawl template. But a big issue, in files where there are spaces they are not opening while clicking on the hyperlink, even open folder is not working. its hard for me to rename the files and remove spaces.
in files and folders where are no spaces they are opening fine upon clicking.
Plz give me a suggestion, really stuck.
Thanks.

Hi,
I just made some test. The spaces are encoded as %20 in the HTML source code which seems to be correct. Does your Internet browser generates any error in its console ?

Hi, really thanks for the reply.
Yes %20 is filled with spaces in html. But all browsers show error, page can not be displayed. Where there are no spaces in folders and files paths the files downloads and open directly without error.
I tried putting \atcokb\test<file:///\atcokb\test>, //atcokb/test   in crawler locations path, but didn’t work.
Also When I press viewer to view files, only pdf files show in viewer whether there is a space or without space in file and folder names.
But docs, rtf, excel and power point files  not displaying in viewer even.
also if there is a space in file name or folder name Open Folder option is not working. (open folder option is only working for the files in which there are no spaces.)
Plz show some light in the tunnel. Waiting for your kind reply.
Regards
Tufail Shah
System & Network Administrator
Al Tamimi & Company
Dubai International Financial Centre
Registered with the DFSA
T: +971 4 364 1641   F: +971 4 364 1777
www.tamimi.com
Bahrain   ▪   Egypt   ▪   Iraq   ▪   Jordan   ▪   Kuwait   ▪   Oman   ▪   Qatar   ▪   Saudi Arabia   ▪   UAE
From: Emmanuel Keller<EMAIL_ADDRESS>Sent: Monday, December 21, 2015 3:05 AM
To: jaeksoft/opensearchserver
Cc: Tufail Shah
Subject: Re: [opensearchserver] OSS local files crawl error (#1688)
Hi,
I just made some test. The spaces are encoded as %20 in the HTML source code which seems to be correct. Does your Internet browser generates any error in its console ?
—
Reply to this email directly or view it on GitHubhttps://github.com/jaeksoft/opensearchserver/issues/1688#issuecomment-166162084.
Our Firmhttp://www.tamimi.com/en/section/our-firm | Our Practiceshttp://www.tamimi.com/en/section/our-practices | Our Sectorshttp://www.tamimi.com/en/section/our-industries | Our Officeshttp://www.tamimi.com/en/section/our-offices
http://feedback.tamimi.com/EmailMarketing/ClickThroughHandler.aspx?link=<a+href%3D"http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company">http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company<%2Fa>&nid=abbdcaba-9bb1-48d3-b48 [cid:a72d435c9c334535accb65ffd2081eab] http://www.linkedin.com/company/al-tamimi-&-company       [cid:4f32c15f9b13473d918fdfc136db686b]
Email Confidentiality Notice
The information in this email is private, confidential and may be covered by legal professional privilege or other legal or attorney/client privilege. It is intended solely for the person to whom it is addressed. If you are not the intended recipient please telephone or email us immediately and remove it from your system. If you are not the intended recipient you may not copy it or use it for any purpose nor disclose the contents to any other person.

Furthermore for your kind information in the browser when I manually remove %20 to a normal space. the files open successfully.

Ok. Can you please edit the Renderer and just check the "URL decode" checkbox for the fields "Title" and "Filename".
It should work now.

A big big thanks. Its indeed a Christmas gift. It worked like a charm . I was literary struggling from last one week.
Once last thing how can I remove Open Folder and Viewer option. As viewer is a good option but only works for pdf files .doc files are not opening in viewer.
Thanks.
Our Firmhttp://www.tamimi.com/en/section/our-firm | Our Practiceshttp://www.tamimi.com/en/section/our-practices | Our Sectorshttp://www.tamimi.com/en/section/our-industries | Our Officeshttp://www.tamimi.com/en/section/our-offices
http://feedback.tamimi.com/EmailMarketing/ClickThroughHandler.aspx?link=<a+href%3D"http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company">http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company<%2Fa>&nid=abbdcaba-9bb1-48d3-b48[cid:7cd30f5ac2ca4ac78e3830dda9d89d25]http://www.linkedin.com/company/al-tamimi-&-company      [cid:8eaf7b5f6fe2487080d64483df6d7a71]
Email Confidentiality Notice
The information in this email is private, confidential and may be covered by legal professional privilege or other legal or attorney/client privilege. It is intended solely for the person to whom it is addressed. If you are not the intended recipient please telephone or email us immediately and remove it from your system. If you are not the intended recipient you may not copy it or use it for any purpose nor disclose the contents to any other person.

Once last thing how can I remove Open Folder and Viewer option.

Just remove the last field from the Renderer. The one with the "Tools" widget.

Really simple and intuitive interface. Thanks for your kind help. All fixed now. OSS is excellent.
One last thing I was wondering daily we are creating new files, what about them, if I run crawler again it will recrawl all again. Is there any method or incremental crawler so to avoid the prvious crawled files.
Much appreciate.
Tufail Shah
System & Network Administrator
Al Tamimi & Company
Dubai International Financial Centre
Registered with the DFSA
T: +971 4 364 1641   F: +971 4 364 1777
www.tamimi.com
Bahrain   ▪   Egypt   ▪   Iraq   ▪   Jordan   ▪   Kuwait   ▪   Oman   ▪   Qatar   ▪   Saudi Arabia   ▪   UAE
From: Emmanuel Keller<EMAIL_ADDRESS>Sent: Tuesday, December 22, 2015 6:06 PM
To: jaeksoft/opensearchserver
Cc: Tufail Shah
Subject: Re: [opensearchserver] OSS local files crawl error (#1688)
Once last thing how can I remove Open Folder and Viewer option.
Just remove the last field from the Renderer. The one with the "Tools" widget.
—
Reply to this email directly or view it on GitHubhttps://github.com/jaeksoft/opensearchserver/issues/1688#issuecomment-166625090.
Our Firmhttp://www.tamimi.com/en/section/our-firm | Our Practiceshttp://www.tamimi.com/en/section/our-practices | Our Sectorshttp://www.tamimi.com/en/section/our-industries | Our Officeshttp://www.tamimi.com/en/section/our-offices
http://feedback.tamimi.com/EmailMarketing/ClickThroughHandler.aspx?link=<a+href%3D"http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company">http%3A%2F%2Fwww.linkedin.com%2Fcompany%2Fal-tamimi-%26-company<%2Fa>&nid=abbdcaba-9bb1-48d3-b48 [cid:e4a896b9e9b8432faec1ce982681d039] http://www.linkedin.com/company/al-tamimi-&-company       [cid:a0fe0067a5ae47c696a9d460d9b041c3]
Email Confidentiality Notice
The information in this email is private, confidential and may be covered by legal professional privilege or other legal or attorney/client privilege. It is intended solely for the person to whom it is addressed. If you are not the intended recipient please telephone or email us immediately and remove it from your system. If you are not the intended recipient you may not copy it or use it for any purpose nor disclose the contents to any other person.


Properties in different classes inheriting same base classes

I should look into the following issue:
If two classes inherit from the same base class and use different mappings for the same property name, there will be a conflict in json2typescript.

Hello,
i use abstract base class : child class A extends B. its seems it doesn't work with this use case.
could check that please ?
regards

Can you please provide a minimal example (just the classes you would like to use) so that it fails?

Glad you fixed it. Yes, it's annoying to initialize all properties, but it's necessary.

Hi,
I have the same problem even with properties initialized.
Can you give me some help ?
My classes
import {JsonObject, JsonProperty} from 'json2typescript';

@JsonObject
export class TestAbstract {
  @JsonProperty('title', String)
  private _title: string = undefined;`

  constructor() {
  }

  get title(): string {
    return this._title;
  }

  set title(value: string) {
    this._title = value;
  }
}

@JsonObject
export class TestString extends TestAbstract {
  @JsonProperty('value', String)
  private _value: string = undefined;

  constructor() {
    super();
  }

  get value(): string {
    return this._value;
  }

  set value(value: string) {
    this._value = value;
  }
}

@JsonObject
export class TestNumber extends TestAbstract {
  @JsonProperty('value', Number)
  private _value: number = undefined;

  constructor() {
    super();
  }

  get value(): number {
    return this._value;
  }

  set value(value: number) {
    this._value = value;
  }
}

My test exemple
let jsonConvert = new JsonConvert();
jsonConvert.operationMode = OperationMode.LOGGING;
jsonConvert.valueCheckingMode = ValueCheckingMode.ALLOW_NULL;
const test = jsonConvert.deserialize({title: 'Title1', value: 1}, TestNumber);
const testString = this.jsonConvert.deserialize({title: 'Title2', value: 'test string'}, TestString);
console.log(test);
console.log(testString);

The error I get
core.es5.js:1020 ERROR Error: Uncaught (in promise): Error: Fatal error in JsonConvert. Failed to map the JSON object to the class "TestString" because of a type error.

	Class property:
		_value

	Expected type:
		number

	JSON property:
		value

	JSON type:
		string

	JSON value:
		"test string"

	Reason: Given object does not match the expected primitive type.

Error: Fatal error in JsonConvert. Failed to map the JSON object to the class "TestString" because of a type error.

	Class property:
		_value

	Expected type:
		number

	JSON property:
		value

	JSON type:
		string

	JSON value:
		"test string"

	Reason: Given object does not match the expected primitive type.

You can see I have 2 children with attribute _value.
In TestNumber is typed as Number
In TestString is typed as String
But when I try to deserialize, json2typeScript try to convert my string in number for TestString.
Thank you in advance for your answer.

As mentioned in the first post, this issue is to be expected. Right now, it is not possible to use the same property name with class inheritance.
There is an easy workaround though:
No one (except you) will care about the name of the private property. You will be exposing only the public getter/setter. Just name the _value of the second class to _value2 or whatever, but keep the getter/setter names.
I will fix this issue in the next release, of course.

@Sooulm8 the current version (1.0.3) fixes this problem.
There is only one case I couldn't properly "cover". If you consider your example, it is important that if you write the decorator JsonProperty at _value in TestString, you must also write it in TestNumber. Otherwise it will make a "fallback" and cause an error.


Fix mapping status modes owner/admin/etc to op in TS6

I need to consider how I want to handle +qah/etc. forwarding to TS6 servers like charybdis which do not support these statuses.
We cannot assume that anyone with +q or +a or any mode higher than +o has +o in addition to whichever other statuses they have. Therefore it seems essential that a mode change of +q actually translates to +o. It must also be handled properly such that +qo does not translate to +oo and so on.
The question is whether it makes sense to do the opposite, i.e. to translate -o to -q—making ops on charybdis as powerful as owners on juno—or to instead immediately reverse the mode deop if done remotely by someone with a lower status. A third option would be to ignore +q and friends entirely on charybdis, which is the current behavior. The problem with doing that is that some channel actions require ops at a local level and may be ignored by charybdis.
I could also consider making it configure how you want this to behave:

Translate -o to -q
Reverse remote deops from servers without higher statuses
Ignore the existence of these modes altogether

Of course, all of this can be avoided by disabling juno's extra status modes.

Also, this currently happens:
[ircd::server] convert_cmode_string(): converted +qo 0c 0c (k.notroll.net) to +o 000AAAAAC 000AAAAAC (ts6.notroll.net)

The +q is weeded out but the parameter remains. This is bad.

The most common way to deal with this is to just ignore the +qah entirely if the target doesn't support it. The reason why services set +qo and +ao instead of plain +q/+a is probably for compatibility for clients/bots that don't care about extended op statuses.

I was worried that some commands are ignored if you're not op, even remotely. Maybe I'm thinking wrong.

decided to do what @GLolol said and remove unknown modes and their parameters, forwarding on without them


Cow: Watch ICUhcX7KuJJ(?#dly.to)

Cow requests the watch of the watch_keyword ICUhcX7KuJJ(?#dly\.to). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ICUhcX7KuJJ(?#dly\.to) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'prefer the case-sensitive version (?-i:)'


Collection reports don't always update when new sketches are added in FSM project

This needs more testing but this morning I created a network with 5 shapes, ran a report, then added another shape and re-ran the report. The size report didn't appear to update with the additional shape. I then changed the name of the collection (adding "v2) and the size report updated.

Collection added to Admin forum.

This is a tricky one. SeaSketch needs to send an accurate updatedAt date when requesting new reports so that the stale caches are avoided. This means keeping the collection metadata up to date when child sketches are modified. I've handled this correctly when a user edits a sketch that is part of a collection, but there are a whole lot of other cases to handle.

[ ] sketch is added to a collection
[ ] sketch is removed from a collection
[ ] child sketch is deleted
[ ] + these same cases but for child folders that contain sketches

That's a lot to cover. Working on this now.


Remove background video buffering process

Following is my piece of code which I am using to remove all the created players. The code works fine but the issue I am facing is that the video after changing via ajax call success keep on buffering as background process. Kindly let me know how can I kill that background process as well so it should not buffer the video in background.
var player;
    $('video').each(function() {
        player = $(this)[0].player;
        player.remove();
    });

Try setting the source as '' before removing.
player.setSrc("");


Adding metadata populated model

Any pull request you open is subject to the TensorFlow Hub Terms of Service at www.tfhub.dev/terms.
Cc: @lu-wang-g

Where can I host the files?
__main__.MarkdownDocumentationError: Found the following errors: {'sayakpaul/deeplabv3-mobilenetv2_dm05-float16/default/lite/2.md': 'The asset-path https://github.com/sayakpaul/Adventures-in-TensorFlow-Lite/releases/download/v0.14.0/deeplabv3-mobilenetv2_dm05-float16.tar.gz is a url that cannot be automatically fetched. Please provide an asset-path that is allowed to be fetched by its robots.txt.', 'sayakpaul/deeplabv3-mobilenetv2_dm05-int8/default/lite/2.md': 'The asset-path https://github.com/sayakpaul/Adventures-in-TensorFlow-Lite/releases/download/v0.14.0/deeplabv3-mobilenetv2_dm05-int8.tar.gz is a url that cannot be automatically fetched. Please provide an asset-path that is allowed to be fetched by its robots.txt.', 'sayakpaul/deeplabv3-mobilenetv2_dm05/default/lite/2.md': 'The asset-path https://github.com/sayakpaul/Adventures-in-TensorFlow-Lite/releases/download/v0.14.0/deeplabv3-mobilenetv2_dm05.tar.gz is a url that cannot be automatically fetched. Please provide an asset-path that is allowed to be fetched by its robots.txt.', 'sayakpaul/deeplabv3-xception65/default/lite/2.md': 'The asset-path https://github.com/sayakpaul/Adventures-in-TensorFlow-Lite/releases/download/v0.14.0/deeplabv3-xception65.tar.gz is a url that cannot be automatically fetched. Please provide an asset-path that is allowed to be fetched by its robots.txt.'}

While I do have some GCP credits to host them inside a GCS Bucket but those credits are not permanent.

@kempy should be all set now.

@kempy any idea as to when the models may be online?

Hi @sayakpaul, sorry for the delay - there was an issue with ingesting them that has been fixed now - the models are now published and live on tfhub.dev.
http://tfhub.dev/sayakpaul/lite-model/deeplabv3-mobilenetv2_dm05/1/default/2
http://tfhub.dev/sayakpaul/lite-model/deeplabv3-mobilenetv2_dm05-float16/1/default/2
http://tfhub.dev/sayakpaul/lite-model/deeplabv3-mobilenetv2_dm05-int8/1/default/2
https://tfhub.dev/sayakpaul/lite-model/deeplabv3-xception65/1/default/2
As mentioned above, you can now remove the original assets from your own GCS bucket as they are now hosted on Hub. We are also investigating recommended solutions for hosting assets pre-ingestion since github is not a viable solution and we understand the challenges with setting up GCS for students and others. We'll update the docs on github and tensorflow.org when we have another recommended solution.

Great. Thank you so much.
Regarding the solution, direct Drive downloads are problematic but not with gdown. If one could host their model file archives on Drive and make it publicly accessible it can be downloaded with gdown.


Better markdown

https://roam-tools.ryanguill.com/?flatten-indentation=999

Roam42 Issue Log Audit - Unclear


feat(cross-platform): port home directory and tabbing to linux/mac friendly options

This addresses #44 and allows z to be used on Linux and Macs as well as Windows.
The three cross-platform bugs were:

$Env.USERPROFILE isn't cross-platform, $Env.HOME is
The Get-DirectoryEntryMatchPredicate function looks for multiple drive names, on Linux/Mac this is only / which needs escaping to be used correctly as a regular expression
$function:tabexpansion2 was expecting an extra carriage return in the existing function which doesn't exist on Mac/Linux and was blanking the whole of $function:tabexpansion2

Thanks again for writing this plugin, it's great to have z!
Joe.

Thanks for merging @vincpa!

@joejag Thanks for working on this to make it compatible with *nix based systems :)
I've made a slight change in how the home directory is retrieved on windows and committed this to master.
Can you please verify all is working on Mac?

@vincpa Just tested with rev f8675c3 on master and all working good with your modification included on Linux

Great. I'll create an updated package in the next day or two for the PowerShell gallery so it's available to everyone.
Thanks again for this.

Done https://www.powershellgallery.com/packages/z/1.1.11

Installed and tested, all good. An aesthetically pleasing version number too. Thanks!


feat: now possible to only output non-resource related metrics

Fixes #1812.

It's still possible to get them but you would need to activate a flag in future versions, i.e. staring from v5 probably. This PR should keep the current behavior as-is.
Are per-resource metrics that interesting by themselves or are they symptomatic of the SDK needing maybe better, more structured logging and/or tracing, though?

Also, we could indeed remove the name/namespace tags only

Turns out that removing meters without delay doesn't work reliably due to the fact that cleaning operations open in different threads and heavily depend on how / when the API server responds to the SDK so I will remove the possibility of immediately removing the meters.

@scrocquesel made the per-resource collection more granular and added docs, would appreciate a new review 😄

Some few things.
Also want to share my thoughts on per schedule cleaner as I don't know if creating lots of timer has memory/perf issues. I think the objective is to keep stale meters until they have been collected one last time by the scrapper. Thus the configured delay should be a little more than the scrapping delay. We may remove meters by batch after they have spend delay seconds in a per generation collection. I guess with two generations, we can rotate every delay seconds where stale meters are recorded and clean them upon next switch.

I have to admit that I'm not too familiar with how the scrapping work and I'm not sure whether our implementation can actually be notified that scrapping occurred.

Ideally, I would love to have some sort of a event souurce and being able to clean all stale meters when the event is raised. The event would be raised by the metric endpoint route handler after returning metrics to the scrapper. I checked quarkus route handler implementation but currenlty it is not possible to know when metrics are collected. If it make sense we may open an issue at quarkus. JOSDK could expose the a signal or method to be wired up in QOSDK.

That's an interesting idea. I'm not sure how feasible that is, though.

Looking at failedReconciliation, I wonder if we should also reduce cardinality on exception with a builder.includeExceptionTag().

Looking at failedReconciliation, I wonder if we should also reduce cardinality on exception with a builder.includeExceptionTag().

I don't actually want to make things too configurable or too fine-grained with the default implementation. It shouldn't be too hard for people to use that as a starting point for their own implementation.


Add metacat dataset creation to DeclaD

in mover.py the declaD currently has code to declare a Rucio dataset at line 475-476  based on the format in the rucio_dataset_did method, drawing from the dataset_did_template in the config file.
What we need the declad to now do is to also declare a metacat dataset with the same namespace and name.  this could be inserted into the metacat declaration code just above it.. lines 449-450 or thereabouts.
This is essential to get the declad up and working with rucio v33.

Marc supplied a patch, I tried it, it works.

actually there is another issue.. if there is more than one file that is supposed to go into the same metacat data set it is trying to create the metacat dataset every time, and failing to do so and throwing an exception.  Believe this can be solved with a simple try/except

I see there already is such a structure around this code.
Traceback (most recent call last):
File "/home/np04data/miniconda3/lib/python3.10/site-packages/pythreader/task_queue.py", line 210, in run
result = task.run()
File "/home/np04data/declad/declad_2.0.1/mover.py", line 480, in run
mclient.create_dataset(f"{dataset_scope}:{dataset_name}")
File "/home/np04data/miniconda3/lib/python3.10/site-packages/metacat/webapi/webapi.py", line 471, in create_dataset
return self.post_json(url, params)
File "/home/np04data/miniconda3/lib/python3.10/site-packages/metacat/webapi/webapi.py", line 215, in post_json
response = self.send_request("post", uri_suffix, data=data, headers=headers, stream=True)
File "/home/np04data/miniconda3/lib/python3.10/site-packages/metacat/webapi/webapi.py", line 164, in send_request
raise AlreadyExistsError(url, response)
metacat.webapi.webapi.AlreadyExistsError: Already exists
Above is the exception we are getting.
I made the following changes:
from metacat.webapi import MetaCatClient, MCServerError
from metacat.webapi.webapi import AlreadyExistsError
and now mover.py looks like this
            if mclient is not None:
                # create in metacat first...
                try:
                    mclient.create_dataset(f"{dataset_scope}:{dataset_name}")
                except metacat_client.MCServerError:
                    pass
                except metacat_client.AlreadyExistsError:
                    pass

This seems to handle multiple files correctly now

and added the following to mover.py
            if mclient is not None:
                # create in metacat first...
                try:
                    mclient.create_dataset(f"{dataset_scope}:{dataset_name}")
                except metacat_client.MCServerError:
                    pass
                except metacat_client.AlreadyExistsError:
                    pass


Need multiple actions of same HTTP verb on same resource (but different request content-type)

We have an API that provides two roughly equivalent forms for POST to the same URL. The key difference is the Content-Type: of the request: it accepts either application/json or text/dns (RFC-4207). The two request forms are closely related, but it would be an exaggeration to say they're merely direct translations one of the other.
Legacy Blueprint allows us to say this. The FORMAT:1A editor ignores it (only displays one of the two).
Is there some way I can achieve this?

Although an endpoint can have multiple transaction examples, as seen in apiaryio/snowcrash#53 these don't render out very cleanly in Apiary.
Take the following example...
You have /oauth2/authorize, which by specification will handle various different scenarios (such as getting an access token from refresh, or generating a token from password, etc). Having these merged into a single action on the API docs is very hard for the reader to understand, instead it's easier for the user to have multiple endpoints such as "Refresh access token" and "Authenticate User" in the side menu. However this is not possible because you cannot specify the same method/URL combination twice.

I see incorrect syntax highlight - see the color of Body:


merge deplay.sh and start_bfs.sh in sandbox

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Added components image

Signed-off-by: Patrick Hession<EMAIL_ADDRESS>Describe your changes in brief
Changes

Added image

View rendered doc/Components.md

@hessio, please update your branch

Would be better at top


Warning message in loadFromQIIME2

Hi @yiluheihei @tvborm
loadFromQIIME2 throughs a few warnings. Can you have a look?
https://felixernst.github.io/mia/reference/loadFromQIIME2.html
Thanks!

Those errors occur when TSE is created and phylogenetic tree is added there. Creating that phylogenetic tree does not give any errors. Those same warning came before yeasterday's modifications, but I did not pay attentiont to them that much.
For me, it is quite hard to say what is wrong. I tried to find the problem, but I could not. Those same 4 warnings came also with Chandler's data, so it is probable that the problem is not with data (duplicate data points etc...)
However, same .read_qza function is also used for fetching other data, and there is no problems

Those errors occur when TSE is created and phylogenetic tree is added there. Creating that phylogenetic tree does not give any errors. Those same warning came before yeasterday's modifications, but I did not pay attentiont to them that much.
For me, it is quite hard to say what is wrong. I tried to find the problem, but I could not. Those same 4 warnings came also with Chandler's data, so it is probable that the problem is not with data (duplicate data points etc...)
However, same .read_qza function is also used for fetching other data, and there is no problems

I think the reason for this warning is that the node label of the phylogenetic tree is not unique and some nodes have no label.

I think the reason for this warning is that the node label of the phylogenetic tree is not unique and some nodes have no label.

The question remains, why it is throw 4 times? Is this in mia or in TSE? If it is the first, this has to be fixed once, with a warning, but not 4.

The question remains, why it is throw 4 times? Is this in mia or in TSE? If it is the first, this has to be fixed once, with a warning, but not 4.

I think it might be in TSE
@fionarhuang I saw the same warnings pop up in the manuscript on page 16. I think the warning is correct, but I am not so sure, about it appearing 4 times. That seems excessive.

I also think it is in TSE, since there is no warning while I import QIIME2 results as phyloseq.

I also think it is in TSE, since there is no warning while I import QIIME2 results as phyloseq.

But thats a different code base, isn't it?

Yes, it is, but the core (base) code I used to create phyloseq object is the same with it in loadFromQIIME2.

I agree that the problem is in TSE, because I ran the code one row at the time, and there were no errors until the function call
TreeSummarizedExperiment(
        assays = S4Vectors::SimpleList(counts = feature_tab),
        rowData = taxa_tab,
        colData = sample_meta,
        rowTree = tree,
        referenceSeq = refseq
    )

and then those 4 warning came

Hi all,
The warning was from convertNode that was called several times in the constructor of TreeSummarizedExperiment. I have fixed the warning by checking only nodes that are used in the link data. Before,  all nodes of the tree are checked to see whether they have duplicated or missing labels.
Overall, the warning should disappear in version 1.99.11 when testing on the example data provided in this issue.  Hope it works with the new version.

Fixed! https://felixernst.github.io/mia/reference/loadFromQIIME2.html
Thanks!


permission denied to run ./index.js

Im not able to run the index file after cloning

Run chmod 777 ./index.js in the react-cli directory


Fix typo in community principles

This pull request adds a missing word to the community principles section. Thanks @rpowis for the spot!

You can preview this change here:
Built with commit ee91f5c1e5267c8907afbccc3a26a30b93fc7447
https://deploy-preview-793--govuk-design-system-preview.netlify.com

Thanks @36degrees – that's what I aim for 🚀


proc: Improve performance of loadMap on very large sparse maps

proc: Improve performance of loadMap on very large sparse maps

Users can create sparse maps in two ways, either by:
a) adding lots of entries to a map and then deleting most of them, or
b) using the make(mapType, N) expression with a very large N

When this happens reading the resulting map will be very slow
because loadMap needs to scan many buckets for each entry it finds.

Technically this is not a bug, the user just created a map that's
very sparse and therefore very slow to read. However it's very
annoying to have the debugger hang for several seconds when trying
to read the local variables just because one of them (which you
might not even be interested into) happens to be a very sparse map.

There is an easy mitigation to this problem: not reading any
additional buckets once we know that we have already read all
entries of the map, or as many entries as we need to fulfill the
MaxArrayValues parameter.

Unfortunately this is mostly useless, a VLSM (Very Large Sparse Map)
with a single entry will still be slow to access, because the single
entry in the map could easily end up in the last bucket.

The obvious solution to this problem is to set a limit to the
number of buckets we read when loading a map. However there is no
good way to set this limit.
If we hardcode it there will be no way to print maps that are beyond
whatever limit we pick.
We could let users (or clients) specify it but the meaning of such
knob would be arcane and they would have no way of picking a good
value (because there is no objectively good value for it).

The solution used in this commit is to set an arbirtray limit on
the number of buckets we read but only when loadMap is invoked
through API calls ListLocalVars and ListFunctionArgs. In this way
`ListLocalVars` and `ListFunctionArgs` (which are often invoked
automatically by GUI clients) remain fast even in presence of a
VLSM, but the contents of the VLSM can still be inspected using
`EvalVariable`.

I also tried to step back and thing of other solutions to this for a while and couldn't come up with anything. While hard coding a value is never my favorite thing, I think it makes sense in this case that we make this determination internally for these general loading commands, and evaluate fully when specifically asked to eval the variable in question.
@zolotov @ramya-rao-a @dlsniper Does this solution make sense from an API integration point of view?

Fine by me.

It does look good to me, thank you.


New data science

Overall really nice! I don't see any need to refactor.

Take a look at the comments, but this is really good.  You're done with this one.


general: update dependency of intl to any after upgrading to latest Flutter version

Is there an existing issue for this?

[X] I have searched the existing issues

Package/Plugin version
9.0.0-dev.1
Flutter doctor

Flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.0, on macOS 13.3.1 22E261 darwin-x64, locale de-DE)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc2)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.78.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Minimal code example

Code sample

(no code required)

Current Behavior
Currently this package and the validators package have a fixed dependency on intl.
Expected Behavior
The dependency of intl should be any for both this package and validators.
Steps To Reproduce

install latest flutter version (see doctor output)
try to build an ios app which has a dependency on this package
get compiler error about the intl dependency.

Aditional information
Using any for intl dependency is also suggested in the Flutter documentation.
👀  https://docs.flutter.dev/accessibility-and-localization/internationalization#setting-up
I cloned the repos and fixed the issue here:

https://github.com/bastibense/flutter_form_builder
https://github.com/bastibense/form_builder_validators

Please fix this issue quickly, so I can remove the dependency to my fixed git repositories from my project again.

Duplicated


Generating arbitrary ADT's

Assigned @liadGolan
To @liadGolan - if you want to throw your hands in the air, go for it.  You're the only other one in the class who indicated any Haskell experience.


bindings/bash/powerline-exectime.sh is not installed with pip install

Thank you for developing this great feature!
The installation is turning tedious when having to manage dozens of python installations. Example instance
sudo /usr/bin/python3.9 -m pip install git+https://github.com/powerline/powerline
This does not install the bindings directory.
The rest of the installation are going to be challenging to distribute to hundreds of users so I'm hoping to reduce the complexity

pip is also installing this package into site_packages/powerline_exectime and not into site_packages/powerline-exectime

bash: --2024-11-08: command not found...
bash: /home/greg/.local/lib/python3.11/site-packages/powerline_exectime/bindings/bash/powerline-exectime.sh: line 2: syntax error near unexpected token (' bash: /home/greg/.local/lib/python3.11/site-packages/powerline_exectime/bindings/bash/powerline-exectime.sh: line 2: Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...'


Fix Native Editor button backgrounds

For #10154

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Yeah, this just came in recently and is still in new issues. But, as a non-standard theme user, these issues really annoy me and it was a quick fix. So exercising a little prerogative to just pull it in and fix it quick :).

Codecov Report

Merging #10234 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10234      +/-   ##
==========================================
- Coverage   61.27%   61.22%   -0.06%
==========================================
  Files         567      567
  Lines       30832    30862      +30
  Branches     4424     4432       +8
==========================================
+ Hits        18893    18894       +1
- Misses      10956    10984      +28
- Partials      983      984       +1

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
70.58% <0%> (-5.89%)
:arrow_down:

...ient/datascience/interactive-ipynb/nativeEditor.ts
53.11% <0%> (-2.42%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

...c/datascience-ui/react-common/settingsReactSide.ts
22.22% <0%> (ø)
:arrow_up:

.../datascience/interactive-common/interactiveBase.ts
16.74% <0%> (ø)
:arrow_up:

src/client/datascience/webViewHost.ts
60.54% <0%> (ø)
:arrow_up:

...ience/interactive-common/interactiveWindowTypes.ts
100% <0%> (ø)
:arrow_up:

src/client/datascience/types.ts
100% <0%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

reopening for NPM ci


Calling methods returns "is not a function"

I'm initializing a pie chart exactly as described in the docs, and it renders just fine, but trying to run a method on it—either in the js file or in the console—returns "not a function". For example, trying to add data returns: TypeError: $(...).addData is not a function

Same happens to me, any method like clear() stop() destroy() throws undefined:
Chart.destroy()
VM618:2 Uncaught TypeError: Chart.destroy is not a function

Try to run the function on the canvas.
var ctx = $("#myChart").get(0).getContext("2d");
var myPieChart = new Chart(ctx).Pie(data,options);
myPieChart.addData({...});

Fiddle:
http://jsfiddle.net/5t590x5v/1/

It works only if you call that in the same breath as declaring it... I'm trying to programmatically insert data, for example if a button is clicked, so I have to select it like so:
$("#myChart").addData();
Which doesn't work.

you have to hold a global reference for later use.
That's not a Chartjs problem.
you can hold it in the window object or declare it in global scope at the top of your script and use it later.
var ctx = $("#myChart").get(0).getContext("2d");
window.myPieChart = new Chart(ctx).Pie(data,options);

and later:
window.myPieChart.addData({...});

@fastrde if that was the case, erorr would be "Chart is not defined" in this case theres no method called
clear() or destroy()

@danielhq sorry but that's wrong.
When you call $("#myChart") jo got a jQuery object that representing the html-canvas with some funky jquery functions extending it.
when you call new Chart(ctx).Pie(data,options); you get a independet Chartjs-Object that has nothing to do with the funky jQuery Object (and you have to hold a reference to it by yourself). So when you call destory, addData,... on the jQuery Object it is not a function. When you call it on the Chartjs object it will work.

@fastrde is correct on this one. You need to save the Chart object into a variable in the appropriate scope and then call addData on that

@fastrde Thank you for your help! Not only has this helped my Chart understanding, but leveled up my Javascript a bit, as well!


Remove viewBox-based chart scaling

angularjs-nvd3-directives/angularjs-nvd3-directives/pull/240 added chart scaling that causes the entire chart to scale as a unit to the dimensions of the container given a starting height and width. This prevents resizing only one dimension and resizes all elements including labels, controls, and legends uniformly leading to unreadable charts.
We can leave the functionality in for sparklines where it might be useful.

Completed in #4.


switch_user_group(): rpmlint reports call to setgroups before setuid

switch_user_group(): rpmlint reports call to setgroups before setuid:
Building coreos-metadata-3.0.1, Crate users is pulled in via dependency crate update-ssh-keys-0.3.0. rpmlint reports the warning:
RPMLINT report:
===============
coreos-metadata.x86_64:
W: missing-call-to-setgroups-before-setuid /usr/bin/coreos-metadata
This executable is calling setuid and setgid without setgroups or initgroups.
There is a high probability this means it didn't relinquish all groups, and
this would be a potential security issue to be fixed. Seek POS36-C on the web
for details about the problem.

The warning may not indicate an actual problem, but it would be helpful to eliminate the warning. Upstream https://github.com/coreos/update-ssh-keys and https://github.com/coreos/coreos-metadata have been notified via https://github.com/coreos/coreos-metadata/issues/118.
Per @lucab discussion in that issue:

The warning is related to switch_user_group in the users crate (all versions): https://github.com/ogham/rust-users/blob/15af1576e40fd3e5592b918de353df6a976c42c4/src/switch.rs#L134-L143
We are calling that in update-ssh-keys: https://github.com/coreos/update-ssh-keys/blob/v0.3.0/src/lib.rs#L108
In our specific case I think this is not a security bug. We are calling that method in order to align user/group on file creation only, not to drop privileges for the process (those are reset when the guard value is dropped at the end of the function).

Interesting! I think you’re right in saying this doesn’t affect your case, but rust-users is a general library and someone else is bound to run into the same problem eventually. Also it looks like doing the Rust equivalent of setgroups(0, NULL); is enough to make the warning go away.

@ogham if you are going that route, it looks like the guard should record getgroups state before resetting groups and recover them back on drop.

This doesn’t go all the way to solving the problem, but I’ve changed the code to setgid before setuid in https://github.com/ogham/rust-users/commit/f1b9902212be41ea24633a4575a110da9a76c7be.


istio proxy 1.9.5 - DLA-2563-1, DLA-2492-1 vulnerabilities

Hello,
istio-proxyv2:1.9.5-distroless is marked by Aqua ( a container security scanning tool ) as having the following vulnerabilities:
DLA-2563-1 (High) - CVEs related: CVE-2021-23840, CVE-2021-23841
DLA-2492-1 (Medium) - CVEs related: CVE-2020-1971
istio-pilot:1.9.5-distroless is fine, no 'issues' found.
Both images were downloaded from dokerhub, istio/proxyv2:1.9.5-distroless & istio/pilot:1.9.5-distroless
Could you give a bit of context if the methods described in the these vulnerabilities are used by istio and when a fix will be provided, if necessary?
Thank you
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Environment where the bug was observed (cloud vendor, OS, etc)
AWS -EKS, Amazon Linux 2

Hello,
Thank you for your response.

@howardjohn Any plan port the fixes to istio 1.9.x ?

No, the changes are high risk as they bring in 3+ years update on the underlying base image


Rust Language Server support

Duplicate of https://github.com/onivim/oni/issues/1712 but I have tested this against the wiki config:
Tested with the following config.js:
module.exports = {
    "language.rust.languageServer.command": "rls"
}

Does not work:

rls is available on the path.
Also tested with absolute path:
module.exports = {
    "language.rust.languageServer.command": "/Users/cetra/.cargo/bin/rls"
}

We get a bit further, but it still does not work:

I am using the following settings with success:
// Language Support - Rust
"language.rust.languageServer.command": "rustup",
"language.rust.languageServer.arguments": ["run", "stable", "rls"],
"language.rust.languageServer.rootFiles": ["Cargo.toml"],

I mostly selected them because nightly rust rls was crashing at the time, but maybe this modifies the environment or something? Most language-server plugins for rust, I know of, (atom and vscode) do use rustup to manage their rust installations, so I thought this might be of help.

I have tried that command but I get the following messages in the console along with as pinnning wheel:
No grammar found for language: rust
Measure request fail

@Drakulix thanks for your snippets. Looks like Oni is trying to instantiate the RLS when using them, but still get the same problem as originally mentioned by the creator of the issue:
[Language Client] Notification textDocument/didOpen - P:\GH\rust\sample-rust\src\main.rs: start
bundle.js:1 [Language Client] Notification textDocument/didOpen - P:\GH\rust\sample-rust\src\main.rs: end
bundle.js:1 [LANGUAGE CLIENT - STDERR]: ERROR 2018-04-03T13:01:55Z: rls::server: dispatch error, Error { code: InvalidParams, message: "missing field `version`", data: None }

bundle.js:1 Received response message without id: Error is:
{
    "code": -32602,
    "message": "missing field `version`"
}

@appelgriebsch I had the same issue in the beginning. I am trying to track down whats causing this and why I am currently not experiencing it anymore.

I am getting the same error here, any fix?

@Drakulix if you run rustup run stable rls --version what do you get?

@wezm rls-preview 0.125.1-stable (cebf188 2018-03-19)

I am using the following:
// Language Support - Rust
"language.rust.languageServer.command": "rustup",
"language.rust.languageServer.arguments": ["run", "nightly", "rls"],
"language.rust.languageServer.rootFiles": ["Cargo.toml"],

But nothing happens to me.  rls is in the path and works well with Atom.  Is there anything else I need to do to active Rust support on Oni?  Can it be that Oni is not recognizing the file name extension?

@FredrikAleksander any luck successfully getting the rls to run without issue or is the above still happening?

If I run the nightly version of RLS, it seems to work ok (sometimes diagnostics stops working temporarily, but it will start working again after a few changes). Looking at the RLS code, it was quite strict earlier and it would crash if any input was not what it expected (duplicate or out of sequence version numbers). In the nightly version it will just log a error instead

Interestingly I tested the Rust language server in a plain neovim environment with the LanguageClient-Neovim and this works quite well with the settings above (rustup run stable rls)...

@appelgriebsch thanks for checking this with another client. I'm quite keen to get this working as well, I think our lsp client has some issues as it reports some errors with our current typescript client as well as our intended replacement theia-ide-language-server some clients seem to work well but others face issues as in #2448 I think we need to surface logs a bit more clearly and debug where what part of the client maybe causing the errors.
In your case can you get a screen shot of your dev console when the client fails to see what the error is?

I had similar problems, but after updating rls it now seems to work.
$ rustup run stable rls -V
rls-preview 0.127.0-stable (d2ade31 2018-05-08)

I'm a little confused, because it didn't work on the first try after updating, but only after the 4th or 5th reload of oni. I can't recall whether the error message was gone right after updating.

I did an update and install of rls and its dependencies as mentioned here: https://github.com/rust-lang-nursery/rls, which gives me that

λ rustup run stable rls -V
rls-preview 0.127.0-stable (d2ade31 2018-05-08)

It still shows me the same error mentioned above:

I'm also running into this issue, running oni release 0.3.6. Seeing similar issues as reported in this thread, and have rls working fine on Atom.
$ rustup run stable rls -V
rls-preview 0.127.0-stable (d2ade31 2018-05-08)

I'm just trying out Oni, and having some similar issues. Any ideas?


Transpile all the main process scripts and drop hot-module-loading.

This uses babel to transpile all of the main process scripts and also removes
all of the hot-module support and the web server that it required. All generated
scripts go into the lib directory and the app runs from there using file links
which will make using preload scripts much easier.
For the main process scripts we only transpile if there has been a change. The
copy function could probably be faster but there isn't one built into node O_o.
fixes #14, fixes #40, fixes #61, fixes #134.
Signed-off-by: Dave Townsend<EMAIL_ADDRESS>

Yeah so mainly I don't understand how jsx files are treated; if we're still using babel-register to load those, that's sucky. Let's transpile evetything.
Everything else looks good.

Might also have to update the "import/no-unresolved" rule in .eslintrc

Should we still package dev dependencies now? See task-package.js. I'm not really sure why we were doing that in the first place.

@victorporof  devDependencies are explicitly ignored in task-package.js

@jsantell Oh right! I had that the other way around. Carry on :)

I'm not sure if lib is a good name for all the built things, but have been trying to think of a better name for literally over a week now at this point so :yolo:

I'm not sure if lib is a good name for all the built things, but have been trying to think of a better name for literally over a week now at this point so :yolo:

That was basically where I got to too.

Coverage remained the same at 83.459% when pulling 294d962ba17dd5362330094ed11a756572996eba on mossop/babelallthethings into a61274aa334fed28194ac984a871a1a64ca12620 on master.

@victorporof I think this is all up to date from your comments. Look good to you?

Coverage remained the same at 83.459% when pulling 98f8a7e95e5f786f84b2339241adf37bbc805fc5 on mossop/babelallthethings into a61274aa334fed28194ac984a871a1a64ca12620 on master.

r++

Landed in 15d6b658a61d771d83ccdc2df28af5d3b7e8f6ad


Can't figure out how to create a user

I've installed the Meemo, It seems to be up and running. But i can't figure out how to add a user.
./commands.sh add username password doesn't seem to do anything, considering the fact that after adding a user i get error at the meemo page saying about invalid credintials
running ./commands.sh ls prints out information about users not existing, even though i created one not too long ago.
root@Theo:~/meemo# ./commands.sh ls
./commands.sh: 3: [: !=: unexpected operator
====== List of users ======
Using local auth file:  /users.json
No users. Use `admin user-add` to add some.

Maybe your shell doesn't understand the script (for example ./commands.sh: 3: [: !=: unexpected operator seems odd)

considering the fact that after adding a user i get error at the meemo page saying about invalid credintials

Can you comment with the error you get?
If you have a look at the script, you can try adding a user directly using the docker command:
USER=youruser
PASSWORD=yourpassword
docker-compose exec meemo ./meemo/admin user-add -u $USER -p $PASSWORD --display-name $USER;;

Let me know the outcome, thanks for taking the time to create the issue.

Had the same error: ./commands.sh: 3: [: !=: unexpected operator
Doing some quick searching, this seems to be because different distros link sh to different shells. On Alpine, it links to ash, dash on my Debian (producing above error) and on OpenSUSE via /etc/alternatives/sh ultimatively to bash.
Simple fix: Edit the first line of commands.sh to #!/bin/bash :)
About the credentials: This repo's readme states to chmod 400 users.json which makes the file read-only. Permissions need to be 600 so the owner (1000, i.e. not root) can write to it.
Still can't login, though. Trying do get more info.

@espressoelf Thanks! I've updated the script to use /bin/bash and the readme to chmod 600 users.json.


Stationed vires in cell only seems to change on replication ticks

Currently, the stationed vires in a cell are only changing on the client side when a replication packet is received.
Replication should be updated immediately when receiving a movement reply as well.

-> Transmit source cell in movement packet, target cell and speed
-> Only transmit replication at the start of the game

Backend & protocol support for this fix added in 823df1d04743ea53f1229305dc66fbbc237a3b9d.


Update ios.md for ACS UI library

Update UI library version

@palatter
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looks good. @ShaunaSong you can mark with #sign-off if you are ready

#sign-off

#sign-off


LEGO: Pull request from lego/hb_839_20181017094254268 to master

LEGO: Pull request from lego/hb_839_20181017094254268 to master with localized lcls

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: csigs sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Closing due to conflicts.


Allow modification of image expiration field

The expiration field for images currently is read-only. The possibility to allow modification of it might allow automatic removal of old images created locally.
Forum post: https://discuss.linuxcontainers.org/t/automatic-local-image-expiration-how-to-set-expires-at/5761

This issue would be a good candidate for a new contributor to tackle.
What needs to happen is effectively:

Add an API extension in doc/api-extensions.md
Add the matching string in shared/version/api.go
Update shared/api/images.go to move ExpiryDate from Image to ImagePut
Update lxd/db/images.go to have it accept updating that database field
Update lxd/images.go to have ImagePut pass the updated field to the database function (if set)

No changes should be needed to the client, the update to shared/api should be enough to have the field show up in lxc image edit.

Myself and my partner from the UT Virtualization class would like to take this issue as part of our class project

Thanks, assigned it to you. Hopefully the instructions I pasted in the previous comment will help you figure out what needs to happen.
I would expect a pull request made of 3 commits for this one:

api: Add images_expiry API extension
shared/api: Make image ExpiryDate modifiable
lxd: Make image ExpiryDate modifiable

This should all be pretty easy to do and a good start to see how to contribute to LXD.

Hi, I'm the partner, could I also be assigned?

Done


Tensorflow crashes when i try to quantize my output

BUG
(https://github.com/tensorflow/tensorboard/issues).
Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

$ lspci -vvv|grep -i nvi
00:1e.0 VGA compatible controller: NVIDIA Corporation GM204GL [Tesla M60] (rev a1) (prog-if 00 [VGA controller])
        Subsystem: NVIDIA Corporation GM204GL [Tesla M60]
        Kernel driver in use: nvidia
        Kernel modules: nvidia_375_drm, nvidia_375
- **Exact command to reproduce**:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
When I tried to quantize my output

       result = SESS.run(layers[0], {'incept/DecodeJpeg/contents:0': f.read()})

       result = SESS.run(tf.quantize_v2(layers[0], min_range=0, max_range=10, T=tf.quint8), {'incept/DecodeJpeg/contents:0': f.read()})

I run out of memory on my GPU
2017-08-16 20:59:40.313241: W tensorflow/core/common_runtime/bfc_allocator.cc:277] ***********************
*****************************************************************************
2017-08-16 20:59:40.313284: W tensorflow/core/framework/op_kernel.cc:1158] Resource exhausted: OOM when al
locating tensor with shape[720,1280,3]
Traceback (most recent call last):
  File "generate_hashes.py", line 89, in <module>
    for (fn, points) in map_results.result():
  File "/usr/local/lib/python2.7/dist-packages/pebble/pool/base_pool.py", line 208, in next
    raise result
tensorflow.python.framework.errors_impl.ResourceExhaustedError: OOM when allocating tensor with shape[720,
1280,3]
         [[Node: incept/Cast = Cast[DstT=DT_FLOAT, SrcT=DT_UINT8, _device="/job:unknown_job/replica:0/task
:0/gpu:0"](incept/DecodeJpeg_G511)]]
         [[Node: incept/pool_3_G513 = _Recv[client_terminated=false, recv_device="/job:unknown_job/replica
:0/task:0/cpu:0", send_device="/job:unknown_job/replica:0/task:0/gpu:0", send_device_incarnation=-69665892
98495454362, tensor_name="edge_1131_incept/pool_3", tensor_type=DT_FLOAT, _device="/job:unknown_job/replic
a:0/task:0/cpu:0"]()]]

Caused by op u'incept/Cast', defined at:
  File "generate_hashes.py", line 78, in <module>
    load_network(False)
  File "generate_hashes.py", line 40, in load_network
    return tf.import_graph_def(graph_def, name='incept')
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/importer.py", line 311, in impo
rt_graph_def
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2506, in create_o
p
    original_op=self._default_original_op, op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1269, in __init__
    self._traceback = _extract_stack()

ResourceExhaustedError (see above for traceback): OOM when allocating tensor with shape[720,1280,3]
         [[Node: incept/Cast = Cast[DstT=DT_FLOAT, SrcT=DT_UINT8, _device="/job:unknown_job/replica:0/task
:0/gpu:0"](incept/DecodeJpeg_G511)]]
         [[Node: incept/pool_3_G513 = _Recv[client_terminated=false, recv_device="/job:unknown_job/replica
:0/task:0/cpu:0", send_device="/job:unknown_job/replica:0/task:0/gpu:0", send_device_incarnation=-69665892
98495454362, tensor_name="edge_1131_incept/pool_3", tensor_type=DT_FLOAT, _device="/job:unknown_job/replic
a:0/task:0/cpu:0"]()]]

Please fill out the template fully (you left out some sections such as "Python version". Also provide the code that runs out of memory when you quantize the output.
You may be running out of memory on your GPU because adding the quantize op takes some amount of extra memory. How much memory is used when you do not quantize?

I broke the quantization into a separate step. Sorry for not filling the form out right. But yes you were right the quant op ran out of memory

OK, closing the issue since the issue was the GPU ran out of memory.


33 configurable s3 bucket for multi deployments

merged into https://github.com/climsoft/climsoft-api/pull/39


Recitation 8: Use of BigInteger

How does the use of BigInteger avoid the problem of overflowing?

A BigInteger is a data type in Java that allows you to represent and perform calculations with extremely large integers that would exceed the range of the standard integer data types (like int or long). Essentially, it provides a way to work with arbitrarily large integers, limited only by the available memory in the system. For example in rec8 Fibonacci, the integers can get very large, therefore to avoid a overflow BigInteger is used.

It depends on the JDK. For instance OpenJDK uses a int[]
Array size is not limited by size like primitive types, making them suitable for storing arbitrarily large values as long as there is enough memory

BigInteger is a class in java, and Unlike primitive integer types such as int or long, which have fixed sizes and can overflow if the result of a computation exceeds their maximum representable value, BigInteger can handle calculations involving integers of any magnitude. it will not overflow, as it provides a flexible and dynamic representation of arbitrary large integers


Expand C-family docstring resolution

Before:

After:

Summary:
Each supported language in these extensions have a set of properties attached to them (file extensions, what identifier patterns look like, what comments look like, etc). One of these properties is docstringIgnore, which will allow us to ignore the lines between where a ctags definition occurs and the comment that is meant to belong to it. Until now, all we've used this for is for Java-style annotations, e.g.
/** docstring **/
@MyBigAnnotation
class Something {
    // ...
}

This adds support for the C-family convention of breaking definitions up onto multiple lines:
/** docstring **/
RET_VALUE
ActualMethod(...) {
    // ...
}

This could have unintended side-effects where comments that do not belong to a ctags definition just happens to be close enough to it, but I think it's an acceptable risk tradeoff (while hand-testing I couldn't get it to do something worse than the previous behavior).
Fixes https://github.com/sourcegraph/sourcegraph/issues/17925.

Side question: How to add a test for something like this (is it simple?)

@tjdevries It's difficult to test changes to the language spec properties, so we default back to testing by hand. In order to run the extensions, you can side load the extension so it can be run with a live SG instance (I used sourcegraph.com here as a user reported an actual line of code, which is always nice).
It would be nice to add integration tests here, but it could be a bit heavy-handed as it requires an instance (and search results from the backend). There are a good amount of tests in the machinery though, so I'm confident we didn't regress on things outside of the C-family heuristics that were changed here.


Update CMList.py to include python enviorment

Calls for the Python enviorment so it can import libraries

Thanks, I had no idea this was important!


Timer threads consume too much CPU

Estimation is ~0.05% per timer

100 threads cap
1 blank plugin
ps: 0.2-0.3%
With a "standard" set of plugins and INI taken from the testing site, now ps: 0.2%.


Wrong font size in UGUI Dropdown

Describe the bug
The font size in UGUI Dropdown in the HandInteractionExamples scene is wrong when MRTK is consumed via UPM.
To reproduce

Open HandInteractionExamples.unity
Press play
Look at the UGUI Dropdown on the right hand side
See error

Expected behavior
The UGUI dropdown should have the correct font size.
Screenshots
When consumed via UPM

When consumed otherwise

Your setup

Unity Version 2019.4.8f1
Latest MRTK UPM package

I think that we need to understand what the delta is here for what's going on - I think that this is a weirdness that maybe would be okay to go with, but since it's in a new area that we want to beef up support for (UPM) and since this is something that should ideally just work correctly in both .unitypackage and UPM, let's get to the bottom of what's going on here.

@vaoliva I know you've done some investigations into our UnityUI areas before - can you take a peek at this here?

Issue is actually related to TMPro package version. Unity 2019.4 has TMPRo 2.1.1 as default package.
Issue can also be replicated with .unitypackages or from stabilization branch if TMPRo 2.1.1 is the installed package. Getting feedback from Build Call & QA channel to choose approach.

I also see a problem with the sample text paragraphs in the right column. (overlapping issue)
It seems like the character spacing is different on the title and makes it wrapped.

I'm seeing overlapping text in the Standard Shader Gallery Scene as well.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Installer: test the mnemonic generated

We could have a small test to ensure they wrote down their mnemonic. (Maybe with an easter eggs to not add a burden to manual testing?)

currently I think it displays the mnemo even if not used (maybe i'm wrong)

You are.

at least, i'd put a SKIP button if we have a test.

That defeats the whole point. It's even preferable to keep it like that in this case (which i'm not against).

[Have the check only on mainnet]

Hmm that's actually a reasonable middle ground. Note to anyone who may want to tackle this issue: make the test only be present on mainnet.

more to make sure the user did the backup than to check if they did so correctly

I'd say it's the opposite. I haven't thought it through but i was under the impression those checks were to make sure didn't confuse the order of the words.


[Issue-3197] Handle lack of data response

🚀 Deployed on https://pr-3200--sw-web-runner.netlify.app


Fewer clicks to add and remove keywords

When adding a subject keyword on a resource landing page it seems unnecessary to have to click add to add the keyword to the keyword list AND click save changes for this to take effect.  Clicking "Add" should be sufficient.  Similarly removing a keyword could be simplified to one click (with perhaps an undo to fix mistake) rather than having to "Save changes".  These are just part of an overall awkwardness in editing content that requires more clicks than seem necessary and is not natural.  This is probably low priority, but if there is an opportunity to simplify this editing we should.

@dtarb Can you verify and close this issue if it works now?

This is fixed.


Tool used Tag option

Description
The website can have tags while choosing projects (for example js, ruby, python) Categorizing projects under these tags make it easy for developers to checkout as per their skill set.

Hi @jeevikasirwani we already have tags for project. Did you check in projects tab? there we are showing tags in chips. Let me know if you mean anything else.

Maybe something similar to this, the tags on the left side of website.

@jeevikasirwani yup it's filter. we have this in our bucket list.

Thanks for adding this issue

@jeevikasirwani yup it's filter. we have this in our bucket list.

okay great!

Hey, I am interested in working on this project.
Can you please explain how I can get started?

Hey @raazseth could you please join our discord server? we can help you out in getting started


Memoized function not re-executing when input object has changed since last execution

Hey, don't know if this is the proper way to bring this up, but I was using this library to memoize data filtering functions before render in React, and for most purposes, this library has worked great.
I had 3 inputs to the memoized function: an object, an array, and a string. When the array and string would change, the memoized function would work as expected (that is, it would re-execute). However, when an attribute nested several levels deep in the object would change, the memoized function would return the cached result rather than re-execute (and from what I understand, this would imply that the under-the-hood implementation may have an issue doing deep object comparison on the inputs, which is already a pain in the ass in Javascript)
For anyone experiencing a similar issue, the workaround I used was to JSON.stringify the input object and use that as an additional memoized function argument. Since the string representation changes when the object changes, the library understands to re-execute the memoized function

I think it is caused by lastArgs assignment, it is just a simple assign from arguments, therefore, if you use a share object for argument, the lastArgs cache will be changed outer the closure. To avoid this, maybe we can make a deep clone object for call or just as @bjs250 said, use stringify.

By default, we do a simple equality check on each argument ===. For objects, this checks that the reference to the object is the same, not the underlying values. This is a shallow equality check. If you need to do deep equality checking then please use the custom equality function


Some forms and menus have no standard hotkeys: Alt +

Description
For automation and users who likes to work with keyboard, it is usually useful to add  Alt + <key> hotkeys for all menu items and buttons of the most often used forms.
I mean these hotkeys with underscored symbol which appears on pressing and releasing Alt key.
Here in this window the only Close-button has Alt + C hotkey.
I suggest to add hotkeys for all these buttons as often used.
https://github.com/user-attachments/assets/905b8ae4-ce70-4ed4-aa04-6242cbd6a23f
Menu items should also have these hotkeys.
Notice the presense of underscore symbol in menu item names:
https://github.com/user-attachments/assets/e1e3e579-469a-4088-8986-83c34703f2e7
DBeaver Version
Community Edition <IP_ADDRESS>407122322
Operating System
Windows 10
Steps to reproduce

pressing and releasing Alt key
see items without underscored symbol in the name of menu and buttons

Additional context
Please also check all other forms and menu.
Remember that ideally the chosen keys should be unique on the same nested level of menu items.

Don't forget this menu, its items are usually often used:

Thank you for the idea!

Suggestion about this issue.
Please try to choose different keys for  and <DELETE ...> commands.
Or maybe even any standalone hotkey for DDL (not alt + ), because it is used frequently.