Cannot connect to the server

Happened on latest 3.0.5 version
Keeps on prompting cannot connect to server without attempting cloudflare verification
The website version works just fine
and i suggest that changing url from .net to .xxx as .xxx version has more galleries compared to .net version

I'm sorry I'm new to this but How to change the url thou

I'm sorry I'm new to this but How to change the url thou

My java IDE expired so I only tried looking at the code on Vs code
And I found out multiple codes referring to .net version
Just search nhentai.net and their should be several results in the IDE


feat: use better systemd build from hyperscale sig

The CentOS systemd build is very limited, this should enable a ton of systemd modules that are super useful for desktop usage, like systemd-homed, userdbd, timesyncd, (...) that we can use instead of stuff like chrony for example.
Fixes: #35

Still needs some figuring out. Apparently the hyperscale sig is still working on it


[stable/airflow] install-requirements.sh not running with installRequirements true

Describe the bug
The script install-requirements.sh won't run during pod startup with "dags: installRequirements: true" and using git-sync. The packages in dags/requirements.txt do not get installed.
Version of Helm and Kubernetes:
Helm: 3.2.1
Kubernetes: 1.15.7
Which chart:
stable/airflow: 7.1.1
What happened:
The script install-requirements.sh won't run during pod startup. The packages in dags/requirements.txt is not installed even after restarting the pods multiple times.
I exec'd into the scheduler container and did cat dags/requirements.txt and I do see my list of requirements.
I checked the logs of the pods and it shows *** installing requirements... immediately followed by *** installing global extra pip packages..., meaning nothing was printed when install-requirements.sh ran (which seems to me that it never ran), but installRequirements is defiantly true because *** installing requirements... did print.
What you expected to happen:
When the pods start the packages listed in dags/requirements.txt should be installed
How to reproduce it (as minimally and precisely as possible):

Have a git repo with a requirements.txt file (can contain any package such as azure-keyvault==1.1.0)
Set the values in values.yaml:

"dags: installRequirements: true"
the values for the git sidecar in "dags: git" using a SSH key
"dags: gitSync: enabled: true"

Install the stable/airflow Helm chart using the values.yaml file with the above values
After the pods start the package "azure-keyvault" won't be installed

Anything else we need to know:
The dags values I am using is
dags:
  installRequirements: true
  git:
    url<EMAIL_ADDRESS>    ref: master
    repoHost: bitbucket.org
    secret: airflow-git-keys
    privateKeyName: currencyairflowdags_access_key
    gitSync:
      enabled: true
      refreshTime: 60

The dags folder does have the requirements.txt after the pod starts, so the git-sync is working fine
If I exec into the container and run /home/airflow/scripts/install-requirements.sh manually it does install the packages from my dags/requirements.txt file (so the script is fine), but if I delete the pods and they get recreated then the packages are not installed.
I think && /home/airflow/scripts/install-requirements.shis not working in the entrypoint args

I found out with the PR https://github.com/helm/charts/pull/22552 it was closed explaining that dags.installRequirements as true will only work with dags.initContainer.enabled as true. I can have both dags.git.gitSync.enabled and dags.initContainer.enabled and this no longer is an issue. Closing the ticket


Add simplified chinese translation

Excellent! Thanks for your contribution
@all-contributors please add @KevinTroyT for translation


Incorrect version info

Version 1.3.3 still says 1.3.1 in web-gui.

I'm closing this as it's now obsolete


Update Loki documentation, Agent-->Alloy

This issue tracks the work for updating Agent-->Alloy in the Loki documentation.
### Tasks
- [ ] Identify the topics that reference or document using Agent with Alloy
- [ ] Assess the contents for minor update, rewrite, or remove.

Initial scan of Loki docs shows multiple hits on Agent in these topics:

send-data/_index.md
get-started/overview.md
get-started/_index.md
setup/install/helm/reference.md
setup/install/helm/concepts.md
setup/install/helm/monitor-and-alert/with-grafana-cloud.md
setup/upgrade/upgrade-from-2x/index.md

Status Update:
Completed PRs
Part 1 - https://github.com/grafana/loki/pull/12602
Part 2 -https://github.com/grafana/loki/pull/13056
Part 3 - https://github.com/grafana/loki/pull/13336
References to Agent in the Upgrade and Migration docs have been left in place, as they are correct for people who are moving off of Grafana Agent.
The Loki Helm chart still uses the Grafana Agent, so those references are also still in place until the Helm Chart gets updated.

Hey squad! So I am just going through the helm docs and references. So the primary use of Grafana Agent that I can see is for self-monitoring which we have marked as depricated. I have added additional deprecation notices to a PR but I do wonder when we decide to delete this section since the meta-monitoring chart is now replacing this. Here was the original message in the helm ref
############################################## WARNING ###############################################################
#
# DEPRECATED VALUES
#
# The following values are deprecated and will be removed in a future version of the helm chart!
#
############################################## WARNING ##############################################################

# -- DEPRECATED Monitoring section determines which monitoring features to enable, this section is being replaced
# by https://github.com/grafana/meta-monitoring-chart


Add MediaError.message sample

Hello @wolenetz,
May you review this s{i,a}mple for MediaError.message?
Source: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/yDIyWozG1xk

This quick sample LGTM. Perhaps a small note that this sample demonstrates only a tiny fraction of such messages.

Thank you @wolenetz! I've addressed your feedback.

Here it is live: https://googlechrome.github.io/samples/media/error-message.html

FYI @mounirlamouri


Wall Height breaks credits

Every time I join my world I'm greeted with the following

Interesting. I will look into this, its odd as I use that module.
Sorry for the impossible long delay, I work 4 jobs over the summer months and my free time dies off.

I am going to close this. It appears newer version of wall height no longer include or link to these files so this issue isn't something I can replicate. If you are still getting this issue after v2 of MM+ for Foundry v10 please reopen this issue and let me know the versions of everything you are running.


ModuleNotFoundError: No module named 'demo'

Whenever i tried to create a model and loadcheckpoint .with launching this code in Google colab
from demo.py import load_checkpoints
generator, kp_detector = load_checkpoints(config_path='config/vox-256.yaml',
checkpoint_path='/content/gdrive/My Drive/first-order-motion-model/vox-cpk.pth.tar')
I end up getting this ERROR
from demo.py import load_checkpoints
generator, kp_detector = load_checkpoints(config_path='config/vox-256.yaml',
checkpoint_path='/content/gdrive/My Drive/first-order-motion-model/vox-cpk.pth.tar')
CAN SOMEONE HELP ME PLEASE THANK YOU

YOU DID NOT RUN, EITHER git clone ... or cd ....  OPEN NOTEBOOK AND RUN ALL THE CELLS ONE BY ONE.  THANK YOU.


Bump all editions, and required versions for new release

It's an incompatible version bump because the dof API changed slightly. The usdt crate version is also incompatible since the dof crate is a transitive dep.


Separate base data from scenario data in ScenarioCalculationEnvironment

This PR implements a refactoring of scenarios in preparation for local scenarios.
ScenarioCalculationEnvironment now embeds an instance of CalculationEnvironment containing the base data, i.e. any data that is the same in all scenarios. When data is requested the scenario data is checked first and then the request is delegated to the base data.
This separation of base and scenario data is a closer fit to the mental model of most users. It also means the market data containers will be much smaller for environments where only a subset of the data is perturbed in each scenario.
The downside of the changes are additional complexity when building market data but this is unavoidable.

Be careful with the definition of "base data" in the description of this PR. It wouldn't normally mean "any data that is the same in all scenarios"; it would normally mean the set of market data corresponding to the base scenario. From the users' perspective, there may not be a scenario that can be identified as the "base" scenario.

Any other comments? I've addressed everything so far.


Added result_ttl and ttl to cron arguments

Mind mixing the conflict?

Mind mixing the conflict?

Just updated my branch

Thanks!

Hello @marcelblijleven from the documentation I don't understand if I can set ttl in infinite tasks.
For example:
    scheduler.schedule(
        scheduled_time=datetime.utcnow(),  # Time for first execution, in UTC timezone
        func=func,                           # Function to be queued
        kwargs={'size': 1000},
        interval=6 * 60 * 60,            # Time before the function is called again, in seconds
        repeat=None,                       # Repeat this number of times (None means repeat forever)
        queue_name='default',       # In which queue the job should be put in
        result_ttl=0,                          # Result gets deleted immediately
    )

Is this a valid setting? If not what can you suggestion so that I don't store the result in cache.
Thanks in advance!


#2609 Added installation type env var to bridge

Closes #2609

Codecov Report

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

@@             Coverage Diff             @@
##           master    #2633       +/-   ##
===========================================
- Coverage   36.46%   23.50%   -12.97%
===========================================
  Files         163       50      -113
  Lines        9351     1051     -8300
  Branches      185      185
===========================================
- Hits         3410      247     -3163
+ Misses       5396      778     -4618
+ Partials      545       26      -519

Flag
Coverage Δ

moduleA
23.50% <ø> (ø)

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

Impacted Files
Coverage Δ

jmeter-service/common.go

cli/cmd/send_event.go

...ouse-service/event_handler/evaluate_sli_handler.go

api/handlers/evaluation.go

helm-service/pkg/mesh/ingress_config.go

cli/pkg/platform/platform_openshift.go

helm-service/controller/mock_handler.go

...mediation-service/handler/problem_event_handler.go

platform-support/openshift-route-service/main.go

lighthouse-service/event_handler/common.go

... and 102 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 02842a8...39fbecb. Read the comment docs.


rename secure_element to ecc608a_plus_winsim

Description

Change name from secure_element to ecc608a_plus_winsim and rename folders.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

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

/bot run checks


Package dependencies for oasis3-mct include versions

I was getting an error building oasis3-mct because I didn't have<EMAIL_ADDRESS>available. This is because versions are included in the depends_on
https://github.com/ACCESS-NRI/spack_packages/blob/main/packages/oasis3-mct/package.py#L24-L25
I don't think we should specify versions in depends_on unless there is a technical reason, e.g. API incompatibility, and in that case it should be >= so as not to lock to a single version.
Versions should be specified in the spec.

Are there any historical reasons that the following versions were chosen?:
module load intel-compiler/2019.5.281
module load netcdf/4.7.4
module load openmpi/4.0.2

Should we keep them as the minimum requirement?

The history of the config.gadi file
https://github.com/COSIMA/oasis3-mct/commits/master/util/make_dir/config.gadi
has some useful pointers, namely this issue:
https://github.com/COSIMA/access-om2/issues/157
which largely seems to be that these were the most recent versions available at the time of transitioning to gadi (or close to the most recent).

I'll keep them as the minimum requirement for the time being.

Seems reasonable.


Add libgfortran to the Toronto Hub

Description
We've been requested to add libgfortran to the U Toronto hub
Value / benefit
This is a support request for a hub's community
Implementation details
it should be add-able from conda forge with this recipe:
https://anaconda.org/conda-forge/libgfortran
here's the hub repo: https://github.com/utoronto-2i2c/jupyterhub-deploy

cc'ing @consideRatio on this since he's the support steward currently - though I went ahead and gave a shot at implementing in https://github.com/utoronto-2i2c/jupyterhub-deploy/pull/126 as well - maybe this just needs a review+merge

Let's close this once the changes get into the production hub.

Update from team meeting
This issue has been deployed to staging. We should deploy to production and then notify the person that opened the support ticket to tell them it should work now. Updated the top comment with these steps

Deployed to prod here :arrow_right:  https://github.com/utoronto-2i2c/jupyterhub-deploy/pull/128
Freshdesk ticked updated and resolved  :arrow_right: https://2i2c.freshdesk.com/a/tickets/39
:tada:


Remove unused localizations using pre_install hook

prepare_command works only in podspec. Latest CocoaPods have changed interface of installer, so original hook wasn't working. Tested on CocoaPods 0.38.2.

@otaran thank you, @mattt please merge this

Works perfectly! Thanks @otaran ! :+1:

FormatterKit now can be added as a framework via CocoaPods (use_frameworks!) which means you don't have to remove other localizations (#208). Please reopen if you still have any issues.


Create a pull request

follow to the link, add title "First release to production" and add the metadata then merge

done


Responsive visibleSlides

Hello,
Can we make the visibleSlides responsive ?
On my phone, I'd like to have only one slide but on my 4k monitor, I would like to display at least 4 slides.
Thank you

We tackle this on our website by using vanilla javascript to determine the
size of the viewport, or a modal or other parent container, and adjust the
visible slides accordingly.   In whatever container that holds your
carousel provider, set up a listener for when the viewport changes size
inside your componentDidMount lifecycle event.  We've actually discussed
how to do this on some closed tickets.   I can try and find issue for you
and share it.
On Sun, Jul 7, 2019 at 6:16 AM Yo1L<EMAIL_ADDRESS>wrote:

Hello,
Can we make the visibleSlides responsive ?
On my phone, I'd like to have only one slide but on my 4k monitor, I would
like to display at least 4 slides.
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/express-labs/pure-react-carousel/issues/157?email_source=notifications&email_token=AABHKLBAYI2YF5RELHJM36DP6G663A5CNFSM4H6VKG72YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G5WL5RQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABHKLE2U3E65AVHFKIEZG3P6G663ANCNFSM4H6VKG7Q
.

@Yo1L https://github.com/express-labs/pure-react-carousel/issues/126

Hi,
That's what I've finally done but I had to also tweak the natural height because my vignettes are responsive too...
I know customers always want weird stuff :)
class CentreInteret extends React.Component {
  state = {
    showItems: 5
  }

  handleWindowResize = () => {
    if (window) {
      let showItems = 8
      let naturalSlideHeight = 150
      if (window.innerWidth < 760) {
        showItems = 1
        naturalSlideHeight = 80
      } else if (window.innerWidth < 800) {
        showItems = 2
      } else if (window.innerWidth < 1050) {
        showItems = 3
      } else if (window.innerWidth < 1440) {
        showItems = 4
      } else if (window.innerWidth < 1980) {
        showItems = 6
      }
      this.setState({
        showItems: showItems,
        naturalSlideHeight: naturalSlideHeight
      })
    }
  }

  componentDidMount() {
    if (window) {
      window.addEventListener('resize', this.handleWindowResize.bind(this))
      this.handleWindowResize()
    }
  }

  componentWillUnmount() {
    if (window) {
      window.removeEventListener('resize', this.handleWindowResize.bind(this))
    }
  }

  render() {
    const props = this.props
    const { showItems, naturalSlideHeight } = this.state

    return (

        <CarouselProvider
          naturalSlideWidth={100}
          naturalSlideHeight={naturalSlideHeight}
          visibleSlides={showItems}
          totalSlides={props.edges.length}>

            <CarouselButtons>

                Sélection de centres d'intérêt

            </CarouselButtons>

          <Slider className="flex flex-col">
            {props.edges.map(({node}, index) => (
              <Slide index={index} key={index} className="flex-1">
                <div className="mx-12 md:mx-2 rounded-lg shadow-md flex h-vignette" style={{
                  backgroundImage: `url(${node.data.S_lection_Photo_de_couverture[0].thumbnails.large.url})`,
                  backgroundSize: `cover`,
                  backgroundRepeat: `no-repeat`,
                }}>

                    {node.data.Nom}

              </Slide>
            ))}
          </Slider>
        </CarouselProvider>

    )
  }
}

Thanks for your help

No problem.  Glad I could help.

Using a hook with resize observer seems to work too. 😄
First, you probably want a polyfill:
yarn add resize-observer-polyfill

Second, here's the hook:
import { useRef, useState, useEffect } from "react"
import ResizeObserver from "resize-observer-polyfill"

export default function useMeasure() {
  const ref = useRef()
  const [bounds, set] = useState({ left: 0, top: 0, width: 0, height: 0 })
  const [ro] = useState(
    () => new ResizeObserver(([entry]) => set(entry.contentRect))
  )
  useEffect(() => {
    let didCancel = false

    if (!didCancel) {
      ro.observe(ref.current)
    }

    return () => {
      didCancel = true
      ro.disconnect()
    }
  }, [ref, ro])
  return [{ ref }, bounds]
}

And finally the carousel:
import React from "react"
import useMeasure from "./useMeasure"
import {
  CarouselProvider,
  Slider,
  Slide,
  ButtonBack,
  ButtonNext,
} from "pure-react-carousel"
import "pure-react-carousel/dist/react-carousel.es.css"

export const Carousel = () => {
  const [bind, { width }] = useMeasure()
  const visibleSlides = width > 768 ? 3 : 1

  return (
    <CarouselProvider
      naturalSlideWidth={100}
      naturalSlideHeight={75}
      totalSlides={3}
      visibleSlides={visibleSlides}
    >

        <Slider>
          <Slide index={0}>I am the first Slide.</Slide>
          <Slide index={1}>I am the second Slide.</Slide>
          <Slide index={2}>I am the third Slide.</Slide>
        </Slider>
        <ButtonBack>Back</ButtonBack>
        <ButtonNext>Next</ButtonNext>

    </CarouselProvider>
  )
}


Build issue fix in docs

Recent changes have started causing build errors. - https://github.com/meshery/meshery/commit/b852bb09e6931c0aa56de2d51eb7dab34476b1dd
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 4d7c439cfc26aa8026aedd6645691265297a4ca0 at: https://667bea7e7ae555e1497e122d--meshery-docs-preview.netlify.app


added python3-gtsam-pip to rosdep python

Please add the following dependency to the rosdep database.
Package name:
gtsam
Package Upstream Source:
https://github.com/borglab/gtsam
Purpose of using this:
Python Wrapper for GTSAM for Smoothing and Mapping (SAM) in robotics.
Links to Distribution Packages

PyPi

https://pypi.org/project/gtsam/

Please add this package to the rosdep database.

I am withdrawing my review approval. I didn't realize that gtsam is packaged in ROS for some distributions and thus using the pip version could create conflicts. It will remain possible to use the version from pip together with custom rosdep sources but I do not think that pip-alternatives to packaged ROS software should be added to the official rosdep db.

No response to this in a long time, and there are concerns here.  I'm going to close this out for now, but feel free to reopen if you'd like to continue the discussion.


Document no-system-llvm

This was approved but not merged.  I don't have permissions to merge it.

Sorry, got side tracked...

No problem!  Thanks for doing that.


notcalculator

Issue Id you have worked upon -
#542
Briefly explain your program logic -
created a not calculator
Screenshots(Attach 2 screenshots of your own input and output) -

By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

Congratulations 🎉 @Upkar0904 on your contribution.
Thanks for adding value to this repository. 😊
Let's celebrate this fest 😃 by providing more developers 👨💻👩💻 an opportunity to contribute. Help us make this repository reach far and more by sharing this to your friends and colleagues.
Your efforts can make difference.

Hey 👋, may you also encourage us by clicking the star icon on the repository main page. ⭐


GCE driver should name the instance after the machine name

It would be nice if machine created with the GCE driver (and others) would be named (or even prefixed) after the machine name by default if no flag value for --google-instance-name is specified.
The current behavior is a bit confusing as machine will create multiple machine with the same underlying VM, if you don't specific --google-instance-name.
proppy@noir:~$ machine create --driver google --google-project proppy-stuff foo
proppy@noir:~$ machine create --driver google --google-project proppy-stuff bar
FATA[0004] Instance docker-machine already exists.
proppy@noir:~$ machine ls
NAME             ACTIVE   DRIVER   STATE     URL
bar              google   Running   tcp://<IP_ADDRESS>:2376
foo     *        google   Running   tcp://<IP_ADDRESS>:2376

Possibly related to #59 #219
/cc @dlorenc

All machines will take the name from the command as the VM name in whatever account/hypervisor

@sthulb that doesn't seems to be the case right now, at least for GCE:
https://github.com/docker/machine/blob/master/drivers/google/google.go#L96

Not right now. It's something I'm working on for #59

Closing as #59 will handle this (as @sthulb mentioned).


Question: Is it possible with know the current route?

I'd like to know my current route, there is a way with the router? or should I use regular JS?

I have the same issue.  How do you quickly query the current route?

Hey @pmcalabrese
Do you need to transform/adjust the current route (ie. are you trying to extracting a value from it), or is your use cases similar to what @bminer described (ie. emphasize links which match the current route)?

Closing this issue because it's related to an old router version. Please update to the latest @riotjs/route version if you can.


[Enhancement] Per-channel filter lists

From #suggestions on the server:

Kanjo: Ability to ignore certain filtered words on a per channel basis.

Currently you can only have 1 list of filtered words and enable or disable the entire list on a channel by channel basis. This means if you add 2 words to the filtered word list and enable it on two channels those channels loses access to both words. This makes it impossible to have the 1st word filtered from the 1st channel and the 2nd word filtered from the 2nd channel without filtering the other word in the channel at the same time

Basically, If I have #room1 and #room2, they would each have their own list of filtered words.

This would remove backwards compatibility ;s I will think about it

I've been working on filtering words for 1.0 today, it will be tough to control. I want to keep it simple. When we get a dashboard ill reconsider this

would it be possible to add a silent delete option for filtered words? (delete without notifying the offending user)

Do ;verbose 0 to turn off the feedback when a filtered word is deleted


conn_is_closed should return 1 if get_last_sys_error is WSAECONNRESET

CLA: trivial
conn_is_closed should check for WSAECONNRESET  on Windows because it has a different value than ECONNRESET.  Without this check ssl tests fail in the Python standard tests because the SSL_do_handshake() on the server side returns an error if the client closes the socket too quickly.

@paulmon please can you amend your commit message to include the "CLA: trivial" text in the commit description (to keep the CLA bot happy).

I tried editing the commit message with git --amend, but the buildbot still doesn't like it.
I also put the CLA: trivial on a seperate line in the commit message like it says in CONTRIBUTING.
Is that what you were suggesting?  Do I need to close this PR and create a new one or should that have worked if I did it right?

Close/reopen to kick CLA bot

Not sure why the bot doesn't like it. It looks ok to me. I removed it by hand for now.

Pushed. Thanks.


Add 2.6.5 bug fixes to master

Master has some bugs that were fixed in v2.6.5, so I created this PR to merge the 2.6.5 fixes to master. @Sean-McQ maybe we can sync up about how to resolve the merge conflicts?

I forgot the code base was restructured due to plugin work, so I'm going to close this PR and manually copy-paste the needed changes to avoid the merge conflicts.


Dimensions[] no longer returns accurate data

The coordinates are relative to the player's position

The coordinates are still accurate and absolute except now they are not of the entire playfield, but of a rectangle centered at the player's position, and are in a randomly shifted coordinate system.

Yes, I've noticed. I made a change to the wiki

Closing since I will not fix this.
Expose should be as dumb as possible to make its maintaining easy.
All complex logic to detect actual world borders' coordinates should be inside scripts using expose.


issue #10

Title
Fix issue with social links container in Edit Panel
Description
Resolved the issue with the social links container not being displayed properly in the Edit Panel section. Added the necessary HTML and CSS to create the social links container inside the Edit Panel. The container now displays social media icons with proper spacing and hover effects.
Dependencies: None

Issue Ticket Number
Fixes #10

Type of change

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

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

i have rectified the issue, i think, take a look at the image, i will make a PR soon.


Background sampling

In CenterPoint original paper, the authors cited Faster R-CNN on sampling 128 boxes in 1:1 positive to negative ratio.  May I ask if this is implemented in the current version of CenterPoint in mmDetection3D?  I am trying to find the code but I cannot find it.  If this is not implemented, may I ask if there is any similar piece of code that comes into your mind that I can start with, to add this feature to the CenterPoint in mmdetection3D repo, please?

May I ask if the orignal authors adding CenterPoint to the repo can kindly comment @yinchimaoliang @zhezhao1989

For the time being, two stage version of CenterPoint is not supported. If you want to implement the above features, you may have to implement the second stage of CenterPoint first. FYI, you can add the second stage head under roi_heads and extend the centerpoint.py under the detectors to two stages. As for your question, you can refer to the related samplers in mmdetection project.

@yinchimaoliang Thank you for getting back to me, Yinhao.  Are you referring to the db_sampler that add samples onto the scene?  So are you suggesting us to also store the background points and add them randomly to the scene?
Also, for the current implementation, do you mean we do not backprop with any negative samples?
Thank you so much!

@Code-Gratefully You are welcome.

db_sampler is a kind of augmentation method, I'm afraid that's not what I was refering to. I was suggesting that you can refer to the samplers in mmdetection project like random_sampler to help you to implement the functions you mentioned before .
Since we have not implemented two stage version of CenterPoint, we do not backprop with any negative samples.

For the time being, two stage version of CenterPoint is not supported. If you want to implement the above features, you may have to implement the second stage of CenterPoint first. FYI, you can add the second stage head under roi_heads and extend the centerpoint.py under the detectors to two stages. As for your question, you can refer to the related samplers in mmdetection project.

In addition, random_sampler is for reference only, it is used for sample gt, you need to use it to sample predictions.

Thanks for the detailed comment, I will look into those.


Failed to build mxnet from source

Description
Hello all, I was trying to build mxnet from source (for tensorrt support), but errors occurred during build process.
Error Message
Performing C SOURCE FILE Test CMAKE_HAVE_LIBC_PTHREAD failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_4bef8 && [1/2] Building C object CMakeFiles/cmTC_4bef8.dir/src.c.o
[2/2] Linking C executable cmTC_4bef8
FAILED: cmTC_4bef8
: && /usr/bin/cc -Wall -Wno-sign-compare -O3 -DCMAKE_HAVE_LIBC_PTHREAD  CMakeFiles/cmTC_4bef8.dir/src.c.o -o cmTC_4bef8   && :
CMakeFiles/cmTC_4bef8.dir/src.c.o: In function main': src.c:(.text.startup+0x23): undefined reference to pthread_create'
src.c:(.text.startup+0x2c): undefined reference to pthread_detach' src.c:(.text.startup+0x35): undefined reference to pthread_cancel'
src.c:(.text.startup+0x40): undefined reference to pthread_join' src.c:(.text.startup+0x4b): undefined reference to pthread_atfork'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
Source file was:
#include <pthread.h>
void* test_func(void* data)
{
return data;
}
int main(void)
{
pthread_t thread;
pthread_create(&thread, NULL, test_func, NULL);
pthread_detach(thread);
pthread_cancel(thread);
pthread_join(thread, NULL);
pthread_atfork(NULL, NULL, NULL);
pthread_exit(NULL);
return 0;
}
Determining if the function pthread_create exists in the pthreads failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_3f333 && [1/2] Building C object CMakeFiles/cmTC_3f333.dir/CheckFunctionExists.c.o
[2/2] Linking C executable cmTC_3f333
FAILED: cmTC_3f333
: && /usr/bin/cc -Wall -Wno-sign-compare -O3 -DCHECK_FUNCTION_EXISTS=pthread_create  CMakeFiles/cmTC_3f333.dir/CheckFunctionExists.c.o -o cmTC_3f333  -lpthreads && :
/usr/bin/ld: cannot find -lpthreads
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
Determining if the OPENBLAS_USE64BITINT exist failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_d0797 && [1/2] Building C object CMakeFiles/cmTC_d0797.dir/CheckSymbolExists.c.o
FAILED: CMakeFiles/cmTC_d0797.dir/CheckSymbolExists.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3  -fPIE -o CMakeFiles/cmTC_d0797.dir/CheckSymbolExists.c.o -c CheckSymbolExists.c
CheckSymbolExists.c: In function ‘main’:
CheckSymbolExists.c:8:19: error: ‘OPENBLAS_USE64BITINT’ undeclared (first use in this function); did you mean ‘OPENBLAS___64BIT__’?
return ((int*)(&OPENBLAS_USE64BITINT))[argc];
^~~~~~~~~~~~~~~~~~~~
OPENBLAS___64BIT__
CheckSymbolExists.c:8:19: note: each undeclared identifier is reported only once for each function it appears in
ninja: build stopped: subcommand failed.
File /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c:
/* */
#include </usr/include/x86_64-linux-gnu/openblas_config.h>
int main(int argc, char** argv)
{
(void)argv;
#ifndef OPENBLAS_USE64BITINT
return ((int*)(&OPENBLAS_USE64BITINT))[argc];
#else
(void)argc;
return 0;
#endif
}
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_UNUSED_LOCAL_TYPEDEF_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_df535 && [1/2] Building C object CMakeFiles/cmTC_df535.dir/src.c.o
FAILED: CMakeFiles/cmTC_df535.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_UNUSED_LOCAL_TYPEDEF_FLAG -fPIE   -Wunused-local-typedef -o CMakeFiles/cmTC_df535.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wunused-local-typedef'; did you mean '-Wunused-local-typedefs'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_COVERED_SWITCH_DEFAULT_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_9e84d && [1/2] Building C object CMakeFiles/cmTC_9e84d.dir/src.c.o
FAILED: CMakeFiles/cmTC_9e84d.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_COVERED_SWITCH_DEFAULT_FLAG -fPIE   -Wcovered-switch-default -o CMakeFiles/cmTC_9e84d.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wcovered-switch-default'; did you mean '-Wno-switch-default'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_DEPRECATED_REGISTER_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_507ea && [1/2] Building C object CMakeFiles/cmTC_507ea.dir/src.c.o
FAILED: CMakeFiles/cmTC_507ea.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_DEPRECATED_REGISTER_FLAG -fPIE   -Wdeprecated-register -o CMakeFiles/cmTC_507ea.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wdeprecated-register'; did you mean '-frename-registers'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_GNU_ANONYMOUS_STRUCT_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_eadb2 && [1/2] Building C object CMakeFiles/cmTC_eadb2.dir/src.c.o
FAILED: CMakeFiles/cmTC_eadb2.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_GNU_ANONYMOUS_STRUCT_FLAG -fPIE   -Wgnu-anonymous-struct -o CMakeFiles/cmTC_eadb2.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wgnu-anonymous-struct'
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_SELF_ASSIGN_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_9923d && [1/2] Building C object CMakeFiles/cmTC_9923d.dir/src.c.o
FAILED: CMakeFiles/cmTC_9923d.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_SELF_ASSIGN_FLAG -fPIE   -Wself-assign -o CMakeFiles/cmTC_9923d.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wself-assign'; did you mean '-Wcast-align'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_VLA_EXTENSION_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_43847 && [1/2] Building C object CMakeFiles/cmTC_43847.dir/src.c.o
FAILED: CMakeFiles/cmTC_43847.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_VLA_EXTENSION_FLAG -fPIE   -Wvla-extension -o CMakeFiles/cmTC_43847.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wvla-extension'; did you mean '-fms-extensions'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_WNO_FORMAT_PEDANTIC_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_b1681 && [1/2] Building C object CMakeFiles/cmTC_b1681.dir/src.c.o
FAILED: CMakeFiles/cmTC_b1681.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_WNO_FORMAT_PEDANTIC_FLAG -fPIE   -Wformat-pedantic -o CMakeFiles/cmTC_b1681.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-Wformat-pedantic'; did you mean '-Wno-pedantic'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_MMIC_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_0792e && [1/2] Building C object CMakeFiles/cmTC_0792e.dir/src.c.o
FAILED: CMakeFiles/cmTC_0792e.dir/src.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_MMIC_FLAG -mmic -fPIE   -mmic -o CMakeFiles/cmTC_0792e.dir/src.c.o -c src.c
cc: error: unrecognized command line option '-mmic'; did you mean '-fpic'?
cc: error: unrecognized command line option '-mmic'; did you mean '-fpic'?
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Performing C SOURCE FILE Test LIBOMP_HAVE_M32_FLAG failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_8e862 && [1/2] Building C object CMakeFiles/cmTC_8e862.dir/src.c.o
[2/2] Linking C executable cmTC_8e862
FAILED: cmTC_8e862
: && /usr/bin/cc -Wall -Wno-sign-compare -O3 -DLIBOMP_HAVE_M32_FLAG -m32  CMakeFiles/cmTC_8e862.dir/src.c.o -o cmTC_8e862   && :
/usr/bin/ld: cannot find Scrt1.o: No such file or directory
/usr/bin/ld: cannot find crti.o: No such file or directory
/usr/bin/ld: skipping incompatible /usr/lib/gcc/x86_64-linux-gnu/7/libgcc.a when searching for -lgcc
/usr/bin/ld: cannot find -lgcc
/usr/bin/ld: skipping incompatible /usr/lib/gcc/x86_64-linux-gnu/7/libgcc.a when searching for -lgcc
/usr/bin/ld: cannot find -lgcc
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
Source file was:
int main(void) { return 0; }
Determining if files windows.h;psapi.h exist failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_056e1 && [1/2] Building C object CMakeFiles/cmTC_056e1.dir/LIBOMP_HAVE_PSAPI_H.c.o
FAILED: CMakeFiles/cmTC_056e1.dir/LIBOMP_HAVE_PSAPI_H.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3  -fPIE -o CMakeFiles/cmTC_056e1.dir/LIBOMP_HAVE_PSAPI_H.c.o -c /home/shunfan/Documents/mxnet/build/CMakeFiles/CheckIncludeFiles/LIBOMP_HAVE_PSAPI_H.c
/home/shunfan/Documents/mxnet/build/CMakeFiles/CheckIncludeFiles/LIBOMP_HAVE_PSAPI_H.c:2:10: fatal error: windows.h: No such file or directory
#include <windows.h>
^~~~~~~~~~~
compilation terminated.
ninja: build stopped: subcommand failed.
Source:
/* */
#include <windows.h>
#include <psapi.h>
int main(void){return 0;}
Determining if the function EnumProcessModules exists in the psapi failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_1b5a9 && [1/2] Building C object CMakeFiles/cmTC_1b5a9.dir/CheckFunctionExists.c.o
[2/2] Linking C executable cmTC_1b5a9
FAILED: cmTC_1b5a9
: && /usr/bin/cc -Wall -Wno-sign-compare -O3 -DCHECK_FUNCTION_EXISTS=EnumProcessModules  CMakeFiles/cmTC_1b5a9.dir/CheckFunctionExists.c.o -o cmTC_1b5a9  -lpsapi && :
/usr/bin/ld: cannot find -lpsapi
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
Determining if the function __atomic_load_1 exists failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_79cae && [1/2] Building C object CMakeFiles/cmTC_79cae.dir/CheckFunctionExists.c.o
:0:23: warning: conflicting types for built-in function ‘__atomic_load_1’ [-Wbuiltin-declaration-mismatch]
/usr/local/share/cmake-3.18/Modules/CheckFunctionExists.c:7:3: note: in expansion of macro ‘CHECK_FUNCTION_EXISTS’
CHECK_FUNCTION_EXISTS(void);
^~~~~~~~~~~~~~~~~~~~~
[2/2] Linking C executable cmTC_79cae
FAILED: cmTC_79cae
: && /usr/bin/cc -Wall -Wno-sign-compare -O3 -DCHECK_FUNCTION_EXISTS=__atomic_load_1 -rdynamic CMakeFiles/cmTC_79cae.dir/CheckFunctionExists.c.o -o cmTC_79cae   && :
CMakeFiles/cmTC_79cae.dir/CheckFunctionExists.c.o: In function main': CheckFunctionExists.c:(.text.startup+0xc): undefined reference to __atomic_load_1'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
Determining if the fopen64 exist failed with the following output:
Change Dir: /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/ninja cmTC_33bc3 && [1/2] Building C object CMakeFiles/cmTC_33bc3.dir/CheckSymbolExists.c.o
FAILED: CMakeFiles/cmTC_33bc3.dir/CheckSymbolExists.c.o
/usr/bin/cc   -Wall -Wno-sign-compare -O3 -fopenmp  -fPIE -o CMakeFiles/cmTC_33bc3.dir/CheckSymbolExists.c.o -c CheckSymbolExists.c
CheckSymbolExists.c: In function ‘main’:
CheckSymbolExists.c:8:19: error: ‘fopen64’ undeclared (first use in this function); did you mean ‘fopen’?
return ((int*)(&fopen64))[argc];
^~~~~~~
fopen
CheckSymbolExists.c:8:19: note: each undeclared identifier is reported only once for each function it appears in
ninja: build stopped: subcommand failed.
File /home/shunfan/Documents/mxnet/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c:
/* */
#include <stdio.h>
int main(int argc, char** argv)
{
(void)argv;
#ifndef fopen64
return ((int*)(&fopen64))[argc];
#else
(void)argc;
return 0;
#endif
}
To Reproduce
cmake -DUSE_CUDA=1 -DUSE_TENSORRT=1 -G Ninja ..
Environment
----------Python Info----------
Version      : 3.6.9
Compiler     : GCC 8.4.0
Build        : ('default', 'Oct  8 2020 12:12:24')
Arch         : ('64bit', 'ELF')
------------Pip Info-----------
Version      : 20.2.4
Directory    : ~/.local/lib/python3.6/site-packages/pip
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
Platform     : Linux-5.4.0-56-generic-x86_64-with-Ubuntu-18.04-bionic
system       : Linux
node         : shunfan-pc
release      : 5.4.0-56-generic
version      : #62~18.04.1-Ubuntu SMP Tue Nov 24 10:07:50 UTC 2020
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              6
On-line CPU(s) list: 0-5
Thread(s) per core:  1
Core(s) per socket:  6
Socket(s):           1
NUMA node(s):        1
Vendor ID:           GenuineIntel
CPU family:          6
Model:               158
Model name:          Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz
Stepping:            10
CPU MHz:             3999.147
CPU max MHz:         4100.0000
CPU min MHz:         800.0000
BogoMIPS:            5799.77
Virtualization:      VT-x
L1d cache:           32K
L1i cache:           32K
L2 cache:            256K
L3 cache:            9216K
NUMA node0 CPU(s):   0-5

Please provide the build configuration log.
cc @Zha0q1 due to the  CheckSymbolExists.c:8:19: error: ‘OPENBLAS_USE64BITINT’ undeclared (first use in this function); did you mean ‘OPENBLAS___64BIT__’? above

Please provide the build configuration log.
cc @Zha0q1 due to the  CheckSymbolExists.c:8:19: error: ‘OPENBLAS_USE64BITINT’ undeclared (first use in this function); did you mean ‘OPENBLAS___64BIT__’? above

Were you talking about CMakeOutput.log? It is quite long, I've uploaded to Drive and shared with this link. Thanks,

I mean the output of the cmake command prior to running make or ninja. Sorry for the confusion

I mean the output of the cmake command prior to running make or ninja. Sorry for the confusion

Thanks for the help. All I did was:
cp config/linux_gpu.cmake config.cmake

mkdir build && cd build

cmake -DUSE_CUDA=1 -DUSE_TENSORRT=1 ..

Yes, please provide the output of cmake -DUSE_CUDA=1 -DUSE_TENSORRT=1 .. with an empty build directory

cmake -DUSE_CUDA=1 -DUSE_TENSORRT=1 ..

Hi, I shared a file of full output in here, thanks!

You can see that -- Configuring incomplete, errors occurred!.
Please install gfortran, for example via sudo apt install gfortran as the /usr/lib/x86_64-linux-gnu/libopenblas.a uses the libgfortran.
cc @mseth10 do you think we should detect libgfortran even if gfortran compiler is not present?

We didn't have to install gfortran separately before. But now with @Zha0q1 's recent commit on statically linking openblas, we have to install it separately. @Zha0q1 can you update the build instructions with this update?

It is all working fine now, thanks a lot!

Please provide the build configuration log.
cc @Zha0q1 due to the  CheckSymbolExists.c:8:19: error: ‘OPENBLAS_USE64BITINT’ undeclared (first use in this function); did you mean ‘OPENBLAS___64BIT__’? above

I think this is how cmake checks if a symbol exists. It tries to compile some dummy project and see it it succeeds

I will update the build from source doc in instruct the user to install gfortran then


CP2K fatal error getting atomic numbers

When processing files like supercell.xyz in the following entry, the parser fails in get_atomic_number, not finding the correct atomic number.

@ladinesa Naive question, but the function's logic looks contrived to me.
In this error message, there are element labels, but no atomic numbers. The former should suffice to populate.
@aalbino2 Is a user allowed to freely switch between element labels and atomic numbers in CP2K?

Hi @ndaelman-hu,
there is the case in which you declare multiple kinds with same atomic number and different electronic structure, e.g., alpha and beta polarized carbon atoms. In that case the label is automatically set to C1 and C2

Hi @ndaelman-hu, there is the case in which you declare multiple kinds with same atomic number and different electronic structure, e.g., alpha and beta polarized carbon atoms. In that case the label is automatically set to C1 and C2

Ow, that's superuseful to know, thx! However, is the user obliged to identify them as carbon C*?
Please link any documentation that you know of.

This is an example of the input file section where the kinds are defined, so you have an element attribute that is the C in thi scase and this one can't vary from the available set of atoms existing
     &KIND C1
       BASIS_SET DZVP-MOLOPT-SR-GTH
       ELEMENT C
       POTENTIAL GTH-PBE-q4
       &BS  T
         &ALPHA
           NEL  2
           L  1
           N  2
         &END ALPHA
         &BETA
           NEL  -2
           L  1
           N  2
         &END BETA
       &END BS
     &END KIND
     &KIND C2
       BASIS_SET DZVP-MOLOPT-SR-GTH
       ELEMENT C
       POTENTIAL GTH-PBE-q4
       &BS  T
         &ALPHA
           NEL  -2
           L  1
           N  2
         &END ALPHA
         &BETA
           NEL  2
           L  1
           N  2
         &END BETA
       &END BS
     &END KIND

Okay, then based on your example and the docs, I'm going to derive elements from the name (when given).

Okay, the extraction function wasn't the problem, but rather the modified header...
Still, I think the former is better now too.


Is arm linux supported?

I have test the demo under ubuntu x64, but I don't have a arm linux system right now, so I was wondering, is this system also work under arm linux?
hope to receive you advice.

Hi,
basically, it does - however, since I haven't implemented NEON SIMD intrinsics so far, you can only use the scalar mode.


cors error

I think we need to add some headers

@mehmetkose could you provide the URL you've passed to oembed-parser, I believe that the oembed provider always enabled CORS because that's public service.

Uh now it's working, I'm not sure how I got there. Thanks anyway!

Uh now it's working, I'm not sure how I got there. Thanks anyway!


fix JSON encoding XSS vunerability

The to_json filter tag did not escape HTML special characters within strings meaning that any resource which had, e.g., a description with closing

2.1 is now in master, so you are free to use 2.1 new features

Unfortunately json_script isn't a drop-in replacement. It's worth seeing if we can migrate to it in future but, at least in the case of JSON-LD, we still need a customised version since json_script doesn't allow us to set the type of the script tag.


Captions of PlantUML diagrams don't respect markdown formatting

See https://github.com/jgm/pandocfilters/issues/81

Fixed (at least for LaTeX) with https://github.com/fuhrmanator/log210-notes-de-cours/commit/221b9374df8bdf2daa5736a764c93d50954f45a6#diff-bbb1f26b8a6a11294322c68624e4f5459c77f2366ea38b8485dda6524adf7243

Thanks to @dilawar for help at https://github.com/dilawar/dilawar/commit/ada7ee93a839e3fabf14e981ac8ce307984e781a#diff-de78ea8273d8480556c6ca3761357ae0bb8a0e4bfdfcc2c4937b31271001e2a6

I am glad I could help.


JS: add query js/regex/missing-regexp-anchor

This adds another query for a common regular expression mistake, this time for missing anchors. Similar to js/incomplete-hostname-regexp, the query is identifies suspicious regular expression patterns, but it does not check that the patterns are used in a security setting.
An earlier iteration of the query used the RegExp AST classes, but I am happier with the string heuristic implementation as it is reasonably simple, requires less QL code, and flags more results (due to support for string literal results). For the record, using the RegExp AST classes during prototyping is very useful for homing in on the syntactic cases we really care about.
The results have been a mixed bag, and required a bunch of iterations, but I think they are quite good now. Unfortunately, most results are not security relevant. If we split the query into one that flags URL patterns, and one for non-URL patterns, then the flagged URL patterns will be mostly security relevant. I think we should run the query on LGTM, but not show it by default for a while, and then decide if we want to split, sharpen or display the query.
Running js/incomplete-hostname-regexp with the refactored RegExpPatternSource shows no performance change.
The performance for the new query is excellent. Running the query next to js/incomplete-hostname-regexp appears to have no measurable overhead (in fact there seems to be a speedup, I am trying to reproduce the numbers. Update: the speedup is not reproducible, but there still is no overhead).
I would like this to make it to 1.21, but it will probably not make it through the doc review today.

Some results for the curious.

Opened https://jira.semmle.com/browse/ODASA-7961 with some ideas for future improvements that are out of scope for this round

Performance evaluation links.

js/incomplete-hostname-regexp vs js/incomplete-hostname-regexp+js/regex/missing-regexp-anchor: irreproducible speedups

small reproduction attempt at big-apps.slugs, the performance is now just unchanged if comparisons are made for the big-apps.slugs entries

For results, I refer to https://lgtm.com/query/3732291658122574474/ which has the most up to date whitelist.

Evaluation: https://git.semmle.com/esben/dist-compare-reports/tree/js/unanchored-url-regex_1559437972495

All comments addressed.

Why me?

Oops, sorry, muscle memory.

All comments addressed

PR checks are failing due to rogue tabs in the query examples.


ion-sfu update offer to client. but the offer is longer and longer as the participant come and leave

start ion-sfu server.

go build ./cmd/signal/json-rpc/main.go && ./main -c config.toml

disable video and enable audio in examples/pubsubtest/main.js to publish and subscribe audio
open twice index.html. one publish and one subscribe. and the subscriber will receive offer from ion-sfu
open more and more pages and publish audio. the subscriber receives more and more mediaSection in offer
close those publisher one by one. the subscriber receieves offers but the count of mediaSection don't decrease, but those closed with a=inactive.

Apart from the network cost while negotiating  sdp, are there any disadvantages to longer and longer offers?
I don't known why don't remove those media section directly. Could you please help confirm why? Thanks a lot!

Hello @r1a2y3, thats the default behavior of webrtc, before we have experimented with recycling transceivers, but different browsers behaved different causing issues.
So besides the fact that you are getting big SDP there is no other side effect.

Hello @OrlandoCo ,thanks for your patient answer.
btw, I experimented with Google Meet. It won't get a big SDP while participants come and leave. So I think there may have a solution to  resolve it.
In a larger meeting. the big SDP may have tens of thousands lines, it's Intolerable and inconvenient to log out and troubleshoot.


Migrate config/params TOML to YAML

This may not belong here, but I've found myself having to manually diff changes between config.toml and params.toml between Wowchemy versions.
I'm not sure what the failure modes might be, but adding new config/params options anytime those new changes are made would be good, I think?
The use of migrating from TOML to YAML is that ruamel.yaml supports round-trip comments. I don't believe the toml pypi package supports that.

It's interesting that you mentioned this.
We use YAML for some internal config.yaml files, such as for the Test site at https://github.com/wowchemy/wowchemy-hugo-modules/blob/master/wowchemy/test/config.yaml , primarily to keep things simpler so both front matter and config are in YAML. We have been converting the remaining TOML front matter in templates to YAML as GitHub still does not support TOML (even though TOML was created by and named after Tom, the founder of GitHub).
We have worked on some ideas for an upgrader tool in the past. If we were to create a new repo for such a tool, we would probably use either Go (to avoid any client dependencies and align with Hugo) or JS (to create a web-based tool, similar in some respects to how Hugo Server spawns a browser window). Would you be interested in contributing to such a project?


[DO NOT MERGE] Correct Jenkins job credentials path

[DO NOT MERGE] Failing tests.
Because that's where it is: https://github.com/alphagov/govuk-puppet/blob/master/modules/govuk_jenkins/templates/google_api/credentials.json.erb
https://trello.com/c/B1dtJ48q/420-improve-alerting-information-on-the-platform-health-dashboard

The issue was not coming from the path being wrong, but a JSON parsing issue on the private key. I'll open another PR to correct that.


Flickering when outputting characters on a white background (ncurses on WSL)

Windows Terminal version
1.19.10821.0
Windows build number
10.0.22631.0
Other Software
I'm using ncurses on WSL.
WSL kernel: <IP_ADDRESS>-microsoft-standard-WSL2
ncurses: 6.4
gcc: 13.2.1
glibc: 2.38
Steps to reproduce
Extra info
This (bug) reproduces every single time that I run the following program. I have tried changing the rendering settings, i.e. using AtlasEngine and not using AtlasEngine. Also using software rendering and not etc. All of the options give exactly the same result as shown in the video below. Note that the program below never ever prints something with a black background. It only every outputs characters with a white background. It works as expected on all terminals (that I have tried) on a native Linux installation. I have tried commenting the code, in the case that you are not familiar with the ncurses library. If you need more information than provided I will be happy to help.
Steps to reproduce
Compile and run the code that I have submitted. The code is compiled with
gcc -O2 main.c -lncurses -ltinfo
Depending on if your wsl-distribution has built the tinfo library separetely or not it is possible that you should instead compile it with
gcc -O2 main.c -lncurses
This is the most minimal example I could find that reproduces the bug. It was found while working on a larger project which outputs scrollable text.
#include <unistd.h>
#include <ncurses.h>

const int arr[262] = {
	0, 19, 18, 17, 0, 17, 17, 16, 16, 17, 19, 17, 19,
	0, 32, 0, 42, 0, 57, 25, 38, 23, 9, 0, 23, 21, 20,
	17, 39, 39, 35, 17, 40, 35, 57, 35, 17, 22, 22, 17,
	51, 39, 35, 17, 61, 35, 41, 35, 17, 22, 18, 36, 39,
	47, 31, 17, 33, 17, 36, 22, 16, 22, 9, 0, 52, 70,
	39, 1, 0, 35, 120, 19, 0, 34, 0, 18, 0, 29, 32, 74,
	26, 30, 0, 31, 49, 50, 41, 45, 17, 0, 55, 33, 0, 49,
	42, 83, 71, 50, 84, 45, 87, 30, 47, 107, 25, 59, 67,
	67, 63, 72, 37, 43, 86, 36, 82, 51, 86, 25, 42, 74,
	87, 25, 30, 83, 74, 25, 36, 74, 25, 45, 87, 25, 17,
	0, 32, 9, 26, 19, 1, 0, 24, 18, 35, 23, 17, 1, 0, 38,
	20, 22, 31, 23, 9, 28, 25, 30, 0, 45, 0, 15, 37, 30,
	32, 9, 14, 43, 0, 17, 1, 0, 64, 25, 44, 23, 9, 0,
	103, 33, 25, 9, 0, 18, 118, 0, 27, 27, 32, 22, 25,
	30, 27, 30, 22, 9, 0, 26, 26, 1, 0, 46, 28, 28, 22,
	21, 41, 25, 27, 0, 23, 60, 61, 11, 60, 1, 0, 43, 37,
	35, 47, 42, 71, 25, 0, 18, 32, 47, 33, 27, 38, 25,
	30, 39, 33, 28, 0, 49, 0, 64, 78, 88, 77, 77, 86, 75,
	81, 83, 84, 33, 0, 48, 41, 9, 0, 41, 27, 40, 9, 61,
	52, 0, 17, 1,
};

int main(void) {
	/* Start ncurses. */
	initscr();
	curs_set(0);

	/* Initialize colors and create the necessary color with white
	 * background. */
	start_color();
	init_pair(1, COLOR_BLACK, COLOR_WHITE);

	/* Set the color to (COLOR_BLACK, COLOR_WHITE). */
	attrset(COLOR_PAIR(1));

	/* This loop fills the entire screen with our background color.
	 * Our color was set to (COLOR_BLACK, COLOR_WHITE) on the previous
	 * line which means that the entire screen will be filled with white.
	 */
	for (int y = 0; y < LINES; y++)
		mvhline(y, 0, ' ', COLS);
	refresh();

	/* This loop successively fills the screen with the char 'x'
	 * in random places. There is nothing special about the character
	 * 'x', it works the same with any character (except the space
	 * character ' ').
	 */
	for (int i = 0; i < 100; i++) {
		/* Add the chars. */
		for (int y = 0; y <= COLS; y++)
			mvaddch(y, arr[y + i], 'x');

		/* Refresh the screen and sleep. */
		refresh();
		sleep(1);
	}

	/* End ncurses. */
	endwin();
}

Expected Behavior
I'm expecting it not to flicker with random black characters when I never output those characters.
Actual Behavior
It flickers with random black characters which I never output.
https://github.com/microsoft/terminal/assets/90049674/264ff3c4-6055-40e6-a9fc-800cd2122666

I have a small piece of additional information. If I refresh the screen more frequently the flickering disappears. That is, if I for example call refresh() after each call to mvaddch() on line 56 in the provided code.
It seems to be that the flickering starts when I output many characters on different lines which I all send to the terminal at the same time with a call to refresh().
I can thus get around the issue and get my program working as intended. However the bug still stands because I shouldn't have to call refresh() more than once.


Feature request

Harbor developer,
Clair project is a project that analyse the container after push for security vulnerabilities.
add to harbor CoreOS\Clair will upgrade harbor to a new level of container registry for on prem.
My email is<EMAIL_ADDRESS>

Wow i saw that you already did that COOL (-: now i need to learn how to install it.

It's a WIP, targeting next release hopefully an RC will be around end of Jul

Closing the issue as the original question is answered.


Compose preview changes

Two things:

Increase opacity of composebox buttons
Change "Preview" to be a link/text button, instead of a icon

Testing plan: Manually tested
GIFs or screenshots:

I merged the opacity commit as 5c4db6ec41d5b76f4ee9c73d73d4bc9b9d213e67; I figure we should let the other commit get discussion for a bit longer.

Closing this, since we're currently pursuing the tabbed approach for Write/Preview, and if that fails, it'll be easy to reopen this.


invalid range syntax errors in some circumstances

Description
I get SyntaxError in some rules as simple as letter ::= [A-Z]. It seems to be caused by another rule being just after it, but it does not seem that simple. I attach a program to show the cases I checked.
The error produced is like this:
$ irb --nocolorize --simple-prompt -r ebnf
>> EBNF.parse("letter ::= [A]\nx ::= 'X'")
Traceback (most recent call last):
       10: from /usr/local/bin/irb:23:in `<main>'
        9: from /usr/local/bin/irb:23:in `load'
        8: from /var/lib/gems/2.7.0/gems/irb-1.2.7/exe/irb:11:in `<top (required)>'
        7: from (irb):1
        6: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf.rb:26:in `parse'
        5: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf.rb:26:in `new'
        4: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf/base.rb:124:in `initialize'
        3: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf/base.rb:124:in `new'
        2: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf/parser.rb:265:in `initialize'
        1: from /var/lib/gems/2.7.0/gems/ebnf-2.1.1/lib/ebnf/parser.rb:302:in `rescue in initialize'
SyntaxError (ERROR [line: 1] syntax error, expecting :HEX, :SYMBOL, :O_RANGE, :RANGE, :STRING1, :STRING2, "(", :primary, :postfix, :seq, :alt, :expression (found "[A]\nx ::= 'X'"))
>>

To reproduce
Run the attached ebnf_debug.rb.txt to see examples and counterexamples.
§ denotes newline as you can check in the program.
For me it prints:
$ ruby ebnf_debug.rb
Ruby: 2.7.0,  ebnf: 2.1.1
letter ::= [A-Z]                          : OK
letter ::= [A]                            : OK
digit ::= [0-9]                           : OK
letter ::= [A-Z] §                        : OK
letter ::= [A-Y] | 'Z'                    : OK
word ::= [A-Z]+                           : OK
letter ::= [A-Z] /* x */                  : OK
letter ::= [A-Z]       § digit ::= [0-9]  : SyntaxError
digit ::= [0-9]        § letter ::= [A-Z] : SyntaxError
letter ::= [A-Z]       § word ::= letter+ : SyntaxError
letter ::= [A-Z]       § x ::= 'X'        : SyntaxError
letter ::= [A]         § x ::= 'X'        : SyntaxError
letter ::= [A] | 'B'   § x ::= 'X'        : OK
letter ::= 'B' | [A]   § x ::= 'X'        : SyntaxError
letter ::= [A] | [B]   § x ::= 'X'        : SyntaxError
letter ::= [A] x       § x ::= 'X'        : OK
letter ::= [A-Z] x     § x ::= 'X'        : OK
letter ::= x [A]       § x ::= 'X'        : OK
letter ::= x [A-Z]     § x ::= 'X'        : SyntaxError
letter ::= [A] /* x */ § x ::= 'X'        : OK
letter ::= [A-Z] /* */ § x ::= 'X'        : OK
letter ::= [^A]        § x ::= 'X'        : OK
                                          : OK

Expected behavior
A successful parse of each above indicated example.
Desktop (please complete the following information):

OS: Ubuntu 20.04.1 LTS
Browser: (?) not applicable

This comes from a conflict in the EBNF grammar between LHS (of rule) and RANGE.
[11] LHS        ::= ('[' SYMBOL ']' ' '+)? SYMBOL ' '* '::='
[14] RANGE      ::= '[' ((R_CHAR '-' R_CHAR) | (HEX '-' HEX) | R_CHAR | HEX)+ '-'? ']' - LHS

In spit of the - LHS it can still get confused.
Many (most) EBNF grammars use the [1] identifier before the rule name, but it is not officially part of the grammar, so the LHS terminal tries to accommodate this.
I think the fix is to branch the grammar depending on if it starts with '[' SYMBOL ']' or not, for example:
[1] ebnf ::= ((rulei (rulei | declaration)*) | (rules (rules | declaration)*))?

[3i] rulei ::= IDENTIFIER SYMBOL '::='

[3s] rules ::= SYMBOL '::='

That way, it should avoid the ambiguity.

As a workaround, before I get around to an update, you can try adding comments after the rule, or introducing the [xx] identifier prefix before the rule. You might also have some success using --input-format native, which uses a custom parser that has it's own issues, but may work for your particular grammar.

OK, thanks, I get it... I am new to this whole topic, and I was only looking at https://www.w3.org/TR/REC-xml/#sec-notation which does not mention the identifier. Maybe that is a point to clarify in the documentation?
And it seems that in the format recognized by this gem, all whitespace are created equal (is it preprocessed?), and this makes a newline acceptable also within the LHS.
I think another pretty unobtrusive way to remove this ambiguity (aside from your suggestion) could be requiring a line break before the LHS but not allowing a line break within the LHS - the latter seems already the case based on your comment, so I assume it is only applied after whitespace preprocessing. This way the rule identifier would not need to be uniformly used or unused, one could decide on a rule by rule basis.

The XML spec is the only citable reference for W3C EBNF, but many specs have taken liberties with the syntax. I document what I believe is colloquial EBNF in the README, this includes the use of [xxx] to identify productions by number, but also the merger of RANGE and ENUM, as is common, along with the @pass and @terminals declarations. I had a mind to create a CG to try to lead to a normative description of W3C EBNF, but haven't gotten around to it.
From the XML spec, whitespace is discarded between non-terminal productions, and newlines have no particular meaning. Within a terminal, you can include whitespace, and a terminal defined as an expression of other terminals has no implicit whitespace. In the existing grammar, LHS is a terminal, and doesn't allow whitespace, but it still leads to the ambiguities.
The update I'm working on separates ID from LHS, and would allow whitespace (including newline) between them, but it would branch depending on if the first non-terminal encountered is ID ([xxx]) or not. But, updating the grammar in the gem is non-trivial.
I did work on this previously, but wasn't able to come up with a consistent grammar which was unambiguous; going into the details of that would be challenging to reconstruct now, but if you turn on debugging, you can sort of figure it out.
As I said, in the mean time, there are some workarounds you might try.

My mistake, now I remember skimming through that part of the README, but later kept reading the W3C page instead.
Now that the cause is clear, the workarounds are clear and easy too 8-)

Documented, as there is no good way to make an unambiguous grammar having both RANGE and rule identifiers, but it can be worked around.


Use asdf to support requesting specific elixir/erlang versions

For reference, see how https://github.com/GoogleCloudPlatform/ruby-docker uses rbenv.

Done as of release elixir-2017-11-29-234522


P3 preview

Connects to https://github.com/vaadin/components-team-tasks/issues/368

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: manolo:white_check_mark: YuriyVaadin:white_check_mark: tomivirkki:x: web-padawanYou have signed the CLA already but the status is still pending? Let us recheck it.

Reviewed 1 of 6 files at r1, 1 of 2 files at r2, 1 of 4 files at r3, 3 of 4 files at r4, 1 of 1 files at r5.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable

bower.json, line 35 at r5 (raw file):

    "iron-component-page": "^3.0.0",
    "iron-meta": "^2.0.0",
    "vaadin-demo-helpers": "vaadin/vaadin-demo-helpers#p3-preview",

Should not mention p3-preview anymore

Comments from Reviewable

package.json, line 3 at r5 (raw file):

{
  "name": "@vaadin/vaadin-icons",
  "version": "4.1.4-pre.2",

should not be changed

Comments from Reviewable

Review status: 5 of 7 files reviewed, 2 unresolved discussions (waiting on @manolo and @tomivirkki)

bower.json, line 35 at r5 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Should not mention p3-preview anymore

Done.

package.json, line 3 at r5 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
should not be changed

Done.

Comments from Reviewable

bower.json, line 35 at r5 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Done.

The line below still has it

Comments from Reviewable

Review status: 5 of 7 files reviewed, 2 unresolved discussions (waiting on @manolo and @tomivirkki)

bower.json, line 35 at r5 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
The line below still has it

Waiting until vaadin-text-field is released

Comments from Reviewable

Reviewed 1 of 6 files at r1, 1 of 2 files at r2, 2 of 4 files at r4, 1 of 1 files at r5, 2 of 2 files at r6.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable


led.py not detecting neopixel

I am working on a raspberry pi 3+ and I had everything working perfectly until I got to running 'visualization.py'. I tested everything with the 'strandtest.py' from rpi_ws281x and it worked perfectly. However, when I try to run 'visualization.py', I get the error:
Traceback (most recent call last):      File "visualization.py", line 9, in <module>           import led      File "/home/pi/audio-reactive-led-strip-master/python/led.py", line 15, in <module>           strip = neopixel.Adafruit_NeoPixel(config.N_PIXELS, config.LED_PIN, AttributeError: module 'neopixel' has no attribute 'Adafruit_NeoPixel'
Everything is downloaded and run through Python 3.7.3
Not sure what this could be as the library was linking up just fine for the strandtest which calls on neopixel. The 'audio-reactive-led-strip', 'adafruit-circuitpython-bundle-6' and 'rpi_ws281x' all reside in the 'home' folder
Please let me know if I need to add anymore information to aid in support

把rpi_ws281x/python/neopixel.py拷贝到audio-reactive-led-strip/python/ 中即可解决~~
cp /path/../rpi_ws281x/python/neopixel.py /path/../audio-reactive-led-strip/python/
python visualization.py
和版本无关

把rpi_ws281x/python/neopixel.py拷贝到audio-reactive-led-strip/python/ 中即可解决~~
cp /path/../rpi_ws281x/python/neopixel.py /path/../audio-reactive-led-strip/python/
python visualization.py
和版本无关

Ran in to the same problem, thanks wangwy1015 for the help.
To clarify what they mean, in your audio-reactive-led-strip/python directory, create a new neopixel.py file and copy the contents from here:
https://raw.githubusercontent.com/jgarff/rpi_ws281x/master/python/neopixel.py
I'm not actually sure if the neopixel libraries need to be installed through pip, but I am able to run visualization.py using "sudo python3 visualization.py" and I had run the neopixel installation commands earlier, from here:
https://learn.adafruit.com/neopixels-on-raspberry-pi/python-usage

Ran in to the same problem, thanks wangwy1015 for the help.
To clarify what they mean, in your audio-reactive-led-strip/python directory, create a new neopixel.py file and copy the contents from here:
https://raw.githubusercontent.com/jgarff/rpi_ws281x/master/python/neopixel.py
I'm not actually sure if the neopixel libraries need to be installed through pip, but I am able to run visualization.py using "sudo python3 visualization.py" and I had run the neopixel installation commands earlier, from here:
https://learn.adafruit.com/neopixels-on-raspberry-pi/python-usage

having this same issue now, is there a new fix for this? the link to the contents for the neopixel.py linked above is dead so just curious what to do

@kylerobert425 try out this one, I think it's the same file:
https://github.com/richardghirst/rpi_ws281x/blob/master/python/neopixel.py

Ok yup that works, thanks!


[Bug] Unify the timezone

When we get default system time zone, it will return PRC, which is not supported by us, thus
will cause dynamic partition create failed. Fix #3919
This CL mainly changes:

Use a unified method to get the system default time zone
Now the default variable system_time_zone and time_zone is set to the default system
time zone, which is Asia/Shanghai.
Modify related unit test.
Support time zone PRC.

LGTM


Interface not centered after switching to/from external display (Cube interface)

If Cube is centered on my 15" laptop, & then I connect 27" display, the next time I open QS it appears in lower left. If I recenter on the external display, the next time I launch it on the laptop it moves to the far top right. http://pic.twitter.com/66nkmtD9uf
Bezel interface works fine, maybe because it isn't draggable like Cube is? I'd be happy to lose draggability in Cube to keep it centered - don't want to drag it anyway.
Using QS 1.1.3 (4007) on OS X 10.9.1
I'd be comfortable copying and pasting code from another interface to patch my copy of Cube if that's possible, but I don't know how without instructions...
Thanks!

Unfortunately Cube interface is no longer officially supported, and so will no longer be updated.


Not possible to find a valid npm package

It is not possible to find this package via npmsearch
https://www.npmjs.com/package/hain-plugin-npm
http://npmsearch.com/?q=hain-plugin-npm
I have no idea where is the problem. Is it the package name?

Sometimes it takes a while for new packages to get indexed. http://npmsearch.com/?q=hain-plugin-npm is now available.


Added foreign_key to endorsement and subscription

Story: https://www.pivotaltracker.com/story/show/133967733
This guarantees referential integrity on db so prevents article_endorsement and article_subscription exist without proper reference to an article.
QA:

Add new article and click subscription and endorsement
Delete the article and the subscription and endorsement also destroyed

make sure to run the migration and push up schema.rb changes


For posterity, confusion around custom sprites (ES6 Class) preUpdate

Leaving this here, hopefully will help someone out. Not sure if it's technically a bug:
After updating to 3.17 all the animations in my derived Sprites would only play the first frame, and then stop updating. In the past you use to need to use
this.scene.add.existing() when using a custom Sprite.
You also had to make sure your Sprite had a preUpdate method, which all of mine did.
As of 3.17 however this is no longer necessary! If you had things setup the way I did with preUpdate methods in your derived Sprites, animations on those sprites will not work unless you manually call
this.anims.update() inside your custom preUpdate method!
TLDR; If your ES6 Sprites animation are not working, make sure you remove any preUpdate method you may have added OR manually call this.anims.update() inside preUpdate.

Some Game objects have preUpdate method, just add super.preUpdate() in custom preUpdate method.


Make introduction of important people

While working on the index, I found Claude 3 Opus's comment interesting, so I decided to write it down as a note for future versions.

Some key people are mentioned repeatedly in different contexts, like Alexis de Tocqueville, Danielle Allen, John Dewey, etc. Brief bios when first introduced would help readers not already familiar with their work.

I feel it is very helpful of most readers.

 Gov4Git notices
On Thursday, 11-Apr-24 16:13:08 UTC by Gov4Git dev
Notice 6jpdox
Started managing this issue, using the Plural Management Protocol v1,  as Gov4Git concern 872 with initial priority score of 0.000000.
This project is managed by Gov4Git, a decentralized governance system for collaborative git projects.
To participate in governance, install the Gov4Git desktop app.
Notice hz3ltl
This issue's priority score is now 0.000000.
The cost of priority is 0.000000.
The projected bounty is now 0.000000.
Notice zbl6ft
The set of eligible proposals claiming this issue is empty.

 Gov4Git notices
On Wednesday, 24-Apr-24 16:34:39 UTC by Gov4Git dev
Notice izrakl
This issue's priority score is now 90000.000000.
The cost of priority is 90000.000000.
The projected bounty is now 90000.000000.
Notice g3j7bj
The set of eligible proposals claiming this issue is empty.


adds 'delete' and 'cordon' Operations to KRIB

adds delete and cordon Operations to the "operators" patterns

Tested to work.  Can we please get this reviewed and pushed to tip ASAP?

:+1:


Remove extra logging from getFetch()

Problem
getFetch() contained an extra log statement which is executed every time the API is used.
Solution
Remove the log
Type of Change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Infrastructure change (CI configs, etc)
[ ] Non-code change (docs, etc)
[ ] None of the above: (explain here)

Test Plan
Ran code that accessed and interacted with Pinecone (fetched index, queried vectors)

fixes #254

@aulorbe can we merge this 🥹

+1

+1

merge please

+1 Can we please get this merged in


Add experimental support for LMDB

On top of #468
LMDB is a high-performance embedded transactional database in the form of a key-value store. It's minimal and focused on modern hardware, 64bit CPU & SSD.
This PR makes use of lmdb optional, by passing the parameter -db=lmdb.
Performance results on my development machine:
Rotating disk:
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=lmdb -reindex-chainstate
real    63m39,459s
user    15m29,719s
sys     1m2,658s
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=leveldb -reindex-chainstate
real    24m40,074s
user    23m9,927s
sys     2m27,688s
SSD:
Testes with a 6 year old SSD disks.
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=lmdb -reindex-chainstate;
real    17m56,519s
user    14m21,408s
sys     1m52,948s
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=leveldb -reindex-chainstate
real    25m36,895s
user    23m43,988s
sys     4m2,225s
SSD without forced OS flushing
I also added the option -dbsafemode, which, when disabled removes forced disk flush for every transaction commit. This gives performance boost, at the cost of higher risk of db corruption in case of abnormal program termination.
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=lmdb -reindex-chainstate -dbsafemode=0
real    16m49,922s
user    14m21,606s
sys     1m36,634s
Smaller dbcache
Because of the memory mapping mechanism, the performance is less dependent on the size of dbcache set. I suspect the performance degradation here is due to CPU being busy with serialization and not I/O blockage.
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=lmdb -reindex-chainstate -dbsafemode=0 -dbcache=100
real    17m28,172s
user    14m13,753s
sys     2m5,633s
$ time ./src/bitcoind -stopatheight=300000 -testnet=0 -db=leveldb -reindex-chainstate -dbsafemode=0 -dbcache=100
real    33m30,324s
user    32m24,312s
sys     7m36,293s
Future work:

Auto-grow max database size. LMDB requires that you set max database size up front. This can be changed under certain conditions after opening the database.
Warn if used with rotating disks.
Implement utxo size estimation
Investigate more optimal use of LMDB properties

Can we use the lock-free parallel read support?
Can we remove dbcache altogether

[qa] Fix qa tests with leveldb assumptions (bip100-sizelimit, blockchain, walletbackup)
[qa] Remove cache if it's for the wrong db backend

Except for these, all qa tests pass:

bip100-sizelimit.py - bypasses framework for bitcoin.conf, thus uses leveldb
blockchain.py - tests for size, not implemented for lmdb
walletbackup - assumes blockstate folder

laanwj has an experimental branch replacing leveldb with lmdb in 2016, here is their notes: https://github.com/laanwj/bitcoin/blob/2016_04_mdb/README.md. This work is in part based on laanwj's implementation. LMDB is also used by Monero.

Added dynamic DB mapsize adjusting and utxo size estimation.

@kyuupichan Can you explain a little about the scenario where you found LMDB did not perform well? Do you recall whether you were using rotating disk or SSD?

@dgenr8 I was trying to use LMDB for (address) history storage in ElectrumX.  We had it for a while but couldn't make it perform well; once we got to height about 250k it was crawling compared to leveldb.  The access pattern of history is purely appending new key-values; no lookup at all is done.  i.e. it's write-only.  I used a spinning disk.  My results were confirmed by @bauerj; maybe he can chime in on his configuration.
See also https://github.com/kyuupichan/electrumx/commit/c958b3af494094aa9b99e7c153344444c0b51cb2.
It is possible we were not using LMDB in the right way or optimally, but we did try.

Added disk space check to DBResizeManager. Added unit tests for DBResizeManager.


Windows: MSB4016 error on build

Hi, I'm following the instructions from the README but I'm running into an MSBuild failure when I try to run dotnet-releaser build.  I've put together a minimal repro here.
Steps to Reproduce
Create a console app:
dotnet new console

Initialize the dotnet-releaser config:
dotnet-releaser new --project ReleaserRepro.csproj

Run dotnet-releaser build, observe the "Failing to run dotnet msbuild" error:
❯ dotnet-releaser build --force dotnet-releaser.toml
info: dotnet-releaser[0]
      Loading configuration from C:\Users\reill\source\releaser-repro\dotnet-releaser.toml
fail: dotnet-releaser[1]
      The changelog.path was not setup
info: dotnet-releaser[2]
      Adding default profile for platform [win-x64] with [Zip] package
      Adding default profile for platform [win-arm] with [Zip] package
      Adding default profile for platform [win-arm64] with [Zip] package
      Adding default profile for platform [linux-x64] with [Deb, Tar] packages
      Adding default profile for platform [linux-arm] with [Deb, Tar] packages
      Adding default profile for platform [linux-arm64] with [Deb, Tar] packages
      Adding default profile for platform [rhel-x64] with [Rpm, Tar] packages
      Adding default profile for platform [osx-x64] with [Tar] package
      Adding default profile for platform [osx-arm64] with [Tar] package

fail: dotnet-releaser[3]
      Failing to run dotnet msbuild -p:Configuration=Release -p:CustomAfterMicrosoftCommonTargets="C:\Users\reill\.dotnet\tools\.store\dotnet-releaser\0.1.0\dotnet-releaser\0.1.0\tools\net6.0\any\dotnet-releaser.targets" -nologo -logger:BinaryLogger,"C:\Users\reill\.dotnet\tools\.store\dotnet-releaser\0.1.0\dotnet-releaser\0.1.0\tools\net6.0\any\dotnet-releaser-binary-logger.dll";"C:\Users\reill\AppData\Local\Temp\tmpC534.tmp.binlog" -t:DotNetReleaserGetPackageInfo C:\Users\reill\source\releaser-repro\ReleaserRepro.csproj. Reason: MSBUILD : error MSB4016: The build stopped unexpectedly because the "ReusableLogger" logger failed unexpectedly during initialization. Object reference not set to an instance of an object.

Version Info
dotnet-releaser v0.1.0, Windows 11, and the .NET 6 SDK that comes installed with VS 2022 17.1.0 Preview 4:
.NET SDK (reflecting any global.json):
 Version:   6.0.200-preview.22055.15
 Commit:    a3c3a2bf3b

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.22000
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\6.0.200-preview.22055.15\

Host (useful for support):
  Version: 6.0.1
  Commit:  3a25a7f1cc

P.S. Thank you so much for making dotnet-releaser - it fills a really important gap in the .NET OSS ecosystem!

Might be a Windows-specific issue, I'm unable to reproduce it on Linux (Debian 11 x64, .NET 6.0.101).

Thanks for testing. It looks like it's probably the net6.0 preview that is failing with something here.
@KirillOsenkov I'm using StructuredLogger here, but does it ring a bell this error with ReusableLogger?
error MSB4016: The build stopped unexpectedly because the "ReusableLogger" logger failed unexpectedly during initialization. Object reference not set to an instance of an object.

It doesn't, and I can't repro on my machine, but I'd certainly be the right person to investigate this and it's likely my fault somehow :) Do you have a full call stack?
You can use the Child Process Attach VS extension:
https://marketplace.visualstudio.com/items?itemName=vsdbgplat.MicrosoftChildProcessDebuggingPowerTool
Then debug dotnet-releaser.exe with Mixed mode (Managed + Native) in VS, enable Child Process Attach, enable first-chance exceptions (Debug -> Exceptions -> All .NET exceptions), then debug and it will get you a call stack of the exception. Paste that callstack here (and ideally the source location where it happens) and we can investigate further.
It looks like a bug in MSBuild currently (nothing a user is doing should be causing a NullRef in MSBuild), but we need to understand what is going on first (and whether it's MSBuild or the SDK).
Excellent repro steps @rgwood, I appreciate good bugs when I see them!

This is what I see on my machine:

Happy to try debugging in VS but I'm not entirely sure how (I've just been using the CLI outside of VS).
Would I need to clone this repo or is there a way to debug the dotnet-releaser.exe I installed as a global tool?

@KirillOsenkov I've installed the extension but I'm not getting a call stack when I debug. Does this look right?
Mixed-mode debugging enabled:

Child Process Debugging enabled:

All CLR exceptions enabled:

Debugging with F5 fails as expected but VS never breaks on an exception:

There's a way to debug an .exe in VS if you don't have the sources. In VS, open the .exe as a Project (File -> Open Project), it will open as a pseudo-project, you can right-click the "project" in Solution Explorer, open Properties, fill out the arguments, working directory, environment variables, debug engine (has to be Mixed for Child Process Attach to work), then just F5.
This trick is generally useful if you don't have the source.
Or you can attach to an already running process, but it's less convenient and since JIT optimizes all modules on load by default, you won't see any variables in the locals window.

Can you try to uncheck Tools -> Options -> Debugging -> Just my code? But it's strange, I'd expect it to work. When it runs, do you see additional processes showing up in Debug -> Windows -> Processes?
It may be that it isn't really attaching to the spawned dotnet msbuild process?

Also you can try setting the MSBuildDebugEngine environment variable, it will dump binlogs for everything. Not sure if that will help here or not.

Disabling Just My Code worked!
System.NullReferenceException
  HResult=0x80004003
  Message=Object reference not set to an instance of an object.
  Source=StructuredLogger
  StackTrace:
   at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsWriter.GetMessageFlags(BuildMessageEventArgs e, BuildEventArgsFieldFlags flags, Boolean writeMessage, Boolean writeImportance) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsWriter.cs:line 667

oh oh, some reflection going on here 😄 Seems that the field arguments was renamed 2 months ago 😨

Thanks for reproducing the bug @rgwood and investigating it!

Do you have many places like this in StructuredLogger @KirillOsenkov ? Does it mean that to completely protect it against such changes, we would need MSBuild to expose proper public API for these?

Could these reflections be more protected by putting them separately somewhere with a validator that could check frequently with a recent MSBuild DLL?

Ouch.
And fantastic job @rgwood, this is really helpful.
I guess we haven't seen this because not a lot of people use the BinaryLogger copy from StructuredLogger.dll (almost everyone uses the original from MSBuild). Yeah, let's think about what's the right thing to do is here.
I filed https://github.com/KirillOsenkov/MSBuildStructuredLog/issues/556 so feel free to watch that for further development. For now the cheapest tactical fix is to probe for both field names and use the one that's not null.

@xoofx would you please try to update to https://www.nuget.org/packages/MSBuild.StructuredLogger/2.1.617 and see if this fixes the issue?
Thanks!

And to answer your question about why Reflection is necessary - these fields shouldn't be exposed to the general public, but I need access for obscure reasons (binary serialization). Maybe it's worth thinking about opening them up, but in MSBuild land, once we open anything up, we can never change it, so we've been cautious about making more foundational stuff public. The perf improvement two months ago that broke my reflection wouldn't have been possible had we had those fields public.

@xoofx would you please try to update to https://www.nuget.org/packages/MSBuild.StructuredLogger/2.1.617 and see if this fixes the issue?

I couldn't reproduce it as I avoid installing previews on my dev machine. But I will ship a new version of dotnet-releaser with it! Thanks for the quick fix!

I think the error is The changelog.path was not setup in the log above, so in dotnet-releaser. Gonna push a fix because it's supposed to skip it when it is not set.

Confirmed that build succeeds on v0.1.2 if I add this to the config:
[changelog]
publish = false

Great. The changelog issue should be fixed in 0.1.3+

Yup, confirmed that build (usually) works with the default config on v0.1.3. Thank you both for fixing things so quickly!
I am running into occasional build failures related to StructuredLogger but I can raise that in another issue elsewhere if it would help:
info: dotnet-releaser[24]
      Building target platform [linux-arm64] / [tar] package
fail: dotnet-releaser[25]
      Failing to run dotnet msbuild -p:PublishTrimmed=true -p:PublishSingleFile=true -p:SelfContained=true -p:PublishReadyToRun=true -p:CopyOutputSymbolsToPublishDirectory=false -p:SkipCopyingSymbolsToOutputDirectory=true -p:RuntimeIdentifier=linux-arm64 -p:Configuration=Release -p:CustomAfterMicrosoftCommonTargets=C:\Users\reill\.dotnet\tools\.store\dotnet-releaser\0.1.3\dotnet-releaser\0.1.3\tools\net6.0\any\dotnet-releaser.targets -nologo "-logger:BinaryLogger,\"C:\Users\reill\.dotnet\tools\.store\dotnet-releaser\0.1.3\dotnet-releaser\0.1.3\tools\net6.0\any\dotnet-releaser-binary-logger.dll\";\"C:\Users\reill\AppData\Local\Temp\tmp48E8.tmp.binlog\"" -t:Restore C:\Users\reill\source\releaser-repro\ReleaserRepro.csproj. Reason: MSBUILD : error MSB4017: The build stopped unexpectedly because of an unexpected logger failure.
      Microsoft.Build.Exceptions.InternalLoggerException: The build stopped unexpectedly because of an unexpected logger failure.
       ---> System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
         at System.Collections.Generic.Dictionary`2.Enumerator.MoveNext()
         at Microsoft.Build.Internal.Utilities.EnumerateItems(IEnumerable items, Action`1 callback) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\Utilities.cs:line 86
         at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsWriter.WriteProjectItems(IEnumerable items) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsWriter.cs:line 852
         at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsWriter.Write(ProjectEvaluationFinishedEventArgs e) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsWriter.cs:line 283
         at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsWriter.WriteCore(BuildEventArgs e) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsWriter.cs:line 196
         at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsWriter.Write(BuildEventArgs e) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsWriter.cs:line 137
         at Microsoft.Build.Logging.StructuredLogger.BinaryLogger.Write(BuildEventArgs e) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BinaryLogger.cs:line 249
         at Microsoft.Build.Logging.StructuredLogger.BinaryLogger.EventSource_AnyEventRaised(Object sender, BuildEventArgs e) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BinaryLogger.cs:line 239
         at Microsoft.Build.Evaluation.ProjectCollection.ReusableLogger.AnyEventRaisedHandler(Object sender, BuildEventArgs e)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.RaiseAnyEvent(Object sender, BuildEventArgs buildEvent)
         --- End of inner exception stack trace ---
         at Microsoft.Build.Exceptions.InternalLoggerException.Throw(Exception innerException, BuildEventArgs e, String messageResourceName, Boolean initializationException, String[] messageArgs)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.RaiseAnyEvent(Object sender, BuildEventArgs buildEvent)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.RaiseStatusEvent(Object sender, BuildStatusEventArgs buildEvent)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.Consume(BuildEventArgs buildEvent)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.Consume(BuildEventArgs buildEvent, Int32 sinkId)
         at Microsoft.Build.BackEnd.Logging.EventRedirectorToSink.Microsoft.Build.Framework.IEventRedirector.ForwardEvent(BuildEventArgs buildEvent)
         at Microsoft.Build.BackEnd.Logging.CentralForwardingLogger.EventSource_AnyEventRaised(Object sender, BuildEventArgs buildEvent)
         at Microsoft.Build.BackEnd.Logging.EventSourceSink.RaiseAnyEvent(Object sender, BuildEventArgs buildEvent)

Thanks, I’ve filed https://github.com/KirillOsenkov/MSBuildStructuredLog/issues/558 and will investigate it there.

One thing you said @KirillOsenkov that I wanted to clarify:

I guess we haven't seen this because not a lot of people use the BinaryLogger copy from StructuredLogger.dll (almost everyone uses the original from MSBuild). Yeah, let's think about what's the right thing to do is here.

The reason I'm using StructuredLogger binary logger is to make more guarantee that I can read the results from any kind of dotnet+msbuild version. Based also on this comment from the readme:

Alternatively (useful for older versions of MSBuild) you can attach the logger to any MSBuild-based build using the logger library

So does it make sense to use StructuredLogger like this in my case or should I use instead the MSBuild built-in version?

I think it’ll be better to just use -bl (the logger built into MSBuild). It has been supported since 15.3 (2017). Both the viewer as well as StructuredLogger.dll support reading binlogs of any version. I was wondering why you’re using StructuredLogger.dll. I don’t think you’ll support ancient MSBuilds such as 14.0?

I think it’ll be better to just use -bl (the logger built into MSBuild). It has been supported since 15.3 (2017). Both the viewer as well as StructuredLogger.dll support reading binlogs of any version. I was wondering why you’re using StructuredLogger.dll. I don’t think you’ll support ancient MSBuilds such as 14.0?

Oh, ok, so I thought that using the logger of StructuredLogger would make sure that I could actually re-read it after. If you say that using MSBuild gives more guarantee then definitely, I will revert that. It will remove the hack I had to go through to have the StructuredLogger dll still packed on the side.

Removed by commit e0825f8a6a73a631b39ffec952e65b9f17695558

Plus, I think even StructuredLogger.dll of recent versions has lost any hope of working with MSBuild < 16.0, so there’s really no scenario I can think of where StructuredLogger.dll should be used for recording.
On the other hand it’s main usage is for reading binlogs (it is used by the viewer) as well as API (to read and analyze binlogs)
It all stems from historical reasons when MSBuild didn’t have /bl built-in. Now /bl is ubiquitous in any MSBuild since 2017 so the copy in StructureLogger.dll is redundant. I still keep it up-to-date (poorly, as we have seen) just in case.

Plus, I think even StructuredLogger.dll of recent versions has lost any hope of working with MSBuild < 16.0, so there’s really no scenario I can think of where StructuredLogger.dll should be used for recording.

Great, thanks! I have removed using it from 0.1.8+


Feature request: Highlight lines

It would be cool to be able to select lines to highlight similar to how you can do it in the github viewer.
The interaction would probably be similar. Click to the left of a line to highlight it, and if we're holding the Shift key down we can click another line to select a range of lines.
Codemirror supports this via the Active Line option, so it should be feasible.
I would also be down to work on a PR for this.

Unless I am mistaken, you can already highlight the lines of the Carbon editor, and those highlights even remain in the image after saving as well  . . . Is this what you mean? 🙂

Oh haha. I guess it kinda does. It's a bit finicky if you want to highlight the entire line, and it looks like it the highlight color renders funky once the editor is not in focus so it makes it a bit hard to read (for the record I haven't tested if using the Active Line option in code mirror is more readable). This also doesn't really let me select more than one selected range.

I think if using the Active Line option makes it look better this feature would be worth it. I'll experiment and post and update here to see if it's better.

@jalvarado91 please do! Thanks 🙂

After looking into this more, you are already able to highlight the lines, and then click away and the highlighting remains. You can also click save and the highlights will be in the PNG. Closing . . .


Add support for 429 response codes

We recently added rate limiting to our NPR One API (hi Irakli!) but were bummed to find that we couldn't return 429 as the HTTP status code because Zaphpa doesn't consider it valid. It's pretty much canonically accepted these days as the status code for rate limiting, so... pretty please? :)

Awesome. Thanks for the pull request. Hope you guys are doing great.
cheers