[Bug]: Puppeteer can run on Docker locally, but not on Cloud Run

Bug expectation
I expected no error when trying to run the Puppeteer function in Cloud Run, but it does. On my local Docker Engine (Windows), it runs fine.
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
  const puppeteer = require('puppeteer')

  try {
    const html = `<h1>Hello</h1>`
    const browser = await puppeteer.launch({
      headless: 'new',
      args: ['--disable-dev-shm-usage', '--no-sandbox'],
    })
    const page = await browser.newPage()
    await page.setContent(html, { waitUntil: 'networkidle0' })
    const pdfBuffer = await page.pdf({
      landscape: true,
      printBackground: true,
      width: 681,
    })
    await browser.close()
  } catch (e) {
    // Some logic to handle errors (omitted)
  }

Error string
Could not find Chrome (ver. 113.0.5672.63). This can occur if either\n 1. you did not perform an installation before running the script (e.g. npm install) or\n 2. your cache path is incorrectly configured (which is: /home/.cache/puppeteer).\nFor (2), check out our guide on configuring puppeteer at https://pptr.dev/guides/configuration.
Dockerfile configuration
FROM node:16

# Install latest chrome dev package and fonts to support major charsets (Chinese, Japanese, Arabic, Hebrew, Thai and a few others)
# Note: this installs the necessary libs to make the bundled version of Chromium that Puppeteer
# installs, work.
RUN apt-get update \
    && apt-get install -y wget gnupg \
    && wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add - \
    && sh -c 'echo "deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google.list' \
    && apt-get update \
    && apt-get install -y google-chrome-stable fonts-ipafont-gothic fonts-wqy-zenhei fonts-thai-tlwg fonts-kacst fonts-freefont-ttf libxss1 \
      --no-install-recommends \
    && rm -rf /var/lib/apt/lists/*

# Create app directory
WORKDIR /usr/src/app

# Uncomment to skip the chromium download when installing puppeteer. If you do,
# you'll need to launch puppeteer with:
#     browser.launch({executablePath: 'google-chrome-stable'})
ENV PUPPETEER_SKIP_CHROMIUM_DOWNLOAD true

# Install app dependencies
# A wildcard is used to ensure both package.json AND package-lock.json are copied
# where available (npm@5+)
COPY package*.json ./

RUN npm install
# If you are building your code for production
# RUN npm ci --only=production

# Bundle app source
COPY . .

Puppeteer version
20.2.0
Node version
16.15.1
Package manager
npm
Package manager version
8.11.0
Operating system
Windows

Have you configured the cache dir like the error message says? https://pptr.dev/guides/configuration#changing-the-default-cache-directory it does not look like the binary is added to the image.

Have you configured the cache dir like the error message says? https://pptr.dev/guides/configuration#changing-the-default-cache-directory it does not look like the binary is added to the image.

I have added the .puppeteerrc.cjs file as mentioned in the link above, but it makes it such that Puppeteer cannot run on my local Docker. I am not sure what binary you are referring to, could you please elaborate?

Puppeteer downloads the browser binary during npm install and stores it to the cache folder. That folder needs to be included into the Docker image. So you need to either 1) run npm install when you build a docker image or 2) copy the cached folder installed locally to the docker image (but local binaries need to match the platform + architecture used by docker, so linux x64 binaries need to be used locally).

Puppeteer downloads the browser binary during npm install and stores it to the cache folder. That folder needs to be included into the Docker image. So you need to either 1) run npm install when you build a docker image or 2) copy the cached folder installed locally to the docker image (but local binaries need to match the platform + architecture used by docker, so linux x64 binaries need to be used locally).

Noted. My current situation is that when I include .puppeteerrc.cjs, the Puppeteer part of the code can't run in my local Docker container. Without that config file, it runs fine locally. I have not tried running it with .puppeteerrc.js on Cloud Run. Do I need to try running it in Cloud Run even though it can't run locally?
In my Dockerfile, I have run npm install, and Puppeteer is included in the package.json file, which means that the binary file in my local machine does not need to be copied to the Docker image running on Cloud Run. Is that right?

That's right, you also call COPY . . which might overwrite your Puppeteer version (depending on how your versions are pinned in package.json). In any case, just make sure that the npm install puts the binary into the cache folder and that the Puppeteer version is not overwriten.

Also, .puppeteerrc.cjs would need to be there before you run npm install

@OrKoN Just wanted to update you on this. It turns out the error is as you mentioned, I added .puppeteerrc.cjs after npm install and this seems to caused some error. I updated the Dockerfile to the following and it works.
FROM node:16

# See https://crbug.com/795759
RUN apt-get update && apt-get -yq upgrade && apt-get install \
    && apt-get autoremove && apt-get autoclean

# Install latest chrome dev package and fonts to support major charsets (Chinese, Japanese, Arabic, Hebrew, Thai and a few others)
# Note: this installs the necessary libs to make the bundled version of Chromium that Puppeteer
# installs, work.
RUN apt-get update \
    && apt-get install -y wget gnupg \
    && wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add - \
    && sh -c 'echo "deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google.list' \
    && apt-get update \
    && apt-get install -y google-chrome-stable fonts-ipafont-gothic fonts-wqy-zenhei fonts-thai-tlwg fonts-kacst fonts-freefont-ttf libxss1 \
      --no-install-recommends \
    && rm -rf /var/lib/apt/lists/*

# Create app directory
WORKDIR /usr/src/app

# Bundle app source
COPY . .

# Install app dependencies
# A wildcard is used to ensure both package.json AND package-lock.json are copied
# where available (npm@5+)
# COPY package*.json ./

RUN npm install
# If you are building your code for production
# RUN npm ci --only=production

EXPOSE 8080
CMD [ "node", "./bin/www" ]

Thanks for the help!


GDScript: Preload should make native type

reduce_preload() was calling type_from_variant() which made datatype with kind set to BUILTIN, not NATIVE
type_from_variant() now returns a variant type when passed a null value, fixes #61159

The CI build failure is unrelated, it's a temporary inconsistency between godot and godot-cpp, which should be fixed when #67751 is merged.

Thanks!


Could I use this with "AutoForm"?

Could I use this with "AutoForm"?
Please example.

I get this error when trying to use autoform in the schema.

Invalid definition for createdAt field: "autoform" is not a supported property

It is because auto form is in its current state still referencing the simple-schema meteor package and not this one.
I already created a PR for autoform, where I solved this issue:
https://github.com/aldeed/meteor-autoform/pull/1557
But @aldeed needs to check and confirm it, before it is merged to the develop branch. It seems that he is very busy (there are many PRs left in autoForm).
@aldeed: do you have someone in the community you trust enough, so this person can check the validity of the PR? It would be great if auto form is available with dependencies to this package. This may also wake up some sleeping contributors who may again be willing to improve the package.

@thearabbit / @abhirakshit  and @all others if you depend on the package please vote for my PR. I am depending on autoForm and would work on an integration of node-simple-schema until it's working flawlessly.

Oh, I would to use Schema not Collection2.
Waiting...

Yes, node-simple-schema is included in the PR I proposed. That's why I made this PR. Collection2-core is also included.
For examples, see: https://github.com/aldeed/meteor-autoform#example

I'm about 80% done with a new version of AutoForm that will work with the latest collection2 and Node version of SimpleSchema. I expect to get it released within the next 2 weeks.

I would like to plan use SimpleSchema, AutoForm with Ms Sql.
Look great @aldeed, I waiting you reply about new version of Autoform.
Many people talk about alternative of Autoform such as Uniform...
Excuse me, What about AutoForm base on Blaze or React?

Is there a chance the new Autoform will be frontend agnostic, with plugins for frontend components?

The version I'm working on is still pretty much the same as the existing version, still Blaze. I am just updating it to work with latest C2 and SS breaking changes, that's it. I am planning a new package similar to autoform but internally different, not tied to Meteor, and using React. I don't know when that will be done. There are other form packages already using React that I haven't used but seem like good alternatives.

@aldeed It's a very good news. Is this project has a repository now? I want to star and track it man.

@aldeed is there any update to the status of the new version of autoform. You mentioned you where about 2 weeks away from a release however i haven't seen anything. Any update to the status would be fantastic.

@bp12345 It isn't published yet but the devel branch of autoform seems to work with latest SS and C2. I'm doing some final checks updating a few apps and I need to document a couple gotchas. I should be able to officially publish 6.0 this week.

@aldeed Any news ? :)) can't use autoform object in schema :/

@dukye, sorry yeah it was published. See https://github.com/aldeed/meteor-autoform#note-autoform-60. I haven't announced it much yet because I haven't quite finished updating the demo app, and I need to update a few add-on packages as well.

@abhirakshit :
import SimpleSchema from 'simpl-schema';
SimpleSchema.extendOptions(['autoform']);

Can someone please take a look at my stackoverflow question.  I think there is an issue with aldeed:autoform and web3 api calls. http://stackoverflow.com/questions/42540029/web3-api-call-error-in-meteor-autovalue-function-and-collection-insert?noredirect=1#comment72265645_42540029


Volume change

changing the volume doesn't effect anything?

It doesn't?! I'll do some tests and see what I can do.

@jpmcglone , It does with the "Ring And Alerts" sound not the volume sound


conditional use of domains

hi,
in my usecase i have two packages, lets say standard and premium, only the premium package has the need of domains, i would like to identify the users on standard package by id or something..
is it possible?

i would like to identify the users on standard package by id or something

How would this ID be sent?

i use a normal user model on my central app, when the user pays for a package i create the database, do migrations and seed the created user in central app also to the tenants database.
the domain to the tenants app is shown inside the profile of the user..
so maybe i could do something after the payment.. but no clue

I can't help with your application's design.
I'm just saying that if you don't want to let them use domains with some plan, then it's your responsibility to send the tenant id from your front end in some way.


requiring sudo is just plain wrong

A program that at runtime checks if it's running as root, rather than checking for the actual permissions that it requires.  I don't think I've seen such bad practise any time before, to be honest.
I don't really see why the program would require to run as root.  Root privileges are way too powerful.  All that's needed appears to be access to some USB devices.  So the proper solution is to ship udev rules for those vid/pid combinations, rather than forcing people to run code with way too high privileges.

Sure, would be happy to look at the pull request

Isn't that #27?

I guess this can be closed, now that #27 has been merged?


Generalize CheckAlreadySynced

Context
When the stop node in the recursive selector is the same as our sync node, we will never stop early because the stop node is only checked after traversal has started.
We previously did a check to see if our sync cid was our latestSync and exit early. This is a special case of the more general form where the stop node in the selector is the cid of the node we are going to start traversal on.
Then we added a special flag to always do this special case.
Proposed Solution
Instead of having the user remember to pass in the flag for the special case check, lets try to determine if we're already in the case where we are trying to stop on the same cid as where we are starting.
Note that this uses knowledge of the Recursive Selector structure to get the stop node. There's a test that will break if this knowledge is ever wrong. It's not ideal, but I think it's better than having to remember to pass in the special flag.

Codecov Report

Merging #97 (2dec055) into main (45a7a69) will increase coverage by 0.30%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
+ Coverage   63.95%   64.25%   +0.30%
==========================================
  Files          21       21
  Lines        1878     1877       -1
==========================================
+ Hits         1201     1206       +5
+ Misses        523      515       -8
- Partials      154      156       +2

Impacted Files
Coverage Δ

option.go
38.63% <ø> (+3.21%)
:arrow_up:

selector.go
62.00% <57.14%> (-1.89%)
:arrow_down:

subscriber.go
80.71% <100.00%> (+1.36%)
:arrow_up:

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 45a7a69...2dec055. Read the comment docs.

Please bump the version and I will refer to that in https://github.com/filecoin-project/storetheindex/pull/234


Plans to add rust based SWA as the bundler?

Do you think its worth it to use the new ultra fast rust based SWA bundler?
great boilerplate by the way!

@john1625b the first time I tried it it was not in good shape, but I believe it is not the case now. Unfortunately, I don't have enough time to try it now, but would love to check a PR, if someone is interested

i am interested

i am interested

I would love to check your PR


chore(main): release 0.71.0

:robot: I have created a release beep boop
0.71.0 (2024-06-18)
Features

Pipelines: add authentication to wehbook (#714) (2a6b3ce)
pipelines: Configuration on pipeline's version (acfa76b)
Workspaces: retrieve workspace connection by slug (#711) (9617063)

Miscellaneous

Connections: stop injecting connections creds in env var (#715) (217f2c1)
GraphQL: Update documentation of graphql files (#712) (2be6c87)
Update database port in docker-compose.yaml (#709) (1a5378b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-app/releases/tag/0.71.0 :sunflower:


Add compare viewer webpage

This PR adds a compare viewer webpage to the ems-basemap editor. It can be opened by clicking the "Compare to Production Styles" button at the top of the Maputnik app. This viewer shows production styles in four zoom levels/locations on the left side. The right side shows the same zoom levels/locations but applies the current style from the Maputnik app. This will allow us to quickly compare local style changes to production styles across different scales.
As the style is changed in Maputnik, the style is automatically updated in the compare viewer. Since Maputnik saves styles in localStorage, we can listen to for those changes and apply the style to the maps in the compare viewer.
I suspect the easiest way to work with this is by opening both Maputnik and the Compare Viewer side by side as shown below.

To test out the PR locally clone and checkout this branch, then run npm install and npm start. The application should be available at http://localhost:8888.
cc: @thomasneirynck @nyurik @kmartastic @miukimiu

Anyway to make that public and submit to maputnik or OMT ? :)

Anyway to make that public and submit to maputnik or OMT ? :)

Not in it's current form. The concept may be feasible, but we'd want to refactor the viewer into react rather than it's current form.


Continuous batching in Llama

Implement a continuous batching test in Llama which demonstrates our continuous batching algorithm:
while True:
  if not is_full(current_batch) and not prefill_q.empty():
    model_prefill(prefill_q.pop())
  elif not is_empty(current_batch):
    model_decode(current_batch)
  else:
    break

Latest work is on branch cglagovich/continuous_batch. It uses paged_update_cache to pass an index vector, and implements basic logic in demo_continuous_batching.py to test continuous batching.


Import 90% of machine code in crc

Only drivers (rpc, driver interface, hyperkit and libvirt structs) and state are left in code-ready/machine.
It will be easier to refactor machine code: better tests in the future, etc. All the persistence is now done in crc side. code-ready/machine is stateless and also all drivers.
requires https://github.com/code-ready/machine/pull/54

Ready for an other round of review :)

Ready for an other round of review :)

I also hesitated for hyperv driver but at the end, it is much easier to have machine reduced to the strict minimum. The major benefits is the ability to share code between preflights and the driver (how we invoke powershell, etc).

I also hesitated for hyperv driver but at the end, it is much easier to have machine reduced to the strict minimum. The major benefits is the ability to share code between preflights and the driver (how we invoke powershell, etc).

/needs-rebase

/needs-rebase

rebased!

rebased!


Trigger a flow based on data dependency availability

It is common for a scheduled flow to have data dependencies. With the data availability check feature for Azkaban flows, it will be easier for jobs in an Azkaban workflow to start only when the data they depend on becomes available.
Motivation

Improve user productivity by standardizing how Azkaban flows specify data dependencies. Users don’t have to build their own jobs that check if the data their flows depend on are available.

Reduce overall flow latency and error rate. Some flows are scheduled based on estimated data availability time. The flows will fail when the data is not available when they start. To reduce such failure rates, users will add some buffer time e.g. schedule a flow to start at 10 AM when the dependent dataset is typically available at 8 AM. Despite the buffer, the flow may still fail if the data is delayed for longer than 2 hours.  This feature allows these flows to be scheduled at an earlier time than they currently do so. In the example, the flow can be scheduled to run at 8 AM without worrying out failures due to data availability.  By removing the failures caused by data delays, the feature makes it easier to see true flow logical failures.

Improve overall infrastructure efficiency and stability.  Azkaban can check the data availability efficiently. See the detailed design section below for details. A typical implementation of a custom job that checks for data availability relies on polling HDFS/Hive Metastore. A bad implementation or too many such jobs running concurrently can put pressure on the HDFS namenode/ Hive Metastore.

Encourage users to provide the data dependency information as metadata that Azkaban can understand. Azkaban can leverage this information in the future to gain more flexibility or optimize the system. E.g. schedule the flow in a cluster where the data lives.
bring the data to the cluster that the flow runs. This can happen if the flow relies on GPU in a certain Hadoop cluster for example.

👍  for this first of all.
I'm mostly curious about how this will be reflected in the UI and what it will looks like in job configuration. Would you have any examples to share about those?
After all, from my perspective it maybe seems like this interface may not be as useful as it can be limiting when compared to custom polling scripts.. With an explicit job to poll for data it is quite clearly visible from the Job List where the flow is going, how long it has been "blocked by a data dependency", and why (you can log details in the job log of the data polling job). Personally I would really hope to have a similar level of visibility in the UI for the dependency checks.

Yes - we spent lot of time thinking if data dependency should be modeled as as a job - or should it be designed at a flow level. After lot of thinking we have decided to design this at a flow level. There are couple of reasons for this:

We feel data dependency is similar to any existing or future trigger we may have. Such as: schedule based triggers, manual triggers, flow-to-flow dependency trigger etc. While the actual flow is "logic" of the flow. Trigger is logic of how flow is "triggered". This separation of concepts makes it easy to handle entire flow as a simpler concept which can be moved around if needed.

This choice does make it harder to display this on UI - compared to if we just made it a simple job type. So we will spend sometime updating UI to reflect some of the key points you made earlier.

One other important change on the radar is - introducing "flow" as a concept at compile time. Today flow is purely "run time" concept. As a result only way to add "flow" level properties is via UI. eg. schedule, SLA etc. We want to take this opportunity and create "flow" as a compile time construct - so that config of the flow such as schedule, data dependancies, sla's can be defined at compile time as well.


Reduce functional utilities call depth

Reduce the number of function calls necessary to invoke the target callable in the functional utilities.
This PR relays on reverting the workaround introduced in #2404, and instead fixes the underlying bug that is believed to have caused the original issue (for which there is no report and no tests).
Each "flatten" commit removes one frame from the call stack. Most changes result in code simplification, but two commits (titled "flatten ... some more") result in more complex code in order to embed index sequences into the wrappers.

It seems like const isn't propagated properly:
https://stellar-group.gitlab.io/-/hpx/-/jobs/158592760/artifacts/tests.unit.html#1b048cec-0235-474d-8aae-fbcc0372a85e

There are also additional test failures caused by the proposed changes: https://stellar-group.gitlab.io/-/hpx/-/jobs/159443858/artifacts/tests.unit.html#775cc3d3-3e99-462e-91fa-e374cdeb7c28

It seems like const isn't propagated properly:

Caused by the different semantics between decltype(e) and decltype((e)).


(Feature request) local repos for lxd

Problem:
I am using many different containers on the same computers/servers/local network and most of them run the same OS (Debian in my case).
Now when I update them, each container connects to the internet and downloads the same packages.
I think that is unnecessary traffic and a waste of resources (because obviously the internet consumes a lot of energy).
Solution:
LXD could include an option for local repos from which all containers could download their updates from.
A concept could look like this:

Download package updates to local repo (could be a folder or a seperate container; maybe on a set schedule (like daily))
automatically add the local repo to a container (as a disk device or something else (like anetwork related solution))
Containers download updates

One more thought about the local repo:
It could be a copy of the complete repo of a distro, but that could be to big, so the alternative would be a program/script looking into the containers, which packages are installed, to only download and include those.
Alternatives:

While some might say that there are other ways to achieve that, I would prefer a solution in lxd itself
I am aware of the image auto update function (image handling docs), but thats not the same, because the containers can be different from the (official) images and updating each image (that could be created from containers) would lead to the same problem.

Not something that really makes sense to do at the LXD layer.
You should create a container that either runs some kind of proxy (look at squid-deb-proxy for example) or even has a full archive mirror of your distro, then configure your other containers to pull from that.
That's not really any different than what organizations do with their physical systems/VMs.


add school gduf

School Website: https://www.gduf.edu.cn/

@yongPhone I am sorry, the domain m.gduf.edu.cn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Custom questions for users

I've installed this dating site. I thank the developers for this amazing piece of software. Very simple to install and works great !!
I did see the question #356, and I was saying that having the possibility to create questions as well as well questions with options and dropdown list would be a great feature if added. In this way, user can search based on these options as well !!
Thanks !!

By the way, is it very complex to add some more questions with some options directly in the database or in a script somewhere so that the user can directly answer these questions by selecting the options. Is yes, can I get some directives how to do it ?


Cant run server

I have tried everything, I have tried running the server-0.1-SNAPSHOT.jar which throws an exception about the domains.xml not found, and the original-server-0.1-SNAPSHOT.jar gives an error about the manifest being missing. I have followed the readme to the last letter

@Pho3niX90 can you please try latest code, should be easier.


pyvisfile and dependencies

I want to use http://mathema.tician.de/software/pyvisfile/ in a project I'm working on. Unfortunately it's a little tricky to compile all the dependencies, so I made conda packages so I don't have to do it again.
I'm not sure exactly if we've decided on the scope of this repo, but this seemed like a better place to put the recipes than anywhere else?

Sounds reasonable!  We have a few packages like that in here too.

I'll test the build out on Mac before merging.
On Apr 21, 2015 4:35 PM, "John Chodera"<EMAIL_ADDRESS>wrote:

Sounds reasonable! We have a few packages like that in here too.
—
Reply to this email directly or view it on GitHub
https://github.com/omnia-md/conda-recipes/pull/175#issuecomment-94974702
.

This looks good to me.

Okay. Tested fine on os-x py27 and linux-64 py34, so I think this is good to merge.


TableMultiHeader Bug with UI5 Sort/Filter Popup

Hi,
there is a major bug when using multilabels and sorting/filtering in:
https://github.com/SAP/openui5/blob/master/src/sap.ui.table/test/sap/ui/table/samples/TableMultiHeader.html
The popup for rows where the top label is "over-spanned" does not get rendered at the correct position but anywhere (bottom or way left) on the page!
Sincerely,
Daniel

Hi Daniel,
I am not sure if I get you right, but you mean this issue when clicking on the "Name of the person" column header. The dropdown popup gets displayed on the left side of the table (even slightly above it).
Can you confirm that this is the issue you are referring to?
Best regards,
Matthias

Hi Matthias,
Yes. And when employed in a Design Studio SDK component it gets displayed even further away since the CSS "right" and "top" are set wrong (to 0 and basically to the bottom left if the screen) from the calling object o.
Thanks and best regards,
Daniel

Sent from my mobile

Am 02.03.2015 um 16:49 schrieb Matthias Oßwald<EMAIL_ADDRESS>Hi Daniel,
I am not sure if I get you right, but you mean this issue when clicking on the "Name of the person" column header. The dropdown popup gets displayed on the left side of the table (even slightly above it).
Can you confirm that this is the issue you are referring to?
Best regards,
Matthias
—
Reply to this email directly or view it on GitHub.

Thanks! I will forward this issue.
Internal reference:<PHONE_NUMBER>

Hi Daniel,
thanks for reporting this. The issue will be fixed in version 1.28.5
https://github.com/SAP/openui5/commit/2b8c0f371b26b51cb6423e3aadf42133bd9974d2
Best regards
Sebastian

@SebastianRied,
for multiHeader with sortable, as default, the sortable menu will appear when  clicking both the header and subheader, could it be possible that the sortmenu will only appear when clicking subheader(bound to the subheader)

when i test accessibility "readability", it can't be read as "current ascending", can ui5 table support it?


Add Are We Game Yet updates to the newsletter

First draft, suggestions on the wording are welcomed :)
Rendered:

What do you think about adding an image to the beginning of the section?
Something like

Also, maybe it should be mentioned that the transfer to AWGY was finished recently?

Yeah, sounds good.

Done - new preview:


Docker build of DataFusion cli fails on WSL Ubuntu 22.04.3 LTS

Describe the bug
❯ docker run -it -v /tmp:/data datafusion-cli
datafusion-cli: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.35' not found (required by datafusion-cli)
datafusion-cli: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by datafusion-cli)
datafusion-cli: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by datafusion-cli)
datafusion-cli: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by datafusion-cli)

Updating the debian release in the Dockerfile from
FROM debian:bullseye-slim

to
FROM debian:bookworm-slim

fixes the issue
❯ docker -v
Docker version 25.0.2, build 29cf629

To Reproduce
Build DataFusion CLI on WSL with latest Ubuntu version using the docker build then docker run cmds
docker build -f datafusion-cli/Dockerfile . --tag datafusion-cli
...
docker run -it -v /tmp:/data datafusion-cli

Expected behavior
No response
Additional context
No response

I think we could use more specific version tags in Dockerfile. Would easier to spot the issue.
For example:
FROM rust:1.72-bookworm as builder
...
FROM debian: bookworm-slim
...

or
FROM rust:1.72-bullseye as builder
...
FROM debian: bullseye-slim
...

I tested the second option and found it works.
- FROM rust:1.72 as builder
+ FROM rust:1.72-bullseye as builder
...
FROM debian: bullseye-slim
...

Shall we create a PR to change the CI to the second option to address this issue? If so, I am happy to do it.


Support Java15+ text block (""" foo """)

Problem description
Java Text Blocks are a new features of Java not supported yet by tree-sitter-java.
See https://www.baeldung.com/java-text-blocks for example of use of the feature.
This is needed for Semgrep: https://github.com/returntocorp/semgrep/issues/3787

I think this has been fixed.


Server crash when accidentaly put a ` character before exit name

This is the error log
mud_1           | Traceback (most recent call last):
mud_1           |   File "./server.py", line 106, in <module>
mud_1           |     sessions[sender_client].process_message(message)
mud_1           |   File "./server.py", line 24, in process_message
mud_1           |     self.current_verb.process(message)
mud_1           |   File "/usr/src/app/verbs/go.py", line 17, in process
mud_1           |     elif [exit in room_exit for room_exit in self.session.user.room.exits.keys()].count(True) > 1:
mud_1           |   File "/usr/src/app/verbs/go.py", line 17, in <listcomp>
mud_1           |     elif [exit in room_exit for room_exit in self.session.user.room.exits.keys()].count(True) > 1:
mud_1           | TypeError: 'in <string>' requires string as left operand, not Quitter

Can't replicate, seems to be fixed now.


【freshman】how to apply this model into a binary classification work.

I am a newer student in the NLP domain. I have read your paper and want to apply your model into a binary classification work. There are some questions that i am facing:
1.if I want to run a demo that you've provided, I don't know which file to run first to start.
2.If I want to apply your model into a sentences binary classification, I don't know how to use your model do a embedding work
sorry to disturb you for such easy question, I am looking forward your reply. Thank you in advance.

We don't have an easy to use demo, but you could make one from one of the pretrained models (see here: https://github.com/allenai/kb#evaluating-fine-tuned-models).  Then use the allennlp predict or allennlp evaluate commands to make predictions.

For binary classification you will need to fine tune the model on your dataset.  The WiC dataset is binary classification you could use as a example. See the config file here: https://github.com/allenai/kb/blob/master/training_config/downstream/wic.jsonnet and the model code is here: https://github.com/allenai/kb/blob/master/kb/evaluation/classification_model.py


jest.setTimeout in setup file does not properly apply such timeout globally

When using a global setup file.
I'm overriding the default global timeout in a setupFile with:
jest.setTimeout(10);

Test should fail but it looks like the default timeout of 5000ms keeps getting used (I'm using --runInBand).
Is this expected behavior? If yes, what is the proper way to override globally this default timeout?
Thanks.

Had to use setupTestFrameworkScriptFile to make it work.


Warn that the API is unstable in README.md.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #322   +/-   ##
=========================================
  Coverage     90.59%   90.59%
  Complexity      497      497
=========================================
  Files            79       79
  Lines          1977     1977
  Branches        151      151
=========================================
  Hits           1791     1791
  Misses          150      150
  Partials         36       36

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 2719346...9bb8adf. Read the comment docs.


I.D. scan add reservation not adding reservation

Describe the bug
Nothing happens when clicking 'add reservation' in the scanner window.
To Reproduce
Steps to reproduce the behavior:

Navigate to staff page
Click on a room and then add reservation
Scan ID
Nothing occurs and the reservation is not added

Expected behavior
Reservation should be added and it should show the message "reservation added".

Might actually not be a bug but something local


Write a FEDERATION.md file

https://socialhub.activitypub.rocks/t/documenting-federation-behavior-in-a-semi-standard-way/453
https://github.com/gabek/fedidocs/issues/4

I'm not seeing much adoption of this in web searches. (Unscientific, I know.)

I wrote a FEP draft for FEDERATION.md: https://codeberg.org/silverpill/feps/src/branch/main/67ff/fep-67ff.md
It contains an up to date list of services which adopted this convention. Bridgy Fed could be one of them :)

Done! https://github.com/snarfed/bridgy-fed/blob/main/FEDERATION.md

Good job! Adding Bridgy Fed to implementation list: https://codeberg.org/fediverse/fep/pulls/314


After clicking "Login wtih Google" errors...

Here is a screenshot...

I'm on Mac el capitan. I also tried on windows 10, but I got the same error message.
Any ideas?
Thanks!

I had the same issue. Most of the time the solution is  to enable the Google Java API  from the Google API Console. If it is already enabled then double check if the API key was typed in correctly .  I got this error because the API key was not typed in correctly. ( for some reason I'm unable to paste the API in the login module)


slow down conversations to a natural pace

Now that we've sped up the loop with Pinecone some convos are happening too fast to follow, e,g., one paragraph per second. This needs a more natural delay.

fixed


Add a Carousel Landing page template

Description
Added a custom editted carousel landing page with a responsive design & neat UI. It can be used as a homepage template for volunteering communities.
Fixes #251

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system: Windows, Linux
Version: 10, Ubuntu
Text-editors used: Sublime

Checklist:

[x] My code follows the style guidelines of this project
[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 to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@arpit456jain souldn't GSSoC22 label be assigned to the PR?

@arpit456jain @VidhiBhatt01 Have been checking the leaderboard today and found this one not counted. Both me ad Arpit forgot about adding GSSoC22 label to the PR. So, is it possible for it to be added?

Added, thanks for reminding!


absl::flat_hash_map - unexpected key lookup failures in runtime loaded library.

I'm seeing key lookup failures while adapting code from Google Deepmind's Reverb project which I don't fully understand.
Normal work flow (works as expected):

A Reverb server runs a Reverb table that receives data from distributed workers via grpc, stores it in an absl::flat_hash_map, and then serves that data via grpc to distributed samplers. All of this code runs via single compiled pybind module, which I believe is loaded at runtime. The absl::flat_hash_map is only manipulated within this module.
A sampler runs a tensorflow dataset op that makes gRPC calls to retrieve this data. The dataset op is also compiled as a library and loaded at runtime.

Our work flow
In our use case, we only have one sampler that runs in the same python process as the reverb server. To increase throughput, I am trying to short the gRPC comms between the server and the sampler. I pass a pointer to the Reverb table from the server module to the dataset module and get it to sample data directly instead of over gRPC. The server module still accepts data as it did so previously from distributed workers.
Key lookup issue: If I iterate over the keys in the hash map and print them, my key is present. However, things like map_object.contains(key) are failing and map_object[key] creates new objects in the map instead of returning existing objects. If I iterate over the keys, I can see duplicates in there. Both absl::fast_hash_map and absl::node_hash_map fail this way, while std::unordered_map works as expected.
It'll take a bit of time to create a clean working example that demonstrates this behavior, and there's definitely some chance I'm using things incorrectly -- the code is very experimental. But I wanted to check here to see if passing a pointer to the object containing the container from 1 runtime loaded library to another is inherently incorrect. Any advice would be appreciated.

We mention at https://abseil.io/about/compatibility that dynamic loading is asking for trouble. From the symptoms you describe, I would guess that you are running into the randomization of absl::Hash. It sounds like the loaded libraries have different hash seeds, which is probably why you can find objects by iteration, but not by lookup.

Thanks! That makes a lot of sense. I'll test this out on Monday and update/close this thread.

I've verified that using std::hash with absl::flat_hash_map (which doesn't do randomization) works as expected, and the absl::hash for the same key was different in the both loaded libraries.
Switching to absl::flat_hash_map with std::hash provides a 10% boost over std::unordered_map. The key was a uint64_t generated via absl::Uniform<uint64_t> and should be sufficiently randomized already, which works fine with std::hash` typically returning the value itself for integer types.
Thanks for your help! I will close this question.


DR-976 Deal with exceptions functionality

Changes:

Add error handling in Domain controller.
Change error handling in Main controller.
Modify error modal html.
Modify error modal Ctrl.

WIP:

Content

:+1:


DO NOT MERGE! Fast Forward stable to 9bd405703a4e775c2d0fbbe5ff7af2b5d6b20553.

This PR will be merged automatically once checks are successful.

Jenkins job manager/postsubmit passed

Jenkins job manager/e2e-suite passed


Adapter ready4stable text

Adapt the ready4stable text so thats it more clear this is no personal message form bluefox but an automatic generated message

Text has been adapted.
Link to adapter.dev has been added.


add normalizer, tokenizer to rouge

What does this PR do?
Fixes #825
I wrote some code but I don't know how to test it sorry.
plz review me hard
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@stancld how is it going here? :)

@hookSSi how is it going here? have you had a change to check recent review/suggestion from @stancld :rabbit:

@hookSSi how is it going here? have you had a change to check recent review/suggestion from @stancld 🐰

Sorry, I'm checking on it now

@hookSSi how is it going here? have you had a change to check recent review/suggestion from @stancld 🐰

Sorry, I'm checking on it now

Thanks for the update (I'll go through the code once again today). May you just add some tests suggested in the code review comment?

Thanks for the update (I'll go through the code once again today). May you just add some tests suggested in the code review comment?

I added test that just checking value. Have a look and review me please.

@hookSSi could you please mark addressed comments as resolved so we easily see what has been left? :rabbit:

@hookSSi could you please mark addressed comments as resolved so we easily see what has been left? 🐰

Oh I didn't know that I can mark these sorry about that.


Command line tool

So as drift is going to have a command line tool, I think we should discuss how the command should work and what should be available.
My first idea is the following
# Change the code to pest test classes
$ drift migrate {path}

# Show the diff instead of actually changing the code.
$ drift show {$path}

Would love to hear if you guys have some better ideas for naming 👍
For example rector uses the following syntax
# Change code based in the specified path based on the specified set
$ vendor/bin/rector process  {path} --set {set-name}

# Show the diffs instead
$ vendor/bin/rector process  {path} --set {set-name} --dry-run

An issue that came to me is how a user should be able to specify if they just want to run some rectors. Let's say a user has pest tests already and want to change them to higher order tests, how should this be specified?
Ex.
$ drift migrate {path} --only=higher-order-tests

Since migrating tests will mostly consist in "cleaning" them up*, we could go with a more fun & expressive syntax:
# Change the code to pest test classes:
$ drift polish {path}

# Show the diff instead of actually changing the code:
$ drift show|check {$path}

That way we keep the car metaphor going on 🙂
I'm not sure which is best between show & check. I think both would work and are short though!

Unless I'm missing the point on what you intend Drift to do?

An issue that came to me is how a user should be able to specify if they just want to run some rectors. Let's say a user has pest tests already and want to change them to higher order tests, how should this be specified?
Ex.
$ drift migrate {path} --only=higher-order-tests

How would that work with tests that use named routes?

Since migrating tests will mostly consist in "cleaning" them up*, we could go with a more fun & expressive syntax:
# Change the code to pest test classes:
$ drift polish {path}

# Show the diff instead of actually changing the code:
$ drift show|check {$path}

That way we keep the naming metaphor going on 🙂
I'm not sure which is best between show & check. I think both would work and are short though!

Unless I'm missing the point on what you intend Drift to do?

I like the idea of having some naming that is more in the style of the naming, keeping it fun while still explaining 👍
I like show the most, as the command will "show" a diff.

How would that work with tests that use named routes?

Not sure I follow your question 😃 Could you explain a little more in detail what you mean.

Let's say you want to refactor this test to use Higher Order Functions:
test('has home page', function () {
    $this->get(route('home'))
        ->assertSuccessful();
});

It is using a named route, home, which you can't access with the arrow function syntax.
So you'd need to replace that with the actual route segment, like so:
test('has home page')->get('/')->assertSuccessful();

Perhaps it's something you can do, I just don't see how for the moment 🙂

Ah, thanks for explaining. The parameter where I specified higher order methods was just an example it could for example also be order tests alphabetically or something else.
I don't have a solution for that scenario yet. I opened an issue about creating that rector. So I think figuring out a way to avoid that problem should be left to that issue. :)

You are right, I digress.
1 - Back on track (no pun intended), we'll need a help function to explain the API to a first time user:
# Display drift's API
$ drift --help | drift -h

2 - What if a user want to revert back to PHPUnit tests? Should he just go back to his previous commits (not optimal imho), or use that kind of command?
# Converts back to PHPUnit
$ drift reverse {$path}

@AlexMartinFR Yep, we totally need a help method 👍
Hmm, I like the idea about a reverse command, however it would require us to make a rector that converts pest to phpunit.

We got a basic command line now, let's close this issue and open specific ones things missing in it or improvements that we could make.


Add a JupyterHub service to manage user images

Add a service managed by JupyterHub to handle user images via a UI.

[ ] Update the documentation to show how to find the UI
[ ] Update the overview diagram to show how the service interacts with the rest of the stack

This is what the UI could look like:

As a first step, we should be able to start with something simple that doesn't involve a database or a shared file / state to keep track of the images.
Instead we could look up the existing images directly using the docker client.
We also need a way to manage the builds that are in progress, maybe by looking at the running repo2docker build containers.
cc @pierrepo if you have any feedback on this

Current status:

We can drop the "Build" button. Instead one can remove the images and add them again.
Still need to:

retrieve images that are being built
add a form to add a new repo
remove images (can be done in a separate PR)
populate the list of built images in the spawner

Hi @jtpio :wave:
This looks very promising. I did not have in mind that the image under construction might be an issue. But you are right to underline we have to deal with it since a user might want to re-create an image under construction if he does not see it in the interface...

One more thought, if a user want to refresh an existing image. He first updates his repo but then, does he need to delete the actual image and register a new one? Would a refresh button be useful here?

It depends on how we think about the Docker images.
Docker images are immutable, so it might not be clear what a refresh button should do. If the ref is master, then we could say that it would grab the latest master and rebuild the image. But if the ref is a commit hash, then refreshing wouldn't update anything.
Otherwise if we want to promote pinning on specific commits (so it's easier to investigate issues and they correspond to a specific point in time), that would indeed mean adding a new image with a new commit hash, and removing the old one after.
For now we could probably try the add / remove workflow and see afterwards if some of this should be made more intuitive in terms of UX.


'Microsoft.Web/serverfarms/firstPartyApps/settings' doesn't have types available

Bicep version
0.4.1008
Describe the bug
Resource type 'Microsoft.Web/serverfarms/firstPartyApps/settings' doesn't have types available
To Reproduce
I need define Geneva configs as below, but get type unavailable. Looks like bicep has no support to Geneva configs
resource antMDSConfigXML 'Microsoft.Web/serverfarms/firstPartyApps/settings@2019-08-01' = {
  name: '${aspName}/AntMDS/MdsConfigXml'
  location: location
  properties: {
    firstPartyId: 'AntMDS'
    settingName: 'MdsConfigXml'
    settingValue: genevaConfig.configXml
  }
}

There's any update here?

This will need to be resolved by the Web Resource Provider team. Tagging @naveedaz / @seligj95 to triage. If the issue is more urgent, I would recommend opening a support ticket or internal incident.

gentle reminder that this is still missing :)


update to new pulp href

Changes to pulp href, where api is preprended to the path, breaks the href parsing function. This shifts the indexes over one to account for that.
And update zest version

probably want to update https://github.com/content-services/tang/blob/main/compose_files/pulp/docker-compose.yml  to use       PULP_API_ROOT: /api/pulp/

thanks!


git_get doesn't work properly yet

git_get -cr /tmp/code https://github.com/freeflowuniverse/crystallib/tree/development_kristof/src

this should checkout branch development_kristof only depth 1
this needs to put it under /tmp/code

there is error

doesn't execute
doesn't use right path

this works now, closing


这个错误提示是什么原因？该如何调整？

2021-12-20 00:09:38.140 Session "SID-GUEST-249": Although the virtual hub has attempt to assign a new IP address <IP_ADDRESS> was made, 4 IP addresses have already been assigned for this service. According to the security policy, this session is denied routing and is therefore allowed to hold no more than 4 IP addresses. The packet has been discarded.

Please write in English and use our template. Please also describe the steps to reproduce the issue.

@domosekai Thanks for your reply. I will resubmit my question

By default a VPN client is not supposed to be bridged with another network. That's what a VPN bridge does.
You can try enabling "Bridge / router mode" in the VPN client, or replacing VPN client with VPN bridge.

Thanks for your quick reply.Let me try it. @domosekai

Thank you, as you said. My problem is solved.


Rename lib/babel.js to lib/child.js

See #374.

Ok @sindresorhus, that should be fixed now.
I have no clue why that happened... It looks like WebStorm automagically un-babel'd it.

WebStorm asks you about adding a file watcher whenever it sees ES6 code. If you accidentally click yes, this can happen. I keep meaning to figure out where the setting is just to turn that off.

No, I get that. But it appears that WebStorm un- babel'd the code. It's
very strange.
On Sat, Dec 26, 2015 at 9:46 PM James Talmage<EMAIL_ADDRESS>wrote:

WebStorm asks you about adding a file watcher whenever it sees ES6 code.
If you accidentally click yes, this can happen. I keep meaning to figure
out where the setting is just to turn that off.
—
Reply to this email directly or view it on GitHub
https://github.com/sindresorhus/ava/pull/375#issuecomment-167382407.

When I see child.js I am asking myself a question "child of what?". In my opinion, it is just as unclear as babel.js. How about forked-process.js, worker.js, test-worker.js?

@vdemedes: what about "subprocess.js"?

@ariporad subprocess.js works too. Let's see what other guys think.

test-file-launcher.js

Also, please paint the bikeshed green.

@jamestalmage a bit long, I think.

test-launcher.js or test-worker.js

@jamestalmage, @vdemedes: I still like subprocess.js or child.js.
And this bikeshed is to be blue.

@jamestalmage++

:+1: test-worker.js

Ok, I'll change it to test-worker and update the commit.


Export-CrescendoModule has odd support for -WhatIf

With most PowerShell commands, if you call a command inside a function that supports should process, the preference is passed to the underlying command.
function foo-test {
 [cmdletbinding(SupportsShouldProcess)]
param($name)
Stop-Service $name
}

foo-test bits -WhatIf
What if: Performing the operation "Stop-Service" on target "Background Intelligent Transfer Service (bits)".

But when Export-CrescendoModule is used, it doesn't appear to pickup SupportsShouldProcerss. Yet, the command will recognize it when used directly:
Export-CrescendoModule -ConfigurationFile .\CmdKey-crescendo.json -modulename foo -WhatIf

What if: Performing the operation "Output to File" on target "foo.psm1".
What if: Performing the operation "Output to File" on target "foo.psm1".
What if: Performing the operation "Output to File" on target "foo.psm1".

Although oddly, it appears to be writing the file 3 times.

The reason this happens 3 times is a side effect of how the .psm1 file is created (multiple redirection). It's a bit clunky and I'll take a look at cleaning this code up.

I'm still hoping this will be merged with a few other open issues such as having multiple definitions in a single json file. I'd love to have a process where I create the crescendo json, and the "conversion" process creates a function file for each command, a psm1 file that dot sources the files, and a manifest.

Thank you - This issue has been addressed in Preview.2.


We compiled it and nothing happens

Nothing happens when we compile it. Only thing we changed is the E-Mail

italics


Forward function cannot call nested functions

The following example
import tangent

def f(x):
    def a(x):
        return x * x
    return a(x)

df = tangent.grad(f, verbose=0)

would fail with the exception
  File "/usr/local/lib/python2.7/dist-packages/tangent/annotate.py", line 64, in resolve
    node.id, self.func.__name__))
AttributeError: Failed to resolve name "a" used by "f".

This is because of that ResolveCalls. visit_FunctionDef
https://github.com/google/tangent/blob/3318ecab3eec71ef97702df66a59d1a2197c8de3/tangent/annotate.py#L44-L46
doesn't add the function definition of a into self.namespace, so that the following code snippet
https://github.com/google/tangent/blob/3318ecab3eec71ef97702df66a59d1a2197c8de3/tangent/annotate.py#L54-L56
cannot resolve the call from f to a.
I don't think it is necessary to support nested functions. But it seems reasonable to restrict the forms of inputs to tangent by introducing something like the Google C++ Code Style.

I am also attaching the AST of the function definition of f for your reference:
        FunctionDef(name='f',
                    args=arguments(args=[Name(id='x', ctx=Param())],
                                   vararg=None,
                                   kwarg=None,
                                   defaults=[]),
                    body=[
                        FunctionDef(name='a',
                                    args=arguments(args=[Name(id='x', ctx=Param())],
                                                   vararg=None,
                                                   kwarg=None,
                                                   defaults=[]),
                                    body=[Return(value=BinOp(left=Name(id='x', ctx=Load()),
                                                             op=Mult(),
                                                             right=Name(id='x', ctx=Load())))],
                                    decorator_list=[]),
                        Return(
                            value=Call(func=Name(id='a', ctx=Load()),
                                       args=[Name(id='x', ctx=Load())],
                                       keywords=[],
                                       starargs=None,
                                       kwargs=None))],

This is indeed a known limitation - closures are not yet supported. Addressing it is a bit more involved because not only the side effects of the closure would need to be detected, but the inner function would need to be autodiffed as well.
Can you rewrite the code and pull a outside of f? I realize doing that might not always be a practical refactoring.

Oh, I see the limitation listed here.  Sure, I can rewrite the application code to pull a out from f. Thank you @mdanatg !


Add XSUM Datasheet

Hey, are you planning to fill in the remaining sections in this PR or should I merge and you create a follow-up?

Hey, are you planning to fill in the remaining sections in this PR or should I merge and you create a follow-up?

Hey, are you planning to fill in the remaining sections in this PR or should I merge and you create a follow-up?

I was waiting for @shashiongithub's  review, but I think merging this right now would be fine.

Hey, are you planning to fill in the remaining sections in this PR or should I merge and you create a follow-up?

I was waiting for @shashiongithub's  review, but I think merging this right now would be fine.

ping about this

ping about this

ping again @Shashi456

ping again @Shashi456

I'll get in done asap, sorry for the delay.

I'll get in done asap, sorry for the delay.

@sebastianGehrmann Done, I'll add the baseline and links for the notebook after talking to the person who set up the baseline.

@sebastianGehrmann Done, I'll add the baseline and links for the notebook after talking to the person who set up the baseline.

Okay, thank you! Merging for now

Okay, thank you! Merging for now


Tests?

THIS IS A GREAT IDEA. I'd love to use it in a project, but there are no tests 😞
You should add some, then this project would be so popular.

The code is perfect. No tests needed. Closed.


Code incorrectly added in project

Hi, I'm following Cloud Firestore iOS Codelab, and in step 4 the tutorial instructs to add the following lines of code to RestaurantTableViewCell.populate(restaurant:):
nameLabel.text = restaurant.name
cityLabel.text = restaurant.city
categoryLabel.text = restaurant.category
starsView.rating = Int(restaurant.averageRating.rounded())
priceLabel.text = priceString(from: restaurant.price)

Project already has those lines added :) The tutorial has to be updated or those lines should be removed from the code
Greetings

Fixed in 3b2cd30444d5d5c6d9b43747c740544e317d57ec.
Thanks!


Strange result when fetching variables under a distributed strategy

System information

Have I written custom code: yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Mojave 10.14.5
TensorFlow installed from (source or binary): binary (tf-nightly)
TensorFlow version (use command below): 1.15.0-dev20190729
Python version: 3.7.4

Describe the current behavior
When I am using any distributed strategy, and I fetch a tf.Variable, I get a result that looks like: [array([( 10,), ( 44,), ( 47,),  ...], dtype=[('resource', 'u1')])]. Instead, I have to wrap the variable in a tf.identity call in order to get its value properly.
Describe the expected behavior
The fetched result should be the value of the variable.
Code to reproduce the issue
import tensorflow as tf
import numpy as np
import sys

class RunHook(tf.train.SessionRunHook):
  def before_run(self, run_context):
    return tf.train.SessionRunArgs(fetches=['var:0'])

  def after_run(self, run_context, run_values):
    print(run_values.results)
    sys.exit(0)

def model_fn(features, labels, mode, params):
  var = tf.get_variable(
    initializer=tf.constant([1.0], dtype=tf.float32),
    name="var",
    dtype=tf.float32,
    trainable=True,
  )
  loss = tf.identity(var)
  opt = tf.train.AdamOptimizer(0.001)
  global_step = tf.train.get_or_create_global_step()
  train_op = opt.minimize(loss, global_step=global_step)
  return tf.estimator.EstimatorSpec(mode=mode, loss=loss, train_op=train_op)

strategy = tf.distribute.MirroredStrategy()
session_config = tf.ConfigProto()
config = tf.estimator.RunConfig(train_distribute=strategy, session_config=session_config,
                                log_step_count_steps=1, save_checkpoints_steps=float('inf'))
classifier = tf.estimator.Estimator(model_fn=model_fn, config=config)

x = np.array([1, 2, 3, 4])
y = np.array([5, 6, 7, 8])
train_input_fn = tf.estimator.inputs.numpy_input_fn(x, y, batch_size=1, num_epochs=None, shuffle=True)

tf.estimator.train_and_evaluate(
  classifier,
  train_spec=tf.estimator.TrainSpec(input_fn=lambda: train_input_fn, hooks=[RunHook()]),
  eval_spec=tf.estimator.EvalSpec(input_fn=lambda: train_input_fn)
)

I have tried on colab with TF version 1.15.0-dev20190729 and i was getting the below error`ImportError: cannot import name 'dense_features' .Please, find the gist here.However i tried with latest nightly version and was able to reproduce the issue.Please, find the gist here.Thanks!

This shouldn't affect correctness.
It is caused by the variable becoming a resource variable.  Your current code leave use_resource=None, yet the distribution strategy force use_resource=True on the variable.  That's why it looks like the behavior is different between the case without distribution strategy and the case with distribution strategy.   You can also see that if you remove distribution strategy, but add use_resource=True to your variable you will get the confusing print out.
Fetching the resource variables by python reference works, however.  You can accept the references in the constructor argument of RunHook() and return the hook using EstimatorSpec.training_hooks.
Hope this helps.


RPTerm & PicoTerm

Hi Daniel,
This is not really an issue... but a request for advise regarding the PicoTerm project.
We already get in touch about development of PicoTerm for RC2014.
I'm working on the PicoTerm version (because it does fit my RC2014).
Presently the USB keyboard is not always detected at power up on the Picoterm Project. Unplug and plug again the USB keyboard does the trick. This behavior is ennoying.
I would like to know:

If RPTerm suffer of the same issue ?
If you find a way to solver it ?

Thank for your response.
Cheers,
Dominique

short answer: RPTerm always detects my keyboard at power up, but I don't know if any of my changes affect that or if things behave different with other keyboards.
long answer:
I did not have much time to work on RPTerm. I am very curious about what changes were made to PicoTerm to make it compatible with more keyboards, but a quick look to the now available sources came empty.
If I can find the time, I intend to do a more throughout analysis of the current PicoTerm code and update both my fork and RPTerm. Also on the list is to try a few different keyboards, and do some tests with PicoTerm.
One last point: it may be a power issue, where the Pico cannot provide the needed voltage/current for the keyboard at power on, leaving the keyboard in some strange state.

@dquadros
I have implemented a delayed USB Power up line (5 sec) with a MOSFET and this is now working perfectly!

Thank for suggestion!
Dominique


How can I build this repository?

I tried to improve this repository by fork.
But I can't build . npm run build is failed with error.
><EMAIL_ADDRESS>build
> rm -rf dist && rollup -c rollup.config.ts --configPlugin @rollup/plugin-typescript && ncc build src/github-action.cjs -o action

loaded rollup.config.ts with warnings
(!) Plugin typescript: @rollup/plugin-typescript: Rollup 'sourcemap' option must be set to generate source maps.
[!] RollupError: Rollup transpiled your configuration to an  ES module even though it appears to contain CommonJS elements. To resolve this, you can pass the "--bundleConfigAsCjs" flag to Rollup or change your configuration to only contain valid ESM code.

Original error: exports is not defined in ES module scope
https://rollupjs.org/command-line-interface/#bundleconfigascjs
ReferenceError: exports is not defined in ES module scope

Also, I couldn't run the test code.
><EMAIL_ADDRESS>test
> jest

 FAIL  test/index.test.ts
  ● Test suite failed to run

    test/index.test.ts:6:8 - error TS1192: Module '"/Users/yonghoonlee/gitroot/ChatGPT-CodeReview/src/index"' has no default export.

    6 import myProbotApp from "../src";
             ~~~~~~~~~~~

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.431 s
Ran all test suites.

How can I build and test?

Hi, the test code is not runnable. And  for the build issue, I guess it's caused by your node version is old and doesn't support esm module, try to upgrade it to new rts version.

Hi, the test code is not runnable. And for the build issue, I guess it's caused by your node version is old and doesn't support esm module, try to upgrade it to new rts version.

Hi, I'm using LTS(18.17.1) but I can't build with the error :(

I am using node v20, I tried switching to v18, and still works fine for me.
Make sure that you didn't change any code based on the main branch, and in the package.json, have the right type declaration.
{
  "type": "module"
}

if you did this and still not work, try the suggestion by the console
RollupError: Rollup transpiled your configuration to an  ES module even though it appears to contain CommonJS elements. To resolve this, you can pass the "--bundleConfigAsCjs" flag to Rollup or change your configuration to only contain valid ESM code.


Custom URI schema that allows users to click on a link in a browser to connect to a server

I believe I completed this a while back on the feature branch.


Always call nav.enable() to make nav collapsing work

This used to depend on the sticky nav being enabled.
Fixes #481

Yea, I ran into this one myself, thanks for fixing it!


Support .NET Core and other platforms

I don't think Scientist.NET uses much that requires it to target .NET 4.5+ only. If someone wants to make this work on .NET core I'd love it.

I will work in FileHelpers to support .Net Core, maybe I can start with Scientist that looks easier
Did you have a timeframe for this issue or I can look at it in the next week or two ?
Thanks

@Haacked @MarcosMeli I'd like to start on this if that's cool. I've already been working on https://github.com/davezych/shience in .NET Core so I think it would make sense for me to migrate this.
How cutting-edge would you like to be? If we support the current "stable" runtime of rc1-update1 we can target dnx451 and dnxcore50, but moving up to some nightlies will allow targeting Net Platform Standard which will get UWP, Windows Phone, DNX, Mono, Win8.1, etc with a single target. (You can view the table here: https://github.com/dotnet/corefx/blob/master/Documentation/architecture/net-platform-standard.md#mapping-the-net-platform-standard-to-platforms).

@davezych fine with me if you've already done the work. :smile: The key thing for me is that the NuGet package still work for .NET 4.5+ projects. So, feel free to go to town on what you think is best here.

@Haacked it looks like I accidentally left the csproj file in the main project. That should probably be removed to avoid confusion.
I'll open a new issue for scripting the nuget package.

@Haacked it looks like I accidentally left the csproj file in the main project. That should probably be removed to avoid confusion.

Ok, I'll do that.

@Haacked close?

I've been trying out the latest package as it looks from https://github.com/Haacked/Scientist.net/pull/37 like it should work in .NET Core, but I can't get it to build.
If I remove all references to the full .NET Framework so that the relevant section in my project.json looks like this:
  "frameworks": {
    "dnxcore50": { }
  },

Then I get this build error:
NU1002	The dependency Scientist 1.0.0-alpha5 in project ... does not support framework DNXCore,Version=v5.0.

Does this only support the latest drops of .NET Core or is RC1 still supported?
Any ideas?

@jpsingleton rc1-final should be supported. I'll check this out and get back to you. I opened #54 to track it.

@davezych great thanks. I was doing some basic research for a book and found this didn't work, although it looked like it should. I'm building a list of .NET Core supported libraries and frameworks at https://github.com/jpsingleton/ANCLAFS (work in progress), so I'll update that when this is working.
Cheers.


Stuck at loading MNIST dataset

Hello guys, I tried to run train.py using this command
python train.py CNN --group groupCNN --model modelCNN --lr 0.001 --lr 0.001 --batchSize 32 --maxIter 2000 --warpType translation
But somehow I stuck at this loading MNIST dataset. My Internet works normally so I dont know what is going on. I also dont know if the parameter I used on the command above is correct or not since the readme doesnt provide any examples to compare.
Thanks

The command line works fine on my side. Could you double-check whether there are files downloaded to data and MNIST_data? The code should automatically download MNIST if doesn't exist in your directory.


Topic Modelling in NLP [ GSSOC'23]

Is your feature request related to a problem? Please describe.
I will be using the sentences to create a corpora and then I will use LDA from gensim to do the topic modelling.
Describe the solution you'd like
I will be using the sentences to create a corpora and then I will use LDA from gensim to do the topic modelling.
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Pritamstudent - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.


Yakuake: hardcode path to konsole

I have no idea how this worked before.
Yakuake uses KPluginLoader, which looks for the library in lots of different weird places but not the correct one. For me it whined that it couldn’t find Konsole and strace gave this:
access("/home/kirrun/.kde/lib/kde4/konsolepart.so", R_OK) = -1 ENOENT (No such file or directory)
access("/nix/store/2qj7f5r1c3jz1n3zbn2dmj8n6damlfc9-kdelibs-4.14.3/lib/kde4/konsolepart.so", R_OK) = -1 ENOENT (No such file or directory)

access("/home/kirrun/.kde/lib/./libkonsolepart", R_OK) = -1 ENOENT (No such file or directory)
access("/nix/store/2qj7f5r1c3jz1n3zbn2dmj8n6damlfc9-kdelibs-4.14.3/lib/./libkonsolepart", R_OK) = -1 ENOENT (No such file or directory)

access("/home/kirrun/.kde/lib/kde4/konsolepart.so", R_OK) = -1 ENOENT (No such file or directory)
access("/nix/store/2qj7f5r1c3jz1n3zbn2dmj8n6damlfc9-kdelibs-4.14.3/lib/kde4/konsolepart.so", R_OK) = -1 ENOENT (No such file or directory)

access("/home/kirrun/.kde/lib/./libkonsolepart", R_OK) = -1 ENOENT (No such file or directory)
access("/nix/store/2qj7f5r1c3jz1n3zbn2dmj8n6damlfc9-kdelibs-4.14.3/lib/./libkonsolepart", R_OK) = -1 ENOENT (No such file or directory)

@urkud

Cherry-picked into master. Thanks!


Leaderboards issues

So, while ago I found 2 issues and I completly forgot to report them here, so first is
1. Selected Mods Leaderboard
When looking at selected mods lb, only pb is shown. What does that mean
For example user a has 2 plays on Harumachi Clover, One with DT and one No Mod.
User a has personal best on No Mod score, when someone, including user a tries to look at selected mods lb with DT, he won't see his score because it's not his PB.
2. PP sorting, but also score at the same time
So, I don't know if it's on purpose or not but in mania this is very annoying, anyways. This got tested in mania, idk if it works the same in other modes but probably yes.
When user a submits score on map xyz with 850,000 score and 220 pp for example, his score will be first
Then user a submits another score but this time with DT, he managed to get 770k score and 320pp. His DT score will be displayed as first even though it's not PB, same thing happens on HT. Due to mania score cap being 1 milion, it's really annoying and if it's on purpose then it doesn't make sense in mania. Also I think that this may be related to first bug.

About the first thing: it's not a bug I suppose because it's like this on a lot of osu servers including ripple based ones. But yeah it would be much better to check for status = 1 or 2 instead of just 2.
To the second one: mania uses PP override, therefore a score is considered the best score on a map based on pp, haven't checked but based on what you've described I suppose that it is like this

minisbett is basically correct on the first one, however i don't really understand the issue you're trying to describe on the second point. plays override based on pp, as the point of pp is to evaluate what is the better score. what's the problem

minisbett is basically correct on the first one, however i don't really understand the issue you're trying to describe on the second point. plays override based on pp, as the point of pp is to evaluate what is the better score. what's the problem

So basically in mania scores don't override on leaderboards, highest score = pb but pp is counted from best play
Here are examples
Score 1: HT 220pp  449k score - PP is calculated from this one
Score 2: NoMod 100pp 600k - This one is displayed on LB
Score 1: DT 300pp 750k score - PP is calculated from this one
Score 2: NoMod 220pp, 920k score - This one is displayed on LB

i’m really not understanding lol
scores always override by PP, so there’s no way the score it’s showing on leaderboards is different to the one the api/db says the best; the one it says is the best will also definitely be the highest pp one
also the 2nd and 3rd screenshot are from bancho so where is the relevance??
i do not understand, lol.

No the problem is just, you see score in the leaderboard but it uses PP override in the background, it should display pp in the leaderboard instead because otherwise it's confusing. Same goes for vanilla, scores override by score and score is in the leaderboard. For relax, pp is the score override and it shows pp in the leaderboard

Oh nvm now I understood

Seems to be a bug I guess

1st is planned yes - per-mod best scores is something i've wanted to implement for a long while
2nd - are you asking for your highest score to be displayed on map leaderboards, but highest pp to count towards global pp?

also - each github issue should represent a single problem - if you have two issues, please create two issues on github, even if they are from the same category (e.g. leaderboards).
categories can be represented through issue tags.

2nd - are you asking for your highest score to be displayed on map leaderboards, but highest pp to count towards global pp?

presumably, since this is how it works on bancho.

first issue already exists at #560, second issue i don't really see happening because there's a lot of semantics and opinion around it so not really worth it


Versioned API documentation: proposal

📚 Documentation
In order to enable version-specific API documentation for napari, we need a place to host and archive it. Previously, this was done on napari/napari.github.io, and the proposal here is to enable that again - but this may change the way the docs are deployed.
As of now, this is the process:

A PR is merged to the napari/napari main branch;
The Build and Deploy docs action is triggered, and pushes the built html artifacts to the gh-pages branch of napari/napari.github.io, overwriting whatever content is already there.

One way we could orchestrate this process is the following:

A PR is merged to the napari/napari main branch;
The Build and Deploy docs action is triggered, and pushes the built html artifacts to the dev subfolder of the gh-pages branch of napari/napari.github.io, keeping the existing deployments for other versions.

This of course implies changing the subfolder structure of the current documentation (released versions would show up under napari.org/<version>, the latest release under napari.org/stable, and the current version under development would show under napari.org/dev or something similar.) This could mean a few broken links especially if people are linking from other pages to the napari docs.
Another change this would imply is an extra action item on releases, to set up the symlink from stable to the newest release.
cc @jni @neuromusic @codemonkey800

I've just been working on this for the plugin skan (with the idea that it could pretty much be copied over to napari since the napari theme is based off of pydata theme) - it's pretty much working see: https://lucyleeow.github.io/skan/dev/, relevant PRs: https://github.com/jni/skan/pull/175 and https://github.com/jni/skan/pull/173/files

and pushes the built html artifacts to the dev subfolder of the gh-pages branch of napari/napari.github.io, keeping the existing deployments for other versions.

would this keep all artifacts from all pushes to main?
Another option we could do is to  github actions to save tag commits to a folder named as per the tag e.g.
      - name: Deploy stable docs 🚀
        uses<EMAIL_ADDRESS>        if: startsWith(env.TAG, 'v')
        with:
          branch: gh-pages
          folder: doc/_build/html
          target-folder: ${{ env.DOC_DIR }}
          clean: true

Another action item that may need to be added to release is to update the version switcher json file (unless we can get it auto-populated by github actions).
Relevant scipy PR adding the version switcher: https://github.com/scipy/scipy/pull/15380/files


Cross-Browser validation

Perform cross-browser testing to ensure compatibility with:

IE 9, 10, 11
Edge
Firefox
Safari
Chrome

We could leave this item as a reference note to the browser's we will be supporting.
I will be performing cross-browser Quality Assurance upon releasing a given completed module.
I will be checking on the browsers noted above on both PC & Mac.
Will also be testing on Safari in iOS7+ on iPad2+ & iPhone5+

This issue is no longer applicable as it will be completed as per component to the chandelier branch.


Sign PHAR releases

We install tools on our CI with phive.
This allows downloading PHAR files and checking their signature automatically.
However, it requires that PHAR files come with a signature that is used to verify the PHAR.
Would it be possible to provide signatures for twigcs PHAR files?

@prudloff-insite the latest release of twigcs has been signed. Please let me know if you encounter any issue with it. Thank you for your patience!


Update to allow sequences in MariaDB >= 10.3

Sequences appeared in MariaDB 10.3.
Existing logic does not reflect that, as a major version > 10 would also need a minor version >= 3
Reflects logic change pointed out by Lijun Liao in issue CORE-3457
┆Issue is synchronized with this Jira Story by Unito

Hi @mykelalvis
Thank you for this Pull Request.
I’m putting together a core team of developers to help us with community requests. We’ll make Liquibase improvements faster together!
If you are willing to help, please reply to this comment and I’ll let you know the next steps.
Your friendly neighborhood developer advocate,
@molivasdat


Use appveyor for CI on Windows

ping @thesamovar @nippoo

YAGNI

well, maybe YAGNI..?

yeahhh... let's reopen this


Fix #98: Update User Agent to current most common UA (Chrome 61 on Windows 10 x64)

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #98)

Testing
N/A

Coverage remained the same at 30.402% when pulling 1713f2052791beab016ef6d9dda9f1aadaefb549 on metaprime:update-ua into 66d962fc3b6748424768612b2fed93e009ed1df6 on RipMeApp:master.


ipsec: Test

The first commit simplifies the optional templates used in most XFRM IN and FWD policies. The second commit simplifies the XFRM IN policies into a single policy. The third moves a bit of code around to avoid unnecessary XFRM policy updates. See commits for details.

/test

/test

/test

/test

/test

/test

/test

/test

/test

/test

According to the XFRM guide a policy is needed to make sure packets are delivered to local processes, and we can combine them into one single policy because after a packet passes the XFRM decode stage delivery to local process is the only thing we care about correct?

Yep, we don't try to filter what the local processes receive. That's the job of Cilium & k8s network policies.

We still recreate the catch-all IN policy for every node in the cluster, because state and policy creation are quite tightly coupled at the moment, correct? If correct, this could be a potential follow-up issue.

Yep, that's a good point. We have the same issue for the FWD policies, as Louis noted before. We could probably install all of those once in the newNode.IsLocal() functions.


Feature/add ability to register/unregister formats

Add the ability to register and unregister formats for reading and writing with JDeli

I am merging this as discussed yesterday and optimisations for this will be made after


Ensure the catalog controller's Reconcile() function is being properly unit tested

#196 Updated the catalog controller's unit tests which resulted in the removal of using envtest for unit testing purposes. It was called out as part of that PR that we should look into refactoring the Reconcile() function in such a way that makes it easier to test only the logic that runs inside of the Reconcile() function:

A future tech debt item could be to investigate refactoring the Reconcile() function to make it easier to test the logic without using envtest.

Factoring the apply logic into a library (example) and testing that in isolation is likely a better way to ensure correctness. If you do that, the rest of Reconcile() ends up being ~10 lines of code, which will remain static and unchanging forevermore. Visually inspecting the un-recoverable error case may be sufficient then.


:sparkles: feat: Default polling mode sync

Codecov Report

Merging #42 (dcf41f3) into main (090fa68) will decrease coverage by 1.69%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff              @@
##               main      #42      +/-   ##
============================================
- Coverage     71.93%   70.24%   -1.69%
+ Complexity      315      307       -8
============================================
  Files            36       36
  Lines          1126     1126
  Branches         79       79
============================================
- Hits            810      791      -19
- Misses          259      278      +19
  Partials         57       57

Impacted Files
Coverage Δ

...ain/java/com/featureprobe/sdk/server/FPConfig.java
50.76% <100.00%> (ø)

... and 3 files with indirect coverage changes


🔗 Forge nemeton-1 chain

Prepare nemeton chain for incentivized program.

[x] forge the genesis
[x] set register dates
[x] disable update-peers job (will be re-enabled when chain starts)
[x] provide README.md.gtpl description
[x] remove peers list for nemeton
[x] set discontinued status in the readme for nameton

@amimart I think we're done for this PR. I've deleted all account balances. I will introduce the different required accounts in another PR. It does not interfere with the registration of druites during the registration period.


[do not merge] building images for testing #853+#854

This PR is just to trigger image building on a combination of #853 and #854, rebased on top of v0.26.1. Planning to test the image on staging.

@sharnoff do you still need it?

Ah, yup, thanks. Don't need it anymore.


⚠ Bofrost.de has degraded performance

In c94c292, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 905 ms

Resolved: Bofrost.de performance has improved in 46b3cf8 after 2 hours, 52 minutes.


Make xieite::threads::Pool use a circular buffer

To replace the queue + lock

also research alternatives


自己修改的emmet的snippets被更新掉了

自己修改了emmet的snippets，更新后snippets被覆盖了，还要自己重新修改一下，是不是也能像settings那样加写自己的设置比较好？

@karl20888 can you edit your post into english please?

Sorry, I'm Chinese and poor English, I just said, The 0.10.1 update cover the file what I edited the snippets.json in the Emmet. So I need to edit it again. Is it can improve?

@bpasero Sorry, I'm Chinese and poor English, I just said, The 0.10.1 update cover the file what I edited the snippets.json in the Emmet. So I need to edit it again. Is it can improve?

Hi @bpasero , @karl20888 has modified the default snippets. After he updated the vs code to version 0.10.1, he lost the modification. He suggests that could he overwrite the default snippets using "User" or "Workplace" snippets,  just like the way we overwrite the default settings.

@ywmail Thank you for your translation!

Yes, don't modify files inside the VS Code installation. They will be replaced on every update.
We have user definable snippets. Go to File > Preferences > User Snippets.
They appear on Ctrl + Space.


Install target-specific scripts

Install target-specific scripts in sbin directory.

I would suggest the following changes:
- git mv scripts/dpdk/os_ver_details.sh -> scripts/common/os_ver_details
- git rm scripts/es2k/os_ver_details.h
- git mv scripts/dpdk/set_hugepages.sh -> scripts/common/set_hugepages
- git rm scripts/es2k/set_hugepages.sh
- git mv scripts/dpdk/setup_env.sh -> scripts/dpdk/setup.env
- chmod a-x scripts/dpdk/setup.env
- git mv scripts/es2k/setup_env.sh -> scripts/es2k/setup.env
- chmod a-x scripts/es2k/setup.env

Update scripts to reflect new names and paths.
Consider creating symlinks from target-specific subdirectories to files in common directory

Since os_ver_Details.sh is common for DPDK and ES2K, moved it to scripts/common. Please see latest changes on - https://github.com/ipdk-io/networking-recipe/pull/170/


[Bug] RpcException with no provider avaliable messages will be thrown if the newly dubbo reference is immediately called.

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
3.2.12
Steps to reproduce this issue

delete local registry cache files.
build a springboot app that has @DubboReference(check = false)  to provider A.
create dynamic reference to provider B when the state of DubboApplicationStateEvent is started.

        ReferenceConfig<ProviderB> referenceConfig = new ReferenceConfig<>();
        referenceConfig.setInterface(ProviderB.class);
        referenceConfig.setVersion("...");
        referenceConfig.setGroup("...");
        referenceConfig.setCheck(false);
        referenceConfig.setAsync(true);
        // using DubboBootstrap.getInstance() as the single instance is already created.
        ProviderB providerB = DubboBootstrap.getInstance().getCache().get(referenceConfig);

call any method of providerB immediatly.

What you expected to happen
ReferenceCache shoud provide a get method with timeout paramter to pull the provider instance info from the registry center at the reference creation.
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

we take a temporary solution:
ReferenceCache refCache = DubboBootstrap.getInstance().getCache();
ProviderB providerB = refCache.get(referenceConfig);
// set check true temporarily for invoker available checking.
referenceConfig.setCheck(true);
try {
    // check invoker available status.
    refCache.check(referenceConfig, checkTimeoutMs);
} finally {
    // restore check option.
    referenceConfig.setCheck(false);
}
// call provider method.
providerB.methodXXX(....);

because you have  set  this  'referenceConfig.setAsync(true);'   which   is  similar  to   send  oneway, do not  care response, and if you wan't to use 'async and return value', you can define your interface's resonse format this  =>  Future


fix: npm pkg name

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

GitHub release
v1.4.0

Your semantic-release bot :package::rocket:


requires containing '~=' result in broken spec file

In spec files ~= is not an valid operator to compare versions and I didn`t find anything about debian dsc-files
Here is a first draft how it could be fixed, but it would need more work
    for req in filtered_req_list:
        # Convert '~=' operator to something rpm understands
        # abc ~= 1.1.1
        # should be converted into something like
        # python-abc >= 1.1.1, python-abc < 1.2
        # but not for build-requires
        if req[1] == '~=':
            req[1]='>='
            v = req[2].split('.')
            v.pop()
            v[-1] = str(int(v[-1]) + 1)
            req.append(", %s < %s" % (req[0], ".".join(v)))
        out_list.append(req)

TODO:

Distinguish build_requires/install_requires
handle requirement output for dsc and spec different

I could help with the implementation.
I just wanted to start a discussion what the best way to handle it before starting the implementation.

Quoting https://www.debian.org/doc/debian-policy/ch-relationships.html:
The relations allowed are <<, <=, =, >= and >> for strictly earlier, earlier or equal, exactly equal, later or equal and strictly later, respectively. The exception is the Provides field, for which only = is allowed.

To answer your Debian .dsc files question

@s-t-e-v-e-n-k I already seen it.
Do you know if something like the following would work in dsc?
Depends: python-abc (>= 1.1.1, < 1.2)
or
Depends: python-abc (>= 1.1.1 < 1.2)

Neither:
Depends: python-abc (>= 1.1.1), python-abc (< 1.2)

@s-t-e-v-e-n-k
Thanks for clarification, this is how I implemented it in my first draft.
Just for the record:
I guess you meant
Depends: python-abc (>= 1.1.1), python-abc (<< 1.2)
(Sorry for my nitpicking)


[BUG] - Using emitConsoleCommand  causes  the Unreal Engine application to crash

Frontend Version:
UE5.3
When use the command pixelStreaming.emitConsoleCommand('open xxx.xxx.xxx.xxx:xxxx '), it causes the Unreal Engine application to crash. However, when entering open (xxx.xxx.xxx.xxx:xxxx) through the key press (commonly used to open the Unreal Engine console), the Unreal Engine application runs normally.
Error log
[2024.02.26-12.41.32:318][416]LogLoad: Error: WorldContext requested with invalid context object.
[2024.02.26-12.41.32:319][416]LogOutputDevice: Warning:
Script Stack (0 frames) :
[2024.02.26-12.41.32:323][416]LogWindows: Error: appError called: Assertion failed: false [File:D:\UnrealEngineSource\UnrealEngine\Engine\Source\Runtime\Engine\Private\UnrealEngine.cpp] [Line: 15553]

@BravoDD44
The issue has been moved to the new repository: https://github.com/EpicGamesExt/PixelStreamingInfrastructure/issues/20
It should be live shortly, so you can continue following the issue there. Thank you for your contribution!


app-harness is blank screen on real device

Describe the bug
app-harness is blank screen on real device
To Reproduce

Have real device connected (no matter the platform)
Go to app-harness
Do npx rnv run -p android -d -> blank screen white or black

Expected behavior
app opens with ui
Desktop (please complete the following information):

OS: macOS Ventura 13.5.2
Node Version 18.19.0
RNV Version 1.0.0-rc.10
@rnv/template-starter 1.0.0-rc.10

AndroidTV simulator also has blank screen, other platforms don't including ios, android, tizen, webos, androidwear
tvOS run fails with


Just do the thing already!

When a new version of the bot is available, and it gets auto-deployed, emit an event to any socket.io listeners to refresh so that new features are available ASAP

Instead of emitting an event, we're auto-refreshing on socket.io client reconnects.


How to generate low dim embeddings the for Kitchen Envs

In the dataset, the observation sequence is sized N*60, is there any code snippets to generate the low dim embeddings?

Hi @xavihart I'm not quite sure I understand your question. The native observation space for the kitchen task is 60 (though a lot of these dimensions are empty), and the dataset is directly extracted from *.mjl files published in the original relay-policy-learining repo inside KitchenMjlLowdimDataset

I got it, so it is not vision-based?

Yes, we only have state-based version of the kitchen task in this repo.


please add better explanation about android_disable_fullscreen

i used this package and it worked fine, but then suddenly my app became un-scrollable and the keyboard was covering the TextField, i spent hours trying every solution available online and nothing worked until i came across a guy with the same problem that said you may have <item name="android:windowFullscreen">true</item> in styles.xml but i didn't thought of that because i never touched that file, then i opened the file and it was inside it, i removed it and finally everything went back to normal, i know you guys provide the option to disable the full screen but at least explain the consequences of making it full screen, or at least make it false by default so no one else go through my problem and miss around with all of his layout for hours, thanks.

I agree with the OP here. Same thing happened with me.

Me too, Took me a quite while to figure it out..

Should be resolved by 48981828e831768c4ad89a35475e8f9f24e0aca5.

Should be resolved by 48981828e831768c4ad89a35475e8f9f24e0aca5.


Graphics from AAI Loaders

I like the look of AAI loaders but i do not like having to put lubricant in them, I see there is a graphics only option for that mod is it possible to replace the graphics of this mod with that one?
https://mods.factorio.com/mod/aai-loaders

Added by b0ff64


Its getting late message disable

is there any way to disable the message that appears in the middle of your screen when it gets to the night the message of
"it's getting a bit late, better hit the hay"
it's distracting to some of the server members.

Yes, there is! Go to settings, under messages, set "title: false".


Fix encrypted identity issues

I think we will simply make this trade-off for now. Kybra Beta does not allow you to use encrypted identities for the time being. dfx chunk uploading will hopefully allow us to remove the post_install process and thus get this functionality back.

[!Warning]
Using encrypted identities can apparently cause huge memory usage.

Multiple users have tried to use encrypted identities and then run into issues in the post_install script when trying to clear chunks:
pub fn clear_chunks(canister_name: &str) -> Result<(), String> {
    let output = dfx("canister", "call", &vec![canister_name, "clear_chunks"])?;

    if !output.status.success() {
        return Err(create_error_string(&String::from_utf8_lossy(
            &output.stderr,
        )));
    }

    Ok(())
}

The first line of that function is the first dfx command the the script executes and the assumption is it waits to decrypt the identity and gets stuck here. We're not sure why this uses lots of ram, but WSL users will use up all the ram in their VM until the process just exits.

One potential way to detect if they are using an identity would be to use a combination of dfx identity whoami and checking for the presence of a  ~/.config/dfx/identity/THEIR_IDENTITY_NAME_HERE/identity.pem.encrypted file.

dfx now supports larger Wasm binaries, and this issue is resolved


Issue with apply method of MultibuyDiscountBenefit

Issue Summary
apply method of MultibuyDiscountBenefit does not work as it should. With Count condition it makes all items in the basket FOC.
Steps to Reproduce
Failed test in PR https://github.com/django-oscar/django-oscar/pull/3342
Technical details
If in the test above we will have more than two items in the basket. E.g.
add_product(self.basket, D('20'), 2)
add_product(self.basket, D('100'), 5)

this loop will make 10_000 iterations:
https://github.com/django-oscar/django-oscar/blob/2429ad9e88e9a432dfa60aaca703d99860f85389/src/oscar/apps/offer/applicator.py#L33-L40
Suggested change to apply method, please review in PR https://github.com/django-oscar/django-oscar/pull/3343

was it not possible to just set max cart applications = 1 ?


Simple Q about net core mismatch by build

By building i am Getting this error.

A compatible installed .NET Core SDK for global.json version [3.1.101] from [C:\Users\Remco\source\repos\Avalonia\global.json] was not found
Install the [3.1.101] .NET Core SDK or update [C:\Users\Remco\source\repos\Avalonia\global.json] with an installed .NET Core SDK:
3.1.200 [C:\Program Files\dotnet\sdk]
3.1.202 [C:\Program Files\dotnet\sdk]

Should i update Global.json or download an older version [3.1.101] of the SDK
Regards,
Remco

.NET Core update has somehow broken one of the MSBuild tasks (missing method exception for System.Xml).

So i should install an older version?

Yes. You have to install .Net Core SDK 3.1.101

tnx, will do