How to use findStrings flat API

I attempted to findStrings in the Python interpreter:
>>> findStrings(None, 5, 1, True, False)

And all my results come back with a state of state=NOT_DEFINED.  Additionally, some of the results point to instructions and not string data.
I expected this method to act just like the Search >> For Strings... menu option, but it doesn't.  Am I missing something here?

I was able to reproduce the problem.  I'll debug it a bit and let you know.

The findStrings() method in FlatProgramAPI is currently implemented the same way the StringAnalyzer is implemented.  The Search >> For Strings... feature uses the CombinedStringSearcher which does extra things like return already defined strings.  The API should probably either use the CombinedStringSearcher (which unfortunately currently references GUI-only things) or the StringSearcher that the analyzer/API uses should be modified to give more consistent results (at least making it aware of defined strings).

Thanks @ryanmkurtz for your help and quick reply.  Creating a StringSearcher object in Jython is pretty straight forward, but I'm a bit stumped on how to make a call back.
I took a look at the findStrings() code, and tried to implement it in Jython:
def callback(s):
    l = list()
    yield l.append(s)
ss = StringSearcher(currentProgram, 5, 1, False, True)
ss.search(None, callback, True, monitor)

But it only returns the address set.  Any ideas on how to implement the call back?  For those playing at home, using a print(s) instead of the yield line will print the repr of the FoundString object.
Thanks again.


self.new method not work on subclasses if refefined initialize method

Here is error:
class A
  def initialize(@a : Int32)
  end

  def self.new(s : String)
    new(s.to_i)
  end
end

class B < A
  def initialize(a : Int32)
    @a = a + 100
  end
end

B.new("1")

 16 | B.new("1")
        ^--
Error: no overload matches 'B.new' with type String

But this works:
class A
  def initialize(@a : Int32)
  end

  def self.new(s : String)
    new(s.to_i)
  end
end

class B < A
end

B.new("1")

This is not a bug, but expected and documented behaviour: https://crystal-lang.org/reference/syntax_and_semantics/inheritance.html

If a class defines a new or initialize then its superclass constructors are not inherited

I understand this may be surprising. And I'm not sure this is the best solution. But that's how the language works. It can be changed, though, of course.
Related current discussion: https://forum.crystal-lang.org/t/constructor-variants-and-inheritance/3310


3 information

Hello sir can you can provide these 3 information's
RESULT
CONCLUSION
PROPOSAL / FUTURE WORK

Thanks !


realizable entity "missing" in some outputs in patient status module

Mark said it may be a bug

this is fixed in new model


Look into Airflow client generation from OpenAPI spec

E.g. https://github.com/oxidecomputer/progenitor

Some endpoints have query params that have as type Vec<String>. Progenitor does not seem to support parsing the necessary information from the OpenAPI spec on how to serialize these query params to a query string: https://github.com/oxidecomputer/progenitor/issues/692


Forbid the change of podSets in Workload

Signed-off-by: Jian Zeng<EMAIL_ADDRESS>
What type of PR is this?
/kind feature

What this PR does / why we need it:
Which issue(s) this PR fixes:

Ref https://github.com/kubernetes-sigs/kueue/issues/171#issuecomment-1121699517
Special notes for your reviewer:

/assign @ahg-g

friendly ping

@ahg-g Sorry for the late response, I have been busy these days 😢 . I would get back to my PRs at this weekend.

yes, sorry I didn't get notified that the comments were addressed

thanks, please squash

@ahg-g Hi! I have squashed the commits.

thanks @knight42
/lgtm


FailedCreatePodSandBox when scheduling pods

Hi,
I'm getting errors from pods on nodes that were recently inserted into kubernetes.
Logs error:
Normal SandboxChanged  9m30s (x183 over 6h15m)  kubelet  Pod sandbox changed, it will be killed and re-created.
Warning  FailedCreatePodSandBox  2m1s kubelet  Failed to create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "5d8c43f3b8179b87e01a6740a92bf485fe95cb16ca310e0c853ef731197f302b" network for pod "restic-2jsjf": networkPlugin cni failed to set up pod "restic-2jsjf_psc-velero" network: error getting ClusterInformation: Get "https://[<IP_ADDRESS>]:443/apis/crd.projectcalico.org/v1/clusterinformations/default": dial tcp <IP_ADDRESS>:443: i/o timeout

Calico version: v3.16.5
Orchestrator version: Kubernetes: v1.19.10
Operating System and version: SUSE Linux Enterprise Server 15 SP3

Calico version: v3.16.5

This is a very old version of Calico / Kubernetes. Could you please try upgrading and letting us know if you still see this?
v3.16 is no longer in support, so I'm going to close this, but please re-open if you are able to reproduce this on a modern version of Calico.


Chatterino window always resets

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
I have 2 monitors and I always use chatterino on the left as my main monitor is the one i use in front of me while the 2nd one is on the left, the problem is there are many programs games etc. that when open fullscreen cannot be minimized to show my desktop without pressing Windows + D or the line on the taskbar, so when that happens everything minimizes, which is ok but the problem is when chatterino is maximized back it goes back to the main monitor, it simply won't remember which monitor it was on or the position on the screen it was on
Screenshots
No response
OS and Chatterino Version
Chatterino 2.3.4 commit 4e5170799 on Windows 10 21H1 Build 19043.1466

While this is a different issue, could you try steps outlined in this comment: https://github.com/Chatterino/chatterino2/issues/2205#issuecomment-868001183 ?

Hello I tried the fix including changing it for all users and then restart chatterino but it did not work

https://superuser.com/questions/738081/how-to-make-applications-open-on-the-correct-monitor-when-using-multiple-monitor hope this helps


BreezeObservationAdapter.handles Property tweak?

Why check both entityAspect and entityType? Why not assume that if an object has an entityType, it must be a breeze entity. Who else is going to define that property for an object in an application that uses Breeze?
Could replace this:
// breeze entities have entityAspect and entityType properties.
if (!object.entityAspect || !(entityType = object.entityType))
  return false;

// get or create the lookup used to avoid reflecting on the breeze entityType multiple times.
canObserve = entityType.__canObserve__ || createCanObserveLookup(entityType);

// return canObserve- coerce undefined values to false.
return !!canObserve[propertyName];

With this:
var type = object.entityType
return type ? !!(type.__canObserve__ || createCanObserveLookup(type))[propertyName] : false;

Agree- overkill with the two checks.  Will make these changes.


[Security] Workflow size.yml is using vulnerable action actions/checkout

The workflow size.yml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Fixed, thanks!


Update install-zos.md

I've added a description of how to use the supplied zowe-config-stc.sh script.

Changed the base branch to docs-staging which hosts the 1.0 content.


add namespace + CPM compliant

This allows developers to include your project by simply adding the following to their CMakeLists.txt:
CPMAddPackage(
  NAME progresscpp
  GITHUB_REPOSITORY prakhar1989/progress-cpp
  GIT_TAG origin/master
)
target_link_libraries(App progresscpp)

And use the project like so, in their c++ files:
#include "progresscpp/ProgressBar.hpp"
...
progresscpp::ProgressBar progressBar(total, 70, '#', '-');
...

The functionality is the same.
Based off of this: https://github.com/TheLartians/ModernCppStarter
Uses CPM: https://github.com/TheLartians/CPM.cmake
affects #2

Thank you!


Class "Opcodes\LogViewer\Level" not found when upgrading from v2 to v3.

I encountered an issue when attempting to upgrade from version 2 to version 3 in my Laravel project. The problem is that the package throws an error related to the class "Opcodes\LogViewer\Level" which is not found after the upgrade.

Environment

PHP Version: 8.2
Laravel Version: 10.x
Operating System: Linux (Ubuntu)

I have followed the upgrade instructions provided by the package documentation
Please is there a way to resolve this issue?

hey @sefakor20 , please see the upgrade guide and remove the unused 'patterns' option from your config/log-viewer.php.
With that, also remove any Opcodes\LogViewer\Level namespace imports as that is no longer used.
Hope this helps! :)

Greetings @arukompas,
Thank you so much for your help in resolving this issue! I appreciate your quick response and effort to fix the problem.
The upgrade to v3 of the package is now working perfectly in my project.


New Namibia phone numbers not validating

This phone number is confirmed valid for accepting phone calls: +26463683160
Libphone says that it is a possible number but not a valid number.
The business that uses it is the Sandfontein Lodge located in Karasburg 9000, Namibia. http://www.sandfontein.com/
How can we go about ensuring that this number and others similarly formatted can be validated in libphone?

Thanks for the report. We will fix it in the next release.

Fixed in release 7.0.11.


Allow filtered access to /events

This PR adds filtered access to /events.   This is part of work I am doing to get buildkite-agent building our stuff properly in a Kubernetes environment.

Awesome! Mind ticking the box in the README for this route?

done :)

Thanks for the contribution @chrissnell!


Profile not working for great britain

Hi!
Thanks for thoses greats profiles! It really helped me a lot!
They work very fine for all continental europe, but when it comes to great britain, it seems very unstable.
For example, if I generate my osrm map with this profile (freight.lua), from the european-latest.osm.pbf (That I've filtered with osmium nw/railway), osrm is unable to compute some basics route.
Example, a route between : [-0.160675,51.187951],[-0.168915,51.220432]
(It doesn't go on the good coordinates, the road begin very far from it)
I tried to change the restriction, but nothing has effect on it.
Would you have an idea ?

Hi kushia !
I never tried the profile in the UK. At the moment, the profile filters out gauges that are not 1435 mm, could it be that the zone you are working on is using another gauge ?
If not, could you take a look at the debug map ? It could help to see if it's a filtering or connectivity problem.
You could also try with the basic.lua profile to see if it changes anything.

Hi C4ptainCrunch!
I made a lot of tests, and it appears that it was because of the variable "data.maxspeed".
It seems than for the ways in UK, data.maxspeed returns sometimes a string like this "X mph" instead of an integer.
So, the script was failing for the way who was evaluating, and the way was then ignored. (osrn-extract doesn't throw an error for those cases and then pass to the next way to evaluate)
You can correct it like that :
speed = tostring(speed)
speed = speed:gsub(" mph", "")
speed = speed:gsub("mph", "")
speed = tonumber (speed)
Before :
result.forward_speed = speed
result.backward_speed = speed

I tested the proposed patch and it works well for UK and the rest of Europe.


Cinder no vg default

Disable in role but enable in CI defaults

retest

This needs to be fixed on master too.


Reduce Math object binary size.

Roughly one KByte saving: 141024 -> 140256

Fixed all requests.

Thank you for the review. Issues were fixed.


[ALLUXIO-2370] rename 'Under Storage' to 'the under storage' in docs

https://alluxio.atlassian.net/browse/ALLUXIO-2370
@calvinjia

alluxio-bot, check this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12237/
Test PASSed.

Merged build finished. Test PASSed.

LGTM


Add readme file

Add a readme file that includes the following:

Application concept and description
Onboarding - Running the application locally
Running the application via Docker

I want to contribute to this.

Can you assign me


Add stackbit.config.ts

👷 Deploy Preview for blog-nsi-890809 processing.

Name
Link

🔨 Latest commit
7ba620fe0d7fd1a62bcdc328a24b238e4e0c1c57

🔍 Latest deploy log
https://app.netlify.com/sites/blog-nsi-890809/deploys/66b5c30f9800d7000890cb96


feat: use anyhow

Closes #24
Change the hand rolled Box<dyn Error> approach for error handling to simply using anyhow.
This is a breaking change if someone uses errors. However, the migration should be fairly straightforward:

Change ffmpeg_sidecar::error::Result to anyhow::Result
Change ffmpeg_sidecar::error::Error to anyhow::Error

If someone is using Error directly (i.e., getting the message or the source) then they should consult the anyhow docs to find the equivalent functionality for their use case.

Great job with this PR. CI passes, all examples work, and the code changes are minimal.
I also appreciate how utilities in anyhow make the source code cleaner, things like anyhow::bail!(...) instead of return Err(Error::msg(...)), and .context(...) instead of .ok_or_else(|| ...).
I will merge this and plan to publish a release on Cargo next week. Since this is technically a (small) breaking change, I'll bump the major version number. Coincidentally, next week will be one year since the first released version, v0.1 in Feb 2023.


Weird annoying fontspec BUG

Description
If you use Graphite Renderer and arabic script, then fontspec complain that the selcted font does not contain arabic script and the text is not rendered correctly.
For the purpose of this bug report, you can test using AWAMI NASTALIQ font.
This issue does not happen in Plain TeX or when using \font control sequence directly in LaTeX. See my second minimal example.
The Script=Arabic is needed since XeTeX does some unicode bidi; without this, parentheses, etc will be wrong.
Check/indicate

[x] Relevant for XeTeX
[ ] Relevant for LuaTeX
[ ] Issue tracker has been searched for similar issues?
[ ] Links to <tex.stackexchange.com> discussion if appropriate

Minimal example demonstrating the issue
\documentclass{article}
\usepackage{fontspec}
\setmainfont{AwamiNastaliq-Regular}[Path,Renderer=Graphite,Script=Arabic]
\begin{document}
وفا
\end{document}

and here is the plain tex one.
\font\test="[AwamiNastaliq-Regular]/GR:script=arab" at 20pt
\test
وفا
\bye

Further details

Well the documentation of fontspec writes

Currently fontspec does not support a convenient interface to select Graphite font features
and all selection must be done via ‘raw’ font feature selection.

so use
  \setmainfont{AwamiNastaliq-Regular}[Path,Renderer=Graphite,RawFeature={script=arab;}]

@wspr Please have a look at this.

@niruvt  — thanks for making the connection

@wspr Thanks a lot for the fix!


Tribute Page Project...(ul issue)

I'm fairly new to coding and am currently doing a  Tribute Page Project. I'm just having issues centering my text along the bullet points. I CANNOT USE CSS to change any of my html so any help would be appreciated, Thanks! Here's the code.........

Dr. Norman Borlaug
A man who saved a billion lives

Dr. Norman Borlaug, third from left, trains biologists in Mexico on how to increase wheat yields - part of his life-long war on hunger.

Here's a time line of Dr. Blorlaug's life:
1914-Born in Cresco, Iowa

1933-Leaves his family's farm to attend the University of Minnesota, thanks to a Depression era program known as the "National Youth Administration"
1935-Has to stop school and save up more money. Works in the Civilian Conservation Corps, helping starving Americans. "I saw how food changed them", he said. "All of this left scars on me."
1937-Finishes university and takes a job in the US Forestry Service
1938-Marries wife of 69 years Margret Gibson. Gets laid off due to budget cuts. Inspired by Elvin Charles Stakman, he returns to school study under Stakman, who teaches him about breeding pest-resistent plants.
1941-Tries to enroll in the military after the Pearl Harbor attack, but is rejected. Instead, the military asked his lab to work on waterproof glue, DDT to control malaria, disenfectants, and other applied science.
1942-Receives a Ph.D. in Genetics and Plant Pathology
1944-Rejects a 100% salary increase from Dupont, leaves behind his pregnant wife, and flies to Mexico to head a new plant pathology program. Over the next 16 years, his team breeds 6,000 different strains of disease resistent wheat - including different varieties for each major climate on Earth.
1945-Discovers a way to grown wheat twice each season, doubling wheat yields
1953-crosses a short, sturdy dwarf breed of wheat with a high-yeidling American breed, creating a strain that responds well to fertalizer. It goes on to provide 95% of Mexico's wheat.
1962-Visits Delhi and brings his high-yielding strains of wheat to the Indian subcontinent in time to help mitigate mass starvation due to a rapidly expanding population
1970-receives the Nobel Peace Prize
1983-helps seven African countries dramatically increase their maize and sorghum yields
1984-becomes a distinguished professor at Texas A&M University
2005- states "we will have to double the world food supply by 2050." Argues that genetically modified crops are the only way we can meet the demand, as we run out of arable land. Says that GM crops are not inherently dangerous because "we've been genetically modifying plants and animals for a long time. Long before we called it science, people were selecting the best breeds."
2009-dies at the age of 95.

"Borlaug's life and achievment are testimony to the far-reaching contribution that one man's towering intellect, persistence and scientific vision can make to human peace and progress."
-Indian Prime Minister Manmohan Singh
If you have time, you should read more about this incredibly amazing human being on his Wikipedia entry

    Written and coded by DesiWeb

Hi @dsteib11 looking at https://github.com/dsteib11/Tribute-Page I don't see any Express.js code anywhere to help with, and it doesn't sound like you're describing an Express.js issue. Are you sure you opened this in the right place? Express.js is a server-side JavaScript framework.

Sorry about that


Change HashDiff Constant to Hashdiff

Why This Change Is Necessary
The name of this gem is hashdiff. There is another gem named
hash_diff. When they are both required via dependencies, errors are
thrown. Additionally since the behavior is different, gems that are
relying on hashdiff's behavior, may get hash_diff's behavior and
vice versa.
The reason for the conflict is because, based on ruby conventions, _'s
are the separators for camel case.
So:
'hashdiff'.camelize # => "Hashdiff"

'hash_diff'.camelize # => "HashDiff"

Unfortunately instead of making the base module of this gem Hashdiff,
it's called HashDiff and it conflicts.
How These Changes Address the Issue
We change all of the constants to the new constant, including in
documentation and tests.
Then we add a shim to allow the old constant to be used and therefore
maintain backward compatibility.
Finally we add a warning message for users of the gem that the old
constant will be removed in the next major version bump.
Side Effects Caused By This Change
A new warning is added when loading the gem.

References:

#45

Test failures seem to be Rubocop issues that existed before this PR.

Thanks a lot @jfelchner . WDYT @krzysiek1507 ?

@liufengyun just remember this needs to be version 0.4.0.

Thank a lot @jfelchner 🎉
And thanks for reviewing @krzysiek1507 @luke-hill !


ccmlib/cluster.py: Workaroud adding multiple nodes in parallel

Adding multiple nodes to the cluster in parallel is not supported by
Scylla, at least for now. Scylla is becoming more strict[1] to check if
there are other pending node operations when a node bootstraps. It fails
the bootstrap if there are pending node operations. To workaround the
existing dtests that adding multiple in parallel, add delay to allow the
second bootstrap node to detect the first boostrap is in process and
wait until the first node has finished. Starting nodes at the exact same
time makes it very hard to detect other pending operations in the
cluster.
[1] Switch to use NODE_OPS_CMD for decommission and bootstrap operation #8481
https://github.com/scylladb/scylla/pull/8481

Issue not found with latest ccmlib and dtest. Closing it.


Updates purgeCacheByRelevantURLs filter to allow arrays

Closes #488
This PR should fix the issue where the cache for a URL won't get purged if it's defined as an array, for example if there are headers .

Hey @jacobbednarz 👋  Any chance of getting a re-review on this? We're having to manually flush cache each time a blog post is published, which thankfully isn't too frequent, but annoying, nonetheless. If it'll be a little while longer, no worries, we can plan in a little work on our end to work around the issue

apologies for the delay; i've been occupied with other work and thinking about this.
i'm thinking this implementation is too specific for the plugin to cater for. within this we create some assumptions (such as the array key being "url") and only checking a single level of depth. which, to me, seems like we shouldn't be doing this in the first place and whatever is calling the hook needs to instead just pass on only the individual URL? i understand you have a pretty bespoke setup but i'm not sure if that should be supported here for all users. the whole purpose of hooks is that the plugin provides an entrypoint to do a thing to avoid bespoke implementations leaking into the plugin itself.
right now, i'm hesitant to accept this as i can't see a use outside of your specific case which i wouldn't want to add (and maintain). am i missing or not quite understanding anything here?

I guess one piece of info missing up to now is where this array format is coming from. It matches the Cloudflare API for purging files by URL, so it would be invalid without a url field, and doesn't support nesting - https://api.cloudflare.com/#zone-purge-files-by-url
Perhaps we made an incorrect assumption that this hook would allow us to use this object format and to be fair, the documentation doesn't say this is supported. It worked for us though until this filter was implemented, since at the end it just gets converted to JSON and POSTed to the endpoint. It affects anyone who wants to purge another URL that has a custom cache key as outlined in the above docs.

gotcha, i see what you mean now. this seems to have only been working by accident as this purge process never had the intention of supporting values outside of the array (despite it being supported in the API). however, in the case where we have URLs and HTTP header for a custom cache key, that seems like we should be supporting it and this PR is no longer about bespoke functionality and actually about allowing full use of the API available.
you've convinced me this should be added, thanks! i'll run this through a few test cases and look to merge.

Thanks @jacobbednarz! Much appreciated 🙂

Hey again @jacobbednarz, sorry to bother you. Any idea when this might get released so we can get it into our WP without having to pull it from master?


MD-1560: Verify DataBase MetaData through JDBC API

This PR exercise all JDBC API related to DB MetaData. @kkhatua please review.
http://<IP_ADDRESS>:8080/job/Framework-Regression/1142/console

LGTM +1

@cchang738

Can you please add a line before dumping the DatabaseMetada information to the log ?
something like, "DatabaseMetadata information"
Please change/update the title for the PR, because here we are not verifying the results, we are only dumping the information to the log.

Thanks,
Khurram


Update verb_database.rb

Updated the pronouns like this:
'Nos' to 'Nós'
'Vos' to 'Vos'
'Ele' to 'Ele/Ela'
'Eles' to 'Eles/Elas'

Thanks! :+1:


Prevent map from zooming out so far as to repeat or show blank space north/south of the poles

Prevent the touch screen map from zooming out so far as to repeat from east to west, or from showing blank space beyond 90 degrees north or south.  Tactile already does this but the standard Maps implementation that we use in Peruse does not do so.  Research ways to prevent too much of a zoom out.

Yes, I know this issue has a long title/subject.


Timeslot datatype change

timeslot start and end dates now have the TimeDate datatype
seed file has been updated again
@clarroquia I kept your time slot and booking seed commented out for now

👍


PHOENIX-6682 Jenkins tests are failing for Java <IP_ADDRESS>

pin Java11 to the Ubuntu 20.04 release version

Last minute testing shows that apt-get also needs --allow-downgrades, otherwise the downgrade does not happen.
Going to commit with that change.


Fixed local data paths to target AnyCPU

@katydalton Please review and merge if ok.

Changed local path directories to target AnyCPU paths ..\..\..\..\ to ..\..\..\
Also fixed one of the urls in create featurelayers sample (code that is not really used). Do you think that the code is ok to be there or should I remove it totally. FeatureLayers are created in XAML and the code in the code behind is commented out but used to show how to do it there.

Looks good. I see what you mean about the Create FeatureLayer from Service sample. I'd say we leave it in, but maybe I'll update the comment to clarify that we're creating the feature layer in Xaml but you could also do it in code-behind?

Thanks. One thing to look when we go to through the samples is how much we have this kind of stuff.


Application Configuration no longer accepts names with a colon

Describe the bug
Adding an application setting with a colon (:) in the name now throws an error in the Applications Configuration page.
To Reproduce
Steps to reproduce the behaviour:

Go to your app Configuration
Add a setting with the name This:MakesNoSense
Set a value of Test
Press OK
Press Save

Expected behaviour
The settings save, as there are a number of .NET libraries with settings that follow this format
Screenshots

Device info (if applicable):
Not relevant
Additional context
Not relevant

Managed to get around this by using double _ i.e. This__MakesNoSense. I feel like this change should be documented somewhere, given that using colons has been working fine until today.

@craigshoemaker Could we specify in the docs that we only support alphanumeric characters, '.', and '_' for app settings? @gavinwilliams I believe we've had this rule in place for a while now. Do you have the last time you made a successful change with a ':'?

@craigshoemaker Could we specify in the docs that we only support alphanumeric characters, '.', and '_' for app settings? @gavinwilliams I believe we've had this rule in place for a while now. Do you have the last time you made a successful change with a ':'?

A little over a month ago

The Application settings doc is now updated with the character allow list.

Suggesting that this issue can be closed now that the documentation article has been updated 🙂 @gavinwilliams @mkarmark


Link PERCENTILE from MEDIAN docstring

🚀 Pull Request
Description

... since PERCENTILE has lazy aggregation.
Closes #4039.  Specifically https://github.com/SciTools/iris/issues/4039#issuecomment-1324871960

let's wait till 2023-01 if anyone really wants MEDIAN to be lazy.
IF not, add a doc note to say "it's not lazy, but percentile is"

Consult Iris pull request check list

😍

Oh nice, I hadn't even noticed that bit!
https://scitools-iris--5128.org.readthedocs.build/en/5128/generated/api/iris/analysis.html#iris.analysis.MEDIAN


Handle hasura issues

Currently, if Hasura isn't running, we get a 500 response with Unexpected token S at position 0. This is a bit crap, frankly. It should really deal with this better, ideally:

Give the user a friendly error message to tell them that the site is experiencing some issues
Automatically email the webmaster to let them know (maybe not every time), we possibly want some way of doing this in a more structured way, but probably have better things to do than wire up some monitoring stuff

The signup form also doesn't give any error information.

The signup form also doesn't give any error information.


chore: log matches

Codecov Report

Merging #110 into main will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
+ Coverage   73.17%   73.30%   +0.13%
==========================================
  Files           6        6
  Lines         205      206       +1
  Branches       34       34
==========================================
+ Hits          150      151       +1
  Misses         54       54
  Partials        1        1

Impacted Files
Coverage Δ

src/annotate.ts
95.23% <100.00%> (+0.11%)
:arrow_up:

src/solidarity.ts
55.17% <100.00%> (ø)

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 ffa8ff4...c58ce04. Read the comment docs.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add PanDirectionGestureRecognizer for Panning on only axis we need (#28)

Since there is a bug in our app, about gesture recognizer races,
we have to add PanDirectionGestureRecognizer for panning on only axis we need.

@kanottonp Should we cancel when direction is horizontal but from right to left?

Related to #28


Windows cats sep

[ ] Tests added / passed
[ ] Passes flake8 dask
[ ] Fully documented, including docs/source/changelog.rst for all changes
and one of the docs/source/*-api.rst files for new API

@mrocklin , are parquet tests run on Windows?

It looks like no?
mrocklin@carbon:~/workspace/dask/continuous_integration/appveyor$ git grep fastpaqrquet
mrocklin@carbon:~/workspace/dask/continuous_integration/appveyor$


remote_activity: Add 'guest users' and 'non guest users' count column.

This PR adds two columns named 'Guest users' and 'Non guest users' to represent count of such users.
We query 'RemoteRealmAuditLog' to get the data.
Steps for manual testing

Uncomment the bottom two lines of zproject/dev_settings.py to use localhost as a push notifications bouncer.
start server: ./tools/run-dev
Run manage.py register_server and register your server (as the local server) with itself (as the bouncer).
Restart server
In update_remote_realm_data_for_server add ignore_conflicts=True

    try:
        RemoteRealm.objects.bulk_create(new_remote_realms, ignore_conflicts=True)

Reason: This is an existing bug in main. So, I'd fix that separately.

In update_analytics_counts.py change (60*10) to (5) -- reduced delay to 5 second
Run management command manage.py update_analytics_counts
visit /activity/remote (login as iago -- is_staff=True needed)

Screenshots and screen captures:
Remote Servers Data

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Pushed tweaks to comments, moved the code to zilencer/models.py for easier reuse.


Emojis don't render properly on all terminals

Emojis cannot be rendered on every terminal. This causes the issue that log messages will look strange and don't have a common alignment.
You can see the issue by running my demo code in different terminals:

NODE_DEBUG=* node index.js

const logdown = require('logdown')
const logger = logdown('MyLogger')

logger.debug('Type "debug"')
logger.error('Type "error"')
logger.info('Type "info"')
logger.log('Type "log"')
logger.warn('Type "warn"')

Because the output isn't consistent across platforms, we should replace emojis with standard ASCII symbols (which can be rendered on most of the systems).
I also noticed that logger.debug is not printed on the console. How can I get debug messages? I already used NODE_DEBUG=*.
Command-line interface (Windows)

PowerShell (Windows)

Ubuntu Bash (Linux Subsystem on Windows 10)

Another good solution would be to make the emojis configurable, so that the end user can decide if they should be printed to the console or not.
Proposal:
var logger = logdown({ icons: false })

WDYT @SleepWalker & @caiogondim ?

Configurable it's good! But is should be enabled by default to keep
backward compatibility.

What would be better?
Completely disabling the icons or configuring them individually?
Or maybe both?
const logger1 = logdown({ icons: false })
const logger2 = logdown({ icons: {
  warning: '!',
  error: '!!'
} })

What do you think?

The easiest to implement (and already useful) config would be turning it on or off completely.

I'm cool with that


Force camera mode

Force 1st person or 3rd person view in a particular region of a scene
This could be achieved by defining a new type of Avatar Modifier area
https://docs.decentraland.org/development-guide/avatar-modifiers/
The player's prior camera mode should be remembered, so as to revert back when the player leaves the area. This should also be the case if the player teleports away from the area, instead of stepping out of the area.

should this area disable the input for the player to freely change the camera mode?
if that so, should we add a new scene permission for this?

renderer: https://github.com/decentraland/unity-renderer/pull/1725
sdk: https://github.com/decentraland/js-sdk-toolchain/pull/75


[BUG] - Dropdown does not work when backdrop is setted on mobile

NextUI Version
@nextui-org/dropdown": "^2.1.29
Describe the bug
I have a Dropdown that opens a modal (which by default has an opaque backdrop). When the Modal opens and I close it, the Dropdown Trigger that initially opened the modal stops working unless I click it multiple times quickly to try to reopen it. This issue only affects the specific Dropdown Trigger that opened the Modal, even if the Modal is controlled outside the dropdown. The problem only occurs on mobile, and I have tested it on my phone as well.
IMPORTANT NOTE: After recording the old video (now is the updated one), I noticed that even without the Modal (just the Dropdown with the backdrop set), it still wouldn’t work. My Dropdown Trigger is a Card, but I also tested it with a Button and text (just the trigger itself).
As a workaround, I removed the backdrop from the Dropdown, and now it seems to work fine on mobile devices. However, this is still a bug.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Trigger dropdown with backdrop
Open modal (not mandatory) or leave the dropdown (mandatory)
Close modal (not mandatory)
Trigger dropdown (bug)

Expected behavior
Open dropdown
Screenshots or Videos
https://github.com/user-attachments/assets/1443b0a7-dc5b-4277-8032-c4ac13ae235f
Operating System Version
Android, Windows
Browser
Chrome

please share a minimal reproducible environment.

same is happening for me on the nextui website. i can produce a minimal reproducible environment if you want me to? just seems to happen when the dropdown with a backdrop is used on any android devices. i've tested it myself and had lots of users report it on my website
https://github.com/user-attachments/assets/c3d4e4bb-e992-4d91-8398-5cf829a793ae


The rendered content via template_include action is not displayed after get_footer() function is called.

I use a rockettheme template called Gemini and also i use a plugin for rent cars and when somebody went to use the forms and put the submit button it must show a general resume form and it didn't show it.
It is a bug in the timber theme, we debugged it and everything is ok until wp_footer is called, there is a timber theme function which not render previously loaded content. The theme used is gantry5 framework so it is not easy to debug something else there. why the rendered content via template_include action is not displayed after get_footer() function is called.

can you answer please?

bump

Hey there
Sorry, there’s a lot of work going on in different areas of Timber. It would definitely help if you could provide more detailed instructions for how to reproduce the error. That’s also why there is an issue template with information that we’d like to know.
It would be good to see how that plugin for renting cars is hooking into the theme. Is it in the official plugin directory? Can you give a link to that? Or is it not specially related to that plugin? Do you have some example code that illustrates the problem?

It is defined as:
add_action('template_include', array('CarRental', 'carrental_summary'));
where is our template file included.

Hey @kataskeviwebsite
It’s really hard to reproduce the error with the information that you’ve given. If you can’t provide us with the steps to reproduce the error, I’m afraid we can’t help you.
Usually, the template_include hook is a filter and not and action. If it’s added as an action and doesn’t return a value, it may break your theme.

there is a timber theme function which not render previously loaded content

So what function is this? Is in the Gantry 5 theme? I’m afraid we can’t provide support for issues that are directly related to theme.
Does the error also occur if you use the Starter Theme?


www subdomain is in google but not accessible

Describe the bug
I searched google.com for site:railsdevs.io and results include www.railsdevs.io URLs, which are not currently accessible.
To Reproduce
Steps to reproduce the behavior:

Go to these results
Click on any of the links
See error

Expected behavior
Application should probably either a) redirect to non-www and/or b) load the page but have a canonical tag that points to the non-www URL.
Screenshots

The site is .com now, not .io. I'm not sure how to remove those results from google.

If you still own the old URL, could we set up some kind of proxy/redirect?

I'm not sure if you your using nginx, traefik, etc.
But you could probably set it up to catch both URL's instead of railsdev.com specifically.
Or just do redirect with the webserver

I have a redirect set up with my registrar but it doesn't work for www. I've debated moving it over to namecheap but don't want to pay for .io for another year if the site is already in .com.
I think the solution here is to not search on Google for an unsupported domain and use the .com.

I'm going to close this. I don't have appetite to try and fix these redirect issues with the old domain.


Separate and refactor integration tests

Currently, the integration tests are just a sub-folder of the main unit tests.  Need to separate those out so we can run integration tests separately.
Also, we need to use EF Core driver for the integration tests.  We can't be using the in-memory driver.

Implemented in #272

Moved the service to Datasync.Common.Test.Server.  The server now uses SQLite in-memory store for everything.
Created a separate Microsoft.Datasync.Integration.Test suite.
Moved all live server tests to the integration test suite.


Suggestion: expose_imports(coolmlproject)

If you call drake::expose_imports(coolmlproject) right after loading your package, then drake will keep track of the R functions in your package and automatically rebuild targets when their upstream function dependencies change nontrivially.

By default, drake tries not to watch package functions because that gets to be too brittle. Except for namespaced calls (i.e. pkg::fun()) drake only looks in your environment for functions. This itself comes with its own form of brittleness because interactive R sessions tend to get cluttered, and r_make() solves the problem by running make() and friends in a fresh new reproducible session.


References to removed entities

The templates' references to data, scripts, tags or xlsxTemplates can get broken if the referenced entity is removed. I don't plan right now to implement this integrity in the server but the studio part should easily deal with it.
Right now, if you delete the referenced data, you need to select another data in the template, save, select no data reference and save to get rid of the invalid reference.
The template should be automatically marked as dirty and remove the invalid reference when it is being loaded. This should be probably done at this place and also accordingly in other extensions
https://github.com/jsreport/jsreport-data/blob/master/studio/DataProperties.js

done


SSL settings

Please consider removing RC4 support etc.
https://www.ssllabs.com/ssltest/analyze.html?d=api.ipify.org&s=<IP_ADDRESS>&latest
See this fine tool:
https://mozilla.github.io/server-side-tls/ssl-config-generator/

And in couple of days your cert expires.

I'm currently working on migrating this project over to a new host (off Heroku), due to poor IPv6 support. When it is fully moved, I'll be passing the A+ ssllabs test =) Also: no worries about the cert -- I have auto-renewal / etc. configured.

OVH is a huge hosting company: https://www.ovh.ie/vps/vps-cloud.xml (only in Canda)
faster instances: https://www.ovh.ie/cloud/instances/cpu.xml

I'm currently evaluating digital ocean, but I'll check them out!

If you go so "low" as DO, then check out Vultr
See the speedtest results: https://github.com/szepeviktor/wordpress-speedtest

Will check it out! I guess you aren't a fan of DO? heh

It's execution time is 60% higher, see the repo above.


Error thrown when using the patch builder on a poco property which has attribute decoration

Describe the bug
On creating a POCO item, derived from Item, which requires additional attributes on the properties the update patch behaviour fails.
To Reproduce

Create a new POCO
Derive from Item
Add string property to POCO
Decorate with attribute eg. [Required]
Save a new instance to a container
Try to update the property, which has the attribute decoration, through the patch builder functionality

Expected behavior
The property is updated with the supplied information.
Actual behavior
Exception is thrown.
Environment summary
SDK Version: Latest source code
OS Version (e.g. Windows, Linux, MacOSX): Windows
Additional context
Pull Request #156 fixes the issue.

Marking as closed as PR has been merged


Some instructions are outdated

[Some of the steps are outdated. For example "search-mailbox" is deprecated. Deleting emails step should be updated with new threat explorer option. It is much better and accurate results to search and delete emails quickly ]

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

ID: daae9c8b-a9c6-01a7-1072-faaab97793bf
Version Independent ID: 08cb659e-3295-e2df-4d9b-e0f426e238e7
Content: Phishing investigation
Content Source: compass/incident-response-playbook-phishing.md
Product: m365-security
Technology: m365d
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Fix sys_fee calculation

What current issue(s) does this address, or what feature is it adding?
Fix https://github.com/CityOfZion/neo-python/issues/440
How did you solve this problem?
bring sys_fee calculation inline with C# (specifically IssueTransaction for #440)
Also looked into all other transaction types and brought them inline where applicable
How did you make sure your solution works?
run the tests from #440
Are there any special changes in the code that we should be aware of?
no
Please check the following, if applicable:

[ ] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[X] Did you add an entry to CHANGELOG.rst? (if not, please do)

I kind of saw that and was thinking, no tests failing so they're probably not used anymore. Anyway, lesson learned; don't rely on tests alone

don't merge, I noticed the bug is back with this last update. Will have to figure out why actually loading the sys_fee breaks it again. will do that later on sunday


Introduce empty_as_none and empty_as_none_vec deserializers

This commit changes the way foo= is deserialized into Option<_>. Previously it was deserialized as None, now it will be deserialized as Some(_), unless the new empty_as_none deserializer is used via #[serde(deserialize_with = "crate::de::empty_as_none")]. This allows users to decide on the deserialization strategy on a field-by-field basis.
Resolves #13

@jplatte I've kept this as a draft PR since it does not contain documentation yet, but I'm curious whether you think this is a direction worth continuing :)


⚠ Everything Hub has degraded performance

In a5df09a, Everything Hub (https://hub.everythingbagel.me) experienced degraded performance:

HTTP code: 200
Response time: 9539 ms

Resolved: Everything Hub performance has improved in 1f1f54c after 10 minutes.


Add type hints

Suggest adding type hints to the code to facilitate rapid development and reduce confusion over variable types (especially useful in method signatures).

This has been completed via Add type hints to all method signatures, fix minor issues related to …


Turbolinks 5

I remember when first tinkering with this project, I was super excited about the concept. Turbolinks 5 is basically dancing behind the curtain now just itching to be unveiled. Have you taken a look at it lately to see about re-writing turbo-react for the latest Turbolinks?

@glennfu I have not had the chance to investigate Turbolinks 5 yet. turbo-react relies on an ugly monkeypatch and knowledge of implementation details of Turbolinks. A major version bump likely means the core of turbo-react will need to be reconsidered.
Have you had the chance to look at it? Got any interesting feedback?

It's a combination of 1: a simpler approach to usage, where all that happens is you either visit a page as normal, or replace the page without a history change. That's like, everything. Then it comes down to the minor quirks of how to deal with corner cases that don't work immediately as you expect. Then the other side is 2: a crazy wrapper for iOS and Android to take the page transitions and make them feel like native app transitions, while giving you a hook into making JS calls to the native app so you can have your native code respond there for things like, adding a button in the nav bar.
Looking into where turbo-react might come into play, it looks like the core rendering methods are here: https://github.com/turbolinks/turbolinks/blob/master/src/turbolinks/snapshot_renderer.coffee#L50
So we have some parentNode.replaceChild calls and one document.body = @newBody. The trick will be, of course, not breaking any of the magic that Turbolinks is trying to quietly do for you. Monkey patching as a proof of concept should now be easier imo, and similarly a pull request for better hooks should be cleaner, too, once it can be proven that turbo-react is viable here.
I still think this project will be really valuable here. The ability to have css transitions between state changes with basically 0 effort is still really awesome.

Just pushed this, seems to work!
https://github.com/ssorallen/turbo-react/pull/29


Implement Argon2 S2K mechanism

This PR aims to implement the new Argon2 S2K mechanism for OpenPGP.
The PR further contains some updates from the crypto-refresh revision of the OpenPGP spec.
~This is still pretty much work in progress, so do not yet merge!~
I added factory methods for different S2K methods to the S2K class.
This should make creating S2K instances more straight forward, since the S2K method is no longer derived from argument ordering.
This also enables creating S2Ks using GNU_DUMMY_S2K methods.

Ready for review :) I can squash the commits if necessary.

Squashed where sensible.

Thanks for the patch! Merged.


Fix Mobile Safari issues.

Toolbox Spec doesn't load on iOS.

Inputs have a border-raidus value from user-agent style sheet on iOS.

@javivelasco I am not sure if I need to do the 'babel-polyfil' for building lib as well.

I think the polyfill might be needed indeed for building lib as well. Also we can change the implementation to skip unsupported APIs. What do you think?

Thank you!


New Testimonial for Madhu Saini

Name
MadhuSaini22
Title
Highly Promising Full-Stack MERN Developer
Description
Madhu Saini is an up-and-coming full-stack developer focusing on the MERN stack. Though I have yet to have the opportunity to work with her directly, I have heard great things about her skills and dedication to her craft. Madhu firmly understands the latest technologies and can deliver high-quality code. I look forward to the opportunity to work with Madhu in the future, and I'm confident that her skills and passion for development will make her an asset to any team.

I think the testimonial workflow not working or the rate limit was exceeded @loftwah

Is there anything wrong with the json?

The error message is
Invalid string: control characters from U+0000 through U+001F must be escaped

I think the newline is causing a problem

Cool I'll fix it soon

I removed the line break and the GitHub Action has re-run

Thanks Eddie I'm not sure how I managed to do that.

OMG, you made my day @loftwah ❤❤
Thank you so much for your kind words, It's my first testimonial and it's boom💥 I appreciate your words :)

Oops testimonial is by @eddiejaoude 😲😭
This is unbelievable for me❤❤❤❤
I haven't even thought of that Open Source God can add a testimonial for me!!
Tons of Thanks to you Eddie :)
Lots of love and respect for you!!

@MadhuSaini22 sorry it was from Dean, the GitHub took my username because I edited the issue


[Feature Request] Can MISE log which token decryption cert is used when there are multiple available certificates?

Is your feature request related to a problem? Please describe.
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/2d18df94bad8de860067cfe0a6bab939ba683043/src/Microsoft.IdentityModel.JsonWebTokens/JsonWebTokenHandler.cs#L1026
Currently, when we pass multiple token decryption certs to MISE, even the token can be successfully decrypted, but we have difficulties to know which cert is actually used. Knowing the current active decryption cert can help us track the cert rotation process.
Describe the solution you'd like
When there are multiple potential token decryption certs (validationParameters.TokenDecryptionKeys), can the library log which cert (i.e. cert thumbprint) is used for token decryption.

@xubinzheng similar to how we record the SigningKey for a JWS, we could add a property EncryptionKey to JsonWebToken.
Would that work for you?


refactor: ProcessRoundEndReward and ParseDemoHeader and removed unnessary comments

some refactors that I thought it was good. I tested couple demos but could you check if its works properly?

I'm closing it since a new major version is out.
Thank you


How dont working local fideo file?

Be sure to search for your issue before opening a new one.
Current Behavior

Local video dont working (url: blob:http://<IP_ADDRESS>:3000/5ed6d3d5-28ee-4815-af0e-b5ce1925684c)
Video dont play
Expected Behavior

Hash local video url
Steps to Reproduce

added local file

Environment

localhost:3000
Browser: Chrome
Operating system: Linux
jsFiddle example: https://jsfiddle.net/6ugmjpk0/

Other Information

Why when specifying a local file in video player hls does not work. How to fix it?
Video in Network is shown as downloaded and you can get url
Sandbox link:
https://codesandbox.io/s/quirky-hypatia-qt6rc?file=/src/utils/content.js

See https://github.com/cookpete/react-player/issues/521#issuecomment-440410426

See #521 (comment)

forceHLS is true, local video dont playing

@cookpete forceHLS is true, local video not playing


SR-5803: NSNumber no longer equates 1 and true

Fix isEqual to work with NSNumbers initialised with different types
(eg, float and UInt).

Match the behaviour of Darwin where true can only equal an NSNumber
with a value of exactly 1.

I essentially added tests that worked correctly on Darwin and then fixed the current code until all of the tests passed.

@parkera Does this look ok now? It mirrors the Darwin results using the same method.

Yup

@swift-ci test and merge

Is there anyway to get a clean build done? I tested on my Linux host (merging into master) and got the same segmentation fault as seen on the CI but when I deleted my foundation-linux-x86_64 build dir it tested successfully.

I think there was something... let me try

@swift-ci clean test

@swift-ci clean

Ok I give up. cc @shahmishal  :)

@swift-ci clean test

@swift-ci test

Clean is not supported with with swift-corelibs-foundation PR testing.

@swift-ci please test


beforeSave signature mismatch

see https://github.com/cakephp/cakephp/blob/master/src%2FORM%2FTable.php#L1439

Thanks!


Handle errors in callbacks given to randomBytes

If you throw an error in  callback given to crypto.randomBytes you otherwise get very strange behaviour.

Yes. Never call a callback within a try block

this also needs to hit pseudoRandomBytes, this was already fixed in the pulled out version

@calvinmetcalf I actually did both in the code but forgot to put it in the description.  We should probably just redirect to the split out version instead though.

ACK, also prevents double calling of the callback.

yes for randomBytes, pseudo doesn't have one (yet I guess)

Couldn't we use the same implementation for randomBytes and pseudoRandomBytes?  They currently have the same implementation except in node.js (https://www.diffchecker.com/yd1lxexd).  Alternatively, we could consider adding a fall-back to insecure Math.random based generation of random bytes in the pseudoRandomBytes case, but that seems a little scary from a library with the name crypto.

publishing pseudoRandomBytes wouldn't be too big a deal, we could then if we really wanted to add an honest to goodness pseudoRandomBytes generator we'd probably want to do something more along the lines of seed chacha20 with a call to randomBytes or something. (I mention chacha20 because it has a getBytes method which can be used in JS with no other dependencies and it's rust uses it for a random bytes source).  We might want to move this into another issue though

Merging as this is good to go.  If you want to discussion pseudoRandomBytes, please do so in a new issue :)


Fusion in Electron

When I use setPetColorMapTransferFunctionForVolumeActor in Electron

May I ask why

hmmm, does it work normally in chrome?

hmmm, does it work normally in chrome?

That problem is solved, thank you.

Could you please add the solution here for future persons seeing the same problem? Thanks!

The problem occurred when I was using a low version of electron, and the code was running in the corresponding low version of chromeium, which is version M96. Just upgrade the electron version to fix the problem.

The problem occurred when I was using a low version of electron, and the code was running in the corresponding low version of chromeium, which is version M96. Just upgrade the electron version to fix the problem.当我使用低版本的电子时，出现了问题，并且代码在相应的低版本的铬（即 M96 版本）中运行。只需升级电子版本即可解决问题。

I have encountered the same problem, but I don't understand the version you are referring to. Is it VTK or Cornerstone? Can you be more specific

Electron

The problem occurred when I was using a low version of electron, and the code was running in the corresponding low version of chromeium, which is version M96. Just upgrade the electron version to fix the problem.当我使用低版本的电子时，出现了问题，并且代码在相应的低版本的铬（即 M96 版本）中运行。只需升级电子版本即可解决问题。

I have encountered the same problem, but I don't understand the version you are referring to. Is it VTK or Cornerstone? Can you be more specific我遇到了同样的问题，但我不明白你所指的版本。是 VTK 还是 Cornerstone？你能说得更具体一点吗

Oh, you're talking about Electron, I'm in a web browser


text_generation: improve parameters check

Improves error reporting for case when internal_bucketing requested without kv-cache enabled
For example command line (note the absence of --use_kv_cache parameter)
python3 run_generation.py --model_name_or_path bigcode/starcoder --batch_size 2 --max_new_tokens 100 --bucket_size 128 --bucket_internal --use_hpu_graphs --bf16  --prompt 'def print_hello_world():'

Before fix:
Traceback (most recent call last):
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 758, in <module>
    main()
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 523, in main
    generate(None, args.reduce_recompile)
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 494, in generate
    outputs = model.generate(
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/var/work/optimum-habana/optimum/habana/transformers/generation/utils.py", line 1442, in generate
    result = self._sample(
  File "/var/work/optimum-habana/optimum/habana/transformers/generation/utils.py", line 2513, in _sample
    if model_kwargs.get("token_idx_cpu") <= (model_kwargs["kv_cache_len"] // bucket_size) * bucket_size:
KeyError: 'kv_cache_len'

After fix:
Traceback (most recent call last):
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 758, in <module>
    main()
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 523, in main
    generate(None, args.reduce_recompile)
  File "/var/work/optimum-habana/examples/text-generation/run_generation.py", line 494, in generate
    outputs = model.generate(
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/var/work/optimum-habana/optimum/habana/transformers/generation/utils.py", line 1065, in generate
    assert generation_config.use_cache, "please set use_cache flag to use bucket_internal"
AssertionError: please set use_cache flag to use bucket_internal

Error message is much more clear with this fix
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Always build embedded

Solves issues with switching between different targets without having to manually clean and recompile.

Resolves #17


Feature/add openssl scripts

Creates a Root CA and server cert for the nodejs https module for development.
The script is flexible to change many parameters via the bash environment, but sane defaults are provided.
Added ./src/certs folder for organization.
Update app.js to point to appropriate location for generated certs.
Update README.md on script for development purposes.

It seems to dump all the files into the /tools directory for me and doesn't create any /certs directory at all. This means the certsPath you've defined doesn't work for me, and I assume this isn't intended. I tested this on macOS 10.15.3 using zsh.
Also, it doesn't work at all in PowerShell but I assume nobody cares about Windows so whatever 😂

Sorry about that Aryu, just a few silly mistakes. Please try again and let me know. I was simply missing a call to the function to install the certs into src/certs, as well as setting some ownership bits on the file.

@Aryuko My mistake on the npm cert script target- I left the relative paths ../src/certs so it was going one directory above the root of the repo.  Latest push fixes thing since npm run-script cert always runs in the root path of the directory.

I assume nobody cares about Windows

D:


document warningsFilter

document warningsFilter as introduced in https://github.com/webpack/webpack/pull/4381

It's better to merge this only after a new stable version is available.


Use source generator for mapping HTTP API to command handlers

Eventuous supports mapping command handlers in command services to HTTP API endpoints. It removes quite a lot of boilerplate code using minimal API feature of latest .NET versions.
However, the feature uses reflections to resolve the service for each API call. Command handler delegates are cached, but it's still not exceptionally fast and also breaks AOT-compatibility because of use of reflections.
It should be relatively straightforward to replace reflections with generated code.

Moving it to after 0.15


Adding the per request trace latency information

Adding per-request latency information will be very useful for debugging, I do many workarounds to get these kinds of data right now but an adaptation of Triton Server per request Tracing https://github.com/triton-inference-server/server/blob/main/docs/user_guide/trace.md would be a very useful addition.

Hey @saeid93 ,
Totally agree - tracing would be a great addition.
That one is already on our sights, and tracked in #39, so I'll go ahead and close this one - happy to continue the discussion on the other issue though!


Tomorrow education domain

Added domain tomorrows.education, the former domain of Tomorrow University of applied sciences, the emails still have the old domain
https://www.tomorrow.university/

@philipto Just as in info, are the requests not answered still in the 1st verification step or have already been considered as not suitable?

@brisu96 Would you please provide us with some more info about this university? In particular:

A link to an official German Ministry of Education website page confirming that Germany recognize this institution as a university. Any other country works too, I mention Germany because of this article: https://assets-global.website-files.com/61b74291129dde846cd732c0/6311bab494970098b63decfe_Press Release EN - State-Recognized University.pdf

Please provide us with a proof, which shows that the school recognizes the domain tomorrows.education that you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Thank you!

@brisu96 You might miss my previous comment. Please react.

@brisu96 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


libs/{libname}/{path} and doc/html/{libname}/{path} redirects are broken

This is an extension of #1007. In the old website, the following URLs (for instance)

https://www.boost.org/libs/multi_index/doc/tutorial/basics.html
https://www.boost.org/libs/json/doc/html/json/quick_look.html
https://www.boost.org/doc/html/poly_collection/tutorial.html

get redirected to

https://www.boost.org/doc/libs/1_84_0/libs/multi_index/doc/tutorial/basics.html
https://www.boost.org/doc/libs/1_84_0/libs/json/doc/html/json/quick_look.html
https://www.boost.org/doc/libs/1_84_0/doc/html/poly_collection/tutorial.html

Note that we have two types of libraries according to where their docs are located:

Normal libraries at libs/{libname}
So-called "integrated libraries" at doc/html/{libname}. These follow an old initative to construct an integrated BoostBook with all the libraries featuring as individual chapters. The list of integrated libraries can be obtained from https://www.boost.org/doc/libs/1_84_0/doc/html/libraries.html. It's been discussed that integrated libraries should eventually revert to nomal doc location, but this move has not been initiated yet.

My proposal is that we support both redirect schemas in the new website:

libs/{libname}/{path} --> doc/libs/{latest_version}/libs/{libname}/{path}
doc/html/{libname}/{path} --> doc/libs/{latest_version}/doc/html/{libname}/{path}

A smarter approach, which I don't think necessary at this moment, is that the schema 2 reverts to

doc/html/{libname}/{path} --> doc/libs/{latest_version}/libs/{libname}/{path}

when the "backend" detects that the affected library has migrated from integrated to normal (and only in that case).

These redirects have been installed. Deploying to staging now.


mismatch in the output of fullyconnected layer(graph API)

Hi,
Compute library- 18.05
arm7a
Platform : linux
Output of 'strings libarm_compute.so | grep arm_compute_version':
I implemented a sample network based on mnist network, using  graph API.  .npy weights are extracted from tensorflow weight file. The outputs of  convolution and pool layers are matching the reference output of tensorflow.
But the output of dense layer (FullyConnected layer) following the pool layer and convolution is not matching the referece output.
input (after pool layer) shape of  FC : 10 x 8 x 64 x 1 ( WHCN )
output of FC : 1024 channels
shape of weight : ( 1024 x 5120) (  of .npy file)
shape of bias : 1024
PS : Relu layer follows this FC layer
Is there any issue with the input ? Is there any issue in the mutators ? Please help with other possible causes  leading to this issue
Awaiting your reply,

Hello @ved27,
Maybe the layout of your fc weights is different as it might have been trained differently, and that seems to be the case if you extracted them from tensorflow.
Can you set NWHC the layout of weights_trained_layout in the FullyConnectedLayerInfo that you pass to the FC layer and check if this will work?

Hi,
FullyConnectedLayerInfo - does this parameter exist in ComputeLibrary ?

Hello @ved27, its part of the 18.08 release. If you need/have to use 18.05 release you have to manually interleave your fully connected weights.

Hi ,
Earlier I extracted FC weights (1024 x 5120) from caffemodel, and used those .npy files to run  on ACL.
It gave correct outputs.
Here, the weights are extracted from tensorflow and I transposed the weights too (from 5120 x 1024 to 1024 x 5120 ) .
What's the difference between tf and caffe ?

Hi @GeorgeARM
Referring to the suggestion of you above
"NWHC the layout of weights_trained_layout in the FullyConnectedLayerInfo"
NWHC - is this layout existing ? or you meant NHWC ?
Also, in https://github.com/ARM-software/ComputeLibrary/blob/master/src/runtime/CL/functions/CLFullyConnectedLayer.cpp   LINE:204
if(_is_fc_after_conv && (input->info()->data_layout() != fc_info.weights_trained_layout))
{
// Convert weights
_convert_weights.configure(weights_to_use,
&_converted_weights_output,
input->info()->tensor_shape(),
fc_info.weights_trained_layout);
    weights_to_use         = &_converted_weights_output;
    _are_weights_converted = false;
}

So, instead of interleaving the weights, can I change the DataLayout of input ?

Hi ,
Apart from this,
I tried the same FC layer with input size 1024x1 (NCHW layout) and output is 10x1 for the same .
I dumped the input of the layer from tensor flow ( shape of input tensor is 1x1024) , extrated weights and passed them as inputs to ACL FC layer. The output has shape 10 x 1 (NCHW layout) . But it does not match the output dumped with tensor flow.

Hello @ved27,
Maybe the layout of your fc weights is different as it might have been trained differently, and that seems to be the case if you extracted them from tensorflow.
Can you set NWHC the layout of weights_trained_layout in the FullyConnectedLayerInfo that you pass to the FC layer and check if this will work?

This helped solving the same issue as reported by @ved27. We have dimension mismatch when we used our model from tensorflow on ACL. Using FullyConnectedLayerInfo solved our problem. However, we had to still transpose the weights when extracting the weight/bias files (as discussed in  #517)
Thanks @GeorgeARM  for your inputs.
Regards,
San

Fixed


Allow edit of Sqlproj in place and Reload

Sql project should allow righclick open and edit of Sqlproj file and a toast notif for reload after the update.

related for reload : https://github.com/microsoft/azuredatastudio/issues/11047