/reveal results in a crash and restart

Describe the bug
Running /reveal results in

poke-bot is thinking...

and spins continuously. It doesn't fail until a very long time passes, but it outputs error into console:
poke-guesser-bot       | /usr/src/app/reveal.js:20
poke-guesser-bot       |                     let lowercaseName = encounter[i].name.toLowerCase();
poke-guesser-bot       |                                                           ^
poke-guesser-bot       |
poke-guesser-bot       | TypeError: Cannot read properties of undefined (reading 'toLowerCase')
poke-guesser-bot       |     at Commands.reveal (/usr/src/app/reveal.js:20:59)
poke-guesser-bot       |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
poke-guesser-bot       |
poke-guesser-bot       | Node.js v19.7.0
poke-guesser-bot exited with code 1
poke-guesser-bot       |
poke-guesser-bot       | ><EMAIL_ADDRESS>start
poke-guesser-bot       | > node index.js
poke-guesser-bot       |
poke-guesser-bot       | Logged in as poke-bot#3409!

To Reproduce
Steps to reproduce the behavior:

Start poke-bot
Run /explore to generate pokemon
Run /reveal to result in error

Expected behavior
/reveal is supposed to output the pokemon
Desktop (fill in or erase if not applicable):

OS: MacOS Ventura 13.4
Docker

Possible solutions
The cause of the issue is unknown. It is possible poke-api is providing a different object than what it used to show. Another possible place to look is the below documentation:
https://sequelize.org/docs/v6/core-concepts/getters-setters-virtuals/#getters
https://sequelize.org/docs/v6/core-concepts/model-querying-finders/

Should be fixed by #103 because I've written To make it work, I had to fix #108 as well into the description of that PR. Needs to be tested, tho, to be sure.

Tested but still doesn't seem to be working. But as you know, I still have issues loading up the lightning rounds commands.
When I try to /reveal, It says Pokebot is Thinking for a very long time. It will probably fail like before.
I do not know if this is also just a local issue, so are you able to test the latest develop branch to make sure?

I just tested it with docker-compose, and it worked without any problem. See below:

git status says:
On branch develop
Your branch is up to date with 'origin/develop'.

nothing to commit, working tree clean

git pull says:
Already up to date.

git log's latest commit is:
commit a70bf8c040d00ccf907a250e3ac31c015f036530 (HEAD -> develop, origin/develop)
Merge: f70a049 9c9abcf
Author: George Ciesinski<EMAIL_ADDRESS>Date:   Fri Sep 29 21:58:53 2023 -0400

    Merge pull request #103 from Wissididom/lightning-rounds

I tested again:
Testing
/explore
poke-guesser-bot       | ><EMAIL_ADDRESS>start
poke-guesser-bot       | > node index.js
poke-guesser-bot       |
poke-guesser-bot       | Logged in as poke-bot#3409!
poke-guesser-bot       | Generating a new Pokemon
poke-guesser-bot       | (sequelize) Warning: Model "encounter" is declaring public class fields for attribute(s): "name", "language".
poke-guesser-bot       | These class fields are shadowing Sequelize's attribute getters & setters.
poke-guesser-bot       | See https://sequelize.org/main/manual/model-basics.html#caveat-with-public-class-fields
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: カラカラ, Language: ja-Hrkt
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Karakara, Language: roomaji
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: 탕구리, Language: ko
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: 卡拉卡拉, Language: zh-Hant
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Osselait, Language: fr
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Tragosso, Language: de
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Cubone, Language: es
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Cubone, Language: it
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: Cubone, Language: en
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: カラカラ, Language: ja
poke-guesser-bot       | Guild:<PHONE_NUMBER>87659581, Channel:<PHONE_NUMBER>87659584, Name: 卡拉卡拉, Language: zh-Hans
poke-guesser-bot       | https://raw.githubusercontent.com/PokeAPI/sprites/master/sprites/pokemon/104.png
poke-guesser-bot       | https://raw.githubusercontent.com/PokeAPI/sprites/master/sprites/pokemon/other/official-artwork/104.png
poke-guesser-bot       | (node:19) ExperimentalWarning: buffer.File is an experimental feature and might change at any time
poke-guesser-bot       | (Use `node --trace-warnings ...` to show where the warning was created)

/reveal
poke-guesser-bot       | isMod: true
poke-guesser-bot       | /usr/src/app/reveal.js:20
poke-guesser-bot       |                     let lowercaseName = encounter[i].name.toLowerCase();
poke-guesser-bot       |                                                           ^
poke-guesser-bot       |
poke-guesser-bot       | TypeError: Cannot read properties of undefined (reading 'toLowerCase')
poke-guesser-bot       |     at Commands.reveal (/usr/src/app/reveal.js:20:59)
poke-guesser-bot       |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
poke-guesser-bot       |
poke-guesser-bot       | Node.js v19.7.0
poke-guesser-bot       | npm notice
poke-guesser-bot       | npm notice New major version of npm available! 9.5.0 -> 10.1.0
poke-guesser-bot       | npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.1.0>
poke-guesser-bot       | npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
poke-guesser-bot       | npm notice
poke-guesser-bot exited with code 1
poke-guesser-bot       |
poke-guesser-bot       | ><EMAIL_ADDRESS>start
poke-guesser-bot       | > node index.js
poke-guesser-bot       |
poke-guesser-bot       | Logged in as poke-bot#3409!

Results in:

poke-bot is thinking...

A few seconds later, it outputs:
poke-guesser-postgres  | 2023-09-30 17:05:55.813 UTC [26] LOG:  checkpoint starting: time
poke-guesser-postgres  | 2023-09-30 17:05:57.275 UTC [26] LOG:  checkpoint complete: wrote 17 buffers (0.1%); 0 WAL file(s) added, 0 removed, 0 recycled; write=1.444 s, sync=0.007 s, total=1.463 s; sync files=12, longest=0.004 s, average=0.001 s; distance=60 kB, estimate=60 kB

So it looks like a crash.
Branch & Commit
georgeciesinski@Georges-MacBook-Air poke-guesser-bot % git log
commit a70bf8c040d00ccf907a250e3ac31c015f036530 (HEAD -> develop, origin/develop)

So I'm on the same commit. This is really strange because I am running Docker Compose, so I thought that the container would work the same on both of our machines. For some reason on mine I am consistently getting this error.

I have not tried making code changes yet. I think that we are getting very inconsistent behaviour on your setup and on my setup, so I think this is more important to understand why it is happening before we attempt to resolve this with code changes.

Only thing I noticed is that you seem to be using Node.js v19.7.0, while my run used Node.js 20.3.1. According to the Dockerfile I see the image node:20.3-bookworm-slim being used.

Yeah I agree, I will see if I can find out why this is failing. If I do figure it out I'll update this issue and start the change.

Seems to be fixed now


CSRF optional token regeneration

Related issues
Issue #192 https://github.com/EllisLab/CodeIgniter/issues/192
Issue #843 https://github.com/EllisLab/CodeIgniter/issues/843
Added an option to determine whether or not the token will be regenerated on every submission (regardless or success or failure). This makes it a bit easier to use tabs, back/forward navigation, asynchronous file uploads, etc.

This looks ok but can you please add a note to the changelog and document the feature, explaining when and how it would be useful in the user guide?
Thank you :)

This looks ok but can you please add a note to the changelog and document the feature, explaining when and how it would be useful in the user guide?
Thank you :)

What do you think?


Allow empty style elements in components

Otherwise <style></style> is reported as a syntax error. <script> in components already allowed an empty body, this just extends the same rule to style elements.
Also added a regression test for <script></script>, to make sure that case stays working.

Nice, thanks !


Update docs

Docs are pretty much in a state of disrepair. What I would suggest is using a tool like https://docsify.js.org/ to render a new, more professional looking docs for this lib, salvaging what can be salvaged from the content of the docs and removing the rest.

@aeneasr one extra thing for today, could you please set up github-pages from master docs folder? I will be pushing a PR from brand new docs in a minute.

Sure!
http://react-page.github.io/react-page/

Lovely, thanks!


Figure out why python sdk performance drops with decreasing of DEFAULT_SPLIT_SIZE

It looks like python sdk has internal rpm limits cause it slows down quite much when  DEFAULT_SPLIT_SIZE is changed.
DEFAULT_SPLIT_SIZE = 1
Average response time: 235s for 500 isochrones
DEFAULT_SPLIT_SIZE = 10
Average response time: 51s for 500 isochrones

https://medium.com/@mkaanerkoc/how-to-send-bulk-http-requests-with-aiohttp-asyncio-in-python-62aee008e057

It would be great to add a section in readme which explains why it happens and add some examples how to send parallel requests with python sdk.


Snow and Rain doesn't show

Rain and snow is invisible whenever the mod is installed (have a look into the attached screenshots)
Minecraft version: 1.20.1
Fabric loader version: fabric-loader-14.21-1.20.1 and up to fabric-loader-14.23-1.20.1
Mod version: watson-fabric-1.15.0+1.20.1
Malilib version: malilib-fabric-1.20.1-0.16.1 and up to malilib-fabric-1.20.1-0.16.3
Single-/Multiplayer, other mods, etc. doesn't matter.

I am also seeing this but in my case it is related to the use of my AMD Radeon R5 M330 GPU. When using the built-in Intel HD Graphics 520 GPU, rain/snow show as normal.
Additionally, this also affects causes the small block rendering (stone, andesite, etc.) and vectors to not display when using the AMD GPU.

I use nvidia gpu myself and havent noticed anything so it will be a difficult one to debug what the actual cause is of this happening

If there is anything I should test for you Im happy to help you out

Small block rendering: In Config, "Lists", "Smaller Render Box" shows common stone types with a smaller outline. If any items are in this list, the render box does not render.
Vectors are the line vectors that appear between rendered blocks showing the order (from Config, "Edits", "vectorShown" setting.

I have no clue why the weird behaviour is happening atm, i guess you tried with just watson and malilib

I did, yes. with several Verions of fabric / malilib.

This is how it normally should display, example with smaller render box and vector:

with following CPU and GPU:


[MINOR][PYTHON] refactor PythonWrite to prepare for supporting python data source streaming write

What changes were proposed in this pull request?
Move PythonBatchWrite out of PythonWrite.
Why are the changes needed?
This is to prepare for supporting python data source streaming write in the future.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Trivial code refactoring, existing test sufficient.
Was this patch authored or co-authored using generative AI tooling?
No.

@HyukjinKwon @HeartSaVioR @allisonwang-db PTAL, thanks!

Would you create a Spark JIRA https://issues.apache.org/jira/browse/SPARK and add it to the PR title? Please refer to https://spark.apache.org/contributing.html for details. Thanks!

Thanks, merging to master!


resources with the same name should be either merged together or discarded/dropped

Currently in a multi project when multiple resources gets added to the docker container it will make problems if files with the same name appear.
This is especially true for resource.conf. These should be applied with a MergeStrategy like it is done in sbt-assembly.

Will be fixed in #1


Not showing network stats

Hello, guys!
I'm having very weird issue.
For some coins I can't see network statistics on frontend.
Sometimes flushing all from Redis helps me to fix it, sometimes not.

Do u have any thoughts about that?
Thanks!

how did u fix it ? i have the same problem now .

super easy, just enable paymentProcessing in coin config


Unit of isolation = process and unit of scheduling = test class or custom container?

Currently we are executing our unit and integration tests in parallel using the following settings in the .runsettings file:
<RunSettings>
    <RunConfiguration>
        <MaxCpuCount>0</MaxCpuCount>
    </RunConfiguration>
</RunSettings>

This enables parallelization, but just per assembly. In our case, this is not really ideal, because in some cases we have a few fast assemblies, and just 1 or 2 very large and long running assemblies. So parallelization per assembly does not help much there.
Here's a screenshot, how that looks like (every line is an assembly and every bar with a different color is a test):

I read here that it's possible to change the "unit of scheduling" from assembly to "test class" or even "test method". This seems exactly what we need, but the document states, that this will change the "unit of isolation" from process to thread and I'm pretty sure, our code will not support that (Singletons, static code, ...).
So what I'm asking is, as I wrote in the title: Is it possible to still use an unit of isolation of process, but change the unit of scheduling from assembly to test class or maybe be able to pass custom containers?
I also stumbled upon this tool, which seems to do exactly what we want, but it's outdated and it would be nice to not need an additional tool for that.
Thank you in advance!

@rkieslinger moved it under test platform, because that is where it would have to be implemented.

This is a new feature and won't be implemented, we are focusing on adding new features to Testing.Platform instead.  https://learn.microsoft.com/en-us/dotnet/core/testing/unit-testing-platform-intro?tabs=dotnetcli


Dev rest api

I have finished a version of carmin rest with catidb (/api/carmin/*). I make some updates to carmin.yaml:

unify study name
add missing error codes and messages
return the status of executions instead of only execution identities when list all the executions. It is like ps in linux.
add /pipelines/pipeline get the definition of specified pipeline.
there is a problem with mixed type in swagger.yaml. I cannot define a mixed type for the value of keyValuePair. Therefore I put an object as value of keyValuePair for the moment.

Great, thanks for this!


matTooltip triggers on focus capture since 2.0.0-beta.11

Bug, feature request, or proposal:
Bug?
What is the expected behavior?
2.0.0-beta.10 didn't trigger tooltips on focus capture, e. g. by getting focus after closing a dialog or after closing a sidenav.
What is the current behavior?
2.0.0-beta.11 triggers tooltips on focus capture. Perhaps this is intended when using the keyboard to focus elements with tooltips, but in my case this is not intended.
Maybe provide a way to disable triggering tooltips on focus capture?
What are the steps to reproduce?
http://plnkr.co/edit/5O7j6Mt033VyDEbC05ai?p=preview
Open the sidenav, then press close. Watch the tooltip reappear.
It will stay until something else catches focus or the button gets hovered.
What is the use-case or motivation for changing an existing behavior?
I want the behavior of 2.0.0-beta.10 back.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Material 2.0.0-beta.11
Chrome 61 / Firefox 55

Same issue when reproduced from a dialog: http://plnkr.co/edit/HBaHnOcg1SPQcEvY684f?p=preview


NIFI-2643 Fixed bootstrap issue with vanilla instance

The RunNiFi invocation checked if the key was null instead of blank.

+1, merging


Unable to publish a GoalId without stamp.

I am trying to cancel all goals by sending a GoalId message with empty id and stamp, as was mentioned at ROS wiki.

So far I tried publishing goal like:
Messages.actionlib_msgs.GoalID goal = new Messages.actionlib_msgs.GoalID
            {
                id = "",
                stamp = null
            };

But it would throw an exception as:

Info: System.NullReferenceException at Messages.actionlib_msgs.GoalID.Serialize(Boolean) at Messages.IRosMessage.Serialize() at Ros_CSharp.TransportSubscriberLink.startMessageWrite(Boolean) at Ros_CSharp.TransportSubscriberLink.enqueueMessage(Ros_CSharp.MessageAndSerializerFunc) at Ros_CSharp.Publication.EnqueueMessage(Ros_CSharp.MessageAndSerializerFunc) at Ros_CSharp.Publication.processPublishQueue() at Ros_CSharp.Poll_Signal.threadFunc() at System.Threading.ThreadHelper.ThreadStart_Context(System.Object) at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean) at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean) at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object) at System.Threading.ThreadHelper.ThreadStart()

I'm quite sure, the problem is with the stamp as I was able to publish successfully with same empty id and a filled-up stamp.

Have you tried setting the stamp to 0,0?

I'll try and get back to this.

Thanks

Hi @nuclearmistake ! My publisher is on one computer while my subscriber/Ros master is on another computer. I specifically need to get the .now of the Ros master.
May I know how could I do it?
I saw you have a function Ros_CSharp.ROS.GetTime() without argument. May I know what this is doing?

GetTime() should be synced to either wall time or Ros time if it's available... If I recall

///
///     Gets the current time as secs/nsecs
///
///
public static m.Time GetTime()
{
return GetTime(DateTime.Now);
}
It's in ROS_Comm/_init.cs

Usually NTP and windows network time are advised, as Ros /time is usually more of bag / stage / gazebo thing

hmm, I see. So I really just have to find a way to set stamp to null. Is it possible with this library?

You could publish a goalid from a regular Ros not with a "null" stamp, and watch what is actually sent in rostopic echo... Then emulate that behavior manually in ros.net.
I guarantee transmitting a null stamp on rospy or roscpp does actually send something... Make your ros.net code sent that
That should behave like you want, although you're not literally using a null stamp

Ok, will try that. Thanks for the help!

You're welcome

This solves the issue:
Messages.actionlib_msgs.GoalID goal = new Messages.actionlib_msgs.GoalID
            {
                id = "",
                stamp = new Messages.std_msgs.Time
                        (
                            new Messages.TimeData
                            {
                                sec = 0,
                                nsec = 0
                            }
                        )

            };

Thanks @nuclearmistake ! You may now close this issue!

Glad to hear that you got it working.


Feat/tabs

Created according to best practices - https://www.w3.org/TR/wai-aria-practices/examples/tabs/tabs-2/tabs.html.
With a little help from material-ui source - https://github.com/mui-org/material-ui/blob/next/packages/material-ui-lab/src/TabList/TabList.js and https://github.com/mui-org/material-ui/blob/next/packages/material-ui/src/Tab/Tab.js

verify

verify


requireRootUser exit 0 regardless of the current user

There are multiple places that function "requireRootUser" is called. This function currently has not been implemented and it returns 0 regardless of the current user.
Actual behavior should be checking the current user and return 1 (false) if the user is not 0 otherwise it can returns 0.
The function is used in different places:

scripts/join.sh
scripts/upgrade.sh
scripts/common/preflights.sh

Hi @amiraliucsc,
This one is very old and it seems like sorted out for the latest releases.
In this away, I am closing this issue. However, if you see that still required after check the latest version please feel free to re-open this one and let us know more about.
Thank you for your collaboration.


total storage requires size metrics

layout.xsl should include:
<li title="account size">
 <xsl:choose>
  <xsl:when test="size >=<PHONE_NUMBER>">
   <xsl:value-of select="format-number(size div<PHONE_NUMBER>, '#,###')"/>
   <xsl:text>Gb</xsl:text>
  </xsl:when>
  <xsl:when test="size >= 1048576">
   <xsl:value-of select="format-number(size div 1048576, '#,###')"/>
   <xsl:text>Mb</xsl:text>
  </xsl:when>
  <xsl:when test="size >= 1024">
   <xsl:value-of select="format-number(size div 1024, '#,###')"/>
   <xsl:text>Kb</xsl:text>
  </xsl:when>
  <xsl:when test="size > 0 and size < 1024">
   <xsl:value-of select="format-number(size div 0, '#,###')"/>
   <xsl:text>bytes</xsl:text>
  </xsl:when>
  <xsl:otherwise>
   <xsl:text>0 bytes</xsl:text>
  </xsl:otherwise>
 </xsl:choose>
</li>

@yegor256 let me know if this code checks out.

@k0r FYI, see https://help.github.com/articles/github-flavored-markdown/

@k0r yes, it looks correct to me

@yegor256 thanks.

@yegor256 would we use size or $size' for when test = ` ?

@rultor hello

@rultor hello

@k0r Hi there! I understand a few simple commands: "merge", "deploy", etc. This page explains them briefly. Any questions or bug reports are welcome. Have fun :)

@rultor release, tag is 0.3.24

@rultor release, tag is 0.3.24

@k0r Thanks for your request. @yegor256 Please confirm this.

@rultor release, tag is 0.3.24

@rultor release, tag is 0.3.24

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.3.24

@yegor256 Done! FYI, the full log is here (took me 10min)

@yegor256 thanks for the confirmation.


Add row and column indices to the doConvertToJavaObject method to provide detailed context when an error occurs.

Add row and column indices to the doConvertToJavaObject method to provide detailed context when an error occurs.
fix #132

LGTM


Go Slow

No merge!

The issue was that I pasted benaadams:patch-1 as the branch name when it should have been patch-1, but I tried it, and it was still fast ;) maybe because it's the platform benchmark.


Make sure scripts with symbols in path name work

This PR is intended as a patch for https://github.com/lihaoyi/Ammonite/issues/418.
Before this PR any script having special chars in path name fails because while loadingClasses wrapperNames are encoded whereas while findClass is given non-encoded Name as argument.
This PR fixes this issue.

I have verified that this fixes the problem, but could you add a unit test that replicates the original problem? i.e. run a script that lives outside of the working directory of the shell used to start the script? Ammonite-Ops lets you pass in a working directory when it runs external processes via %%.

Just one tweak; when you're done squash it and I'll merge it


Outdated personal data statement

Where to find the issue

https://www.coronawarn.app/en/
https://www.coronawarn.app/de/

Describe the issue
The statement on https://www.coronawarn.app/en/ under the heading "Data Privacy and Safety", sub-heading "Two objectives" is very outdated. It says:
Only personal data needed for the following two objectives will be processed:

Assess personal risk of infection
Learn COVID-19 test results faster.

The Privacy notice 16 March 2022, version 2.19 identifies many other data categories which are not mentioned in the above statement:

Data about your COVID vaccination (data in the COVID vaccination certificate)
Data about your recovery (data in the COVID recovery certificate)
Data in the COVID test certificate
Booking information
Usage data (data sharing)
Participation in a survey
Contents of the error reports
Confirmation of the authenticity of your app

Suggested change
Review the section "Two objectives".
Possibly:

remove it or
bring it up to date in detail or
make it more general and refer to the Privacy notice document

Related issue

https://github.com/corona-warn-app/cwa-website/issues/1729

I think this and the personal_data FAQ entry should be changed to just point to the Privacy notice. Everything else does not really make sense, IHMO.

@Ein-Tim

I think this and the personal_data FAQ entry should be changed to just point to the Privacy notice. Everything else does not really make sense, IHMO.

This is also my feeling. I believe that the Open Source Team themselves need to make the decision on this important topic. It should not have to be a community decision.

IHMO

Friendly reminder: if you are trying to write an abbreviation for "In My Honest Opinion", then the letters come in the order IMHO, not IHMO! 🙂

@MikeMcC399 @Ein-Tim It looks like the Data Privacy Document piece within that same section already contains links to the current PDF as well as a link to /en/privacy for previous versions of the PDF. I would think having another subsection right next to the Data Privacy Document subsection that merely points to the PDF as well would be a bit repetitive/unnecessary.
I would think removing this subsection entirely or updating the objectives to be a bit more up to date would be the two better options. If agreed upon, once the Open Source Team weighs in I can get a PR together (if that's ok with you @MikeMcC399 )

@ahodzic2
Yes, I'm in favour of just removing that part. It's really up to the Open Source Team to comment now. Right now though they are busy trying to get the new FAQ structure fixed and "out-of-the-door".
Normally for issues that I have submitted myself I would also submit the corresponding PR myself, providing I have the technical ability to solve the issue. That is the case here as well. If there is any doubt about what is wanted by the Open Source Team I would clarify that first before PR submission, as it would only create unnecessary work to submit something which would not get approved.
Thanks for your offer in any case.

@ahodzic2
Yes, I'm in favour of just removing that part. It's really up to the Open Source Team to comment now. Right now though they are busy trying to get the new FAQ structure fixed and "out-of-the-door".
Normally for issues that I have submitted myself I would also submit the corresponding PR myself, providing I have the technical ability to solve the issue. That is the case here as well. If there is any doubt about what is wanted by the Open Source Team I would clarify that first before PR submission, as it would only create unnecessary work to submit something which would not get approved.
Thanks for your offer in any case.

Thank you for the clarification @MikeMcC399 . Looking to get more involved with open source projects as part of my graduate studies so that breakdown definitely helps with understanding the process better :)

@dsarkar
Do you want me to submit a PR for this? It's a non-urgent tidy-up task.

@MikeMcC399 Thanks for your offer, let me clarify internally first. I guess your proposal

make it more general and refer to the Privacy notice document

is the way to go.

@dsarkar
Thanks for picking this up.

make it more general and refer to the Privacy notice document - is best, as it should then not need updating again
remove it - is simplest

@MikeMcC399 Suggestion:
Objectives
Only personal data needed for the following objectives will be processed:

Assess personal risk of infection
Learn COVID-19 test results faster.
Regarding other objectives see the privacy notice.

PR #2750 submitted to resolve this issue


Refactor setup.py and move most hardcoded information into spec.yaml files

This pull request partially resolves #2490 by moving the most of the hardcoded information about forks into spec.yaml, prefix.py and suffix.py files in the respective fork directories.

Getting the linting errors cleaned up would probably make this easier to merge/consider.

Have force pushed a commit to fix the linter errors. I've also made setup.py get linted and fixed the lint errors inside it.
A very slight regression is that the autogenerated files no longer meet the requirements of E402 (imports at top of file) and E3XX (wrong number of empty lines). I've added linter.ini ignores for those.

I'm fine reviewing the python here, but I think it'd be good to have someone with more hands-on experience with working with this spec weigh in prior to that to verify that this is a change they are interested in adopting.

I think ignoring E402 is okay

not sure how involved we want this particular PR to be but i think a more complete solution here takes the code in {prefix, suffix}.py and patches that into the spec AST/IR we informally have in setup.py.  the reason to prefer this solution is that we may have various modifications to do to the spec content before rendering a final Python file (say, after any de-duplication/redefinition required is done).
i'm just commenting this here now as it is top of mind given some work i was just doing on spec code coverage -- we re-duplicate some phase0 spec functions into the altair spec and ideally we drop the ones no longer used in subsequent forks
we can definitely keep this PR as is and handle some of these wider reaching refactors in subsequent PRs

I've implemented inheritance between spec.yaml files to eliminate repetition. I've also created a SpecConfig object to hold all the information from spec.yaml, which makes the code neater.


Fixes #197: OCMInvocationMatcher should release recordedInvocation on dealloc

As far as I can tell this is a duplicate of #198 which I just merged. If not, please re-open.


i/o timeout when upgrading vm

When running docker-machine upgrade for a vm, the upgrade operation timesout. I'm running machine on OS X 10.10.3
The output is as follows:
Stopping machine to do the upgrade...
Upgrading machine devdocker...
Downloading https://github.com/boot2docker/boot2docker/releases/download/v1.7.0/boot2docker.iso to /Users/sarvagnan/.docker/machine/cache/boot2docker.iso...
Get https://github.com/boot2docker/boot2docker/releases/download/v1.7.0/boot2docker.iso: dial tcp: i/o timeout
How do I fix this?
Thanks!

In my case (as commented in bug #1274), I had to re-order nameservers in my /etc/resolv.conf to pick up a different nameserver. Check with nslookup if you can resolve the server's address.

This is my resolv.conf

Mac OS X Notice

This file is not used by the host name and address resolution
or the DNS query routing mechanisms used by most processes on
this Mac OS X system.

This file is automatically generated.

nameserver <IP_ADDRESS>
As to nslookup, github.com is resolved, but https://github.com, github.com/boot2docker/boot2docker, https://github.com/boot2docker/boot2docker/releases/download/v1.7.0/boot2docker.iso, and github.com/boot2docker/boot2docker/releases/download/v1.7.0/boot2docker.iso are not. Should I add something to my resolv.conf for this to work?

you can open your network preferences > advanced > DNS and add some DNS servers explicitly.

That worked. Thanks!


DMXSerial has conflict with SD library

I am trying to use the SD library (to read a file from an SD card) and DMXSerial together.
Unfortunately both seem to define the same interrupt - the compiler throws an error - even if I compile a totally empty project with just the two include statements.
My minimized test sketch:

#include <SD.h>
#include <DMXSerial.h>
void setup() {
// put your setup code here, to run once:
}
void loop() {
// put your main code here, to run repeatedly:
}

The error messages of the compiler:
Arduino: 1.8.5 (Windows 10), Board: "Arduino Ethernet"
HardwareSerial0.cpp.o (symbol from plugin): In function `Serial':
(.text+0x0): multiple definition of `__vector_18'
libraries\DMXSerial-master\DMXSerial.cpp.o (symbol from plugin):(.text+0x0): first defined here
HardwareSerial0.cpp.o (symbol from plugin): In function `Serial':
(.text+0x0): multiple definition of `__vector_19'
libraries\DMXSerial-master\DMXSerial.cpp.o (symbol from plugin):(.text+0x0): first defined here
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino Ethernet.
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

This is a know issue of the SD library: It uses the Serial interface for debugging output purpose.

HardwareSerial0.cpp.o (symbol from plugin): In function `Serial':

I propose you to remove all the Serial references in your local SD library.


NO-JIRA: e2e: openstack: fix nil deref in route53 teardown

observed in MCO rehearsal
--- FAIL: TestCreateCluster (2136.19s)
    --- FAIL: TestCreateCluster/ValidateHostedCluster (1908.04s)
    --- FAIL: TestCreateCluster/Teardown (225.96s)
        --- PASS: TestCreateCluster/Teardown/ValidateMetricsAreExposed (0.07s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x712fa07]

goroutine 1383 [running]:
testing.tRunner.func1.2({0x799cd20, 0xb3ccb30})
	/usr/lib/golang/src/testing/testing.go:1631 +0x49e
testing.tRunner.func1()
	/usr/lib/golang/src/testing/testing.go:1634 +0x669
panic({0x799cd20?, 0xb3ccb30?})
	/usr/lib/golang/src/runtime/panic.go:770 +0x136
github.com/openshift/hypershift/test/e2e/util.deleteIngressRoute53Records(0xc000bbab60, {0x8d44be8, 0xc000a9d130}, 0xc00137f008, 0xc001002a08)
	/go/src/github.com/openshift/hypershift/test/e2e/util/util.go:1492 +0xb87
github.com/openshift/hypershift/test/e2e/util.teardownHostedCluster(0xc000bbab60, {0x8d44be8, 0xc000a9d130}, 0xc00137f008, {0x8d574e0, 0xc000eccbd0}, 0xc001002a08, {0xc0014fe810, 0x21}, 0x0)
	/go/src/github.com/openshift/hypershift/test/e2e/util/hypershift_framework.go:402 +0x119d

https://storage.googleapis.com/test-platform-results/pr-logs/pull/openshift_release/57194/rehearse-57194-pull-ci-openshift-machine-config-operator-master-e2e-openstack-hypershift/1839081537698533376/build-log.txt
Seems to be triggered by earlier failure in ValidateHostedCluster
=== RUN   TestCreateCluster/ValidateHostedCluster
    util.go:1505: Creating Ingress Route53 Record for HostedCluster example-zc75b
    util.go:146: Waiting for kubeconfig to be published for HostedCluster e2e-clusters-jl5nh/example-zc75b
    eventually.go:220: observed *v1beta1.HostedCluster e2e-clusters-jl5nh/example-zc75b invalid at RV 11891 after 0s: expected a kubeconfig reference in status
    util.go:146: Successfully waited for kubeconfig to be published for HostedCluster e2e-clusters-jl5nh/example-zc75b in 51s
    util.go:163: Waiting for kubeconfig secret to have data
    eventually.go:220: observed *v1.Secret e2e-clusters-jl5nh/example-zc75b-admin-kubeconfig valid at RV 12689 after 0s: expected secret to contain kubeconfig in data
    util.go:163: Successfully waited for kubeconfig secret to have data in 0s
    util.go:206: Waiting for a successful connection to the guest API server
    eventually.go:100: Failed to get *v1.SelfSubjectReview: Post "https://ac04d3ce43ec7444db9aa372573ea145-23216860.us-east-1.elb.amazonaws.com:6443/apis/authentication.k8s.io/v1/selfsubjectreviews": dial tcp: lookup ac04d3ce43ec7444db9aa372573ea145-23216860.us-east-1.elb.amazonaws.com on <IP_ADDRESS>:53: no such host
    util.go:206: Successfully waited for a successful connection to the guest API server in 57.025s
    util.go:1505:
        failed to get router-default service IP
        Unexpected error:
            <*errors.errorString | 0xc000f8a210>:
            router-default service did't become available: context deadline exceeded
            {
                s: "router-default service did't become available: context deadline exceeded",
            }
        occurred

/lgtm
Thanks


Non accessible Vitis-AI-Copyleft-Model-Zoo repository

In the Vitis-AI 3.5 doc, at https://xilinx.github.io/Vitis-AI/3.5/html/docs/workflow-model-zoo.html, it is written:
Vitis AI Copyleft Model Zoo[¶](https://xilinx.github.io/Vitis-AI/3.5/html/docs/workflow-model-zoo.html#vitis-ai-copyleft-model-zoo)

Many open-source models are released under reciprocal license terms which are not compatible with Apache 2.0. In order to faciliate the support of such models, and clearly distinguish the source license for each, we have created a separate Model Zoo repository. Users will find the training code for these models (for example, YOLOv7) in the [Vitis AI Copyleft Model Zoo](https://github.com/Xilinx/Vitis-AI-Copyleft-Model-Zoo). All other models are found in the primary Vitis AI repository.

However, Vitis-AI-Copyleft-Model-Zoo is not accessible. Maybe you forgot to make it public?
Than you

@iraadit Thanks for mentioning this!  Yes, it has been a bit delayed.  We just received final legal approval in the past 24 hours.  You can expect it to be online very shortly.

Thank you for your prompt answer!
I'll be looking forward to the public release of this repo.
Closing the issue.

@iraadit I can confirm that the repo is now public.  We will be adding some additional guides to this repo for some AGPL models in the near future, but for now this is what we have released:  https://github.com/Xilinx/Vitis-AI-Copyleft-Model-Zoo

Thank you for the code and the follow-up !


WIP: Changes to Remove Open3D

This is a work in progress PR.

@PranjalSahu this is on the master branch. I can't merge this PR as it will propagate to the extension and we are not ready to release it yet.

@PranjalSahu you can clone your own repository locally to your computer and use the extension wizard to to install it. make sure you remove the original slicermorph extension first though. @smrolfe can help you if you need more explicit instructions


Include RFC4646 locales

RFC4646 defines locale strings with a hyphen, not an underscore. Browsers (tested Chrome, Firefox, Safri, Opera, IE) also seem to return an RFC4646 compliant string when probing the window.navigator object. Would be nice to have compatible file names included in the locales directory. <3z

Dear sir/madam
I agree. I sometimes see them both ways, so I think I'll just add the hyphens and leave the underscores. Fair? Balanced?

Fair and balanced. Thank you!


ddclient: Fix capitalization of systemd unit keys

This avoids the following warning:
Apr 19 10:53:48 xen systemd[1]: [/nix/store/...-unit-ddclient.service/ddclient.service:19] Unknown lvalue 'type' in section 'Service'

As Type=simple is the default in systemd, this shouldn't have any
functional changes.

As Type=simple is the default in systemd, this shouldn't have any functional changes.

Then why not drop it entirely?

Well, right above that is that comment about potentially changing it to Type=forking, which would need rewording to potentially something more complex to convey the same message.

Well, I guess I turned it into
# Uncomment this if too many problems occur:
# Type = "forking";

in my head? General systemd attitude is don't repeat the defaults, but I admit it's no big deal either way.

Yes, that's good. Force-pushed.

Thanks! Merged.


Review to shared tests

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Configuration errors are swallowed

What is your Scenario?
Setup testcafe configuration with errors such as missing dependencies or incorrect commonJS require paths.
What is the Current behavior?
Testcafe reports an error relating to the config:
An error has occurred while reading the ".testcaferc.js" configuration file.
ERROR You have not specified a browser.

What is the Expected behavior?
Testcafe should report the underlying error thrown in the readJsConfigurationFileContent method of configuration-base.ts
ERROR Error: Cannot find module 'axe.config.js'

What is your public website URL? (or attach your complete example)
N/A
What is your TestCafe test code?
N/A
Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce
See current behaviour
TestCafe version
1.18.4
Node.js version
No response
Command-line arguments
none
Browser name(s) and version(s)
No response
Platform(s) and version(s)
No response
Other
No response

I researched the issue and found that you can use the internal TestCafe log to show the error message.
To enable the TestCafe log, you need to set the DEBUG environment variable as follows:
DEBUG=testcafe:*

As for the ERROR You have not specified a browser. error message, it looks like expected behavior, because the configuration file was not loaded properly.


Add back dupe header fix in local mode

Brings back https://github.com/awslabs/chalice/pull/314 and adds a fix for the tests in py3.

Codecov Report

Merging #318 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #318      +/-   ##
==========================================
+ Coverage   88.92%   88.93%   +<.01%
==========================================
  Files          18       18
  Lines        1860     1861       +1
  Branches      224      224
==========================================
+ Hits         1654     1655       +1
  Misses        152      152
  Partials       54       54

Impacted Files
Coverage Δ

chalice/local.py
94.11% <100%> (+0.05%)
: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 6164d5b...ed764bc. Read the comment docs.


Crashing

I have followed all the steps, the tracking works perfectly, but when I start steam VR it just crashes. When I launch the vr without the motion controllers on it works perfectly with looking around etc, but when I turn the controllers on it just crashes upon launch.

Also, I am using one PsEye camera right now and I'm using 2 motion controllers.

I'm having a similar issue. Suddenly I started to get a 108 error from SteamVR and the message "Failed to receive UDP connection id: No connection could be made because the target machine actively refused it" from Psmoveservice.
One day was working perfectly, other day it apears these errors.
I already spend 2 days trying to solve this issue, but I got no solutions.
I suspect that the problem came with a windows 10 update.
Damn! Another Bill Gate's headeache!

I figured out my problem, it was a joystick controller that was been detect as a Psnavi.
Unplug all other USB devices of your PC.
Reinstall pseye drivers can resolves the problem too.

I figured out my problem, it was a joystick controller that was been detect as a Psnavi.
Unplug all other USB devices of your PC.
Reinstall pseye drivers can resolves the problem too.

Same here, 1.6 crashing when xbox 360 controller plugged. 1.5 is working fine.


[READY] centralization of the starting point for reader and main app

fixes #882 #897

Note that this PR messes-up the reader/library CSS due to combining both scopes into the same WebPack bundle. This is because we are staging multiple consecutive (small) PRs in order to solve the bigger issues, incrementally. So the develop branch currently runs but has some rendering bugs.


Error messages should indicate the page/block where they occured

Currently the sync outputs warnings everytime it encounters an unsupported block type or similar parsing issue.
This info is also rendered into the generated markdown
While this is great to locate the error in the generated output (e.g. git diffs), it misses critical context when only reviewing the build output e.g. on a CI/CD runner. It would thus be great if the error messages/loggers would indicate the file/block where they encountered the problem.
To implement this, would probably need to smarten up the logger (e.g. use proper logging lib instead of console.log).

Fixed in v0.6,see  856ad54ff26a41bd854c2f39f8a0943c8e5c483e


When I enter "node bot.js" I get errors and the bot is ofline.

When I eneter "node bot.js" into the node.js command prompt, I get lots of errors. Also, in my server, the bot is offline. This is what is says in node.js when I enter the command:
C:\Max\Code\discordbot\test\node_modules\winston-transport\legacy.js:17
throw new Error('Invalid transport, must be an object with a log method.');
^
Error: Invalid transport, must be an object with a log method.
at new LegacyTransportStream (C:\Max\Code\discordbot\test\node_modules\winston-transport\legacy.js:17:11)
at DerivedLogger.add (C:\Max\Code\discordbot\test\node_modules\winston\lib\winston\logger.js:277:7)
at Object.winston.(anonymous function) [as add] (C:\Max\Code\discordbot\test\node_modules\winston\lib\winston.js:84:34)
at Object. (C:\Max\Code\discordbot\test\bot.js:6:8)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Function.Module.runMain (module.js:693:10)
Please help!

I'm getting the same thing, it seems to be failing when it hits logger.add(logger.transports.Console...

Having the same issue. Looking at the Winston API, something like:
logger.add(new logger.transports.Console, {
colorize: true
});

Fixes the error, however it does not affect the offline state of the bot

@IliyanPlIvanov What happens now? Does it still show the error?

WORKS!!!

before doing the solution that @HudsonGraeme put up in his edit, install git to $PATH here: https://git-scm.com/downloads

Having the same issue. Looking at the Winston API, something like:
logger.add(new logger.transports.Console, {
colorize: true
});

Fixes the error, however it does not affect the offline state of the bot
EDIT: Run the below and it should work ;)
npm install discord.io github:woor/discord.io#gateway_v6 winston --save
Author: @orkc #84

before running the npm command you need to install git and add it to your path but this worked perfectly thanks!

I also get errors with my bot offline. but i get these:

@TizwigTFC  Simply add a \ before the ' and you should be fine.


How can scroll to the selected item？(怎样才能滚动到所选项目)

E.gscrollToRow(at indexPath: IndexPath, at scrollPosition: UITableView.ScrollPosition, animated: Bool)
例如scrollToRow(at indexPath: IndexPath, at scrollPosition: UITableView.ScrollPosition, animated: Bool)
What should I do on CollectionKit?
我在CollectionKit上应该如何做？
Thank you very much for this open source library!
非常感谢这个开源库！

First get the items index. I usually do something like provider.dataSource.data.firstIndex { ... }
Then:
if var frame = collectionView.frame(at: indexForItem) {
  collectionView.scrollRectToVisible(frame, animated: true)
}

Thank you very much for your answer, but still have questions.
collectionView.frame(at: indexForItem)
I didn't find the frame(at:XX) method in the CollectionView class.
Can you elaborate on this?
Thanks again!

Ah you are right, this must be a function I added myself. I’m not at work so I don’t know my exact implementation but if I remember correctly it just returns the flattenedProvider.frame(at:)

Thank you solve my problem 👍


Fix baichuan npu long sequence perf

Description
Fix baichuan npu long sequence perf

https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/9977650890


Get Report GET_FLAT_FILE_ACTIONABLE_ORDER_DATA_SHIPPING (getReportDocument )

Hello, I need help.
I'm trying to get a report (GET_FLAT_FILE_ACTIONABLE_ORDER_DATA_SHIPPING) using the function  getReportDocument , but the api return this error :
"errors": [
{
"message": "Access to requested resource is denied.",
"code": "Unauthorized",
"details": ""
}
]
and when we add the path : /reports/2021-06-30/reports, the api return this error :
{
"errors": [
{
"code": "InvalidInput",
"message": "Invalid or unsupported document ID found",
"details": ""
}
]
}
Here are all our app authorization : https://prnt.sc/SR840DY3LRt1

Selling Partner Insights
Amazon Fulfillment
Pricing
Finance and Accounting
Product Listing
Inventory and Order Tracking
Direct-to-Consumer Shipping
Tax Remittance
Tax Invoicing

This is our code to call the function getRdtCredentials :
$cred = $credentials->getRdtCredentials([
'restrictedResources' => [
[
'method' => 'GET',
'path' => '/orders/v0/orders'
],
[
'method' => 'GET',
'path' => '/orders/v0/orders/{orderId}/orderItems'
],
[
'method' => 'GET',
'path' => '/orders/v0/orders/{orderId}/address'
],
[
'method' => 'GET',
'path' => '/orders/v0/orders/{orderId}/buyerInfo'
],
// [
// 	'method' => 'GET',
// 	'path' => '/reports/2021-06-30/reports'
//   ]
]
]);
$request = new Reports($cred, $config);
$reportDocumentId = $params['reportDocumentId'];
$reports = $request->getReportDocument($reportDocumentId)
I really appreciate your help
Thank you.

It seems this report is part of the Restricted Data, so you need to exchange your access token for an RDT token (if your developer account is approved for that)
You can check the README section on RDT
https://github.com/double-break/spapi-php#working-with-restricted-data-apis


build: fix build for g++-10

@ihedvall Hi, I got these errors when building with g++-10. Could you review it? :bow:
mdflib/mdflib/src/ichannelgroup.cpp:44:19: error: 'find_if' is not a member of 'std'; did you mean 'find'?
   44 |   auto itr = std::find_if(cn_list.begin(), cn_list.end(),
      |                   ^~~~~~~
      |                   find

mdflib/mdflib/src/mdfwriter.cpp:824:41: error: call of overloaded 'Parameter(size_t&, long long unsigned int)' is ambiguous
  824 |         cc_length->Parameter(index, 0ULL);
      |                                         ^
In file included from mdflib/mdflib/../include/mdf/ichannel.h:17,
                 from mdflib/mdflib/../include/mdf/ichannelgroup.h:13,
                 from mdflib/mdflib/../include/mdf/idatagroup.h:14,
                 from mdflib/mdflib/src/mdfwriter.cpp:8:
mdflib/mdflib/../include/mdf/ichannelconversion.h:195:8: note: candidate: 'void mdf::IChannelConversion::Parameter(size_t, double)'
  195 |   void Parameter(size_t index, double parameter);
      |        ^~~~~~~~~
mdflib/mdflib/../include/mdf/ichannelconversion.h:215:8: note: candidate: 'void mdf::IChannelConversion::Parameter(size_t, uint64_t)'
  215 |   void Parameter(size_t index, uint64_t parameter);

It seems to be fixed by https://github.com/ihedvall/mdflib/pull/50.


JAVA : Java Int to String

Write your solution to this JAVA problem of hackerrank:-
Problem link : Java Int to String

Please assign me, I'll work on this issue

@Riddhi9570 I want to be assigned

I am willing to contribute to this solution. As a part of hacktober-fest contributor

I am willing to contribute to the solution


Only allow org members to add other users to an org

This change adds a check to see if the user is an org member before allowing link creation between user and org.
This link is used to calculate marker permissions so needs to be specially protected.
It also closes some other holes that would affect the ability to "join" an org.

Users can no longer create a link between a user and an org if they
are not a member or creator of the org
When a user creates an org, they automatically become a member of the org
Users can no longer create "was created by" links manually, as this could be used to bypass org creator validation

Note: This is a temporary solution until we have a better way to handle this. Once we are able to distinctly permission writes in AutumnDB, we can remove this.
Change-type: minor
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@joshbwlng I cracked on with the org membership heuristics - it's not ideal but the code came out quite clean. Let me know what you think!

@balena-ci rebase

@LucianBuzzo Testing in main repo with https://github.com/product-os/jellyfish/pull/8912

@balena-ci rebase

@balena-ci I self-certify!


Update collector.go

I'm closing this for now. We've created an internal development ticket to track adding support for these metrics in a performant way, and we can revisit at that point.
Thanks again @benrurepo


Update view recorded side panel

We need to make some enhancements to the current View recorded code in the script recorder updating it to the new design as following

@liciavale a question about the "copy to clipboard" button - do we need this? The code block component has a copy to clipboard feature in its upper right already. Unless you think it'd be better to have a more prominent UI exposed for this feature, we can hide the copy button on the code block.

Hey, @justinkambic yes, this is a good point, we could probably remove the icon on the top right panel for now. I'll update this design and we keep it simple for now


How can I evaluate daivs17 test-dev performance?

hello hank you for the nice paper.
I want to evaluate the performance of test-dev after reading the paper.
I did the evaluation, but how can I get the score??

https://codalab.lisn.upsaclay.fr/competitions/6812


Map typography style

In the styles  folder, create a typography.SCSS file that will contain all the existing fonts in the project

closed by #258


A11y_SQLAzureDataStudio_Extension_Install-UninstallExtension_ColorContrast: Color contrast ratio is below 4.5:1 for some text present on the screen.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
#A11yMAS;#A11yTCS;#SQL Azure Data Studio;#DesktopApp;#Win32;#Benchmark; #A11y_SQLAzureDataStudioBenchmark;#A11ySev1;#MAS1.4.3;#ColorContrast;
Environment Details:
Version: 1.16.0-insider
Commit: 506c6a5
Date: 2020-02-17T05:43:18.399Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Additional Details:
MAS reference: MAS1.4.3
Repro Steps:

Launch Azure Data Studio.
Select the Connections icon (if not already selected) from left bar.
Hover over the 'Servers' header and select the 'New Connection' icon.
Provide server and windows authentication details, then click 'Connect'.
5.Check if minimum color contrast ratio is met for all texts present on the screen.

Actual:
The minimum color contrast ratio is not met for the text present on the screen.
Expected:
The Color contrast ratio must be =>4.5:1 for the content present in the screen.
User Impact:
Low vision users might face difficulty in visualizing what text is present on the screen.
Attachment link reference:
9211_A11y_SQLAzureDataStudio_Extension_Install-UninstallExtension_ColorContrast.zip
9211_A11y_SQLAzureDataStudio_ConnectToASQLServer_Servers_ColorContrast.zip

both issues are vscode issue, please open an issue for vscode repo.

Verified the issue now issue is not repro'ing.
GitHub Tags: #Closed;

GitHubTags:#SQLADS-Win32-Feb2020;


Add integration with the Hugging Face Hub

Hi everyone!
Here is a PoC to start working on an integration between Keras and the Hugging Face Hub. It allows to save a model to the Hub and load it back. For now the implementation is quite rough as I'm not familiar with the Keras internals + I did some code duplication. My additions are mainly:

save a model to a folder (internal)

=> equivalent to an unzipped .keras file

save model to the Hub

=> huggingface_hub.create_repo + save model to temporary folder + huggingface_hub.upload_folder

load model from a folder (internal)

=> equivalent to load from unzipped .keras file

load model from the Hub

=> huggingface_hub.snapshot_download + load model from downloaded folder

API design:
Save to HF with model.save("hf://...")
>>> import keras
>>> model = keras.Model(...)
>>> model.save("hf://Wauplin/mnist_example")

Load it back with keras.saving.load_model("hf://...")
>>> import keras
>>> model = keras.saving.load_model("hf://Wauplin/mnist_example")

Colab example: https://colab.research.google.com/drive/1aR0bnC5itdFVM12EgGLEx1bL5onWT2K-?usp=sharing
HF repo example: https://huggingface.co/Wauplin/mnist_example

I am not familiar with the Keras internals, so this is more a PoC to discuss before a proper implementation. Let me know what you think!
(disclaimer, I work at HF as maintainer of huggingface_hub :hugs:)

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

Project coverage is 56.86%. Comparing base (11be99a) to head (cbf4ec9).
Report is 1 commits behind head on master.

Files
Patch %
Lines

keras/src/saving/saving_lib.py
11.84%
65 Missing and 2 partials :warning:

keras/src/saving/saving_api.py
71.42%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19854       +/-   ##
===========================================
- Coverage   78.83%   56.86%   -21.98%
===========================================
  Files         498      498
  Lines       45864    45937       +73
  Branches     8450     8464       +14
===========================================
- Hits        36159    26124    -10035
- Misses       7999    18203    +10204
+ Partials     1706     1610       -96

Flag
Coverage Δ

keras
56.86% <16.86%> (-21.83%)
:arrow_down:

keras-jax
?

keras-numpy
56.86% <16.86%> (-0.09%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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

Good idea, I'd suggest just
model.save("./local_dir", zipped=False)
keras.saving.load_model("./local_dir")

Let me take a quick look at implementing it.

Good idea, I'd suggest just
model.save("./local_dir", zipped=False)
keras.saving.load_model("./local_dir")

Let me take a quick look at implementing it.

This is done -- let me know what you think.

Thanks a lot for your comments and changes @fchollet @mattdangerw! I addressed most comments + merged the recent change to use the new zipped parameter. It makes this PR much cleaner :)
About zipped default value when saving the model, what do you think of defaulting to None? If None the model is saved as unzipped if the filepath starts with hf://, otherwise it's saved as zipped. Problem otherwise is that one need to explicitly write model.save("hf://Wauplin/densenet_121_example", zipped=False) which is redundant since we don't support saving as zipped on the Hub. What do you think @fchollet? (this is what I did in this PR but happy to revisit)
And finally, about displaying the model summary plot in the model card, I replied in https://github.com/keras-team/keras/pull/19854#discussion_r1642748190

Thanks you -- Looks good to me!

We're almost ready to merge -- can you please address the last comments from @mattdangerw ?

@fchollet @mattdangerw sorry for the delay, I just pushed the last requested changes :) Please let me know if you have any feedback

Also, do you have an opinion about https://github.com/keras-team/keras/pull/19854#discussion_r1642748190? I'm fine with current implementation that saves a summary_plot.png with the plot of the model (even though it can be quite big).

Nice, thank you!

plot the summary only if number of layers less than a threshold. Mentioning it but I'm not a big fan of this solution as it might be confusing to users.
plot the summary and save it to the repo but do not load it in the model card. We can simply add a link to it in the model card and that's it. This way, we don't end up with enormous pages.

I'd suggest plotting the summary and saving the file in all cases, but then:

If less than 10 (?) layers, in-line the image in the model card
If more, add a link to the image in the card, so it can be opened in a different tab and does not clutter the model card

What do you think?

Sounds like a good plan yes! I pushed https://github.com/keras-team/keras/pull/19854/commits/83d6c68c70eba8b713eb1fce0dcac0b732a861e0 in that sense. A threshold at 10 layers seems good to have decent UX on smaller screens as well.


[Internal] Contract: Add 3.9.0-preview3 contract

Description
Adding 3.9.0-preview3 API surface:

Type of change
Please delete options that are not relevant.

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

LGTM. Thanks @ealsur.


Add ask alias to message

Because I've been using ".tell", it feels a bit weird to type ".tell Someone is this a question?" for example. So it's purely me being weird, I'd like to be able to "ask" a question.

naa has this in his open PR.


I'm getting a redirect_uri_mismatch error after application receives the token

I'm getting a weird error from the binary, which indicates that the redirect uri is invalid.
The redirect url is already listed on my 'Authorized redirect URIs.' of my app.
I don't quite understand why I would be able to get the refresh token, but then given what looks like an error regarding using the wrong redirect URL.
C:\test\google-ads-doctor\oauthdoctor\bin\windows\amd64>oauthdoctor.exe --verbose -language php -oauthtype web -sysinfo
2020/07/22 16:43:25 Client library language: php
Host: ADMINRG-0VC3H2Q
CPUs: 8
OS: windows
Arch: amd64
PageSize: 4096 bytes
Heap: 285176 bytes
IPV4:<IP_ADDRESS>
 IPV4:<IP_ADDRESS>
 Connected to googleads.googleapis.com.:80
2020/07/22 16:43:26 Google Ads API client library config file: C:\Users\shorton\google_ads_php.ini
2020/07/22 16:43:26 Config keys and values:
2020/07/22 16:43:26     ClientID = ******************* (hidden)
2020/07/22 16:43:26     ClientSecret = ******************* (hidden)
2020/07/22 16:43:26     DevToken = ******************* (hidden)
2020/07/22 16:43:26     RefreshToken = ******************* (hidden)
2020/07/22 16:43:26     LoginCustomerID = <empty>
2020/07/22 16:43:26     PrivateKeyPath = <empty>
2020/07/22 16:43:26     DelegatedAccount = <empty>
2020/07/22 16:43:26 Please enter a Google Ads account ID:
102-900-5886
2020/07/22 16:44:36 You will need to enter the URL http://localhost:8080 as a valid redirect URI in your Google APIs Console's project (https://console.developers.google.com/apis/library). Please follow this guide (https://developers.google.com/google-ads/api/docs/oauth/cloud-project) for further instructions.
2020/07/22 16:44:36 Visit the URL for the auth dialog:
https://accounts.google.com/o/oauth2/auth?access_type=offline&client_id=*******************&redirect_uri=http%3A%2F%2Flocalhost%3A8080&response_type=code&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fadwords&state=state
2020/07/22 16:44:36 Running HTTP server in the background at port 8080...
2020/07/22 16:45:02 OAuth code received by the HTTP server handler: *******************
2020/07/22 16:45:03 oauth2: cannot fetch token: 400 Bad Request
Response: {
  "error": "redirect_uri_mismatch",
  "error_description": "Bad Request"
}

I got the Google Ads API to work with the same client key, secret, and developer token, feel free to let me know if you need me to run it again or anything, I'm going to continue to leave this issue open, and check it every few days.

I wasn't aware this was a dead project, it's still mentioned multiple times in the documentation.
To whichever person sees this, can you ask them to remove the mention of this tool from the documentation?
Perhaps you can save people some time, which may add by replacing references to this unmaintained software by asking them to post questions in the support forum.


[detect_compiler] Improve error messages

For a long period of time, users have long wondered which Visual Studio components vcpkg needs in Windows:
CMake Error at C:/Users/James/code/ext/vcpkg/downloads/tools/cmake-3.21.1-windows/cmake-3.21.1-windows-i386/share/cmake-3.21/Modules/CMakeTestCCompiler.cmake:69 (message):
  The C compiler

    "C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: C:/Users/James/code/ext/vcpkg/buildtrees/bzip2/x64-windows-rel/CMakeFiles/CMakeTmp

    Run Build Command(s):C:/Users/James/code/ext/vcpkg/downloads/tools/ninja/1.10.2-windows/ninja.exe cmTC_3abad && [1/2] Building C object CMakeFiles\cmTC_3abad.dir\testCCompiler.c.obj
    [2/2] Linking C executable cmTC_3abad.exe
    FAILED: cmTC_3abad.exe
    cmd.exe /C "cd . && C:\Users\James\code\ext\vcpkg\downloads\tools\cmake-3.21.1-windows\cmake-3.21.1-windows-i386\bin\cmake.exe -E vs_link_exe --intdir=CMakeFiles\cmTC_3abad.dir --rc=rc --mt=CMAKE_MT-NOTFOUND --manifests  -- C:\PROGRA~1\MICROS~4\2022\COMMUN~1\VC\Tools\MSVC\1430~1.307\bin\Hostx64\x64\link.exe /nologo CMakeFiles\cmTC_3abad.dir\testCCompiler.c.obj  /out:cmTC_3abad.exe /implib:cmTC_3abad.lib /pdb:cmTC_3abad.pdb /version:0.0 /machine:x64  /debug /INCREMENTAL /subsystem:console  kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib && cd ."
    RC Pass 1: command "rc /fo CMakeFiles\cmTC_3abad.dir/manifest.res CMakeFiles\cmTC_3abad.dir/manifest.rc" failed (exit code 0) with the following output:
    The system cannot find the file specified
    ninja: build stopped: subcommand failed.

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:2 (project)

This PR aims to improve these error messages so that users can self-check the Visual Studio components that need to be installed:
  Failed to detect the MSVC C compiler.

  Please ensure you've installed the following Visual Studio Components using
  Visual Studio Installer:

    - Windows Universal C Runtime
    - C++ Build Tools core features
    - Visual Studio Build tools:
      * VC++ 2017 version ... latest v141 tools (for Visual Studio 2017)
      * MSVC v142 - VS 2019 C++ x64/x86 build tools (for Visual Studio 2019)
      * MSVC v143 - VS 2022 C++ x64/x86 build tools (for Visual Studio 2022)
    - MSBuild
    - Windows SDK
    - English language Pack

  For ARM / ARM64 or UWP triplet, please also installed:

    - Visual Studio Build tools for ARM:
      * VC++ 2017 version ... Libs for Spectre (ARM) (for Visual Studio 2017)
      * MSVC v142 - VS 2019 C++ ARM build tools (for Visual Studio 2019)
      * MSVC v143 - VS 2022 C++ ARM build tools (for Visual Studio 2022)
    - Visual Studio Build tools for ARM64:
      * VC++ 2017 version ... Libs for Spectre (ARM64) (for Visual Studio 2017)
      * MSVC v142 - VS 2019 C++ ARM64 build tools (for Visual Studio 2019)
      * MSVC v143 - VS 2022 C++ ARM64 build tools (for Visual Studio 2022)
    - Visual Studio Build tools for UWP:
      * Visual C++ runtime for UWP (for Visual Studio 2017)
      * C++ Universal Windows Platform support for v142 build tools (for Visual Studio 2019)
      * C++ Universal Windows Platform support for v143 build tools (for Visual Studio 2022)

Related: https://github.com/microsoft/vcpkg/issues/21580 https://github.com/microsoft/vcpkg/issues/22193 etc.

I found that detect_compiler is not built first for every build. Maybe we should add the same process to vcpkg_execute_build_process?

Is it possible to put this check into vcpkg itself? It seems like we could check for all of these components when we call vswhere and warn the user.

@strega-nil-ms It seems that wswhere couldn't get whether the English language pack installed.

The required product id:
C++:
Microsoft.VisualStudio.Workload.MSBuildTools
Microsoft.VisualStudio.Workload.VCTools
    *Microsoft.VisualStudio.Component.VC.Tools.x86.x64 (for 2017 or later)
    *Microsoft.VisualStudio.Component.VC.140 (for 2015)
Microsoft.VisualStudio.Component.Windows10SDK

ARM/ARM64:
Microsoft.VisualStudio.Component.VC.Tools.arm
Microsoft.VisualStudio.Component.VC.Tools.arm64
UWP:
Microsoft.VisualStudio.ComponentGroup.UWP.VC
Microsoft.VisualStudio.Workload.Universal

@JackBoosY we must be able to, since otherwise we wouldn't print an error if we couldn't find the english language pack.

@JackBoosY, see https://github.com/microsoft/vcpkg-tool/blob/main/src/vcpkg/visualstudio.cpp#L26

@JackBoosY, see https://github.com/microsoft/vcpkg-tool/blob/main/src/vcpkg/visualstudio.cpp#L26

        if (!excluded_toolsets.empty())
        {
            ret.append(msg::format(msgVSExcludedByLanguagePack));
            ret.appendnl();
            for (const Toolset& toolset : excluded_toolsets)
            {
                ret.append(msg::LocalizedString::from_string_unchecked(
                    Strings::concat("    ", toolset.visual_studio_root_path, '\n')));
            }
        }

I don't believe that the VS toolset is related to the English language pack.

Will continue this in https://github.com/microsoft/vcpkg/pull/22214


[CT-1494] [Bug] DBT run on incremental wipes out default privilege

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
In Postgres, we use 'alter default privileges' to give a set of pre-defined grants to newly created tables. This works fine for tables and views in DBT because a drop and create occurs which reapplies the default privilege grants on each new DBT run.
Because an incremental is not re-created on a DBT run when the apply_grants macro runs it wipes out the default privilege grants.
Expected Behavior
After reading through the source code I figured out that the problem statement is with the should_revoke macro. Since should_revoke is set to true for incremental inserts and the table is not recreated, the default privilege grants never get reapplied.
It would be great if there was an option to only perform a revoke on grants when a full-refresh  is performed that way when they are revoked any default privilege grants applied won't be deleted
So maybe an "is_revoke_only_on_refresh": True option added to the incremental config that allows for the following functionality:

Grants will stay the same and can be added to on normal incremental runs
Grants can be removed only on full-refresh runs

Steps To Reproduce

Have a Postgres user account with 'alter default privileges' setup
Perform a dbt run on an incremental ('alter default privileges' are given correctly)
Perform a dbt run on the same existing incremental ('alter default privileges' are lost)

Relevant log output
No response
Environment
- OS: Debian bullseye
- Python: 3.9.12
- dbt: 1.2.1

Which database adapter are you using with dbt?
postgres
Additional Context
No response

@jtcohen6 sorry to post on this closed thread. I have tried to get an answer in the dbt Slack workspace, but without success, so trying it here.
I have tried the suggested approach Nr. 2 to disable the grants config for my incremental models, but it doesn't work in my dbt project. I tested the following 2 ways, but none of them seem to work:

adding grants config to the yaml property file:

models:
  - name: my_model
    config:
      grants: {}

or adding grants={} to the config block of the model

{{
    config(
        materialized="incremental",
        unique_key="surrogate_key",
        incremental_strategy="delete+insert",
        grants={}
    )
}}

Could you please specify how the grants: {}, described in your message, should be applied?
Thanks a lot


Added missing argument

Number of arguments to Infof didn't match the format, resulting in misalignment

I signed it!

Thanks for fixing. LGTM


TST: fix special test that was importing np.testing.utils (deprecated)

This fixes the CI failures with numpy master.

Problems on older numpy releases...
I think the routine is equivalent to assert_allclose(...). Indeed, this code predates assert_allclose.

I'd happily undo the warning at some point if that would help, It was intended to help find these cases, which it looks like it is doing 8-)

Pushed a commit that replaces the function with assert_allclose.

Am going to hit the green button to unblock the CI. Thanks all.


chore: Make Instruction fn const

After merging this PR some constraints got relaxed and we are able to create a const Instruction that contains &mut in its signature.
Currently, this works in the nightly version: cargo 1.75.0-nightly (6fa6fdc76 2023-10-10) and we should wait until this is in stable before merging.

Closing this, and will reopen in few months


Improve DX for scenarios where Next.js and Drupal integration is not working correctly

There are multiple different ways how the Next.js and Drupal integration won't work correctly. This includes (but not limited to):

ACMS installed without default content model
Content model has been modified
Incompatible web server setup (e.g. web server not compatible with JSON:API)
Incorrect env variables generated by the backend

Proposed solution
Do a "preflight check" which ensures that fundamental aspects of the integration are working as expected. Fail early in case it is determined that the integration with the backend is not working as required by the Next.js app. Provide helpful error messages with links to documentation that documents how most common failures can be debugged.
The "preflight check" could check that the content model includes all content types from CONTENT_TYPES in [...slug].tsx. This would happen by checking /jsonapi which would also work as a test for ensuring that the API from the backend can be reached.

Common failures with Next.js and Drupal integration
Content mismatch
In the case of a content mismatch, there are two options to fix this:

Reinstall Acquia CMS with both the content model and demo content enabled to ensure that the backend has all the content that the Next.js site uses.
Check if all the items in CONTENT_TYPES in [...slug].tsx exist in the ACMS backend and if not either add the missing content types to the CMS or edit the list of  CONTENT_TYPES to match what is in ACMS.

Incompatible web server setup
Check if you are able to make a request to /jsonapi on ACMS. If you are getting a valid response but your Next.js site isn't working,  your web server might not be compatible with JSON:API. If you are using valet as your web server, please see https://github.com/laravel/valet/issues/771#issuecomment-485533478 for the known solution.
Incorrect env variables generated by the backend
[i'm not sure of the solution would it just be to reinstall ACMS?]

Added this to https://github.com/acquia/next-acms/wiki/Debugging-common-errors-with-Next.js-and-Drupal-integration

The API for executing these checks have been added: https://github.com/acquia/next-acms/pull/65.
Next step on this is to write some unit tests for the new API which was splitted to another PR since we still need to setup Jest. After that, we can make a new release of next-acms and update basic-starter to use it.


Uncaught SyntaxError: Unexpected token export

Expected behaviour
Run a test with es6
Actual behaviour
Getting  Uncaught SyntaxError: Unexpected token export
Environment Details

Karma version: 1.5.0

    preprocessors: {
      '../client/**/*.js': ['babel'],
      'unit/*.js': ['babel']
    },
    babelPreprocessor: {
      options: {
        presets: ['es2015'],
        plugins: ['transform-es2015-modules-umd'],
        sourceMap: 'inline'
      },
      filename: function (file) {
        return file.originalPath.replace(/\.js$/, '.es5.js');
      },
      sourceFileName: function (file) {
        return file.originalPath;
      }
    },

babel-core: "^6.24.0"
node: v7.4.0

@Hosar karma-babel-preprocessor is better place for this issue because I think the problem in babel setup.
Thanks

Ok got it, I'll check on that. Thank you

@Hosar Did you solve the problem?

No feedback in 8 months; closing.


fix param height bug (with line wrapping)

There was a bug where the height of the param bug was not correctly determined.
This seems a much more solid approach.

lgtm, with a naming change - cursorCoords to getCursorCoords.

Thanks!


XML Parse Error in Dataflow with data provider scheme

XML Parser has no reader for data provider scheme and returns error:
XMLParseError: AttributeError: 'Reader' object has no attribute 'parse_dataproviderscheme'
Stack:
Structure

Structures
OrganisationSchemes

<str:DataProviderScheme xmlns:str="http://www.sdmx.org/resources/sdmxml/schemas/v2_1/structure" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:mes="http://www.sdmx.org/resources/sdmxml/schemas/v2_1/message" xmlns:com="http://www.sdmx.org/resources/sdmxml/schemas/v2_1/common" urn="urn:sdmx:org.sdmx.infomodel.base.DataProviderScheme=UNICEF:DATA_PROVIDERS(1.0)" isExternalReference="false" agencyID="UNICEF" id="DATA_PROVIDERS" isFinal="false" version="1.0">
<com:Name xml:lang="it">DATA_PROVIDERS</com:Name>
<str:DataProvider urn="urn:sdmx:org.sdmx.infomodel.base.DataProvider=UNICEF:DATA_PROVIDERS(1.0).UNICEF" id="UNICEF">
<com:Name xml:lang="en">UNICEF Official Data</com:Name>
</str:DataProvider>
<str:DataProvider urn="urn:sdmx:org.sdmx.infomodel.base.DataProvider=UNICEF:DATA_PROVIDERS(1.0).UNICEF_DRAFT" id="UNICEF_DRAFT">
<com:Name xml:lang="en">UNICEF Draft Data</com:Name>
</str:DataProvider>
</str:DataProviderScheme>

What I had to do to make this call work was to scope the references, like:
req = sdmx.Request('UNICEF')
req.dataflow('NUTRITION',params={'references': 'Descendants'})
So that it does not get the "siblings" such as data providers.
However this seems to create another challenge, not sure if it is related to using references, but my dataflow does not seem to be the right kind of object when I try to start accessing the descendants like "structure", they are dictionary-like objects, and I can't dig further into the attributes like, dsd.dimensions.components. Am I just missing a step?

You have the right intuition. Use the structure attribute of the
dataflow definition to obtain the ID of the datastructure definition
(DSD). Then do something like:
DSD_msg = Request('unicef').datastructure(DSD_id)
On 29/03/2021, Yves Jaques @.***> wrote:

What I had to do to make this call work was to scope the references, like:
req = sdmx.Request('UNICEF')
req.dataflow('NUTRITION',params={'references': 'Descendants'})
So that it does not get the "siblings" such as data providers.
However this seems to create another challenge, not sure if it is related to
using references, but my dataflow does not seem to be the right kind of
object when I try to start accessing the descendants like "structure", they
are dictionary-like objects, and I can't dig further into the attributes
like, dsd.dimensions.components. Am I just missing a step?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dr-leo/pandaSDMX/issues/206#issuecomment-809688627

Probably solved in v1.8.0. Pls reopen if needed.


IMDb mobile

Add support for the mobile site at m.imdb.com (for example, Carol), shouldn't be a lot of work.

Code in the imdb-mobile branch


Allow me to download multi-file datasets

This is possible, but totally untested (and probably doesn't quite work)

This is closely related to #47, but here we are looking into adding the functionality to have multiple download URLs


Custom domain handles for Bluesky => AP

We currently support custom domain handles for web => fediverse. We should do the same for Bluesky => fediverse! Much of the plumbing is already there.
Corresponds to #826, which is equivalent for custom domain handles on accounts bridged into Bluesky.
(Note that this is just to support them at all. Switching from<EMAIL_ADDRESS>handle to [domain]@[domain] has extra complications, tracked in #1074.)

From @elfprince13 in https://github.com/snarfed/bridgy-fed/issues/826#issuecomment-2218108078:

I currently control:

<EMAIL_ADDRESS>(personal Mastodon instance)
@elfprince13.mumak.app (ATproto/Bluesky account)

Even a single-user Mastodon instance does not run particularly well on a small AWS instance, and it requires more attention than I want to allocate for it to constantly switch apps for posting, so I have been thinking of sunsetting the Mastodon account in favor of the Bluesky account; however, I would like to:

retain my existing Mastodon followers if at all possible.
continue using<EMAIL_ADDRESS>for my handle, rather than the clunkier<EMAIL_ADDRESS>which the current domain redirect support appears to require.

@elfprince13 the other difficulty here is that we currently only support custom fediverse usernames for web sites, not Bluesky accounts. I'm reluctant to add user-level settings, since they'd require an authed UI that I don't currently have and am unlikely to build. Maybe we could do it by reading the desired username or @-@ handle from your Bluesky profile bio though.

If there’s an unauthed approach that optimizes around the specific (fairly
common) pattern of user.domain.tld -> @.*** or otherwise checks
for some configuration file from domain.tld (or even a TXT record in the
DNS) I would slightly prefer that to putting it in the bio.
~Thomas

Agreed. Bio is awkward and annoying, DNS or HTTP is inaccessible, authed UI is a burden on my end. I haven't heard a good easy (or at least not hard) answer yet. 🤷

Hey!
Thanks for pointing me to this thread  💣
I'm curious, why is DNS/HTTP inaccessible?
My 2 cents:
I don't think the bio approach is a good idea – ideally the user should verify the domain they're claiming, otherwise one could just say they're<EMAIL_ADDRESS>in their bio and there's no way to verify that, plus there's gonna be duplicates problems (2 users claiming the same domain in their bio). Also yes, it's just awkward.
What @elfprince13 proposes about checking some configuration file from domain.tld sounds ok to me and similar to what Bluesky already does. I don't know if it is technically impossible, the doc for the webside -> AP mentions a similar approach using the .well-known path but surely you know why it can't be used in this scenario and I don't 😛

We may be confusing two different things here: custom domain handles, eg<EMAIL_ADDRESS>and custom usernames in handles, eg<EMAIL_ADDRESS>Let's keep this issue focused on the former; I'll create a new issue for the latter.

^ done!
@frncs-eu:

I don't think the bio approach is a good idea – ideally the user should verify the domain they're claiming, otherwise one could just say they're<EMAIL_ADDRESS>in their bio and there's no way to verify that, plus there's gonna be duplicates problems (2 users claiming the same domain in their bio). Also yes, it's just awkward.

Of course. To use a custom domain in a fediverse handle, you have to serve the Webfinger redirects on it, at minimum, which you can only do if you own it.
@elfprince13 and I were talking about how to set a custom username in a fediverse handle. I'll follow up a bit on that in #1190.

^ done!
@frncs-eu:

I don't think the bio approach is a good idea – ideally the user should verify the domain they're claiming, otherwise one could just say they're<EMAIL_ADDRESS>in their bio and there's no way to verify that, plus there's gonna be duplicates problems (2 users claiming the same domain in their bio). Also yes, it's just awkward.

Of course. To use a custom domain in a fediverse handle, you have to serve the Webfinger redirects on it, at minimum, which you can only do if you own it.
@elfprince13 and I were talking about how to set a custom username in a fediverse handle. I'll follow up a bit on that in #1190.

Sorry for the noise, all clear now! Thank you!

Now that we support DMs/chat, we could let people DM us the domain they want to use for their bridged fediverse address. Easier than a custom authed UI.

Realistically, Bluesky accounts bridged into the fediverse will always start on the @bsky.brid.gy domain. So, the next step here is either

Migrate the account from<EMAIL_ADDRESS>to @domain@domain with Move
OR
Delete the old account and start the new one on @domain@domain fresh.

1 is more work, but not too much, and obviously preferable.


Update MD file on new branch 2017-27-08_08-11-46

:warning: Validation status: warnings

File
Status
Preview URL
Details

split/test_link_to_con.yml
:warning:Warning

Details

split/ref.yml
:warning:Warning

Details

split/ref1.yml
:warning:Warning

Details

split/updateTestRef.md
:white_check_mark:Succeeded

split/test_link_to_con.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref2.yml.

split/ref.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref2.yml.

split/ref1.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref2.yml.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[macOS] Add Xcode 13.1 to macOS 11 & 12

Description
Xcode 13.1RC has been released, and while it claims to only support macOS-12, it looks like macOS-11 is also compatible.

Related issue:
https://github.com/actions/virtual-environments/issues/4300
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run macos110


【代码迁移】任务46：Density Adaptive Attention is All You Need: Robust Parameter-Efficient Fine-Tuning Across Multiple Modalities

基于Mindspore复现了论文《Density Adaptive Attention is All You Need: Robust Parameter-Efficient Fine-Tuning Across Multiple Modalities》代码仓库的GaussianBlock、MultiHeadGaussianAdaptiveAttention、GaussianAdaptiveAttention类
编写了main文件测试复现类的功能，测试任务为在MNIST数据集上训练一个带有GaussianBlock的简单网络。
对比了不同网络的torch实现和mindspore实现的测试结果：

可见，基于Mindspore实现的GaussianBlock等类复现了基于Torch实现的效果。

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


Error plotting result for Robyn in Docker

Project Robyn
Describe issue
I've been building the Robyn MMM in the Docker, everything went well so far except, the plotting
every plotting output was returning this message

Font 'Arial Narrow' is not installed, has other name, or can't be found

above message resulting no plotting folder
I've never got this problem whenever I run on Rstudio
Thanks!
Provide reproducible example
I've got my Docker as below:
FROM rocker/verse

COPY . /app
WORKDIR /app

USER root

RUN apt-get update && \
    apt-get -y upgrade && \
    apt-get -y install --no-install-recommends \
    make git curl g++ gcc&& \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

# INSTALL PYTHON 3
RUN apt-get update && \
    apt-get -y install python3 python3-pip python3-venv

# Adding gsutil to the image
# RUN curl -sSL https://sdk.cloud.google.com | bash && \
#     mv /root/google-cloud-sdk /usr/local

ENV PATH=${PATH}:/usr/local/google-cloud-sdk/bin

RUN python3 -m pip install -r python-requirements/requirements.txt --no-cache-dir

# install R packages
RUN Rscript /app/install_robyn.R
RUN Rscript /app/install_nevergrad.R

Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Check and share Robyn version: 3.10.5
R version 4.3.3 (2024-02-29)

I'm no expert in this but it sounds like your doc env is missing the font. Does thishttps://stackoverflow.com/questions/74745966/how-do-i-add-the-arial-font-to-a-docker-windows-container help?

Probably related to this issue: https://github.com/facebookexperimental/Robyn/issues/973


CORGI-107 remove distgit mount related code before move to MP+

This removes use of the distgit and lookaside NFS shares.

LGTM

I tagged main before this commit as 0.0.1


Rescue from more errors

If anything goes wrong with finding a publication (eg Panopticon is down), let's log the error and render a nice error page.

I'm not sure this is quite right (aware I missed a conversation in campfire, don't have time to dive into it in full detail right now but will pick it up soon).
I think that if panopticon is down we may well still want to render the page since we can still get most of the relevant content from publisher. Either way, I'd like to get the error handling in gds-api-adapters right before spending much time on how we then handle those errors.


Implement RpcMessage handlers

Most of RPC messages have been implemented her :
https://github.com/0xSpaceShard/starknet-rivet/pull/75


Show nothing if there are not campaigns

What changes did you make?

Makes campaigns not show if there aren't any for the service.

Is there a ticket that you are fixing?
Nope
Changelog

[x] I have added a Changie entry or given a reason why this does not need an entry in this section.

Screenshots
Please delete this section if it is not relevant